Skip to content

fix(web): open provider settings on the Models tab - #8543

Open
Gigioxx wants to merge 1 commit into
pingdotgg:mainfrom
Gigioxx:fix/provider-settings-default-models-tab
Open

fix(web): open provider settings on the Models tab#8543
Gigioxx wants to merge 1 commit into
pingdotgg:mainfrom
Gigioxx:fix/provider-settings-default-models-tab

Conversation

@Gigioxx

@Gigioxx Gigioxx commented Aug 28, 2026

Copy link
Copy Markdown

Fixes #8521.

Since #8380 every provider opened its editor on the Configuration tab, so set-once plumbing (display name, accent color, env vars, binary path) was the first thing on the Providers page instead of the Models list users actually change.

ProviderInstanceCard now initialises activeTab to "models". The existing visibleTab guard still forces Configuration for unknown drivers that have no Models tab, so nothing changes for custom fork instances.

Screenshots

Before (Configuration by default) After (Models by default)
Providers page opening on the Configuration form Providers page opening on the Models list

Checks

  • vp test run apps/web/src/components/settings/ProviderInstanceCard.test.ts apps/web/src/components/settings/ProviderSettingsPanel.environment.test.tsx
  • vp run typecheck in apps/web
  • Targeted lint
  • Verified in Chrome at 1536 x 860: default selection and switching providers both land on Models; Configuration stays one click away
  • Codex review of the diff: no findings

Made with Claude Fable 5 in the Claude Code harness through T3 Code.


Note

Low Risk
Single default UI state change with no API, auth, or data-handling impact; unknown-driver fallback behavior is preserved.

Overview
Provider instance editor now opens on the Models tab instead of Configuration, so the Providers page surfaces model visibility/favorites first rather than one-time plumbing (display name, env vars, binary path).

Only the initial activeTab state in ProviderInstanceCard changes from "configuration" to "models". The existing visibleTab rule still pins Configuration when driverOption is missing (unknown/custom fork drivers with no Models tab), so that path is unchanged.

Reviewed by Cursor Bugbot for commit 4eab817. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Set default tab to models in ProviderInstanceCard

Changes the initial activeTab state in ProviderInstanceCard.tsx from configuration to models. The Models tab is now shown first when opening provider settings.

Macroscope summarized 4eab817.

@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

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: 4219831b-aa78-401e-92a8-85133cbd170f

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.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Aug 28, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 4eab817

Macroscope's review found this PR approvable — This is a self-contained one-line UI default change that opens known provider settings on the existing Models tab while preserving the Configuration fallback for unknown drivers. It has no API, data, authentication, billing, or deployment impact.

You can add or adjust custom eligibility rules. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Providers settings page makes the Configuration form the focus instead of provider list and models (regression from #8380)

1 participant