@mjasnikovs/pi-task 0.21.8 → 0.21.10

package/dist/shared/child-process.js

@@ -340,10 +340,29 @@ export function runChild(spawn, invocation, cwd, signal, opts) {
             streamWatch?.note();
             stderr += d.toString();
         });
-        proc.on('close', (code) => {
+        // One idempotent settle path for close/error/abort. Detaching the abort
+        // listener here is the point: `{once: true}` only fires-and-removes on an
+        // ACTUAL abort, so a child that finishes normally used to leave its
+        // listener on the signal forever. A TaskRunner shares ONE AbortController
+        // across every child of a run, so those listeners accumulated linearly and
+        // each one retained, via `killProc`'s closure, the finished child process,
+        // this invocation (including its full prompt), and `opts` — together with
+        // everything the caller's callbacks close over. See GitHub issue #9.
+        let settled = false;
+        const cleanup = () => {
             if (stallTimer)
                 clearInterval(stallTimer);
             streamWatch?.stop();
+            signal?.removeEventListener('abort', killProc);
+        };
+        const settle = (result) => {
+            cleanup();
+            if (settled)
+                return;
+            settled = true;
+            resolve(result);
+        };
+        proc.once('close', (code) => {
             // The child has exited, but anything it backgrounded (a dev server) may
             // still hold its process group and a port — reap the group so the next
             // gate's boot check does not collide with our own orphan. Best-effort:
@@ -355,7 +374,7 @@ export function runChild(spawn, invocation, cwd, signal, opts) {
             if (sink)
                 sink.flush();
             const text = sink ? sink.text : undefined;
-            resolve({
+            settle({
                 stdout,
                 stderr,
                 exitCode: code ?? 0,
@@ -368,11 +387,8 @@ export function runChild(spawn, invocation, cwd, signal, opts) {
                     : {})
             });
         });
-        proc.on('error', () => {
-            if (stallTimer)
-                clearInterval(stallTimer);
-            streamWatch?.stop();
-            resolve({ stdout, stderr, exitCode: 1, aborted });
+        proc.once('error', () => {
+            settle({ stdout, stderr, exitCode: 1, aborted });
         });
         if (signal) {
             if (signal.aborted)

package/dist/task/file-inventory.js

@@ -37,12 +37,31 @@ function runGitLsFiles(cwd, signal) {
         proc.stdout?.on('data', (d) => {
             stdout += d.toString();
         });
-        proc.on('error', () => resolve(''));
-        proc.on('close', code => resolve(code === 0 ? stdout : ''));
-        signal?.addEventListener('abort', () => {
+        // Same detach discipline as runChild (GitHub issue #9): `signal` is the
+        // run-long orchestrator signal, so a listener left behind by a normally
+        // finished `git ls-files` would retain that child for the whole run.
+        const onAbort = () => {
             if (!proc.killed)
                 proc.kill('SIGTERM');
-        });
+        };
+        let settled = false;
+        const settle = (value) => {
+            signal?.removeEventListener('abort', onAbort);
+            if (settled)
+                return;
+            settled = true;
+            resolve(value);
+        };
+        proc.once('error', () => settle(''));
+        proc.once('close', code => settle(code === 0 ? stdout : ''));
+        if (signal) {
+            // An already-aborted signal never emits 'abort', so without this check
+            // a run cancelled before this point would let the child run to term.
+            if (signal.aborted)
+                onAbort();
+            else
+                signal.addEventListener('abort', onAbort, { once: true });
+        }
     });
 }
 /** Cap output to maxLines real (non-blank) paths; tag truncation when cut. */

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.21.8",
+    "version": "0.21.10",
     "description": "Deterministic task planning and spec-orchestration for local models — crash-safe /task pipelines with verify/enforce gates, a real-time remote web view, and web/docs/fetch/worker subagent tools.",
     "type": "module",
     "main": "./dist/index.js",