feat: Refine provider settings and picker availability states#1289
feat: Refine provider settings and picker availability states#1289huxcrux wants to merge 2 commits intopingdotgg:mainfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip CodeRabbit can generate a title for your PR based on the changes.Add |
What Changed
Refined the provider settings and picker experience.
Related PRs: #1288
Why
The provider settings were getting long and repetitive, and that will become more noticeable as more providers are added.
Grouping each provider into its own expandable section makes the settings easier to scan now and gives us a better place to add provider-specific options later. It also makes provider availability more explicit and predictable: if a provider is disabled, not found, or unauthed, it should not appear selectable elsewhere in the app.
UI Changes
Added aa few extra pictures to show the different states when a provider is not found, unauthed or working fine.
Provider picker:
Settings:
Checklist
Note
Add per-provider enable/disable toggles, status badges, and refresh capability to settings
enabledProvidersfield toAppSettings(defaulting bothcodexandclaudeAgenttotrue) with helpersisProviderEnabled,getEnabledProviderOptions, andpatchProviderEnabledin appSettings.tsserver.refreshProviderStatusesWebSocket RPC, backed by arefreshStatuseseffect in ProviderHealth.ts that recomputes and caches statusesMacroscope summarized 8cd00ec.