fix(web): show health banner for selected provider instead of defaulting to Codex#1223
Conversation
…defaulting to Codex The health banner always showed the Codex provider status regardless of which provider the user selected, because activeProvider fell back to "codex" when no session existed yet. Use selectedProvider instead, which correctly resolves from the composer draft selection.
|
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 use oxc to improve the quality of JavaScript and TypeScript code reviews.Add a configuration file to your project to customize how CodeRabbit runs oxc. |
github-actions
bot
added
size:XS
2 participants
Summary
ProviderHealthBanneralways displayed the Codex provider status regardless of which provider the user selected in the composer, becauseactiveProviderfell back to"codex"when no session existed yet (sessions are only created after the first message is sent)."codex"fallback withselectedProvider, which correctly resolves from the composer draft's provider selection."<provider> provider status"for all, instead of special-casing Codex).Note
[!NOTE]
Fix health banner to show status for the selected provider instead of defaulting to Codex
selectedProviderinstead ofactiveThread?.session?.providerwith a"codex"fallback."<Provider> provider status"for all providers, removing prior special-casing that omitted "provider" for non-Codex labels.Macroscope summarized 20e79bd.