@skyramp/mcp 0.2.7 → 0.2.8

package/build/utils/scenarioDrafting.js

@@ -142,7 +142,7 @@ export function inferResourceRelationships(endpoints) {
     }
     return relationships;
 }
-export function draftScenariosFromEndpoints(endpoints, newEndpoints = [], wsAuthType, options = {}) {
+export function draftScenariosFromEndpoints(endpoints, newEndpoints = [], wsAuthType, options = {}, removedEndpoints = []) {
     const scenarios = [];
     const resourceGroups = new Map();
     for (const ep of endpoints) {
@@ -191,6 +191,28 @@ export function draftScenariosFromEndpoints(endpoints, newEndpoints = [], wsAuth
     }
     scenarios.push(...draftDiffDirectScenarios(newEndpoints, resourceGroups, wsAuthType));
     scenarios.push(...draftAttackSurfaceExpansionScenarios(endpoints, options.changedEndpoints ?? [], wsAuthType, options.securityRelevantDiff ?? false));
+    // 404-guard scenarios for removed endpoints — absence assertions confirming
+    // the endpoint is gone. Single-step contract tests, no chaining needed.
+    for (const ep of removedEndpoints) {
+        scenarios.push({
+            scenarioName: `verify-removed-${ep.method.toLowerCase()}-${ep.path.replace(/[^a-z0-9]+/gi, "-").replace(/^-|-$/g, "")}`,
+            description: `Verify ${ep.method} ${ep.path} returns 404 — endpoint was removed in this PR`,
+            category: "breaking_change",
+            priority: "high",
+            steps: [{
+                    order: 1,
+                    method: ep.method,
+                    path: ep.path,
+                    description: `${ep.method} ${ep.path} → 404 (endpoint removed)`,
+                    interactionType: "error",
+                    expectedStatusCode: 404,
+                }],
+            chainingKeys: [],
+            requiresAuth: false,
+            estimatedComplexity: "simple",
+            source: ScenarioSource.CodeInferred,
+        });
+    }
     return capScenarios(scenarios);
 }
 const MAX_TOTAL_SCENARIOS = 30;

package/build/utils/utils.d.ts

@@ -1,4 +1,4 @@
-import { CallToolResult } from "@modelcontextprotocol/sdk/types.js";
+import { CallToolResult, ReadResourceResult } from "@modelcontextprotocol/sdk/types.js";
 export declare function readDiffFile(diffFilePath: string | undefined): string | undefined;
 export declare function toolError(message: string): CallToolResult;
 /**
@@ -7,6 +7,12 @@ export declare function toolError(message: string): CallToolResult;
  * `{ content: [{ type: "text", text }] }` boilerplate themselves.
  */
 export declare function toolText(text: string): CallToolResult;
+/**
+ * Wraps a single text payload in the MCP resource-result shape so resource
+ * handlers don't have to construct
+ * `{ contents: [{ uri, mimeType, text }] }` boilerplate themselves.
+ */
+export declare function resourceText(uri: string, text: string, mimeType?: string): ReadResourceResult;
 export declare const OUTPUT_DIR_FIELD_NAME = "outputDir";
 export declare const TRACE_OUTPUT_FILE_FIELD_NAME = "traceOutputFile";
 export declare const PLAYWRIGHT_OUTPUT_FILE_FIELD_NAME = "playwrightOutput";

package/build/utils/utils.js

@@ -28,6 +28,16 @@ export function toolText(text) {
         content: [{ type: "text", text }],
     };
 }
+/**
+ * Wraps a single text payload in the MCP resource-result shape so resource
+ * handlers don't have to construct
+ * `{ contents: [{ uri, mimeType, text }] }` boilerplate themselves.
+ */
+export function resourceText(uri, text, mimeType = "text/plain") {
+    return {
+        contents: [{ uri, mimeType, text }],
+    };
+}
 export const OUTPUT_DIR_FIELD_NAME = "outputDir";
 export const TRACE_OUTPUT_FILE_FIELD_NAME = "traceOutputFile";
 export const PLAYWRIGHT_OUTPUT_FILE_FIELD_NAME = "playwrightOutput";

package/build/utils/versions.d.ts

@@ -1,3 +1,3 @@
-export declare const SKYRAMP_IMAGE_VERSION = "v1.3.28";
-export declare const EXECUTOR_DOCKER_IMAGE = "skyramp/executor:v1.3.28";
-export declare const WORKER_DOCKER_IMAGE = "skyramp/worker:v1.3.28";
+export declare const SKYRAMP_IMAGE_VERSION = "v1.3.29";
+export declare const EXECUTOR_DOCKER_IMAGE = "skyramp/executor:v1.3.29";
+export declare const WORKER_DOCKER_IMAGE = "skyramp/worker:v1.3.29";

package/build/utils/versions.js

@@ -1,3 +1,3 @@
-export const SKYRAMP_IMAGE_VERSION = "v1.3.28";
+export const SKYRAMP_IMAGE_VERSION = "v1.3.29";
 export const EXECUTOR_DOCKER_IMAGE = `skyramp/executor:${SKYRAMP_IMAGE_VERSION}`;
 export const WORKER_DOCKER_IMAGE = `skyramp/worker:${SKYRAMP_IMAGE_VERSION}`;

package/build/workspace/workspace.d.ts

@@ -3,7 +3,7 @@ export declare const WORKSPACE_DIR = ".skyramp";
 export declare const WORKSPACE_FILENAME = "workspace.yml";
 export declare const serviceSchema: z.ZodObject<{
     serviceName: z.ZodString;
-    repo: z.ZodOptional<z.ZodString>;
+    repository: z.ZodOptional<z.ZodString>;
     language: z.ZodOptional<z.ZodEnum<["python", "typescript", "javascript", "java"]>>;
     framework: z.ZodOptional<z.ZodEnum<["playwright", "pytest", "robot", "junit"]>>;
     testDirectory: z.ZodOptional<z.ZodString>;
@@ -49,7 +49,7 @@ export declare const serviceSchema: z.ZodObject<{
     serviceName: string;
     language?: "python" | "typescript" | "javascript" | "java" | undefined;
     framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
-    repo?: string | undefined;
+    repository?: string | undefined;
     testDirectory?: string | undefined;
     api?: {
         authHeader?: string | undefined;
@@ -69,7 +69,7 @@ export declare const serviceSchema: z.ZodObject<{
     serviceName: string;
     language?: "python" | "typescript" | "javascript" | "java" | undefined;
     framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
-    repo?: string | undefined;
+    repository?: string | undefined;
     testDirectory?: string | undefined;
     api?: {
         authHeader?: string | undefined;
@@ -118,7 +118,7 @@ export declare const workspaceConfigSchema: z.ZodObject<{
     }>>;
     services: z.ZodOptional<z.ZodArray<z.ZodObject<{
         serviceName: z.ZodString;
-        repo: z.ZodOptional<z.ZodString>;
+        repository: z.ZodOptional<z.ZodString>;
         language: z.ZodOptional<z.ZodEnum<["python", "typescript", "javascript", "java"]>>;
         framework: z.ZodOptional<z.ZodEnum<["playwright", "pytest", "robot", "junit"]>>;
         testDirectory: z.ZodOptional<z.ZodString>;
@@ -164,7 +164,7 @@ export declare const workspaceConfigSchema: z.ZodObject<{
         serviceName: string;
         language?: "python" | "typescript" | "javascript" | "java" | undefined;
         framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
-        repo?: string | undefined;
+        repository?: string | undefined;
         testDirectory?: string | undefined;
         api?: {
             authHeader?: string | undefined;
@@ -184,7 +184,7 @@ export declare const workspaceConfigSchema: z.ZodObject<{
         serviceName: string;
         language?: "python" | "typescript" | "javascript" | "java" | undefined;
         framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
-        repo?: string | undefined;
+        repository?: string | undefined;
         testDirectory?: string | undefined;
         api?: {
             authHeader?: string | undefined;
@@ -217,7 +217,7 @@ export declare const workspaceConfigSchema: z.ZodObject<{
         serviceName: string;
         language?: "python" | "typescript" | "javascript" | "java" | undefined;
         framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
-        repo?: string | undefined;
+        repository?: string | undefined;
         testDirectory?: string | undefined;
         api?: {
             authHeader?: string | undefined;
@@ -250,7 +250,7 @@ export declare const workspaceConfigSchema: z.ZodObject<{
         serviceName: string;
         language?: "python" | "typescript" | "javascript" | "java" | undefined;
         framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
-        repo?: string | undefined;
+        repository?: string | undefined;
         testDirectory?: string | undefined;
         api?: {
             authHeader?: string | undefined;
@@ -291,7 +291,7 @@ export declare function validateWorkspaceConfig(config: unknown): z.SafeParseRet
         serviceName: string;
         language?: "python" | "typescript" | "javascript" | "java" | undefined;
         framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
-        repo?: string | undefined;
+        repository?: string | undefined;
         testDirectory?: string | undefined;
         api?: {
             authHeader?: string | undefined;
@@ -324,7 +324,7 @@ export declare function validateWorkspaceConfig(config: unknown): z.SafeParseRet
         serviceName: string;
         language?: "python" | "typescript" | "javascript" | "java" | undefined;
         framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
-        repo?: string | undefined;
+        repository?: string | undefined;
         testDirectory?: string | undefined;
         api?: {
             authHeader?: string | undefined;

package/build/workspace/workspace.js

@@ -27,9 +27,9 @@ export const serviceSchema = z
     .object({
     serviceName: z.string(),
     // owner/repo this service belongs to (multi-repo workspaces). Absent → the
-    // primary repo. Service identity is the composite (repo, serviceName), so the
-    // same serviceName can appear once per repo without colliding.
-    repo: z.string().optional(),
+    // primary repo. Service identity is the composite (repository, serviceName),
+    // so the same serviceName can appear once per repo without colliding.
+    repository: z.string().optional(),
     language: z.enum(["python", "typescript", "javascript", "java"]).optional(),
     framework: z.enum(["playwright", "pytest", "robot", "junit"]).optional(),
     testDirectory: z.string().optional(),
@@ -318,11 +318,11 @@ export class WorkspaceConfigManager {
         const validatedService = svcResult.data;
         const config = await this.read();
         const services = config.services || [];
-        // Upsert keyed on the composite (repo, serviceName). Absent repo normalizes
-        // to "" (primary) so a primary service and a same-named additional-repo
+        // Upsert keyed on the composite (repository, serviceName). Absent repository
+        // normalizes to "" (primary) so a primary service and a same-named related-repo
         // service are distinct entries.
-        const repoKey = validatedService.repo || "";
-        const idx = services.findIndex((s) => (s.repo || "") === repoKey &&
+        const repoKey = validatedService.repository || "";
+        const idx = services.findIndex((s) => (s.repository || "") === repoKey &&
             s.serviceName === validatedService.serviceName);
         if (idx >= 0) {
             services[idx] = validatedService;

package/build/workspace/workspace.test.js

@@ -6,10 +6,10 @@ import yaml from "js-yaml";
 import { serviceSchema, WorkspaceConfigManager, validateWorkspaceConfig, createDefaultConfig, WORKSPACE_DIR, WORKSPACE_FILENAME, } from "./workspace.js";
 describe("workspace module (ported from @skyramp/skyramp, SKYR-3794)", () => {
     describe("serviceSchema", () => {
-        it("accepts an optional repo and rejects a non-string repo", () => {
-            expect(serviceSchema.safeParse({ serviceName: "api", repo: "org/api" }).success).toBe(true);
+        it("accepts an optional repository and rejects a non-string repository", () => {
+            expect(serviceSchema.safeParse({ serviceName: "api", repository: "org/api" }).success).toBe(true);
             expect(serviceSchema.safeParse({ serviceName: "api" }).success).toBe(true);
-            expect(serviceSchema.safeParse({ serviceName: "api", repo: 123 }).success).toBe(false);
+            expect(serviceSchema.safeParse({ serviceName: "api", repository: 123 }).success).toBe(false);
         });
         it("rejects unknown fields (strict)", () => {
             expect(serviceSchema.safeParse({ serviceName: "api", bogus: true }).success).toBe(false);
@@ -42,12 +42,12 @@ describe("workspace module (ported from @skyramp/skyramp, SKYR-3794)", () => {
             const mgr = new WorkspaceConfigManager(repoDir);
             await mgr.initialize();
             await mgr.addService({ serviceName: "svc", testDirectory: "tests/primary" });
-            await mgr.addService({ serviceName: "svc", repo: "letsramp/api-insight", testDirectory: "tests/py" });
+            await mgr.addService({ serviceName: "svc", repository: "letsramp/api-insight", testDirectory: "tests/py" });
             await mgr.addService({ serviceName: "svc", testDirectory: "tests/primary-v2" }); // upsert primary
             const cfg = read();
             expect(cfg.services).toHaveLength(2);
-            expect(cfg.services.find((s) => !s.repo).testDirectory).toBe("tests/primary-v2");
-            expect(cfg.services.find((s) => s.repo === "letsramp/api-insight").testDirectory).toBe("tests/py");
+            expect(cfg.services.find((s) => !s.repository).testDirectory).toBe("tests/primary-v2");
+            expect(cfg.services.find((s) => s.repository === "letsramp/api-insight").testDirectory).toBe("tests/py");
             // repoName auto-detected from the git remote.
             expect(cfg.workspace.repoName).toBe("letsramp/primary");
         });

package/node_modules/playwright/lib/mcp/skyramp/loadTraceTool.js

@@ -36,20 +36,23 @@ const loadTraceSchema = {
   name: "skyramp_load_trace",
   title: "Load and replay a Skyramp trace",
   description: [
-    "Load a previously recorded Skyramp trace (.zip or .jsonl/.txt) and replay its actions against the live browser to restore state, then continue recording from that point.",
-    "The replayed actions are merged into the current recording, so subsequent browser_* steps append to them and skyramp_export_zip writes a combined trace.",
-    "STOPPING EARLY: when the user wants to stop partway through the loaded trace before continuing, set ONE of stopAtStep / stopAtUrl / stopBefore based on their prompt. Omit all three to replay the whole trace.",
-    "- stopAtStep: replay steps 1..N then stop (N is 1-based, from the numbered step list).",
+    "Load a previously recorded Skyramp trace (.zip or .jsonl/.txt) and replay a range of its actions against the live browser to restore state, then continue recording from that point.",
+    "The replayed actions are merged into the current recording, so subsequent browser_* steps append to them and skyramp_export_zip writes a combined trace. You can record some steps FIRST, then load a trace to insert an already-recorded fragment at the current point.",
+    "STARTING LATER (skip a stale prefix): set startAtStep to begin replay partway in, skipping earlier steps. Use this when the live browser is already past the trace's opening steps - e.g. the login page changed, you recorded the NEW login manually, and now want to append only the still-valid post-login part of the old trace. Omit to start from step 1.",
+    "- startAtStep: begin replay at step N (1-based, from the numbered step list); steps 1..N-1 are skipped (neither replayed nor recorded).",
+    "STOPPING EARLY: when the user wants to stop partway through the loaded trace before continuing, set ONE of stopAtStep / stopAtUrl / stopBefore based on their prompt. Omit all three to replay to the end.",
+    "- stopAtStep: replay up to and including step N then stop (N is 1-based, from the numbered step list).",
     "- stopAtUrl: stop once the active page URL matches this glob/regex/substring (checked after each action).",
     '- stopBefore: stop right BEFORE the first action whose description contains this text (case-insensitive), e.g. "Checkout" to stop before clicking Checkout.',
-    "If unsure where to stop, first call with dryRun:true to read the numbered step list, then call again with the chosen stop parameter."
+    "If unsure where to start/stop, first call with dryRun:true to read the numbered step list, then call again with the chosen start/stop parameters."
   ].join(" "),
   inputSchema: import_mcpBundle.z.object({
     path: import_mcpBundle.z.string().describe("Absolute path to the trace file (.zip, .jsonl, or .txt)."),
-    stopAtStep: import_mcpBundle.z.number().int().positive().optional().describe("Replay steps 1..N (1-based), then stop before continuing."),
+    startAtStep: import_mcpBundle.z.number().int().positive().optional().describe("Begin replay at step N (1-based); skip steps 1..N-1. Use to skip a stale prefix (e.g. old login)."),
+    stopAtStep: import_mcpBundle.z.number().int().positive().optional().describe("Replay up to and including step N (1-based), then stop before continuing."),
     stopAtUrl: import_mcpBundle.z.string().optional().describe("Stop once the active page URL matches this glob/regex/substring."),
     stopBefore: import_mcpBundle.z.string().optional().describe("Stop before the first action whose description contains this text (case-insensitive)."),
-    dryRun: import_mcpBundle.z.boolean().optional().describe("Parse and list the steps WITHOUT replaying. Use to choose a stop point."),
+    dryRun: import_mcpBundle.z.boolean().optional().describe("Parse and list the steps WITHOUT replaying. Use to choose start/stop points."),
     speed: import_mcpBundle.z.enum(["fast", "slow"]).optional().describe('Replay speed. "slow" adds a 1s delay between actions (default "fast").')
   }),
   // Replaying drives the live browser and mutates the recording, so this is an
@@ -134,17 +137,30 @@ function describeStopReason(result) {
 async function replayActions(actionsList, callbacks, params) {
   const delay = params.speed === "slow" ? 1e3 : 0;
   const stopBefore = params.stopBefore?.toLowerCase();
-  let completedCount = 0;
-  for (let i = 0; i < actionsList.length; i++) {
+  const startIndex = params.startAtStep ? Math.max(0, Math.min(params.startAtStep - 1, actionsList.length)) : 0;
+  let completedCount = startIndex;
+  for (let i = startIndex; i < actionsList.length; i++) {
     const action = actionsList[i];
     if (stopBefore && describeStep(action, i).toLowerCase().includes(stopBefore))
-      return { completedCount, stopIndex: i, stopReason: "stopBefore" };
+      return { completedCount, startIndex, stopIndex: i, stopReason: "stopBefore" };
+    try {
+      callbacks.beforeAction?.(action);
+    } catch (e) {
+      return {
+        completedCount,
+        startIndex,
+        stopIndex: i,
+        stopReason: "error",
+        error: { message: e.message, actionName: action.action.name, stepIndex: i + 1 }
+      };
+    }
     const alias = action.frame.pageAlias || "page";
     const page = callbacks.pageForAlias(alias);
     if (!METADATA_ONLY_ACTIONS.has(action.action.name)) {
       if (!page) {
         return {
           completedCount,
+          startIndex,
           stopIndex: i,
           stopReason: "error",
           error: { message: `No live page for alias "${alias}"`, actionName: action.action.name, stepIndex: i + 1 }
@@ -152,9 +168,11 @@ async function replayActions(actionsList, callbacks, params) {
       }
       try {
         await performClientAction(page, action);
+        await callbacks.afterAction?.(action);
       } catch (e) {
         return {
           completedCount,
+          startIndex,
           stopIndex: i,
           stopReason: "error",
           error: { message: e.message, actionName: action.action.name, stepIndex: i + 1 }
@@ -163,16 +181,16 @@ async function replayActions(actionsList, callbacks, params) {
     }
     completedCount = i + 1;
     if (params.stopAtStep !== void 0 && completedCount >= params.stopAtStep)
-      return { completedCount, stopIndex: i + 1, stopReason: "stopAtStep" };
+      return { completedCount, startIndex, stopIndex: i + 1, stopReason: "stopAtStep" };
     if (params.stopAtUrl) {
       const currentUrl = page?.url() ?? "";
       if (urlMatchesPattern(currentUrl, params.stopAtUrl))
-        return { completedCount, stopIndex: i + 1, stopReason: "stopAtUrl" };
+        return { completedCount, startIndex, stopIndex: i + 1, stopReason: "stopAtUrl" };
     }
     if (delay)
       await new Promise((resolve) => setTimeout(resolve, delay));
   }
-  return { completedCount, stopIndex: actionsList.length, stopReason: "done" };
+  return { completedCount, startIndex, stopIndex: actionsList.length, stopReason: "done" };
 }
 function urlMatchesPattern(currentUrl, pattern) {
   const regexLiteral = /^\/(.*)\/([a-z]*)$/i.exec(pattern);

package/node_modules/playwright/lib/mcp/skyramp/traceRecordingBackend.js

@@ -111,6 +111,11 @@ class TraceRecordingBackend {
         },
         contextOptions: {
           viewport: import_skyRampExport.RECORDING_VIEWPORT,
+          // Mirror `playwright codegen --load-storage`: seed the recording
+          // browser with a saved session so it records against an authenticated
+          // app. Same source (PLAYWRIGHT_MCP_STORAGE_STATE) that buildJsonlContent
+          // stamps into the export header, so load + emit stay consistent.
+          ...process.env.PLAYWRIGHT_MCP_STORAGE_STATE?.trim() ? { storageState: process.env.PLAYWRIGHT_MCP_STORAGE_STATE.trim() } : {},
           recordHar: { path: this._harPath, mode: "minimal" },
           ignoreHTTPSErrors: true,
           serviceWorkers: "block"
@@ -520,6 +525,9 @@ Failed to load trace from ${params.path}: ${e.message}` }], isError: true };
     if (!loadedActions.length)
       return { content: [{ type: "text", text: `### Error
 No actions found in ${params.path}.` }], isError: true };
+    if (params.startAtStep !== void 0 && params.stopAtStep !== void 0 && params.startAtStep > params.stopAtStep)
+      return { content: [{ type: "text", text: `### Error
+startAtStep (${params.startAtStep}) is after stopAtStep (${params.stopAtStep}).` }], isError: true };
     if (params.dryRun) {
       const stepList = loadedActions.map((a, i) => (0, import_loadTraceTool.describeStep)(a, i)).join("\n");
       return {
@@ -531,7 +539,7 @@ ${loadedActions.length} steps in ${params.path}. Nothing was replayed.
 Steps:
 ${stepList}
 
-Call skyramp_load_trace again with stopAtStep / stopAtUrl / stopBefore (or none of them to replay all) to replay.`
+Call skyramp_load_trace again to replay: set startAtStep to skip a stale prefix (e.g. an old login) and/or one of stopAtStep / stopAtUrl / stopBefore to stop early. Omit all to replay the whole trace.`
         }]
       };
     }
@@ -539,17 +547,83 @@ Call skyramp_load_trace again with stopAtStep / stopAtUrl / stopBefore (or none
       await this._browserBackend.context.ensureTab();
     } catch {
     }
+    const startIndex = params.startAtStep ? Math.max(0, Math.min(params.startAtStep - 1, loadedActions.length)) : 0;
+    let guardEnd = loadedActions.length;
+    if (params.stopAtStep) {
+      guardEnd = Math.min(guardEnd, params.stopAtStep);
+    } else if (params.stopBefore) {
+      const needle = params.stopBefore.toLowerCase();
+      const matchIndex = loadedActions.findIndex((a, i) => i >= startIndex && (0, import_loadTraceTool.describeStep)(a, i).toLowerCase().includes(needle));
+      if (matchIndex !== -1)
+        guardEnd = matchIndex;
+    }
+    const replayRange = loadedActions.slice(startIndex, guardEnd);
+    const primaryAlias = replayRange[0]?.frame.pageAlias || "page";
+    {
+      const available = /* @__PURE__ */ new Set([primaryAlias]);
+      for (const a of replayRange) {
+        if (!available.has(a.frame.pageAlias))
+          return { content: [{ type: "text", text: `### Error
+The selected range acts on tab "${a.frame.pageAlias}" but the step that opens that tab is outside the range (skipped by startAtStep). Lower startAtStep to include the opening step, or pick a range that stays on a single tab.` }], isError: true };
+        for (const s of a.action.signals) {
+          if (s.name === "popup")
+            available.add(s.popupAlias);
+        }
+      }
+    }
+    const liveTabs = () => (this._browserBackend.context?.tabs() ?? []).map((t) => t.page);
+    const seedPage = this._browserBackend.context?.currentTab()?.page ?? liveTabs()[0];
+    const remap = /* @__PURE__ */ new Map();
+    remap.set(primaryAlias, { page: seedPage, liveAlias: this._currentPageAlias });
+    const claimedTabs = /* @__PURE__ */ new Set();
+    if (seedPage)
+      claimedTabs.add(seedPage);
+    let tabsBefore = [];
     let result;
     try {
-      result = await (0, import_loadTraceTool.replayActions)(loadedActions, { pageForAlias: (alias) => this._pageForAlias(alias) }, params);
+      result = await (0, import_loadTraceTool.replayActions)(loadedActions, {
+        pageForAlias: (alias) => remap.get(alias)?.page ?? this._pageForAlias(alias),
+        beforeAction: () => {
+          tabsBefore = liveTabs();
+        },
+        afterAction: async (action) => {
+          for (const signal of action.action.signals) {
+            if (signal.name !== "popup")
+              continue;
+            let fresh;
+            for (let k = 0; k < 100 && !fresh; k++) {
+              fresh = liveTabs().find((p) => !tabsBefore.includes(p) && !claimedTabs.has(p));
+              if (!fresh)
+                await new Promise((r) => setTimeout(r, 50));
+            }
+            if (!fresh) {
+              throw new Error(`Expected a popup ("${signal.popupAlias}") to open after this action, but no new tab appeared \u2014 the fragment has drifted from the live page.`);
+            }
+            claimedTabs.add(fresh);
+            remap.set(signal.popupAlias, { page: fresh, liveAlias: this._currentPageAlias });
+          }
+        }
+      }, params);
     } catch (e) {
       return { content: [{ type: "text", text: `### Error
 Replay failed: ${e.message}` }], isError: true };
     }
-    for (let i = 0; i < result.stopIndex; i++)
-      this._seedTrackedAction(loadedActions[i]);
-    if (result.stopIndex > 0)
+    for (let i = result.startIndex; i < result.stopIndex; i++) {
+      const a = loadedActions[i];
+      const liveAlias = remap.get(a.frame.pageAlias)?.liveAlias;
+      if (liveAlias && liveAlias !== a.frame.pageAlias)
+        a.frame = { ...a.frame, pageAlias: liveAlias };
+      const before = this._trackedActions.length;
+      this._seedTrackedAction(a);
+      const popupSignal = a.action.signals.find((s) => s.name === "popup");
+      if (popupSignal && this._trackedActions.length > before)
+        this._trackedActions[this._trackedActions.length - 1].popupAlias = remap.get(popupSignal.popupAlias)?.liveAlias ?? popupSignal.popupAlias;
+    }
+    if (result.stopIndex > result.startIndex)
       this._currentPageAlias = loadedActions[result.stopIndex - 1].frame.pageAlias || "page";
+    const replayedCount = Math.max(0, result.stopIndex - result.startIndex);
+    const clamped = params.startAtStep !== void 0 && params.startAtStep !== result.startIndex + 1;
+    const skippedNote = result.startIndex > 0 ? ` Skipped the first ${result.startIndex} step${result.startIndex === 1 ? "" : "s"} (startAtStep ${params.startAtStep}${clamped ? `, clamped \u2014 trace has ${loadedActions.length} steps` : ""}).` : "";
     const remaining = (0, import_loadTraceTool.listStepsFrom)(loadedActions, result.stopIndex);
     const stoppedNote = (0, import_loadTraceTool.describeStopReason)(result);
     const errorNote = result.error ? `
@@ -560,7 +634,7 @@ You can continue recording from this point or fix the issue.` : "";
       content: [{
         type: "text",
         text: `### Trace replayed
-${result.completedCount} of ${loadedActions.length} steps replayed from ${params.path}. ${stoppedNote}${errorNote}
+${replayedCount} of ${loadedActions.length} steps replayed from ${params.path}.${skippedNote} ${stoppedNote}${errorNote}
 
 Remaining steps (NOT replayed):
 ${remaining}
@@ -1658,10 +1732,9 @@ Cell row ${params.row}, column ${params.column} has "${actual}", expected "${par
     const url = page.url();
     if (url.startsWith("chrome-extension://") || url.startsWith("about:") || url.startsWith("file://"))
       return;
-    const alreadyEnabled = await page.evaluate(() => {
-      const host = document.querySelector("flt-semantics-host");
-      return host && host.children.length > 0;
-    }).catch(() => false);
+    const alreadyEnabled = await page.evaluate(
+      () => document.querySelector("flt-semantics-placeholder") === null && document.querySelector("flt-semantics-host") !== null
+    ).catch(() => false);
     if (alreadyEnabled) {
       traceDebug("Flutter accessibility already enabled");
       return;
@@ -1689,14 +1762,14 @@ Cell row ${params.row}, column ${params.column} has "${actual}", expected "${par
       traceDebug("Flutter detected but placeholder click failed");
       return;
     }
-    const enabled = await page.waitForFunction(() => {
-      const host = document.querySelector("flt-semantics-host");
-      return host && host.children.length > 0;
-    }, { timeout: 5e3 }).then(() => true).catch(() => false);
+    const enabled = await page.waitForFunction(
+      () => document.querySelector("flt-semantics-placeholder") === null,
+      { timeout: 1e4 }
+    ).then(() => true).catch(() => false);
     if (enabled)
       traceDebug("Flutter accessibility mode enabled");
     else
-      traceDebug("Flutter placeholder clicked but semantics host did not appear");
+      traceDebug("Flutter placeholder clicked but semantics not confirmed active");
   }
   /** Clean up temp directory and optionally recreate it for next session. */
   _cleanupTempDir(recreate = false) {

package/node_modules/playwright/lib/mcp/test/skyRampExport.js

@@ -55,12 +55,13 @@ function modifiersArrayToMask(modifiers) {
   return mask;
 }
 const RECORDING_VIEWPORT = { width: 1280, height: 900 };
-function jsonlHeader(browserName, harPath, viewport) {
+function jsonlHeader(browserName, harPath, viewport, storageState) {
   return JSON.stringify({
     browserName,
     launchOptions: { headless: false },
     contextOptions: {
       viewport,
+      ...storageState ? { storageState } : {},
       ignoreHTTPSErrors: true,
       recordHar: { path: harPath, mode: "minimal" },
       serviceWorkers: "block"
@@ -623,7 +624,7 @@ function mouseActionToJsonl(action, pageGuid, baseTs, alias) {
       return [];
   }
 }
-function buildJsonlContent(actions, browserName, harPath, viewport = RECORDING_VIEWPORT) {
+function buildJsonlContent(actions, browserName, harPath, viewport = RECORDING_VIEWPORT, storageState = process.env.PLAYWRIGHT_MCP_STORAGE_STATE?.trim() || void 0) {
   normalizePrimaryPageAlias(actions);
   const deduplicated = removeRedundantClicks(deduplicateRetries(actions));
   const startTime = deduplicated[0]?.timestamp ?? Date.now();
@@ -635,7 +636,7 @@ function buildJsonlContent(actions, browserName, harPath, viewport = RECORDING_V
   };
   const mainGuid = getPageGuid(DEFAULT_PAGE_ALIAS);
   const lines = [
-    jsonlHeader(browserName, harPath, viewport),
+    jsonlHeader(browserName, harPath, viewport, storageState),
     JSON.stringify({ name: "openPage", url: "about:blank", signals: [], timestamp: String(startTime - 1e3), pageGuid: mainGuid, pageAlias: DEFAULT_PAGE_ALIAS, framePath: DEFAULT_FRAME_PATH })
   ];
   let actionCount = 0;