dcdpr / jp / 26805122083

67%

Build Type

push

github

Committed by web-flow

Commit Message

chore(tools): Add supervised shutdown ladder and stale sandbox sweep (#711)

Debug tools that launch `jp` previously waited indefinitely for the
child to exit. A `jp` command that blocks or hangs would wedge the tool
forever.

This commit adds a time-budgeted supervision layer to the launch
harness. A process that doesn't exit within its run window is shut down
via an escalating ladder: two `SIGINT`s (mirroring a manual double
Ctrl+C, which drives jp's own graceful shutdown and flushes the trace
log / dhat heap profile), each followed by a grace window, then
`SIGKILL` as the backstop for a wedged process. The outcome is recorded
in `LaunchResult::termination` (`Exited`, `Graceful`, or `Forced`), and
a warning banner is prepended to the rendered report whenever jp didn't
exit on its own.

To keep pipe-fill deadlocks out of the picture, stdout and stderr are
now drained on dedicated reader threads rather than via
`wait_with_output`. jp is also placed in its own process group at spawn
so `SIGINT`/`SIGKILL` reach any children it left behind.

`Sandbox::create` now calls `sweep_stale_sandboxes` on entry to reclaim
worktrees left behind by a host process that was killed before its
`Drop` ran. Liveness is checked via `kill(pid, 0)` on unix; the sweep is
a no-op on non-unix targets.

A new `.jp/config/skill/debug.toml` registers the three debug tools and
a system-prompt section that describes when and how to reach for each
one.

Signed-off-by: Jean Mertz <git@jeanmertz.com>

Coverage Stats

152 of 345 new or added lines in 7 files covered. (44.06%)

37 existing lines in 5 files now uncovered.

33288 of 50104 relevant lines covered (66.44%)

282.05 hits per line