@nwire/supervisor 0.12.1 → 0.13.1

package/dist/supervisor.js

@@ -109,9 +109,28 @@ export class RunnerSupervisor extends EventEmitter {
         };
         child.stdout?.on("data", capture("stdout"));
         child.stderr?.on("data", capture("stderr"));
+        // A supervised child must never keep the supervisor's own process alive:
+        // unref the child handle and its stdio pipes so an in-flight or
+        // not-yet-stopped child can't block the parent (or a test runner's
+        // worker) from exiting. Data is still captured — unref only detaches the
+        // handles from the event-loop keep-alive set, it doesn't stop them.
+        child.unref();
+        // The piped stdio streams are Sockets at runtime (they carry `.unref`)
+        // but typed as the base `Readable`, which doesn't declare it.
+        const unref = (s) => {
+            s?.unref?.();
+        };
+        unref(child.stdout);
+        unref(child.stderr);
         child.on("exit", (code, signal) => {
             proc.exitCode = code;
             proc.signal = signal;
+            // Release the child's stdout/stderr pipes. Node keeps the underlying
+            // PipeWrap handles alive until the readable side is destroyed; without
+            // this, an exited child leaves open handles that keep the parent (or a
+            // test runner's worker) from exiting cleanly.
+            child.stdout?.destroy();
+            child.stderr?.destroy();
             // Distinguish clean stop (we asked for it) from a crash.
             if (proc.status === "stopping") {
                 proc.status = "exited";
@@ -193,8 +212,16 @@ export class RunnerSupervisor extends EventEmitter {
         this.emit("status", id, "stopping", state.proc);
         state.child.kill("SIGTERM");
         const exited = new Promise((resolve) => state.child.once("exit", () => resolve()));
-        const grace = new Promise((resolve) => setTimeout(() => resolve("escalate"), graceMs));
+        // Hold the timer handle so we can clear it the moment the child exits —
+        // an uncleared grace timer is a ref'd setTimeout that keeps the process
+        // (and a test runner's worker) alive for the full graceMs window.
+        let graceTimer;
+        const grace = new Promise((resolve) => {
+            graceTimer = setTimeout(() => resolve("escalate"), graceMs);
+        });
         const result = await Promise.race([exited.then(() => "exited"), grace]);
+        if (graceTimer)
+            clearTimeout(graceTimer);
         if (result === "escalate") {
             try {
                 // Windows collapses SIGTERM and SIGKILL into the same hard

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nwire/supervisor",
-  "version": "0.12.1",
+  "version": "0.13.1",
   "private": false,
   "description": "Spawns and watches Nwire wire processes for dev tooling. Used by `nwire dev` and Studio's vite dev server. Generic process supervisor with stdout/stderr ring buffer, health-check polling, SIGTERM-then-SIGKILL graceful stop.",
   "keywords": [