@percy/playwright 1.0.11-alpha.0 → 1.1.0-beta.1

package/index.js

@@ -52,7 +52,15 @@ async function captureSerializedDOM(page, options, percyDOM) {
   // This section only handles cross-origin iframe serialization and resource merging.
   const pageUrl = new URL(page.url());
   const crossOriginFrames = page.frames()
-    .filter(frame => frame.url() !== 'about:blank' && new URL(frame.url()).origin !== pageUrl.origin);
+    .filter(frame => {
+      const frameUrl = frame.url();
+      if (!frameUrl || frameUrl === 'about:blank') return false;
+      try {
+        return new URL(frameUrl).origin !== pageUrl.origin;
+      } catch {
+        return false;
+      }
+    });
 
   // Inject Percy DOM into all cross-origin frames before processing them in parallel
   await Promise.all(crossOriginFrames.map(frame => frame.evaluate(percyDOM)));
@@ -114,9 +122,15 @@ async function captureResponsiveDOM(page, options, percyDOM) {
   // Calculate default height for non-mobile widths
   let defaultHeight = currentHeight;
   if (process.env.PERCY_RESPONSIVE_CAPTURE_MIN_HEIGHT?.toLowerCase() === 'true') {
-    const minHeight = utils.percy?.config?.snapshot?.minHeight;
-    /* istanbul ignore next: no instrumenting injected code */
-    defaultHeight = await page.evaluate((minH) => window.outerHeight - window.innerHeight + minH, minHeight);
+    if (options.minHeight) {
+      defaultHeight = options.minHeight;
+    } else {
+      const configMinHeight = utils.percy?.config?.snapshot?.minHeight;
+      /* istanbul ignore else: CLI always provides default value for config.snapshot */
+      if (configMinHeight) {
+        defaultHeight = configMinHeight;
+      }
+    }
   }
 
   // Get width and height combinations

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@percy/playwright",
   "description": "Playwright client library for visual testing with Percy",
-  "version": "1.0.11-alpha.0",
+  "version": "1.1.0-beta.1",
   "license": "MIT",
   "author": "Perceptual Inc.",
   "repository": "https://github.com/percy/percy-playwright",
@@ -29,13 +29,13 @@
   },
   "publishConfig": {
     "access": "public",
-    "tag": "alpha"
+    "tag": "beta"
   },
   "peerDependencies": {
     "playwright-core": ">=1"
   },
   "devDependencies": {
-    "@percy/cli": "^1.30.9",
+    "@percy/cli": "1.31.10-alpha.0",
     "@playwright/test": "^1.24.2",
     "babel-eslint": "^10.1.0",
     "cross-env": "^7.0.2",