@skyramp/mcp 0.3.2-rc.pom-4 → 0.3.3

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

@@ -114,7 +114,9 @@ class TraceRecordingBackend {
       `--load-extension=${loadExtension.join(",")}`
     ] : void 0;
     const headless = loadExtension?.length ? false : this._options.headless ?? (!!process.env.CI || import_os.default.platform() === "linux" && !process.env.DISPLAY);
+    const testIdAttribute = process.env.PLAYWRIGHT_MCP_TEST_ID_ATTRIBUTE?.trim() || this._options.testIdAttribute?.trim();
     const config = await (0, import_config.resolveConfig)({
+      ...testIdAttribute ? { testIdAttribute } : {},
       browser: {
         isolated: !persistent,
         ...persistent && userDataDir ? { userDataDir } : {},
@@ -562,8 +564,11 @@ Baseline "${input.filename}" (${params.snapshotType})${targetDesc} recorded; the
       return { content: [{ type: "text", text: `### Error
 Pass at most ONE of stopAtStep / stopAtUrl / stopBefore (got ${stopParams.join(", ")}). Omit all three to replay the whole trace.` }], isError: true };
     let loadedActions;
+    let sourceDiagnostics;
     try {
-      loadedActions = await (0, import_skyRampImport.actionsFromPath)(params.path);
+      const source = await (0, import_skyRampImport.traceSourceFromPath)(params.path);
+      loadedActions = source.actions;
+      sourceDiagnostics = source.diagnostics;
     } catch (e) {
       return { content: [{ type: "text", text: `### Error
 Failed to load trace from ${params.path}: ${e.message}` }], isError: true };
@@ -571,6 +576,10 @@ 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 };
+    const diagnosticsNote = sourceDiagnostics.length ? `
+
+Note: loaded from a generated test file \u2014 the replayable interaction skeleton was extracted. ${sourceDiagnostics.length} statement(s) were not converted to steps:
+` + sourceDiagnostics.map((d) => `- line ${d.line}: ${d.text} (${d.reason})`).join("\n") : "";
     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 };
@@ -583,7 +592,7 @@ startAtStep (${params.startAtStep}) is after stopAtStep (${params.stopAtStep}).`
 ${loadedActions.length} steps in ${params.path}. Nothing was replayed.
 
 Steps:
-${stepList}
+${stepList}${diagnosticsNote}
 
 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.`
         }]
@@ -680,7 +689,7 @@ You can continue recording from this point or fix the issue.` : "";
       content: [{
         type: "text",
         text: `### Trace replayed
-${replayedCount} of ${loadedActions.length} steps replayed from ${params.path}.${skippedNote} ${stoppedNote}${errorNote}
+${replayedCount} of ${loadedActions.length} steps replayed from ${params.path}.${skippedNote} ${stoppedNote}${errorNote}${diagnosticsNote}
 
 Remaining steps (NOT replayed):
 ${remaining}
@@ -1650,6 +1659,8 @@ Cell row ${params.row}, column ${params.column} has "${actual}", expected "${par
       if (recId && mcpId && recId !== mcpId)
         continue;
       ri++;
+      if (rec.action?.name === "fill" && rec.action.isPassword)
+        ta.isPassword = true;
       const rebuilt = this._rebuildCodeFromRecorder(rec, ta.code);
       if (rebuilt) {
         traceDebug(`[consume] ${ta.toolName} -> ${rebuilt}`);

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

@@ -24,9 +24,10 @@ function bigSnapshot(chars) {
     (0, import_vitest.expect)(out).toContain(snapshot);
     (0, import_vitest.expect)(out).toContain("```yaml");
   });
-  (0, import_vitest.it)("also handles a ```javascript fence", () => {
+  (0, import_vitest.it)("falls back to line replacement for a non-```js fence (only ```js is spliced)", () => {
     const out = (0, import_resultCode.rewriteResultCodeText)(clickResult(bigSnapshot(500), "javascript"), 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.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);

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

@@ -325,7 +325,7 @@ function trackedActionToJsonl(action, pageGuid, timestamp) {
   if (toolName === "browser_type" || toolName === "browser_press_sequentially") {
     const text = args.text ?? extractFillText(code);
     const name = toolName === "browser_press_sequentially" || args.slowly ? "pressSequentially" : "fill";
-    return JSON.stringify({ name, selector, text, locator: locatorObj, ...base });
+    return JSON.stringify({ name, selector, text, isPassword: action.isPassword || void 0, locator: locatorObj, ...base });
   }
   if (toolName === "browser_fill_form") {
     return null;

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

@@ -42,3 +42,33 @@ function jsonlFor(action) {
     (0, import_vitest.expect)(entry.state).toBe("hidden");
   });
 });
+(0, import_vitest.describe)("browser_type \u2192 isPassword flag", () => {
+  function typeAction(extra = {}) {
+    return {
+      toolName: "browser_type",
+      args: { text: "s3cret" },
+      code: `await page.locator('#login-password').fill('s3cret');`,
+      timestamp: 1e3,
+      ...extra
+    };
+  }
+  (0, import_vitest.it)("marks a fill correlated to a password input", () => {
+    const entry = jsonlFor(typeAction({ isPassword: true }));
+    (0, import_vitest.expect)(entry.name).toBe("fill");
+    (0, import_vitest.expect)(entry.isPassword).toBe(true);
+  });
+  (0, import_vitest.it)("omits the key entirely when not a password", () => {
+    const line = (0, import_skyRampExport.trackedActionToJsonl)(typeAction(), "guid-1", 1e3);
+    (0, import_vitest.expect)(JSON.parse(line).isPassword).toBeUndefined();
+    (0, import_vitest.expect)(line).not.toContain("isPassword");
+  });
+  (0, import_vitest.it)("omits the key when correlation reported false", () => {
+    const line = (0, import_skyRampExport.trackedActionToJsonl)(typeAction({ isPassword: false }), "guid-1", 1e3);
+    (0, import_vitest.expect)(line).not.toContain("isPassword");
+  });
+  (0, import_vitest.it)("carries the flag through pressSequentially too", () => {
+    const entry = jsonlFor(typeAction({ args: { text: "s3cret", slowly: true }, isPassword: true }));
+    (0, import_vitest.expect)(entry.name).toBe("pressSequentially");
+    (0, import_vitest.expect)(entry.isPassword).toBe(true);
+  });
+});