Skip to content

test: synchronize legacy automation race before interruption - #213

Merged
Zongwei9888 merged 1 commit into
mainfrom
codex/stabilize-automation-race-test
Sep 9, 2026
Merged

test: synchronize legacy automation race before interruption#213
Zongwei9888 merged 1 commit into
mainfrom
codex/stabilize-automation-race-test

Conversation

@Zongwei9888

Copy link
Copy Markdown
Collaborator

The legacy-client automation race test could interrupt its foreign Turn before the scripted Agent claimed the first step. The subsequent Automation then consumed that gated step and timed out instead of completing. This caused Python 3.12 to fail on the first main run after #212, despite the PR checks passing.

Wait for the foreign Agent to claim its step before interrupting it. All existing assertions remain, including refusal to adopt the foreign Turn and eventual completion with the Automation’s own Turn. Only seven test lines change; production code and timeouts are unchanged.

Validation:

  • Reproduced the original timeout by delaying the foreign Agent start by 0.2 seconds with a local pytest plugin.
  • All 32 automation goal-run tests pass with that delay applied.
  • The affected test passes 20 consecutive independent delayed-start runs.
  • Pre-commit checks pass.

This follow-up will be merged only after its applicable CI checks pass.

@Zongwei9888
Zongwei9888 merged commit d2ca51d into main Sep 9, 2026
12 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