@browserbridge/bbx 1.0.0 → 1.0.1

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

@@ -3,7 +3,13 @@
 import fs from 'node:fs';
 import path from 'node:path';
 
-import { APP_NAME, getBridgeDir, getLauncherFilename, getManifestInstallDir, PUBLISHED_EXTENSION_ID } from './config.js';
+import {
+  APP_NAME,
+  getBridgeDir,
+  getLauncherFilename,
+  getManifestInstallDir,
+  PUBLISHED_EXTENSION_ID,
+} from './config.js';
 
 export const DEFAULT_EXTENSION_ID_ENV = 'BROWSER_BRIDGE_EXTENSION_ID';
 export const BUILT_IN_EXTENSION_ID_SOURCE = 'built_in';
@@ -21,6 +27,7 @@ export const BUILT_IN_EXTENSION_ID_SOURCE = 'built_in';
  *   bridgeDir?: string | undefined,
  *   stdout?: Pick<NodeJS.WriteStream, 'write'>,
  *   stderr?: Pick<NodeJS.WriteStream, 'write'>,
+ *   preserveCustomExtensionId?: boolean | undefined,
  *   env?: NodeJS.ProcessEnv
  * }} InstallManifestOptions
  */
@@ -62,13 +69,13 @@ export function resolveDefaultExtensionId(env = process.env) {
     const parsed = parseExtensionId(candidate);
     return {
       extensionId: parsed,
-      source: parsed ? 'env' : 'invalid_env'
+      source: parsed ? 'env' : 'invalid_env',
     };
   }
 
   return {
     extensionId: PUBLISHED_EXTENSION_ID || null,
-    source: PUBLISHED_EXTENSION_ID ? 'built_in' : 'none'
+    source: PUBLISHED_EXTENSION_ID ? 'built_in' : 'none',
   };
 }
 
@@ -90,9 +97,10 @@ export function getDefaultExtensionId(env = process.env) {
  * @returns {string[]}
  */
 export function getAllowedOrigins(existingManifest, extensionId) {
-  const existing = (existingManifest && Array.isArray(existingManifest.allowed_origins))
-    ? existingManifest.allowed_origins
-    : [];
+  const existing =
+    existingManifest && Array.isArray(existingManifest.allowed_origins)
+      ? existingManifest.allowed_origins
+      : [];
 
   if (extensionId) {
     const origin = `chrome-extension://${extensionId}/`;
@@ -110,6 +118,25 @@ export function getAllowedOrigins(existingManifest, extensionId) {
   return ['chrome-extension://__REPLACE_WITH_EXTENSION_ID__/'];
 }
 
+/**
+ * @param {string[] | undefined} allowedOrigins
+ * @returns {string[]}
+ */
+function getExtensionIdsFromAllowedOrigins(allowedOrigins) {
+  if (!Array.isArray(allowedOrigins)) {
+    return [];
+  }
+
+  const ids = new Set();
+  for (const origin of allowedOrigins) {
+    const match = /^chrome-extension:\/\/([a-z]{32})\/?$/.exec(origin);
+    if (match?.[1]) {
+      ids.add(match[1]);
+    }
+  }
+  return [...ids];
+}
+
 /**
  * @param {string} value
  * @returns {string}
@@ -144,7 +171,9 @@ export async function installNativeManifest(options) {
     installDir = getManifestInstallDir(browser),
     bridgeDir = getBridgeDir(),
     stdout = process.stdout,
-    env = process.env
+    stderr = process.stderr,
+    preserveCustomExtensionId = false,
+    env = process.env,
   } = options;
 
   const parsedExtensionId = parseExtensionId(extensionIdArg);
@@ -160,19 +189,35 @@ export async function installNativeManifest(options) {
       `Invalid ${DEFAULT_EXTENSION_ID_ENV}: ${env[DEFAULT_EXTENSION_ID_ENV]}\nExpected 32 lowercase letters or chrome-extension://<id>/`
     );
   }
-  const extensionId = parsedExtensionId || defaultExtensionId.extensionId;
+  const requestedExtensionId = parsedExtensionId || defaultExtensionId.extensionId;
   const hostPath = path.join(repoRoot, 'packages', 'native-host', 'bin', 'native-host.js');
   const launcherPath = path.join(bridgeDir, getLauncherFilename());
   const manifestPath = path.join(installDir, `${APP_NAME}.json`);
 
-  const launcher = process.platform === 'win32'
-    ? `@echo off\r\n"${nodePath}" "${hostPath}" %*\r\n`
-    : `#!/bin/sh
+  const launcher =
+    process.platform === 'win32'
+      ? `@echo off\r\n"${nodePath}" "${hostPath}" %*\r\n`
+      : `#!/bin/sh
 exec '${escapeSingleQuotes(nodePath)}' '${escapeSingleQuotes(hostPath)}' "$@"
 `;
 
   const existingManifest = await readExistingManifest(manifestPath);
-  const allowedOrigins = getAllowedOrigins(existingManifest, extensionId);
+  const existingExtensionIds = getExtensionIdsFromAllowedOrigins(existingManifest?.allowed_origins);
+  const hasStoreOrigin = existingExtensionIds.includes(PUBLISHED_EXTENSION_ID);
+  const customExtensionIds = existingExtensionIds.filter((id) => id !== PUBLISHED_EXTENSION_ID);
+  const preservedCustomExtensionId =
+    preserveCustomExtensionId &&
+    !parsedExtensionId &&
+    extensionIdArg == null &&
+    defaultExtensionId.source === BUILT_IN_EXTENSION_ID_SOURCE &&
+    customExtensionIds.length > 0 &&
+    !hasStoreOrigin;
+  const allowedOrigins = preservedCustomExtensionId
+    ? getAllowedOrigins(existingManifest, null)
+    : getAllowedOrigins(existingManifest, requestedExtensionId);
+  const extensionId = preservedCustomExtensionId
+    ? customExtensionIds[0] || requestedExtensionId
+    : requestedExtensionId;
 
   /** @type {{name: string, description: string, path: string, type: 'stdio', allowed_origins: string[]}} */
   const manifest = {
@@ -180,7 +225,7 @@ exec '${escapeSingleQuotes(nodePath)}' '${escapeSingleQuotes(hostPath)}' "$@"
     description: 'Browser Bridge native host',
     path: launcherPath,
     type: 'stdio',
-    allowed_origins: allowedOrigins
+    allowed_origins: allowedOrigins,
   };
 
   await fs.promises.mkdir(installDir, { recursive: true });
@@ -194,7 +239,7 @@ exec '${escapeSingleQuotes(nodePath)}' '${escapeSingleQuotes(hostPath)}' "$@"
   stdout.write(`Wrote ${manifestPath}\n`);
   stdout.write(`Wrote ${launcherPath}\n`);
 
-  if (!parsedExtensionId && extensionIdArg == null && extensionId) {
+  if (!preservedCustomExtensionId && !parsedExtensionId && extensionIdArg == null && extensionId) {
     if (defaultExtensionId.source === 'env') {
       stdout.write(`Used extension ID from ${DEFAULT_EXTENSION_ID_ENV}.\n`);
     } else if (defaultExtensionId.source === BUILT_IN_EXTENSION_ID_SOURCE) {
@@ -202,11 +247,19 @@ exec '${escapeSingleQuotes(nodePath)}' '${escapeSingleQuotes(hostPath)}' "$@"
     }
   }
 
-  const hasPlaceholder = allowedOrigins.some((origin) => origin.includes('__REPLACE_WITH_EXTENSION_ID__'));
+  if (preservedCustomExtensionId) {
+    stderr.write(
+      `Warning: existing native host manifest keeps custom extension ID ${customExtensionIds.join(', ')} instead of the Browser Bridge store ID ${PUBLISHED_EXTENSION_ID}. Leaving allowed_origins unchanged.\n`
+    );
+  }
+
+  const hasPlaceholder = allowedOrigins.some((origin) =>
+    origin.includes('__REPLACE_WITH_EXTENSION_ID__')
+  );
   if (hasPlaceholder) {
     stdout.write(
       'Tip: pass the extension ID to set allowed_origins automatically:\n' +
-      '  bbx install <extension-id>\n'
+        '  bbx install <extension-id>\n'
     );
   }
 
@@ -214,7 +267,7 @@ exec '${escapeSingleQuotes(nodePath)}' '${escapeSingleQuotes(hostPath)}' "$@"
     manifestPath,
     launcherPath,
     allowedOrigins,
-    extensionId
+    extensionId,
   };
 }
 
@@ -228,7 +281,7 @@ export async function uninstallNativeManifest(options = {}) {
     installDir = getManifestInstallDir(browser),
     bridgeDir = getBridgeDir(),
     removeBridgeDir = false,
-    stdout = process.stdout
+    stdout = process.stdout,
   } = options;
 
   const manifestPath = path.join(installDir, `${APP_NAME}.json`);
@@ -237,9 +290,7 @@ export async function uninstallNativeManifest(options = {}) {
     stdout.write(`Removed ${manifestPath}\n`);
   }
 
-  const removedBridgeDir = removeBridgeDir
-    ? await removePathIfExists(bridgeDir)
-    : false;
+  const removedBridgeDir = removeBridgeDir ? await removePathIfExists(bridgeDir) : false;
   if (removedBridgeDir) {
     stdout.write(`Removed ${bridgeDir}\n`);
   }
@@ -248,7 +299,7 @@ export async function uninstallNativeManifest(options = {}) {
     manifestPath,
     bridgeDir,
     removedManifest,
-    removedBridgeDir
+    removedBridgeDir,
   };
 }
 

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

@@ -54,10 +54,10 @@ const daemonEntryPath = path.resolve(__dirname, '../bin/bridge-daemon.js');
  */
 function isHostBridgeRequest(message) {
   return Boolean(
-    message
-    && typeof message === 'object'
-    && /** @type {Record<string, unknown>} */ (message).type === 'host.bridge_request'
-    && typeof /** @type {Record<string, unknown>} */ (message).request === 'object'
+    message &&
+    typeof message === 'object' &&
+    /** @type {Record<string, unknown>} */ (message).type === 'host.bridge_request' &&
+    typeof (/** @type {Record<string, unknown>} */ (message).request) === 'object'
   );
 }
 
@@ -67,10 +67,10 @@ function isHostBridgeRequest(message) {
  */
 function isHostStatusRequest(message) {
   return Boolean(
-    message
-    && typeof message === 'object'
-    && /** @type {Record<string, unknown>} */ (message).type === 'host.setup_status.request'
-    && typeof /** @type {Record<string, unknown>} */ (message).requestId === 'string'
+    message &&
+    typeof message === 'object' &&
+    /** @type {Record<string, unknown>} */ (message).type === 'host.setup_status.request' &&
+    typeof (/** @type {Record<string, unknown>} */ (message).requestId) === 'string'
   );
 }
 
@@ -80,9 +80,9 @@ function isHostStatusRequest(message) {
  */
 function isHostIdentity(message) {
   return Boolean(
-    message
-    && typeof message === 'object'
-    && /** @type {Record<string, unknown>} */ (message).type === 'host.identity'
+    message &&
+    typeof message === 'object' &&
+    /** @type {Record<string, unknown>} */ (message).type === 'host.identity'
   );
 }
 
@@ -92,10 +92,10 @@ function isHostIdentity(message) {
  */
 function isHostAccessUpdate(message) {
   return Boolean(
-    message
-    && typeof message === 'object'
-    && /** @type {Record<string, unknown>} */ (message).type === 'host.access_update'
-    && typeof /** @type {Record<string, unknown>} */ (message).accessEnabled === 'boolean'
+    message &&
+    typeof message === 'object' &&
+    /** @type {Record<string, unknown>} */ (message).type === 'host.access_update' &&
+    typeof (/** @type {Record<string, unknown>} */ (message).accessEnabled) === 'boolean'
   );
 }
 
@@ -105,9 +105,9 @@ function isHostAccessUpdate(message) {
  */
 function isHostActivity(message) {
   return Boolean(
-    message
-    && typeof message === 'object'
-    && /** @type {Record<string, unknown>} */ (message).type === 'host.activity'
+    message &&
+    typeof message === 'object' &&
+    /** @type {Record<string, unknown>} */ (message).type === 'host.activity'
   );
 }
 
@@ -126,7 +126,7 @@ export async function runNativeHost({ socketPath = getSocketPath() } = {}) {
         'native_bootstrap',
         ERROR_CODES.NATIVE_HOST_UNAVAILABLE,
         error instanceof Error ? error.message : String(error)
-      )
+      ),
     });
     return;
   }
@@ -159,22 +159,29 @@ export async function runNativeHost({ socketPath = getSocketPath() } = {}) {
         if (message.type === 'agent.response') {
           await writeNativeMessage(process.stdout, {
             type: 'host.bridge_response',
-            response: message.response
+            response: message.response,
           });
           return;
         }
-        if (message.type === 'extension.setup_status.response' || message.type === 'extension.setup_status.error') {
+        if (
+          message.type === 'extension.setup_status.response' ||
+          message.type === 'extension.setup_status.error'
+        ) {
           await writeNativeMessage(process.stdout, {
-            type: message.type === 'extension.setup_status.response'
-              ? 'host.setup_status.response'
-              : 'host.setup_status.error',
+            type:
+              message.type === 'extension.setup_status.response'
+                ? 'host.setup_status.response'
+                : 'host.setup_status.error',
             requestId: message.requestId,
             status: message.status,
-            error: message.error
+            error: message.error,
           });
         }
       })().catch((err) => {
-        console.error('native-host: socket message handler failed:', err instanceof Error ? err.message : err);
+        console.error(
+          'native-host: socket message handler failed:',
+          err instanceof Error ? err.message : err
+        );
       });
     }
   });
@@ -184,14 +191,14 @@ export async function runNativeHost({ socketPath = getSocketPath() } = {}) {
       if (isHostBridgeRequest(message)) {
         await writeJsonLine(socket, {
           type: 'agent.request',
-          request: message.request
+          request: message.request,
         });
         return;
       }
       if (isHostStatusRequest(message)) {
         await writeJsonLine(socket, {
           type: 'extension.setup_status.request',
-          requestId: message.requestId
+          requestId: message.requestId,
         });
         return;
       }
@@ -199,30 +206,33 @@ export async function runNativeHost({ socketPath = getSocketPath() } = {}) {
         await writeJsonLine(socket, {
           type: 'extension.identity',
           browserName: message.browserName,
-          profileLabel: message.profileLabel
+          profileLabel: message.profileLabel,
         });
         return;
       }
       if (isHostAccessUpdate(message)) {
         await writeJsonLine(socket, {
           type: 'extension.access_update',
-          accessEnabled: message.accessEnabled
+          accessEnabled: message.accessEnabled,
         });
         return;
       }
       if (isHostActivity(message)) {
         await writeJsonLine(socket, {
           type: 'extension.activity',
-          at: message.at
+          at: message.at,
         });
         return;
       }
       await writeJsonLine(socket, {
         type: 'extension.response',
-        response: message
+        response: message,
       });
     })().catch((err) => {
-      console.error('native-host: stdin message handler failed:', err instanceof Error ? err.message : err);
+      console.error(
+        'native-host: stdin message handler failed:',
+        err instanceof Error ? err.message : err
+      );
     });
   });
 }
@@ -235,9 +245,12 @@ export async function runNativeHost({ socketPath = getSocketPath() } = {}) {
  * @param {() => void} [onTerminate]
  * @returns {void}
  */
-export function bindBridgeSocketLifecycle(socket, onTerminate = () => {
-  process.exit(0);
-}) {
+export function bindBridgeSocketLifecycle(
+  socket,
+  onTerminate = () => {
+    process.exit(0);
+  }
+) {
   let terminated = false;
 
   /**
@@ -317,7 +330,7 @@ function connectSocket(socketPath) {
 function spawnBridgeDaemon() {
   const child = spawn(process.execPath, [daemonEntryPath], {
     detached: true,
-    stdio: 'ignore'
+    stdio: 'ignore',
   });
   child.unref();
 }
@@ -327,9 +340,11 @@ function spawnBridgeDaemon() {
  * @returns {boolean}
  */
 function shouldBootstrap(error) {
-  return error instanceof Error
-    && 'code' in error
-    && (error.code === 'ENOENT' || error.code === 'ECONNREFUSED');
+  return (
+    error instanceof Error &&
+    'code' in error &&
+    (error.code === 'ENOENT' || error.code === 'ECONNREFUSED')
+  );
 }
 
 /**

package/packages/protocol/src/budget.js

@@ -4,11 +4,7 @@
 /** @typedef {import('./types.js').BudgetOptions} BudgetOptions */
 /** @typedef {import('./types.js').TruncateResult} TruncateResult */
 
-import {
-  DEFAULT_MAX_DEPTH,
-  DEFAULT_MAX_NODES,
-  DEFAULT_TEXT_BUDGET,
-} from './defaults.js';
+import { DEFAULT_MAX_DEPTH, DEFAULT_MAX_NODES, DEFAULT_TEXT_BUDGET } from './defaults.js';
 
 /**
  * @param {BudgetOptions} [options={}]
@@ -20,7 +16,7 @@ export function applyBudget(options = {}) {
     maxDepth: clamp(options.maxDepth ?? DEFAULT_MAX_DEPTH, 1, 20),
     textBudget: clamp(options.textBudget ?? DEFAULT_TEXT_BUDGET, 32, 10000),
     includeBbox: options.includeBbox !== false,
-    attributeAllowlist: normalizeList(options.attributeAllowlist)
+    attributeAllowlist: normalizeList(options.attributeAllowlist),
   };
 }
 
@@ -41,7 +37,7 @@ export function truncateText(value, budget) {
   return {
     value: `${value.slice(0, Math.max(0, budget - 1))}\u2026`,
     truncated: true,
-    omitted: value.length - budget
+    omitted: value.length - budget,
   };
 }
 

package/packages/protocol/src/capabilities.js

@@ -20,7 +20,7 @@ export const CAPABILITIES = Object.freeze({
   AUTOMATION_INPUT: 'automation.input',
   TABS_MANAGE: 'tabs.manage',
   PERFORMANCE_READ: 'performance.read',
-  NETWORK_READ: 'network.read'
+  NETWORK_READ: 'network.read',
 });
 
 export const DEFAULT_CAPABILITIES = Object.freeze([
@@ -40,7 +40,7 @@ export const DEFAULT_CAPABILITIES = Object.freeze([
   CAPABILITIES.CDP_STYLES,
   CAPABILITIES.TABS_MANAGE,
   CAPABILITIES.PERFORMANCE_READ,
-  CAPABILITIES.NETWORK_READ
+  CAPABILITIES.NETWORK_READ,
 ]);
 
 /** @type {Readonly<Record<CapabilityMethod, Capability | null>>} */
@@ -101,7 +101,7 @@ export const METHOD_CAPABILITIES = Object.freeze({
   'cdp.get_computed_styles_for_node': CAPABILITIES.CDP_STYLES,
   'performance.get_metrics': CAPABILITIES.PERFORMANCE_READ,
   'log.tail': null,
-  'health.ping': null
+  'health.ping': null,
 });
 
 /**

package/packages/protocol/src/errors.js

@@ -12,7 +12,7 @@ export const ERROR_CODES = Object.freeze({
   INVALID_REQUEST: 'INVALID_REQUEST',
   NATIVE_HOST_UNAVAILABLE: 'NATIVE_HOST_UNAVAILABLE',
   EXTENSION_DISCONNECTED: 'EXTENSION_DISCONNECTED',
-  TIMEOUT: 'TIMEOUT'
+  TIMEOUT: 'TIMEOUT',
 });
 
 /**
@@ -23,47 +23,47 @@ export const ERROR_CODES = Object.freeze({
 export const ERROR_RECOVERY = Object.freeze({
   [ERROR_CODES.ACCESS_DENIED]: {
     retry: false,
-    hint: 'Access is off for this window. Ask the user to click Enable in the Browser Bridge popup or side panel. Do not request access again until that window is enabled.'
+    hint: 'Access is off for this window. Ask the user to click Enable in the Browser Bridge popup or side panel. Do not request access again until that window is enabled.',
   },
   [ERROR_CODES.ELEMENT_STALE]: {
     retry: false,
     alternativeMethod: 'dom.query',
-    hint: 'Element was removed from the DOM. Re-query with the same selector to get a fresh elementRef.'
+    hint: 'Element was removed from the DOM. Re-query with the same selector to get a fresh elementRef.',
   },
   [ERROR_CODES.TAB_MISMATCH]: {
     retry: false,
     alternativeMethod: 'tabs.list',
-    hint: 'Tab was closed or not found. Use tabs.list to find an available tab.'
+    hint: 'Tab was closed or not found. Use tabs.list to find an available tab.',
   },
   [ERROR_CODES.TIMEOUT]: {
     retry: true,
     retryAfterMs: 1000,
-    hint: 'Operation exceeded the time limit. Retry once, or simplify the request (smaller maxNodes, narrower selector).'
+    hint: 'Operation exceeded the time limit. Retry once, or simplify the request (smaller maxNodes, narrower selector).',
   },
   [ERROR_CODES.RATE_LIMITED]: {
     retry: true,
     retryAfterMs: 2000,
-    hint: 'Too many requests. Back off and retry after a short delay.'
+    hint: 'Too many requests. Back off and retry after a short delay.',
   },
   [ERROR_CODES.EXTENSION_DISCONNECTED]: {
     retry: true,
     retryAfterMs: 3000,
     alternativeMethod: 'health.ping',
-    hint: 'Extension not connected. Check Chrome is running, then retry. Use health.ping to verify connectivity.'
+    hint: 'Extension not connected. Check Chrome is running, then retry. Use health.ping to verify connectivity.',
   },
   [ERROR_CODES.NATIVE_HOST_UNAVAILABLE]: {
     retry: false,
-    hint: 'Native host not reachable. Run `bbx doctor` to diagnose the installation.'
+    hint: 'Native host not reachable. Run `bbx doctor` to diagnose the installation.',
   },
   [ERROR_CODES.INVALID_REQUEST]: {
     retry: false,
-    hint: 'Malformed method or params. Check the method name and parameter types.'
+    hint: 'Malformed method or params. Check the method name and parameter types.',
   },
   [ERROR_CODES.INTERNAL_ERROR]: {
     retry: true,
     retryAfterMs: 1000,
-    hint: 'Unexpected extension error. Retry once; if persistent, check page.get_console for details.'
-  }
+    hint: 'Unexpected extension error. Retry once; if persistent, check page.get_console for details.',
+  },
 });
 
 /**