@1presence/bridge 0.33.0 → 0.34.0

package/dist/claude.js

@@ -111,9 +111,41 @@ function renderHistoryMessage(msg) {
 }
 // ─── Active processes ─────────────────────────────────────────────────────────
 const active = new Map();
+// Maximum automatic retries when the `claude` CLI exits non-zero BEFORE
+// producing any real output. This covers the known Claude Code print-mode 400
+// regression that surfaces as "API Error: 400 due to tool use concurrency
+// issues" (GitHub anthropics/claude-code#18131, still open) — it is
+// non-deterministic enough that a fresh spawn frequently succeeds. We retry
+// ONLY when the failed attempt produced no real assistant text and no tool
+// calls, so a failure that lands after real work (where retrying could
+// double-execute a side-effectful tool) is surfaced, never silently re-run.
+const MAX_TURN_RETRIES = 1;
+// Map a non-zero CLI exit + any captured "API Error:" line to a concise,
+// user-facing Local Mode message. The raw upstream text stays in operator logs
+// only — we never echo a wall of provider error JSON into the chat. Referring
+// to "Claude Code" here is intentional and consistent with Local Mode's other
+// operational errors (e.g. the "claude CLI not found" message): in Local Mode
+// the user is knowingly running their own Claude Code install.
+//
+// NOTE on the 400 tool-use case: this is an open Claude Code print-mode
+// regression (introduced in 2.1.19, still present in 2.1.146 — the current
+// latest), so upgrading does NOT fix it. We deliberately do not suggest an
+// upgrade; the automatic retry is the real mitigation and resending sometimes
+// gets through.
+function describeCliFailure(code, apiErrorText) {
+    const t = apiErrorText.trim();
+    if (/API Error:\s*400/i.test(t) && /(tool use|concurren|parallel)/i.test(t)) {
+        return 'Local Mode hit a known Claude Code error (a print-mode bug that affects every current version). I retried once automatically — sending the message again sometimes gets through.';
+    }
+    if (/^API Error:/i.test(t)) {
+        return `Local Mode error from Claude Code: ${t.replace(/^API Error:\s*/i, '').trim()}`;
+    }
+    return `Local Mode stopped unexpectedly (claude exited with code ${code ?? 'unknown'}). Please try again.`;
+}
 // ─── Spawn ────────────────────────────────────────────────────────────────────
 function spawnClaude(params) {
     const { conversationId, presenceSessionId, text, uid, history, vaultFileOpen, clientCapabilities, syncedFolders, onEvent, onDone, onError } = params;
+    const attemptIdx = params._attemptIdx ?? 0;
     const systemPromptPath = (0, path_1.join)((0, os_1.tmpdir)(), `agent-${uid}.md`);
     const mcpConfigPath = (0, path_1.join)((0, os_1.tmpdir)(), `mcp-${uid}.json`);
     if (verbose) {
@@ -125,6 +157,13 @@ function spawnClaude(params) {
         process.stderr.write(paint('90', `[bridge:verbose] conversation:  ${conversationId}`) + '\n');
         process.stderr.write(paint('90', `[bridge:verbose] history turns: ${history.length}`) + '\n');
     }
+    // Surface the user's UID before the session line in every mode — it's the
+    // Firestore doc prefix (`sessions/<uid>_<conversationId>`), so logging it
+    // makes a reported bridge failure correlatable to the stored session without
+    // having to ask which account hit it. The CLI's own `--session-id` is
+    // ephemeral and is NOT the Firestore conversationId, so the uid is the key
+    // join column when debugging.
+    process.stderr.write(`[bridge] user ${uid}\n`);
     // Debug transcript: lead with the user prompt for this turn (the clean
     // message, before the gateway's ephemeral-context prefix), plus the session
     // id (correlates with the chat URL / Firestore session doc) and a hint at
@@ -198,6 +237,10 @@ function spawnClaude(params) {
     //   across turns of a chat — even with --no-session-persistence. The
     //   bridge passes the per-spawn `conversationId` here; the presence
     //   sessionId is correlated separately via bridge logs and spool records.
+    // The CLI treats --session-id as "claim this new session ID" and rejects a
+    // reused id with "Session ID X is already in use". A retry is a fresh spawn,
+    // so it MUST use a new uuid; the first attempt keeps the correlation id.
+    const spawnSessionId = attemptIdx === 0 ? presenceSessionId : crypto.randomUUID();
     const args = [
         '--print',
         '--input-format', 'stream-json',
@@ -210,7 +253,7 @@ function spawnClaude(params) {
         '--mcp-config', mcpConfigPath,
         '--strict-mcp-config',
         '--no-session-persistence',
-        '--session-id', presenceSessionId,
+        '--session-id', spawnSessionId,
     ];
     const pinnedModel = (0, config_1.getBridgeModel)();
     if (pinnedModel) {
@@ -266,6 +309,15 @@ function spawnClaude(params) {
     let extractedModel = null;
     let buffer = '';
     let killedForViolation = false;
+    // Retry/error-surfacing tracking for this attempt:
+    //  - sawApiError: the CLI emitted an "API Error:" assistant text event (the
+    //    way Claude Code reports an underlying API failure mid-turn).
+    //  - apiErrorText: that text, captured for describeCliFailure().
+    //  - producedRealOutput: any real assistant text or tool_use was emitted, so
+    //    a later failure must NOT be retried (could double-run a side-effect).
+    let sawApiError = false;
+    let apiErrorText = '';
+    let producedRealOutput = false;
     proc.stdout.on('data', (chunk) => {
         buffer += chunk.toString('utf-8');
         const lines = buffer.split('\n');
@@ -282,6 +334,10 @@ function spawnClaude(params) {
                 continue;
             }
             const type = event['type'];
+            // Set when this event is the CLI's "API Error:" turn — we neither forward
+            // it to the PWA nor let it reach the accumulator (it carries no real
+            // content and would poison history / show a raw error mid-stream).
+            let suppressEvent = false;
             // Extract model + key source info from the first system/init event.
             // No session-id persistence — Firestore is the only source of truth
             // now, and we pin --session-id to presenceSessionId on every spawn.
@@ -322,6 +378,7 @@ function spawnClaude(params) {
                     let wroteText = false;
                     for (const block of content) {
                         if (block['type'] === 'tool_use') {
+                            producedRealOutput = true;
                             const toolName = block['name'];
                             const toolId = block['id'];
                             if (toolId)
@@ -370,13 +427,27 @@ function spawnClaude(params) {
                         else if (block['type'] === 'text') {
                             const text = block['text'];
                             if (text) {
-                                if (debug) {
-                                    // Full text, newlines intact — the readable transcript.
-                                    debugBlock('assistant', exports.SECTION_COLORS.assistant, text);
+                                if (/^API Error:/i.test(text.trimStart())) {
+                                    // The CLI is reporting an underlying API failure as assistant
+                                    // text. Capture it for the user-facing message, and suppress
+                                    // the whole event so the raw error never reaches the PWA or
+                                    // the accumulator (the gateway also blanks it via
+                                    // cleanTurnText — this is the upstream defense).
+                                    sawApiError = true;
+                                    apiErrorText = text.trim();
+                                    suppressEvent = true;
+                                    process.stderr.write(paint(exports.SECTION_COLORS.result, `[bridge] ${text.replace(/\n+/g, ' ')}`) + '\n');
                                 }
                                 else {
-                                    process.stderr.write(paint(exports.SECTION_COLORS.assistant, text.replace(/\n+/g, ' ')));
-                                    wroteText = true;
+                                    producedRealOutput = true;
+                                    if (debug) {
+                                        // Full text, newlines intact — the readable transcript.
+                                        debugBlock('assistant', exports.SECTION_COLORS.assistant, text);
+                                    }
+                                    else {
+                                        process.stderr.write(paint(exports.SECTION_COLORS.assistant, text.replace(/\n+/g, ' ')));
+                                        wroteText = true;
+                                    }
                                 }
                             }
                         }
@@ -412,7 +483,8 @@ function spawnClaude(params) {
                 if (typeof c === 'number')
                     costUsd = c;
             }
-            onEvent(event);
+            if (!suppressEvent)
+                onEvent(event);
         }
     });
     proc.stderr.on('data', (chunk) => {
@@ -434,9 +506,19 @@ function spawnClaude(params) {
             catch { /* ignore */ }
         }
         if (code !== 0 && code !== null) {
+            // Auto-retry once when the CLI failed BEFORE producing any real output —
+            // the signature of the known print-mode 400 regression. A fresh spawn
+            // (new --session-id) usually succeeds. We never retry once real text or a
+            // tool call landed, to avoid double-running a side-effectful tool.
+            if (attemptIdx < MAX_TURN_RETRIES && sawApiError && !producedRealOutput) {
+                process.stderr.write(`[bridge] turn failed before output (${apiErrorText.replace(/\n+/g, ' ').slice(0, 120)}) — retrying once\n`);
+                spawnClaude({ ...params, _attemptIdx: attemptIdx + 1 });
+                return;
+            }
             // Pass any partial token usage we observed before the failure so the
-            // PWA and the gateway's bridge usage store can still record it.
-            onError(`claude exited with code ${code}`, usage, extractedModel);
+            // PWA and the gateway's bridge usage store can still record it. Surface a
+            // classified, user-readable message instead of the opaque exit code.
+            onError(describeCliFailure(code, apiErrorText), usage, extractedModel);
         }
         else {
             onDone(messageCount, costUsd, usage, extractedModel);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@1presence/bridge",
-  "version": "0.33.0",
+  "version": "0.34.0",
   "description": "Run 1Presence on your Mac and use your Claude.ai Pro subscription from any device",
   "bin": {
     "1presence-bridge": "dist/index.js"