next-issue 0.12.0 → 0.12.1

package/README.md

@@ -85,9 +85,10 @@ only between two issues, finishes the issue that it holds, deletes the file and
 then writes the summary as usual. A file from an earlier run is dropped at the
 start, so it stops nothing.
 
-`Ctrl-C` is different: the terminal signals every child too, so the run dies in
-the middle of a step. The saved state lets the next run continue the issue, but
-the issue keeps `status:in-progress` and the run writes no summary.
+`Ctrl-C` is different: the harness passes the signal on to the command that runs
+and to every child of it, so the run dies in the middle of a step. The saved
+state lets the next run continue the issue, but the issue keeps
+`status:in-progress` and the run writes no summary.
 
 Every status label that the harness puts on the issue goes on the pull request
 too, from the first review round. Thus a draft pull request shows work that the

package/dist/exec.mjs

@@ -2,6 +2,41 @@ import { spawn } from "node:child_process";
 let observer;
 let defaultTimeoutMs = 10 * 60_000;
 const STDERR_TAIL = 200_000;
+const groups = new Set();
+function forward(signal) {
+    for (const pid of groups) {
+        try {
+            process.kill(-pid, signal);
+        }
+        catch { }
+    }
+    groups.clear();
+    process.off("SIGINT", onInterrupt);
+    process.off("SIGTERM", onTerminate);
+    process.kill(process.pid, signal);
+}
+const onInterrupt = () => forward("SIGINT");
+const onTerminate = () => forward("SIGTERM");
+function watch(pid) {
+    if (pid === undefined) {
+        return;
+    }
+    if (groups.size === 0) {
+        process.on("SIGINT", onInterrupt);
+        process.on("SIGTERM", onTerminate);
+    }
+    groups.add(pid);
+}
+function unwatch(pid) {
+    if (pid === undefined) {
+        return;
+    }
+    groups.delete(pid);
+    if (groups.size === 0) {
+        process.off("SIGINT", onInterrupt);
+        process.off("SIGTERM", onTerminate);
+    }
+}
 export function setCommandObserver(next) {
     observer = next;
 }
@@ -14,12 +49,15 @@ export function run(command, args, options = {}) {
     return new Promise((resolve, reject) => {
         const child = spawn(command, args, {
             cwd: options.cwd,
+            detached: true,
             stdio: [
                 options.input === undefined ? "ignore" : "pipe",
                 options.show === true ? 2 : "pipe",
                 "pipe",
             ],
         });
+        const pid = child.pid;
+        watch(pid);
         let stdout = "";
         let stderr = "";
         let timedOut = false;
@@ -40,11 +78,21 @@ export function run(command, args, options = {}) {
         });
         const timer = setTimeout(() => {
             timedOut = true;
-            child.kill("SIGKILL");
+            if (pid === undefined) {
+                child.kill("SIGKILL");
+                return;
+            }
+            try {
+                process.kill(-pid, "SIGKILL");
+            }
+            catch {
+                child.kill("SIGKILL");
+            }
         }, limit);
         let done = false;
         const settle = (code) => {
             clearTimeout(timer);
+            unwatch(pid);
             if (done) {
                 return;
             }
@@ -63,6 +111,7 @@ export function run(command, args, options = {}) {
         };
         child.on("error", (error) => {
             clearTimeout(timer);
+            unwatch(pid);
             done = true;
             reject(error);
         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "next-issue",
-  "version": "0.12.0",
+  "version": "0.12.1",
   "description": "Autonomous issue harness driven by the Claude Agent SDK",
   "keywords": [
     "github",