Skip to content

fix(codex): discover the nested ChatGPT CLI bundle - #369

Merged
chattymin merged 1 commit into
mainfrom
codex/fix-chatgpt-codex-path
Sep 28, 2026
Merged

chattymin merged 1 commit into
mainfrom
codex/fix-chatgpt-codex-path

Conversation

@chattymin

Copy link
Copy Markdown
Owner

Summary

ChatGPT now bundles Codex under Contents/Resources/codex-cli/CodexCLI.app/Contents/MacOS/codex. After the old executable disappears, PokeTokenBar cannot discover the CLI and hides Codex limits.

Add the current bundle path while retaining the legacy path and dedicated-install precedence. Add regression tests against the production candidate list and record the missed discovery coverage.

Type of change

  • Bug fix
  • New feature
  • Refactor / cleanup
  • Documentation
  • Other:

Validation

  • swift build passed; swift test: 1,382 tests, 15 skipped, zero failures.

  • Targeted candidate regression tests, including a mutation check that removes the new path.

  • Read-only account/rateLimits/read smoke check using the new executable with a minimal GUI-style PATH.

Checklist

  • swift build and swift test pass locally
  • PR title and description are written in English
  • UI changes are described above (before/after — images optional; not applicable)
  • No copyrighted assets, secrets, or private tooling references are committed (see CONTRIBUTING)
  • Tests were added or updated for this change

@chattymin chattymin self-assigned this Sep 28, 2026
@chattymin
chattymin merged commit 8c08e01 into main Sep 28, 2026
2 checks passed
@chattymin
chattymin deleted the codex/fix-chatgpt-codex-path branch September 28, 2026 04:49
donky-ey pushed a commit to donky-ey/PokeDexBar that referenced this pull request Oct 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant