Skip to content

Stabilize child stdio lifecycle tests - #32

Merged
beiwei30 merged 1 commit into
mainfrom
fix/child-stdio-lifecycle-test-stability
Aug 26, 2026
Merged

Stabilize child stdio lifecycle tests#32
beiwei30 merged 1 commit into
mainfrom
fix/child-stdio-lifecycle-test-stability

Conversation

@beiwei30

Copy link
Copy Markdown
Owner

Summary

  • split normal lifecycle grace, explicit escalation timing, and the outer deadlock watchdog so parallel scheduler delay cannot turn graceful exits into forced termination
  • add a deterministic two-phase broken-stdin fixture handshake while preserving exact StdinIo, pending-request release, and child-reaping assertions
  • document the reproduced causal orderings and closeout evidence; production transport code and the maintenance spawn disposition remain unchanged

Verification

  • cargo test -p orbcode-app-server-client --test child_stdio_transport
  • cargo test -p orbcode-app-server-client --test child_stdio_transport -- --test-threads=1
  • exact canonical initialize, broken-stdin, and outbound-payload tests
  • 20 consecutive default-parallel child_stdio_transport target runs
  • cargo test -p orbcode-app-server-client
  • cargo test -p orbcode --test child_stdio_client_e2e
  • maintenance self-test and audit, docs audit, rustfmt, focused clippy, and focused check
  • two consecutive complete scripts/check.sh runs from a clean worktree

No retries, ignored tests, serial annotations, public API changes, protocol changes, or production Rust changes were introduced.

CopilotAI lite review requested due to automatic review settings August 26, 2026 14:27

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@beiwei30
beiwei30 merged commit 912762e into mainAug 26, 2026
2 checks passed
@beiwei30
beiwei30 deleted the fix/child-stdio-lifecycle-test-stability branch August 26, 2026 14:47
Sign up for freeto 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

@beiwei30