Skip to content

fix(codex): expose code-mode host alongside the CLI - #1155

Open
fdobrovolny wants to merge 1 commit into
coder:mainfrom
fdobrovolny:fix/codex-code-mode-host-path
Open

fdobrovolny wants to merge 1 commit into
coder:mainfrom
fdobrovolny:fix/codex-code-mode-host-path

Conversation

@fdobrovolny

Copy link
Copy Markdown

Link the standalone code-mode helper into the CLI and Coder script bin directories when present. Preserve the current release symlink so upgrades also update the helper, and retain compatibility with older releases.

Description

Codex refuses to run any command as codex-code-mode-host is missing.

I’m blocked by the workspace tooling: commands fail because /home/coder/.local/bin/[codex-code-mode-host](https://github.com/home/coder/.local/bin/codex-code-mode-host) is missing. I haven’t been able to read the README or change the app.

Type of Change

  • New module
  • New template
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Module Information

Path: registry/coder-labs/modules/codex
New version: v5.4.2
Breaking change: [ ] Yes [X] No

Template Information

Path: registry/coder-labs/modules/codex/main.test.ts

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun fmt)
  • Changes tested locally

Related Issues

openai/codex#31831

Link the standalone code-mode helper into the CLI and Coder script bin directories while preserving the active release symlink across upgrades. Create fixture directories before writing mock binaries in the regression test.

Refs: openai/codex#31831
@fdobrovolny
fdobrovolny force-pushed the fix/codex-code-mode-host-path branch from 655f9f7 to 409a462 Compare October 2, 2026 01:05
@fdobrovolny

Copy link
Copy Markdown
Author

@35C4n0r Sorry for the issue, test is now fixed.

Comment on lines +294 to +299
const created = await execContainer(id, [
"mkdir",
"-p",
`${root}/${version}/bin`,
]);
expect(created.exitCode, created.stderr).toBe(0);

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was added to fix the issue.

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