@cotestdev/mcp_playwright 0.0.14 → 0.0.15

package/lib/mcp/browser/tab.js

@@ -25,9 +25,11 @@ __export(tab_exports, {
 module.exports = __toCommonJS(tab_exports);
 var import_events = require("events");
 var import_utils = require("playwright-core/lib/utils");
-var import_utilsBundle = require("playwright-core/lib/utilsBundle");
 var import_utils2 = require("./tools/utils");
 var import_log = require("../log");
+var import_dialogs = require("./tools/dialogs");
+var import_files = require("./tools/files");
+var import_transform = require("../../transform/transform");
 const TabEvents = {
   modalState: "modalState"
 };
@@ -37,27 +39,23 @@ class Tab extends import_events.EventEmitter {
     this._lastTitle = "about:blank";
     this._consoleMessages = [];
     this._recentConsoleMessages = [];
-    this._requests = /* @__PURE__ */ new Map();
+    this._requests = /* @__PURE__ */ new Set();
     this._modalStates = [];
     this._downloads = [];
+    this._needsFullSnapshot = false;
     this.context = context;
     this.page = page;
     this._onPageClose = onPageClose;
     page.on("console", (event) => this._handleConsoleMessage(messageToConsoleMessage(event)));
     page.on("pageerror", (error) => this._handleConsoleMessage(pageErrorToConsoleMessage(error)));
-    page.on("request", (request) => {
-      this._requests.set(request, null);
-      if (request.frame() === page.mainFrame() && request.isNavigationRequest())
-        this._willNavigateMainFrameToNewDocument();
-    });
-    page.on("response", (response) => this._requests.set(response.request(), response));
+    page.on("request", (request) => this._requests.add(request));
     page.on("close", () => this._onClose());
     page.on("filechooser", (chooser) => {
       this.setModalState({
         type: "fileChooser",
         description: "File chooser",
         fileChooser: chooser,
-        clearedBy: "browser_file_upload"
+        clearedBy: import_files.uploadFile.schema.name
       });
     });
     page.on("dialog", (dialog) => this._dialogShown(dialog));
@@ -67,25 +65,34 @@ class Tab extends import_events.EventEmitter {
     page.setDefaultNavigationTimeout(this.context.config.timeouts.navigation);
     page.setDefaultTimeout(this.context.config.timeouts.action);
     page[tabSymbol] = this;
-    void this._initialize();
+    this._initializedPromise = this._initialize();
   }
   static forPage(page) {
     return page[tabSymbol];
   }
-  async _initialize() {
-    const messages = await this.page.consoleMessages().catch(() => []);
+  static async collectConsoleMessages(page) {
+    const result = [];
+    const messages = await page.consoleMessages().catch(() => []);
     for (const message of messages)
-      this._handleConsoleMessage(messageToConsoleMessage(message));
-    const errors = await this.page.pageErrors().catch(() => []);
+      result.push(messageToConsoleMessage(message));
+    const errors = await page.pageErrors().catch(() => []);
     for (const error of errors)
-      this._handleConsoleMessage(pageErrorToConsoleMessage(error));
+      result.push(pageErrorToConsoleMessage(error));
+    return result;
+  }
+  async _initialize() {
+    for (const message of await Tab.collectConsoleMessages(this.page))
+      this._handleConsoleMessage(message);
     const requests = await this.page.requests().catch(() => []);
-    for (const request of requests) {
-      this._requests.set(request, null);
-      void request.response().catch(() => null).then((response) => {
-        if (response)
-          this._requests.set(request, response);
-      });
+    for (const request of requests)
+      this._requests.add(request);
+    for (const initPage of this.context.config.browser.initPage || []) {
+      try {
+        const { default: func } = await (0, import_transform.requireOrImport)(initPage);
+        await func({ page: this.page });
+      } catch (e) {
+        (0, import_log.logUnhandledError)(e);
+      }
     }
   }
   modalStates() {
@@ -98,22 +105,19 @@ class Tab extends import_events.EventEmitter {
   clearModalState(modalState) {
     this._modalStates = this._modalStates.filter((state) => state !== modalState);
   }
-  modalStatesMarkdown() {
-    return renderModalStates(this.context, this.modalStates());
-  }
   _dialogShown(dialog) {
     this.setModalState({
       type: "dialog",
       description: `"${dialog.type()}" dialog with message "${dialog.message()}"`,
       dialog,
-      clearedBy: "browser_handle_dialog"
+      clearedBy: import_dialogs.handleDialog.schema.name
     });
   }
   async _downloadStarted(download) {
     const entry = {
       download,
       finished: false,
-      outputFile: await this.context.outputFile(download.suggestedFilename(), { origin: "web" })
+      outputFile: await this.context.outputFile(download.suggestedFilename(), { origin: "web", reason: "Saving download" })
     };
     this._downloads.push(entry);
     await download.saveAs(entry.outputFile);
@@ -132,9 +136,6 @@ class Tab extends import_events.EventEmitter {
     this._clearCollectedArtifacts();
     this._onPageClose(this);
   }
-  _willNavigateMainFrameToNewDocument() {
-    this._lastAriaSnapshot = void 0;
-  }
   async updateTitle() {
     await this._raceAgainstModalStates(async () => {
       this._lastTitle = await (0, import_utils2.callOnPageNoTrace)(this.page, (page) => page.title());
@@ -147,23 +148,22 @@ class Tab extends import_events.EventEmitter {
     return this === this.context.currentTab();
   }
   async waitForLoadState(state, options) {
+    await this._initializedPromise;
     await (0, import_utils2.callOnPageNoTrace)(this.page, (page) => page.waitForLoadState(state, options).catch(import_log.logUnhandledError));
   }
   async navigate(url) {
+    await this._initializedPromise;
     this._clearCollectedArtifacts();
-    this._willNavigateMainFrameToNewDocument();
-    const downloadEvent = (0, import_utils2.callOnPageNoTrace)(this.page, (page) => page.waitForEvent("download").catch(import_log.logUnhandledError));
+    const { promise: downloadEvent, abort: abortDownloadEvent } = (0, import_utils2.eventWaiter)(this.page, "download", 3e3);
     try {
       await this.page.goto(url, { waitUntil: "domcontentloaded" });
+      abortDownloadEvent();
     } catch (_e) {
       const e = _e;
       const mightBeDownload = e.message.includes("net::ERR_ABORTED") || e.message.includes("Download is starting");
       if (!mightBeDownload)
         throw e;
-      const download = await Promise.race([
-        downloadEvent,
-        new Promise((resolve) => setTimeout(resolve, 3e3))
-      ]);
+      const download = await downloadEvent;
       if (!download)
         throw e;
       await new Promise((resolve) => setTimeout(resolve, 500));
@@ -171,31 +171,34 @@ class Tab extends import_events.EventEmitter {
     }
     await this.waitForLoadState("load", { timeout: 5e3 });
   }
-  consoleMessages() {
-    return this._consoleMessages;
+  async consoleMessages(level) {
+    await this._initializedPromise;
+    return this._consoleMessages.filter((message) => shouldIncludeMessage(level, message.type));
   }
-  requests() {
+  async requests() {
+    await this._initializedPromise;
     return this._requests;
   }
   async captureSnapshot() {
+    await this._initializedPromise;
     let tabSnapshot;
     const modalStates = await this._raceAgainstModalStates(async () => {
-      const snapshot = await this.page._snapshotForAI();
+      const snapshot = await this.page._snapshotForAI({ track: "response" });
       tabSnapshot = {
         url: this.page.url(),
         title: await this.page.title(),
-        ariaSnapshot: snapshot,
-        formattedAriaSnapshotDiff: this._lastAriaSnapshot ? generateAriaSnapshotDiff(this._lastAriaSnapshot, snapshot) : void 0,
+        ariaSnapshot: snapshot.full,
+        ariaSnapshotDiff: this._needsFullSnapshot ? void 0 : snapshot.incremental,
         modalStates: [],
         consoleMessages: [],
         downloads: this._downloads
       };
     });
     if (tabSnapshot) {
-      tabSnapshot.consoleMessages = this._recentConsoleMessages;
+      tabSnapshot.consoleMessages = this._recentConsoleMessages.filter((message) => shouldIncludeMessage(this.context.config.console.level, message.type));
       this._recentConsoleMessages = [];
-      this._lastAriaSnapshot = tabSnapshot.ariaSnapshot;
     }
+    this._needsFullSnapshot = !tabSnapshot;
     return tabSnapshot ?? {
       url: this.page.url(),
       title: "",
@@ -223,18 +226,24 @@ class Tab extends import_events.EventEmitter {
     ]);
   }
   async waitForCompletion(callback) {
+    await this._initializedPromise;
     await this._raceAgainstModalStates(() => (0, import_utils2.waitForCompletion)(this, callback));
   }
   async refLocator(params) {
+    await this._initializedPromise;
     return (await this.refLocators([params]))[0];
   }
   async refLocators(params) {
-    const snapshot = await this.page._snapshotForAI();
-    return params.map((param) => {
-      if (!snapshot.includes(`[ref=${param.ref}]`))
+    await this._initializedPromise;
+    return Promise.all(params.map(async (param) => {
+      try {
+        const locator = this.page.locator(`aria-ref=${param.ref}`).describe(param.element);
+        const { resolvedSelector } = await locator._resolveSelector();
+        return { locator, resolved: (0, import_utils.asLocator)("javascript", resolvedSelector) };
+      } catch (e) {
         throw new Error(`Ref ${param.ref} not found in the current page snapshot. Try capturing new snapshot.`);
-      return this.page.locator(`aria-ref=${param.ref}`).describe(param.element);
-    });
+      }
+    }));
   }
   async waitForTimeout(time) {
     if (this._javaScriptBlocked()) {
@@ -242,7 +251,8 @@ class Tab extends import_events.EventEmitter {
       return;
     }
     await (0, import_utils2.callOnPageNoTrace)(this.page, (page) => {
-      return page.evaluate(() => new Promise((f) => setTimeout(f, 1e3)));
+      return page.evaluate(() => new Promise((f) => setTimeout(f, 1e3))).catch(() => {
+      });
     });
   }
 }
@@ -256,43 +266,60 @@ function messageToConsoleMessage(message) {
 function pageErrorToConsoleMessage(errorOrValue) {
   if (errorOrValue instanceof Error) {
     return {
-      type: void 0,
+      type: "error",
       text: errorOrValue.message,
       toString: () => errorOrValue.stack || errorOrValue.message
     };
   }
   return {
-    type: void 0,
+    type: "error",
     text: String(errorOrValue),
     toString: () => String(errorOrValue)
   };
 }
-function renderModalStates(context, modalStates) {
-  const result = ["### Modal state"];
+function renderModalStates(modalStates) {
+  const result = [];
   if (modalStates.length === 0)
     result.push("- There is no modal state present");
   for (const state of modalStates)
     result.push(`- [${state.description}]: can be handled by the "${state.clearedBy}" tool`);
   return result;
 }
-const tabSymbol = Symbol("tabSymbol");
-function generateAriaSnapshotDiff(oldSnapshot, newSnapshot) {
-  const diffs = (0, import_utils.diffAriaSnapshots)(import_utilsBundle.yaml, oldSnapshot, newSnapshot);
-  if (diffs === "equal")
-    return "<no changes>";
-  if (diffs === "different")
-    return;
-  if (diffs.length > 3 || diffs.some((diff) => diff.newSource.split("\n").length > 100)) {
-    return;
+const consoleMessageLevels = ["error", "warning", "info", "debug"];
+function shouldIncludeMessage(thresholdLevel, type) {
+  const messageLevel = consoleLevelForMessageType(type);
+  return consoleMessageLevels.indexOf(messageLevel) <= consoleMessageLevels.indexOf(thresholdLevel);
+}
+function consoleLevelForMessageType(type) {
+  switch (type) {
+    case "assert":
+    case "error":
+      return "error";
+    case "warning":
+      return "warning";
+    case "count":
+    case "dir":
+    case "dirxml":
+    case "info":
+    case "log":
+    case "table":
+    case "time":
+    case "timeEnd":
+      return "info";
+    case "clear":
+    case "debug":
+    case "endGroup":
+    case "profile":
+    case "profileEnd":
+    case "startGroup":
+    case "startGroupCollapsed":
+    case "trace":
+      return "debug";
+    default:
+      return "info";
   }
-  const lines = [`The following refs have changed`];
-  for (const diff of diffs)
-    lines.push("", "```yaml", diff.newSource.trimEnd(), "```");
-  const combined = lines.join("\n");
-  if (combined.length >= newSnapshot.length)
-    return;
-  return combined;
 }
+const tabSymbol = Symbol("tabSymbol");
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
   Tab,

package/lib/mcp/browser/tools/common.js

@@ -22,10 +22,10 @@ __export(common_exports, {
   default: () => common_default
 });
 module.exports = __toCommonJS(common_exports);
-var import_bundle = require("../../sdk/bundle");
+var import_mcpBundle = require("playwright-core/lib/mcpBundle");
 var import_tool = require("./tool");
-const baseSchema = import_bundle.z.object({
-  toolText: import_bundle.z.string().describe("A human-readable description of the action to perform by the tool.")
+const baseSchema = import_mcpBundle.z.object({
+  toolText: import_mcpBundle.z.string().describe("A human-readable description of the action to perform by the tool.")
 });
 const close = (0, import_tool.defineTool)({
   capability: "core",
@@ -33,8 +33,8 @@ const close = (0, import_tool.defineTool)({
     name: "browser_close",
     title: "Close browser",
     description: "Close the page",
-    inputSchema: baseSchema.extend({}),
-    type: "readOnly"
+    inputSchema: import_mcpBundle.z.object({}),
+    type: "action"
   },
   handle: async (context, params, response) => {
     await context.closeBrowserContext();
@@ -49,10 +49,10 @@ const resize = (0, import_tool.defineTabTool)({
     title: "Resize browser window",
     description: "Resize the browser window",
     inputSchema: baseSchema.extend({
-      width: import_bundle.z.number().describe("Width of the browser window"),
-      height: import_bundle.z.number().describe("Height of the browser window")
+      width: import_mcpBundle.z.number().describe("Width of the browser window"),
+      height: import_mcpBundle.z.number().describe("Height of the browser window")
     }),
-    type: "readOnly"
+    type: "action"
   },
   handle: async (tab, params, response) => {
     response.addCode(`await page.setViewportSize({ width: ${params.width}, height: ${params.height} });`);

package/lib/mcp/browser/tools/console.js

@@ -21,7 +21,7 @@ __export(console_exports, {
   default: () => console_default
 });
 module.exports = __toCommonJS(console_exports);
-var import_bundle = require("../../sdk/bundle");
+var import_mcpBundle = require("playwright-core/lib/mcpBundle");
 var import_tool = require("./tool");
 const console = (0, import_tool.defineTabTool)({
   capability: "core",
@@ -29,11 +29,14 @@ const console = (0, import_tool.defineTabTool)({
     name: "browser_console_messages",
     title: "Get console messages",
     description: "Returns all console messages",
-    inputSchema: import_bundle.z.object({}),
+    inputSchema: import_mcpBundle.z.object({
+      level: import_mcpBundle.z.enum(["error", "warning", "info", "debug"]).default("info").describe('Level of the console messages to return. Each level includes the messages of more severe levels. Defaults to "info".')
+    }),
     type: "readOnly"
   },
   handle: async (tab, params, response) => {
-    tab.consoleMessages().map((message) => response.addResult(message.toString()));
+    const messages = await tab.consoleMessages(params.level);
+    messages.map((message) => response.addResult(message.toString()));
   }
 });
 var console_default = [

package/lib/mcp/browser/tools/dialogs.js

@@ -18,12 +18,13 @@ var __copyProps = (to, from, except, desc) => {
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 var dialogs_exports = {};
 __export(dialogs_exports, {
-  default: () => dialogs_default
+  default: () => dialogs_default,
+  handleDialog: () => handleDialog
 });
 module.exports = __toCommonJS(dialogs_exports);
-var import_bundle = require("../../sdk/bundle");
-var import_common = require("./common");
+var import_mcpBundle = require("playwright-core/lib/mcpBundle");
 var import_tool = require("./tool");
+var import_common = require("./common");
 const handleDialog = (0, import_tool.defineTabTool)({
   capability: "core",
   schema: {
@@ -31,10 +32,10 @@ const handleDialog = (0, import_tool.defineTabTool)({
     title: "Handle a dialog",
     description: "Handle a dialog",
     inputSchema: import_common.baseSchema.extend({
-      accept: import_bundle.z.boolean().describe("Whether to accept the dialog."),
-      promptText: import_bundle.z.string().optional().describe("The text of the prompt in case of a prompt dialog.")
+      accept: import_mcpBundle.z.boolean().describe("Whether to accept the dialog."),
+      promptText: import_mcpBundle.z.string().optional().describe("The text of the prompt in case of a prompt dialog.")
     }),
-    type: "destructive"
+    type: "action"
   },
   handle: async (tab, params, response) => {
     response.setIncludeSnapshot();
@@ -43,15 +44,10 @@ const handleDialog = (0, import_tool.defineTabTool)({
       throw new Error("No dialog visible");
     tab.clearModalState(dialogState);
     await tab.waitForCompletion(async () => {
-      if (params.accept) {
-        if (params.promptText)
-          response.addCode(`await page.on('dialog', dialog => dialog.accept(${JSON.stringify(params.promptText)}));`);
-        else
-          response.addCode(`await page.on('dialog', dialog => dialog.accept());`);
+      if (params.accept)
         await dialogState.dialog.accept(params.promptText);
-      } else {
+      else
         await dialogState.dialog.dismiss();
-      }
     });
   },
   clearsModalState: "dialog"
@@ -59,3 +55,7 @@ const handleDialog = (0, import_tool.defineTabTool)({
 var dialogs_default = [
   handleDialog
 ];
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  handleDialog
+});

package/lib/mcp/browser/tools/evaluate.js

@@ -1,9 +1,7 @@
 "use strict";
-var __create = Object.create;
 var __defProp = Object.defineProperty;
 var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
 var __getOwnPropNames = Object.getOwnPropertyNames;
-var __getProtoOf = Object.getPrototypeOf;
 var __hasOwnProp = Object.prototype.hasOwnProperty;
 var __export = (target, all) => {
   for (var name in all)
@@ -17,29 +15,20 @@ var __copyProps = (to, from, except, desc) => {
   }
   return to;
 };
-var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
-  // If the importer is in node compatibility mode or this is not an ESM
-  // file that has been converted to a CommonJS file using a Babel-
-  // compatible transform (i.e. "__esModule" has not been set), then set
-  // "default" to the CommonJS "module.exports" for node compatibility.
-  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
-  mod
-));
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 var evaluate_exports = {};
 __export(evaluate_exports, {
   default: () => evaluate_default
 });
 module.exports = __toCommonJS(evaluate_exports);
-var import_bundle = require("../../sdk/bundle");
+var import_mcpBundle = require("playwright-core/lib/mcpBundle");
+var import_utils = require("playwright-core/lib/utils");
 var import_tool = require("./tool");
-var javascript = __toESM(require("../codegen"));
-var import_utils = require("./utils");
 var import_common = require("./common");
 const evaluateSchema = import_common.baseSchema.extend({
-  function: import_bundle.z.string().describe("() => { /* code */ } or (element) => { /* code */ } when element is provided"),
-  element: import_bundle.z.string().optional().describe("Human-readable element description used to obtain permission to interact with the element"),
-  ref: import_bundle.z.string().optional().describe("Exact target element reference from the page snapshot")
+  function: import_mcpBundle.z.string().describe("() => { /* code */ } or (element) => { /* code */ } when element is provided"),
+  element: import_mcpBundle.z.string().optional().describe("Human-readable element description used to obtain permission to interact with the element"),
+  ref: import_mcpBundle.z.string().optional().describe("Exact target element reference from the page snapshot")
 });
 const evaluate = (0, import_tool.defineTabTool)({
   capability: "core",
@@ -48,19 +37,19 @@ const evaluate = (0, import_tool.defineTabTool)({
     title: "Evaluate JavaScript",
     description: "Evaluate JavaScript expression on page or element",
     inputSchema: evaluateSchema,
-    type: "destructive"
+    type: "action"
   },
   handle: async (tab, params, response) => {
     response.setIncludeSnapshot();
     let locator;
     if (params.ref && params.element) {
       locator = await tab.refLocator({ ref: params.ref, element: params.element });
-      response.addCode(`await page.${await (0, import_utils.generateLocator)(locator)}.evaluate(${javascript.quote(params.function)});`);
+      response.addCode(`await page.${locator.resolved}.evaluate(${(0, import_utils.escapeWithQuotes)(params.function)});`);
     } else {
-      response.addCode(`await page.evaluate(${javascript.quote(params.function)});`);
+      response.addCode(`await page.evaluate(${(0, import_utils.escapeWithQuotes)(params.function)});`);
     }
     await tab.waitForCompletion(async () => {
-      const receiver = locator ?? tab.page;
+      const receiver = locator?.locator ?? tab.page;
       const result = await receiver._evaluateFunction(params.function);
       response.addResult(JSON.stringify(result, null, 2) || "undefined");
     });

package/lib/mcp/browser/tools/files.js

@@ -18,12 +18,13 @@ var __copyProps = (to, from, except, desc) => {
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 var files_exports = {};
 __export(files_exports, {
-  default: () => files_default
+  default: () => files_default,
+  uploadFile: () => uploadFile
 });
 module.exports = __toCommonJS(files_exports);
-var import_bundle = require("../../sdk/bundle");
-var import_common = require("./common");
+var import_mcpBundle = require("playwright-core/lib/mcpBundle");
 var import_tool = require("./tool");
+var import_common = require("./common");
 const uploadFile = (0, import_tool.defineTabTool)({
   capability: "core",
   schema: {
@@ -31,9 +32,9 @@ const uploadFile = (0, import_tool.defineTabTool)({
     title: "Upload files",
     description: "Upload one or multiple files",
     inputSchema: import_common.baseSchema.extend({
-      paths: import_bundle.z.array(import_bundle.z.string()).optional().describe("The absolute paths to the files to upload. Can be single file or multiple files. If omitted, file chooser is cancelled.")
+      paths: import_mcpBundle.z.array(import_mcpBundle.z.string()).optional().describe("The absolute paths to the files to upload. Can be single file or multiple files. If omitted, file chooser is cancelled.")
     }),
-    type: "destructive"
+    type: "action"
   },
   handle: async (tab, params, response) => {
     response.setIncludeSnapshot();
@@ -52,3 +53,7 @@ const uploadFile = (0, import_tool.defineTabTool)({
 var files_default = [
   uploadFile
 ];
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  uploadFile
+});

package/lib/mcp/browser/tools/form.js

@@ -1,9 +1,7 @@
 "use strict";
-var __create = Object.create;
 var __defProp = Object.defineProperty;
 var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
 var __getOwnPropNames = Object.getOwnPropertyNames;
-var __getProtoOf = Object.getPrototypeOf;
 var __hasOwnProp = Object.prototype.hasOwnProperty;
 var __export = (target, all) => {
   for (var name in all)
@@ -17,24 +15,15 @@ var __copyProps = (to, from, except, desc) => {
   }
   return to;
 };
-var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
-  // If the importer is in node compatibility mode or this is not an ESM
-  // file that has been converted to a CommonJS file using a Babel-
-  // compatible transform (i.e. "__esModule" has not been set), then set
-  // "default" to the CommonJS "module.exports" for node compatibility.
-  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
-  mod
-));
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 var form_exports = {};
 __export(form_exports, {
   default: () => form_default
 });
 module.exports = __toCommonJS(form_exports);
-var import_bundle = require("../../sdk/bundle");
+var import_mcpBundle = require("playwright-core/lib/mcpBundle");
+var import_utils = require("playwright-core/lib/utils");
 var import_tool = require("./tool");
-var import_utils = require("./utils");
-var codegen = __toESM(require("../codegen"));
 var import_common = require("./common");
 const fillForm = (0, import_tool.defineTabTool)({
   capability: "core",
@@ -43,19 +32,19 @@ const fillForm = (0, import_tool.defineTabTool)({
     title: "Fill form",
     description: "Fill multiple form fields",
     inputSchema: import_common.baseSchema.extend({
-      fields: import_bundle.z.array(import_bundle.z.object({
-        name: import_bundle.z.string().describe("Human-readable field name"),
-        type: import_bundle.z.enum(["textbox", "checkbox", "radio", "combobox", "slider"]).describe("Type of the field"),
-        ref: import_bundle.z.string().describe("Exact target field reference from the page snapshot"),
-        value: import_bundle.z.string().describe("Value to fill in the field. If the field is a checkbox, the value should be `true` or `false`. If the field is a combobox, the value should be the text of the option.")
+      fields: import_mcpBundle.z.array(import_mcpBundle.z.object({
+        name: import_mcpBundle.z.string().describe("Human-readable field name"),
+        type: import_mcpBundle.z.enum(["textbox", "checkbox", "radio", "combobox", "slider"]).describe("Type of the field"),
+        ref: import_mcpBundle.z.string().describe("Exact target field reference from the page snapshot"),
+        value: import_mcpBundle.z.string().describe("Value to fill in the field. If the field is a checkbox, the value should be `true` or `false`. If the field is a combobox, the value should be the text of the option.")
       })).describe("Fields to fill in")
     }),
-    type: "destructive"
+    type: "input"
   },
   handle: async (tab, params, response) => {
     for (const field of params.fields) {
-      const locator = await tab.refLocator({ element: field.name, ref: field.ref });
-      const locatorSource = `await page.${await (0, import_utils.generateLocator)(locator)}`;
+      const { locator, resolved } = await tab.refLocator({ element: field.name, ref: field.ref });
+      const locatorSource = `await page.${resolved}`;
       if (field.type === "textbox" || field.type === "slider") {
         const secret = tab.context.lookupSecret(field.value);
         await locator.fill(secret.value);
@@ -65,7 +54,7 @@ const fillForm = (0, import_tool.defineTabTool)({
         response.addCode(`${locatorSource}.setChecked(${field.value});`);
       } else if (field.type === "combobox") {
         await locator.selectOption({ label: field.value });
-        response.addCode(`${locatorSource}.selectOption(${codegen.quote(field.value)});`);
+        response.addCode(`${locatorSource}.selectOption(${(0, import_utils.escapeWithQuotes)(field.value)});`);
       }
     }
   }

package/lib/mcp/browser/tools/install.js

@@ -33,7 +33,7 @@ __export(install_exports, {
 module.exports = __toCommonJS(install_exports);
 var import_child_process = require("child_process");
 var import_path = __toESM(require("path"));
-var import_bundle = require("../../sdk/bundle");
+var import_mcpBundle = require("playwright-core/lib/mcpBundle");
 var import_tool = require("./tool");
 const install = (0, import_tool.defineTool)({
   capability: "core-install",
@@ -41,8 +41,8 @@ const install = (0, import_tool.defineTool)({
     name: "browser_install",
     title: "Install the browser specified in the config",
     description: "Install the browser specified in the config. Call this if you get an error about the browser not being installed.",
-    inputSchema: import_bundle.z.object({}),
-    type: "destructive"
+    inputSchema: import_mcpBundle.z.object({}),
+    type: "action"
   },
   handle: async (context, params, response) => {
     const channel = context.config.browser?.launchOptions?.channel ?? context.config.browser?.browserName ?? "chrome";