@chatpanel/bridge 0.10.38 → 0.10.40

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chatpanel/bridge",
-  "version": "0.10.38",
+  "version": "0.10.40",
   "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/engines/antigravity.js

@@ -50,7 +50,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');

package/src/engines/claude.js

@@ -54,7 +54,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();

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

@@ -77,10 +77,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');

package/src/engines/custom.js

@@ -333,18 +333,23 @@ async function runStableMcpSetup(plan, cwd) {
     let child;
     try { child = spawn(bin, argv, opts); } catch (e) { return reject(new Error(`Failed to start ${plan.command}: ${e.message}`)); }
     let stderr = '';
+    // The CLI often reports WHY it couldn't register on stdout while still exiting 0 (Hermes
+    // prints "requires the 'mcp' Python SDK" and saves nothing). Capturing it is what lets
+    // the caller turn a silent no-op into an actionable message.
+    let stdout = '';
     const timer = setTimeout(() => {
       child.kill('SIGKILL');
       reject(new Error(`${plan.command} MCP setup timed out.`));
     }, 20_000);
     child.stderr.on('data', (d) => (stderr += d.toString()));
+    child.stdout?.on('data', (d) => (stdout += d.toString()));
     child.on('error', (e) => {
       clearTimeout(timer);
       reject(new Error(`Failed to start ${plan.command}: ${e.message}`));
     });
     child.on('close', (code) => {
       clearTimeout(timer);
-      if (code === 0) return resolve();
+      if (code === 0) return resolve(`${stdout}\n${stderr}`.trim());
       reject(new Error(`${plan.command} MCP setup exited ${code}: ${stderr.trim().split('\n').pop() || 'failed'}`));
     });
     try { child.stdin.end(); } catch { /* ignore */ }
@@ -386,9 +391,15 @@ export async function ensureStableMcpConfig(spec, cwd, label, emit, deps = {}) {
   }
 
   emit({ type: 'status', text: `${label} is setting up one-time browser tools...` });
-  await runSetup(plan, cwd);
+  // Keep what the CLI said: it commonly exits 0 while refusing to register (no MCP SDK, no
+  // HTTP transport), and that sentence is the only accurate explanation available.
+  const setupOutput = String((await runSetup(plan, cwd)) || '').trim();
   if (await hasConfig(spec, cwd)) return true;
-  throw new Error(`${label} browser-tool setup completed, but the MCP server is still not visible. Run: ${setupCommand}`);
+  throw new Error(
+    `${label} browser-tool setup did not register the server.`
+    + (setupOutput ? ` ${setupOutput.split('\n').filter(Boolean).slice(-3).join(' ')}` : '')
+    + ` Run: ${setupCommand}`,
+  );
 }
 
 export async function chat({ messages, system, options, images }, emit, { signal } = {}) {

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.38';
+const VERSION = '0.10.40';
 const HOST = process.env.CHATPANEL_BRIDGE_HOST || '127.0.0.1';
 const PORT = Number(process.env.CHATPANEL_BRIDGE_PORT) || 4319;