Update Codex provider banner when Codex path settings change#1185
Update Codex provider banner when Codex path settings change#1185jamesx0416 wants to merge 1 commit 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)
📝 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 generate a title for your PR based on the changes.Add |
What Changed
server.validateCodexCliRPC so the server can revalidate Codex availability for an effectivebinaryPath/homePathpairserver.getConfig()and the existing Codex banner can reflect revalidation resultscodexBinaryPathorcodexHomePathinput blursWhy
The existing banner only reflected a startup-time
codex-from-PATH probe, so changing Codex path settings could leave a stale "not on PATH" error visible even when the configured Codex binary worked. This keeps the banner tied to real validation results for the current Codex settings without weakening the existing session-start hard gate.UI Changes
Small interaction change only:
No screenshots or video attached.
Checklist
Note
Update Codex provider banner when Codex path settings change
server.validateCodexCliWebSocket RPC method that re-runs the Codex CLI health check on demand with optionalbinaryPathandhomePathoverrides, then pushesserverConfigUpdatedto all clients.ProviderHealthLivenow stores statuses in aRefand exposesrevalidateCodexStatus, replacing the startup-only snapshot with live reads.checkCodexProviderStatusForInputskips the OpenAI login probe when a custom model provider is detected in the (possibly overridden)CODEX_HOME, and uses the overridden binary path for the CLI process.server.getConfigand keybinding-change pushes now return the latest provider statuses rather than the value captured at startup.Macroscope summarized 7db778f.