@skyramp/mcp 0.3.8 → 0.4.0-rc.1

package/build/utils/testFixtures.d.ts

@@ -0,0 +1,5 @@
+import { ScenarioStep } from "../types/RepositoryAnalysis.js";
+/** Build a minimal ScenarioStep. Every field a caller could disagree about is a
+ *  parameter: a default here is a value the test did not choose and cannot be
+ *  read off the call. */
+export declare function mkStep(method: string, path: string, description: string): ScenarioStep;

package/build/utils/testFixtures.js

@@ -0,0 +1,13 @@
+/** Build a minimal ScenarioStep. Every field a caller could disagree about is a
+ *  parameter: a default here is a value the test did not choose and cannot be
+ *  read off the call. */
+export function mkStep(method, path, description) {
+    return {
+        order: 0,
+        method,
+        path,
+        description,
+        interactionType: "success",
+        expectedStatusCode: 200,
+    };
+}

package/build/utils/utils.d.ts

@@ -1,5 +1,4 @@
 import { CallToolResult, ReadResourceResult } from "@modelcontextprotocol/sdk/types.js";
-export declare function readDiffFile(diffFilePath: string | undefined): string | undefined;
 export declare function toolError(message: string): CallToolResult;
 /**
  * Build a successful tool result whose payload is delivered on BOTH MCP result

package/build/utils/utils.js

@@ -1,17 +1,6 @@
 import path from "path";
-import * as fs from "fs";
 import { isTestbotEnabled } from "./featureFlags.js";
 import { logger } from "./logger.js";
-export function readDiffFile(diffFilePath) {
-    if (!diffFilePath)
-        return undefined;
-    try {
-        return fs.readFileSync(diffFilePath, "utf-8");
-    }
-    catch {
-        return undefined;
-    }
-}
 export function toolError(message) {
     return {
         content: [{ type: "text", text: message }],

package/build/utils/versions.d.ts

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

package/build/utils/versions.js

@@ -1,4 +1,4 @@
-export const SKYRAMP_IMAGE_VERSION = "v1.3.40";
+export const SKYRAMP_IMAGE_VERSION = "v1.3.44";
 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`).

package/build/workspace/workspace.d.ts

@@ -55,7 +55,7 @@ export declare const serviceSchema: z.ZodObject<{
     }, "strict", z.ZodTypeAny, {
         authHeader?: string | undefined;
         authScheme?: string | undefined;
-        authType?: "none" | "cookie" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | "token" | undefined;
+        authType?: "none" | "cookie" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | undefined;
         baseUrl?: string | undefined;
         schemaPath?: string | undefined;
         defaultQueryParams?: Record<string, string> | undefined;
@@ -67,7 +67,7 @@ export declare const serviceSchema: z.ZodObject<{
     }, {
         authHeader?: string | undefined;
         authScheme?: string | undefined;
-        authType?: "none" | "cookie" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | "token" | undefined;
+        authType?: "none" | "cookie" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | undefined;
         baseUrl?: string | undefined;
         schemaPath?: string | undefined;
         defaultQueryParams?: Record<string, string> | undefined;
@@ -146,7 +146,7 @@ export declare const serviceSchema: z.ZodObject<{
     api?: {
         authHeader?: string | undefined;
         authScheme?: string | undefined;
-        authType?: "none" | "cookie" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | "token" | undefined;
+        authType?: "none" | "cookie" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | undefined;
         baseUrl?: string | undefined;
         schemaPath?: string | undefined;
         defaultQueryParams?: Record<string, string> | undefined;
@@ -181,7 +181,7 @@ export declare const serviceSchema: z.ZodObject<{
     api?: {
         authHeader?: string | undefined;
         authScheme?: string | undefined;
-        authType?: "none" | "cookie" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | "token" | undefined;
+        authType?: "none" | "cookie" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | undefined;
         baseUrl?: string | undefined;
         schemaPath?: string | undefined;
         defaultQueryParams?: Record<string, string> | undefined;
@@ -292,7 +292,7 @@ export declare const workspaceConfigSchema: z.ZodObject<{
         }, "strict", z.ZodTypeAny, {
             authHeader?: string | undefined;
             authScheme?: string | undefined;
-            authType?: "none" | "cookie" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | "token" | undefined;
+            authType?: "none" | "cookie" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | undefined;
             baseUrl?: string | undefined;
             schemaPath?: string | undefined;
             defaultQueryParams?: Record<string, string> | undefined;
@@ -304,7 +304,7 @@ export declare const workspaceConfigSchema: z.ZodObject<{
         }, {
             authHeader?: string | undefined;
             authScheme?: string | undefined;
-            authType?: "none" | "cookie" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | "token" | undefined;
+            authType?: "none" | "cookie" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | undefined;
             baseUrl?: string | undefined;
             schemaPath?: string | undefined;
             defaultQueryParams?: Record<string, string> | undefined;
@@ -383,7 +383,7 @@ export declare const workspaceConfigSchema: z.ZodObject<{
         api?: {
             authHeader?: string | undefined;
             authScheme?: string | undefined;
-            authType?: "none" | "cookie" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | "token" | undefined;
+            authType?: "none" | "cookie" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | undefined;
             baseUrl?: string | undefined;
             schemaPath?: string | undefined;
             defaultQueryParams?: Record<string, string> | undefined;
@@ -418,7 +418,7 @@ export declare const workspaceConfigSchema: z.ZodObject<{
         api?: {
             authHeader?: string | undefined;
             authScheme?: string | undefined;
-            authType?: "none" | "cookie" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | "token" | undefined;
+            authType?: "none" | "cookie" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | undefined;
             baseUrl?: string | undefined;
             schemaPath?: string | undefined;
             defaultQueryParams?: Record<string, string> | undefined;
@@ -466,7 +466,7 @@ export declare const workspaceConfigSchema: z.ZodObject<{
         api?: {
             authHeader?: string | undefined;
             authScheme?: string | undefined;
-            authType?: "none" | "cookie" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | "token" | undefined;
+            authType?: "none" | "cookie" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | undefined;
             baseUrl?: string | undefined;
             schemaPath?: string | undefined;
             defaultQueryParams?: Record<string, string> | undefined;
@@ -514,7 +514,7 @@ export declare const workspaceConfigSchema: z.ZodObject<{
         api?: {
             authHeader?: string | undefined;
             authScheme?: string | undefined;
-            authType?: "none" | "cookie" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | "token" | undefined;
+            authType?: "none" | "cookie" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | undefined;
             baseUrl?: string | undefined;
             schemaPath?: string | undefined;
             defaultQueryParams?: Record<string, string> | undefined;
@@ -571,7 +571,7 @@ export declare function validateWorkspaceConfig(config: unknown): z.SafeParseRet
         api?: {
             authHeader?: string | undefined;
             authScheme?: string | undefined;
-            authType?: "none" | "cookie" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | "token" | undefined;
+            authType?: "none" | "cookie" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | undefined;
             baseUrl?: string | undefined;
             schemaPath?: string | undefined;
             defaultQueryParams?: Record<string, string> | undefined;
@@ -619,7 +619,7 @@ export declare function validateWorkspaceConfig(config: unknown): z.SafeParseRet
         api?: {
             authHeader?: string | undefined;
             authScheme?: string | undefined;
-            authType?: "none" | "cookie" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | "token" | undefined;
+            authType?: "none" | "cookie" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "session" | undefined;
             baseUrl?: string | undefined;
             schemaPath?: string | undefined;
             defaultQueryParams?: Record<string, string> | undefined;

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

@@ -0,0 +1,56 @@
+"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 assertHiddenTool_exports = {};
+__export(assertHiddenTool_exports, {
+  assertHiddenMcpTool: () => assertHiddenMcpTool,
+  assertHiddenToolSchema: () => assertHiddenToolSchema
+});
+module.exports = __toCommonJS(assertHiddenTool_exports);
+var import_mcpBundle = require("playwright-core/lib/mcpBundle");
+var import_tool = require("../sdk/tool");
+const assertHiddenToolSchema = {
+  name: "browser_assert_hidden",
+  title: "Assert element absent or hidden",
+  description: [
+    "Assert that an element is absent from the page or not visible \u2014 the recorded form of expect(locator).toBeHidden().",
+    "Use it for an element a change removed, after navigating to the page that used to show it.",
+    "Takes exactly one target: testId, or role + name, or text, or attribute + value. There is no snapshot ref (a removed element has none).",
+    "role + name and text match EXACTLY (case-sensitive, whole string): the values come verbatim from the removed source, not from snapshot text.",
+    "The tool refuses to record on a page that is not a loaded http(s) application page (about:blank, an error page, a data: URL),",
+    "where every absence would be vacuously true. Only a passing assertion is recorded in the trace."
+  ].join(" "),
+  inputSchema: import_mcpBundle.z.object({
+    element: import_mcpBundle.z.string().describe("Human-readable description of the element that must be absent or not visible"),
+    testId: import_mcpBundle.z.string().optional().describe("The data-testid of the element that must be absent or not visible. Pass exactly one of testId, role (+ name), text, or attribute (+ value)."),
+    role: import_mcpBundle.z.string().optional().describe('ARIA role of the element that must be absent or not visible (button, link, heading, textbox, \u2026). Requires "name".'),
+    name: import_mcpBundle.z.string().optional().describe('Accessible name that goes with "role"; matched exactly (case-sensitive, whole string).'),
+    text: import_mcpBundle.z.string().optional().describe("Exact visible text of the element that must be absent or not visible (whole string, case-sensitive)."),
+    attribute: import_mcpBundle.z.string().optional().describe("A `data-*` attribute name (e.g. `data-cy`, `data-qa`, `data-test-id`) identifying the removed element; pair with `value`. Use `testId` instead when the attribute is `data-testid`."),
+    value: import_mcpBundle.z.string().optional().describe("The literal value of `attribute`.")
+  }),
+  type: "readOnly"
+};
+function assertHiddenMcpTool() {
+  return (0, import_tool.toMcpTool)(assertHiddenToolSchema);
+}
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  assertHiddenMcpTool,
+  assertHiddenToolSchema
+});

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

@@ -35,7 +35,8 @@ const assertToolSchema = {
     '"checked" checks checkbox/radio state, "disabled" checks disabled/enabled state,',
     '"visible" checks the element is visible,',
     `"count" counts elements sharing the referenced element's role (or exact text)`,
-    "across the page, e.g. ref one option of an open dropdown to assert the number of options."
+    "across the page, e.g. ref one option of an open dropdown to assert the number of options.",
+    "To assert an element is absent or not visible, use `browser_assert_hidden`."
   ].join(" "),
   inputSchema: import_mcpBundle.z.object({
     type: import_mcpBundle.z.enum(["text", "value", "checked", "disabled", "visible", "count"]).describe(`Type of assertion: "text" for text content, "value" for input field value, "checked" for checkbox/radio state, "disabled" for disabled/enabled state ("false" asserts the element is enabled), "visible" for element visibility, "count" for the number of elements with the referenced element's role`),

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

@@ -378,6 +378,16 @@ async function performClientAction(page, actionInContext) {
         throw new Error(`assertDisabled failed: expected disabled=${action.disabled}, got ${disabled}`);
       return;
     }
+    case "assertHidden": {
+      try {
+        await locator.waitFor({ ...opts, state: "hidden" });
+      } catch (e) {
+        if (e?.name === "TimeoutError")
+          throw new Error(`assertHidden failed: element is still visible (${e.message})`);
+        throw e;
+      }
+      return;
+    }
     case "assertCount": {
       await locator.first().waitFor(opts).catch(() => {
       });

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

@@ -63,6 +63,7 @@ const ACTION_NAMES = /* @__PURE__ */ new Set([
   "assertVisible",
   "assertDisabled",
   "assertCount",
+  "assertHidden",
   "assertSnapshot",
   "visualSnapshot",
   "assertTableCell",
@@ -138,7 +139,9 @@ function parseLine(line) {
   }
   if (!entry.name || !isActionName(entry.name))
     return null;
-  const { pageGuid, pageAlias, framePath, locator: _locator, ...actionFields } = entry;
+  const { pageGuid, pageAlias, framePath, locator, ...actionFields } = entry;
+  if (entry.name === "assertHidden" && locator && typeof locator === "object")
+    actionFields.locator = locator;
   const frame = {
     pageGuid: pageGuid ?? "",
     pageAlias: pageAlias ?? "page",

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

@@ -45,6 +45,7 @@ var import_log = require("../log");
 var import_skyRampExport = require("../test/skyRampExport");
 var import_exportTool = require("./exportTool");
 var import_assertTool = require("./assertTool");
+var import_assertHiddenTool = require("./assertHiddenTool");
 var import_assertApiRequestTool = require("./assertApiRequestTool");
 var import_loadTraceTool = require("./loadTraceTool");
 var import_skyRampImport = require("./skyRampImport");
@@ -186,7 +187,7 @@ class TraceRecordingBackend {
   }
   async listTools() {
     const browserTools = await this._browserBackend.listTools();
-    return [...browserTools, (0, import_exportTool.exportZipMcpTool)(), (0, import_assertTool.assertMcpTool)(), (0, import_assertApiRequestTool.assertApiRequestMcpTool)(), (0, import_loadTraceTool.loadTraceMcpTool)(), (0, import_mouseActionTool.mouseActionMcpTool)(), (0, import_tableAssertTool.tableAssertMcpTool)(), (0, import_visualSnapshotTool.visualSnapshotMcpTool)(), (0, import_gojsTool.gojsAddNodeMcpTool)(), (0, import_gojsTool.gojsLinkMcpTool)()];
+    return [...browserTools, (0, import_exportTool.exportZipMcpTool)(), (0, import_assertTool.assertMcpTool)(), (0, import_assertHiddenTool.assertHiddenMcpTool)(), (0, import_assertApiRequestTool.assertApiRequestMcpTool)(), (0, import_loadTraceTool.loadTraceMcpTool)(), (0, import_mouseActionTool.mouseActionMcpTool)(), (0, import_tableAssertTool.tableAssertMcpTool)(), (0, import_visualSnapshotTool.visualSnapshotMcpTool)(), (0, import_gojsTool.gojsAddNodeMcpTool)(), (0, import_gojsTool.gojsLinkMcpTool)()];
   }
   async callTool(name, args, progress) {
     if (!this._initialized)
@@ -238,6 +239,10 @@ class TraceRecordingBackend {
       const parsed = import_assertTool.assertToolSchema.inputSchema.parse(args || {});
       return this._handleAssert(parsed);
     }
+    if (name === import_assertHiddenTool.assertHiddenToolSchema.name) {
+      const parsed = import_assertHiddenTool.assertHiddenToolSchema.inputSchema.parse(args || {});
+      return this._handleAssertHidden(parsed, Date.now());
+    }
     if (name === import_assertApiRequestTool.assertApiRequestSchema.name) {
       this._trackedActions.push({
         toolName: "browser_assert_api_request",
@@ -855,6 +860,14 @@ Continue recording with browser_* tools, then call skyramp_export_zip to write t
         return { toolName: "browser_assert", code: `assertChecked:${a.selector}:${!!a.checked}`, args: { type: "checked", selector: a.selector, checked: !!a.checked } };
       case "assertVisible":
         return { toolName: "browser_assert", code: `assertVisible:${a.selector}`, args: { type: "visible", selector: a.selector } };
+      case "assertHidden": {
+        if (typeof a.selector !== "string" || !a.selector)
+          return null;
+        const args = { type: "hidden", selector: a.selector };
+        if (a.locator && typeof a.locator === "object")
+          args.locator = a.locator;
+        return { toolName: "browser_assert", code: `assertHidden:${a.selector}`, args };
+      }
       case "assertDisabled":
         if (typeof a.disabled !== "boolean")
           return null;
@@ -1365,6 +1378,9 @@ Role "${refRole}" is not countable \u2014 reference an exemplar with a semantic
       if (!tab) {
         return { content: [{ type: "text", text: "### Assertion Failed\nNo active page." }], isError: true };
       }
+      const notReady = await this._requireLoadedPage(tab, "domcontentloaded");
+      if (notReady)
+        return notReady;
       countSelector = `internal:role=${refRole}`;
       let countActual;
       try {
@@ -1520,6 +1536,149 @@ Cell row ${params.row}, column ${params.column} equals "${params.expected}".` :
 Cell row ${params.row}, column ${params.column} has "${actual}", expected "${params.expected}".` }]
     };
   }
+  static {
+    /**
+     * How long a still-present browser_assert_hidden target may take to disappear
+     * before the check reports "still visible". Short on purpose: the agent
+     * asserts after the page has settled, so a longer wait only delays a genuine
+     * failure verdict.
+     */
+    this.HIDDEN_WAIT_TIMEOUT_MS = 2e3;
+  }
+  /**
+   * Page-readiness guard shared by the checks whose live query passes
+   * vacuously on a page that has not loaded the application:
+   * browser_assert_hidden (waitFor({ state: 'hidden' }) resolves instantly for
+   * zero matches) and browser_assert type:"count" (getByRole().count() is 0 on
+   * an empty page).
+   *
+   * What it checks: only the URL scheme. about:blank, chrome-error:// and
+   * data: URLs are refused with an error; an http(s)/file: page is then given
+   * up to 5 s to reach the load event (best-effort — a slow page is not an
+   * error, the caller's own wait/timeout decides).
+   *
+   * What it does NOT catch: a typo'd or stale target on a correctly loaded
+   * page still passes here — this guard cannot tell "removed" from "never
+   * existed". There is deliberately no rendered-element check either: a
+   * text-only page or a canvas/Flutter app has few or no elements under <body>
+   * and is still a loaded page. The defence against a wrong target lives on the
+   * mcp side, which enforces (as a lint) that every absence assertion is paired
+   * with a positive assertion on the same page.
+   *
+   * `loadState` is the readiness bar after the URL check: `hidden` waits for
+   * `load` (an absence verdict wants the page fully settled), `count` only for
+   * `domcontentloaded` — a streaming or long-polling SPA may never fire `load`,
+   * and a live count should not pay up to 5 s of latency for that.
+   *
+   * Returns the isError result to hand back, or null when the page is ready.
+   */
+  async _requireLoadedPage(tab, loadState = "load") {
+    const url = tab.page.url();
+    if (!/^(?:https?|file):/.test(url))
+      return { content: [{ type: "text", text: `### Error
+The current page (${url}) is not a loaded application page; navigate to the app before asserting on it.` }], isError: true };
+    await tab.page.waitForLoadState(loadState, { timeout: 5e3 }).catch(() => {
+    });
+    return null;
+  }
+  /**
+   * browser_assert_hidden (SKYR-4305): the element is absent from the page or
+   * not visible — the recorded form of expect(locator).toBeHidden().
+   * Targets are named (testId | role+name | text | attribute+value) rather than
+   * referenced by a snapshot ref because the typical subject no longer exists
+   * at all (a change removed it), and the browser_assert path resolves refs via
+   * browser_hover, which fails actionability on a hidden element anyway.
+   *
+   * Exactness: role+name and text are both recorded EXACT (`s` suffix /
+   * exact: true) — one rule for this tool. The other producers
+   * (_codeToLocator, _extractLocatorForRef) record `i` because their values come
+   * from snapshot text, which may be trimmed or partially rendered; here the
+   * values come verbatim from the deleted source, so exact is both safe and
+   * needed: a substring "Session" match would keep failing until every
+   * "Session…" occurrence on the page is gone.
+   *
+   * The locator object is built here next to the selector and recorded on the
+   * tracked action (args.locator), so the exporter never has to regex-parse a
+   * JSON-quoted value back out of the selector string.
+   *
+   * Verification is live: locator.waitFor({ state: 'hidden' }) has exactly
+   * toBeHidden's contract — resolves for zero matches or one non-visible match,
+   * times out while the match is visible, and strict-fails on multiple matches
+   * (no `.first()`, so the recorder can never record a pass the generated
+   * expect(locator).toBeHidden() would throw on). Because zero matches passes,
+   * _requireLoadedPage runs first.
+   *
+   * Recorded as toolName 'browser_assert' with args.type 'hidden', so
+   * assertActionToJsonl and the re-seed path share one shape with the other
+   * assertions.
+   */
+  async _handleAssertHidden(params, timestamp) {
+    const err = (text) => ({ content: [{ type: "text", text: `### Error
+${text}` }], isError: true });
+    const given = [params.testId, params.role, params.text, params.attribute].filter((v) => typeof v === "string" && v.trim() !== "");
+    if (given.length !== 1)
+      return err('Pass exactly one target: "testId", or "role" together with "name", or "text", or "attribute" together with "value". There is no "ref" \u2014 a removed element has no snapshot ref.');
+    if (!params.role?.trim() && typeof params.name === "string")
+      return err('"name" only goes with "role" (the ARIA role of the element it names).');
+    if (!params.attribute?.trim() && typeof params.value === "string")
+      return err('"value" only goes with "attribute" (the data-* attribute name it is the value of).');
+    let selector;
+    let locator;
+    if (params.testId?.trim()) {
+      const testId = params.testId.trim();
+      selector = `internal:testid=[data-testid=${JSON.stringify(testId)}s]`;
+      locator = { kind: "test-id", body: testId, options: {} };
+    } else if (params.role?.trim()) {
+      const role = params.role.trim();
+      if (!TraceRecordingBackend.ASSERTABLE_ROLES.has(role))
+        return err(`Role "${role}" is not assertable \u2014 use a semantic role (button, link, heading, textbox, \u2026) or pass "testId"/"text" instead.`);
+      const name = params.name?.trim() ?? "";
+      if (!name)
+        return err('By role, "name" is required \u2014 a bare role would assert that no such element exists anywhere on the page.');
+      selector = `internal:role=${role}[name=${JSON.stringify(name)}s]`;
+      locator = { kind: "role", body: role, options: { attrs: [], exact: true, name } };
+    } else if (params.attribute?.trim()) {
+      const attribute = params.attribute.trim().toLowerCase();
+      if (!/^data-[a-z0-9]+(?:-[a-z0-9]+)*$/.test(attribute))
+        return err(`Attribute "${params.attribute.trim()}" is not a data-* attribute. "attribute" accepts only data-* attributes (data-cy, data-qa, data-test-id, \u2026); target other elements by role + name or text instead.`);
+      const value = params.value?.trim() ?? "";
+      if (!value)
+        return err('By attribute, a non-empty "value" is required \u2014 a bare attribute would assert that no element carries it anywhere on the page.');
+      selector = `[${attribute}=${JSON.stringify(value)}]`;
+      locator = { kind: "default", body: selector, options: {} };
+    } else {
+      const text = params.text.trim();
+      selector = `internal:text=${JSON.stringify(text)}s`;
+      locator = { kind: "text", body: text, options: { exact: true } };
+    }
+    const tab = this._browserBackend.context?.currentTab();
+    if (!tab)
+      return err("No active page.");
+    const notReady = await this._requireLoadedPage(tab);
+    if (notReady)
+      return notReady;
+    let passed = true;
+    try {
+      await tab.page.locator(selector).waitFor({ state: "hidden", timeout: TraceRecordingBackend.HIDDEN_WAIT_TIMEOUT_MS });
+    } catch (e) {
+      if (/strict mode violation/i.test(String(e?.message ?? "")))
+        return err(`Target ${selector} matches multiple elements, so toBeHidden() would fail strict mode. Use a more specific target (a data-testid, or role + exact name). ${String(e.message).split("\n")[0]}`);
+      if (e?.name !== "TimeoutError")
+        return err(`Could not verify absence of "${params.element}": ${e?.message ?? e}`);
+      passed = false;
+    }
+    let details;
+    if (passed) {
+      details = `Hidden assertion passed: "${params.element}" is absent or not visible.`;
+      this._trackedActions.push({ toolName: "browser_assert", args: { type: "hidden", selector, locator }, code: `assertHidden:${selector}`, timestamp });
+      traceDebug(`Assert: assertHidden with selector ${selector}`);
+    } else {
+      const count = await tab.page.locator(selector).count().catch(() => -1);
+      details = `Hidden assertion FAILED: "${params.element}" is still visible (${count < 0 ? "unknown" : count} element(s) match ${selector}).`;
+    }
+    return { content: [{ type: "text", text: `### ${passed ? "Assertion Passed" : "Assertion Failed"}
+${details}` }] };
+  }
   /** Convert a Playwright locator expression to a selector + locator object for JSONL. */
   _codeToLocator(expr) {
     const testidMatch = expr.match(/getByTestId\(\s*['"]([^'"]+)['"]\s*\)/);

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

@@ -444,7 +444,7 @@ function assertActionToJsonl(action, pageGuid, timestamp) {
     }
   } else {
     const parts = code.split(":");
-    if (assertType === "visible") {
+    if (assertType === "visible" || assertType === "hidden") {
       selector = parts.slice(1).join(":");
     } else if (assertType === "text") {
       substring = parts[parts.length - 1] === "true";
@@ -455,12 +455,14 @@ function assertActionToJsonl(action, pageGuid, timestamp) {
       selector = parts.slice(1, parts.length - 1).join(":");
     }
   }
-  const locator = selectorToLocator(selector);
+  const locator = args.locator && typeof args.locator === "object" ? args.locator : selectorToLocator(selector);
   switch (assertType) {
     case "text":
       return JSON.stringify({ name: "assertText", selector, text: expected, substring, locator, ...base });
     case "visible":
       return JSON.stringify({ name: "assertVisible", selector, locator, ...base });
+    case "hidden":
+      return JSON.stringify({ name: "assertHidden", selector, locator, ...base });
     case "value":
       return JSON.stringify({ name: "assertValue", selector, value: expected, locator, ...base });
     case "checked":

package/node_modules/playwright/node_modules/playwright-core/lib/server/codegen/skyramp/jsonlReader.js

@@ -130,6 +130,7 @@ const _ACTION_NAMES = /* @__PURE__ */ new Set([
   "assertVisible",
   "assertDisabled",
   "assertCount",
+  "assertHidden",
   "assertSnapshot",
   "visualSnapshot",
   "assertTableCell",

package/node_modules/playwright/node_modules/playwright-core/lib/server/recorder/recorderSignalProcessor.js

@@ -50,6 +50,8 @@ class RecorderSignalProcessor {
         generateGoto = false;
       else if (signal.isReload)
         generateGoto = !recentReloadUserAction;
+      else if (signal.fromBlankPage)
+        generateGoto = true;
       else if (!recentNavUserAction)
         generateGoto = true;
       if (generateGoto) {

package/node_modules/playwright/node_modules/playwright-core/lib/server/recorder.js

@@ -558,7 +558,8 @@ class Recorder extends import_events.default {
     const isSameDocument = !!event && !event.newDocument;
     this._prevFrameUrls.set(frame, frame.url());
     const url = userInitiatedNavigationUrl(event) ?? frame.url();
-    this._signalProcessor.signal(pageAlias, frame, { name: "navigation", url, isReload, isSameDocument });
+    const fromBlankPage = isBlankUrl(prevUrl) && !isBlankUrl(url);
+    this._signalProcessor.signal(pageAlias, frame, { name: "navigation", url, isReload, isSameDocument, fromBlankPage });
   }
   _onPopup(page, popup) {
     const pageAlias = this._pageAliases.get(page);
@@ -587,6 +588,9 @@ function userInitiatedNavigationUrl(event) {
     request = request.redirectedFrom();
   return request.url();
 }
+function isBlankUrl(url) {
+  return !url || url === "about:blank";
+}
 function languageForFile(file) {
   if (file.endsWith(".py"))
     return "python";

package/node_modules/playwright/node_modules/playwright-core/lib/vite/traceViewer/{index.-Id052Lr.js → index.B7KbSQcC.js}

@@ -1,2 +1,2 @@
-import{T as x,r,a as C,W as F,j as e,D as z,b as I,c as P,d as A,e as O,f as V}from"./assets/defaultSettingsView-Bwr1eMKC.js";const $=()=>{const[o,c]=r.useState(!1),[n,l]=r.useState(),[h,u]=r.useState(),[p,b]=r.useState(N),[f,j]=r.useState({done:0,total:0}),[k,v]=r.useState(!1),[y,m]=r.useState(null),[T,R]=r.useState(null),[U,E]=r.useState(!1),g=r.useCallback(t=>{const s=new URL(window.location.href);if(!t.length)return;const i=t.item(0),a=URL.createObjectURL(i);s.searchParams.append("trace",a);const d=s.toString();window.history.pushState({},"",d),l(a),u(i.name),v(!1),m(null)},[]);r.useEffect(()=>{const t=async s=>{var a;if(!((a=s.clipboardData)!=null&&a.files.length))return;const i=["application/zip","application/x-zip-compressed"];for(const d of s.clipboardData.files)if(!i.includes(d.type))return;s.preventDefault(),g(s.clipboardData.files)};return document.addEventListener("paste",t),()=>document.removeEventListener("paste",t)}),r.useEffect(()=>{const t=s=>{const{method:i,params:a}=s.data;if(i!=="load"||!((a==null?void 0:a.trace)instanceof Blob))return;const d=new File([a.trace],"trace.zip",{type:"application/zip"}),w=new DataTransfer;w.items.add(d),g(w.files)};return window.addEventListener("message",t),()=>window.removeEventListener("message",t)});const W=r.useCallback(t=>{t.preventDefault(),g(t.dataTransfer.files)},[g]),M=r.useCallback(t=>{t.preventDefault(),t.target.files&&g(t.target.files)},[g]);r.useEffect(()=>{const t=new URL(window.location.href).searchParams,s=t.get("trace");if(c(t.has("isServer")),s!=null&&s.startsWith("file:")){R(s||null);return}if(t.has("isServer")){const i=new URLSearchParams(window.location.search).get("ws"),a=new URL(`../${i}`,window.location.toString());a.protocol=window.location.protocol==="https:"?"wss:":"ws:";const d=new C(new F(a));d.onLoadTraceRequested(async w=>{l(w.traceUrl),v(!1),m(null)}),d.initialize({}).catch(()=>{})}else s&&!s.startsWith("blob:")&&l(s)},[]);const S=r.useCallback(async t=>{const s=new URLSearchParams;s.set("trace",t);const i=await fetch(`contexts?${s.toString()}`);if(!i.ok){const{error:w}=await i.json();return m(w),w}const a=await i.json(),d=new x(t,a);j({done:0,total:0}),m(null),b(d)},[]);r.useEffect(()=>{(async()=>{if(!n){b(N);return}const t=s=>{s.data.method==="progress"&&j(s.data.params)};try{navigator.serviceWorker.addEventListener("message",t),j({done:0,total:1});let s=await S(n);s!=null&&s.includes("please grant permission for Local Network Access")&&(await fetch(n,{method:"HEAD",headers:{"x-pw-serviceworker":"skip"}}),s=await S(n)),s&&(o||l(void 0))}finally{navigator.serviceWorker.removeEventListener("message",t)}})()},[o,n,h,S]);const D=f.done!==f.total&&f.total!==0&&!y;r.useEffect(()=>{if(D){const t=setTimeout(()=>{E(!0)},200);return()=>clearTimeout(t)}else E(!1)},[D]);const L=!!(!o&&!k&&!T&&(!n||y));return e.jsxs("div",{className:"vbox workbench-loader",onDragOver:t=>{t.preventDefault(),t.dataTransfer.types.includes("Files")&&v(!0)},children:[e.jsxs("div",{className:"hbox header",...L?{inert:!0}:{},children:[e.jsx("div",{className:"logo",children:e.jsx("img",{src:"playwright-logo.svg",alt:"Playwright logo"})}),e.jsx("div",{className:"product",children:"Playwright"}),p.title&&e.jsx("div",{className:"title",children:p.title}),e.jsx("div",{className:"spacer"}),e.jsx(z,{icon:"settings-gear",title:"Settings",dialogDataTestId:"settings-toolbar-dialog",children:e.jsx(I,{location:"trace-viewer"})})]}),e.jsx(P,{model:p,inert:L}),T&&e.jsxs("div",{className:"drop-target",children:[e.jsx("div",{children:"Trace Viewer uses Service Workers to show traces. To view trace:"}),e.jsxs("div",{style:{paddingTop:20},children:[e.jsxs("div",{children:["1. Click ",e.jsx("a",{href:T,children:"here"})," to put your trace into the download shelf"]}),e.jsxs("div",{children:["2. Go to ",e.jsx("a",{href:"https://trace.playwright.dev",children:"trace.playwright.dev"})]}),e.jsx("div",{children:"3. Drop the trace from the download shelf into the page"})]})]}),e.jsx(A,{open:U,isModal:!0,className:"progress-dialog",children:e.jsxs("div",{className:"progress-content",children:[e.jsx("div",{className:"title",role:"heading","aria-level":1,children:"Loading Playwright Trace..."}),e.jsx("div",{className:"progress-wrapper",children:e.jsx("div",{className:"inner-progress",style:{width:f.total?100*f.done/f.total+"%":0}})})]})}),L&&e.jsxs("div",{className:"drop-target",children:[e.jsx("div",{className:"processing-error",role:"alert",children:y}),e.jsx("div",{className:"title",role:"heading","aria-level":1,children:"Drop Playwright Trace to load"}),e.jsx("div",{children:"or"}),e.jsx("button",{onClick:()=>{const t=document.createElement("input");t.type="file",t.click(),t.addEventListener("change",s=>M(s))},type:"button",children:"Select file"}),e.jsx("div",{className:"info",children:"Playwright Trace Viewer is a Progressive Web App, it does not send your trace anywhere, it opens it locally."}),e.jsxs("div",{className:"version",children:["Playwright v","1.58.2-skyramp.8.14.6"]})]}),o&&!n&&e.jsx("div",{className:"drop-target",children:e.jsx("div",{className:"title",children:"Select test to see the trace"})}),k&&e.jsx("div",{className:"drop-target",onDragLeave:()=>{v(!1)},onDrop:t=>W(t),children:e.jsx("div",{className:"title",children:"Release to analyse the Playwright Trace"})})]})},N=new x("",[]),q=({traceJson:o})=>{const[c,n]=r.useState(void 0),[l,h]=r.useState(0),u=r.useRef(null);return r.useEffect(()=>(u.current&&clearTimeout(u.current),u.current=setTimeout(async()=>{try{const p=await B(o);n(p)}catch{const p=new x("",[]);n(p)}finally{h(l+1)}},500),()=>{u.current&&clearTimeout(u.current)}),[o,l]),e.jsx(P,{isLive:!0,model:c})};async function B(o){const c=new URLSearchParams;c.set("trace",o);const l=await(await fetch(`contexts?${c.toString()}`)).json();return new x(o,l)}(async()=>{const o=new URLSearchParams(window.location.search);if(O(),window.location.protocol!=="file:"){if(o.get("isUnderTest")==="true"&&await new Promise(h=>setTimeout(h,1e3)),!navigator.serviceWorker)throw new Error(`Service workers are not supported.
+import{T as x,r,a as C,W as F,j as e,D as z,b as I,c as P,d as A,e as O,f as V}from"./assets/defaultSettingsView-Bwr1eMKC.js";const $=()=>{const[o,c]=r.useState(!1),[n,l]=r.useState(),[h,u]=r.useState(),[p,b]=r.useState(N),[f,j]=r.useState({done:0,total:0}),[k,v]=r.useState(!1),[y,m]=r.useState(null),[T,R]=r.useState(null),[U,E]=r.useState(!1),g=r.useCallback(t=>{const s=new URL(window.location.href);if(!t.length)return;const i=t.item(0),a=URL.createObjectURL(i);s.searchParams.append("trace",a);const d=s.toString();window.history.pushState({},"",d),l(a),u(i.name),v(!1),m(null)},[]);r.useEffect(()=>{const t=async s=>{var a;if(!((a=s.clipboardData)!=null&&a.files.length))return;const i=["application/zip","application/x-zip-compressed"];for(const d of s.clipboardData.files)if(!i.includes(d.type))return;s.preventDefault(),g(s.clipboardData.files)};return document.addEventListener("paste",t),()=>document.removeEventListener("paste",t)}),r.useEffect(()=>{const t=s=>{const{method:i,params:a}=s.data;if(i!=="load"||!((a==null?void 0:a.trace)instanceof Blob))return;const d=new File([a.trace],"trace.zip",{type:"application/zip"}),w=new DataTransfer;w.items.add(d),g(w.files)};return window.addEventListener("message",t),()=>window.removeEventListener("message",t)});const W=r.useCallback(t=>{t.preventDefault(),g(t.dataTransfer.files)},[g]),M=r.useCallback(t=>{t.preventDefault(),t.target.files&&g(t.target.files)},[g]);r.useEffect(()=>{const t=new URL(window.location.href).searchParams,s=t.get("trace");if(c(t.has("isServer")),s!=null&&s.startsWith("file:")){R(s||null);return}if(t.has("isServer")){const i=new URLSearchParams(window.location.search).get("ws"),a=new URL(`../${i}`,window.location.toString());a.protocol=window.location.protocol==="https:"?"wss:":"ws:";const d=new C(new F(a));d.onLoadTraceRequested(async w=>{l(w.traceUrl),v(!1),m(null)}),d.initialize({}).catch(()=>{})}else s&&!s.startsWith("blob:")&&l(s)},[]);const S=r.useCallback(async t=>{const s=new URLSearchParams;s.set("trace",t);const i=await fetch(`contexts?${s.toString()}`);if(!i.ok){const{error:w}=await i.json();return m(w),w}const a=await i.json(),d=new x(t,a);j({done:0,total:0}),m(null),b(d)},[]);r.useEffect(()=>{(async()=>{if(!n){b(N);return}const t=s=>{s.data.method==="progress"&&j(s.data.params)};try{navigator.serviceWorker.addEventListener("message",t),j({done:0,total:1});let s=await S(n);s!=null&&s.includes("please grant permission for Local Network Access")&&(await fetch(n,{method:"HEAD",headers:{"x-pw-serviceworker":"skip"}}),s=await S(n)),s&&(o||l(void 0))}finally{navigator.serviceWorker.removeEventListener("message",t)}})()},[o,n,h,S]);const D=f.done!==f.total&&f.total!==0&&!y;r.useEffect(()=>{if(D){const t=setTimeout(()=>{E(!0)},200);return()=>clearTimeout(t)}else E(!1)},[D]);const L=!!(!o&&!k&&!T&&(!n||y));return e.jsxs("div",{className:"vbox workbench-loader",onDragOver:t=>{t.preventDefault(),t.dataTransfer.types.includes("Files")&&v(!0)},children:[e.jsxs("div",{className:"hbox header",...L?{inert:!0}:{},children:[e.jsx("div",{className:"logo",children:e.jsx("img",{src:"playwright-logo.svg",alt:"Playwright logo"})}),e.jsx("div",{className:"product",children:"Playwright"}),p.title&&e.jsx("div",{className:"title",children:p.title}),e.jsx("div",{className:"spacer"}),e.jsx(z,{icon:"settings-gear",title:"Settings",dialogDataTestId:"settings-toolbar-dialog",children:e.jsx(I,{location:"trace-viewer"})})]}),e.jsx(P,{model:p,inert:L}),T&&e.jsxs("div",{className:"drop-target",children:[e.jsx("div",{children:"Trace Viewer uses Service Workers to show traces. To view trace:"}),e.jsxs("div",{style:{paddingTop:20},children:[e.jsxs("div",{children:["1. Click ",e.jsx("a",{href:T,children:"here"})," to put your trace into the download shelf"]}),e.jsxs("div",{children:["2. Go to ",e.jsx("a",{href:"https://trace.playwright.dev",children:"trace.playwright.dev"})]}),e.jsx("div",{children:"3. Drop the trace from the download shelf into the page"})]})]}),e.jsx(A,{open:U,isModal:!0,className:"progress-dialog",children:e.jsxs("div",{className:"progress-content",children:[e.jsx("div",{className:"title",role:"heading","aria-level":1,children:"Loading Playwright Trace..."}),e.jsx("div",{className:"progress-wrapper",children:e.jsx("div",{className:"inner-progress",style:{width:f.total?100*f.done/f.total+"%":0}})})]})}),L&&e.jsxs("div",{className:"drop-target",children:[e.jsx("div",{className:"processing-error",role:"alert",children:y}),e.jsx("div",{className:"title",role:"heading","aria-level":1,children:"Drop Playwright Trace to load"}),e.jsx("div",{children:"or"}),e.jsx("button",{onClick:()=>{const t=document.createElement("input");t.type="file",t.click(),t.addEventListener("change",s=>M(s))},type:"button",children:"Select file"}),e.jsx("div",{className:"info",children:"Playwright Trace Viewer is a Progressive Web App, it does not send your trace anywhere, it opens it locally."}),e.jsxs("div",{className:"version",children:["Playwright v","1.58.2-skyramp.8.14.9"]})]}),o&&!n&&e.jsx("div",{className:"drop-target",children:e.jsx("div",{className:"title",children:"Select test to see the trace"})}),k&&e.jsx("div",{className:"drop-target",onDragLeave:()=>{v(!1)},onDrop:t=>W(t),children:e.jsx("div",{className:"title",children:"Release to analyse the Playwright Trace"})})]})},N=new x("",[]),q=({traceJson:o})=>{const[c,n]=r.useState(void 0),[l,h]=r.useState(0),u=r.useRef(null);return r.useEffect(()=>(u.current&&clearTimeout(u.current),u.current=setTimeout(async()=>{try{const p=await B(o);n(p)}catch{const p=new x("",[]);n(p)}finally{h(l+1)}},500),()=>{u.current&&clearTimeout(u.current)}),[o,l]),e.jsx(P,{isLive:!0,model:c})};async function B(o){const c=new URLSearchParams;c.set("trace",o);const l=await(await fetch(`contexts?${c.toString()}`)).json();return new x(o,l)}(async()=>{const o=new URLSearchParams(window.location.search);if(O(),window.location.protocol!=="file:"){if(o.get("isUnderTest")==="true"&&await new Promise(h=>setTimeout(h,1e3)),!navigator.serviceWorker)throw new Error(`Service workers are not supported.
 Make sure to serve the Trace Viewer (${window.location}) via HTTPS or localhost.`);navigator.serviceWorker.register("sw.bundle.js"),navigator.serviceWorker.controller||await new Promise(h=>{navigator.serviceWorker.oncontrollerchange=()=>h()}),setInterval(function(){fetch("ping")},1e4)}const c=o.get("trace"),l=(c==null?void 0:c.endsWith(".json"))?e.jsx(q,{traceJson:c}):e.jsx($,{});V.createRoot(document.querySelector("#root")).render(l)})();

package/node_modules/playwright/node_modules/playwright-core/lib/vite/traceViewer/index.html

@@ -7,7 +7,7 @@
     <link rel="icon" href="./playwright-logo.svg" type="image/svg+xml">
     <link rel="manifest" href="./manifest.webmanifest">
     <title>Playwright Trace Viewer</title>
-    <script type="module" crossorigin src="./index.-Id052Lr.js"></script>
+    <script type="module" crossorigin src="./index.B7KbSQcC.js"></script>
     <link rel="modulepreload" crossorigin href="./assets/defaultSettingsView-Bwr1eMKC.js">
     <link rel="stylesheet" crossorigin href="./defaultSettingsView.7ch9cixO.css">
     <link rel="stylesheet" crossorigin href="./index.BVu7tZDe.css">

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

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

package/node_modules/playwright/node_modules/playwright-core/src/server/codegen/skyramp/jsonlReader.ts

@@ -117,7 +117,7 @@ const _ACTION_NAMES = new Set<string>([
   'check', 'click', 'hover', 'closePage', 'fill',
   'navigate', 'openPage', 'press', 'select', 'uncheck',
   'setInputFiles', 'assertText', 'assertValue', 'assertChecked',
-  'assertVisible', 'assertDisabled', 'assertCount', 'assertSnapshot', 'visualSnapshot', 'assertTableCell',
+  'assertVisible', 'assertDisabled', 'assertCount', 'assertHidden', 'assertSnapshot', 'visualSnapshot', 'assertTableCell',
   'dragTo', 'diagramNodeAdd', 'diagramLinkAdd', 'selectArea', 'tableSnapshot', 'domSnapshot', 'fileChooser',
   'penTool', 'mouse.wheel', 'mouse.move', 'mouse.down', 'mouse.up',
   'modalOpen', 'modalClose', 'waitForTimeout', 'evaluate',

package/node_modules/playwright/node_modules/playwright-core/src/server/recorder/recorderSignalProcessor.ts

@@ -63,6 +63,13 @@ export class RecorderSignalProcessor {
         generateGoto = false;  // SPA pushState/replaceState/popstate: not a real navigation
       else if (signal.isReload)
         generateGoto = !recentReloadUserAction;  // Browser reload button: capture only when no recent user action triggered it
+      // First navigation off the blank start page: never suppress it. A real recording
+      // starts on an empty about:blank, so a click there is a stray focus click and the
+      // navigation came from the address bar. A blank page given content programmatically
+      // could in principle navigate itself, which would make this goto redundant -- but a
+      // redundant goto is a far cheaper failure than a spec with no navigation at all.
+      else if (signal.fromBlankPage)
+        generateGoto = true;
       else if (!recentNavUserAction)
         generateGoto = true;