chrome-bridge-mcp 1.30.1 → 1.30.3

package/extension/manifest.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "Chrome Bridge for Claude Code",
-  "version": "1.30.1",
+  "version": "1.30.2",
   "description": "Bridge your browser to Claude Code: 67 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

@@ -1507,14 +1507,24 @@ function recordTabEnding(tabId, ending) {
   if (tabEndings.size > TAB_ENDINGS_MAX) tabEndings.delete(tabEndings.keys().next().value);
 }
 
+const CLOSE_WAIT_MS = 5000;
+
 async function cmdTabAction({ action, tab_id, bypass_cache = false }) {
   if (!action) throw new Error('Missing required parameter: action');
   const tabId = await resolveTabId(tab_id);
 
   if (action === 'close') {
+    // chrome.tabs.remove aspetta che la pagina lasci andare: con un dialogo
+    // «Vuoi uscire?» aperto o la pagina bloccata in un ciclo resterebbe muta
+    // fino al timeout del comando (30 s, visto il 15/10). Dopo 5 s si risponde
+    // con un errore; la chiusura resta in corso e Chrome la completa quando
+    // la pagina cede (get_tabs la riporta come closed_by_bridge).
     bridgeClosing.add(tabId);
-    try { await chrome.tabs.remove(tabId); } catch (e) { bridgeClosing.delete(tabId); throw e; }
-    return { action, closed: tabId };
+    const removing = chrome.tabs.remove(tabId).then(() => true, (e) => { bridgeClosing.delete(tabId); throw e; });
+    const done = await Promise.race([removing, new Promise((r) => setTimeout(() => r(false), CLOSE_WAIT_MS))]);
+    if (done) return { action, closed: tabId };
+    removing.catch(() => {});
+    throw new Error(`Tab ${tabId} did not close within ${CLOSE_WAIT_MS / 1000} s: the page is not answering, either a "Leave site?" dialog is open (beforeunload) or the page is stuck in a loop. The close is still pending and completes if the page lets go. Look at the tab: answer the dialog, or close it by hand.`);
   }
   if (action === 'discard') {
     // La scheda resta nella barra e si ricarica al focus: il processo muore, la

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "chrome-bridge-mcp",
   "mcpName": "io.github.frsorrentino/chrome-bridge",
-  "version": "1.30.1",
+  "version": "1.30.3",
   "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). 67 tools; 2.8-4.3× cheaper and up to 9.3× fewer turns than Claude in Chrome, measured; ChromeOS included.",
   "main": "server/index.js",

package/server/index.js

@@ -14,6 +14,7 @@ import { WSManager } from './ws-manager.js';
 import { registerTools } from './tools.js';
 import { createObserver } from './observe.js';
 import { BRANDED_CHROME_HINT, launchBrowser } from './launcher.js';
+import { INSTRUCTIONS } from './instructions.js';
 import { DEFAULT_PORT, VERSION } from './protocol.js';
 
 // Launch mode: browser dedicato (profilo effimero + estensione unpacked).
@@ -50,9 +51,7 @@ function parseCaps() {
 }
 
 async function main() {
-  // 1. Crea il server MCP
-  // instructions: dette una volta qui invece che ripetute in ogni descrizione
-  // tool — pesano ~1 volta nel contesto del client anziché ~50.
+  // 1. Crea il server MCP (istruzioni e loro limiti in instructions.js)
   const mcpServer = new McpServer({
     name: 'chrome-bridge',
     version: VERSION,
@@ -60,36 +59,7 @@ async function main() {
     // get_status({enable}) accende N tool uno per uno: una sola notifica,
     // non N (un ciclo di list_changed costava CPU a Claude Code, 2.1.271).
     debouncedNotificationMethods: ['notifications/tools/list_changed'],
-    instructions: [
-      // Eval del 29/09: con Claude in Chrome richiesto e assente, il modello
-      // ripiegava su chrome-bridge senza chiedere (2 casi su 2).
-      'If the user names another browser tool (e.g. Claude in Chrome) and it is not available, say so and ask; do not switch to chrome-bridge on your own.',
-      'Selectors on DOM tools pierce shadow DOM with ">>>" ("my-app >>> button.save").',
-      'tab_id omitted = the tab last navigated/created in this session, else the active tab. frame_id omitted = main frame (list frames with get_frames).',
-      'To find targets use get_interactives, not read_page(html); its refs (n1, n2…) are the ref param of click/type_text/hover and of fill_form fields, and navigate already returns them.',
-      // Il costo dominante sono i TURNI, non i byte: un turno vale ~15-30 volte
-      // un KB di output risparmiato. Queste due clausole si pagano una volta
-      // qui e valgono più di qualunque ottimizzazione di schema.
-      'Several fields: one fill_form (submit_selector submits in the same call), not repeated type_text: one turn instead of N.',
-      'Tables: extract_table (where/columns filtered server-side) or extract, never read_page: a big table through read_page costs tens of thousands of tokens.',
-      // Debug del 29/09: in 7 run su 20 il modello caricava la skill prima di
-      // agire e pagava ~2 turni. La regola che serve sta qui, già in contesto.
-      'Debugging a page: navigate already reports the first console error mapped to its source line and the failed requests with status; get_css_styles names the rule behind a style and flags undefined_vars. Start there; no skill or recipe needed.',
-      'Detail in a screenshot: element_screenshot (selector or region, scale), not another full one. To check an outcome: assert or wait_for poll for you, no screenshot.',
-      // Il blocco più frequente sul campo (Meta Ads Manager, 23/09/2026): tre
-      // attese scadute e due screenshot falliti prima di capire che la
-      // finestra era dietro un'altra.
-      // Il 42% delle sessioni reali caricava i tool con 2+ ToolSearch (media 2,0).
-      'When tools are deferred, batch every tool needed for the next workflow into ONE ToolSearch call using exact names (select:name1,name2,...); do not search one tool at a time.',
-      'Missing optional tools: call get_status({enable:["group",...]}) once for all required groups (audits, visual, network, storage, dom, files, perf; "all" enables all). After tools/list_changed, use them next turn; if still deferred, batch-search their names once. No restart or user configuration needed.',
-      // 14/10/2026: nei flussi di lavoro Claude restava fermo sulla finestra
-      // «Salva con nome», di sistema e fuori portata dell'estensione. Su ChromeOS
-      // con «Chiedi dove salvare ciascun file» si apre anche con saveAs:false.
-      'Saving a file: never click a Download/Export button and wait for the browser download, the OS "Save as" dialog can open and no tool can press it. '
-        + 'A file with a URL: http_request with save_to (browser cookies, bytes written to disk, no dialog). '
-        + 'A file the page builds itself (blob, data URL, showSaveFilePicker, a download the page starts): capture_downloads arm, then click, then capture_downloads wait.',
-      'A tab in a minimized, covered or background window does not render: screenshots fail, timers slow down. get_page_info reports visibility (page_hidden in a result means the same); bring the window on screen, or create_tab new_window with bounds.',
-    ].join(' '),
+    instructions: INSTRUCTIONS,
   });
 
   // 2. Avvia il WebSocket server
@@ -104,16 +74,31 @@ async function main() {
   // durante l'avvio (osservate 3 directory residue, una da 122 MB).
   let browser = null;
   const session = { tools: null };
-  const shutdown = async () => {
+  // Una sola chiusura anche quando fine di stdin, chiusura del trasporto e
+  // segnale arrivano insieme.
+  let closing = null;
+  const shutdown = () => (closing ??= (async () => {
     console.error('[chrome-bridge] Shutting down...');
-    try { await Promise.race([session.tools?.closeEmptyOwnedTabs(), new Promise((r) => setTimeout(r, 2000))]); } catch {}
-    try { if (browser) await browser.stop(); } catch {}
+    // Il browser di --launch per primo: su Windows Claude Code chiude stdin e
+    // dopo 300 ms uccide l'albero dei processi (2.1.296). Le sue schede
+    // muoiono con lui; nel Chrome dell'utente si chiudono quelle rimaste vuote.
+    if (browser) {
+      try { await browser.stop(); } catch {}
+    } else {
+      try { await Promise.race([session.tools?.closeEmptyOwnedTabs(), new Promise((r) => setTimeout(r, 2000))]); } catch {}
+    }
     try { await wsManager.stop(); } catch {}
     try { await mcpServer.close(); } catch {}
     process.exit(0);
-  };
+  })());
   process.on('SIGINT', shutdown);
   process.on('SIGTERM', shutdown);
+  // Su Windows SIGINT e SIGTERM non arrivano, e un client terminato di colpo
+  // non manda segnali: in entrambi i casi resta solo la fine di stdin. L'SDK
+  // non la ascolta, e il WebSocket teneva vivo il processo.
+  process.stdin.once('end', shutdown);
+  process.stdin.once('close', shutdown);
+  mcpServer.server.onclose = shutdown;
   process.on('uncaughtException', (err) => {
     console.error('[chrome-bridge] Uncaught exception:', err);
     shutdown().catch(() => process.exit(1));

package/server/instructions.js

@@ -0,0 +1,48 @@
+/**
+ * Istruzioni del server MCP: dette una volta all'initialize invece che
+ * ripetute in ogni descrizione tool, pesano ~1 volta nel contesto del client
+ * anziché ~50.
+ *
+ * Claude Code le taglia: 4.096 caratteri dalla 2.1.296, 2.048 fino alla
+ * 2.1.295. RULES deve entrare nei primi 2.048; TAIL ripete quello che dicono
+ * già le descrizioni dei tool, quindi un client vecchio la può perdere.
+ * test/unit/instructions.test.js tiene i due limiti.
+ */
+
+export const RULES = [
+  // Eval del 29/09: con Claude in Chrome richiesto e assente, il modello
+  // ripiegava su chrome-bridge senza chiedere (2 casi su 2).
+  'If the user names another browser tool (e.g. Claude in Chrome) and it is not available, say so and ask; do not switch to chrome-bridge on your own.',
+  // Il costo dominante sono i TURNI, non i byte: un turno vale ~15-30 volte
+  // un KB di output risparmiato. Queste due clausole si pagano una volta
+  // qui e valgono più di qualunque ottimizzazione di schema.
+  'Several fields: one fill_form (submit_selector submits in the same call), not repeated type_text: one turn instead of N.',
+  'Tables: extract_table (where/columns filtered server-side) or extract, never read_page: a big table through read_page costs tens of thousands of tokens.',
+  // Debug del 29/09: in 7 run su 20 il modello caricava la skill prima di
+  // agire e pagava ~2 turni. La regola che serve sta qui, già in contesto.
+  'Debugging a page: navigate already reports the first console error mapped to its source line and the failed requests with status; get_css_styles names the rule behind a style and flags undefined_vars. Start there; no skill or recipe needed.',
+  'Detail in a screenshot: element_screenshot (selector or region, scale), not another full one. To check an outcome: assert or wait_for poll for you, no screenshot.',
+  // Il 42% delle sessioni reali caricava i tool con 2+ ToolSearch (media 2,0).
+  'When tools are deferred, batch every tool needed for the next workflow into ONE ToolSearch call using exact names (select:name1,name2,...); do not search one tool at a time.',
+  'Missing optional tools: call get_status({enable:["group",...]}) once for all required groups (audits, visual, network, storage, dom, files, perf; "all" enables all). After tools/list_changed, use them next turn; if still deferred, batch-search their names once. No restart or user configuration needed.',
+  // 14/10/2026: nei flussi di lavoro Claude restava fermo sulla finestra
+  // «Salva con nome», di sistema e fuori portata dell'estensione. Su ChromeOS
+  // con «Chiedi dove salvare ciascun file» si apre anche con saveAs:false.
+  'Saving a file: never click a Download/Export button and wait for the browser download, the OS "Save as" dialog can open and no tool can press it. '
+    + 'A file with a URL: http_request with save_to (browser cookies, bytes written to disk, no dialog). '
+    + 'A file the page builds itself (blob, data URL, showSaveFilePicker, a download the page starts): capture_downloads arm, then click, then capture_downloads wait.',
+  // Il blocco più frequente sul campo (Meta Ads Manager, 23/09/2026): tre
+  // attese scadute e due screenshot falliti prima di capire che la
+  // finestra era dietro un'altra.
+  'A tab in a minimized, covered or background window does not render: screenshots fail, timers slow down. get_page_info reports visibility (page_hidden in a result means the same); bring the window on screen, or create_tab new_window with bounds.',
+];
+
+// Le ripetono le descrizioni di read_page, get_interactives, navigate e
+// fill_form, e dei parametri ref, tab_id, frame_id e selector.
+export const TAIL = [
+  'To find targets use get_interactives, not read_page(html); its refs (n1, n2…) are the ref param of click/type_text/hover and of fill_form fields, and navigate already returns them.',
+  'tab_id omitted = the tab last navigated/created in this session, else the active tab. frame_id omitted = main frame (list frames with get_frames).',
+  'Selectors on DOM tools pierce shadow DOM with ">>>" ("my-app >>> button.save").',
+];
+
+export const INSTRUCTIONS = [...RULES, ...TAIL].join(' ');

package/server/lighthouse.js

@@ -9,6 +9,8 @@
  * chrome-bridge gira da Node 18: sotto si usa la 12.8.2 (Node >= 18.16).
  */
 import { spawn } from 'node:child_process';
+import { existsSync } from 'node:fs';
+import { win32 } from 'node:path';
 
 export function lighthouseVersion(node = process.versions.node) {
   const [maj, min] = node.split('.').map(Number);
@@ -21,9 +23,24 @@ export function lighthouseArgs(url, port, outBase, { categories, formFactor = 'm
     '--quiet', ...(categories?.length ? [`--only-categories=${categories.join(',')}`] : []), ...(formFactor === 'desktop' ? ['--preset=desktop'] : [])];
 }
 
+/**
+ * Come lanciare npx. Su Windows npx è npx.cmd, che spawn senza shell non trova
+ * (ENOENT): si usa npx-cli.js accanto a node.exe con il node del server, senza
+ * shell. Se manca, npx.cmd con la shell e gli argomenti fra virgolette.
+ */
+export function npxCommand({ platform = process.platform, execPath = process.execPath, env = process.env, exists = existsSync } = {}) {
+  if (env.CHROME_BRIDGE_NPX) return { cmd: env.CHROME_BRIDGE_NPX, pre: [], shell: false };
+  if (platform !== 'win32') return { cmd: 'npx', pre: [], shell: false };
+  const cli = win32.join(win32.dirname(execPath), 'node_modules', 'npm', 'bin', 'npx-cli.js');
+  if (exists(cli)) return { cmd: execPath, pre: [cli], shell: false };
+  return { cmd: 'npx.cmd', pre: [], shell: true };
+}
+
 export function runLighthouse(args, { timeoutMs = 240000 } = {}) {
   return new Promise((resolve, reject) => {
-    const p = spawn(process.env.CHROME_BRIDGE_NPX || 'npx', args, { stdio: ['ignore', 'ignore', 'pipe'] });
+    const { cmd, pre, shell } = npxCommand();
+    const argv = [...pre, ...args].map((a) => (shell ? `"${a}"` : a));
+    const p = spawn(cmd, argv, { stdio: ['ignore', 'ignore', 'pipe'], shell });
     let err = '';
     const timer = setTimeout(() => { p.kill('SIGKILL'); reject(new Error(`Lighthouse did not finish within ${timeoutMs / 1000} s`)); }, timeoutMs);
     p.stderr.on('data', (d) => { err += d; });

package/server/protocol.js

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