Skip to content

fix: detect Codex in nested ChatGPT macOS app bundle - #2413

Closed
wsgxy986 wants to merge 1 commit into
first-tree-ai:mainfrom
wsgxy986:fix/macos-codex-runtime-discovery
Closed

wsgxy986 wants to merge 1 commit into
first-tree-ai:mainfrom
wsgxy986:fix/macos-codex-runtime-discovery

Conversation

@wsgxy986

Copy link
Copy Markdown

Summary

Fix Codex runtime discovery for newer ChatGPT macOS app bundles. First Tree 0.5.23 failed to start a Codex chat even though the executable was installed and runnable at:

/Applications/ChatGPT.app/Contents/Resources/codex-cli/CodexCLI.app/Contents/MacOS/codex
  • Add the nested app bundle directory for both /Applications and ~/Applications.
  • Preserve existing PATH precedence and legacy app locations.
  • Add regression tests for system-wide and per-user installations.

Creating a ~/.local/bin/codex symlink and restarting the daemon restored discovery on the affected machine. This change makes that workaround unnecessary for this installation layout.

Validation

  • Both new regression cases failed before the fix and passed afterward.
  • All 35 tests in source-repos.test.ts and providers/codex/__tests__/binary.test.ts passed.
  • Client TypeScript check passed.
  • Repository-wide Biome check passed with existing warnings.
  • git diff --check passed.
  • Full repository typecheck and tests were attempted but blocked by the local dependency setup (ERR_PNPM_IGNORED_BUILDS). They are not claimed to have passed.

Notes

  • No package version changes or public CLI changes.
  • Related QA case: packages/qa/cases/runtime/daemon-probe-capability.md.
  • Draft pending full CI validation and contributor identity/CLA checks.
  • Replaces fix: detect Codex in nested ChatGPT macOS app bundle #2412 with a completed description; the code change is unchanged.

@github-actions

Copy link
Copy Markdown

Thank you for your contribution. Before it can be accepted, you must be covered by the First Tree ICLA v1.0. Sign individually with the exact comment below. If you contribute for an organization, an authorized signatory can instead complete the First Tree CCLA v1.0 and send it to legal@first-tree.ai for verification.


I have read the CLA Document and I hereby sign the CLA


gongxinyu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@wsgxy986 wsgxy986 closed this Sep 30, 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