Skip to content

test(sessions): synchronize worker readiness in proxy flow - #39

Merged
YueZh127 merged 1 commit into
qa-tools-fkst-stagingfrom
codex/pr37-worker-readiness-fix
Sep 8, 2026
Merged

YueZh127 merged 1 commit into
qa-tools-fkst-stagingfrom
codex/pr37-worker-readiness-fix

Conversation

@YueZh127

@YueZh127 YueZh127 commented Sep 8, 2026

Copy link
Copy Markdown

Purpose

Fix the deterministic readiness race exposed twice by PR #37 CI. createSession returns submitted; the proxy integration test now waits for the worker to complete its claim and begin executor creation before sending the first action. Production session semantics are unchanged.

Behavior

  • Uses an explicit executor-ready signal, not sleeps or retrying 409.
  • Races readiness against worker failure, early stop, and a bounded diagnostic deadline.
  • Verifies claimed|running through the same NyxID-style proxy.
  • Preserves raw response bodies in HTTP failure diagnostics.

Verification

  • Focused proxy test: pass.
  • Concurrent local runs: 12/12 pass.
  • Independent reviewer repeated 20 runs: 40/40 cases pass.
  • Full suite: 53 files / 370 tests pass.
  • Standalone Mongo contract: 80/80 pass.
  • Build, typecheck, lint, bundle, fixture drift, and both contract demos pass.
  • Independent review: initial P2/P3 findings fixed; same reviewer approved the final diff.

Normal merge commit into staging only. No squash, rebase, force-push, auto-merge, or branch deletion.

@YueZh127
YueZh127 merged commit f104b16 into qa-tools-fkst-staging Sep 8, 2026
4 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