stacklok / toolhive-studio
69%
main: 70%

LAST BUILD BRANCH: renovate/vitejs-plugin-react-6.x
DEFAULT BRANCH: main
Repo Added
Files 522
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH v0.35.0-rc.0
branch: v0.35.0-rc.0
CHANGE BRANCH
x

coverage: 68.673% (+0.02%) from 68.652%
25802159849

push

github

web-flow 
fix(main): prevent orphaned thv serve when parent exits before SIGKILL timer (#2217)

* 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>

* fix(main): use exitCode/signalCode for liveness instead of .killed

Addresses Copilot review feedback on PR #2217.

`ChildProcess.killed` is only set after a successful `kill()` call - it
reflects "we sent a signal", not "the process has exited". After a
graceful SIGTERM the flag becomes true while the child is still running,
which caused the very fix this PR is for to short-circuit: the
parent-exit handler's `stopToolhive({ force: true })` returned early on
`toolhiveProcess.killed` and never escalated to SIGKILL. The mock test
deliberately did not auto-set `.killed`, so the regression was masked.

- isToolhiveRunning() now checks exitCode/signalCode for true liveness
- stopToolhive() early return uses the same liveness check
- exit handler clears the pending killTimer so a quick graceful exit
  does not leave a 2s setTimeout holding the event loop open
- MockProcess gains exitCode/signalCode (null defaults) to match the
  real ChildProcess API; regression test now sets mockProcess.killed
  after the first stop so the escalation path mirrors production

Co-Authored-By: Claude Opus... (continued)

4851 of 7622 branches covered (63.64%)

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

7234 of 10534 relevant lines covered (68.67%)

118.74 hits per line

Relevant lines Covered
10534 RELEVANT LINES 7234 COVERED LINES
118.74 HITS PER LINE
Source Files on v0.35.0-rc.0

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
25802159849 v0.35.0-rc.0 fix(main): prevent orphaned thv serve when parent exits before SIGKILL timer (#2217) * fix(main): prevent orphaned thv serve when parent exits before SIGKILL timer stopToolhive() nulled the module-level toolhiveProcess reference immediately afte... push web-flow github
68.67
See All Builds (2462)