llamastash / llamastash / 26689431081

87%

Build Type

push

github

Committed by deepu105

Commit Message

fix(windows): make ProcessControl a process singleton

WindowsProcessControl owns the per-PID JobHandle map. Each Job is set
to KILL_ON_JOB_CLOSE, so dropping the last handle to a Job kills every
process assigned to it. platform_default() was returning a fresh Arc
on every call, so the controller (and its HashMap, and every
JobHandle inside it) was dropped the instant spawn_supervised
returned — taking the just-spawned child with it. The supervisor's
probe at /health saw nothing because the child had already been
terminated. Routing platform_default() through a OnceLock keeps the
controller alive for the whole process; on Unix the unit struct
backend makes the singleton a no-op.

Fixes the three Windows integration tests that timed out with
"supervisor never reached ready" — agent_script_round_trip,
logs_follow_returns_daemon_unreachable_when_daemon_dies,
start_preset_chain_seeds_supervisor_with_saved_params.

Coverage Stats

7 of 7 new or added lines in 1 file covered. (100.0%)

11 existing lines in 3 files now uncovered.

41683 of 48330 relevant lines covered (86.25%)

1674.81 hits per line