@skyramp/mcp 0.3.0 → 0.3.1-ldw-beta

package/build/utils/mockCompatibility.test.js

@@ -0,0 +1,79 @@
+import { validateMockCompatibility, MockProtocol } from "./mockCompatibility.js";
+describe("validateMockCompatibility", () => {
+    it("REST-only mocks produce no warnings (DNS interception works correctly)", () => {
+        const mocks = [
+            { protocol: MockProtocol.Rest, endpointURL: "http://billing:5001/api/users", method: "GET" },
+            { protocol: MockProtocol.Rest, endpointURL: "http://ams:4000/api/orders", method: "POST" },
+        ];
+        const result = validateMockCompatibility(mocks);
+        expect(result.compatible).toBe(true);
+        expect(result.warnings).toHaveLength(0);
+    });
+    it("REST mocks using loopback URLs get a blocking REST_LOOPBACK_URL warning", () => {
+        const mocks = [
+            { protocol: MockProtocol.Rest, endpointURL: "http://localhost:8080/api/users", method: "GET" },
+            { protocol: MockProtocol.Rest, endpointURL: "http://host.docker.internal:4000/api/orders", method: "POST" },
+            { protocol: MockProtocol.Rest, endpointURL: "http://[::1]:8080/api/status", method: "GET" },
+        ];
+        const result = validateMockCompatibility(mocks, { dockerLocalDev: true });
+        expect(result.compatible).toBe(false);
+        expect(result.hasBlockers).toBe(true);
+        const warning = result.warnings.find((w) => w.code === "REST_LOOPBACK_URL");
+        expect(warning).toBeDefined();
+        expect(warning.blocking).toBe(true);
+        expect(warning.affectedMocks).toContain("http://localhost:8080/api/users");
+        expect(warning.affectedMocks).toContain("http://host.docker.internal:4000/api/orders");
+        expect(warning.affectedMocks).toContain("http://[::1]:8080/api/status");
+    });
+    it("does not block loopback REST URLs outside Docker local-dev preflight mode", () => {
+        const mocks = [
+            { protocol: MockProtocol.Rest, endpointURL: "http://localhost:8080/api/users", method: "GET" },
+        ];
+        const result = validateMockCompatibility(mocks);
+        expect(result.compatible).toBe(true);
+        expect(result.hasBlockers).toBe(false);
+        expect(result.warnings.find((w) => w.code === "REST_LOOPBACK_URL")).toBeUndefined();
+    });
+    it("Kafka mocks produce no compatibility warning when generation payload shape is valid", () => {
+        const mocks = [
+            { protocol: MockProtocol.Kafka, endpointURL: "redpanda:9092", kafkaTopic: "user-events" },
+            { protocol: MockProtocol.Rest, endpointURL: "http://ams:4000/api/users", method: "GET" },
+        ];
+        const result = validateMockCompatibility(mocks);
+        expect(result.compatible).toBe(true);
+        expect(result.hasBlockers).toBe(false);
+        expect(result.warnings.find((w) => w.code === "KAFKA_SDK_INCOMPATIBLE")).toBeUndefined();
+    });
+    it("gRPC mocks produce GRPC_PERSISTENT_CONN warning", () => {
+        const mocks = [
+            { protocol: MockProtocol.Grpc, endpointURL: "grpc://localhost:50051", grpcServiceName: "UserService" },
+            { protocol: MockProtocol.Rest, endpointURL: "http://ams:4000/api/users", method: "GET" },
+        ];
+        const result = validateMockCompatibility(mocks);
+        expect(result.compatible).toBe(true);
+        const grpcWarning = result.warnings.find((w) => w.code === "GRPC_PERSISTENT_CONN");
+        expect(grpcWarning).toBeDefined();
+        expect(grpcWarning.affectedMocks).toContain("UserService");
+    });
+    it("gRPC on non-8080 ports is allowed with only the persistent connection warning", () => {
+        const mocks = [
+            { protocol: MockProtocol.Grpc, endpointURL: "partner-accounts:50051", grpcServiceName: "PaymentService" },
+        ];
+        const result = validateMockCompatibility(mocks);
+        expect(result.compatible).toBe(true);
+        expect(result.hasBlockers).toBe(false);
+        expect(result.warnings).toHaveLength(1);
+        expect(result.warnings[0].code).toBe("GRPC_PERSISTENT_CONN");
+        expect(result.warnings.find((w) => w.code === "GRPC_PORT_MISMATCH")).toBeUndefined();
+    });
+    it("gRPC on port 8080 also gets only GRPC_PERSISTENT_CONN", () => {
+        const mocks = [
+            { protocol: MockProtocol.Grpc, endpointURL: "some-service:8080", grpcServiceName: "ProfileService" },
+        ];
+        const result = validateMockCompatibility(mocks);
+        expect(result.compatible).toBe(true);
+        expect(result.hasBlockers).toBe(false);
+        expect(result.warnings).toHaveLength(1);
+        expect(result.warnings[0].code).toBe("GRPC_PERSISTENT_CONN");
+    });
+});

package/build/utils/progress.js

@@ -10,11 +10,16 @@ export function makeProgressReporter(extra) {
     return async (progress, total, message) => {
         const progressToken = extra._meta?.progressToken;
         if (progressToken !== undefined) {
-            const notification = {
-                method: "notifications/progress",
-                params: { progressToken, progress, total, message },
-            };
-            await extra.sendNotification(notification);
+            try {
+                const notification = {
+                    method: "notifications/progress",
+                    params: { progressToken, progress, total, message },
+                };
+                await extra.sendNotification(notification);
+            }
+            catch {
+                // Progress is best-effort; a disconnected client should not fail the tool.
+            }
         }
     };
 }

package/build/utils/versions.d.ts

@@ -1,3 +1,4 @@
 export declare const SKYRAMP_IMAGE_VERSION = "v1.3.34";
 export declare const EXECUTOR_DOCKER_IMAGE = "skyramp/executor:v1.3.34";
 export declare const WORKER_DOCKER_IMAGE = "skyramp/worker:v1.3.34";
+export declare const WORKER_CONTROL_PORT = 35142;

package/build/utils/versions.js

@@ -1,3 +1,5 @@
 export const SKYRAMP_IMAGE_VERSION = "v1.3.34";
 export const EXECUTOR_DOCKER_IMAGE = `skyramp/executor:${SKYRAMP_IMAGE_VERSION}`;
 export const WORKER_DOCKER_IMAGE = `skyramp/worker:${SKYRAMP_IMAGE_VERSION}`;
+// Control port the Skyramp worker listens on (SDK `CONTAINER_PORT`).
+export const WORKER_CONTROL_PORT = 35142;

package/build/utils/workspaceAuth.d.ts

@@ -125,3 +125,12 @@ export interface WorkspaceAuthConfig {
     authScheme?: string;
 }
 export declare function getWorkspaceAuthConfig(repositoryPath: string): Promise<WorkspaceAuthConfig>;
+/**
+ * Resolve auth header and scheme from workspace config for test generation.
+ * Returns null when auth should be skipped (authType absent or "none").
+ * Single source of truth — used by both TestGenerationService and batch scenario tool.
+ */
+export declare function resolveAuthFromWorkspace(repoPath: string, existingAuthHeader?: string, existingAuthScheme?: string): Promise<{
+    authHeader: string;
+    authScheme?: string;
+} | null>;

package/build/utils/workspaceAuth.js

@@ -77,7 +77,20 @@ export async function getWorkspaceBaseUrl(repositoryPath, testFile, language) {
             const absTestDirectory = path.resolve(repositoryPath, s.testDirectory);
             return absTestFile.startsWith(absTestDirectory + path.sep) || absTestFile === absTestDirectory;
         });
-        // Step 2: narrow by language if still ambiguous
+        // Step 2: longest-prefix match — when multiple services match (e.g.
+        // tests/skyramp is a prefix of tests/skyramp/cart), pick the service
+        // whose testDirectory is the most specific (longest) prefix of the test file.
+        if (matches.length > 1) {
+            matches.sort((a, b) => {
+                const aLen = path.resolve(repositoryPath, a.testDirectory).length;
+                const bLen = path.resolve(repositoryPath, b.testDirectory).length;
+                return bLen - aLen; // longest first
+            });
+            // Keep only services with the longest matching testDirectory length.
+            const longestLen = path.resolve(repositoryPath, matches[0].testDirectory).length;
+            matches = matches.filter((s) => path.resolve(repositoryPath, s.testDirectory).length === longestLen);
+        }
+        // Step 3: narrow by language if still ambiguous, but don't eliminate all matches
         if (matches.length > 1 && language) {
             const byLanguage = matches.filter((s) => s.language?.toLowerCase() === language.toLowerCase());
             if (byLanguage.length > 0)
@@ -330,3 +343,40 @@ export async function getWorkspaceAuthConfig(repositoryPath) {
     const authScheme = svc?.api?.authScheme;
     return { authHeader, authType, authScheme };
 }
+/**
+ * Resolve auth header and scheme from workspace config for test generation.
+ * Returns null when auth should be skipped (authType absent or "none").
+ * Single source of truth — used by both TestGenerationService and batch scenario tool.
+ */
+export async function resolveAuthFromWorkspace(repoPath, existingAuthHeader, existingAuthScheme) {
+    if (existingAuthHeader !== undefined)
+        return null; // caller already specified
+    const wsAuth = await getWorkspaceAuthConfig(repoPath);
+    // Skip auth when explicitly none. When authType is absent, still resolve if
+    // authHeader or authScheme is set (some workspaces specify only the header).
+    if (wsAuth.authType === WorkspaceAuthType.None)
+        return null;
+    if (!wsAuth.authType && !wsAuth.authHeader && !wsAuth.authScheme)
+        return null;
+    let resolvedHeader = wsAuth.authHeader ?? getDefaultAuthHeader(wsAuth.authType);
+    if (!resolvedHeader && wsAuth.authScheme !== undefined) {
+        resolvedHeader = "Authorization";
+    }
+    if (wsAuth.authType === WorkspaceAuthType.ApiKey && !resolvedHeader) {
+        logger.warning("workspace.yml has authType: apiKey but no authHeader is set — " +
+            "requests will be sent without an auth header and will likely receive 401/403. " +
+            "Add authHeader: <X-Your-Key-Header> to .skyramp/workspace.yml.");
+        return null;
+    }
+    if (!resolvedHeader)
+        return null;
+    logger.info("authHeader resolved from workspace config", {
+        authHeader: resolvedHeader,
+        authType: wsAuth.authType,
+    });
+    let authScheme = existingAuthScheme;
+    if (isAuthorizationHeaderName(resolvedHeader)) {
+        authScheme = authScheme ?? wsAuth.authScheme ?? getAuthScheme(wsAuth.authType);
+    }
+    return { authHeader: resolvedHeader, authScheme };
+}

package/build/workspace/workspace.d.ts

@@ -28,22 +28,25 @@ export declare const serviceSchema: z.ZodObject<{
     }>>;
     runtimeDetails: z.ZodOptional<z.ZodObject<{
         serverStartCommand: z.ZodOptional<z.ZodString>;
-        runtime: z.ZodEnum<["local", "docker", "k8s"]>;
+        serverStopCommand: z.ZodOptional<z.ZodString>;
+        runtime: z.ZodEffects<z.ZodNativeEnum<typeof import("../types/TestTypes.js").RuntimeEnvironment>, import("../types/TestTypes.js").RuntimeEnvironment, unknown>;
         dockerNetwork: z.ZodOptional<z.ZodString>;
         k8sNamespace: z.ZodOptional<z.ZodString>;
         k8sContext: z.ZodOptional<z.ZodString>;
     }, "strict", z.ZodTypeAny, {
-        runtime: "local" | "docker" | "k8s";
+        runtime: import("../types/TestTypes.js").RuntimeEnvironment;
         dockerNetwork?: string | undefined;
         k8sNamespace?: string | undefined;
         k8sContext?: string | undefined;
         serverStartCommand?: string | undefined;
+        serverStopCommand?: string | undefined;
     }, {
-        runtime: "local" | "docker" | "k8s";
+        runtime?: unknown;
         dockerNetwork?: string | undefined;
         k8sNamespace?: string | undefined;
         k8sContext?: string | undefined;
         serverStartCommand?: string | undefined;
+        serverStopCommand?: string | undefined;
     }>>;
 }, "strict", z.ZodTypeAny, {
     serviceName: string;
@@ -59,11 +62,12 @@ export declare const serviceSchema: z.ZodObject<{
         baseUrl?: string | undefined;
     } | undefined;
     runtimeDetails?: {
-        runtime: "local" | "docker" | "k8s";
+        runtime: import("../types/TestTypes.js").RuntimeEnvironment;
         dockerNetwork?: string | undefined;
         k8sNamespace?: string | undefined;
         k8sContext?: string | undefined;
         serverStartCommand?: string | undefined;
+        serverStopCommand?: string | undefined;
     } | undefined;
 }, {
     serviceName: string;
@@ -79,11 +83,12 @@ export declare const serviceSchema: z.ZodObject<{
         baseUrl?: string | undefined;
     } | undefined;
     runtimeDetails?: {
-        runtime: "local" | "docker" | "k8s";
+        runtime?: unknown;
         dockerNetwork?: string | undefined;
         k8sNamespace?: string | undefined;
         k8sContext?: string | undefined;
         serverStartCommand?: string | undefined;
+        serverStopCommand?: string | undefined;
     } | undefined;
 }>;
 export declare const workspaceConfigSchema: z.ZodObject<{
@@ -143,22 +148,25 @@ export declare const workspaceConfigSchema: z.ZodObject<{
         }>>;
         runtimeDetails: z.ZodOptional<z.ZodObject<{
             serverStartCommand: z.ZodOptional<z.ZodString>;
-            runtime: z.ZodEnum<["local", "docker", "k8s"]>;
+            serverStopCommand: z.ZodOptional<z.ZodString>;
+            runtime: z.ZodEffects<z.ZodNativeEnum<typeof import("../types/TestTypes.js").RuntimeEnvironment>, import("../types/TestTypes.js").RuntimeEnvironment, unknown>;
             dockerNetwork: z.ZodOptional<z.ZodString>;
             k8sNamespace: z.ZodOptional<z.ZodString>;
             k8sContext: z.ZodOptional<z.ZodString>;
         }, "strict", z.ZodTypeAny, {
-            runtime: "local" | "docker" | "k8s";
+            runtime: import("../types/TestTypes.js").RuntimeEnvironment;
             dockerNetwork?: string | undefined;
             k8sNamespace?: string | undefined;
             k8sContext?: string | undefined;
             serverStartCommand?: string | undefined;
+            serverStopCommand?: string | undefined;
         }, {
-            runtime: "local" | "docker" | "k8s";
+            runtime?: unknown;
             dockerNetwork?: string | undefined;
             k8sNamespace?: string | undefined;
             k8sContext?: string | undefined;
             serverStartCommand?: string | undefined;
+            serverStopCommand?: string | undefined;
         }>>;
     }, "strict", z.ZodTypeAny, {
         serviceName: string;
@@ -174,11 +182,12 @@ export declare const workspaceConfigSchema: z.ZodObject<{
             baseUrl?: string | undefined;
         } | undefined;
         runtimeDetails?: {
-            runtime: "local" | "docker" | "k8s";
+            runtime: import("../types/TestTypes.js").RuntimeEnvironment;
             dockerNetwork?: string | undefined;
             k8sNamespace?: string | undefined;
             k8sContext?: string | undefined;
             serverStartCommand?: string | undefined;
+            serverStopCommand?: string | undefined;
         } | undefined;
     }, {
         serviceName: string;
@@ -194,11 +203,12 @@ export declare const workspaceConfigSchema: z.ZodObject<{
             baseUrl?: string | undefined;
         } | undefined;
         runtimeDetails?: {
-            runtime: "local" | "docker" | "k8s";
+            runtime?: unknown;
             dockerNetwork?: string | undefined;
             k8sNamespace?: string | undefined;
             k8sContext?: string | undefined;
             serverStartCommand?: string | undefined;
+            serverStopCommand?: string | undefined;
         } | undefined;
     }>, "many">>;
 }, "strict", z.ZodTypeAny, {
@@ -227,11 +237,12 @@ export declare const workspaceConfigSchema: z.ZodObject<{
             baseUrl?: string | undefined;
         } | undefined;
         runtimeDetails?: {
-            runtime: "local" | "docker" | "k8s";
+            runtime: import("../types/TestTypes.js").RuntimeEnvironment;
             dockerNetwork?: string | undefined;
             k8sNamespace?: string | undefined;
             k8sContext?: string | undefined;
             serverStartCommand?: string | undefined;
+            serverStopCommand?: string | undefined;
         } | undefined;
     }[] | undefined;
 }, {
@@ -260,11 +271,12 @@ export declare const workspaceConfigSchema: z.ZodObject<{
             baseUrl?: string | undefined;
         } | undefined;
         runtimeDetails?: {
-            runtime: "local" | "docker" | "k8s";
+            runtime?: unknown;
             dockerNetwork?: string | undefined;
             k8sNamespace?: string | undefined;
             k8sContext?: string | undefined;
             serverStartCommand?: string | undefined;
+            serverStopCommand?: string | undefined;
         } | undefined;
     }[] | undefined;
 }>;
@@ -301,11 +313,12 @@ export declare function validateWorkspaceConfig(config: unknown): z.SafeParseRet
             baseUrl?: string | undefined;
         } | undefined;
         runtimeDetails?: {
-            runtime: "local" | "docker" | "k8s";
+            runtime?: unknown;
             dockerNetwork?: string | undefined;
             k8sNamespace?: string | undefined;
             k8sContext?: string | undefined;
             serverStartCommand?: string | undefined;
+            serverStopCommand?: string | undefined;
         } | undefined;
     }[] | undefined;
 }, {
@@ -334,11 +347,12 @@ export declare function validateWorkspaceConfig(config: unknown): z.SafeParseRet
             baseUrl?: string | undefined;
         } | undefined;
         runtimeDetails?: {
-            runtime: "local" | "docker" | "k8s";
+            runtime: import("../types/TestTypes.js").RuntimeEnvironment;
             dockerNetwork?: string | undefined;
             k8sNamespace?: string | undefined;
             k8sContext?: string | undefined;
             serverStartCommand?: string | undefined;
+            serverStopCommand?: string | undefined;
         } | undefined;
     }[] | undefined;
 }>;

package/build/workspace/workspace.js

@@ -14,6 +14,7 @@ import { execFile } from "child_process";
 import { promisify } from "util";
 import yaml from "js-yaml";
 import { z } from "zod";
+import { runtimeEnvironmentInputSchema } from "../types/TestTypes.js";
 const execFileAsync = promisify(execFile);
 // ---------------------------------------------------------------------------
 // Constants
@@ -57,7 +58,8 @@ export const serviceSchema = z
     runtimeDetails: z
         .object({
         serverStartCommand: z.string().optional(),
-        runtime: z.enum(["local", "docker", "k8s"]),
+        serverStopCommand: z.string().optional(),
+        runtime: runtimeEnvironmentInputSchema,
         dockerNetwork: z.string().optional(),
         k8sNamespace: z.string().optional(),
         k8sContext: z.string().optional(),

package/node_modules/playwright/lib/mcp/browser/tools/screenshot.js

@@ -40,7 +40,7 @@ const screenshot = (0, import_tool.defineTabTool)({
   schema: {
     name: "browser_take_screenshot",
     title: "Take a screenshot",
-    description: `Take a screenshot of the current page. You can't perform actions based on the screenshot, use browser_snapshot for actions.`,
+    description: `Take a diagnostic screenshot of the current page for your own inspection while recording. This is NOT a test assertion: the image is not recorded into the trace and will NOT appear in the generated test (it is dropped at export). You can't perform actions based on the screenshot \u2014 use browser_snapshot for actions. To ASSERT how a page/element/region looks (visual regression \u2014 e.g. an instruction to "take a screenshot" or capture a baseline), use browser_visual_snapshot instead, which records a toHaveScreenshot() assertion.`,
     inputSchema: screenshotSchema,
     type: "readOnly"
   },

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

@@ -0,0 +1,45 @@
+"use strict";
+var __defProp = Object.defineProperty;
+var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
+var __getOwnPropNames = Object.getOwnPropertyNames;
+var __hasOwnProp = Object.prototype.hasOwnProperty;
+var __export = (target, all) => {
+  for (var name in all)
+    __defProp(target, name, { get: all[name], enumerable: true });
+};
+var __copyProps = (to, from, except, desc) => {
+  if (from && typeof from === "object" || typeof from === "function") {
+    for (let key of __getOwnPropNames(from))
+      if (!__hasOwnProp.call(to, key) && key !== except)
+        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
+  }
+  return to;
+};
+var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
+var resultCode_exports = {};
+__export(resultCode_exports, {
+  rewriteResultCodeText: () => rewriteResultCodeText
+});
+module.exports = __toCommonJS(resultCode_exports);
+function rewriteResultCodeText(text, newCode) {
+  if (!text.includes("await page."))
+    return null;
+  const fence = text.match(/```(?:js|javascript)\n/);
+  if (fence && fence.index !== void 0) {
+    const codeStart = fence.index + fence[0].length;
+    const codeEnd = text.indexOf("\n```", codeStart);
+    if (codeEnd >= 0)
+      return text.slice(0, codeStart) + newCode + text.slice(codeEnd);
+  }
+  const lines = text.split("\n");
+  const codeLineIdx = lines.findIndex((l) => l.includes("await page."));
+  if (codeLineIdx >= 0) {
+    lines[codeLineIdx] = newCode;
+    return lines.join("\n");
+  }
+  return null;
+}
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  rewriteResultCodeText
+});

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

@@ -48,6 +48,7 @@ var import_assertTool = require("./assertTool");
 var import_assertApiRequestTool = require("./assertApiRequestTool");
 var import_loadTraceTool = require("./loadTraceTool");
 var import_skyRampImport = require("./skyRampImport");
+var import_resultCode = require("./resultCode");
 var import_mouseActionTool = require("./mouseActionTool");
 var import_visualSnapshotTool = require("./visualSnapshotTool");
 var import_gojsTool = require("./gojsTool");
@@ -1847,28 +1848,11 @@ Cell row ${params.row}, column ${params.column} has "${actual}", expected "${par
   _modifyResultCode(result, newCode) {
     const modified = { ...result, content: [...result.content] };
     for (let i = 0; i < modified.content.length; i++) {
-      if (modified.content[i].type === "text") {
-        const text = modified.content[i].text || "";
-        if (text.includes("```javascript") || text.includes("await page.")) {
-          const codeBlockMatch = text.match(/([\s\S]*```javascript\n)([\s\S]*?)(```[\s\S]*)/);
-          if (codeBlockMatch) {
-            modified.content[i] = {
-              ...modified.content[i],
-              text: codeBlockMatch[1] + newCode + "\n" + codeBlockMatch[3]
-            };
-          } else {
-            const lines = text.split("\n");
-            const codeLineIdx = lines.findIndex((l) => l.includes("await page."));
-            if (codeLineIdx >= 0) {
-              lines[codeLineIdx] = newCode;
-              modified.content[i] = {
-                ...modified.content[i],
-                text: lines.join("\n")
-              };
-            }
-          }
-        }
-      }
+      if (modified.content[i].type !== "text")
+        continue;
+      const rewritten = (0, import_resultCode.rewriteResultCodeText)(modified.content[i].text || "", newCode);
+      if (rewritten !== null)
+        modified.content[i] = { ...modified.content[i], text: rewritten };
     }
     return modified;
   }

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

@@ -0,0 +1,48 @@
+"use strict";
+var import_vitest = require("vitest");
+var import_resultCode = require("../skyramp/resultCode");
+const NEW_CODE = 'await page.locator("#viewer-cols").getByRole("button", { name: "TextBlock" }).click();';
+function clickResult(snapshot, fence = "js") {
+  return "### Ran Playwright code\n\n```" + fence + '\nawait page.getByRole("button", { name: "TextBlock" }).click();\n```\n\n### Page state\n- Page Snapshot:\n```yaml\n' + snapshot + "\n```\n";
+}
+function bigSnapshot(chars) {
+  let s = "";
+  let i = 0;
+  while (s.length < chars) {
+    s += `- text "\u5024 ${i} \u5186 ${i * 7}" [ref=e${i}]
+`;
+    i++;
+  }
+  return s;
+}
+(0, import_vitest.describe)("rewriteResultCodeText", () => {
+  (0, import_vitest.it)("replaces code inside a ```js fence and keeps the snapshot intact", () => {
+    const snapshot = bigSnapshot(2e3);
+    const out = (0, import_resultCode.rewriteResultCodeText)(clickResult(snapshot, "js"), NEW_CODE);
+    (0, import_vitest.expect)(out).toContain(NEW_CODE);
+    (0, import_vitest.expect)(out).not.toContain('await page.getByRole("button", { name: "TextBlock" }).click();');
+    (0, import_vitest.expect)(out).toContain(snapshot);
+    (0, import_vitest.expect)(out).toContain("```yaml");
+  });
+  (0, import_vitest.it)("also handles a ```javascript fence", () => {
+    const out = (0, import_resultCode.rewriteResultCodeText)(clickResult(bigSnapshot(500), "javascript"), NEW_CODE);
+    (0, import_vitest.expect)(out).toContain(NEW_CODE);
+  });
+  (0, import_vitest.it)("falls back to line replacement when there is no fenced code block", () => {
+    const text = '### Result\nawait page.getByRole("button").click();\n\n### Page state\n' + bigSnapshot(500);
+    const out = (0, import_resultCode.rewriteResultCodeText)(text, NEW_CODE);
+    (0, import_vitest.expect)(out).toContain(NEW_CODE);
+    (0, import_vitest.expect)(out).not.toContain('await page.getByRole("button").click();');
+  });
+  (0, import_vitest.it)("returns null when there is no recorded code to rewrite", () => {
+    (0, import_vitest.expect)((0, import_resultCode.rewriteResultCodeText)("### Page state\n- Page Snapshot:\n" + bigSnapshot(500), NEW_CODE)).toBeNull();
+  });
+  (0, import_vitest.it)("is fast on a ~1.2MB result (no catastrophic backtracking)", () => {
+    const text = clickResult(bigSnapshot(12e5), "js");
+    const start = Date.now();
+    const out = (0, import_resultCode.rewriteResultCodeText)(text, NEW_CODE);
+    const elapsedMs = Date.now() - start;
+    (0, import_vitest.expect)(out).toContain(NEW_CODE);
+    (0, import_vitest.expect)(elapsedMs).toBeLessThan(2e3);
+  });
+});

package/node_modules/playwright/node_modules/playwright-core/package.json

@@ -1,6 +1,6 @@
 {
   "name": "playwright-core",
-  "version": "1.58.2-skyramp.8.14.0",
+  "version": "1.58.2-skyramp.8.14.2",
   "description": "A high-level API to automate web browsers",
   "repository": {
     "type": "git",

package/node_modules/playwright/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyramp/playwright",
-  "version": "1.58.2-skyramp.8.14.0",
+  "version": "1.58.2-skyramp.8.14.2",
   "description": "Skyramp's fork of Playwright with trace recording for UI test generation",
   "publishConfig": {
     "access": "public"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyramp/mcp",
-  "version": "0.3.0",
+  "version": "0.3.1-ldw-beta",
   "main": "build/index.js",
   "exports": {
     ".": "./build/index.js",
@@ -71,7 +71,7 @@
     "dockerode": "^5.0.0",
     "fast-glob": "^3.3.3",
     "js-yaml": "^4.1.1",
-    "playwright": "file:vendor/skyramp-playwright-1.58.2-skyramp.8.14.0.tgz",
+    "playwright": "file:vendor/skyramp-playwright-1.58.2-skyramp.8.14.2.tgz",
     "simple-git": "^3.30.0",
     "typescript": "^5.8.3",
     "zod": "^3.25.3"