stacklok / toolhive-studio / 25744143487

69%

main: 70%

Build Type

Pull #2217

github

Committed by reyortiz3

Commit Message

fix(main): prevent orphaned thv serve when parent exits before SIGKILL timer

stopToolhive() nulled the module-level toolhiveProcess reference immediately
after sending SIGTERM and schedule SIGKILL via setTimeout. When Electron's
parent process exited before the 2s timer fired, the synchronous
process.on('exit') handler called stopToolhive({ force: true }) but it
short-circuited on !toolhiveProcess, leaving the child alive and blocking
the next launch's port.

Hold the reference until the child's 'exit' event fires (tracked via a new
isStopping flag for isToolhiveRunning) so the synchronous parent-exit force
kill actually SIGKILLs the orphaned PID. Guard the exit handler against
clobbering a fast-restart's new spawn.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #2217: fix(main): prevent orphaned thv serve when parent exits before SIGKILL timer

Coverage Stats

4841 of 7608 branches covered (63.63%)

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

16 existing lines in 1 file now uncovered.

7219 of 10514 relevant lines covered (68.66%)

118.95 hits per line