@browserbridge/bbx 1.0.0 → 1.0.1

package/packages/protocol/src/summary.js

@@ -1,9 +1,6 @@
 // @ts-check
 
-import {
-  estimateJsonPayloadCost,
-  getCostClass,
-} from './index.js';
+import { estimateJsonPayloadCost, getCostClass } from './index.js';
 
 /** @typedef {import('./types.js').BridgeResponse} BridgeResponse */
 /** @typedef {import('./types.js').BridgeMethod} SummaryMethod */
@@ -53,16 +50,44 @@ import {
 
 /** @type {Record<string, ActionSummary>} */
 const ACTION_SUMMARIES = {
-  hovered:   { text: r => `Hover ${r.hovered ? 'active' : 'failed'} on ${r.elementRef}.`, evidence: r => r },
-  dragged:   { text: r => `Drag ${r.dragged ? 'completed' : 'failed'}: ${r.sourceRef} → ${r.destinationRef}.`, evidence: r => r },
-  closed:    { text: r => `Tab ${r.tabId} closed.`, evidence: r => r },
-  clicked:   { text: r => `Clicked ${r.elementRef ?? 'element'}.`, evidence: r => ({ elementRef: r.elementRef }) },
-  focused:   { text: r => `Focused ${r.elementRef ?? 'element'}.`, evidence: r => ({ elementRef: r.elementRef }) },
-  typed:     { text: r => `Typed into ${r.elementRef ?? 'element'}.`, evidence: r => ({ elementRef: r.elementRef }) },
-  pressed:   { text: r => `Key pressed${r.key ? ` (${r.key})` : ''}.`, evidence: r => r },
-  navigated: { text: r => `Navigated to ${r.url ?? 'page'}.`, evidence: r => ({ url: r.url }) },
-  scrolled:  { text: r => `Scrolled to (${r.x ?? 0}, ${r.y ?? 0}).`, evidence: r => r },
-  resized:   { text: r => `Viewport resized to ${r.width ?? '?'}×${r.height ?? '?'}.`, evidence: r => r },
+  hovered: {
+    text: (r) => `Hover ${r.hovered ? 'active' : 'failed'} on ${r.elementRef}.`,
+    evidence: (r) => r,
+  },
+  dragged: {
+    text: (r) =>
+      `Drag ${r.dragged ? 'completed' : 'failed'}: ${r.sourceRef} → ${r.destinationRef}.`,
+    evidence: (r) => r,
+  },
+  closed: { text: (r) => `Tab ${r.tabId} closed.`, evidence: (r) => r },
+  clicked: {
+    text: (r) => `Clicked ${r.elementRef ?? 'element'}.`,
+    evidence: (r) => ({ elementRef: r.elementRef }),
+  },
+  focused: {
+    text: (r) => `Focused ${r.elementRef ?? 'element'}.`,
+    evidence: (r) => ({ elementRef: r.elementRef }),
+  },
+  typed: {
+    text: (r) => `Typed into ${r.elementRef ?? 'element'}.`,
+    evidence: (r) => ({ elementRef: r.elementRef }),
+  },
+  pressed: {
+    text: (r) => `Key pressed${r.key ? ` (${r.key})` : ''}.`,
+    evidence: (r) => r,
+  },
+  navigated: {
+    text: (r) => `Navigated to ${r.url ?? 'page'}.`,
+    evidence: (r) => ({ url: r.url }),
+  },
+  scrolled: {
+    text: (r) => `Scrolled to (${r.x ?? 0}, ${r.y ?? 0}).`,
+    evidence: (r) => r,
+  },
+  resized: {
+    text: (r) => `Viewport resized to ${r.width ?? '?'}×${r.height ?? '?'}.`,
+    evidence: (r) => r,
+  },
 };
 
 /**
@@ -74,21 +99,24 @@ const ACTION_SUMMARIES = {
  * @returns {AnnotatedBridgeSummary}
  */
 export function annotateBridgeSummary(summary, response) {
-  const transportBytes = getNumericMetaField(response.meta, 'transport_bytes')
-    ?? getNumericMetaField(response.meta, 'response_bytes')
-    ?? estimateJsonPayloadCost(response.ok ? response.result : { error: response.error }).bytes;
-  const transportTokens = getNumericMetaField(response.meta, 'transport_approx_tokens')
-    ?? getNumericMetaField(response.meta, 'approx_tokens')
-    ?? estimateJsonPayloadCost(response.ok ? response.result : { error: response.error }).approxTokens;
+  const transportBytes =
+    getNumericMetaField(response.meta, 'transport_bytes') ??
+    getNumericMetaField(response.meta, 'response_bytes') ??
+    estimateJsonPayloadCost(response.ok ? response.result : { error: response.error }).bytes;
+  const transportTokens =
+    getNumericMetaField(response.meta, 'transport_approx_tokens') ??
+    getNumericMetaField(response.meta, 'approx_tokens') ??
+    estimateJsonPayloadCost(response.ok ? response.result : { error: response.error }).approxTokens;
   const summaryCost = estimateJsonPayloadCost(summary);
 
   return {
     ...summary,
     transportBytes,
     transportTokens,
-    transportCostClass: getMetaCostClass(response.meta, 'transport_cost_class')
-      ?? getMetaCostClass(response.meta, 'cost_class')
-      ?? getCostClass(transportTokens),
+    transportCostClass:
+      getMetaCostClass(response.meta, 'transport_cost_class') ??
+      getMetaCostClass(response.meta, 'cost_class') ??
+      getCostClass(transportTokens),
     summaryBytes: summaryCost.bytes,
     summaryTokens: summaryCost.approxTokens,
     summaryCostClass: summaryCost.costClass,
@@ -132,28 +160,32 @@ export function summarizeBridgeResponse(response, method) {
         `${response.error.code}: ${response.error.message}${hint ? ` ${hint}` : ''}`,
         protocolWarning
       ),
-      evidence: response.error.details ?? null
+      evidence: response.error.details ?? null,
     };
   }
 
   const result = toRecord(response.result);
   if (typeof result.daemon === 'string') {
-    const access = result.access && typeof result.access === 'object'
-      ? /** @type {Record<string, unknown>} */ (result.access)
-      : null;
-    const accessSummary = access == null
-      ? ''
-      : access.enabled
-        ? ` Access: ${access.routeReady ? `ready on tab ${access.routeTabId}.` : `enabled${typeof access.reason === 'string' ? ` (${access.reason})` : '.'}`}`
-        : ' Access: disabled.';
+    const access =
+      result.access && typeof result.access === 'object'
+        ? /** @type {Record<string, unknown>} */ (result.access)
+        : null;
+    const accessSummary =
+      access == null
+        ? ''
+        : access.enabled
+          ? ` Access: ${access.routeReady ? `ready on tab ${access.routeTabId}.` : `enabled${typeof access.reason === 'string' ? ` (${access.reason})` : '.'}`}`
+          : ' Access: disabled.';
     const connectedExtensions = Array.isArray(result.connectedExtensions)
       ? /** @type {Array<Record<string, unknown>>} */ (result.connectedExtensions)
       : [];
     const extensionSummary = result.extensionConnected
-      ? `connected (${connectedExtensions.length}: ${connectedExtensions.map((ext) => {
-        const label = `${ext.browserName ?? 'unknown'}${ext.profileLabel ? '/' + ext.profileLabel : ''}`;
-        return ext.accessEnabled ? `${label}*` : label;
-      }).join(', ')})`
+      ? `connected (${connectedExtensions.length}: ${connectedExtensions
+          .map((ext) => {
+            const label = `${ext.browserName ?? 'unknown'}${ext.profileLabel ? '/' + ext.profileLabel : ''}`;
+            return ext.accessEnabled ? `${label}*` : label;
+          })
+          .join(', ')})`
       : 'disconnected';
     return {
       ok: true,
@@ -161,22 +193,36 @@ export function summarizeBridgeResponse(response, method) {
         `Daemon: ${result.daemon}. Extension: ${extensionSummary}.${accessSummary}`,
         protocolWarning
       ),
-      evidence: result
+      evidence: result,
     };
   }
   if (Array.isArray(result.mcpClients) && Array.isArray(result.skillTargets)) {
-    const configuredMcp = result.mcpClients.filter((entry) => entry && typeof entry === 'object' && /** @type {Record<string, unknown>} */ (entry).configured).length;
-    const installedSkills = result.skillTargets.filter((entry) => entry && typeof entry === 'object' && /** @type {Record<string, unknown>} */ (entry).installed).length;
+    const configuredMcp = result.mcpClients.filter(
+      (entry) =>
+        entry &&
+        typeof entry === 'object' &&
+        /** @type {Record<string, unknown>} */ (entry).configured
+    ).length;
+    const installedSkills = result.skillTargets.filter(
+      (entry) =>
+        entry &&
+        typeof entry === 'object' &&
+        /** @type {Record<string, unknown>} */ (entry).installed
+    ).length;
     return {
       ok: true,
       summary: appendProtocolWarning(
         `Setup: MCP configured for ${configuredMcp}/${result.mcpClients.length} clients; skill installed for ${installedSkills}/${result.skillTargets.length} targets.`,
         protocolWarning
       ),
-      evidence: result
+      evidence: result,
     };
   }
-  if (typeof result.url === 'string' && typeof result.title === 'string' && typeof result.origin === 'string') {
+  if (
+    typeof result.url === 'string' &&
+    typeof result.title === 'string' &&
+    typeof result.origin === 'string'
+  ) {
     /** @type {string[]} */
     const hints = [];
     if (result.hints && typeof result.hints === 'object') {
@@ -190,10 +236,18 @@ export function summarizeBridgeResponse(response, method) {
         `Page: ${result.title} (${result.origin})${hints.length ? ` [${hints.join(', ')}]` : ''}.`,
         protocolWarning
       ),
-      evidence: { url: result.url, origin: result.origin, title: result.title, hints: result.hints }
+      evidence: {
+        url: result.url,
+        origin: result.origin,
+        title: result.title,
+        hints: result.hints,
+      },
     };
   }
-  if ((typeof result.text === 'string' || typeof result.value === 'string') && typeof result.truncated === 'boolean') {
+  if (
+    (typeof result.text === 'string' || typeof result.value === 'string') &&
+    typeof result.truncated === 'boolean'
+  ) {
     const text = /** @type {string} */ (result.text ?? result.value);
     const len = typeof result.length === 'number' ? result.length : text.length;
     const label = method === 'dom.get_text' ? 'Element text' : 'Page text';
@@ -203,7 +257,11 @@ export function summarizeBridgeResponse(response, method) {
         `${label}: ${len} chars${result.truncated ? ' (truncated)' : ''}.`,
         protocolWarning
       ),
-      evidence: { text: text.slice(0, 500), length: len, truncated: result.truncated }
+      evidence: {
+        text: text.slice(0, 500),
+        length: len,
+        truncated: result.truncated,
+      },
     };
   }
   if (Array.isArray(result.tabs)) {
@@ -215,14 +273,24 @@ export function summarizeBridgeResponse(response, method) {
         tabId: tab.tabId,
         active: tab.active,
         origin: tab.origin,
-        title: tab.title
-      }))
+        title: tab.title,
+      })),
     };
   }
-  if (Array.isArray(result.nodes) && typeof result.total === 'number' && result.nodes[0]?.role !== undefined) {
+  if (
+    Array.isArray(result.nodes) &&
+    typeof result.total === 'number' &&
+    result.nodes[0]?.role !== undefined
+  ) {
     const nodes = /** @type {Array<Record<string, unknown>>} */ (result.nodes);
     const interactive = nodes.filter((/** @type {Record<string, unknown>} */ n) => n.interactive);
-    const shown = interactive.length > 0 ? interactive : nodes.filter((/** @type {Record<string, unknown>} */ n) => n.role && n.role !== 'none' && n.role !== 'generic');
+    const shown =
+      interactive.length > 0
+        ? interactive
+        : nodes.filter(
+            (/** @type {Record<string, unknown>} */ n) =>
+              n.role && n.role !== 'none' && n.role !== 'generic'
+          );
     return {
       ok: true,
       summary: appendProtocolWarning(
@@ -235,7 +303,7 @@ export function summarizeBridgeResponse(response, method) {
         if (n.interactive) entry.interactive = true;
         if (n.value) entry.value = n.value;
         return entry;
-      })
+      }),
     };
   }
   if (Array.isArray(result.nodes)) {
@@ -247,14 +315,20 @@ export function summarizeBridgeResponse(response, method) {
       if (typeof n.attrs === 'object' && n.attrs !== null) {
         const attrs = /** @type {Record<string, unknown>} */ (n.attrs);
         if (attrs.id) entry.id = attrs.id;
-        if (typeof attrs.class === 'string') entry.cls = attrs.class.split(' ').slice(0, 3).join(' ');
+        if (typeof attrs.class === 'string')
+          entry.cls = attrs.class.split(' ').slice(0, 3).join(' ');
         if (attrs.role) entry.role = attrs.role;
         if (attrs['aria-label']) entry.label = attrs['aria-label'];
         if (attrs['data-testid']) entry.testId = attrs['data-testid'];
       }
       if (!entry.role && n.role) entry.role = n.role;
       if (!entry.label && n.name) entry.label = n.name;
-      const text = typeof n.textExcerpt === 'string' ? n.textExcerpt : typeof n.text === 'string' ? n.text : '';
+      const text =
+        typeof n.textExcerpt === 'string'
+          ? n.textExcerpt
+          : typeof n.text === 'string'
+            ? n.text
+            : '';
       if (text) {
         entry.text = text.length > 80 ? `${text.slice(0, 79)}\u2026` : text;
       }
@@ -275,21 +349,21 @@ export function summarizeBridgeResponse(response, method) {
         `${label} ${nodes.length} element(s)${nodes.length > 15 ? '; showing first 15' : ''}.`,
         protocolWarning
       ),
-      evidence: compact
+      evidence: compact,
     };
   }
   if (typeof result.rolledBack === 'boolean' || typeof result.rolled_back === 'boolean') {
     return {
       ok: true,
       summary: appendProtocolWarning('Patch rolled back.', protocolWarning),
-      evidence: result
+      evidence: result,
     };
   }
   if (typeof result.patchId === 'string') {
     return {
       ok: true,
       summary: appendProtocolWarning(`Patch ${result.patchId} applied.`, protocolWarning),
-      evidence: result
+      evidence: result,
     };
   }
   if (typeof result.found === 'boolean') {
@@ -301,7 +375,10 @@ export function summarizeBridgeResponse(response, method) {
           : `Element not found (timed out after ${result.duration ?? 0}ms).`,
         protocolWarning
       ),
-      evidence: { elementRef: result.elementRef ?? null, duration: result.duration }
+      evidence: {
+        elementRef: result.elementRef ?? null,
+        duration: result.duration,
+      },
     };
   }
   if (typeof result.value !== 'undefined' && typeof result.type === 'string') {
@@ -313,7 +390,11 @@ export function summarizeBridgeResponse(response, method) {
       repr = '';
     } else if (typeof result.value === 'string') {
       repr = result.value.length > 200 ? `${result.value.slice(0, 199)}\u2026` : result.value;
-    } else if (typeof result.value === 'object' && result.value !== null && Object.keys(result.value).length === 0) {
+    } else if (
+      typeof result.value === 'object' &&
+      result.value !== null &&
+      Object.keys(result.value).length === 0
+    ) {
       repr = '(empty - may be a Promise, Map, or non-serializable value)';
     } else {
       repr = JSON.stringify(result.value);
@@ -325,20 +406,33 @@ export function summarizeBridgeResponse(response, method) {
         repr ? `Evaluated to ${typeLabel}: ${repr}` : `Evaluated to ${typeLabel}.`,
         protocolWarning
       ),
-      evidence: result
+      evidence: result,
     };
   }
-  if (Array.isArray(result.entries) && result.entries.length > 0 && typeof result.entries[0]?.at === 'string' && typeof result.entries[0]?.method === 'string') {
+  if (
+    Array.isArray(result.entries) &&
+    result.entries.length > 0 &&
+    typeof result.entries[0]?.at === 'string' &&
+    typeof result.entries[0]?.method === 'string'
+  ) {
     const entries = /** @type {Array<Record<string, unknown>>} */ (result.entries);
     return {
       ok: true,
       summary: appendProtocolWarning(`Log: ${entries.length} entries.`, protocolWarning),
       evidence: entries.slice(-10).map((/** @type {Record<string, unknown>} */ e) => ({
-        at: e.at, method: e.method, ok: e.ok, ...(typeof e.source === 'string' && e.source ? { source: e.source } : {})
-      }))
+        at: e.at,
+        method: e.method,
+        ok: e.ok,
+        ...(typeof e.source === 'string' && e.source ? { source: e.source } : {}),
+      })),
     };
   }
-  if (Array.isArray(result.entries) && (result.entries.length > 0 ? (result.entries[0]?.type === 'fetch' || result.entries[0]?.type === 'xhr') : method === 'page.get_network')) {
+  if (
+    Array.isArray(result.entries) &&
+    (result.entries.length > 0
+      ? result.entries[0]?.type === 'fetch' || result.entries[0]?.type === 'xhr'
+      : method === 'page.get_network')
+  ) {
     const entries = /** @type {Array<Record<string, unknown>>} */ (result.entries);
     return {
       ok: true,
@@ -347,8 +441,11 @@ export function summarizeBridgeResponse(response, method) {
         protocolWarning
       ),
       evidence: entries.slice(0, 20).map((/** @type {Record<string, unknown>} */ e) => ({
-        method: e.method, url: truncateUrl(/** @type {string} */ (e.url)), status: e.status, duration: e.duration
-      }))
+        method: e.method,
+        url: truncateUrl(/** @type {string} */ (e.url)),
+        status: e.status,
+        duration: e.duration,
+      })),
     };
   }
   if (Array.isArray(result.entries)) {
@@ -366,7 +463,7 @@ export function summarizeBridgeResponse(response, method) {
           entry.ago = formatRelativeTime(e.ts);
         }
         return entry;
-      })
+      }),
     };
   }
   if (typeof result.html === 'string') {
@@ -376,7 +473,10 @@ export function summarizeBridgeResponse(response, method) {
         `HTML fragment: ${result.html.length} chars${result.truncated ? ' (truncated)' : ''}.`,
         protocolWarning
       ),
-      evidence: { html: result.html.slice(0, 500), truncated: result.truncated }
+      evidence: {
+        html: result.html.slice(0, 500),
+        truncated: result.truncated,
+      },
     };
   }
   for (const [field, handler] of Object.entries(ACTION_SUMMARIES)) {
@@ -384,13 +484,12 @@ export function summarizeBridgeResponse(response, method) {
       return {
         ok: true,
         summary: appendProtocolWarning(handler.text(result), protocolWarning),
-        evidence: handler.evidence(result)
+        evidence: handler.evidence(result),
       };
     }
   }
   if (typeof result.tabId === 'number' && typeof result.url === 'string') {
-    const actionMethod =
-      typeof result.method === 'string' ? result.method : method;
+    const actionMethod = typeof result.method === 'string' ? result.method : method;
     if (actionMethod === 'navigation.navigate') {
       return {
         ok: true,
@@ -405,16 +504,22 @@ export function summarizeBridgeResponse(response, method) {
     }
     return {
       ok: true,
-      summary: appendProtocolWarning(`Tab ${result.tabId} created${result.url ? ` (${result.url})` : ''}.`, protocolWarning),
-      evidence: result
+      summary: appendProtocolWarning(
+        `Tab ${result.tabId} created${result.url ? ` (${result.url})` : ''}.`,
+        protocolWarning
+      ),
+      evidence: result,
     };
   }
   if (typeof result.metrics === 'object' && result.metrics !== null) {
     const keys = Object.keys(result.metrics);
     return {
       ok: true,
-      summary: appendProtocolWarning(`Performance: ${keys.length} metrics collected.`, protocolWarning),
-      evidence: result.metrics
+      summary: appendProtocolWarning(
+        `Performance: ${keys.length} metrics collected.`,
+        protocolWarning
+      ),
+      evidence: result.metrics,
     };
   }
   if (typeof result.count === 'number' && typeof result.type === 'string' && result.entries) {
@@ -427,16 +532,26 @@ export function summarizeBridgeResponse(response, method) {
     }
     return {
       ok: true,
-      summary: appendProtocolWarning(`Storage (${result.type}): ${result.count} entries.`, protocolWarning),
-      evidence: safe
+      summary: appendProtocolWarning(
+        `Storage (${result.type}): ${result.count} entries.`,
+        protocolWarning
+      ),
+      evidence: safe,
     };
   }
-  if (typeof result.tag === 'string' && typeof result.elementRef === 'string' && typeof result.bbox === 'object') {
+  if (
+    typeof result.tag === 'string' &&
+    typeof result.elementRef === 'string' &&
+    typeof result.bbox === 'object'
+  ) {
     const desc = [result.tag];
     if (result.id) desc[0] += `#${result.id}`;
-    const textValue = typeof result.text === 'object' && result.text !== null && 'value' in result.text
-      ? /** @type {{ value: string }} */ (result.text).value
-      : typeof result.text === 'string' ? result.text : '';
+    const textValue =
+      typeof result.text === 'object' && result.text !== null && 'value' in result.text
+        ? /** @type {{ value: string }} */ (result.text).value
+        : typeof result.text === 'string'
+          ? result.text
+          : '';
     if (textValue) desc.push(String(textValue).slice(0, 60));
     const bbox = /** @type {Record<string, number>} */ (result.bbox);
     if (bbox.width && bbox.height) {
@@ -445,15 +560,29 @@ export function summarizeBridgeResponse(response, method) {
     return {
       ok: true,
       summary: appendProtocolWarning(`Element ${desc.join(', ')}.`, protocolWarning),
-      evidence: { elementRef: result.elementRef, tag: result.tag, id: result.id, role: result.role, text: textValue, bbox: result.bbox }
+      evidence: {
+        elementRef: result.elementRef,
+        tag: result.tag,
+        id: result.id,
+        role: result.role,
+        text: textValue,
+        bbox: result.bbox,
+      },
     };
   }
-  if (typeof result.properties === 'object' && result.properties !== null && typeof result.elementRef === 'string') {
+  if (
+    typeof result.properties === 'object' &&
+    result.properties !== null &&
+    typeof result.elementRef === 'string'
+  ) {
     const props = Object.keys(/** @type {object} */ (result.properties));
     return {
       ok: true,
-      summary: appendProtocolWarning(`Computed ${props.length} style(s) for ${result.elementRef}.`, protocolWarning),
-      evidence: result.properties
+      summary: appendProtocolWarning(
+        `Computed ${props.length} style(s) for ${result.elementRef}.`,
+        protocolWarning
+      ),
+      evidence: result.properties,
     };
   }
   if (method === 'styles.get_computed') {
@@ -461,23 +590,41 @@ export function summarizeBridgeResponse(response, method) {
     return {
       ok: true,
       summary: appendProtocolWarning(`Computed ${props.length} style(s).`, protocolWarning),
-      evidence: result
+      evidence: result,
     };
   }
   if (typeof result.content === 'object' && typeof result.border === 'object') {
     const c = /** @type {Record<string, number>} */ (result.content);
     return {
       ok: true,
-      summary: appendProtocolWarning(`Box model: ${c.width ?? '?'}×${c.height ?? '?'} at (${c.x ?? 0}, ${c.y ?? 0}).`, protocolWarning),
-      evidence: result
+      summary: appendProtocolWarning(
+        `Box model: ${c.width ?? '?'}×${c.height ?? '?'} at (${c.x ?? 0}, ${c.y ?? 0}).`,
+        protocolWarning
+      ),
+      evidence: result,
     };
   }
-  if (typeof result.x === 'number' && typeof result.y === 'number' && typeof result.width === 'number' && typeof result.height === 'number' &&
-      !('clicked' in result || 'hovered' in result || 'focused' in result || 'resized' in result || 'tag' in result || 'elementRef' in result)) {
+  if (
+    typeof result.x === 'number' &&
+    typeof result.y === 'number' &&
+    typeof result.width === 'number' &&
+    typeof result.height === 'number' &&
+    !(
+      'clicked' in result ||
+      'hovered' in result ||
+      'focused' in result ||
+      'resized' in result ||
+      'tag' in result ||
+      'elementRef' in result
+    )
+  ) {
     return {
       ok: true,
-      summary: appendProtocolWarning(`Box model: ${result.width}×${result.height} at (${result.x}, ${result.y}).`, protocolWarning),
-      evidence: result
+      summary: appendProtocolWarning(
+        `Box model: ${result.width}×${result.height} at (${result.x}, ${result.y}).`,
+        protocolWarning
+      ),
+      evidence: result,
     };
   }
   if (Array.isArray(response.result) && looksLikePatchArray(response.result, method)) {
@@ -485,21 +632,24 @@ export function summarizeBridgeResponse(response, method) {
     return {
       ok: true,
       summary: appendProtocolWarning(`${patches.length} active patch(es).`, protocolWarning),
-      evidence: patches.slice(0, 10)
+      evidence: patches.slice(0, 10),
     };
   }
   if (Array.isArray(result.patches)) {
     return {
       ok: true,
       summary: appendProtocolWarning(`${result.patches.length} active patch(es).`, protocolWarning),
-      evidence: result.patches.slice(0, 10)
+      evidence: result.patches.slice(0, 10),
     };
   }
   const keys = Object.keys(result);
   return {
     ok: true,
-    summary: appendProtocolWarning(`Bridge method succeeded with ${keys.length} top-level fields.`, protocolWarning),
-    evidence: keys.slice(0, 10)
+    summary: appendProtocolWarning(
+      `Bridge method succeeded with ${keys.length} top-level fields.`,
+      protocolWarning
+    ),
+    evidence: keys.slice(0, 10),
   };
 }
 
@@ -518,7 +668,7 @@ export function summarizeBatchResponseItem({ method, tabId, response, durationMs
     approxTokens: cost.approxTokens,
     meta: response.meta,
     error: response.ok ? null : response.error,
-    response: response.ok ? response.result : null
+    response: response.ok ? response.result : null,
   };
 }
 
@@ -535,9 +685,9 @@ export function summarizeBatchErrorItem({ method, tabId, error, durationMs }) {
     error: {
       code: 'INTERNAL_ERROR',
       message,
-      details: null
+      details: null,
     },
-    meta: { protocol_version: '1.0' }
+    meta: { protocol_version: '1.0' },
   });
   const summary = annotateBridgeSummary(summarizeBridgeResponse(response, method), response);
   return {
@@ -548,7 +698,7 @@ export function summarizeBatchErrorItem({ method, tabId, error, durationMs }) {
     approxTokens: 0,
     meta: response.meta,
     error: response.error,
-    response: null
+    response: null,
   };
 }
 
@@ -557,9 +707,7 @@ export function summarizeBatchErrorItem({ method, tabId, error, durationMs }) {
  * @returns {Record<string, unknown>}
  */
 function toRecord(value) {
-  return value && typeof value === 'object'
-    ? /** @type {Record<string, unknown>} */ (value)
-    : {};
+  return value && typeof value === 'object' ? /** @type {Record<string, unknown>} */ (value) : {};
 }
 
 /**
@@ -623,12 +771,7 @@ function looksLikePatchArray(arr, method) {
   if (!Array.isArray(arr)) return false;
   if (arr.length === 0) return method === 'patch.list';
   const first = arr[0];
-  return (
-    first !== null &&
-    typeof first === 'object' &&
-    'patchId' in first &&
-    'kind' in first
-  );
+  return first !== null && typeof first === 'object' && 'patchId' in first && 'kind' in first;
 }
 
 /**

package/packages/protocol/src/types.js

@@ -38,7 +38,7 @@ export {};
  *   budget_applied?: boolean,
  *   budget_truncated?: boolean,
  *   continuation_hint?: string | null,
-   *   [key: string]: unknown
+ *   [key: string]: unknown
  * }} BridgeMeta
  */
 

package/skills/browser-bridge/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: browser-bridge
-description: "Token-efficient Chrome tab inspection, interaction, and patching via local bridge extension (CLI: bbx). Reads live DOM, styles, console, network, and storage from a real Chrome tab with lower token cost than screenshots."
+description: 'Token-efficient Chrome tab inspection, interaction, and patching via local bridge extension (CLI: bbx). Reads live DOM, styles, console, network, and storage from a real Chrome tab with lower token cost than screenshots.'
 ---
 
 # Browser Bridge

package/skills/browser-bridge/agents/openai.yaml

@@ -1,4 +1,4 @@
 interface:
-  display_name: "Browser Bridge"
-  short_description: "Token-efficient Chrome tab inspection, interaction, and patching via local bridge"
-  default_prompt: "Use the browser-bridge skill for Chrome tab inspection, interaction, and patching. Start with `bbx status` to confirm connectivity. If ACCESS_DENIED, ask the user to click Enable in the extension popup/side panel, then retry. Default routing follows the active tab; use `--tab` only for a different tab. Prefer structured reads (`dom.query`, `styles.get_computed`) over screenshots. Batch independent reads with `bbx batch`. Use `bbx skill` for runtime presets."
+  display_name: 'Browser Bridge'
+  short_description: 'Token-efficient Chrome tab inspection, interaction, and patching via local bridge'
+  default_prompt: 'Use the browser-bridge skill for Chrome tab inspection, interaction, and patching. Start with `bbx status` to confirm connectivity. If ACCESS_DENIED, ask the user to click Enable in the extension popup/side panel, then retry. Default routing follows the active tab; use `--tab` only for a different tab. Prefer structured reads (`dom.query`, `styles.get_computed`) over screenshots. Batch independent reads with `bbx batch`. Use `bbx skill` for runtime presets.'