feat(web): redesign provider editor and models list

[StiensWout]

The provider editor was a long stack of full-width label-over-input fields, a four-column table for usually zero to two environment variables, and a Models tab capped at 160px with four always-visible icon buttons per row. Codex models also never showed "Fast mode" because the label only matched the boolean fastMode descriptor, not Codex's serviceTier select.

What changes:

• Configuration tab regrouped into a label-left grid (new grid variant of ProviderSettingsForm; the Add wizard's dialog variant is untouched): display name and accent swatches on one line, then Runtime and Environment groups under small divider labels. Each driver field is one line, with its description beside a fixed-width input. Environment variables are compact NAME = value rows with a lock toggle for sensitive values. Delete instance moves to the editor header.
• Models tab: no height cap, rows are a fixed-column grid (star, name and slug, capabilities, reorder and remove, picker switch) so capabilities line up and the reorder arrows stay in the row instead of appearing on hover. Picker visibility is a per-row switch (custom models show a disabled checked switch since they are always offered), favorites are grouped first with hidden models sunk to the bottom, a filter input appears above eight models, and "Add custom model" is a ghost button that expands into the input. The Models tab label shows the model count and how many are hidden.
• Capability labels count a Codex serviceTier select with a "Fast" tier as Fast mode, matching the composer's logic.
• Section header: "Checked Xs ago" and refresh are one compact ghost button next to Add provider.

Before

Configuration tab:

Models tab:

After

Screenshots predate the last commit, which made each driver field one line, aligned the model columns, and added the model count to the tab.

Configuration tab:

Models tab:

Checks

• vp test run on the four provider settings test files (21 tests), including a new groupModelsForDisplay test
• vp run --filter @t3tools/web typecheck
• Focused lint and format on the changed files

Built with Claude Fable 5.1 in the Claude Code harness via T3 Code.

---

Note

Medium Risk
Model list ordering and hidden-model grouping change persisted modelOrder semantics and what users see for edge cases (e.g. custom slugs in hidden prefs); otherwise UI-only settings changes with new test coverage for grouping.

Overview
Redesigns the provider instance Configuration and Models tabs plus related settings chrome.

The Configuration tab moves to a label-left two-column grid: display name shares a row with an inline accent swatch row (ProviderAccentColorPickerlayout="inline"), driver fields use a new grid variant on ProviderSettingsForm (description beside fixed-width controls), and Runtime / Environment sections get divider labels. Environment variables drop the bordered table for compact NAME = value rows with lock/unlock for sensitivity and inline remove. Delete instance moves from the list header icon to a text button in the editor header.

The Models tab removes the short height cap and info popovers in favor of a fixed-column row layout: inline capability tags (including Fast mode for Codex serviceTier selects), always-visible reorder/remove actions, and a picker visibility switch instead of eye icons. Lists are ordered via new groupModelsForDisplay (favorites, then visible, then hidden built-ins; custom models never treated as hidden). Reorder only swaps neighbors within the same group; filtering appears when there are more than eight models; add-custom-model is collapsible with cancel/Escape and scroll-to-new-row. The Models tab label shows total and hidden counts.

Providers section header combines refresh and “Checked … ago” into one compact button (sr-only label for tests); read-only mode shows last-checked text and keeps Advanced health-interval labels readable while controls stay inert.

^{Reviewed by Cursor Bugbot for commit d6fb0bb. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add inline layout option to ProviderAccentColorPicker

Extracts the custom picker, preset swatches, and clear action into a reusable swatch row. The new optional stacked/inline layout prop defaults to stacked, which keeps the label and description. inline mode renders only the swatch row.

^{Macroscope summarized d6fb0bb.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a4150b38-4d2a-4471-af42-52f97c78b786

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the provider list/editor redesign for shared-primitive and Tailwind-ownership consistency. Three findings, all in changed lines, about icon-color indirection through Button and a call-site override of Switch geometry. Layout/inert/read-only handling, the new grid variant of ProviderSettingsForm, and the models grouping helper look consistent with the existing contracts.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review

Four findings, all in changed lines. Three are shared-primitive contract issues where a call-site class never reaches the element it is meant to style:

• Button routes icon color through --control-icon-color ([&_svg:not([class*='text-'])]:text-[var(--control-icon-color)]), so text-* on the button does not tint a bare icon child. The favorite star and the sensitive/lock toggle lose their state colors as a result.
• Input applies className to the outer input-control wrapper, not the inner <input>, so h-8 / text-xs at the call site do not resize the control; size="compact" is the existing contract for that.

No issues found with the grid variant in ProviderSettingsForm, the layout="inline" accent picker, the read-only inert scoping in ProviderInstanceCard / ProviderSettingsPanel, or the --thumb-size switch override (an established call-site pattern).

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This is a substantial provider-settings redesign with new model filtering and visibility interactions plus changed persistence semantics for model ordering. The configuration and environment editor behavior also changes across the production settings surface, warranting human review.

^{You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

One accessibility regression found in the reworked Providers section header. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete buttons, size="compact" on the env and filter inputs, switch thumb sizing) is addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the provider editor header/configuration grid. Prior comments on icon-color indirection and compact input sizing have been addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two layout findings in the redesigned provider list row / environment rows. Everything flagged in earlier runs (icon-color indirection on the star and lock buttons, size="compact" on the env and filter inputs, the switch thumb override, and the refresh button's accessible name) looks resolved.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review: two findings on the redesigned provider panels. Earlier findings (icon-color indirection on the star/lock buttons, size="compact" on the env/filter inputs, switch thumb sizing, refresh-button accessible name) look resolved on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One new finding on the redesigned models list; details inline. The coarse-pointer issue flagged on the previous push (hover-only reorder / remove-custom-model actions in ProviderModelsSection) still stands and is not re-posted here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the redesigned provider panel. Everything flagged in earlier runs (icon-color indirection, compact input sizing, coarse-pointer row actions, refresh button accessible name, fixed row height) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One layout finding on the redesigned models list; details inline. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete/update buttons, size="compact" on the env and filter inputs, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, the coarse-pointer row actions, and the missing height cap below lg) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings on this head; details inline. Everything flagged in earlier runs (icon-color indirection, compact input sizing, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, coarse-pointer row actions, the missing height cap below lg, the 3rem actions track, and the filter clearing on add) reads as addressed.

^{Posted via Macroscope — UI Consistency}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 71ceb4f. Configure here.}

[StiensWout]

The provider editor was a long stack of full-width label-over-input fields, a four-column table for usually zero to two environment variables, and a Models tab capped at 160px with four always-visible icon buttons per row. Codex models also never showed "Fast mode" because the label only matched the boolean fastMode descriptor, not Codex's serviceTier select.

What changes:

• Configuration tab regrouped into a label-left grid (new grid variant of ProviderSettingsForm; the Add wizard's dialog variant is untouched): display name and accent swatches on one line, then Runtime and Environment groups under small divider labels. Each driver field is one line, with its description beside a fixed-width input. Environment variables are compact NAME = value rows with a lock toggle for sensitive values. Delete instance moves to the editor header.
• Models tab: no height cap, rows are a fixed-column grid (star, name and slug, capabilities, reorder and remove, picker switch) so capabilities line up and the reorder arrows stay in the row instead of appearing on hover. Picker visibility is a per-row switch (custom models show a disabled checked switch since they are always offered), favorites are grouped first with hidden models sunk to the bottom, a filter input appears above eight models, and "Add custom model" is a ghost button that expands into the input. The Models tab label shows the model count and how many are hidden.
• Capability labels count a Codex serviceTier select with a "Fast" tier as Fast mode, matching the composer's logic.
• Section header: "Checked Xs ago" and refresh are one compact ghost button next to Add provider.

Before

Configuration tab:

Models tab:

After

Screenshots predate the last commit, which made each driver field one line, aligned the model columns, and added the model count to the tab.

Configuration tab:

Models tab:

Checks

• vp test run on the four provider settings test files (21 tests), including a new groupModelsForDisplay test
• vp run --filter @t3tools/web typecheck
• Focused lint and format on the changed files

Built with Claude Fable 5.1 in the Claude Code harness via T3 Code.

---

Note

Medium Risk
Model list ordering and hidden-model grouping change persisted modelOrder semantics and what users see for edge cases (e.g. custom slugs in hidden prefs); otherwise UI-only settings changes with new test coverage for grouping.

Overview
Redesigns the provider instance Configuration and Models tabs plus related settings chrome.

The Configuration tab moves to a label-left two-column grid: display name shares a row with an inline accent swatch row (ProviderAccentColorPickerlayout="inline"), driver fields use a new grid variant on ProviderSettingsForm (description beside fixed-width controls), and Runtime / Environment sections get divider labels. Environment variables drop the bordered table for compact NAME = value rows with lock/unlock for sensitivity and inline remove. Delete instance moves from the list header icon to a text button in the editor header.

The Models tab removes the short height cap and info popovers in favor of a fixed-column row layout: inline capability tags (including Fast mode for Codex serviceTier selects), always-visible reorder/remove actions, and a picker visibility switch instead of eye icons. Lists are ordered via new groupModelsForDisplay (favorites, then visible, then hidden built-ins; custom models never treated as hidden). Reorder only swaps neighbors within the same group; filtering appears when there are more than eight models; add-custom-model is collapsible with cancel/Escape and scroll-to-new-row. The Models tab label shows total and hidden counts.

Providers section header combines refresh and “Checked … ago” into one compact button (sr-only label for tests); read-only mode shows last-checked text and keeps Advanced health-interval labels readable while controls stay inert.

^{Reviewed by Cursor Bugbot for commit d6fb0bb. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add inline layout option to ProviderAccentColorPicker

Extracts the custom picker, preset swatches, and clear action into a reusable swatch row. The new optional stacked/inline layout prop defaults to stacked, which keeps the label and description. inline mode renders only the swatch row.

^{Macroscope summarized d6fb0bb.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a4150b38-4d2a-4471-af42-52f97c78b786

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the provider list/editor redesign for shared-primitive and Tailwind-ownership consistency. Three findings, all in changed lines, about icon-color indirection through Button and a call-site override of Switch geometry. Layout/inert/read-only handling, the new grid variant of ProviderSettingsForm, and the models grouping helper look consistent with the existing contracts.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review

Four findings, all in changed lines. Three are shared-primitive contract issues where a call-site class never reaches the element it is meant to style:

• Button routes icon color through --control-icon-color ([&_svg:not([class*='text-'])]:text-[var(--control-icon-color)]), so text-* on the button does not tint a bare icon child. The favorite star and the sensitive/lock toggle lose their state colors as a result.
• Input applies className to the outer input-control wrapper, not the inner <input>, so h-8 / text-xs at the call site do not resize the control; size="compact" is the existing contract for that.

No issues found with the grid variant in ProviderSettingsForm, the layout="inline" accent picker, the read-only inert scoping in ProviderInstanceCard / ProviderSettingsPanel, or the --thumb-size switch override (an established call-site pattern).

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This is a substantial provider-settings redesign with new model filtering and visibility interactions plus changed persistence semantics for model ordering. The configuration and environment editor behavior also changes across the production settings surface, warranting human review.

^{You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

One accessibility regression found in the reworked Providers section header. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete buttons, size="compact" on the env and filter inputs, switch thumb sizing) is addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the provider editor header/configuration grid. Prior comments on icon-color indirection and compact input sizing have been addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two layout findings in the redesigned provider list row / environment rows. Everything flagged in earlier runs (icon-color indirection on the star and lock buttons, size="compact" on the env and filter inputs, the switch thumb override, and the refresh button's accessible name) looks resolved.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review: two findings on the redesigned provider panels. Earlier findings (icon-color indirection on the star/lock buttons, size="compact" on the env/filter inputs, switch thumb sizing, refresh-button accessible name) look resolved on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One new finding on the redesigned models list; details inline. The coarse-pointer issue flagged on the previous push (hover-only reorder / remove-custom-model actions in ProviderModelsSection) still stands and is not re-posted here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the redesigned provider panel. Everything flagged in earlier runs (icon-color indirection, compact input sizing, coarse-pointer row actions, refresh button accessible name, fixed row height) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One layout finding on the redesigned models list; details inline. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete/update buttons, size="compact" on the env and filter inputs, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, the coarse-pointer row actions, and the missing height cap below lg) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings on this head; details inline. Everything flagged in earlier runs (icon-color indirection, compact input sizing, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, coarse-pointer row actions, the missing height cap below lg, the 3rem actions track, and the filter clearing on add) reads as addressed.

^{Posted via Macroscope — UI Consistency}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 71ceb4f. Configure here.}

[StiensWout]

The provider editor was a long stack of full-width label-over-input fields, a four-column table for usually zero to two environment variables, and a Models tab capped at 160px with four always-visible icon buttons per row. Codex models also never showed "Fast mode" because the label only matched the boolean fastMode descriptor, not Codex's serviceTier select.

What changes:

• Configuration tab regrouped into a label-left grid (new grid variant of ProviderSettingsForm; the Add wizard's dialog variant is untouched): display name and accent swatches on one line, then Runtime and Environment groups under small divider labels. Each driver field is one line, with its description beside a fixed-width input. Environment variables are compact NAME = value rows with a lock toggle for sensitive values. Delete instance moves to the editor header.
• Models tab: no height cap, rows are a fixed-column grid (star, name and slug, capabilities, reorder and remove, picker switch) so capabilities line up and the reorder arrows stay in the row instead of appearing on hover. Picker visibility is a per-row switch (custom models show a disabled checked switch since they are always offered), favorites are grouped first with hidden models sunk to the bottom, a filter input appears above eight models, and "Add custom model" is a ghost button that expands into the input. The Models tab label shows the model count and how many are hidden.
• Capability labels count a Codex serviceTier select with a "Fast" tier as Fast mode, matching the composer's logic.
• Section header: "Checked Xs ago" and refresh are one compact ghost button next to Add provider.

Before

Configuration tab:

Models tab:

After

Screenshots predate the last commit, which made each driver field one line, aligned the model columns, and added the model count to the tab.

Configuration tab:

Models tab:

Checks

• vp test run on the four provider settings test files (21 tests), including a new groupModelsForDisplay test
• vp run --filter @t3tools/web typecheck
• Focused lint and format on the changed files

Built with Claude Fable 5.1 in the Claude Code harness via T3 Code.

---

Note

Medium Risk
Model list ordering and hidden-model grouping change persisted modelOrder semantics and what users see for edge cases (e.g. custom slugs in hidden prefs); otherwise UI-only settings changes with new test coverage for grouping.

Overview
Redesigns the provider instance Configuration and Models tabs plus related settings chrome.

The Configuration tab moves to a label-left two-column grid: display name shares a row with an inline accent swatch row (ProviderAccentColorPickerlayout="inline"), driver fields use a new grid variant on ProviderSettingsForm (description beside fixed-width controls), and Runtime / Environment sections get divider labels. Environment variables drop the bordered table for compact NAME = value rows with lock/unlock for sensitivity and inline remove. Delete instance moves from the list header icon to a text button in the editor header.

The Models tab removes the short height cap and info popovers in favor of a fixed-column row layout: inline capability tags (including Fast mode for Codex serviceTier selects), always-visible reorder/remove actions, and a picker visibility switch instead of eye icons. Lists are ordered via new groupModelsForDisplay (favorites, then visible, then hidden built-ins; custom models never treated as hidden). Reorder only swaps neighbors within the same group; filtering appears when there are more than eight models; add-custom-model is collapsible with cancel/Escape and scroll-to-new-row. The Models tab label shows total and hidden counts.

Providers section header combines refresh and “Checked … ago” into one compact button (sr-only label for tests); read-only mode shows last-checked text and keeps Advanced health-interval labels readable while controls stay inert.

^{Reviewed by Cursor Bugbot for commit d6fb0bb. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add inline layout option to ProviderAccentColorPicker

Extracts the custom picker, preset swatches, and clear action into a reusable swatch row. The new optional stacked/inline layout prop defaults to stacked, which keeps the label and description. inline mode renders only the swatch row.

^{Macroscope summarized d6fb0bb.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a4150b38-4d2a-4471-af42-52f97c78b786

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the provider list/editor redesign for shared-primitive and Tailwind-ownership consistency. Three findings, all in changed lines, about icon-color indirection through Button and a call-site override of Switch geometry. Layout/inert/read-only handling, the new grid variant of ProviderSettingsForm, and the models grouping helper look consistent with the existing contracts.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review

Four findings, all in changed lines. Three are shared-primitive contract issues where a call-site class never reaches the element it is meant to style:

• Button routes icon color through --control-icon-color ([&_svg:not([class*='text-'])]:text-[var(--control-icon-color)]), so text-* on the button does not tint a bare icon child. The favorite star and the sensitive/lock toggle lose their state colors as a result.
• Input applies className to the outer input-control wrapper, not the inner <input>, so h-8 / text-xs at the call site do not resize the control; size="compact" is the existing contract for that.

No issues found with the grid variant in ProviderSettingsForm, the layout="inline" accent picker, the read-only inert scoping in ProviderInstanceCard / ProviderSettingsPanel, or the --thumb-size switch override (an established call-site pattern).

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This is a substantial provider-settings redesign with new model filtering and visibility interactions plus changed persistence semantics for model ordering. The configuration and environment editor behavior also changes across the production settings surface, warranting human review.

^{You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

One accessibility regression found in the reworked Providers section header. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete buttons, size="compact" on the env and filter inputs, switch thumb sizing) is addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the provider editor header/configuration grid. Prior comments on icon-color indirection and compact input sizing have been addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two layout findings in the redesigned provider list row / environment rows. Everything flagged in earlier runs (icon-color indirection on the star and lock buttons, size="compact" on the env and filter inputs, the switch thumb override, and the refresh button's accessible name) looks resolved.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review: two findings on the redesigned provider panels. Earlier findings (icon-color indirection on the star/lock buttons, size="compact" on the env/filter inputs, switch thumb sizing, refresh-button accessible name) look resolved on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One new finding on the redesigned models list; details inline. The coarse-pointer issue flagged on the previous push (hover-only reorder / remove-custom-model actions in ProviderModelsSection) still stands and is not re-posted here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the redesigned provider panel. Everything flagged in earlier runs (icon-color indirection, compact input sizing, coarse-pointer row actions, refresh button accessible name, fixed row height) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One layout finding on the redesigned models list; details inline. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete/update buttons, size="compact" on the env and filter inputs, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, the coarse-pointer row actions, and the missing height cap below lg) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings on this head; details inline. Everything flagged in earlier runs (icon-color indirection, compact input sizing, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, coarse-pointer row actions, the missing height cap below lg, the 3rem actions track, and the filter clearing on add) reads as addressed.

^{Posted via Macroscope — UI Consistency}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 71ceb4f. Configure here.}

[StiensWout]

The provider editor was a long stack of full-width label-over-input fields, a four-column table for usually zero to two environment variables, and a Models tab capped at 160px with four always-visible icon buttons per row. Codex models also never showed "Fast mode" because the label only matched the boolean fastMode descriptor, not Codex's serviceTier select.

What changes:

• Configuration tab regrouped into a label-left grid (new grid variant of ProviderSettingsForm; the Add wizard's dialog variant is untouched): display name and accent swatches on one line, then Runtime and Environment groups under small divider labels. Each driver field is one line, with its description beside a fixed-width input. Environment variables are compact NAME = value rows with a lock toggle for sensitive values. Delete instance moves to the editor header.
• Models tab: no height cap, rows are a fixed-column grid (star, name and slug, capabilities, reorder and remove, picker switch) so capabilities line up and the reorder arrows stay in the row instead of appearing on hover. Picker visibility is a per-row switch (custom models show a disabled checked switch since they are always offered), favorites are grouped first with hidden models sunk to the bottom, a filter input appears above eight models, and "Add custom model" is a ghost button that expands into the input. The Models tab label shows the model count and how many are hidden.
• Capability labels count a Codex serviceTier select with a "Fast" tier as Fast mode, matching the composer's logic.
• Section header: "Checked Xs ago" and refresh are one compact ghost button next to Add provider.

Before

Configuration tab:

Models tab:

After

Screenshots predate the last commit, which made each driver field one line, aligned the model columns, and added the model count to the tab.

Configuration tab:

Models tab:

Checks

• vp test run on the four provider settings test files (21 tests), including a new groupModelsForDisplay test
• vp run --filter @t3tools/web typecheck
• Focused lint and format on the changed files

Built with Claude Fable 5.1 in the Claude Code harness via T3 Code.

---

Note

Medium Risk
Model list ordering and hidden-model grouping change persisted modelOrder semantics and what users see for edge cases (e.g. custom slugs in hidden prefs); otherwise UI-only settings changes with new test coverage for grouping.

Overview
Redesigns the provider instance Configuration and Models tabs plus related settings chrome.

The Configuration tab moves to a label-left two-column grid: display name shares a row with an inline accent swatch row (ProviderAccentColorPickerlayout="inline"), driver fields use a new grid variant on ProviderSettingsForm (description beside fixed-width controls), and Runtime / Environment sections get divider labels. Environment variables drop the bordered table for compact NAME = value rows with lock/unlock for sensitivity and inline remove. Delete instance moves from the list header icon to a text button in the editor header.

The Models tab removes the short height cap and info popovers in favor of a fixed-column row layout: inline capability tags (including Fast mode for Codex serviceTier selects), always-visible reorder/remove actions, and a picker visibility switch instead of eye icons. Lists are ordered via new groupModelsForDisplay (favorites, then visible, then hidden built-ins; custom models never treated as hidden). Reorder only swaps neighbors within the same group; filtering appears when there are more than eight models; add-custom-model is collapsible with cancel/Escape and scroll-to-new-row. The Models tab label shows total and hidden counts.

Providers section header combines refresh and “Checked … ago” into one compact button (sr-only label for tests); read-only mode shows last-checked text and keeps Advanced health-interval labels readable while controls stay inert.

^{Reviewed by Cursor Bugbot for commit d6fb0bb. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add inline layout option to ProviderAccentColorPicker

Extracts the custom picker, preset swatches, and clear action into a reusable swatch row. The new optional stacked/inline layout prop defaults to stacked, which keeps the label and description. inline mode renders only the swatch row.

^{Macroscope summarized d6fb0bb.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a4150b38-4d2a-4471-af42-52f97c78b786

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the provider list/editor redesign for shared-primitive and Tailwind-ownership consistency. Three findings, all in changed lines, about icon-color indirection through Button and a call-site override of Switch geometry. Layout/inert/read-only handling, the new grid variant of ProviderSettingsForm, and the models grouping helper look consistent with the existing contracts.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review

Four findings, all in changed lines. Three are shared-primitive contract issues where a call-site class never reaches the element it is meant to style:

• Button routes icon color through --control-icon-color ([&_svg:not([class*='text-'])]:text-[var(--control-icon-color)]), so text-* on the button does not tint a bare icon child. The favorite star and the sensitive/lock toggle lose their state colors as a result.
• Input applies className to the outer input-control wrapper, not the inner <input>, so h-8 / text-xs at the call site do not resize the control; size="compact" is the existing contract for that.

No issues found with the grid variant in ProviderSettingsForm, the layout="inline" accent picker, the read-only inert scoping in ProviderInstanceCard / ProviderSettingsPanel, or the --thumb-size switch override (an established call-site pattern).

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This is a substantial provider-settings redesign with new model filtering and visibility interactions plus changed persistence semantics for model ordering. The configuration and environment editor behavior also changes across the production settings surface, warranting human review.

^{You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

One accessibility regression found in the reworked Providers section header. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete buttons, size="compact" on the env and filter inputs, switch thumb sizing) is addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the provider editor header/configuration grid. Prior comments on icon-color indirection and compact input sizing have been addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two layout findings in the redesigned provider list row / environment rows. Everything flagged in earlier runs (icon-color indirection on the star and lock buttons, size="compact" on the env and filter inputs, the switch thumb override, and the refresh button's accessible name) looks resolved.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review: two findings on the redesigned provider panels. Earlier findings (icon-color indirection on the star/lock buttons, size="compact" on the env/filter inputs, switch thumb sizing, refresh-button accessible name) look resolved on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One new finding on the redesigned models list; details inline. The coarse-pointer issue flagged on the previous push (hover-only reorder / remove-custom-model actions in ProviderModelsSection) still stands and is not re-posted here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the redesigned provider panel. Everything flagged in earlier runs (icon-color indirection, compact input sizing, coarse-pointer row actions, refresh button accessible name, fixed row height) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One layout finding on the redesigned models list; details inline. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete/update buttons, size="compact" on the env and filter inputs, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, the coarse-pointer row actions, and the missing height cap below lg) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings on this head; details inline. Everything flagged in earlier runs (icon-color indirection, compact input sizing, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, coarse-pointer row actions, the missing height cap below lg, the 3rem actions track, and the filter clearing on add) reads as addressed.

^{Posted via Macroscope — UI Consistency}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 71ceb4f. Configure here.}

[StiensWout]

The provider editor was a long stack of full-width label-over-input fields, a four-column table for usually zero to two environment variables, and a Models tab capped at 160px with four always-visible icon buttons per row. Codex models also never showed "Fast mode" because the label only matched the boolean fastMode descriptor, not Codex's serviceTier select.

What changes:

• Configuration tab regrouped into a label-left grid (new grid variant of ProviderSettingsForm; the Add wizard's dialog variant is untouched): display name and accent swatches on one line, then Runtime and Environment groups under small divider labels. Each driver field is one line, with its description beside a fixed-width input. Environment variables are compact NAME = value rows with a lock toggle for sensitive values. Delete instance moves to the editor header.
• Models tab: no height cap, rows are a fixed-column grid (star, name and slug, capabilities, reorder and remove, picker switch) so capabilities line up and the reorder arrows stay in the row instead of appearing on hover. Picker visibility is a per-row switch (custom models show a disabled checked switch since they are always offered), favorites are grouped first with hidden models sunk to the bottom, a filter input appears above eight models, and "Add custom model" is a ghost button that expands into the input. The Models tab label shows the model count and how many are hidden.
• Capability labels count a Codex serviceTier select with a "Fast" tier as Fast mode, matching the composer's logic.
• Section header: "Checked Xs ago" and refresh are one compact ghost button next to Add provider.

Before

Configuration tab:

Models tab:

After

Screenshots predate the last commit, which made each driver field one line, aligned the model columns, and added the model count to the tab.

Configuration tab:

Models tab:

Checks

• vp test run on the four provider settings test files (21 tests), including a new groupModelsForDisplay test
• vp run --filter @t3tools/web typecheck
• Focused lint and format on the changed files

Built with Claude Fable 5.1 in the Claude Code harness via T3 Code.

---

Note

Medium Risk
Model list ordering and hidden-model grouping change persisted modelOrder semantics and what users see for edge cases (e.g. custom slugs in hidden prefs); otherwise UI-only settings changes with new test coverage for grouping.

Overview
Redesigns the provider instance Configuration and Models tabs plus related settings chrome.

The Configuration tab moves to a label-left two-column grid: display name shares a row with an inline accent swatch row (ProviderAccentColorPickerlayout="inline"), driver fields use a new grid variant on ProviderSettingsForm (description beside fixed-width controls), and Runtime / Environment sections get divider labels. Environment variables drop the bordered table for compact NAME = value rows with lock/unlock for sensitivity and inline remove. Delete instance moves from the list header icon to a text button in the editor header.

The Models tab removes the short height cap and info popovers in favor of a fixed-column row layout: inline capability tags (including Fast mode for Codex serviceTier selects), always-visible reorder/remove actions, and a picker visibility switch instead of eye icons. Lists are ordered via new groupModelsForDisplay (favorites, then visible, then hidden built-ins; custom models never treated as hidden). Reorder only swaps neighbors within the same group; filtering appears when there are more than eight models; add-custom-model is collapsible with cancel/Escape and scroll-to-new-row. The Models tab label shows total and hidden counts.

Providers section header combines refresh and “Checked … ago” into one compact button (sr-only label for tests); read-only mode shows last-checked text and keeps Advanced health-interval labels readable while controls stay inert.

^{Reviewed by Cursor Bugbot for commit d6fb0bb. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add inline layout option to ProviderAccentColorPicker

Extracts the custom picker, preset swatches, and clear action into a reusable swatch row. The new optional stacked/inline layout prop defaults to stacked, which keeps the label and description. inline mode renders only the swatch row.

^{Macroscope summarized d6fb0bb.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a4150b38-4d2a-4471-af42-52f97c78b786

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the provider list/editor redesign for shared-primitive and Tailwind-ownership consistency. Three findings, all in changed lines, about icon-color indirection through Button and a call-site override of Switch geometry. Layout/inert/read-only handling, the new grid variant of ProviderSettingsForm, and the models grouping helper look consistent with the existing contracts.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review

Four findings, all in changed lines. Three are shared-primitive contract issues where a call-site class never reaches the element it is meant to style:

• Button routes icon color through --control-icon-color ([&_svg:not([class*='text-'])]:text-[var(--control-icon-color)]), so text-* on the button does not tint a bare icon child. The favorite star and the sensitive/lock toggle lose their state colors as a result.
• Input applies className to the outer input-control wrapper, not the inner <input>, so h-8 / text-xs at the call site do not resize the control; size="compact" is the existing contract for that.

No issues found with the grid variant in ProviderSettingsForm, the layout="inline" accent picker, the read-only inert scoping in ProviderInstanceCard / ProviderSettingsPanel, or the --thumb-size switch override (an established call-site pattern).

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This is a substantial provider-settings redesign with new model filtering and visibility interactions plus changed persistence semantics for model ordering. The configuration and environment editor behavior also changes across the production settings surface, warranting human review.

^{You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

One accessibility regression found in the reworked Providers section header. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete buttons, size="compact" on the env and filter inputs, switch thumb sizing) is addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the provider editor header/configuration grid. Prior comments on icon-color indirection and compact input sizing have been addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two layout findings in the redesigned provider list row / environment rows. Everything flagged in earlier runs (icon-color indirection on the star and lock buttons, size="compact" on the env and filter inputs, the switch thumb override, and the refresh button's accessible name) looks resolved.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review: two findings on the redesigned provider panels. Earlier findings (icon-color indirection on the star/lock buttons, size="compact" on the env/filter inputs, switch thumb sizing, refresh-button accessible name) look resolved on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One new finding on the redesigned models list; details inline. The coarse-pointer issue flagged on the previous push (hover-only reorder / remove-custom-model actions in ProviderModelsSection) still stands and is not re-posted here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the redesigned provider panel. Everything flagged in earlier runs (icon-color indirection, compact input sizing, coarse-pointer row actions, refresh button accessible name, fixed row height) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One layout finding on the redesigned models list; details inline. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete/update buttons, size="compact" on the env and filter inputs, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, the coarse-pointer row actions, and the missing height cap below lg) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings on this head; details inline. Everything flagged in earlier runs (icon-color indirection, compact input sizing, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, coarse-pointer row actions, the missing height cap below lg, the 3rem actions track, and the filter clearing on add) reads as addressed.

^{Posted via Macroscope — UI Consistency}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 71ceb4f. Configure here.}

[StiensWout]

The provider editor was a long stack of full-width label-over-input fields, a four-column table for usually zero to two environment variables, and a Models tab capped at 160px with four always-visible icon buttons per row. Codex models also never showed "Fast mode" because the label only matched the boolean fastMode descriptor, not Codex's serviceTier select.

What changes:

• Configuration tab regrouped into a label-left grid (new grid variant of ProviderSettingsForm; the Add wizard's dialog variant is untouched): display name and accent swatches on one line, then Runtime and Environment groups under small divider labels. Each driver field is one line, with its description beside a fixed-width input. Environment variables are compact NAME = value rows with a lock toggle for sensitive values. Delete instance moves to the editor header.
• Models tab: no height cap, rows are a fixed-column grid (star, name and slug, capabilities, reorder and remove, picker switch) so capabilities line up and the reorder arrows stay in the row instead of appearing on hover. Picker visibility is a per-row switch (custom models show a disabled checked switch since they are always offered), favorites are grouped first with hidden models sunk to the bottom, a filter input appears above eight models, and "Add custom model" is a ghost button that expands into the input. The Models tab label shows the model count and how many are hidden.
• Capability labels count a Codex serviceTier select with a "Fast" tier as Fast mode, matching the composer's logic.
• Section header: "Checked Xs ago" and refresh are one compact ghost button next to Add provider.

Before

Configuration tab:

Models tab:

After

Screenshots predate the last commit, which made each driver field one line, aligned the model columns, and added the model count to the tab.

Configuration tab:

Models tab:

Checks

• vp test run on the four provider settings test files (21 tests), including a new groupModelsForDisplay test
• vp run --filter @t3tools/web typecheck
• Focused lint and format on the changed files

Built with Claude Fable 5.1 in the Claude Code harness via T3 Code.

---

Note

Medium Risk
Model list ordering and hidden-model grouping change persisted modelOrder semantics and what users see for edge cases (e.g. custom slugs in hidden prefs); otherwise UI-only settings changes with new test coverage for grouping.

Overview
Redesigns the provider instance Configuration and Models tabs plus related settings chrome.

The Configuration tab moves to a label-left two-column grid: display name shares a row with an inline accent swatch row (ProviderAccentColorPickerlayout="inline"), driver fields use a new grid variant on ProviderSettingsForm (description beside fixed-width controls), and Runtime / Environment sections get divider labels. Environment variables drop the bordered table for compact NAME = value rows with lock/unlock for sensitivity and inline remove. Delete instance moves from the list header icon to a text button in the editor header.

The Models tab removes the short height cap and info popovers in favor of a fixed-column row layout: inline capability tags (including Fast mode for Codex serviceTier selects), always-visible reorder/remove actions, and a picker visibility switch instead of eye icons. Lists are ordered via new groupModelsForDisplay (favorites, then visible, then hidden built-ins; custom models never treated as hidden). Reorder only swaps neighbors within the same group; filtering appears when there are more than eight models; add-custom-model is collapsible with cancel/Escape and scroll-to-new-row. The Models tab label shows total and hidden counts.

Providers section header combines refresh and “Checked … ago” into one compact button (sr-only label for tests); read-only mode shows last-checked text and keeps Advanced health-interval labels readable while controls stay inert.

^{Reviewed by Cursor Bugbot for commit d6fb0bb. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add inline layout option to ProviderAccentColorPicker

Extracts the custom picker, preset swatches, and clear action into a reusable swatch row. The new optional stacked/inline layout prop defaults to stacked, which keeps the label and description. inline mode renders only the swatch row.

^{Macroscope summarized d6fb0bb.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a4150b38-4d2a-4471-af42-52f97c78b786

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the provider list/editor redesign for shared-primitive and Tailwind-ownership consistency. Three findings, all in changed lines, about icon-color indirection through Button and a call-site override of Switch geometry. Layout/inert/read-only handling, the new grid variant of ProviderSettingsForm, and the models grouping helper look consistent with the existing contracts.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review

Four findings, all in changed lines. Three are shared-primitive contract issues where a call-site class never reaches the element it is meant to style:

• Button routes icon color through --control-icon-color ([&_svg:not([class*='text-'])]:text-[var(--control-icon-color)]), so text-* on the button does not tint a bare icon child. The favorite star and the sensitive/lock toggle lose their state colors as a result.
• Input applies className to the outer input-control wrapper, not the inner <input>, so h-8 / text-xs at the call site do not resize the control; size="compact" is the existing contract for that.

No issues found with the grid variant in ProviderSettingsForm, the layout="inline" accent picker, the read-only inert scoping in ProviderInstanceCard / ProviderSettingsPanel, or the --thumb-size switch override (an established call-site pattern).

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This is a substantial provider-settings redesign with new model filtering and visibility interactions plus changed persistence semantics for model ordering. The configuration and environment editor behavior also changes across the production settings surface, warranting human review.

^{You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

One accessibility regression found in the reworked Providers section header. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete buttons, size="compact" on the env and filter inputs, switch thumb sizing) is addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the provider editor header/configuration grid. Prior comments on icon-color indirection and compact input sizing have been addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two layout findings in the redesigned provider list row / environment rows. Everything flagged in earlier runs (icon-color indirection on the star and lock buttons, size="compact" on the env and filter inputs, the switch thumb override, and the refresh button's accessible name) looks resolved.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review: two findings on the redesigned provider panels. Earlier findings (icon-color indirection on the star/lock buttons, size="compact" on the env/filter inputs, switch thumb sizing, refresh-button accessible name) look resolved on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One new finding on the redesigned models list; details inline. The coarse-pointer issue flagged on the previous push (hover-only reorder / remove-custom-model actions in ProviderModelsSection) still stands and is not re-posted here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the redesigned provider panel. Everything flagged in earlier runs (icon-color indirection, compact input sizing, coarse-pointer row actions, refresh button accessible name, fixed row height) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One layout finding on the redesigned models list; details inline. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete/update buttons, size="compact" on the env and filter inputs, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, the coarse-pointer row actions, and the missing height cap below lg) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings on this head; details inline. Everything flagged in earlier runs (icon-color indirection, compact input sizing, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, coarse-pointer row actions, the missing height cap below lg, the 3rem actions track, and the filter clearing on add) reads as addressed.

^{Posted via Macroscope — UI Consistency}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 71ceb4f. Configure here.}

[StiensWout]

The provider editor was a long stack of full-width label-over-input fields, a four-column table for usually zero to two environment variables, and a Models tab capped at 160px with four always-visible icon buttons per row. Codex models also never showed "Fast mode" because the label only matched the boolean fastMode descriptor, not Codex's serviceTier select.

What changes:

• Configuration tab regrouped into a label-left grid (new grid variant of ProviderSettingsForm; the Add wizard's dialog variant is untouched): display name and accent swatches on one line, then Runtime and Environment groups under small divider labels. Each driver field is one line, with its description beside a fixed-width input. Environment variables are compact NAME = value rows with a lock toggle for sensitive values. Delete instance moves to the editor header.
• Models tab: no height cap, rows are a fixed-column grid (star, name and slug, capabilities, reorder and remove, picker switch) so capabilities line up and the reorder arrows stay in the row instead of appearing on hover. Picker visibility is a per-row switch (custom models show a disabled checked switch since they are always offered), favorites are grouped first with hidden models sunk to the bottom, a filter input appears above eight models, and "Add custom model" is a ghost button that expands into the input. The Models tab label shows the model count and how many are hidden.
• Capability labels count a Codex serviceTier select with a "Fast" tier as Fast mode, matching the composer's logic.
• Section header: "Checked Xs ago" and refresh are one compact ghost button next to Add provider.

Before

Configuration tab:

Models tab:

After

Screenshots predate the last commit, which made each driver field one line, aligned the model columns, and added the model count to the tab.

Configuration tab:

Models tab:

Checks

• vp test run on the four provider settings test files (21 tests), including a new groupModelsForDisplay test
• vp run --filter @t3tools/web typecheck
• Focused lint and format on the changed files

Built with Claude Fable 5.1 in the Claude Code harness via T3 Code.

---

Note

Medium Risk
Model list ordering and hidden-model grouping change persisted modelOrder semantics and what users see for edge cases (e.g. custom slugs in hidden prefs); otherwise UI-only settings changes with new test coverage for grouping.

Overview
Redesigns the provider instance Configuration and Models tabs plus related settings chrome.

The Configuration tab moves to a label-left two-column grid: display name shares a row with an inline accent swatch row (ProviderAccentColorPickerlayout="inline"), driver fields use a new grid variant on ProviderSettingsForm (description beside fixed-width controls), and Runtime / Environment sections get divider labels. Environment variables drop the bordered table for compact NAME = value rows with lock/unlock for sensitivity and inline remove. Delete instance moves from the list header icon to a text button in the editor header.

The Models tab removes the short height cap and info popovers in favor of a fixed-column row layout: inline capability tags (including Fast mode for Codex serviceTier selects), always-visible reorder/remove actions, and a picker visibility switch instead of eye icons. Lists are ordered via new groupModelsForDisplay (favorites, then visible, then hidden built-ins; custom models never treated as hidden). Reorder only swaps neighbors within the same group; filtering appears when there are more than eight models; add-custom-model is collapsible with cancel/Escape and scroll-to-new-row. The Models tab label shows total and hidden counts.

Providers section header combines refresh and “Checked … ago” into one compact button (sr-only label for tests); read-only mode shows last-checked text and keeps Advanced health-interval labels readable while controls stay inert.

^{Reviewed by Cursor Bugbot for commit d6fb0bb. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add inline layout option to ProviderAccentColorPicker

Extracts the custom picker, preset swatches, and clear action into a reusable swatch row. The new optional stacked/inline layout prop defaults to stacked, which keeps the label and description. inline mode renders only the swatch row.

^{Macroscope summarized d6fb0bb.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a4150b38-4d2a-4471-af42-52f97c78b786

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the provider list/editor redesign for shared-primitive and Tailwind-ownership consistency. Three findings, all in changed lines, about icon-color indirection through Button and a call-site override of Switch geometry. Layout/inert/read-only handling, the new grid variant of ProviderSettingsForm, and the models grouping helper look consistent with the existing contracts.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review

Four findings, all in changed lines. Three are shared-primitive contract issues where a call-site class never reaches the element it is meant to style:

• Button routes icon color through --control-icon-color ([&_svg:not([class*='text-'])]:text-[var(--control-icon-color)]), so text-* on the button does not tint a bare icon child. The favorite star and the sensitive/lock toggle lose their state colors as a result.
• Input applies className to the outer input-control wrapper, not the inner <input>, so h-8 / text-xs at the call site do not resize the control; size="compact" is the existing contract for that.

No issues found with the grid variant in ProviderSettingsForm, the layout="inline" accent picker, the read-only inert scoping in ProviderInstanceCard / ProviderSettingsPanel, or the --thumb-size switch override (an established call-site pattern).

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This is a substantial provider-settings redesign with new model filtering and visibility interactions plus changed persistence semantics for model ordering. The configuration and environment editor behavior also changes across the production settings surface, warranting human review.

^{You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

One accessibility regression found in the reworked Providers section header. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete buttons, size="compact" on the env and filter inputs, switch thumb sizing) is addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the provider editor header/configuration grid. Prior comments on icon-color indirection and compact input sizing have been addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two layout findings in the redesigned provider list row / environment rows. Everything flagged in earlier runs (icon-color indirection on the star and lock buttons, size="compact" on the env and filter inputs, the switch thumb override, and the refresh button's accessible name) looks resolved.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review: two findings on the redesigned provider panels. Earlier findings (icon-color indirection on the star/lock buttons, size="compact" on the env/filter inputs, switch thumb sizing, refresh-button accessible name) look resolved on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One new finding on the redesigned models list; details inline. The coarse-pointer issue flagged on the previous push (hover-only reorder / remove-custom-model actions in ProviderModelsSection) still stands and is not re-posted here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the redesigned provider panel. Everything flagged in earlier runs (icon-color indirection, compact input sizing, coarse-pointer row actions, refresh button accessible name, fixed row height) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One layout finding on the redesigned models list; details inline. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete/update buttons, size="compact" on the env and filter inputs, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, the coarse-pointer row actions, and the missing height cap below lg) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings on this head; details inline. Everything flagged in earlier runs (icon-color indirection, compact input sizing, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, coarse-pointer row actions, the missing height cap below lg, the 3rem actions track, and the filter clearing on add) reads as addressed.

^{Posted via Macroscope — UI Consistency}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 71ceb4f. Configure here.}

[StiensWout]

The provider editor was a long stack of full-width label-over-input fields, a four-column table for usually zero to two environment variables, and a Models tab capped at 160px with four always-visible icon buttons per row. Codex models also never showed "Fast mode" because the label only matched the boolean fastMode descriptor, not Codex's serviceTier select.

What changes:

• Configuration tab regrouped into a label-left grid (new grid variant of ProviderSettingsForm; the Add wizard's dialog variant is untouched): display name and accent swatches on one line, then Runtime and Environment groups under small divider labels. Each driver field is one line, with its description beside a fixed-width input. Environment variables are compact NAME = value rows with a lock toggle for sensitive values. Delete instance moves to the editor header.
• Models tab: no height cap, rows are a fixed-column grid (star, name and slug, capabilities, reorder and remove, picker switch) so capabilities line up and the reorder arrows stay in the row instead of appearing on hover. Picker visibility is a per-row switch (custom models show a disabled checked switch since they are always offered), favorites are grouped first with hidden models sunk to the bottom, a filter input appears above eight models, and "Add custom model" is a ghost button that expands into the input. The Models tab label shows the model count and how many are hidden.
• Capability labels count a Codex serviceTier select with a "Fast" tier as Fast mode, matching the composer's logic.
• Section header: "Checked Xs ago" and refresh are one compact ghost button next to Add provider.

Before

Configuration tab:

Models tab:

After

Screenshots predate the last commit, which made each driver field one line, aligned the model columns, and added the model count to the tab.

Configuration tab:

Models tab:

Checks

• vp test run on the four provider settings test files (21 tests), including a new groupModelsForDisplay test
• vp run --filter @t3tools/web typecheck
• Focused lint and format on the changed files

Built with Claude Fable 5.1 in the Claude Code harness via T3 Code.

---

Note

Medium Risk
Model list ordering and hidden-model grouping change persisted modelOrder semantics and what users see for edge cases (e.g. custom slugs in hidden prefs); otherwise UI-only settings changes with new test coverage for grouping.

Overview
Redesigns the provider instance Configuration and Models tabs plus related settings chrome.

The Configuration tab moves to a label-left two-column grid: display name shares a row with an inline accent swatch row (ProviderAccentColorPickerlayout="inline"), driver fields use a new grid variant on ProviderSettingsForm (description beside fixed-width controls), and Runtime / Environment sections get divider labels. Environment variables drop the bordered table for compact NAME = value rows with lock/unlock for sensitivity and inline remove. Delete instance moves from the list header icon to a text button in the editor header.

The Models tab removes the short height cap and info popovers in favor of a fixed-column row layout: inline capability tags (including Fast mode for Codex serviceTier selects), always-visible reorder/remove actions, and a picker visibility switch instead of eye icons. Lists are ordered via new groupModelsForDisplay (favorites, then visible, then hidden built-ins; custom models never treated as hidden). Reorder only swaps neighbors within the same group; filtering appears when there are more than eight models; add-custom-model is collapsible with cancel/Escape and scroll-to-new-row. The Models tab label shows total and hidden counts.

Providers section header combines refresh and “Checked … ago” into one compact button (sr-only label for tests); read-only mode shows last-checked text and keeps Advanced health-interval labels readable while controls stay inert.

^{Reviewed by Cursor Bugbot for commit d6fb0bb. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add inline layout option to ProviderAccentColorPicker

Extracts the custom picker, preset swatches, and clear action into a reusable swatch row. The new optional stacked/inline layout prop defaults to stacked, which keeps the label and description. inline mode renders only the swatch row.

^{Macroscope summarized d6fb0bb.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a4150b38-4d2a-4471-af42-52f97c78b786

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the provider list/editor redesign for shared-primitive and Tailwind-ownership consistency. Three findings, all in changed lines, about icon-color indirection through Button and a call-site override of Switch geometry. Layout/inert/read-only handling, the new grid variant of ProviderSettingsForm, and the models grouping helper look consistent with the existing contracts.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review

Four findings, all in changed lines. Three are shared-primitive contract issues where a call-site class never reaches the element it is meant to style:

• Button routes icon color through --control-icon-color ([&_svg:not([class*='text-'])]:text-[var(--control-icon-color)]), so text-* on the button does not tint a bare icon child. The favorite star and the sensitive/lock toggle lose their state colors as a result.
• Input applies className to the outer input-control wrapper, not the inner <input>, so h-8 / text-xs at the call site do not resize the control; size="compact" is the existing contract for that.

No issues found with the grid variant in ProviderSettingsForm, the layout="inline" accent picker, the read-only inert scoping in ProviderInstanceCard / ProviderSettingsPanel, or the --thumb-size switch override (an established call-site pattern).

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This is a substantial provider-settings redesign with new model filtering and visibility interactions plus changed persistence semantics for model ordering. The configuration and environment editor behavior also changes across the production settings surface, warranting human review.

^{You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

One accessibility regression found in the reworked Providers section header. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete buttons, size="compact" on the env and filter inputs, switch thumb sizing) is addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the provider editor header/configuration grid. Prior comments on icon-color indirection and compact input sizing have been addressed in this revision.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two layout findings in the redesigned provider list row / environment rows. Everything flagged in earlier runs (icon-color indirection on the star and lock buttons, size="compact" on the env and filter inputs, the switch thumb override, and the refresh button's accessible name) looks resolved.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review: two findings on the redesigned provider panels. Earlier findings (icon-color indirection on the star/lock buttons, size="compact" on the env/filter inputs, switch thumb sizing, refresh-button accessible name) look resolved on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One new finding on the redesigned models list; details inline. The coarse-pointer issue flagged on the previous push (hover-only reorder / remove-custom-model actions in ProviderModelsSection) still stands and is not re-posted here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings in the redesigned provider panel. Everything flagged in earlier runs (icon-color indirection, compact input sizing, coarse-pointer row actions, refresh button accessible name, fixed row height) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One layout finding on the redesigned models list; details inline. Everything flagged in earlier runs (icon-color indirection on the star/lock/delete/update buttons, size="compact" on the env and filter inputs, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, the coarse-pointer row actions, and the missing height cap below lg) reads as addressed on this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Two findings on this head; details inline. Everything flagged in earlier runs (icon-color indirection, compact input sizing, the fixed h-19 list row, the refresh button's accessible name, the accent group label, the disabled-switch tooltip trigger, coarse-pointer row actions, the missing height cap below lg, the 3rem actions track, and the filter clearing on add) reads as addressed.

^{Posted via Macroscope — UI Consistency}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 71ceb4f. Configure here.}