Skip to content

fix(cli): set codex process title in launcher#2

Draft
middleDuckAi wants to merge 2 commits intoupstream-main-20260324from
middleDuck/codex-15238-process-title
Draft

fix(cli): set codex process title in launcher#2
middleDuckAi wants to merge 2 commits intoupstream-main-20260324from
middleDuck/codex-15238-process-title

Conversation

@middleDuckAi
Copy link
Copy Markdown
Owner

@middleDuckAi middleDuckAi commented Mar 24, 2026

Summary

  • set the Node launcher process title to codex
  • keep the launcher surface small by moving the assignment into a dedicated helper
  • add an automated Node builtin regression test for the launcher process-title helper

Why

Issue openai#15238 reports that the launcher shows up as node in process listings even though users invoke codex. Setting the launcher process title to codex makes process listings and terminal tooling reflect the actual command being run.

Testing

  • node --check codex-cli/bin/codex.js
  • node --check codex-cli/bin/process_title.js
  • node --check codex-cli/tests/process_title.test.js
  • node --test codex-cli/tests/process_title.test.js
  • manual process-title repro with a temporary local vendor binary and ps -o pid=,comm=,command=

Notes

  • This is intentionally a launcher-only change in codex-cli/bin/codex.js.
  • The new test keeps coverage lightweight by using Node's builtin test runner instead of adding extra test infrastructure.

@middleDuckAi middleDuckAi changed the base branch from main to upstream-main-20260324 March 24, 2026 11:30
@middleDuckAi middleDuckAi force-pushed the middleDuck/codex-15238-process-title branch from 0444bca to e06db85 Compare March 24, 2026 12:24
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