palmier 1.1.6 → 1.1.7

package/dist/rpc-handler.js

@@ -4,7 +4,7 @@ import * as path from "path";
 import { listTasks, parseTaskFile, writeTaskFile, getTaskDir, readTaskStatus, writeTaskStatus, readHistory, deleteHistoryEntry, appendTaskList, removeFromTaskList, isTaskInList, appendHistory, createRunDir, appendRunMessage, getRunDir, writeFollowupStatus, readFollowupStatus, deleteFollowupStatus, isRunStarred, setRunStarred } from "./task.js";
 import { resolvePending, getPending, listPending } from "./pending-requests.js";
 import { getPlatform } from "./platform/index.js";
-import { resolveCommand, spawnCommand } from "./spawn-command.js";
+import { onChildDone, resolveCommand, spawnCommand } from "./spawn-command.js";
 import crossSpawn from "cross-spawn";
 import { getAgent, getPromptCommandLine, getNpmInstalledVersion } from "./agents/agent.js";
 import { validateClient, revokeClient } from "./client-store.js";
@@ -419,7 +419,7 @@ export function createRpcHandler(config, nc) {
                 const chunks = [];
                 child.stdout?.on("data", (d) => chunks.push(d));
                 child.stderr?.on("data", (d) => process.stderr.write(d));
-                child.on("close", async (code) => {
+                onChildDone(child, async (code) => {
                     activeFollowups.delete(followupKey);
                     deleteFollowupStatus(followupRunDir);
                     // stop_followup already wrote the stopped status.

package/dist/spawn-command.d.ts

@@ -8,6 +8,12 @@ export declare function resolveCommand(command: string, args: string[]): {
     command: string;
     args: string[];
 };
+/**
+ * Like the "close" event, but doesn't wait for grandchildren: a background process
+ * the child started (e.g. a browser daemon) can inherit its stdio pipes on Windows
+ * and hold them open long after the child itself exits.
+ */
+export declare function onChildDone(child: ChildProcess, callback: (code: number | null) => void): void;
 export interface SpawnStreamingOptions {
     cwd: string;
     env?: Record<string, string>;

package/dist/spawn-command.js

@@ -50,6 +50,32 @@ function treeKill(child) {
     }
     child.kill();
 }
+const STDIO_DRAIN_GRACE_MS = 1000;
+/**
+ * Like the "close" event, but doesn't wait for grandchildren: a background process
+ * the child started (e.g. a browser daemon) can inherit its stdio pipes on Windows
+ * and hold them open long after the child itself exits.
+ */
+export function onChildDone(child, callback) {
+    let done = false;
+    let grace;
+    const finish = (code) => {
+        if (done)
+            return;
+        done = true;
+        if (grace)
+            clearTimeout(grace);
+        callback(code);
+    };
+    child.on("exit", (code) => {
+        grace = setTimeout(() => {
+            child.stdout?.destroy();
+            child.stderr?.destroy();
+            finish(code);
+        }, STDIO_DRAIN_GRACE_MS);
+    });
+    child.on("close", finish);
+}
 /**
  * Spawn with shell interpretation for piped commands like "tail -f log | grep".
  * Returns the ChildProcess with stdout piped so the caller can read it directly
@@ -104,7 +130,7 @@ export function spawnCommand(command, args, opts) {
                 reject(new Error("command timed out"));
             }, opts.timeout);
         }
-        child.on("close", (code) => {
+        onChildDone(child, (code) => {
             if (timer)
                 clearTimeout(timer);
             const output = Buffer.concat(chunks).toString("utf-8");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "palmier",
-  "version": "1.1.6",
+  "version": "1.1.7",
   "description": "Palmier host CLI - provisions, executes tasks, and serves NATS RPC",
   "license": "Apache-2.0",
   "author": "Hongxu Cai",