@chromatic-com/playwright 0.14.7 → 0.14.9

package/dist/browser.mjs

@@ -5275,7 +5275,6 @@ postcss$1.Root;
 postcss$1.Node;
 
 // src/browser.ts
-window.__chromatic_takeSnapshot = takeSnapshot;
 async function takeSnapshot() {
   const mirror = createMirror();
   const domSnapshot = snapshot(document, {
@@ -5325,3 +5324,4 @@ async function toDataURL(url) {
   });
 }
 __name(toDataURL, "toDataURL");
+window.__chromatic_takeSnapshot = takeSnapshot;

package/dist/index.js

@@ -3919,9 +3919,13 @@ var STORIES_FILE_EXT = "stories.json";
 var uniqueId = {
   value: 1
 };
-function storiesFileName(testTitle) {
+function storiesFileName(testTitle, overwriteDuplicateNames = true) {
+  let title = sanitize(testTitle);
+  if (!overwriteDuplicateNames) {
+    title += "-" + uniqueId.value++;
+  }
   const fileName = [
-    sanitize(testTitle) + "-" + uniqueId.value++,
+    title,
     STORIES_FILE_EXT
   ].join(".");
   const maxByteLength = MAX_FILE_NAME_BYTE_LENGTH - 25;
@@ -4022,7 +4026,7 @@ function findDefaultViewport(viewports) {
 __name(findDefaultViewport, "findDefaultViewport");
 
 // ../shared/src/write-archive/index.ts
-async function writeTestResult(e2eTestInfo, domSnapshots, archive, chromaticStorybookParams) {
+async function writeTestResult(e2eTestInfo, domSnapshots, archive, chromaticStorybookParams, testRunner) {
   const { titlePath, outputDir, pageUrl } = e2eTestInfo;
   const titlePathWithoutFileExtensions = titlePath.map((pathPart) => (
     // make sure we remove file extensions, even if the file name doesn't have .spec or .test or.cy
@@ -4057,7 +4061,7 @@ async function writeTestResult(e2eTestInfo, domSnapshots, archive, chromaticStor
     const snapshotFile = snapshotFileName(snapshotId(title, name), domSnapshot.viewport);
     await outputFile(path2.join(archiveDir, snapshotFile), mappedSnapshot);
   }));
-  const storiesFile = storiesFileName(title);
+  const storiesFile = storiesFileName(title, testRunner !== "vitest");
   const storiesJson = createStories(title, domSnapshots, chromaticStorybookParams);
   await outputJSONFile(path2.join(finalOutputDir, storiesFile), storiesJson);
   const errors = Object.entries(archive).filter(([, r]) => "error" in r);
@@ -4100,8 +4104,8 @@ __name(trackComplete, "trackComplete");
 
 // ../shared/src/constants.ts
 var DEFAULT_GLOBAL_RESOURCE_ARCHIVE_TIMEOUT_MS = 1e4;
-
-// src/takeSnapshot.ts
+var browserEntry = __require.resolve("@chromatic-com/playwright/browser");
+var browserScript = fs.readFileSync(browserEntry, "utf-8");
 var chromaticSnapshots = /* @__PURE__ */ new Map();
 async function takeSnapshot(page, nameOrTestInfo, maybeTestInfo) {
   let name;
@@ -4155,14 +4159,15 @@ async function takeSnapshot(page, nameOrTestInfo, maybeTestInfo) {
 }
 __name(takeSnapshot, "takeSnapshot");
 async function executeSnapshotScript(context) {
-  await context.addScriptTag({
-    type: "module",
-    path: __require.resolve("@chromatic-com/playwright/browser")
-  });
-  await context.waitForFunction(() => typeof window.__chromatic_takeSnapshot === "function");
-  const snapshot = await context.evaluate(async () => {
-    return JSON.stringify(await window.__chromatic_takeSnapshot());
-  });
+  const snapshot = await context.evaluate(
+    // Wrapper is needed to scope all inlined definitions of browser script, as
+    // it contains bundled rrweb-snapshot that inlines variables like "const node = ...".
+    `async function wrapper() {
+      ${browserScript}
+
+      return JSON.stringify(await takeSnapshot());
+    }()`
+  );
   return JSON.parse(snapshot);
 }
 __name(executeSnapshotScript, "executeSnapshotScript");
@@ -4255,7 +4260,7 @@ var performChromaticSnapshot = /* @__PURE__ */ __name(async ({ page, delay, diff
       prefersReducedMotion,
       cropToViewport,
       ignoreSelectors
-    });
+    }, "playwright");
     trackComplete();
   } finally {
     chromaticSnapshots.delete(testId);

package/dist/index.mjs

@@ -2,7 +2,7 @@ import { createRequire } from 'module';
 import { test as test$1 } from '@playwright/test';
 export { expect } from '@playwright/test';
 import path2, { join } from 'path';
-import { existsSync, mkdirSync } from 'fs';
+import { readFileSync, existsSync, mkdirSync } from 'fs';
 import { writeFile } from 'fs/promises';
 import { createHash } from 'crypto';
 import { toId, storyNameFromExport } from 'storybook/internal/csf';
@@ -3913,9 +3913,13 @@ var STORIES_FILE_EXT = "stories.json";
 var uniqueId = {
   value: 1
 };
-function storiesFileName(testTitle) {
+function storiesFileName(testTitle, overwriteDuplicateNames = true) {
+  let title = sanitize(testTitle);
+  if (!overwriteDuplicateNames) {
+    title += "-" + uniqueId.value++;
+  }
   const fileName = [
-    sanitize(testTitle) + "-" + uniqueId.value++,
+    title,
     STORIES_FILE_EXT
   ].join(".");
   const maxByteLength = MAX_FILE_NAME_BYTE_LENGTH - 25;
@@ -4016,7 +4020,7 @@ function findDefaultViewport(viewports) {
 __name(findDefaultViewport, "findDefaultViewport");
 
 // ../shared/src/write-archive/index.ts
-async function writeTestResult(e2eTestInfo, domSnapshots, archive, chromaticStorybookParams) {
+async function writeTestResult(e2eTestInfo, domSnapshots, archive, chromaticStorybookParams, testRunner) {
   const { titlePath, outputDir, pageUrl } = e2eTestInfo;
   const titlePathWithoutFileExtensions = titlePath.map((pathPart) => (
     // make sure we remove file extensions, even if the file name doesn't have .spec or .test or.cy
@@ -4051,7 +4055,7 @@ async function writeTestResult(e2eTestInfo, domSnapshots, archive, chromaticStor
     const snapshotFile = snapshotFileName(snapshotId(title, name), domSnapshot.viewport);
     await outputFile(join(archiveDir, snapshotFile), mappedSnapshot);
   }));
-  const storiesFile = storiesFileName(title);
+  const storiesFile = storiesFileName(title, testRunner !== "vitest");
   const storiesJson = createStories(title, domSnapshots, chromaticStorybookParams);
   await outputJSONFile(join(finalOutputDir, storiesFile), storiesJson);
   const errors = Object.entries(archive).filter(([, r]) => "error" in r);
@@ -4094,8 +4098,8 @@ __name(trackComplete, "trackComplete");
 
 // ../shared/src/constants.ts
 var DEFAULT_GLOBAL_RESOURCE_ARCHIVE_TIMEOUT_MS = 1e4;
-
-// src/takeSnapshot.ts
+var browserEntry = __require.resolve("@chromatic-com/playwright/browser");
+var browserScript = readFileSync(browserEntry, "utf-8");
 var chromaticSnapshots = /* @__PURE__ */ new Map();
 async function takeSnapshot(page, nameOrTestInfo, maybeTestInfo) {
   let name;
@@ -4149,14 +4153,15 @@ async function takeSnapshot(page, nameOrTestInfo, maybeTestInfo) {
 }
 __name(takeSnapshot, "takeSnapshot");
 async function executeSnapshotScript(context) {
-  await context.addScriptTag({
-    type: "module",
-    path: __require.resolve("@chromatic-com/playwright/browser")
-  });
-  await context.waitForFunction(() => typeof window.__chromatic_takeSnapshot === "function");
-  const snapshot = await context.evaluate(async () => {
-    return JSON.stringify(await window.__chromatic_takeSnapshot());
-  });
+  const snapshot = await context.evaluate(
+    // Wrapper is needed to scope all inlined definitions of browser script, as
+    // it contains bundled rrweb-snapshot that inlines variables like "const node = ...".
+    `async function wrapper() {
+      ${browserScript}
+
+      return JSON.stringify(await takeSnapshot());
+    }()`
+  );
   return JSON.parse(snapshot);
 }
 __name(executeSnapshotScript, "executeSnapshotScript");
@@ -4249,7 +4254,7 @@ var performChromaticSnapshot = /* @__PURE__ */ __name(async ({ page, delay, diff
       prefersReducedMotion,
       cropToViewport,
       ignoreSelectors
-    });
+    }, "playwright");
     trackComplete();
   } finally {
     chromaticSnapshots.delete(testId);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chromatic-com/playwright",
-  "version": "0.14.7",
+  "version": "0.14.9",
   "description": "Chromatic Visual Regression Testing for Playwright",
   "repository": {
     "type": "git",