Skip to content

fix(cli): treat async accepted sends as success#481

Merged
jiashuoz merged 3 commits into
mainfrom
codex/cli-async-accepted
Jul 15, 2026
Merged

fix(cli): treat async accepted sends as success#481
jiashuoz merged 3 commits into
mainfrom
codex/cli-async-accepted

Conversation

@jiashuoz

@jiashuoz jiashuoz commented Jul 15, 2026

Copy link
Copy Markdown
Member

Summary

  • report normal async accepted send and reply results as successful CLI outcomes
  • reserve exit 3 exclusively for pending_review holds
  • add frozen exit 7 for terminal failed and unknown persisted outcomes so callers inspect the returned message id instead of risking a duplicate retry

Test plan

  • npm test --workspace @e2a/cli (112 passed)
  • npm run build --workspace @e2a/cli

@jiashuoz jiashuoz marked this pull request as ready for review July 15, 2026 05:32
@jiashuoz jiashuoz merged commit 1dcc26a into main Jul 15, 2026
14 of 15 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