@chatpanel/bridge 0.10.40 → 0.10.41

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chatpanel/bridge",
-  "version": "0.10.40",
+  "version": "0.10.41",
   "type": "module",
   "description": "Local bridge that exposes the AI coding agents installed on your machine \u2014 Claude Code (CLI), Codex (CLI), and Antigravity CLI (formerly Gemini CLI, which remains available for business/enterprise) \u2014 to the ChatPanel Chrome extension over a localhost SSE endpoint. Bring your own agent.",
   "keywords": [

package/src/cli-errors.js

@@ -0,0 +1,64 @@
+// Turning a CLI agent's dying breath into something a person can act on.
+//
+// When an agent exits non-zero we used to surface its raw stderr. For a crash that is fine;
+// for the common real failure — one of the AGENT'S OWN MCP servers refusing to authenticate —
+// it is not: those servers answer with an HTML error page, so the user got kilobytes of
+// markup, inline CSS and an SVG logo in the chat, with the one useful sentence buried inside.
+//
+// The failure is also usually not ChatPanel's to fix. Saying whose it is, and naming the
+// server, is the difference between "something broke" and "re-auth that server".
+
+const NOISE = [
+  /^\s*$/,
+  /^\s*at\s+/,            // stack frames
+  /^\s*[.#]?[\w-]+\s*\{/, // CSS rules
+  /^\s*[\w-]+:\s*[^;]+;$/,// CSS declarations
+  /^\s*<\//,              // closing tags
+];
+
+// Strip HTML/CSS/SVG so an error page collapses to whatever prose it contained.
+export function stripMarkup(text) {
+  return String(text || '')
+    .replace(/<script[\s\S]*?<\/script>/gi, ' ')
+    .replace(/<style[\s\S]*?<\/style>/gi, ' ')
+    .replace(/<svg[\s\S]*?<\/svg>/gi, ' ')
+    .replace(/<[^>]+>/g, ' ')
+    .replace(/&nbsp;/g, ' ')
+    .replace(/[ \t]+/g, ' ');
+}
+
+// The named causes worth translating. Each returns a sentence that says what to DO.
+function knownCause(text) {
+  const server = /refresh OAuth tokens for server ([\w.-]+)/i.exec(text)?.[1]
+    || /server ['"]?([\w.-]+)['"]? (?:requires|failed) auth/i.exec(text)?.[1];
+  if (/refresh token (?:does not exist|was rejected)|invalid_grant/i.test(text)) {
+    return `${server ? `its MCP server "${server}"` : 'one of its MCP servers'} needs re-authentication — its saved OAuth token has expired or been revoked. Re-login to that server in the agent's own config; ChatPanel can't refresh it.`;
+  }
+  if (/invalid_token|AuthRequired|www-authenticate|\b401\b/i.test(text)) {
+    return `${server ? `its MCP server "${server}"` : 'one of its MCP servers'} rejected the agent's token (401/invalid_token). Re-authenticate that server in the agent, then retry.`;
+  }
+  if (/HTTP 403|\b403\b/.test(text)) {
+    return `${server ? `its MCP server "${server}"` : 'one of its MCP servers'} returned 403 and an error page — that server is refusing requests or is temporarily down. This is outside ChatPanel; retry when it recovers.`;
+  }
+  if (/ENOENT|command not found/i.test(text)) return 'the command could not be found on PATH.';
+  return null;
+}
+
+/**
+ * A short, actionable summary of why a CLI agent exited. Never returns markup, and never more
+ * than a couple of lines — the full output stays in the bridge log for anyone debugging.
+ */
+export function summarizeCliError(label, code, stderr, stdout = '') {
+  const raw = `${stderr || ''}\n${stdout || ''}`;
+  const cause = knownCause(raw);
+  if (cause) return `${label} exited ${code}: ${cause}`;
+
+  const lines = stripMarkup(raw)
+    .split('\n')
+    .map((l) => l.trim())
+    .filter((l) => l && !NOISE.some((re) => re.test(l)));
+  // Prefer the last line that reads like an error, else the last line at all.
+  const errish = lines.filter((l) => /error|fail|fatal|panic|refused|denied|timeout/i.test(l));
+  const pick = (errish.length ? errish : lines).pop() || 'failed';
+  return `${label} exited ${code}: ${pick.length > 300 ? `${pick.slice(0, 300)}…` : pick}`;
+}

package/src/engines/antigravity.js

@@ -16,6 +16,7 @@ import os from 'node:os';
 import path from 'node:path';
 import { findAgentBin } from '../env.js';
 import { buildCliPrompt } from './prompt.js';
+import { summarizeCliError } from '../cli-errors.js';
 import { killOnAbort, spawnGroupOpts } from '../proc.js';
 import { pushExtraArgs, FORBIDDEN } from './args.js';
 import { resolveWorkdir } from '../workdir.js';
@@ -153,7 +154,7 @@ export async function chat({ messages, system, options, images }, emit, { signal
         emit({ type: 'done', text: '' });
         resolve();
       } else {
-        reject(new Error(`Antigravity exited ${code}: ${err.trim() || out.trim() || 'failed'}`));
+        reject(new Error(summarizeCliError('Antigravity', code, err, out)));
       }
     });
   });

package/src/engines/claude.js

@@ -19,6 +19,7 @@ import os from 'node:os';
 import path from 'node:path';
 import { resolveClaude, buildSpawnSpec, isCompiledBinary, selfMcpStdio } from '../env.js';
 import { buildCliPrompt } from './prompt.js';
+import { summarizeCliError } from '../cli-errors.js';
 import { killOnAbort } from '../proc.js';
 import { pushExtraArgs, FORBIDDEN } from './args.js';
 import { displayPath, resolveWorkdir } from '../workdir.js';
@@ -160,7 +161,7 @@ function runClaude({ prompt, args, cwd, emit, signal }) {
       detach();
       if (signal?.aborted) { resolve({ streamedAny, resultText }); return; } // Stop pressed — end quietly
       if (code === 0) resolve({ streamedAny, resultText });
-      else reject(new Error(`Claude Code exited ${code}: ${stderr.trim().split('\n').pop() || 'failed'}`));
+      else reject(new Error(summarizeCliError('Claude Code', code, stderr)));
     });
 
     child.stdin.write(prompt);

package/src/engines/codex.js

@@ -24,6 +24,7 @@ import { findAgentBin, selfMcpStdio } from '../env.js';
 import { buildCliPrompt } from './prompt.js';
 import { pushExtraArgs, FORBIDDEN } from './args.js';
 import { resolveWorkdir } from '../workdir.js';
+import { summarizeCliError } from '../cli-errors.js';
 
 // Idle timeout: re-armed on every stdout/stderr chunk, so a long run that keeps
 // streaming never trips it — only true silence does. Override with
@@ -238,7 +239,7 @@ export async function chat({ messages, system, options, images }, emit, { signal
         emit({ type: 'done', text: '' });
         resolve();
       } else {
-        reject(new Error(`Codex exited ${code}: ${stderr.trim() || 'failed'}`));
+        reject(new Error(summarizeCliError('Codex', code, stderr)));
       }
     });
 

package/src/engines/custom.js

@@ -621,7 +621,7 @@ export async function runSpec(spec, { messages, system, options = {}, images },
         emit({ type: 'done', text: parser.streamed ? '' : parser.finish() });
         resolve();
       } else {
-        reject(new Error(`${label} exited ${code}: ${stderr.trim().split('\n').pop() || 'failed'}`));
+        reject(new Error(summarizeCliError(label, code, stderr)));
       }
     });
 

package/src/server.js

@@ -67,7 +67,7 @@ import {
 // Hardcoded (not read from package.json) so it survives Bun's single-file
 // --compile, where package.json isn't on a readable FS. CI fails the publish if
 // this drifts from package.json, so the two can't silently diverge.
-const VERSION = '0.10.40';
+const VERSION = '0.10.41';
 const HOST = process.env.CHATPANEL_BRIDGE_HOST || '127.0.0.1';
 const PORT = Number(process.env.CHATPANEL_BRIDGE_PORT) || 4319;