pi-goosedump 0.12.68 → 0.12.69

package/dist/index.ts

@@ -7,7 +7,7 @@ import { randomUUID as randomUUID2 } from "node:crypto";
 // src/cli.ts
 import { Type as Type2 } from "typebox";
 import { Value as Value2 } from "typebox/value";
-import { execFile } from "node:child_process";
+import { spawn } from "node:child_process";
 import { accessSync, constants as fsConstants, existsSync, readFileSync as readFileSync2, statSync } from "node:fs";
 import { createRequire } from "node:module";
 import { dirname, isAbsolute, join } from "node:path";
@@ -513,18 +513,66 @@ function resolveNativeGoosedumpBinary(environment = process.env) {
   }
 }
 function runGoosedumpAsync(args, signal, executablePath = resolveNativeGoosedumpBinary()) {
+  if (signal?.aborted)
+    return Promise.reject(signal.reason);
   return new Promise((resolve, reject) => {
-    execFile(executablePath, args, {
-      encoding: "utf-8",
-      maxBuffer: 16 * 1024 * 1024,
-      signal,
-      ...process.platform === "linux" ? { env: { ...process.env, APPIMAGE_EXTRACT_AND_RUN: "1" } } : {}
-    }, (err, stdout, stderr) => {
-      if (err) {
-        const message = stderr.trim() || err.message;
-        reject(new Error(message));
+    const processGroup = process.platform !== "win32";
+    const output = { stdout: "", stderr: "" };
+    const bytes = { stdout: 0, stderr: 0 };
+    let child;
+    let failure;
+    const abort = () => {
+      if (child?.pid === undefined)
+        return;
+      try {
+        if (processGroup)
+          process.kill(-child.pid, "SIGTERM");
+        else
+          child.kill("SIGTERM");
+      } catch (error) {
+        if (error.code !== "ESRCH") {
+          failure = new Error("failed to stop goosedump process", { cause: error });
+        }
+      }
+    };
+    const capture = (stream, chunk) => {
+      if (failure)
+        return;
+      bytes[stream] += Buffer.byteLength(chunk);
+      if (bytes[stream] > 16 * 1024 * 1024) {
+        failure = new Error(`goosedump ${stream} exceeded the 16 MiB buffer limit`);
+        abort();
+      } else {
+        output[stream] += chunk;
+      }
+    };
+    signal?.addEventListener("abort", abort, { once: true });
+    try {
+      child = spawn(executablePath, args, {
+        detached: processGroup,
+        signal,
+        stdio: ["ignore", "pipe", "pipe"],
+        ...process.platform === "linux" ? { env: { ...process.env, APPIMAGE_EXTRACT_AND_RUN: "1" } } : {}
+      });
+    } catch (error) {
+      signal?.removeEventListener("abort", abort);
+      reject(error);
+      return;
+    }
+    child.stdout?.setEncoding("utf8");
+    child.stderr?.setEncoding("utf8");
+    child.stdout?.on("data", (chunk) => capture("stdout", chunk));
+    child.stderr?.on("data", (chunk) => capture("stderr", chunk));
+    child.on("error", (error) => {
+      failure ??= error;
+    });
+    child.once("close", (code, exitSignal) => {
+      signal?.removeEventListener("abort", abort);
+      if (failure || code !== 0) {
+        const message = output.stderr.trim() || failure?.message || `goosedump exited with ${exitSignal ?? `code ${code}`}`;
+        reject(new Error(message, { cause: failure }));
       } else {
-        resolve({ stdout, stderr });
+        resolve(output);
       }
     });
   });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goosedump",
-  "version": "0.12.68",
+  "version": "0.12.69",
   "description": "Pi extension for goosedump session search, durable memory, session management, and compaction",
   "keywords": [
     "compaction",
@@ -37,7 +37,7 @@
     "ci:test": "npm test"
   },
   "dependencies": {
-    "@jarkkojs/goosedump-api": "^0.12.68"
+    "@jarkkojs/goosedump-api": "^0.12.69"
   },
   "devDependencies": {
     "@earendil-works/pi-coding-agent": "^0.84.2",