Skip to content

[codex] fix(smithy): update Codex interactive CLI flags#109

Open
fnnzzz wants to merge 1 commit into
stoneforge-ai:masterfrom
fnnzzz:fix/codex-interactive-cli-flags
Open

[codex] fix(smithy): update Codex interactive CLI flags#109
fnnzzz wants to merge 1 commit into
stoneforge-ai:masterfrom
fnnzzz:fix/codex-interactive-cli-flags

Conversation

@fnnzzz
Copy link
Copy Markdown
Contributor

@fnnzzz fnnzzz commented May 6, 2026

Summary

  • Replace Codex interactive --full-auto launches with supported --sandbox workspace-write --ask-for-approval never flags.
  • Apply the replacement to both fresh interactive sessions and resumed sessions.
  • Update Smithy provider tests and add a patch changeset for @stoneforge/smithy.

Root Cause

Recent Codex CLI versions reject --full-auto, so Director and persistent-worker interactive PTYs exit before the Codex TUI starts.

Evidence

Local reproduction against the currently installed Codex CLI:

$ codex --version
codex-cli 0.128.0
$ codex --full-auto --help
error: unexpected argument '--full-auto' found

  tip: to pass '--full-auto' as a value, use '-- --full-auto'

Usage: codex [OPTIONS] [PROMPT]
       codex [OPTIONS] <COMMAND> [ARGS]

For more information, try '--help'.

codex --help lists the replacement flags used here:

-s, --sandbox <SANDBOX_MODE>
-a, --ask-for-approval <APPROVAL_POLICY>
-C, --cd <DIR>

Validation

  • bun test src/providers/codex/provider.bun.test.ts
  • pnpm run typecheck
  • pnpm run build
  • pnpm run test

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 6, 2026

CLA assistant check
All committers have signed the CLA.

@fnnzzz fnnzzz marked this pull request as ready for review May 6, 2026 15:53
@fnnzzz fnnzzz force-pushed the fix/codex-interactive-cli-flags branch from 7d671a5 to 3149011 Compare May 6, 2026 15:53
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.

2 participants