Skip to content

fix: detect Codex runtime in desktop app - #252

Merged
SaladDay merged 1 commit into
mainfrom
fix/codex-desktop-runtime
Sep 2, 2026
Merged

fix: detect Codex runtime in desktop app#252
SaladDay merged 1 commit into
mainfrom
fix/codex-desktop-runtime

Conversation

@SaladDay

@SaladDay SaladDay commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

What changed

  • Treat a non-empty CODEX_THREAD_ID as an active Codex runtime.
  • Preserve the existing PATH-based detection for all coding agents.
  • Cover runtime-only, PATH merge, missing, empty, and whitespace marker cases.
  • Add the repository blind-review workflow used for this change.

Scope

This PR only addresses Codex running inside ChatGPT Desktop. It does not add runtime detection for other agents or change installation and configuration behavior.

Validation

  • 500 tests passed.
  • Type check passed.
  • Production build passed.
  • Lint passed with one existing warning in test/sdk/speech.test.ts.
  • Final blind review found no issues.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@SaladDay
SaladDay merged commit cbf7089 into main Sep 2, 2026
5 checks passed
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