Skip to content

fix(agents): accept Codex turns after reconnect - #27

Merged
futrime merged 1 commit into
mainfrom
fix/codex-transient-reconnect
Aug 28, 2026
Merged

fix(agents): accept Codex turns after reconnect#27
futrime merged 1 commit into
mainfrom
fix/codex-transient-reconnect

Conversation

@antoinegg1

Copy link
Copy Markdown
Contributor

Summary

  • Treat an intermediate Codex app-server error as superseded when the same turn later completes successfully.
  • Preserve immediate failure behavior for terminal failed or interrupted turns.
  • Cover a Reconnecting... 1/5 event followed by successful completion.

Validation

  • uv run pre-commit run --all-files
  • uv run pytest tests/agents/test_appservers.py -q: 45 passed
  • uv run pytest --ignore=tests/coganchor: 1469 passed, 36 skipped

The complete suite was also attempted. Coganchor tests cannot run in this container because pidfd_getfd returns EPERM without CAP_SYS_PTRACE; before interruption, 500 tests passed and 65 skipped, with failures limited to tests/coganchor.

@futrime
futrime merged commit c929e31 into main Aug 28, 2026
4 checks passed
@futrime
futrime deleted the fix/codex-transient-reconnect branch August 28, 2026 02:34
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