Skip to content

CI only: Windows deadline test waits for readiness - #17

Merged
ualtinok merged 3 commits into
mainfrom
ci/windows-readiness
Oct 13, 2026
Merged

ualtinok merged 3 commits into
mainfrom
ci/windows-readiness

Conversation

@basal-alfonso

@basal-alfonso basal-alfonso Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

CI-only draft for a worker's Windows proof. Not for merging; the change reaches main through the gated push.


Summary by cubic

Makes the Windows subprocess deadline tests reliable by starting the output deadline only after confirming the descendant process is actually running.

  • Adds output_until_ready, which polls a readiness predicate on Windows before starting the output deadline timer.
  • Updates the Windows pipe tests to wait for the descendant PID to be live instead of assuming it starts immediately.
  • Adds a guarded mutation proving the deadline test fails when readiness gating is removed.

Written for commit 33478e8. Summary will update on new commits.

View guided diff Turn on auto-fix

@ualtinok
ualtinok merged commit 03616ab into main Oct 13, 2026
14 checks passed
@basal-alfonso

basal-alfonso Bot commented Oct 13, 2026

Copy link
Copy Markdown
Author

Closing: CI-only branch; its change reaches main through the gated push.

@ualtinok
ualtinok deleted the ci/windows-readiness branch October 13, 2026 17:13
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