@chatpanel/bridge 0.10.39 → 0.10.41

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chatpanel/bridge",
-  "version": "0.10.39",
+  "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';
@@ -50,7 +51,9 @@ let lastProbe = 0;
 export async function available() {
   // Cache a positive result; keep re-probing (throttled) while not found so it
   // self-heals once agy appears on PATH — never cache a negative forever.
-  if (!installed && Date.now() - lastProbe > 4000) {
+  // Re-probe in both directions (see codex.js): an uninstalled CLI must stop reporting itself
+  // available without waiting for a bridge restart.
+  if (Date.now() - lastProbe > (installed ? 30_000 : 4000)) {
     lastProbe = Date.now();
     try {
       installed = !!findAgentBin('agy');
@@ -151,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';
@@ -54,7 +55,9 @@ let cachedOk = false;
 export async function available() {
   // Availability = "can we launch claude somehow" (native / cli.js / WSL / SDK),
   // NOT "does `claude --version` exit 0" (which fails when it just needs login).
-  if (!cachedOk && Date.now() - lastProbe > 4000) {
+  // Re-probe in both directions (see codex.js): caching success forever meant an uninstalled
+  // CLI still reported itself available.
+  if (Date.now() - lastProbe > (cachedOk ? 30_000 : 4000)) {
     lastProbe = Date.now();
     try {
       const spec = resolveClaude();
@@ -158,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/cli-agents.js

@@ -21,9 +21,13 @@ function makeCliAgent(command, spec, notFoundHint, overrides = {}) {
   return {
     spec: resolvedSpec,
     async available() {
-      // Cache a positive result; re-probe (throttled) while not found so it
-      // self-heals once the CLI appears on PATH.
-      if (!installed && Date.now() - lastProbe > 4000) {
+      // Re-probe in BOTH directions. Caching a positive result forever meant an UNINSTALLED
+      // CLI kept reporting itself available until the bridge restarted — the picker showed a
+      // green dot for an agent that could no longer run, and the turn failed with "couldn't
+      // find it on your PATH". A found CLI is re-checked on a slow interval (it rarely
+      // disappears, and `which` is cheap but not free); a missing one keeps the fast interval
+      // so it still self-heals the moment it is installed.
+      if (Date.now() - lastProbe > (installed ? 30_000 : 4000)) {
         lastProbe = Date.now();
         try {
           installed = !!findAgentBin(command);

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
@@ -77,10 +78,10 @@ function ensureIsolatedHome() {
 let installed = false;
 let lastProbe = 0;
 export async function available() {
-  // Availability = "is codex findable on PATH", not "does `codex --version` exit
-  // 0" (which fails when the CLI just needs login). Cache positives; re-probe
-  // (throttled) while not found so it self-heals once codex appears on PATH.
-  if (!installed && Date.now() - lastProbe > 4000) {
+  // Availability = "is codex findable on PATH", not "does `codex --version` exit 0" (which
+  // fails when the CLI just needs login). Re-probed in BOTH directions: caching a positive
+  // forever kept an uninstalled CLI reporting itself available until the bridge restarted.
+  if (Date.now() - lastProbe > (installed ? 30_000 : 4000)) {
     lastProbe = Date.now();
     try {
       installed = !!findAgentBin('codex');
@@ -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.39';
+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;