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.Closes
Closes Healthcheck appears to not be using the custom
CODEX_HOMEand path binary settings #693Closes [Bug]: CODEX_HOME path override is ignored in some parts of the app #1158
Closes Codex binary path stuck on stale value; app keeps checking /Users/username/bin/codex even after reset #1022
Closes Codex provider status following installation can't be dismissed without closing and re-opening app #197
Closes Persistent "Codex CLI not found" error after installation of codex CLI #372
Should close 'codex' not found error but it actually works #630
UI Changes
Small interaction change only:
No screenshots or video attached.
Checklist