chrome-bridge-mcp 1.15.0 → 1.15.1

package/extension/lib/download-state.js

@@ -0,0 +1,43 @@
+/**
+ * Legge lo stato reale di un download.
+ *
+ * `chrome.downloads.download()` restituisce un id e basta: dire "started: true"
+ * e fermarsi lì è come fidarsi di un HTTP 200 senza guardare il corpo. Su
+ * ChromeOS, con l'impostazione «Chiedi dove salvare ogni file» attiva, Chrome
+ * ignora `saveAs: false` e apre il selettore: il file viene scaricato per intero
+ * ma non viene scritto da nessuna parte finché un umano non sceglie.
+ *
+ * La firma di quella condizione è precisa e verificata dal vivo:
+ * `state: 'in_progress'`, `filename: ''`, `bytesReceived === totalBytes`.
+ * Senza distinguerla, un agente aspetta un file che non arriverà mai da solo.
+ */
+export function classifyDownload(item) {
+  if (!item) return { state: 'unknown', done: false, reason: 'download not found' };
+
+  const { state, filename, bytesReceived = 0, totalBytes = 0, error } = item;
+
+  if (state === 'complete') return { state: 'complete', done: true, filename };
+  if (state === 'interrupted') {
+    return { state: 'interrupted', done: true, error: error || 'interrupted', filename };
+  }
+
+  // Nessuna destinazione decisa: il selettore è aperto e aspetta una persona.
+  if (!filename) {
+    const fetched = totalBytes > 0 && bytesReceived >= totalBytes;
+    return {
+      state: 'waiting_for_user',
+      done: false,
+      bytes: { received: bytesReceived, total: totalBytes },
+      reason: fetched
+        ? 'bytes fetched, but Chrome is asking where to save — the "Ask where to save each file" setting overrides saveAs:false'
+        : 'no destination chosen yet — Chrome is showing the save dialog',
+    };
+  }
+
+  return {
+    state: 'in_progress',
+    done: false,
+    filename,
+    bytes: { received: bytesReceived, total: totalBytes },
+  };
+}

package/extension/manifest.json

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

@@ -8,6 +8,7 @@
 import './telemetry.js';
 import { buildMarkdown } from './lib/page-markdown.js';
 import { computeTiles } from './lib/tile-layout.js';
+import { classifyDownload } from './lib/download-state.js';
 const { pushError } = globalThis.__cbTelemetry;
 
 const DEFAULT_PORT = 8765;
@@ -1092,7 +1093,7 @@ async function cmdTileWindows({ window_ids, reference_window_id, layout = 'grid'
   };
 }
 
-async function cmdMoveTab({ tab_id, window_id, new_window = false, left, top, width, height, index = -1 }) {
+async function cmdMoveTab({ tab_id, window_id, new_window = false, window_type, left, top, width, height, index = -1 }) {
   if (typeof tab_id !== 'number') throw new Error('Missing required parameter: tab_id');
   if (typeof window_id !== 'number' && !new_window) throw new Error('Provide window_id or new_window: true');
 
@@ -1100,8 +1101,11 @@ async function cmdMoveTab({ tab_id, window_id, new_window = false, left, top, wi
 
   if (new_window) {
     // tabs.move vuole una finestra che esiste già; per estrarre in una finestra
-    // nuova (e posizionata) l'unica via è windows.create({tabId}).
+    // nuova (e posizionata) l'unica via è windows.create({tabId}). windows.create
+    // conosce solo normal e popup: le finestre app sono della SWA, non nostre —
+    // popup è il massimo di "finestra terminale" ottenibile da un'estensione.
     const winOpts = { tabId: tab_id };
+    if (window_type === 'popup') winOpts.type = 'popup';
     for (const [k, v] of [['left', left], ['top', top], ['width', width], ['height', height]]) {
       if (typeof v === 'number') winOpts[k] = v;
     }
@@ -1111,6 +1115,9 @@ async function cmdMoveTab({ tab_id, window_id, new_window = false, left, top, wi
       url: before.url,
       from_window: before.windowId,
       to_window: win.id,
+      // Il tipo lo dice Chrome, non la richiesta: se popup viene declassato a
+      // normal il chiamante deve vederlo da qui.
+      window_type: win.type,
       new_window: true,
       index: 0,
       same_window: before.windowId === win.id,
@@ -1910,7 +1917,7 @@ async function cmdViewportResize({ preset, width, height, left, top, state, read
     if (Object.keys(updateOpts).length === 0) {
       if (state) {
         const w = await chrome.windows.get(tab.windowId);
-        return { requested: { state }, actual: { left: w.left, top: w.top, width: w.width, height: w.height, state: w.state } };
+        return { requested: { state }, window: { left: w.left, top: w.top, width: w.width, height: w.height, state: w.state } };
       }
       throw new Error('Provide preset, width, height, left, top or state');
     }
@@ -1920,24 +1927,40 @@ async function cmdViewportResize({ preset, width, height, left, top, state, read
     await new Promise((r) => setTimeout(r, 200));
   }
 
-  const results = await chrome.scripting.executeScript({
-    target: { tabId },
-    func: () => ({
-      viewportWidth: window.innerWidth,
-      viewportHeight: window.innerHeight,
-      devicePixelRatio: window.devicePixelRatio,
-    }),
-    world: 'MAIN',
-  });
-  const actual = results?.[0]?.result ?? {};
-  if (read_only) return { actual };
-  const win = await chrome.windows.get(tab.windowId);
+  // La misura del viewport passa da un'injection, e `chrome://` e
+  // `chrome-untrusted://` la rifiutano: le finestre del Terminale e di
+  // media-app sono esattamente quelle. Prima questo errore veniva propagato e
+  // il tool falliva DOPO aver spostato la finestra — l'azione riuscita
+  // riportata come guasto, che è il modo di far ritentare chi chiama qualcosa
+  // che è già successo. La geometria della finestra non richiede injection:
+  // arriva da chrome.windows.get e c'è sempre.
+  let actual = {};
+  let viewport_error = null;
+  try {
+    const results = await chrome.scripting.executeScript({
+      target: { tabId },
+      func: () => ({
+        viewportWidth: window.innerWidth,
+        viewportHeight: window.innerHeight,
+        devicePixelRatio: window.devicePixelRatio,
+      }),
+      world: 'MAIN',
+    });
+    actual = results?.[0]?.result ?? {};
+  } catch (e) {
+    viewport_error = e.message;
+  }
+  const winNow = await chrome.windows.get(tab.windowId);
+  const geometry = { left: winNow.left, top: winNow.top, width: winNow.width, height: winNow.height, state: winNow.state };
+  if (read_only) return { actual, window: geometry, viewport_error };
+  const win = winNow;
   return {
     requested: { width: targetW, height: targetH, preset: preset || null, left, top, state },
     actual,
+    viewport_error,
     // Il window manager può accettare e ignorare: senza il confronto una
     // finestra rimasta ferma passerebbe per spostata.
-    window: { left: win.left, top: win.top, width: win.width, height: win.height, state: win.state },
+    window: geometry,
   };
 }
 
@@ -3687,7 +3710,13 @@ async function cmdManageDownloads({ action, url, filename, timeout = 30000, limi
     const opts = { url, saveAs: false };
     if (filename) opts.filename = filename;
     const id = await chrome.downloads.download(opts);
-    return { started: true, id, url };
+    // Non basta l'id: su ChromeOS l'impostazione «Chiedi dove salvare ogni file»
+    // scavalca saveAs:false e apre il selettore, quindi il download resta fermo
+    // ad aspettare una persona. Una breve occhiata allo stato reale distingue
+    // "partito" da "in attesa di te".
+    await new Promise((r) => setTimeout(r, 600));
+    const [item] = await chrome.downloads.search({ id });
+    return { id, url, ...classifyDownload(item) };
   }
 
   if (action === 'list') {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "chrome-bridge-mcp",
   "mcpName": "io.github.frsorrentino/chrome-bridge",
-  "version": "1.15.0",
+  "version": "1.15.1",
   "type": "module",
   "description": "MCP server with 63 tools for browser automation via Chrome extension. Works on ChromeOS/Crostini.",
   "main": "server/index.js",

package/server/tools.js

@@ -1349,7 +1349,10 @@ export function registerTools(server, wsManager, caps = 'all') {
     'tab_action',
     'Tab lifecycle: close, activate (focus), reload (optional cache bypass), back, forward. close discards the '
       + 'tab and anything unsaved in it, and cannot be undone — it may be a tab the user is working in. '
-      + 'reload and navigation drop injected CSS, emulations and page hooks.',
+      + 'reload and navigation drop injected CSS, emulations and page hooks. duplicate works where create_tab is '
+      + 'forbidden (chrome-untrusted://, verified on a ChromeOS Terminal tab) and lands in the source tab\'s own '
+      + 'window, app windows included — the only way to open a new Terminal session inside the Terminal window. '
+      + 'discard replaces the tab id: the result carries the new one, saved ids go stale.',
     {
       action: z.enum(['close', 'activate', 'reload', 'back', 'forward', 'discard', 'mute', 'unmute', 'duplicate']).describe('close cannot be undone; discard frees memory, the tab reloads on focus; reload drops injected CSS and hooks'),
       bypass_cache: z.boolean().optional().default(false).describe('reload only'),
@@ -1666,9 +1669,12 @@ export function registerTools(server, wsManager, caps = 'all') {
   // --- manage_downloads ---
   server.tool(
     'manage_downloads',
-    'List downloads or wait for one to complete. Files land in the browser Downloads folder, not on the server.',
+    'List downloads, start one, or wait for the newest to finish. Files land in the browser Downloads folder, '
+      + 'not on the server. action=download reports the download\'s real state rather than just an id: with '
+      + 'Chrome set to ask where to save each file, saveAs:false is overridden and it comes back as '
+      + 'waiting_for_user — every byte fetched, no destination chosen, and nothing will move until someone picks one.',
     {
-      action: z.enum(['list', 'wait_for_complete', 'download']).describe('download starts one with the browser cookie jar; wait_for_complete blocks until the newest finishes'),
+      action: z.enum(['list', 'wait_for_complete', 'download']).describe('download fetches with the browser cookie jar and reports the real state; wait_for_complete blocks until the newest finishes'),
       url: z.string().optional().describe('What to download (action=download); sent with the session cookies of its origin'),
       filename: z.string().optional().describe('Relative path inside the Downloads folder (action=download)'),
       timeout: z.number().optional().default(30000).describe('Max ms (wait_for_complete)'),
@@ -1757,25 +1763,30 @@ export function registerTools(server, wsManager, caps = 'all') {
   // --- move_tab ---
   server.tool(
     'move_tab',
-    'Move an existing tab into another window (chrome.tabs.move), e.g. to consolidate windows that were '
-      + 'each opened separately. Does not create or close anything: the tab keeps its id, its history and its '
-      + 'page state. Both windows must be normal browser windows — Chrome refuses to move a tab into a popup or '
-      + 'an app window, and the refusal is reported verbatim rather than retried.',
+    'Move an existing tab into another window (chrome.tabs.move). Nothing is created or closed: the tab keeps '
+      + 'its id, history and page state, and this works even on chrome-untrusted:// tabs where creating one is '
+      + 'forbidden — verified on a ChromeOS Terminal tab. The **destination** must be a normal window: moving out '
+      + 'of an app or popup window is fine, moving into one is refused with "Tabs can only be moved to and from '
+      + 'normal windows". So a ChromeOS Terminal session can be pulled out to its own window (new_window, then '
+      + 'position it with viewport_resize) but cannot be merged back into the Terminal window — the closest look '
+      + 'is window_type popup: no tab strip, no omnibox, visually a terminal window. An extension cannot create '
+      + 'app windows: normal and popup are all Chrome offers.',
     {
       tab_id: z.number().describe('Tab to move; get it from get_tabs'),
       window_id: z.number().optional().describe('Destination window; get_tabs reports windowId for every tab'),
       new_window: z.boolean().optional().default(false).describe('Extract the tab into a fresh window instead — tabs.move needs an existing window, this does not'),
+      window_type: z.enum(['normal', 'popup']).optional().default('normal').describe('new_window only: popup has no tab strip and no omnibox — the terminal-window look for detached chrome-untrusted://terminal tabs'),
       left: z.number().optional().describe('New window x (new_window)'),
       top: z.number().optional().describe('New window y (new_window)'),
       width: z.number().optional().describe('New window width px (new_window)'),
       height: z.number().optional().describe('New window height px (new_window)'),
       index: z.number().optional().default(-1).describe('Position in the destination window; -1 appends at the end'),
     },
-    async ({ tab_id, window_id, new_window, left, top, width, height, index }) => {
+    async ({ tab_id, window_id, new_window, window_type, left, top, width, height, index }) => {
       if (window_id == null && new_window !== true) {
         throw new Error('Provide window_id (an existing window) or new_window: true');
       }
-      const data = await send(MessageType.MOVE_TAB, { tab_id, window_id, new_window: new_window === true || undefined, left, top, width, height, index: index ?? -1 });
+      const data = await send(MessageType.MOVE_TAB, { tab_id, window_id, new_window: new_window === true || undefined, window_type, left, top, width, height, index: index ?? -1 });
       return { content: [{ type: 'text', text: jsonText(data) }] };
     }
   );