@browserbridge/bbx 1.7.3 → 1.7.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@browserbridge/bbx",
-  "version": "1.7.3",
+  "version": "1.7.5",
   "private": false,
   "keywords": [
     "agent-tools",
@@ -73,13 +73,13 @@
     "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@types/chrome": "^0.1.43",
-    "@types/node": "^25.9.2",
+    "@types/chrome": "^0.2.2",
+    "@types/node": "^26.1.1",
     "c8": "^11.0.0",
-    "linkedom": "^0.18.12",
-    "oxfmt": "^0.54.0",
-    "oxlint": "^1.69.0",
-    "typescript": "^6.0.3"
+    "linkedom": "^0.18.13",
+    "oxfmt": "^0.58.0",
+    "oxlint": "^1.73.0",
+    "typescript": "^7.0.2"
   },
   "engines": {
     "node": ">=18"

package/packages/agent-client/src/atomic-write.js

@@ -0,0 +1,99 @@
+// @ts-check
+
+import { randomUUID } from 'node:crypto';
+import fs from 'node:fs';
+import path from 'node:path';
+
+/**
+ * Replace a file through a sibling temporary file so readers never observe a
+ * partial write. Existing permissions are preserved unless an explicit mode is
+ * requested.
+ *
+ * @param {string} targetPath
+ * @param {string | NodeJS.ArrayBufferView} data
+ * @param {{ encoding?: BufferEncoding, mode?: number, preserveMode?: boolean }} [options={ }]
+ * @returns {Promise<void>}
+ */
+export async function atomicWriteFile(targetPath, data, options = {}) {
+  const directory = path.dirname(targetPath);
+  const temporaryPath = path.join(
+    directory,
+    `.${path.basename(targetPath)}.${process.pid}.${randomUUID()}.tmp`
+  );
+  await fs.promises.mkdir(directory, { recursive: true });
+
+  let existingMode;
+  if (options.preserveMode !== false && options.mode === undefined) {
+    try {
+      existingMode = (await fs.promises.stat(targetPath)).mode & 0o777;
+    } catch (error) {
+      if (!isMissingFileError(error)) {
+        throw error;
+      }
+    }
+  }
+
+  const mode = options.mode ?? existingMode;
+  try {
+    await fs.promises.writeFile(temporaryPath, data, {
+      ...(options.encoding ? { encoding: options.encoding } : {}),
+      ...(mode === undefined ? {} : { mode }),
+      flag: 'wx',
+    });
+    if (process.platform !== 'win32' && mode !== undefined) {
+      await fs.promises.chmod(temporaryPath, mode);
+    }
+    try {
+      await fs.promises.rename(temporaryPath, targetPath);
+    } catch (error) {
+      if (process.platform !== 'win32' || !isReplaceError(error)) {
+        throw error;
+      }
+      try {
+        await fs.promises.stat(targetPath);
+      } catch {
+        throw error;
+      }
+
+      const backupPath = `${temporaryPath}.backup`;
+      await fs.promises.rename(targetPath, backupPath);
+      try {
+        await fs.promises.rename(temporaryPath, targetPath);
+      } catch (replaceError) {
+        await fs.promises.rename(backupPath, targetPath).catch(() => {});
+        throw replaceError;
+      }
+      await fs.promises.rm(backupPath, { force: true }).catch(() => {});
+    }
+    if (process.platform !== 'win32' && options.mode !== undefined) {
+      await fs.promises.chmod(targetPath, options.mode);
+    }
+  } catch (error) {
+    await fs.promises.rm(temporaryPath, { force: true }).catch(() => {});
+    throw error;
+  }
+}
+
+/**
+ * Windows does not replace an existing file with rename().
+ *
+ * @param {unknown} error
+ * @returns {boolean}
+ */
+function isReplaceError(error) {
+  const code =
+    error && typeof error === 'object' ? /** @type {{ code?: unknown }} */ (error).code : null;
+  return code === 'EEXIST' || code === 'EPERM' || code === 'EACCES';
+}
+
+/**
+ * @param {unknown} error
+ * @returns {boolean}
+ */
+function isMissingFileError(error) {
+  return Boolean(
+    error &&
+    typeof error === 'object' &&
+    /** @type {{ code?: unknown }} */ (error).code === 'ENOENT'
+  );
+}

package/packages/agent-client/src/cli-helpers.js

@@ -96,7 +96,7 @@ export function parseJsonObject(value) {
 
 /**
  * Parse a CLI argument as a positive integer, throwing a user-friendly error
- * if the value is missing or not a finite number.
+ * if the value is missing or not a positive integer.
  *
  * @param {string | undefined} value
  * @param {string} argName - The argument name shown in the error message
@@ -104,12 +104,29 @@ export function parseJsonObject(value) {
  */
 export function parseIntArg(value, argName) {
   const n = Number(value);
-  if (!value || !Number.isFinite(n)) {
-    throw new Error(`${argName} must be a number (got ${JSON.stringify(value)}).`);
+  if (!value || !Number.isInteger(n) || n <= 0) {
+    throw new Error(`${argName} must be a positive integer (got ${JSON.stringify(value)}).`);
   }
   return n;
 }
 
+/**
+ * Parse a CLI argument as a finite number. This is intentionally separate from
+ * positive integer parsing because viewport coordinates may be fractional or
+ * negative.
+ *
+ * @param {string | undefined} value
+ * @param {string} argName
+ * @returns {number}
+ */
+export function parseNumberArg(value, argName) {
+  const number = Number(value);
+  if (value === undefined || value === '' || !Number.isFinite(number)) {
+    throw new Error(`${argName} must be a number (got ${JSON.stringify(value)}).`);
+  }
+  return number;
+}
+
 /**
  * @param {string} method
  * @returns {boolean}