browser-commander 0.2.0 → 0.3.0

package/CHANGELOG.md

@@ -1,5 +1,33 @@
 # Changelog
 
+## 0.3.0
+
+### Minor Changes
+
+- 03e9ccb: Add isTimeoutError function for detecting timeout errors
+
+  Adds a new `isTimeoutError` function exported from the library that helps detect timeout errors from selector waiting operations. This function is complementary to `isNavigationError` and allows automation loops to handle timeout errors gracefully without crashing.
+
+  Usage example:
+
+  ```javascript
+  import { isTimeoutError } from 'browser-commander';
+
+  try {
+    await page.waitForSelector('.button');
+  } catch (error) {
+    if (isTimeoutError(error)) {
+      console.log('Timeout occurred, continuing with next item...');
+    }
+  }
+  ```
+
+## 0.2.1
+
+### Patch Changes
+
+- Test patch release
+
 ## 0.2.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "browser-commander",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "Universal browser automation library that supports both Playwright and Puppeteer with a unified API",
   "main": "src/index.js",
   "type": "module",

package/src/core/navigation-safety.js

@@ -158,3 +158,32 @@ export function withNavigationSafety(fn, options = {}) {
     }
   };
 }
+
+/**
+ * Check if an error is a timeout error from selector waiting
+ * These errors should be treated as non-fatal in automation loops
+ * @param {Error} error - The error to check
+ * @returns {boolean} - True if this is a timeout error
+ */
+export function isTimeoutError(error) {
+  if (!error) {
+    return false;
+  }
+
+  // Check error name first (most reliable)
+  if (error.name === 'TimeoutError') {
+    return true;
+  }
+
+  // Check error message patterns (case-insensitive)
+  const message = (error.message || '').toLowerCase();
+  const timeoutErrorPatterns = [
+    'waiting for selector',
+    'timeout',
+    'timeouterror',
+    'timeout exceeded',
+    'timed out',
+  ];
+
+  return timeoutErrorPatterns.some((pattern) => message.includes(pattern));
+}

package/src/exports.js

@@ -10,6 +10,7 @@ export { disableTranslateInPreferences } from './core/preferences.js';
 export { detectEngine } from './core/engine-detection.js';
 export {
   isNavigationError,
+  isTimeoutError,
   safeOperation,
   makeNavigationSafe,
   withNavigationSafety,

package/tests/unit/core/navigation-safety.test.js

@@ -2,6 +2,7 @@ import { describe, it } from 'node:test';
 import assert from 'node:assert';
 import {
   isNavigationError,
+  isTimeoutError,
   safeOperation,
   makeNavigationSafe,
   withNavigationSafety,
@@ -67,6 +68,80 @@ describe('navigation-safety', () => {
     });
   });
 
+  describe('isTimeoutError', () => {
+    it('should return false for null', () => {
+      assert.strictEqual(isTimeoutError(null), false);
+    });
+
+    it('should return false for undefined', () => {
+      assert.strictEqual(isTimeoutError(undefined), false);
+    });
+
+    it('should return false for error without message or name', () => {
+      assert.strictEqual(isTimeoutError({}), false);
+    });
+
+    it('should return false for regular error', () => {
+      const error = new Error('Some regular error');
+      assert.strictEqual(isTimeoutError(error), false);
+    });
+
+    it('should return true for error with name "TimeoutError"', () => {
+      const error = new Error('Something failed');
+      error.name = 'TimeoutError';
+      assert.strictEqual(isTimeoutError(error), true);
+    });
+
+    it('should return true for "waiting for selector" error', () => {
+      const error = new Error('Waiting for selector ".button" timed out');
+      assert.strictEqual(isTimeoutError(error), true);
+    });
+
+    it('should return true for "timeout" error (case-insensitive)', () => {
+      const error = new Error('Timeout while waiting for element');
+      assert.strictEqual(isTimeoutError(error), true);
+    });
+
+    it('should return true for "timeout exceeded" error', () => {
+      const error = new Error('timeout exceeded');
+      assert.strictEqual(isTimeoutError(error), true);
+    });
+
+    it('should return true for "timed out" error', () => {
+      const error = new Error('Operation timed out after 30000ms');
+      assert.strictEqual(isTimeoutError(error), true);
+    });
+
+    it('should return true for Playwright locator timeout error', () => {
+      const error = new Error(
+        'locator.click: Timeout 30000ms exceeded. Waiting for selector "button.submit"'
+      );
+      assert.strictEqual(isTimeoutError(error), true);
+    });
+
+    it('should return true for Puppeteer waitForSelector timeout', () => {
+      const error = new Error(
+        'waiting for selector `#myElement` failed: timeout 30000ms exceeded'
+      );
+      assert.strictEqual(isTimeoutError(error), true);
+    });
+
+    it('should handle uppercase TIMEOUT in message', () => {
+      const error = new Error('TIMEOUT ERROR occurred');
+      assert.strictEqual(isTimeoutError(error), true);
+    });
+
+    it('should not match navigation errors as timeout errors', () => {
+      const error = new Error('Execution context was destroyed');
+      assert.strictEqual(isTimeoutError(error), false);
+    });
+
+    it('should not match frame detached errors as timeout errors', () => {
+      const error = new Error('frame was detached');
+      assert.strictEqual(isTimeoutError(error), false);
+    });
+  });
+
   describe('safeOperation', () => {
     it('should return success result on successful operation', async () => {
       const result = await safeOperation(async () => 'success');