@ai-sdk/harness-acp 1.0.0 → 1.0.1

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # @ai-sdk/harness-acp
 
+## 1.0.1
+
+### Patch Changes
+
+- 6f54e1d: feat(harness-acp): further simplify public `createACP()` API
+- 15d0475: fix(harness-acp): defer auth resolution until session start to correctly recognize available credentials
+- faaabb0: fix(harness-acp): improve bridge error handling to prevent infinite hang on certain errors
+- Updated dependencies [81cd026]
+  - @ai-sdk/provider-utils@5.0.25
+  - @ai-sdk/harness@1.0.64
+
 ## 1.0.0
 
 ### Major Changes

package/README.md

@@ -27,14 +27,13 @@ import { createVercelSandbox } from '@ai-sdk/sandbox-vercel';
 
 const codexACP = createACP({
   harnessId: 'acp-codex',
-  implementation: {
-    type: 'npm',
-    mode: 'simple',
+  source: {
+    type: 'npm-simple',
     packageName: '@agentclientprotocol/codex-acp',
-    version: '1.1.4',
-    executable: 'codex-acp',
-    forwardEnv: ['CODEX_API_KEY', 'OPENAI_API_KEY'],
+    packageVersion: '1.1.4',
   },
+  executable: 'codex-acp',
+  forwardEnv: ['CODEX_API_KEY', 'OPENAI_API_KEY'],
   permissionModeMapping: {
     'allow-reads': null,
     'allow-edits': null,

package/dist/bridge/host-tool-mcp.mjs

@@ -12,7 +12,7 @@ import {
   ListToolsRequestSchema,
   McpError
 } from "@modelcontextprotocol/sdk/types.js";
-var VERSION = true ? "1.0.0" : "0.0.0-test";
+var VERSION = true ? "1.0.1" : "0.0.0-test";
 function createHostToolMCPServer({
   tools: tools2,
   revision = 1,

package/dist/bridge/index.mjs

@@ -820,10 +820,25 @@ function createACPBridgeError({
   stage,
   cause
 }) {
-  const error = new Error(`ACP ${stage} failed.`);
+  const causeMessage = getErrorMessage({ error: cause });
+  const error = new Error(
+    causeMessage == null ? `ACP ${stage} failed.` : `ACP ${stage} failed: ${causeMessage}`
+  );
   error.cause = cause;
   return error;
 }
+function getErrorMessage({ error }) {
+  if (error instanceof Error) {
+    return error.message;
+  }
+  if (error != null && typeof error === "object" && "message" in error && typeof error.message === "string") {
+    return error.message;
+  }
+  if (typeof error === "string") {
+    return error;
+  }
+  return void 0;
+}
 function stripMetadata({ value }) {
   if (Array.isArray(value)) {
     return value.map((item) => stripMetadata({ value: item }));
@@ -836,6 +851,42 @@ function stripMetadata({ value }) {
   return result;
 }
 
+// src/v1/bridge/agent-stderr-monitor.ts
+import { stripVTControlCharacters } from "util";
+function monitorACPAgentStderr({
+  stderr,
+  onStderrLine
+}) {
+  let rejectFailure;
+  const failure = new Promise((_, reject) => {
+    rejectFailure = reject;
+  });
+  void failure.catch(() => {
+  });
+  void (async () => {
+    stderr.setEncoding("utf8");
+    let pending = "";
+    for await (const chunk of stderr) {
+      pending += chunk;
+      const lines = pending.split("\n");
+      pending = lines.pop() ?? "";
+      for (const line of lines) handleStderrLine({ line });
+    }
+    if (pending.length > 0) handleStderrLine({ line: pending });
+  })().catch((error) => rejectFailure(error));
+  return failure;
+  function handleStderrLine({ line }) {
+    if (line.length === 0) return;
+    onStderrLine(line);
+    const normalizedLine = stripVTControlCharacters(line);
+    if (normalizedLine.toLowerCase().includes("failed to deserialize responsestreamevent from stream")) {
+      rejectFailure(
+        new Error("ACP agent failed to deserialize a streamed response.")
+      );
+    }
+  }
+}
+
 // src/v1/bridge/stream-translator.ts
 function createACPStreamTranslator({
   emit,
@@ -2713,6 +2764,7 @@ var implementation = await readImplementationDescriptor({
 });
 var bridgeConfiguration = await readACPBridgeEnvironment({ env: processEnv });
 var child;
+var agentResponseStreamFailure;
 var connection;
 var session;
 var recoveredSessionUpdates;
@@ -2767,6 +2819,12 @@ async function runTurn(start, turn) {
   if (activeSession == null) {
     throw new Error("ACP session initialization did not produce a session.");
   }
+  const activeAgentResponseStreamFailure = agentResponseStreamFailure;
+  if (activeAgentResponseStreamFailure == null) {
+    throw new Error(
+      "ACP session initialization did not start stderr monitoring."
+    );
+  }
   if (start.recoveryMode?.type === "lossy-rerun") {
     const marker = {
       type: "acp-recovery",
@@ -2883,7 +2941,8 @@ async function runTurn(start, turn) {
       try {
         message = await Promise.race([
           activeSession.nextUpdate(),
-          cancellationFailure
+          cancellationFailure,
+          activeAgentResponseStreamFailure
         ]);
       } catch (error) {
         for (const rawValue of streamCapture?.drainRawValues() ?? []) {
@@ -2977,7 +3036,16 @@ async function ensureSession({
       stdio: ["pipe", "pipe", "pipe"]
     }
   );
-  forwardAgentStderr({ child, turn });
+  agentResponseStreamFailure = monitorACPAgentStderr({
+    stderr: child.stderr,
+    onStderrLine: (line) => {
+      turn.bridgeLog({
+        level: "warn",
+        subsystem: "acp.agent.stderr",
+        message: line
+      });
+    }
+  });
   const input = Writable.toWeb(child.stdin);
   const output = Readable.toWeb(child.stdout);
   const capturedStream = captureACPStream({
@@ -3207,35 +3275,6 @@ function createChildEnvironment({
 function isRecord6(value) {
   return value != null && typeof value === "object" && !Array.isArray(value);
 }
-function forwardAgentStderr({
-  child: child2,
-  turn
-}) {
-  void (async () => {
-    child2.stderr.setEncoding("utf8");
-    let pending = "";
-    for await (const chunk of child2.stderr) {
-      pending += chunk;
-      const lines = pending.split("\n");
-      pending = lines.pop() ?? "";
-      for (const line of lines) {
-        if (line.length === 0) continue;
-        turn.bridgeLog({
-          level: "warn",
-          subsystem: "acp.agent.stderr",
-          message: line
-        });
-      }
-    }
-    if (pending.length > 0) {
-      turn.bridgeLog({
-        level: "warn",
-        subsystem: "acp.agent.stderr",
-        message: pending
-      });
-    }
-  })();
-}
 async function readImplementationDescriptor({
   path
 }) {