@alwaysmeticulous/cli 2.45.1 → 2.45.3

package/dist/commands/replay/replay.command.d.ts

@@ -1,21 +1,5 @@
 /// <reference types="yargs" />
-import { Replay, ScreenshotDiffOptions } from "@alwaysmeticulous/api";
-import { ReplayExecutionOptions, ReplayTarget } from "@alwaysmeticulous/sdk-bundles-api";
-export interface RawReplayCommandHandlerOptions extends ScreenshotDiffOptions, Omit<ReplayExecutionOptions, "maxDurationMs" | "maxEventCount"> {
-    screenshot: boolean;
-    appUrl: string | null | undefined;
-    simulationIdForAssets: string | null | undefined;
-    maxDurationMs: number | null | undefined;
-    maxEventCount: number | null | undefined;
-    storyboard: boolean;
-    apiToken: string | null | undefined;
-    commitSha: string | null | undefined;
-    sessionId: string;
-    cookiesFile: string | null | undefined;
-    debugger: boolean;
-    baseReplayId: string | null | undefined;
-}
-export declare const rawReplayCommandHandler: ({ apiToken, commitSha, sessionId, appUrl, simulationIdForAssets, headless, devTools, bypassCSP, screenshot, baseReplayId, diffThreshold, diffPixelThreshold, shiftTime, networkStubbing, moveBeforeClick, cookiesFile, disableRemoteFonts, noSandbox, skipPauses, maxDurationMs, maxEventCount, storyboard, essentialFeaturesOnly, debugger: enableStepThroughDebugger, }: RawReplayCommandHandlerOptions) => Promise<Replay>;
+import { ReplayTarget } from "@alwaysmeticulous/sdk-bundles-api";
 export declare const getReplayTarget: ({ appUrl, simulationIdForAssets, }: {
     appUrl: string | null;
     simulationIdForAssets: string | null;

package/dist/commands/replay/replay.command.js

@@ -1,13 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.replayCommand = exports.getReplayTarget = exports.rawReplayCommandHandler = void 0;
+exports.replayCommand = exports.getReplayTarget = void 0;
 const common_1 = require("@alwaysmeticulous/common");
 const replay_orchestrator_launcher_1 = require("@alwaysmeticulous/replay-orchestrator-launcher");
 const command_builder_1 = require("../../command-utils/command-builder");
 const common_options_1 = require("../../command-utils/common-options");
 const out_of_date_client_error_1 = require("../../utils/out-of-date-client-error");
 const replay_debugger_ui_1 = require("./utils/replay-debugger.ui");
-const rawReplayCommandHandler = async ({ apiToken, commitSha, sessionId, appUrl, simulationIdForAssets, headless, devTools, bypassCSP, screenshot, baseReplayId, diffThreshold, diffPixelThreshold, shiftTime, networkStubbing, moveBeforeClick, cookiesFile, disableRemoteFonts, noSandbox, skipPauses, maxDurationMs, maxEventCount, storyboard, essentialFeaturesOnly, debugger: enableStepThroughDebugger, }) => {
+const replayCommandHandler = async ({ apiToken, commitSha, sessionId, appUrl, simulationIdForAssets, headless, devTools, bypassCSP, screenshot, baseReplayId, diffThreshold, diffPixelThreshold, shiftTime, networkStubbing, moveBeforeClick, cookiesFile, disableRemoteFonts, noSandbox, skipPauses, maxDurationMs, maxEventCount, storyboard, essentialFeaturesOnly, debugger: enableStepThroughDebugger, }) => {
     if (!screenshot && storyboard) {
         throw new Error("Cannot take storyboard screenshots without taking end state screenshots. Please set '--screenshot' to true, or '--storyboard' to false.");
     }
@@ -81,8 +81,7 @@ const rawReplayCommandHandler = async ({ apiToken, commitSha, sessionId, appUrl,
             getOnBeforeUserEventCallback.resolve(stepThroughDebuggerUI.onBeforeUserEvent);
             getOnClosePageCallback.resolve(stepThroughDebuggerUI.close);
         }
-        const { replay } = await replayExecution.finalResult;
-        return replay;
+        await replayExecution.finalResult;
     }
     catch (error) {
         if ((0, out_of_date_client_error_1.isOutOfDateClientError)(error)) {
@@ -93,7 +92,6 @@ const rawReplayCommandHandler = async ({ apiToken, commitSha, sessionId, appUrl,
         }
     }
 };
-exports.rawReplayCommandHandler = rawReplayCommandHandler;
 const getReplayTarget = ({ appUrl, simulationIdForAssets, }) => {
     if (simulationIdForAssets) {
         return { type: "snapshotted-assets", simulationIdForAssets };
@@ -149,5 +147,5 @@ exports.replayCommand = (0, command_builder_1.buildCommand)("simulate")
     ...common_options_1.SCREENSHOT_DIFF_OPTIONS,
 })
     .handler(async (options) => {
-    await (0, exports.rawReplayCommandHandler)(options);
+    await replayCommandHandler(options);
 });

package/dist/commands/replay/utils/replay-debugger.ui.js

@@ -36,15 +36,42 @@ const openStepThroughDebuggerUI = async ({ onCloseReplayedPage, onLogEventTarget
         index: 0,
         loading: false,
     };
+    let readyPromiseResolve;
+    const readyPromise = new Promise((resolve) => {
+        readyPromiseResolve = resolve;
+    });
+    // This function is called by the UI itself
+    await debuggerPage.exposeFunction("__meticulous__replayDebuggerDispatchEvent", (eventType, data) => {
+        if (eventType === "ready") {
+            return onReady();
+        }
+        if (eventType === "check-next-target") {
+            if (state.index >= replayableEvents.length) {
+                logger.info("End of replay!");
+                return;
+            }
+            const nextEvent = state.events[state.index];
+            return onLogEventTarget(nextEvent);
+        }
+        if (eventType === "play-next-event") {
+            return onPlayNextEvent();
+        }
+        if (eventType === "set-index") {
+            return onAdvanceToIndex(data.index || 0);
+        }
+        logger.info(`[debugger-ui] Warning: received unknown event "${eventType}"`);
+    });
+    const onReady = async () => {
+        readyPromiseResolve();
+        await setState(state);
+    };
     const setState = async (newState) => {
+        await readyPromise;
         state = { ...state, ...newState };
         await debuggerPage.evaluate((state) => {
             window.__meticulous__replayDebuggerSetState(state);
         }, state);
     };
-    const onReady = async () => {
-        await setState(state);
-    };
     let advanceToEvent = null;
     const onBeforeUserEvent = async ({ userEventIndex, }) => {
         await setState({
@@ -72,27 +99,6 @@ const openStepThroughDebuggerUI = async ({ onCloseReplayedPage, onLogEventTarget
         await setState({ loading: true });
         advanceToEvent === null || advanceToEvent === void 0 ? void 0 : advanceToEvent({ nextEventIndexToPauseBefore: targetIndex });
     };
-    // This function is called by the UI itself
-    await debuggerPage.exposeFunction("__meticulous__replayDebuggerDispatchEvent", (eventType, data) => {
-        if (eventType === "ready") {
-            return onReady();
-        }
-        if (eventType === "check-next-target") {
-            if (state.index >= replayableEvents.length) {
-                logger.info("End of replay!");
-                return;
-            }
-            const nextEvent = state.events[state.index];
-            return onLogEventTarget(nextEvent);
-        }
-        if (eventType === "play-next-event") {
-            return onPlayNextEvent();
-        }
-        if (eventType === "set-index") {
-            return onAdvanceToIndex(data.index || 0);
-        }
-        logger.info(`[debugger-ui] Warning: received unknown event "${eventType}"`);
-    });
     logger.info(`[debugger-ui] Navigating to ${uiServer.url}...`);
     const res = await debuggerPage.goto(uiServer.url, {
         waitUntil: "domcontentloaded",

package/dist/utils/sentry.utils.js

@@ -38,13 +38,17 @@ exports.setOptions = setOptions;
 const wrapHandler = function wrapHandler_(handler) {
     return async (args) => {
         await handler(args)
-            .then(() => {
+            .then(async () => {
             var _a;
             const transaction = (_a = Sentry.getCurrentHub().getScope()) === null || _a === void 0 ? void 0 : _a.getTransaction();
             if (transaction !== undefined) {
                 transaction.setStatus("ok");
                 transaction.finish();
             }
+            await Sentry.flush(sentry_1.SENTRY_FLUSH_TIMEOUT.toMillis());
+            // This is required: otherwise the process will hang for a while,
+            // presumably waiting on some setTimeout to trigger
+            process.exit(0);
         })
             .catch(async (error) => {
             var _a;
@@ -54,6 +58,7 @@ const wrapHandler = function wrapHandler_(handler) {
                 transaction.setStatus("unknown_error");
                 transaction.finish();
             }
+            await Sentry.flush(sentry_1.SENTRY_FLUSH_TIMEOUT.toMillis());
             // Don't display the help text which can obscure the error
             process.exit(1);
         });
@@ -67,5 +72,4 @@ const reportHandlerError = async (error) => {
     logger.info("");
     logger.info("Tip: run `meticulous <command> --help` for help on a particular command, or `meticulous --help` for a list of the available commands.");
     Sentry.captureException(error);
-    await Sentry.flush(sentry_1.SENTRY_FLUSH_TIMEOUT.toMillis());
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alwaysmeticulous/cli",
-  "version": "2.45.1",
+  "version": "2.45.3",
   "description": "The Meticulous CLI",
   "license": "ISC",
   "main": "dist/index.js",
@@ -32,9 +32,9 @@
     "@alwaysmeticulous/common": "^2.44.1",
     "@alwaysmeticulous/downloading-helpers": "^2.44.1",
     "@alwaysmeticulous/record": "^2.44.1",
-    "@alwaysmeticulous/replay-debugger-ui": "^2.45.1",
-    "@alwaysmeticulous/replay-orchestrator-launcher": "^2.45.0",
-    "@alwaysmeticulous/sdk-bundles-api": "^2.45.0",
+    "@alwaysmeticulous/replay-debugger-ui": "^2.45.2",
+    "@alwaysmeticulous/replay-orchestrator-launcher": "^2.45.3",
+    "@alwaysmeticulous/sdk-bundles-api": "^2.45.3",
     "@alwaysmeticulous/sentry": "^2.40.0",
     "@sentry/node": "^7.36.0",
     "axios": "^1.2.6",
@@ -79,5 +79,5 @@
     "coverageDirectory": "../coverage",
     "testEnvironment": "node"
   },
-  "gitHead": "05823c058b305c65ca379238d5a8ddf475681a61"
+  "gitHead": "d4e536d8ea889e75077bb9889399016d35082339"
 }