fix: kill the whole process tree on Windows spawn-test timeout (#13) #50

Merged

the_og merged 1 commits from fix/13-windows-tree-kill into main

2026-07-12 13:31:15 -04:00

Author	SHA1	Message	Date
Cowork Supervisor	1384ed9703	fix: kill the whole process tree on Windows spawn-test timeout (#13 ) CI / Lint (ruff) (pull_request) Successful in 7s Details CI / Tests (py3.10 / ubuntu-latest) (pull_request) Successful in 8s Details CI / Tests (py3.12 / windows-latest) (pull_request) Successful in 20s Details CI / Tests (py3.12 / ubuntu-latest) (pull_request) Successful in 9s Details CI / Tests (py3.13 / ubuntu-latest) (pull_request) Successful in 8s Details Popen.kill() only terminated the direct child, so runner-style commands (npx -> node -> server) leaked the real server process on every Windows spawn test. taskkill /PID <pid> /T /F walks the descendant tree. Also sets CREATE_NO_WINDOW on the spawned test process so the windowed exe doesn't flash a console per test. Closes #13 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>	2026-07-12 13:30:02 -04:00