@browserbridge/bbx 1.1.0 → 1.3.0

package/packages/native-host/src/framing.js

@@ -24,6 +24,19 @@ export async function writeNativeMessage(stream, message) {
   }
 }
 
+/**
+ * @param {NodeJS.WritableStream} stream
+ * @returns {(message: unknown) => Promise<void>}
+ */
+export function createNativeMessageWriter(stream) {
+  let queue = Promise.resolve();
+  return (message) => {
+    const writePromise = queue.then(() => writeNativeMessage(stream, message));
+    queue = writePromise.catch(() => {});
+    return writePromise;
+  };
+}
+
 /**
  * @param {NodeJS.ReadableStream} stream
  * @param {(message: unknown) => void} onMessage

package/packages/native-host/src/native-host.js

@@ -5,7 +5,7 @@ import net from 'node:net';
 import { createFailure, ERROR_CODES } from '../../protocol/src/index.js';
 import { createSocketBridgeTransport, getBridgeTransport } from './config.js';
 import { spawnBridgeDaemonProcess } from './daemon-process.js';
-import { createNativeMessageReader, writeJsonLine, writeNativeMessage } from './framing.js';
+import { createNativeMessageReader, createNativeMessageWriter, writeJsonLine } from './framing.js';
 
 /** @typedef {import('./config.js').BridgeTransport} BridgeTransport */
 
@@ -117,11 +117,13 @@ export async function runNativeHost({
   socketPath = undefined,
 } = {}) {
   const resolvedTransport = socketPath ? createSocketBridgeTransport(socketPath) : transport;
+  const writeNativeMessageQueued = createNativeMessageWriter(process.stdout);
+
   let socket;
   try {
     socket = await connectWithBootstrap(resolvedTransport);
   } catch (error) {
-    await writeNativeMessage(process.stdout, {
+    await writeNativeMessageQueued({
       type: 'agent.response',
       response: createFailure(
         'native_bootstrap',
@@ -164,11 +166,11 @@ export async function runNativeHost({
       }
       void (async () => {
         if (message.type === 'extension.request') {
-          await writeNativeMessage(process.stdout, message.request);
+          await writeNativeMessageQueued(message.request);
           return;
         }
         if (message.type === 'agent.response') {
-          await writeNativeMessage(process.stdout, {
+          await writeNativeMessageQueued({
             type: 'host.bridge_response',
             response: message.response,
           });
@@ -178,7 +180,7 @@ export async function runNativeHost({
           message.type === 'extension.setup_status.response' ||
           message.type === 'extension.setup_status.error'
         ) {
-          await writeNativeMessage(process.stdout, {
+          await writeNativeMessageQueued({
             type:
               message.type === 'extension.setup_status.response'
                 ? 'host.setup_status.response'

package/packages/protocol/src/capabilities.js

@@ -105,6 +105,7 @@ export const METHOD_CAPABILITIES = Object.freeze({
   'performance.get_metrics': CAPABILITIES.PERFORMANCE_READ,
   'log.tail': null,
   'health.ping': null,
+  'daemon.metrics': null,
 });
 
 /**

package/packages/protocol/src/protocol.js

@@ -7,6 +7,7 @@ import {
   DEFAULT_A11Y_MAX_NODES,
   DEFAULT_CONSOLE_LIMIT,
   DEFAULT_EVAL_TIMEOUT_MS,
+  DEFAULT_LOG_TAIL_LIMIT,
   DEFAULT_MAX_DEPTH,
   DEFAULT_MAX_HTML_LENGTH,
   DEFAULT_MAX_NODES,
@@ -29,6 +30,7 @@ import { BRIDGE_METHODS, METHOD_SET, createBridgeMethodGroups } from './registry
 /** @typedef {import('./types.js').BridgeSuccessResponse} BridgeSuccessResponse */
 /** @typedef {import('./types.js').CheckedActionParams} CheckedActionParams */
 /** @typedef {import('./types.js').CdpDispatchKeyEventParams} CdpDispatchKeyEventParams */
+/** @typedef {import('./types.js').CdpNodeIdParams} CdpNodeIdParams */
 /** @typedef {import('./types.js').ConsoleParams} ConsoleParams */
 /** @typedef {import('./types.js').DomQueryParams} DomQueryParams */
 /** @typedef {import('./types.js').DragParams} DragParams */
@@ -38,11 +40,13 @@ import { BRIDGE_METHODS, METHOD_SET, createBridgeMethodGroups } from './registry
 /** @typedef {import('./types.js').GetHtmlParams} GetHtmlParams */
 /** @typedef {import('./types.js').HoverParams} HoverParams */
 /** @typedef {import('./types.js').InputActionParams} InputActionParams */
+/** @typedef {import('./types.js').LogTailParams} LogTailParams */
 /** @typedef {import('./types.js').NavigationActionParams} NavigationActionParams */
 /** @typedef {import('./types.js').NetworkParams} NetworkParams */
 /** @typedef {import('./types.js').NormalizedAccessibilityTreeParams} NormalizedAccessibilityTreeParams */
 /** @typedef {import('./types.js').NormalizedCheckedAction} NormalizedCheckedAction */
 /** @typedef {import('./types.js').NormalizedCdpDispatchKeyEventParams} NormalizedCdpDispatchKeyEventParams */
+/** @typedef {import('./types.js').NormalizedCdpNodeIdParams} NormalizedCdpNodeIdParams */
 /** @typedef {import('./types.js').NormalizedConsoleParams} NormalizedConsoleParams */
 /** @typedef {import('./types.js').NormalizedDomQuery} NormalizedDomQuery */
 /** @typedef {import('./types.js').NormalizedDragParams} NormalizedDragParams */
@@ -52,6 +56,7 @@ import { BRIDGE_METHODS, METHOD_SET, createBridgeMethodGroups } from './registry
 /** @typedef {import('./types.js').NormalizedGetHtmlParams} NormalizedGetHtmlParams */
 /** @typedef {import('./types.js').NormalizedHoverParams} NormalizedHoverParams */
 /** @typedef {import('./types.js').NormalizedInputAction} NormalizedInputAction */
+/** @typedef {import('./types.js').NormalizedLogTailParams} NormalizedLogTailParams */
 /** @typedef {import('./types.js').NormalizedNavigationAction} NormalizedNavigationAction */
 /** @typedef {import('./types.js').NormalizedNetworkParams} NormalizedNetworkParams */
 /** @typedef {import('./types.js').NormalizedPageTextParams} NormalizedPageTextParams */
@@ -209,6 +214,12 @@ export function validateBridgeRequest(request) {
       'session_id is no longer supported. Use tab_id or window-scoped default routing.'
     );
   }
+  if (
+    candidate.params != null &&
+    (typeof candidate.params !== 'object' || Array.isArray(candidate.params))
+  ) {
+    throw new BridgeError(ERROR_CODES.INVALID_REQUEST, 'Request params must be an object.');
+  }
   const parsedTabId = Number(candidate.tab_id);
 
   const method = /** @type {BridgeMethod} */ (candidate.method);
@@ -242,6 +253,8 @@ export function validateBridgeRequest(request) {
  */
 function normalizeRequestParams(method, params) {
   switch (method) {
+    case 'log.tail':
+      return normalizeLogTailParams(params);
     case 'dom.query':
       return normalizeDomQuery(params);
     case 'page.evaluate':
@@ -285,6 +298,9 @@ function normalizeRequestParams(method, params) {
       return normalizeInputAction(params);
     case 'cdp.dispatch_key_event':
       return normalizeCdpDispatchKeyEventParams(params);
+    case 'cdp.get_box_model':
+    case 'cdp.get_computed_styles_for_node':
+      return normalizeCdpNodeIdParams(params);
     case 'input.set_checked':
       return normalizeCheckedAction(params);
     case 'input.select_option':
@@ -429,6 +445,20 @@ export function normalizeCdpDispatchKeyEventParams(params = {}) {
   };
 }
 
+/**
+ * @param {CdpNodeIdParams} [params={}]
+ * @returns {NormalizedCdpNodeIdParams}
+ */
+export function normalizeCdpNodeIdParams(params = {}) {
+  if (typeof params.nodeId !== 'number' || !Number.isFinite(params.nodeId)) {
+    throw new BridgeError(ERROR_CODES.INVALID_REQUEST, 'nodeId must be a finite number.');
+  }
+
+  return {
+    nodeId: params.nodeId,
+  };
+}
+
 /**
  * @param {CheckedActionParams} [params={}]
  * @returns {NormalizedCheckedAction}
@@ -763,6 +793,16 @@ export function normalizePageTextParams(params = {}) {
   };
 }
 
+/**
+ * @param {LogTailParams} [params={}]
+ * @returns {NormalizedLogTailParams}
+ */
+export function normalizeLogTailParams(params = {}) {
+  return {
+    limit: clampInt(params.limit, 1, 200, DEFAULT_LOG_TAIL_LIMIT),
+  };
+}
+
 /**
  * @param {ViewportResizeParams} [params={}]
  * @returns {NormalizedViewportResizeParams}

package/packages/protocol/src/registry.js

@@ -82,6 +82,7 @@ const BRIDGE_METHOD_DESCRIPTIONS = Object.freeze({
   'performance.get_metrics': 'Read browser performance metrics.',
   'log.tail': 'Tail recent bridge log entries.',
   'health.ping': 'Check daemon, extension, and access-routing health.',
+  'daemon.metrics': 'Daemon health and performance metrics.',
 });
 
 /**
@@ -122,8 +123,9 @@ export const BRIDGE_METHOD_REGISTRY = Object.freeze({
     ['kind', 'target'],
     'trivial'
   ),
-  'log.tail': createRegistryEntry('log.tail', 'system', false, [], 'trivial'),
+  'log.tail': createRegistryEntry('log.tail', 'system', false, ['limit'], 'trivial'),
   'health.ping': createRegistryEntry('health.ping', 'system', false, [], 'trivial'),
+  'daemon.metrics': createRegistryEntry('daemon.metrics', 'system', false, [], 'trivial'),
   // tabs — trivial
   'tabs.list': createRegistryEntry('tabs.list', 'tabs', false, [], 'trivial'),
   'tabs.create': createRegistryEntry('tabs.create', 'tabs', false, ['url', 'active'], 'trivial'),
@@ -411,18 +413,12 @@ export const BRIDGE_METHOD_REGISTRY = Object.freeze({
   // cdp — high (raw protocol, large payloads)
   'cdp.get_document': createRegistryEntry('cdp.get_document', 'cdp', true, [], 'high'),
   'cdp.get_dom_snapshot': createRegistryEntry('cdp.get_dom_snapshot', 'cdp', true, [], 'high'),
-  'cdp.get_box_model': createRegistryEntry(
-    'cdp.get_box_model',
-    'cdp',
-    true,
-    ['elementRef'],
-    'high'
-  ),
+  'cdp.get_box_model': createRegistryEntry('cdp.get_box_model', 'cdp', true, ['nodeId'], 'high'),
   'cdp.get_computed_styles_for_node': createRegistryEntry(
     'cdp.get_computed_styles_for_node',
     'cdp',
     true,
-    ['elementRef'],
+    ['nodeId'],
     'high'
   ),
   'cdp.dispatch_key_event': createRegistryEntry(