fix(hooks): replace PTY with subprocess.PIPE for reliable output capture#837

Open

raballew wants to merge 23 commits into

jumpstarter-dev:mainfrom

raballew:fix/pty-read-before-stop

Commits on Jun 30, 2026

fix(hooks): eliminate macOS PTY output race condition

raballew
and
claude
committed
ci: temporarily enable macOS tests on PRs to validate PTY fix

raballew
and
claude
committed
test(hooks): cover reader_stop + BlockingIOError exit path

raballew
and
claude
committed
test(hooks): use real grandchild process to cover reader_stop path

raballew
and
claude
committed
ci: retrigger #2
raballew
committed
fix(test): don't set eof_seen on transient BlockingIOError in PtyTracker

raballew
and
claude
committed
ci: run macOS pytest 20x to validate PTY fix

raballew
and
claude
committed
fix(hooks): drain polls for full timeout instead of breaking on first empty

raballew
and
claude
committed
fix(hooks): extend reader grace period to DRAIN_TIMEOUT_SECONDS

raballew
and
claude
committed
ci: bump e2e timeout from 30m to 35m

raballew
and
claude
committed
test(hooks): remove macos_pty_xfail markers

raballew
and
claude
committed
fix(hooks): replace PTY with subprocess.PIPE for output capture

raballew
and
claude
committed
fix(hooks): use async non-blocking read on pipe fd for real-time output

raballew
and
claude
committed
fix(hooks): yield to event loop after reader to flush LogStream

raballew
and
claude
committed
revert(ci): remove temporary validation CI changes

raballew
and
claude
committed
test(hooks): add edge case tests for pipe-based output capture

raballew
and
claude
committed

Commits on Jul 1, 2026

Commits on Jul 2, 2026

ci: remove temporary stress test workflow

raballew
and
claude
committed