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

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-WNMIFR4A.js")).main(commandArgs);
     case "claim":
       return await (await import("./claim-J654JAS6.js")).main(commandArgs);
     case "report":

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

@@ -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) {
@@ -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;
@@ -1051,10 +1053,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 +1068,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 +1395,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") {
@@ -8524,8 +8528,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 +13851,4 @@ export {
   buildBaselineCaptureArgs,
   main
 };
-//# sourceMappingURL=chunk-2EVRJIRQ.js.map
+//# sourceMappingURL=chunk-L2AOGS53.js.map