Skip to content

feat(providers): redesign setup and connect model pickers - #5706

Merged
georgi merged 2 commits into
mainfrom
codex/provider-setup-ui
Sep 10, 2026
Merged

feat(providers): redesign setup and connect model pickers#5706
georgi merged 2 commits into
mainfrom
codex/provider-setup-ui

Conversation

@georgi

@georgi georgi commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

What changed

Model pickers now open provider setup when no configured provider supports the requested model type, and every picker includes an Add providers action. Provider discovery finishes before prompting, and local, OAuth, and managed providers count as configured. The setup dialog groups account sign-in and API keys, filters choices by task, and refreshes provider/model caches after connecting. The Providers settings page uses grouped rows with clearer status and actions. OpenAI sign-in is offered for chat, while image, speech, and embedding setup requires an API key.

Verification

  • npm run test:affected — 539 suites passed, 4,610 tests passed, 3 skipped. Run with localhost access for UI image fixtures.
  • NODE_OPTIONS=--max-old-space-size=8192 npm run typecheck
  • npm run lint
  • npm run dev:nodetool -- harness gate --base origin/main — 3 selfchecks passed.
  • Inspected the provider dialog at desktop and phone widths and the model picker in the running app.

The automatic-opening assertion was inverted once in useModelProviderSetup.test.tsx and failed as expected, then restored and rerun successfully:

Expected number of calls: 0
Received number of calls: 1
Test Suites: 1 failed, 1 total
Tests: 8 failed, 9 passed, 17 total

Screenshots

Desktop provider setup

Desktop provider setup

Mobile provider setup

Mobile provider setup

@georgi
georgi marked this pull request as ready for review September 10, 2026 07:20
@georgi
georgi enabled auto-merge (squash) September 10, 2026 07:20
@georgi
georgi merged commit 61439c8 into main Sep 10, 2026
22 of 23 checks passed
@georgi
georgi deleted the codex/provider-setup-ui branch September 10, 2026 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant