chrome-bridge-mcp 1.23.3 → 1.23.4

package/extension/lib/command-outcome.js

@@ -0,0 +1,57 @@
+/**
+ * Esiti dei comandi che prima si fidavano di un valore di ripiego.
+ *
+ * Due casi osservati nel test Windows del 27/09/2026, validi su ogni OS:
+ * - `click #checkout-button` con `query_dom` a count 0 rispondeva
+ *   `{"clicked":true}` (type_text, hover e press_key allo stesso modo): la
+ *   funzione iniettata lancia «Element not found», `executeScript` risolve
+ *   con `result` undefined e il ripiego `?? { clicked: true }` lo
+ *   trasformava in un successo.
+ * - `navigate` verso una porta chiusa rispondeva `{url, title}` senza errore:
+ *   la scheda mostra la pagina d'errore di Chrome, che arriva comunque a
+ *   status 'complete'. L'errore vero passa solo da webNavigation.onErrorOccurred.
+ */
+
+/**
+ * Risultato di click, type_text, hover e press_key: senza un risultato della
+ * funzione iniettata il comando non è avvenuto, qualunque sia il motivo. Tutte
+ * e quattro rispondevano con un successo di ripiego ({clicked|typed|hovered|
+ * pressed: true}) su un selettore che non trova niente.
+ */
+export function elementOutcome(results, selector) {
+  const first = results?.[0];
+  if (first?.result != null) return first.result;
+  if (first?.error?.message) throw new Error(first.error.message);
+  if (!selector) throw new Error('The page returned no result for the command');
+  throw new Error(`Element not found: ${selector} (no element matches; check it with query_dom)`);
+}
+
+/**
+ * net::ERR_ABORTED non è un fallimento da riferire: è un download, una
+ * risposta 204 o una navigazione sostituita da un'altra.
+ */
+export function isReportableNavError(details) {
+  return details?.frameId === 0 && !!details.error && details.error !== 'net::ERR_ABORTED';
+}
+
+/**
+ * Raccoglie gli errori di rete del frame principale, per scheda, dal momento
+ * della chiamata. Va aperta PRIMA di tabs.create/tabs.update: un rifiuto di
+ * connessione in locale arriva in pochi millisecondi, prima che l'id della
+ * scheda nuova sia noto.
+ */
+export function watchNavErrors(onErrorOccurred) {
+  const byTab = new Map();
+  const listener = (details) => {
+    if (isReportableNavError(details)) byTab.set(details.tabId, details);
+  };
+  onErrorOccurred.addListener(listener);
+  return {
+    errorFor: (tabId) => byTab.get(tabId) ?? null,
+    stop: () => onErrorOccurred.removeListener(listener),
+  };
+}
+
+export function navErrorMessage(details, tabId) {
+  return `${details.error} loading ${details.url} (tab ${tabId} shows the browser's error page)`;
+}

package/extension/manifest.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "Chrome Bridge for Claude Code",
-  "version": "1.23.0",
+  "version": "1.23.4",
   "description": "Bridge your browser to Claude Code: 60 token-efficient web-dev automation tools over a local WebSocket. ChromeOS included.",
   "homepage_url": "https://github.com/frsorrentino/chrome-bridge",
   "minimum_chrome_version": "135",

package/extension/service-worker.js

@@ -12,6 +12,7 @@ import { createPacer } from './lib/capture-pacing.js';
 import { computeTiles } from './lib/tile-layout.js';
 import { classifyDownload } from './lib/download-state.js';
 import { findTextInPage } from './lib/find-text.js';
+import { elementOutcome, watchNavErrors, navErrorMessage } from './lib/command-outcome.js';
 const { pushError } = globalThis.__cbTelemetry;
 
 const DEFAULT_PORT = 8765;
@@ -779,29 +780,40 @@ async function cmdGetTabs({ include_windows = false, ended } = {}) {
 async function cmdNavigate({ url, tab_id }) {
   if (!url) throw new Error('Missing required parameter: url');
 
-  // Senza tab_id esplicito: apri SEMPRE in una nuova scheda (attiva) per non
-  // sovrascrivere la scheda/sessione corrente dell'utente. Con tab_id: naviga
-  // quella scheda specifica.
-  if (!tab_id) {
-    const newTab = await chrome.tabs.create({ url, active: true });
-    // Se la pagina è già 'complete' (es. cache) non aspettare il listener
-    if (newTab.status !== 'complete') {
-      await waitForComplete(newTab.id);
+  // La pagina d'errore di Chrome (connessione rifiutata, DNS) arriva a
+  // 'complete' come una pagina vera: l'errore si legge solo qui.
+  const navErrors = watchNavErrors(chrome.webNavigation.onErrorOccurred);
+  try {
+    // Senza tab_id esplicito: apri SEMPRE in una nuova scheda (attiva) per non
+    // sovrascrivere la scheda/sessione corrente dell'utente. Con tab_id: naviga
+    // quella scheda specifica.
+    if (!tab_id) {
+      const newTab = await chrome.tabs.create({ url, active: true });
+      // Se la pagina è già 'complete' (es. cache) non aspettare il listener
+      if (newTab.status !== 'complete') {
+        await waitForComplete(newTab.id);
+      }
+      const failed = navErrors.errorFor(newTab.id);
+      if (failed) throw new Error(navErrorMessage(failed, newTab.id));
+      const createdTab = await chrome.tabs.get(newTab.id);
+      return { url: createdTab.url, title: createdTab.title, tabId: newTab.id };
     }
-    const createdTab = await chrome.tabs.get(newTab.id);
-    return { url: createdTab.url, title: createdTab.title, tabId: newTab.id };
-  }
 
-  const tabId = await resolveTabId(tab_id);
+    const tabId = await resolveTabId(tab_id);
 
-  // Registra il listener PRIMA di tabs.update: una navigazione veloce (cache)
-  // può emettere 'complete' prima che il listener esista
-  const done = waitForComplete(tabId);
-  await chrome.tabs.update(tabId, { url });
-  await done;
+    // Registra il listener PRIMA di tabs.update: una navigazione veloce (cache)
+    // può emettere 'complete' prima che il listener esista
+    const done = waitForComplete(tabId);
+    await chrome.tabs.update(tabId, { url });
+    await done;
 
-  const updatedTab = await chrome.tabs.get(tabId);
-  return { url: updatedTab.url, title: updatedTab.title, tabId };
+    const failed = navErrors.errorFor(tabId);
+    if (failed) throw new Error(navErrorMessage(failed, tabId));
+    const updatedTab = await chrome.tabs.get(tabId);
+    return { url: updatedTab.url, title: updatedTab.title, tabId };
+  } finally {
+    navErrors.stop();
+  }
 }
 
 async function cmdScreenshot({ tab_id }) {
@@ -907,7 +919,7 @@ async function cmdClick({ selector, tab_id, frame_id, force = false, button = 'l
     world: 'MAIN',
   });
 
-  return results?.[0]?.result ?? { clicked: true };
+  return elementOutcome(results, selector);
 }
 
 async function cmdTypeText({ selector, text, mode = 'set', tab_id, frame_id }) {
@@ -982,7 +994,7 @@ async function cmdTypeText({ selector, text, mode = 'set', tab_id, frame_id }) {
     world: 'MAIN',
   });
 
-  return results?.[0]?.result ?? { typed: true };
+  return elementOutcome(results, selector);
 }
 
 // Impronta per page_changed: pochi interi letti nell'ISOLATED world (nessuna
@@ -3331,7 +3343,7 @@ async function cmdHover({ selector, tab_id, frame_id }) {
     args: [selector],
     world: 'MAIN',
   });
-  return results?.[0]?.result ?? { hovered: true };
+  return elementOutcome(results, selector);
 }
 
 // --- press_key ---
@@ -3398,7 +3410,7 @@ async function cmdPressKey({ key, selector, ctrl = false, shift = false, alt = f
     args: [key, selector || null, { ctrl, shift, alt, meta }],
     world: 'MAIN',
   });
-  return results?.[0]?.result ?? { pressed: true };
+  return elementOutcome(results, selector);
 }
 
 // --- get_frames ---

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "chrome-bridge-mcp",
   "mcpName": "io.github.frsorrentino/chrome-bridge",
-  "version": "1.23.3",
+  "version": "1.23.4",
   "type": "module",
   "description": "Chrome Bridge by frsorrentino: MCP server that lets Claude Code borrow a tab from your own logged-in Chrome and hand it back — logins, 2FA and CAPTCHAs stay with you (handoff). 60 tools; about half the cost of Claude in Chrome, measured; ChromeOS included.",
   "main": "server/index.js",

package/server/protocol.js

@@ -93,7 +93,7 @@ export const MessageType = Object.freeze({
 });
 
 // Versione
-export const VERSION               = '1.23.3';
+export const VERSION               = '1.23.4';
 
 // Configurazione
 export const DEFAULT_PORT          = 8765;