Skip to content

fix(cli): avoid app-local agent shims - #10

Merged
devswha merged 2 commits into
mainfrom
codex/fix-external-cli-executable-resolution
Jul 22, 2026
Merged

fix(cli): avoid app-local agent shims#10
devswha merged 2 commits into
mainfrom
codex/fix-external-cli-executable-resolution

Conversation

@devswha

@devswha devswha commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove app-local node_modules/.bin directories from external-agent executable lookup
  • resolve the first executable user installation to an absolute path before starting tmux
  • retain the existing bare-command failure behavior when no executable can be resolved

Root cause

npm run dev prepends ChatMux's own node_modules/.bin to PATH. A new Codex tmux therefore launched the repository SDK dependency (0.141.0) instead of the user's installed Codex CLI (0.144.6). The older binary could not run the configured gpt-5.6-sol model. This was reproduced twice and confirmed from the live process tree.

Verification

  • external session service tests (28/28)
  • npm run typecheck
  • npm run lint
  • npm run build

@devswha

devswha commented Jul 22, 2026

Copy link
Copy Markdown
Owner Author

E2E QA evidence (2026-07-22): a newly created Codex tmux showed global Codex 0.144.6 instead of the repository-local 0.141.0 shim, then completed the requested file edit and validation commands.

# Conflicts:
#	server/modules/providers/tests/external-cli-sessions.service.test.ts
@devswha
devswha merged commit 69af51b into main Jul 22, 2026
2 checks passed
@devswha
devswha deleted the codex/fix-external-cli-executable-resolution branch August 13, 2026 13:40
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