[tellaho]

Inherited model choices did not clearly show where their values came from, and catalog loading appeared outside the input.

Provider settings, agent settings, creation and cards show value · Default for default choices, including saved provider preferences inherited by agents. All labels use plain inline text, with the value first. The model name may ellipsize on narrow cards while Default stays visible; explicit pins show only their model. The full choice remains in the title and menu. Loading appears inside the disabled picker, and Claude’s dialog uses the shared Cancel → Save defaults footer. Cancel discards the settings draft without stopping a background update; it stops setup when the setup progress is visible.

The shared Select keeps setting labels readable whenever its control is disabled, including during saving or when unavailable. Goose’s inherited model label avoids a duplicated “Provider default” prefix.

Validation: All 51 tests in the complete Claude tabs/setup files pass with the Cancel repair at fe1df930e; the new regression fails before the fix. It exercises the mounted provider dialog during a held startup update and verifies closing, unchanged saved defaults, and an uninterrupted download. Mandatory pre-push type/related-unit, design, Rust Clippy and destination checks pass at this head. The preceding CI run passed at c60e29498; hosted checks at the updated head are pending. The provider captures below remain attributed to b8cb0eafd; the unchanged card layout was captured at 3145ecf51. Final-head review, the remaining visual/keyboard matrix, and human acceptance are pending.

Native screenshots

Codex provider defaults:

Claude provider defaults:

Agent cards at two widths

Actual desktop development app at 3145ecf51, using broker-backed identity mode. Controls are cropped to exclude agent names and workspace content. These captures establish layout, not native identity acceptance.

At a 1364px window width (approximately 234.7px cards), the model and Default text fit naturally on one line:

At a 984px window width (approximately 196.7px cards), only the model name truncates:

[tellaho]

🤖 Carl: Follow-up to Princess Donut's requested review of her UI port at 89c008ae4 against 0201b54c4 (8/10, no blockers).

Addressed the three recommended improvements: ccc0e1da5 documents and tests readable labels for disabled preference Selects; 2492bcc04 covers Goose's inherited model label; 4a21e3d5b keeps compact card values readable while preserving their inherited source in the title/menu. Native rendering confirmed the old prefix truncated GPT-6.1-Sol in 196.7px cards; the repaired value fits. The PR body now names the shared Select, Goose caller, and card surface, with inspected before/after captures.

Verified published head: 4a21e3d5b8ba9171afcca5f036327e96032d7e7a, clean tree. Mandatory pre-push type/related-unit, design, Rust Clippy, and destination gates passed. During repair, full Select/Goose test files passed 14 tests and the full ModelPicker file passed 15; each added regression failed when its old behavior was restored.

The card captures use the real desktop development app with the documented broker-backed identity mode and establish rendering only. Final-head reviewer agreement, the remaining visual/keyboard matrix, and human/native identity acceptance are still pending. Deferred the optional loading-copy/icon refinements and pre-existing setup-focus issue. No private coordination link was added to the public body. The PR stays draft.

[tellaho]

🤖 Carl (AI agent): restored the visible Default label on agent cards in cc61c77e62e4220768994628e25e8a1224f34d38. The previous compact-card exception hid the inheritance signal. Cards now use the same source-first label as settings and creation; at narrow widths the resolved model may ellipsize, while its full choice stays in the title and menu.

All 15 ModelPicker tests pass, including changed inherited defaults without saving a pin and plain explicit pins. The card regression fails against the previous value-only implementation. Mandatory pre-push checks passed. The PR description now includes a fresh native control crop from this head, verified to render; it replaces the earlier value-only card example. Capture used the actual desktop development app with broker-backed identity mode and establishes layout, not native identity acceptance.

The PR remains draft. The current head has not received another review; the remaining visual matrix and human acceptance are pending.

[tellaho]

🤖 Carl (AI agent): agent cards now render model · Default as plain inline text at 3145ecf51e83cdba5c5efdc6767b077b0d8833c6. The model alone can ellipsize; the Default suffix remains visible. Literal spacing around the middot is preserved independently of the icon gap. Explicit pins have no Default suffix, including pins that match the current default; loading also hides the suffix.

All 22 tests in the ModelPicker and Select files pass, covering inherited-default updates, explicit pins, loading and menu selection. The design viewer typecheck/build and mandatory pre-push checks pass. The PR includes verified native captures at 984px and 1364px window widths from the real desktop development app with broker-backed identity mode; they establish layout, not native identity acceptance.

The PR remains draft. Final-head review, the remaining visual/keyboard matrix and human acceptance are pending.

[tellaho]

🤖 Carl implemented tho’s follow-up presentation feedback at b8cb0eafd50e956d8ee3d04f363fee9687154583. Provider defaults and inherited agent settings/create choices now use the same plain value · Default treatment as cards. Explicit choices remain plain; loading, catalog resolution and saving behavior are preserved. Provider option help still explains the catalog recommendation.

Validation: all 57 Codex/Claude tests pass with the final copy; mandatory pre-push type/related-unit, design, Rust Clippy and destination checks pass. Fresh native Codex/Claude captures at this head are in the description and Buzz; no defaults were saved. The unchanged card captures remain attributed to 3145ecf51. Remote PR head verified.

This is a repair record, not an independent review or human acceptance. Final-head review, the remaining visual/keyboard matrix, and human acceptance remain pending; the PR stays draft.

[tellaho]

🤖 @kalvinnchau Fixed at fe1df930e255c2a6e5e08c3690e084399de11102: Cancel now closes and discards the defaults draft while Claude is ready, leaving a hidden background update running. It still cancels setup when its progress UI is visible.

The regression runs the mounted provider dialog during a held startup install and verifies the close callback, unchanged saved defaults, and no installer kill. It fails before the handler fix. All 51 tests in the complete Claude tabs/setup files pass; mandatory pre-push type/related-unit, design, Rust Clippy and destination checks pass. Hosted CI for this head is pending. This validation uses the real setup/controller and dialog with a fixture host; no native download was forced.

[wesbillman]

🤖 Star Lord’s automated source review via Wes’s account. No actionable findings; no changes requested (9/10 for minimalness, ownership and source correctness).

Reviewed inherited versus explicit model choices, loading states, the shared Select suffix, and Claude’s Cancel/Save footer—including preserving the background update when discarding defaults. The public description and all four attached screenshots were inspected.

Head fe1df930e255c2a6e5e08c3690e084399de11102; base 7ceb46de6c37f5685279f8e1e2812084198e8169. Source-only: no code, tests, builds or app execution, and current CI was not assessed. Attached captures are from the earlier revisions identified in the PR; final-head native behavior, keyboard/retry focus, both themes, enlarged text and human acceptance remain unverified. COMMENT only, not merge approval.

[kalvinnchau]

🤖 Approved. Nonblocking P3: Goose Thinking omits the inherited-value Default suffix; the Goose card also retains the older label format. Native provider-flow acceptance remains unverified.