@tempo-ai/visual-review 0.0.108 → 0.0.109-staging.2

package/dist/bin.js

@@ -6,7 +6,7 @@ async function main(argv = process.argv.slice(2)) {
   const [command, ...commandArgs] = argv;
   switch (command) {
     case "capture":
-      return await (await import("./cli-ASUZAROR.js")).main(commandArgs);
+      return await (await import("./cli-JDSQSM3K.js")).main(commandArgs);
     case "claim":
       return await (await import("./claim-J654JAS6.js")).main(commandArgs);
     case "report":

package/dist/{chunk-2EVRJIRQ.js → chunk-7P5FPOFE.js}

@@ -419,7 +419,7 @@ async function installChromium(log, spawnImpl = spawn) {
 }
 
 // ../canvas-capture-headless/src/fingerprint.ts
-var KIT_VERSION = "3";
+var KIT_VERSION = "5";
 var FONTS_BUNDLE_VERSION = "system-1";
 var DEVICE_SCALE_FACTOR = 2;
 function buildEnvFingerprint(input) {
@@ -855,8 +855,7 @@ function readNavigatedMark(error) {
   return error !== null && typeof error === "object" && navigatedErrors.has(error);
 }
 function documentReplaced(start, now, expectedNavigations) {
-  const ownSameDocument = now.sameDocumentIssued - start.sameDocumentIssued;
-  return now.navigationRequests > start.navigationRequests + expectedNavigations || now.frameNavigations > start.frameNavigations + expectedNavigations + ownSameDocument;
+  return now.navigationRequests > start.navigationRequests + expectedNavigations || now.documentCommits > start.documentCommits + expectedNavigations;
 }
 var BOARD_ATTEMPT_LIMIT = 3;
 function decideBoardRetry(attempt, attemptIndex, attemptLimit = BOARD_ATTEMPT_LIMIT) {
@@ -910,7 +909,7 @@ function describeRetryReason(attempt) {
   }
   return "document reloaded mid-capture";
 }
-var DEFAULT_VIEWPORT = { width: 1600, height: 1200 };
+var INITIAL_VIEWPORT = { width: 1600, height: 1200 };
 async function startCaptureSession(options) {
   const log = options.log ?? (() => {
   });
@@ -936,7 +935,7 @@ async function startCaptureSession(options) {
     const contextStart = Date.now();
     const context = await browser.newContext({
       deviceScaleFactor: capture.deviceScaleFactor ?? DEVICE_SCALE_FACTOR,
-      viewport: capture.viewport ?? DEFAULT_VIEWPORT,
+      viewport: capture.viewport ?? INITIAL_VIEWPORT,
       reducedMotion: "reduce",
       colorScheme: "light",
       timezoneId: "UTC",
@@ -946,6 +945,7 @@ async function startCaptureSession(options) {
     await context.addInitScript(buildFreezeInitScript());
     await context.addInitScript(installRenderProfileInPage);
     const page = await context.newPage();
+    const documentEvents = await context.newCDPSession(page);
     const worker = {
       context,
       page,
@@ -954,12 +954,13 @@ async function startCaptureSession(options) {
       hasNavigated: false,
       counters: {
         navigationRequests: 0,
-        frameNavigations: 0,
-        sameDocumentIssued: 0
+        documentCommits: 0
       },
       documentsLoaded: 0,
       navigationsInFlight: 0,
       async dispose() {
+        await documentEvents.detach().catch(() => {
+        });
         await page.close().catch(() => {
         });
         await context.close().catch(() => {
@@ -983,9 +984,10 @@ async function startCaptureSession(options) {
       if (!isMainFrameNavigation(request)) return;
       worker.navigationsInFlight = Math.max(0, worker.navigationsInFlight - 1);
     });
-    page.on("framenavigated", (frame) => {
-      if (frame === page.mainFrame()) worker.counters.frameNavigations += 1;
+    documentEvents.on("Page.frameNavigated", ({ frame }) => {
+      if (!frame.parentId) worker.counters.documentCommits += 1;
     });
+    await documentEvents.send("Page.enable");
     page.on("domcontentloaded", () => {
       worker.documentsLoaded += 1;
       worker.navigationsInFlight = 0;
@@ -1030,8 +1032,12 @@ async function startCaptureSession(options) {
       target.storyboardId
     );
     const selector = buildStoryboardRenderSelector(renderId);
+    const viewport = capture.viewport ?? {
+      width: Math.ceil(target.width),
+      height: Math.ceil(target.height)
+    };
     const ownsWorker = reusableWorker === void 0;
-    const worker = reusableWorker ?? await createCaptureWorker(capture);
+    const worker = reusableWorker ?? await createCaptureWorker({ ...capture, viewport });
     const { page } = worker;
     if (!worker.contextClaimed) {
       timings.contextMs = worker.contextMs;
@@ -1043,6 +1049,10 @@ async function startCaptureSession(options) {
     try {
       const navStart = Date.now();
       try {
+        const currentViewport = page.viewportSize();
+        if (currentViewport?.width !== viewport.width || currentViewport?.height !== viewport.height) {
+          await page.setViewportSize(viewport);
+        }
         const targetUrl = buildDirectRenderUrl2(options.hostUrl, target);
         if (!worker.hasNavigated) {
           await page.goto(targetUrl, { waitUntil: "domcontentloaded" });
@@ -1051,10 +1061,11 @@ async function startCaptureSession(options) {
         } else {
           if (worker.currentSelector === selector) {
             await page.evaluate((url) => {
-              history.replaceState(null, "", url);
-              window.dispatchEvent(new PopStateEvent("popstate"));
+              history.replaceState(history.state, "", url);
+              window.dispatchEvent(
+                new PopStateEvent("popstate", { state: history.state })
+              );
             }, buildDirectRenderIdleUrl(options.hostUrl));
-            worker.counters.sameDocumentIssued += 1;
             await page.waitForSelector(worker.currentSelector, {
               state: "detached",
               timeout: capture.settleTimeoutMs ?? options.settleTimeoutMs ?? 3e4
@@ -1065,12 +1076,13 @@ async function startCaptureSession(options) {
               const captureWindow = window;
               captureWindow.__tempoResetCaptureDeterminism?.();
               captureWindow.__tempoCaptureRenderProfile?.begin(nextSelector);
-              history.replaceState(null, "", url);
-              window.dispatchEvent(new PopStateEvent("popstate"));
+              history.replaceState(history.state, "", url);
+              window.dispatchEvent(
+                new PopStateEvent("popstate", { state: history.state })
+              );
             },
             { url: targetUrl, nextSelector: selector }
           );
-          worker.counters.sameDocumentIssued += 1;
         }
         worker.currentSelector = selector;
       } finally {
@@ -1391,7 +1403,7 @@ async function startCaptureSession(options) {
             if (isReloadStorm(recentReloadRetries, stormWindow)) {
               aborted = true;
               throw new CaptureReloadStormError(
-                `capture aborted: the devserver reloaded during the last ${stormWindow} storyboards in a row (${reloadRetriesTotal} re-captures so far) \u2014 it is re-optimizing or restarting continuously; see the "devserver:" lines above`
+                `capture aborted: the document reloaded during the last ${stormWindow} storyboards in a row (${reloadRetriesTotal} re-captures so far) \u2014 check the devserver output and storyboard navigation for repeated reloads`
               );
             }
             if (result.status === "render-error") {
@@ -4813,7 +4825,7 @@ async function parcelEventToFileEvent(event, rootInfo) {
 // ../tempo-sdk/package.json
 var package_default = {
   name: "tempo-sdk",
-  version: "0.0.42",
+  version: "0.0.44",
   type: "module",
   description: "Tempo SDK \u2014 Vite, Next.js, and Expo plugins for JSX annotation and shared page/storyboard types",
   repository: {
@@ -4894,7 +4906,7 @@ var package_default = {
     demo: "TEMPO=true pnpm --filter @tempo-modules/vite-annotate-plugin-demo dev",
     "demo:nextjs": "TEMPO=true pnpm --filter @tempo-modules/tempo-sdk-nextjs-demo dev",
     test: "vitest run",
-    "test:browser": "vitest run canvas-host-vite-e2e.test.ts canvas-host-packed-sdk-e2e.test.ts error-tracking-e2e.test.ts",
+    "test:browser": "vitest run canvas-capture-determinism-browser.test.ts canvas-host-vite-e2e.test.ts canvas-host-packed-sdk-e2e.test.ts error-tracking-e2e.test.ts",
     "test:watch": "vitest",
     "test:nextjs:unit": "vitest run nextjs-loader.test.ts nextjs-wrapper.test.ts nextjs-fixtures.test.ts nextjs-published.test.ts",
     "test:nextjs:integration": "vitest run nextjs-e2e-correctness.test.ts",
@@ -8524,8 +8536,9 @@ Hint: ${current.hint}` : "";
     if (!publicUrl) {
       const crashed = this.lastExitCrashed;
       const probeBlocked = !crashed && !this.stopped && !probeResult.sawResponder;
-      const wrongResponder = !crashed && !this.stopped && probeResult.sawResponder && !probeResult.identityFailure;
-      const identityFailure = !crashed && !this.stopped && probeResult.identityFailure;
+      const canClassifyResponder = !crashed && this.hasLivenessAuthority();
+      const wrongResponder = canClassifyResponder && probeResult.sawResponder && !probeResult.identityFailure;
+      const identityFailure = canClassifyResponder && probeResult.identityFailure;
       let message = crashed ? "Dev server exited before its URL became reachable" : "Tempo host URL did not become reachable in time";
       if (wrongResponder) {
         const target = publicUrlCandidates[0] ?? "";
@@ -13846,4 +13859,4 @@ export {
   buildBaselineCaptureArgs,
   main
 };
-//# sourceMappingURL=chunk-2EVRJIRQ.js.map
+//# sourceMappingURL=chunk-7P5FPOFE.js.map