Fix Codex provider banner for custom overrides#1218
Fix Codex provider banner for custom overrides#1218varundeepsaini wants to merge 1 commit intopingdotgg:mainfrom
Conversation
Signed-off-by: Varun Deep Saini <varun.23bcs10048@ms.sst.scaler.com>
|
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)
📝 Coding Plan
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 suggest fixes for GitHub Check annotations.Configure the |
|
Picture please? |
What Changed
Why
The server provider health snapshot uses the default Codex environment, while new sessions can use browser-configured Codex overrides. This removes the false unavailable banner in that override case and keeps the override handling consistent in one place.
Checklist
Note
Fix Codex provider banner to hide health status when custom overrides are configured
getCodexProviderOverridesin appSettings.ts to trimcodexBinaryPathandcodexHomePathfromAppSettings, returningundefinedwhen both are blank.resolveProviderHealthBannerStatusin ChatView.logic.ts to suppress the Codex provider health banner when local overrides are present; other providers are unaffected.ProviderHealthBannerno longer shows Codex health when a custom binary path orCODEX_HOMEis configured.codexBinaryPathorcodexHomePathis set to a non-blank value.Macroscope summarized b8485a5.