Skip to content

fix(codex): remove invalid resume flags#1277

Closed
Gujiassh wants to merge 1 commit intogarrytan:mainfrom
Gujiassh:fix/codex-resume-flags
Closed

fix(codex): remove invalid resume flags#1277
Gujiassh wants to merge 1 commit intogarrytan:mainfrom
Gujiassh:fix/codex-resume-flags

Conversation

@Gujiassh
Copy link
Copy Markdown

@Gujiassh Gujiassh commented Apr 30, 2026

Summary

  • remove -C and -s from the codex exec resume consult-mode command in the Codex skill
  • switch the resumed-session example to -c "sandbox_mode="read-only"", which the resume subcommand actually accepts
  • document that resume inherits working directory from the current shell/session context

Why

Issue #1258 reports that /codex consult-mode resume currently documents flags that codex exec resume rejects on codex-cli 0.125.0, causing the skill flow to fail with exit 2 instead of resuming the session.

Testing

  • bun run gen:skill-docs
  • bun test test/skill-validation.test.ts

View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews

@garrytan
Copy link
Copy Markdown
Owner

garrytan commented May 9, 2026

Closing in favor of #1270 (also by @jbetala7) which lands the same fix plus a regression test. Merged as part of the v1.30.0.0 fix wave (#1391). Thanks for the contribution!

@garrytan garrytan closed this May 9, 2026
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