@percy/core 1.31.6 → 1.31.7

package/dist/browser.js

@@ -123,6 +123,27 @@ export class Browser extends EventEmitter {
     var _this$ws;
     return ((_this$ws = this.ws) === null || _this$ws === void 0 ? void 0 : _this$ws.readyState) === WebSocket.OPEN;
   }
+  async restart() {
+    this.log.info('Restarting browser after disconnection');
+
+    // Force close the existing browser instance
+    /* istanbul ignore next: Hard to mock */
+    if (this.readyState !== null) {
+      await this.close(true).catch(err => {
+        this.log.debug('Error during force close:', err);
+      });
+    }
+
+    // Reset state for fresh launch
+    this.readyState = null;
+    this._closed = null;
+    this.#callbacks.clear();
+    this.sessions.clear();
+
+    // Launch a new browser instance
+    await this.launch();
+    this.log.info('Browser restarted successfully');
+  }
   async close(force = false) {
     var _this$percy$config$di, _this$process, _this$ws2;
     // Check for the new closeBrowser option
@@ -194,6 +215,11 @@ export class Browser extends EventEmitter {
     return this._closed;
   }
   async page(options = {}) {
+    // Check if browser is connected, restart if needed
+    if (!this.isConnected()) {
+      this.log.warn('Browser disconnected, attempting restart');
+      await this.restart();
+    }
     let {
       targetId
     } = await this.send('Target.createTarget', {

package/dist/config.js

@@ -482,7 +482,7 @@ export const configSchema = {
       },
       retry: {
         type: 'boolean',
-        default: false
+        default: true
       },
       launchOptions: {
         type: 'object',
@@ -1176,6 +1176,35 @@ export const comparisonSchema = {
       properties: {
         considerElementsData: regionsSchema
       }
+    },
+    elementSelectorsData: {
+      type: 'object',
+      additionalProperties: {
+        type: 'object',
+        properties: {
+          success: {
+            type: 'boolean'
+          },
+          top: {
+            type: ['number', 'null']
+          },
+          left: {
+            type: ['number', 'null']
+          },
+          bottom: {
+            type: ['number', 'null']
+          },
+          right: {
+            type: ['number', 'null']
+          },
+          message: {
+            type: ['string', 'null']
+          },
+          stacktrace: {
+            type: ['string', 'null']
+          }
+        }
+      }
     }
   }
 };

package/dist/discovery.js

@@ -541,8 +541,19 @@ export function createDiscoveryQueue(percy) {
         }
       }, {
         count: snapshot.discovery.retry ? 3 : 1,
-        onRetry: () => {
+        onRetry: async error => {
+          var _error$message, _error$message2, _error$message3, _error$message4;
           percy.log.info(`Retrying snapshot: ${snapshotLogName(snapshot.name, snapshot.meta)}`, snapshot.meta);
+          // If browser disconnected or crashed, restart it before retrying
+          if (error !== null && error !== void 0 && (_error$message = error.message) !== null && _error$message !== void 0 && _error$message.includes('Browser not connected') || error !== null && error !== void 0 && (_error$message2 = error.message) !== null && _error$message2 !== void 0 && _error$message2.includes('Browser closed') || error !== null && error !== void 0 && (_error$message3 = error.message) !== null && _error$message3 !== void 0 && _error$message3.includes('Session closed') || error !== null && error !== void 0 && (_error$message4 = error.message) !== null && _error$message4 !== void 0 && _error$message4.includes('Session crashed')) {
+            percy.log.warn('Detected browser disconnection, restarting browser before retry');
+            try {
+              await percy.browser.restart();
+            } catch (restartError) {
+              percy.log.error(`Failed to restart browser: ${restartError}`);
+              throw restartError;
+            }
+          }
         },
         signal: snapshot._ctrl.signal,
         throwOn: ['AbortError']

package/dist/utils.js

@@ -419,7 +419,7 @@ export async function withRetries(fn, {
       // if this error should not be retried on, we want to skip errors
       let throwError = throwOn === null || throwOn === void 0 ? void 0 : throwOn.includes(e.name);
       if (!throwError && run < count) {
-        await (onRetry === null || onRetry === void 0 ? void 0 : onRetry());
+        await (onRetry === null || onRetry === void 0 ? void 0 : onRetry(e));
         continue;
       }
       throw e;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@percy/core",
-  "version": "1.31.6",
+  "version": "1.31.7",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -43,12 +43,12 @@
     "test:types": "tsd"
   },
   "dependencies": {
-    "@percy/client": "1.31.6",
-    "@percy/config": "1.31.6",
-    "@percy/dom": "1.31.6",
-    "@percy/logger": "1.31.6",
-    "@percy/monitoring": "1.31.6",
-    "@percy/webdriver-utils": "1.31.6",
+    "@percy/client": "1.31.7",
+    "@percy/config": "1.31.7",
+    "@percy/dom": "1.31.7",
+    "@percy/logger": "1.31.7",
+    "@percy/monitoring": "1.31.7",
+    "@percy/webdriver-utils": "1.31.7",
     "content-disposition": "^0.5.4",
     "cross-spawn": "^7.0.3",
     "extract-zip": "^2.0.1",
@@ -61,5 +61,5 @@
     "ws": "^8.17.1",
     "yaml": "^2.4.1"
   },
-  "gitHead": "f5575bd2cd974b24fe3425ee5095fc825e30585e"
+  "gitHead": "a0b5ff9e0224d0ab4559f20170256320e91c6161"
 }