Skip to content

[codex] stabilize stress descendant test - #74

Merged
greenthree merged 1 commit into
mainfrom
codex/stress-pid-handshake
Aug 12, 2026
Merged

[codex] stabilize stress descendant test#74
greenthree merged 1 commit into
mainfrom
codex/stress-pid-handshake

Conversation

@greenthree

Copy link
Copy Markdown
Owner

Summary

  • atomically publish the stress checker child PID used by the descendant cleanup test
  • keep the real timeout and process-tree death assertions unchanged

Root cause

Main CI run 31511756536 failed only on Windows / Python 3.10 because Path.write_text briefly exposed an empty PID file before writing the PID. The checker timeout could terminate the child during that window, leaving the test to parse "" as an integer. The production process cleanup completed; the failure was in the test handshake and is unrelated to the typeid(type-id) mutation change.

Validation

  • focused descendant cleanup test: 30/30 consecutive passes
  • python -m unittest tests.test_stress -v: 15 passed
  • npm run check: 713 passed, 30 skipped
  • npm run pack:check: probhub 223 files, probhub-skill 5 files
  • local phtest nine-problem lint: ok
  • git diff --check: passed

@greenthree
greenthree marked this pull request as ready for review August 12, 2026 07:12
@greenthree
greenthree merged commit 9f9063a into main Aug 12, 2026
11 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