@zvada/agent-server 0.3.1 → 0.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@zvada/agent-server",
-	"version": "0.3.1",
+	"version": "0.3.2",
 	"description": "Harness-agnostic agent execution engine: run Claude Code, Codex (SDK/CLI + app-server), and any ACP agent behind one interface with a normalized event stream, multi-turn sessions, and resume. Root export is the wire contract; /core, /server, /client are the seats.",
 	"license": "MIT",
 	"type": "module",
@@ -25,7 +25,13 @@
 	"engines": {
 		"bun": ">=1.2.0"
 	},
-	"files": ["src", "docs", "README.md", "CHANGELOG.md", "LICENSE"],
+	"files": [
+		"src",
+		"docs",
+		"README.md",
+		"CHANGELOG.md",
+		"LICENSE"
+	],
 	"publishConfig": {
 		"access": "public"
 	},

package/src/core/agents/codex-app-server/client.ts

@@ -1,6 +1,9 @@
 import { type ChildProcess, spawn as nodeSpawn } from "node:child_process";
 import { StringDecoder } from "node:string_decoder";
 
+/** Cap the retained stderr tail — enough for the CLI's error, never a log flood. */
+const STDERR_TAIL_MAX = 2048;
+
 /** A JSON-RPC notification (server → client, no id). */
 export interface CodexNotification {
 	method: string;
@@ -54,6 +57,11 @@ export class CodexAppServerClient {
 	private readonly closeHandlers = new Set<(error?: Error) => void>();
 	private requestHandler?: ServerRequestHandler;
 	private exited = false;
+	/** Bounded rolling tail of the subprocess's stderr — folded into the exit
+	 *  error so failures carry the CLI's own explanation. */
+	private stderrTail = "";
+	/** Chunk-boundary-safe UTF-8 decoding for the stderr tail; reset per spawn. */
+	private stderrDecoder = new StringDecoder("utf8");
 
 	constructor(private readonly opts: CodexAppServerClientOptions = {}) {}
 
@@ -140,14 +148,32 @@ export class CodexAppServerClient {
 			stdio: ["pipe", "pipe", "pipe"],
 		});
 		this.proc = proc;
+		this.stderrTail = "";
+		this.stderrDecoder = new StringDecoder("utf8");
 		proc.stdout?.on("data", (chunk: Buffer) => this.onStdout(chunk));
 		proc.stdin?.on("error", (err) => this.onExit(err));
 		proc.stdout?.on("error", (err) => this.onExit(err));
-		proc.stderr?.on("data", () => {});
+		// Keep a bounded tail of stderr: it is the ONLY place the CLI explains
+		// itself (a missing/expired login prints there and exits 1). Discarding
+		// it reduced every failure to "exited (code=1)" — indistinguishable
+		// from a crash for the integrator staring at the turn error. Decoded
+		// via StringDecoder so a UTF-8 sequence split across chunks survives.
+		proc.stderr?.on("data", (chunk: Buffer) => {
+			this.stderrTail = (this.stderrTail + this.stderrDecoder.write(chunk)).slice(-STDERR_TAIL_MAX);
+		});
 		proc.on("error", (err) => this.onExit(err));
-		proc.on("exit", (code, signal) =>
-			this.onExit(new Error(`codex app-server exited (code=${code} signal=${signal})`)),
-		);
+		// Build the exit error on CLOSE, not exit: `exit` can fire before the
+		// stderr stream flushes its final data, and the final flush is usually
+		// the line that says WHY. `close` is guaranteed after both.
+		proc.on("close", (code, signal) => {
+			this.stderrTail = (this.stderrTail + this.stderrDecoder.end()).slice(-STDERR_TAIL_MAX);
+			const tail = this.stderrTail.trim().replace(/\s+/g, " ");
+			this.onExit(
+				new Error(
+					`codex app-server exited (code=${code} signal=${signal})${tail ? `: ${tail}` : ""}`,
+				),
+			);
+		});
 	}
 
 	private onStdout(chunk: Buffer): void {