pullfrog 0.1.0 → 0.1.1

package/dist/cli.mjs

@@ -109979,6 +109979,7 @@ async function spawn(options) {
   const startTime = performance3.now();
   let stdoutBuffer = "";
   let stderrBuffer = "";
+  const killGroup = options.killGroup ?? false;
   return new Promise((resolve3, reject) => {
     const child = nodeSpawn(options.cmd, options.args, {
       env: options.env || {
@@ -109986,9 +109987,20 @@ async function spawn(options) {
         HOME: process.env.HOME || ""
       },
       stdio: options.stdio || ["pipe", "pipe", "pipe"],
-      cwd: options.cwd || process.cwd()
+      cwd: options.cwd || process.cwd(),
+      detached: killGroup
     });
-    trackChild({ child });
+    const killSelf = (signal) => {
+      if (killGroup && child.pid) {
+        try {
+          process.kill(-child.pid, signal);
+          return;
+        } catch {
+        }
+      }
+      child.kill(signal);
+    };
+    trackChild({ child, killGroup });
     let timeoutId;
     let sigkillEscalatorId;
     let activityCheckIntervalId;
@@ -109999,10 +110011,10 @@ async function spawn(options) {
     if (options.timeout) {
       timeoutId = setTimeout(() => {
         isTimedOut = true;
-        child.kill("SIGTERM");
+        killSelf("SIGTERM");
         sigkillEscalatorId = setTimeout(() => {
           if (!child.killed) {
-            child.kill("SIGKILL");
+            killSelf("SIGKILL");
           }
         }, 5e3);
       }, options.timeout);
@@ -110012,6 +110024,11 @@ async function spawn(options) {
         `spawn activity timer: pid=${child.pid} cmd=${options.cmd} timeout=${activityTimeoutMs}ms`
       );
       activityCheckIntervalId = setInterval(() => {
+        if (options.isPausedExternally?.()) {
+          lastActivityTime = performance3.now();
+          log.debug(`spawn activity check: pid=${child.pid} paused externally`);
+          return;
+        }
         const idleMs = performance3.now() - lastActivityTime;
         log.debug(
           `spawn activity check: pid=${child.pid} idle=${Math.round(idleMs)}ms / ${activityTimeoutMs}ms`
@@ -110021,9 +110038,9 @@ async function spawn(options) {
           killedAtIdleMs = idleMs;
           const idleSec = Math.round(idleMs / 1e3);
           log.info(
-            `no output for ${idleSec}s from pid=${child.pid} (${options.cmd}), killing process`
+            `no output for ${idleSec}s from pid=${child.pid} (${options.cmd}), killing process${killGroup ? " group" : ""}`
           );
-          child.kill("SIGKILL");
+          killSelf("SIGKILL");
           clearInterval(activityCheckIntervalId);
           try {
             options.onActivityTimeout?.();
@@ -142532,7 +142549,7 @@ var import_semver = __toESM(require_semver2(), 1);
 // package.json
 var package_default = {
   name: "pullfrog",
-  version: "0.1.0",
+  version: "0.1.1",
   type: "module",
   bin: {
     pullfrog: "dist/cli.mjs",
@@ -147467,6 +147484,12 @@ async function runClaude(params) {
       activityTimeout: 3e5,
       onActivityTimeout: params.onActivityTimeout,
       stdio: ["ignore", "pipe", "pipe"],
+      // run claude in its own process group so SIGKILL on activity timeout /
+      // outer cancellation reaches any subprocesses it spawns (rg, file
+      // watchers, mcp transports, etc). claude itself is a node bundle so
+      // there's no shim-orphan issue like opencode-ai/bin/opencode, but
+      // detached + killGroup is the right default for any agent runtime.
+      killGroup: true,
       onStdout: async (chunk) => {
         const text = chunk.toString();
         output += text;
@@ -147826,6 +147849,9 @@ async function runOpenCode(params) {
   const taskDispatchByCallID = /* @__PURE__ */ new Map();
   const pendingTaskDispatches = [];
   const knownNonTaskCallIDs = /* @__PURE__ */ new Set();
+  function isSubagentInFlight() {
+    return taskDispatchByCallID.size > 0 || pendingTaskDispatches.length > 0;
+  }
   function emitSubagentFinished(dispatch, status, output2, matchKind) {
     const subagentDuration = performance7.now() - dispatch.startedAt;
     const outputStr = typeof output2 === "string" ? output2 : "";
@@ -148078,6 +148104,20 @@ async function runOpenCode(params) {
       activityTimeout: 3e5,
       onActivityTimeout: params.onActivityTimeout,
       stdio: ["ignore", "pipe", "pipe"],
+      // node_modules/opencode-ai/bin/opencode is a Node shim that spawnSyncs
+      // the native opencode-<plat>-<arch> binary with stdio:"inherit". without
+      // a process-group kill, SIGKILL hits only the shim, the native binary
+      // is reparented to PID 1, holds our stdout pipe open, and `child.close`
+      // never fires — producing zombie runs. detached + killGroup nukes the
+      // whole tree.
+      killGroup: true,
+      // suspend the inner activity timer while a `task` subagent is in flight.
+      // opencode's task tool encapsulates subagent execution in-process — the
+      // subagent's internal events don't surface on the parent NDJSON stream,
+      // so without this the 5min timeout would falsely fire mid-subagent.
+      // suspend/resume is preferable to a heartbeat because there's no race
+      // between a periodic tick and a subagent finishing between ticks.
+      isPausedExternally: isSubagentInFlight,
       onStdout: async (chunk) => {
         const text = chunk.toString();
         output += text;
@@ -156036,7 +156076,7 @@ async function run2() {
 }
 
 // cli.ts
-var VERSION10 = "0.1.0";
+var VERSION10 = "0.1.1";
 var bin = basename2(process.argv[1] || "");
 var PROG = bin === "pf" || bin === "pullfrog" ? bin : "pullfrog";
 var rawArgs = process.argv.slice(2);

package/dist/index.js

@@ -109696,6 +109696,7 @@ async function spawn(options) {
   const startTime = performance3.now();
   let stdoutBuffer = "";
   let stderrBuffer = "";
+  const killGroup = options.killGroup ?? false;
   return new Promise((resolve3, reject) => {
     const child = nodeSpawn(options.cmd, options.args, {
       env: options.env || {
@@ -109703,9 +109704,20 @@ async function spawn(options) {
         HOME: process.env.HOME || ""
       },
       stdio: options.stdio || ["pipe", "pipe", "pipe"],
-      cwd: options.cwd || process.cwd()
+      cwd: options.cwd || process.cwd(),
+      detached: killGroup
     });
-    trackChild({ child });
+    const killSelf = (signal) => {
+      if (killGroup && child.pid) {
+        try {
+          process.kill(-child.pid, signal);
+          return;
+        } catch {
+        }
+      }
+      child.kill(signal);
+    };
+    trackChild({ child, killGroup });
     let timeoutId;
     let sigkillEscalatorId;
     let activityCheckIntervalId;
@@ -109716,10 +109728,10 @@ async function spawn(options) {
     if (options.timeout) {
       timeoutId = setTimeout(() => {
         isTimedOut = true;
-        child.kill("SIGTERM");
+        killSelf("SIGTERM");
         sigkillEscalatorId = setTimeout(() => {
           if (!child.killed) {
-            child.kill("SIGKILL");
+            killSelf("SIGKILL");
           }
         }, 5e3);
       }, options.timeout);
@@ -109729,6 +109741,11 @@ async function spawn(options) {
         `spawn activity timer: pid=${child.pid} cmd=${options.cmd} timeout=${activityTimeoutMs}ms`
       );
       activityCheckIntervalId = setInterval(() => {
+        if (options.isPausedExternally?.()) {
+          lastActivityTime = performance3.now();
+          log.debug(`spawn activity check: pid=${child.pid} paused externally`);
+          return;
+        }
         const idleMs = performance3.now() - lastActivityTime;
         log.debug(
           `spawn activity check: pid=${child.pid} idle=${Math.round(idleMs)}ms / ${activityTimeoutMs}ms`
@@ -109738,9 +109755,9 @@ async function spawn(options) {
           killedAtIdleMs = idleMs;
           const idleSec = Math.round(idleMs / 1e3);
           log.info(
-            `no output for ${idleSec}s from pid=${child.pid} (${options.cmd}), killing process`
+            `no output for ${idleSec}s from pid=${child.pid} (${options.cmd}), killing process${killGroup ? " group" : ""}`
           );
-          child.kill("SIGKILL");
+          killSelf("SIGKILL");
           clearInterval(activityCheckIntervalId);
           try {
             options.onActivityTimeout?.();
@@ -142249,7 +142266,7 @@ var import_semver = __toESM(require_semver2(), 1);
 // package.json
 var package_default = {
   name: "pullfrog",
-  version: "0.1.0",
+  version: "0.1.1",
   type: "module",
   bin: {
     pullfrog: "dist/cli.mjs",
@@ -147184,6 +147201,12 @@ async function runClaude(params) {
       activityTimeout: 3e5,
       onActivityTimeout: params.onActivityTimeout,
       stdio: ["ignore", "pipe", "pipe"],
+      // run claude in its own process group so SIGKILL on activity timeout /
+      // outer cancellation reaches any subprocesses it spawns (rg, file
+      // watchers, mcp transports, etc). claude itself is a node bundle so
+      // there's no shim-orphan issue like opencode-ai/bin/opencode, but
+      // detached + killGroup is the right default for any agent runtime.
+      killGroup: true,
       onStdout: async (chunk) => {
         const text = chunk.toString();
         output += text;
@@ -147543,6 +147566,9 @@ async function runOpenCode(params) {
   const taskDispatchByCallID = /* @__PURE__ */ new Map();
   const pendingTaskDispatches = [];
   const knownNonTaskCallIDs = /* @__PURE__ */ new Set();
+  function isSubagentInFlight() {
+    return taskDispatchByCallID.size > 0 || pendingTaskDispatches.length > 0;
+  }
   function emitSubagentFinished(dispatch, status, output2, matchKind) {
     const subagentDuration = performance7.now() - dispatch.startedAt;
     const outputStr = typeof output2 === "string" ? output2 : "";
@@ -147795,6 +147821,20 @@ async function runOpenCode(params) {
       activityTimeout: 3e5,
       onActivityTimeout: params.onActivityTimeout,
       stdio: ["ignore", "pipe", "pipe"],
+      // node_modules/opencode-ai/bin/opencode is a Node shim that spawnSyncs
+      // the native opencode-<plat>-<arch> binary with stdio:"inherit". without
+      // a process-group kill, SIGKILL hits only the shim, the native binary
+      // is reparented to PID 1, holds our stdout pipe open, and `child.close`
+      // never fires — producing zombie runs. detached + killGroup nukes the
+      // whole tree.
+      killGroup: true,
+      // suspend the inner activity timer while a `task` subagent is in flight.
+      // opencode's task tool encapsulates subagent execution in-process — the
+      // subagent's internal events don't surface on the parent NDJSON stream,
+      // so without this the 5min timeout would falsely fire mid-subagent.
+      // suspend/resume is preferable to a heartbeat because there's no race
+      // between a periodic tick and a subagent finishing between ticks.
+      isPausedExternally: isSubagentInFlight,
       onStdout: async (chunk) => {
         const text = chunk.toString();
         output += text;

package/dist/utils/subprocess.d.ts

@@ -26,6 +26,8 @@ export interface SpawnOptions {
     stdio?: ("pipe" | "ignore" | "inherit")[];
     onStdout?: (chunk: string) => void;
     onStderr?: (chunk: string) => void;
+    killGroup?: boolean;
+    isPausedExternally?: () => boolean;
 }
 export interface SpawnResult {
     stdout: string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pullfrog",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "type": "module",
   "bin": {
     "pullfrog": "dist/cli.mjs",