Repository navigation
feat(tui): offer sign-in for signed-out providers in the model picker - #302
Merged
Merged
Conversation
The provider page lists providers that advertise a terminal sign-in method but have no models yet below the active ones, greyed out. Entering one starts that provider's login flow. /model opens the picker when only sign-in methods are available. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
daviddanialy
marked this pull request as draft
October 22, 2026 19:36
The agent tags each terminal auth method with its provider under the `kit/provider` metadata key and rebuilds the session's model catalog when the set of credentialed providers changes, so the post-login refresh advertises the new provider's models. The TUI treats a provider as signed in when the catalog has a group for it, even if model discovery returned nothing, and offers sign-in only once a catalog is known. Picker sign-in waits for the current turn, keeps the selection highlighted, and reopens the provider's models with the same defaults choice after a successful login. /model no longer opens an empty picker. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
daviddanialy
marked this pull request as ready for review
October 22, 2026 20:23
A signed-in provider whose model discovery failed now shows why its model page is empty, a search with no results says so, and the footer drops the enter hint when there is nothing to select. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The
/modelprovider page now lists providers you can sign in to but haven't yet, below the signed-in providers and greyed out. Pressing enter on one starts that provider's terminal sign-in (the same flow as/login <id>). When sign-in succeeds, the picker reopens on that provider's models with the same "tab defaults" setting.Agent
kit/providerfield in its_meta, naming the provider it signs in to.TUI
/loginand model switches./modelnever opens an empty picker.enter sign inwhile a greyed-out row is selected.--remotethe picker shows the signed-in providers as before./modelsection ofdocs/user/tui-and-sessions.mddescribes the new behaviour.Demo
Testing
mise run lintcargo test --lib: 2001 passed. Two shell tests fail locally, and they also fail onmain: a shell startup file on the test machine prints a warning into their captured stderr.plugins::tests::system_git_runner_disables_configured_askpass_*was skipped because it hangs locally waiting on git. This PR doesn't touch the shell, git or plugin code.kit/providerfield on sign-in methods馃 Generated with Claude Code