fix(desktop): show readable Claude Code model names and descriptions

[wpfleger96]

Claude Code agents' model dropdown showed raw IDs (opus[1m], claude-fable-5-1[1m], haiku) and its effort choices showed low/xhigh. Claude Code's ACP adapter labels config options and choices with name, but Buzz read displayName, which no adapter sends.

Adapter labels and descriptions

• normalize_agent_models reads name, keeps each model's description, and uses the model option's currentValue as the agent's default model (ahead of the unstable currentModelId).
• parse_config_options / parse_option_values read name for option and choice labels, so effort shows "Low"/"Xhigh".
• The create (PersonaModelCombobox) and edit (PersonaDropdownField) model pickers show the description as a muted second line, through both option builders (modelDropdownOptions and buildTemplateModelDropdownOptions).

Generated names for full Claude model IDs

• Claude Code's own names drop versions ("Sonnet" for claude-sonnet-5). The desktop's Databricks label grammar already parses a bare Claude ID once its wrapper is removed, so that body parse is exposed as generateBareModelLabel; Databricks entry points and outputs are unchanged.
• For the claude runtime, claudeCodeModelLabel formats full IDs and keeps a [1m] suffix as "(1M context)". Short names (opus, haiku, opus[1m]), unknown bracket suffixes, and IDs containing any non-ASCII character keep the adapter's name.
• This applies to the model dropdowns in the create and edit dialogs and the Agent defaults editor. The "Default model (…)" row appears in the edit dialog and the Agent defaults editor; the create dialog still requires an explicit model and shows no Default row. Model labels on agent cards are unchanged.
• In the edit dialog, the Default row keeps the discovered label (e.g. "Default model (Opus)") unless a global default model is set, in which case it shows that model. When the adapter's default is its own default entry (e.g. "Default (recommended)"), the row reads plain "Default model" and keeps that entry's description.

models debug command

• The harness's readable models output reads option and choice labels from name and the option id from id (falling back to configId), so it no longer prints raw IDs for Claude Code. JSON output is unchanged.

Model ID	Before	After
opus[1m]	opus[1m]	Opus · Opus with 1M context · …
claude-fable-5-1[1m]	claude-fable-5-1[1m]	Claude Fable 5.1 (1M context) · Fable 5.1 · …
claude-sonnet-5	claude-sonnet-5	Claude Sonnet 5 · Sonnet 5 · …
claude-sonnet-4-6	claude-sonnet-4-6	Claude Sonnet 4.6 · Previous Sonnet version
claude-sonnet-4-6[1m]	claude-sonnet-4-6[1m]	Claude Sonnet 4.6 (1M context) · Sonnet 4.6 for long sessions · …
haiku	haiku	Haiku · Haiku 4.5 · …

🤖

[github-actions]

🔐 Codex Security Review

Note: This is an automated, security-focused review generated by Codex.
Use it as a supplement to human review; false positives are possible.

Scope

• Exact PR diff: 133fb9869228dab4c00cb2b9e259bf834a55dbf7...473e97b9469a50e4f4eb851449ff8e0c689e2b50
• Model: gpt-5.6-sol

💡 Click "edited" above to see earlier reviews for this PR.

---

Review Summary

Overall Risk: LOW

One backward-compatibility regression affects ACP adapters using the previously supported displayName field.

Findings

[LOW] Preserve displayName compatibility when normalizing ACP models

• Category: Reliability
• Location: desktop/src-tauri/src/commands/agent_models.rs:746 (source)
• Description: Stable configOptions now read only name, although the existing supported ACP shape uses displayName. These entries are added to seen_ids before unstable models are processed, so a matching unstable entry cannot restore its human-readable name.
• Impact: Affected model pickers display raw, potentially cryptic model IDs instead of adapter-provided labels. The same field replacement in session configuration also removes friendly effort labels.
• Recommendation: Prefer name but fall back to displayName in model normalization, session configuration parsing, and human-readable CLI output. Add mixed-schema compatibility tests.

Notes

• Per review constraints, no builds or tests were executed.

---

_{Generated by Codex Security Review |
Requested by: @wpfleger96 |
Workflow run}

[wesbillman]

Carl, an automated reviewer, commenting via Wes’s GitHub account.

No material correctness or security blockers found. One optional wording suggestion is inline; this is a comment review, not approval.

Reviewed head 39e8e7d69db7bed3c81a3fa67d3bb35d44294cf2 against base/merge-base 448407a972ca9da0c1e13d49ee2c2170821be8a2.

• Traced discovery, default precedence, and create/edit/defaults label propagation. Targeted source-extracted probes preserved Databricks grammar outputs on 378 inputs and checked Claude fallback/ID preservation; these are not package-suite or live-UI results.
• The existing bot’s Cursor compatibility claim does not hold for Cursor 2026.10.01-e373342: its ACP builders translate internal displayName into wire name (buildModelSelectOption, buildVariantModelConfigOptions, getConfigOptionValues in 3351.index.js, published bundle). Claude v0.36.1 and the ACP v1 schema also use name. Other adapter/version combinations were not exhaustively checked.
• Current CI has passed Rust unit tests/lint and desktop Windows/macOS builds; Desktop Core, remaining smoke/integration jobs, and Windows Rust are still running. Let those finish. No native picker or VoiceOver session was exercised in this review.