@evan_zhong/browser-attach 0.17.0 → 0.17.2

package/extension/connection.js

@@ -126,7 +126,7 @@ export function createConnection(chrome, CONNECTION, controller) {
       respond({ ok: true });
     }
     if (message.type === 'release_all') {
-      controller.releaseAll().then(() => respond({ ok: true }));
+      release().then(() => respond({ ok: true }), error => respond({ ok: false, error: error.message }));
       return true;
     }
   });

package/extension/controller.js

@@ -343,7 +343,7 @@ export function createController(api, notify = () => {}) {
       }
       const rendered = renderAX(nodes, token, refs, frame.id, target.sessionId, options);
       if (rendered.length || options.query == null) entries.push({ node: false, frameId: frame.id, line: `Frame ${frame.id} ${frame.url}` });
-      entries.push(...rendered);
+      for (const entry of rendered) entries.push(entry);
     }
     const metadata = value(await send(tabId, 'Runtime.evaluate', {
       expression: '({title: document.title, url: location.href})', returnByValue: true,

package/extension/manifest.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "BrowserAttach",
-  "version": "0.17.0",
+  "version": "0.17.2",
   "description": "Connect existing Chrome and Edge tabs to a locally paired MCP client.",
   "minimum_chrome_version": "125",
   "permissions": [

package/extension/popup.js

@@ -4,7 +4,13 @@ async function refresh() {
   document.querySelector('#profile').textContent = `${result.browser.family} · ${result.browser.profile_name}`;
   document.querySelector('#claims').textContent = `正在控制 ${result.claims} 个标签页`;
 }
-document.querySelector('#release').onclick = async () => { await chrome.runtime.sendMessage({ type: 'release_all' }); await refresh(); };
+document.querySelector('#release').onclick = async () => {
+  try {
+    const result = await chrome.runtime.sendMessage({ type: 'release_all' });
+    await refresh();
+    if (!result.ok) document.querySelector('#status').textContent = `释放失败：${result.error}`;
+  } catch (error) { document.querySelector('#status').textContent = `释放失败：${error.message}`; }
+};
 document.querySelector('#reconnect').onclick = async () => { await chrome.runtime.sendMessage({ type: 'reconnect' }); await refresh(); };
 document.querySelector('#settings').onclick = () => chrome.runtime.openOptionsPage();
 refresh().catch(error => { document.querySelector('#status').textContent = error.message; });

package/extension/snapshot.js

@@ -21,11 +21,15 @@ export function renderAX(nodes, token, refs, frameId, sessionId, options = snaps
     const parents = new Map();
     for (const node of nodes) for (const child of node.childIds || []) parents.set(child, node.nodeId);
     function keepText(node) {
-      if (!node || kept.has(node.nodeId)) return;
-      const role = node.role?.value;
-      if (!textRoles.has(role) && !node.ignored && !['none', 'generic'].includes(role)) return;
-      kept.add(node.nodeId);
-      for (const child of node.childIds || []) keepText(byId.get(child));
+      const pending = [node];
+      while (pending.length) {
+        const current = pending.pop();
+        if (!current || kept.has(current.nodeId)) continue;
+        const role = current.role?.value;
+        if (!textRoles.has(role) && !current.ignored && !['none', 'generic'].includes(role)) continue;
+        kept.add(current.nodeId);
+        for (const child of current.childIds || []) pending.push(byId.get(child));
+      }
     }
     for (const node of nodes) {
       if (node.ignored || !node.backendDOMNodeId || !(controls.has(node.role?.value) ||
@@ -37,19 +41,33 @@ export function renderAX(nodes, token, refs, frameId, sessionId, options = snaps
     }
   }
   function plainText(node) {
-    if (!node) return null;
-    const role = node.role?.value;
-    if (textRoles.has(role)) return String(node.name?.value || '');
-    if (!node.ignored && !['none', 'generic'].includes(role)) return null;
-    const parts = (node.childIds || []).map(id => plainText(byId.get(id)));
-    return parts.includes(null) ? null : parts.join(' ');
+    const pending = [node], parts = [];
+    while (pending.length) {
+      const current = pending.pop();
+      if (current === ' ') { parts.push(' '); continue; }
+      if (!current) return null;
+      const role = current.role?.value;
+      if (textRoles.has(role)) { parts.push(String(current.name?.value || '')); continue; }
+      if (!current.ignored && !['none', 'generic'].includes(role)) return null;
+      const children = current.childIds || [];
+      for (let index = children.length - 1; index >= 0; index--) {
+        pending.push(byId.get(children[index]));
+        if (index > 0) pending.push(' ');
+      }
+    }
+    return parts.join('');
   }
-  function render(node, depth, parentName = '') {
-    if (!node || visited.has(node.nodeId)) return;
+  // Preserve preorder and ref numbering without using the JavaScript call stack.
+  const children = new Set();
+  for (const node of nodes) for (const child of node.childIds || []) children.add(child);
+  const pending = nodes.filter(node => !children.has(node.nodeId)).reverse().map(node => ({ node, depth: 0, parentName: '' }));
+  while (pending.length) {
+    const { node, depth, parentName } = pending.pop();
+    if (!node || visited.has(node.nodeId)) continue;
     visited.add(node.nodeId);
-    if (kept && !kept.has(node.nodeId)) return;
+    if (kept && !kept.has(node.nodeId)) continue;
     const role = node.role?.value || 'node';
-    if (role === 'InlineTextBox') return;
+    if (role === 'InlineTextBox') continue;
     let name = String(node.name?.value || '');
     let merged = false;
     if (['listitem', 'paragraph'].includes(role) && !name) {
@@ -77,11 +95,13 @@ export function renderAX(nodes, token, refs, frameId, sessionId, options = snaps
       }
       nextDepth++;
     }
-    if (!merged) for (const child of node.childIds || []) render(byId.get(child), nextDepth, skip ? parentName : identityName);
+    if (!merged) {
+      const childIds = node.childIds || [];
+      for (let index = childIds.length - 1; index >= 0; index--) {
+        pending.push({ node: byId.get(childIds[index]), depth: nextDepth, parentName: skip ? parentName : identityName });
+      }
+    }
   }
-  // AX trees normally have a single root, but ignored roots can be disconnected.
-  const children = new Set(nodes.flatMap(node => node.childIds || []));
-  for (const node of nodes) if (!children.has(node.nodeId)) render(node, 0);
   return entries;
 }
 

package/extension/wait.js

@@ -70,7 +70,7 @@ export function inspectWait(query, inspect, args) {
   if (!element) return { satisfied: false, state: 'absent' };
   if (args.condition === 'attached') return { satisfied: true, state: 'attached' };
   if (args.condition === 'text') {
-    const text = element.textContent || '';
+    const text = element.innerText ?? '';
     return { satisfied: args.previous != null ? text !== args.previous : text.includes(args.expected), text: text.slice(0, 500) };
   }
   const bounds = inspect.call(element);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@evan_zhong/browser-attach",
-  "version": "0.17.0",
+  "version": "0.17.2",
   "description": "MCP for existing signed-in Chrome and Edge tabs through a locally paired extension",
   "type": "module",
   "license": "MIT",

package/src/config.mjs

@@ -15,7 +15,7 @@ export const DATA_DIR = configuredHome
   : process.platform === 'win32' ? join(homedir(), '.browser-attach')
     : join(process.env.XDG_DATA_HOME || join(homedir(), '.local', 'share'), 'browser-attach');
 export const DEFAULT_CONFIG = join(DATA_DIR, 'config.json');
-export const VERSION = '0.17.0';
+export const VERSION = '0.17.2';
 
 export async function loadConfig(path = DEFAULT_CONFIG) {
   let config;

package/src/mcp.mjs

@@ -147,7 +147,7 @@ export function createMcp(client) {
   }, null, args => { dragOptions(args); return args; });
   tool('wait', 'Wait without scrolling or input. actionable requires a stable, visible, enabled, unobscured target in the viewport; network_idle uses observed page requests. Timeout includes the last observed state.', {
     ...handle, ...dialogPolicy, timeout_ms: integerRange('timeout_ms', 100, 30000).default(5000), ...selector, ...frameScope, nth: target.nth,
-    condition: z.enum(['attached', 'visible', 'actionable', 'text', 'url', 'network_idle']), expected: z.string().max(2000).optional().describe('For text: textContent substring; for url: main-frame URL substring. Text may use previous instead.'), previous: z.string().max(100000).optional().describe('Text only: wait for full textContent to differ; alternative to expected.'), idle_ms: integerRange('idle_ms', 100, 10000).default(500),
+    condition: z.enum(['attached', 'visible', 'actionable', 'text', 'url', 'network_idle']), expected: z.string().max(2000).optional().describe('For text: rendered innerText substring; for url: main-frame URL substring. Text may use previous instead.'), previous: z.string().max(100000).optional().describe('Text only: wait for full rendered innerText to differ; alternative to expected.'), idle_ms: integerRange('idle_ms', 100, 10000).default(500),
   }, null, args => { waitOptions(args); return args; });
   tool('select_option', 'Set one HTML select option without native UI, using synthetic input/change (events_trusted=false). Verifies retained selection only.', {
     ...handle, ...inputTimeout, ...target, value: z.string().optional().describe('Choose exactly one of value, label or zero-based index; value/label match exactly.'), label: z.string().optional(), index: integerRange('index', 0, Number.MAX_SAFE_INTEGER).optional(),