poe-code 3.0.266 → 3.0.267

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "poe-code",
-  "version": "3.0.266",
+  "version": "3.0.267",
   "description": "CLI tool to configure Poe API for developer workflows.",
   "type": "module",
   "main": "./dist/index.js",

package/packages/agent-child-process/dist/index.d.ts

@@ -1,4 +1,4 @@
-import type { Readable, Writable } from "node:stream";
+import { type Readable, type Writable } from "node:stream";
 import type { SpawnResult, SpawnUsage } from "../../agent-spawn/dist/index.js";
 export type SpawnProcess = typeof import("node:child_process").spawn;
 export type AgentChildProcessKind = "exec" | "execFile" | "spawn";

package/packages/agent-child-process/dist/index.js

@@ -1,4 +1,5 @@
 import { spawn as defaultSpawn } from "node:child_process";
+import { PassThrough } from "node:stream";
 import { StringDecoder } from "node:string_decoder";
 export class AgentChildProcessError extends Error {
     result;
@@ -9,7 +10,10 @@ export class AgentChildProcessError extends Error {
     }
 }
 export async function exec(command, options) {
-    const shell = process.platform === "win32" ? process.env.ComSpec ?? "cmd.exe" : process.env.SHELL ?? "sh";
+    assertNonBlank(command, "command");
+    const shell = process.platform === "win32"
+        ? (options?.env?.ComSpec ?? process.env.ComSpec ?? "cmd.exe")
+        : (options?.env?.SHELL ?? process.env.SHELL ?? "sh");
     const shellArgs = process.platform === "win32" ? ["/d", "/s", "/c", command] : ["-c", command];
     return runExecution({
         kind: "exec",
@@ -21,6 +25,7 @@ export async function exec(command, options) {
     });
 }
 export async function execFile(file, argsOrOptions = [], maybeOptions) {
+    assertNonBlank(file, "file");
     const args = Array.isArray(argsOrOptions) ? argsOrOptions : [];
     const options = Array.isArray(argsOrOptions) ? maybeOptions : argsOrOptions;
     return runExecution({
@@ -33,10 +38,11 @@ export async function execFile(file, argsOrOptions = [], maybeOptions) {
     });
 }
 export function spawn(file, argsOrOptions = [], maybeOptions) {
-    const args = Array.isArray(argsOrOptions) ? argsOrOptions : [];
-    const options = Array.isArray(argsOrOptions) ? maybeOptions : argsOrOptions;
     let child;
     try {
+        assertNonBlank(file, "file");
+        const args = Array.isArray(argsOrOptions) ? argsOrOptions : [];
+        const options = Array.isArray(argsOrOptions) ? maybeOptions : argsOrOptions;
         child = startChildProcess({
             kind: "spawn",
             file,
@@ -59,8 +65,8 @@ export function spawn(file, argsOrOptions = [], maybeOptions) {
     return {
         pid: child.process.pid,
         stdin: child.process.stdin,
-        stdout: child.process.stdout,
-        stderr: child.process.stderr,
+        stdout: child.stdout,
+        stderr: child.stderr,
         kill(signal) {
             return child.process.kill(signal);
         },
@@ -72,8 +78,13 @@ async function runExecution(spec) {
 }
 function startChildProcess(spec) {
     const child = spawnChild(spec);
-    const result = collectResult(child, spec).then((commandResult) => applyExitPolicy(commandResult, spec.options));
-    return { process: child, result };
+    const stdout = createOutputPipes(child.stdout, spec.kind === "spawn");
+    const stderr = createOutputPipes(child.stderr, spec.kind === "spawn");
+    const result = collectResult(child, spec, {
+        stdout: stdout.result,
+        stderr: stderr.result
+    }).then((commandResult) => applyExitPolicy(commandResult, spec.options));
+    return { process: child, stdout: stdout.user, stderr: stderr.user, result };
 }
 function spawnChild(spec) {
     const options = spec.options;
@@ -89,22 +100,22 @@ function spawnChild(spec) {
 function getStdio(kind) {
     return kind === "spawn" ? ["pipe", "pipe", "pipe"] : ["ignore", "pipe", "pipe"];
 }
-function collectResult(child, spec) {
+function collectResult(child, spec, streams) {
     const stdoutDecoder = new StringDecoder("utf8");
     const stderrDecoder = new StringDecoder("utf8");
     let stdout = "";
     let stderr = "";
     let childClosed = false;
-    let stdoutFinished = child.stdout === null;
-    let stderrFinished = child.stderr === null;
+    let stdoutFinished = streams.stdout === null;
+    let stderrFinished = streams.stderr === null;
     let exitCode = 1;
     let exitSignal = null;
     let processError;
     let outputError;
-    child.stdout?.on("data", (chunk) => {
+    streams.stdout?.on("data", (chunk) => {
         stdout += typeof chunk === "string" ? chunk : stdoutDecoder.write(chunk);
     });
-    child.stderr?.on("data", (chunk) => {
+    streams.stderr?.on("data", (chunk) => {
         stderr += typeof chunk === "string" ? chunk : stderrDecoder.write(chunk);
     });
     return new Promise((resolve) => {
@@ -127,20 +138,20 @@ function collectResult(child, spec) {
             });
             resolve({ ...attempt, attempts: [attempt] });
         };
-        child.stdout?.once("end", () => {
+        streams.stdout?.once("end", () => {
             stdoutFinished = true;
             finish();
         });
-        child.stderr?.once("end", () => {
+        streams.stderr?.once("end", () => {
             stderrFinished = true;
             finish();
         });
-        child.stdout?.once("error", (error) => {
+        streams.stdout?.once("error", (error) => {
             outputError = error;
             stdoutFinished = true;
             finish();
         });
-        child.stderr?.once("error", (error) => {
+        streams.stderr?.once("error", (error) => {
             outputError = error;
             stderrFinished = true;
             finish();
@@ -161,6 +172,22 @@ function collectResult(child, spec) {
         });
     });
 }
+function createOutputPipes(source, exposeUserStream) {
+    if (source === null) {
+        return { result: null, user: null };
+    }
+    const result = new PassThrough();
+    const user = exposeUserStream ? new PassThrough() : null;
+    source.once("error", (error) => {
+        result.destroy(error);
+        user?.destroy();
+    });
+    source.pipe(result);
+    if (user !== null) {
+        source.pipe(user);
+    }
+    return { result, user };
+}
 function createAttempt(spec, output) {
     return {
         kind: spec.kind,
@@ -195,6 +222,14 @@ async function maybeRunAgent(result, options) {
             cause: error
         });
     }
+    try {
+        validateExitPolicy(policy);
+    }
+    catch (error) {
+        throw new AgentChildProcessError("Agent exit policy is invalid", result, {
+            cause: error
+        });
+    }
     const runAgent = options.runAgent ?? defaultRunAgent;
     let agentResult;
     try {
@@ -225,6 +260,15 @@ async function maybeRunAgent(result, options) {
         }
     };
 }
+function validateExitPolicy(policy) {
+    assertNonBlank(policy.agent, "onExit.agent");
+    assertNonBlank(policy.prompt, "onExit.prompt");
+}
+function assertNonBlank(value, field) {
+    if (value.trim().length === 0) {
+        throw new TypeError(`${field} must not be empty`);
+    }
+}
 async function defaultRunAgent(input) {
     const { spawn: spawnAgent } = await import("@poe-code/agent-spawn");
     return spawnAgent(input.agent, {