@cotestdev/mcp_playwright 0.0.52 → 0.0.53

package/lib/mcp/browser/browserContextFactory.js

@@ -304,9 +304,7 @@ class SharedContextFactory {
   }
 }
 async function computeTracesDir(config, clientInfo) {
-  if (!config.saveTrace && !config.capabilities?.includes("devtools"))
-    return;
-  return await (0, import_config.outputFile)(config, clientInfo, `traces`, { origin: "code" });
+  return import_path.default.resolve((0, import_config.outputDir)(config, clientInfo), "traces");
 }
 async function browserContextOptionsFromConfig(config, clientInfo) {
   const result = { ...config.browser.contextOptions };

package/lib/mcp/program.js

@@ -77,8 +77,9 @@ Please run the command below. It will install a local copy of ffmpeg and will no
 Daemon listening on ${socketPath}`);
         console.log("<EOF>");
       } catch (error) {
+        const message = process.env.PWDEBUGIMPL ? error.stack || error.message : error.message;
         console.log(`### Error
-${error.message}`);
+${message}`);
         console.log("<EOF>");
       }
       return;

package/lib/mcp/terminal/command.js

@@ -56,11 +56,11 @@ function zodParse(schema, data, type) {
         const label = type === "option" ? `'--${prop}' option` : `'${prop}' argument`;
         switch (issue.code) {
           case "invalid_type":
-            return "error: " + label + ": " + issue.message.toLowerCase().replace(/invalid input:/, "").trim();
+            return "error: " + label + ": " + issue.message.replace(/Invalid input:/, "").trim();
           case "unrecognized_keys":
             return "error: unknown " + label;
           default:
-            return "error: " + label + ": " + issue.message.toLowerCase();
+            return "error: " + label + ": " + issue.message;
         }
       });
     }).flat().join("\n"));

package/lib/mcp/terminal/commands.js

@@ -23,6 +23,17 @@ __export(commands_exports, {
 module.exports = __toCommonJS(commands_exports);
 var import_mcpBundle = require("../../mcpBundle");
 var import_command = require("./command");
+const numberArg = import_mcpBundle.z.preprocess((val, ctx) => {
+  const number = Number(val);
+  if (Number.isNaN(number)) {
+    ctx.issues.push({
+      code: "custom",
+      message: `expected number, received '${val}'`,
+      input: val
+    });
+  }
+  return number;
+}, import_mcpBundle.z.number());
 const open = (0, import_command.declareCommand)({
   name: "open",
   description: "Open the browser",
@@ -131,8 +142,8 @@ const mouseMove = (0, import_command.declareCommand)({
   description: "Move mouse to a given position",
   category: "mouse",
   args: import_mcpBundle.z.object({
-    x: import_mcpBundle.z.number().describe("X coordinate"),
-    y: import_mcpBundle.z.number().describe("Y coordinate")
+    x: numberArg.describe("X coordinate"),
+    y: numberArg.describe("Y coordinate")
   }),
   toolName: "browser_mouse_move_xy",
   toolParams: ({ x, y }) => ({ x, y })
@@ -162,8 +173,8 @@ const mouseWheel = (0, import_command.declareCommand)({
   description: "Scroll mouse wheel",
   category: "mouse",
   args: import_mcpBundle.z.object({
-    dx: import_mcpBundle.z.number().describe("Y delta"),
-    dy: import_mcpBundle.z.number().describe("X delta")
+    dx: numberArg.describe("Y delta"),
+    dy: numberArg.describe("X delta")
   }),
   toolName: "browser_mouse_wheel",
   toolParams: ({ dx: deltaY, dy: deltaX }) => ({ deltaY, deltaX })
@@ -317,8 +328,8 @@ const resize = (0, import_command.declareCommand)({
   description: "Resize the browser window",
   category: "core",
   args: import_mcpBundle.z.object({
-    w: import_mcpBundle.z.number().describe("Width of the browser window"),
-    h: import_mcpBundle.z.number().describe("Height of the browser window")
+    w: numberArg.describe("Width of the browser window"),
+    h: numberArg.describe("Height of the browser window")
   }),
   toolName: "browser_resize",
   toolParams: ({ w: width, h: height }) => ({ width, height })
@@ -356,7 +367,7 @@ const tabClose = (0, import_command.declareCommand)({
   description: "Close a browser tab",
   category: "tabs",
   args: import_mcpBundle.z.object({
-    index: import_mcpBundle.z.number().optional().describe("Tab index. If omitted, current tab is closed.")
+    index: numberArg.optional().describe("Tab index. If omitted, current tab is closed.")
   }),
   toolName: "browser_tabs",
   toolParams: ({ index }) => ({ action: "close", index })
@@ -366,7 +377,7 @@ const tabSelect = (0, import_command.declareCommand)({
   description: "Select a browser tab",
   category: "tabs",
   args: import_mcpBundle.z.object({
-    index: import_mcpBundle.z.number().describe("Tab index")
+    index: numberArg.describe("Tab index")
   }),
   toolName: "browser_tabs",
   toolParams: ({ index }) => ({ action: "select", index })
@@ -424,7 +435,7 @@ const cookieSet = (0, import_command.declareCommand)({
   options: import_mcpBundle.z.object({
     domain: import_mcpBundle.z.string().optional().describe("Cookie domain"),
     path: import_mcpBundle.z.string().optional().describe("Cookie path"),
-    expires: import_mcpBundle.z.number().optional().describe("Cookie expiration as Unix timestamp"),
+    expires: numberArg.optional().describe("Cookie expiration as Unix timestamp"),
     httpOnly: import_mcpBundle.z.boolean().optional().describe("Whether the cookie is HTTP only"),
     secure: import_mcpBundle.z.boolean().optional().describe("Whether the cookie is secure"),
     sameSite: import_mcpBundle.z.enum(["Strict", "Lax", "None"]).optional().describe("Cookie SameSite attribute")
@@ -552,7 +563,7 @@ const routeMock = (0, import_command.declareCommand)({
     pattern: import_mcpBundle.z.string().describe('URL pattern to match (e.g., "**/api/users")')
   }),
   options: import_mcpBundle.z.object({
-    status: import_mcpBundle.z.number().optional().describe("HTTP status code (default: 200)"),
+    status: numberArg.optional().describe("HTTP status code (default: 200)"),
     body: import_mcpBundle.z.string().optional().describe("Response body (text or JSON string)"),
     ["content-type"]: import_mcpBundle.z.string().optional().describe("Content-Type header"),
     header: import_mcpBundle.z.union([import_mcpBundle.z.string(), import_mcpBundle.z.array(import_mcpBundle.z.string())]).optional().transform((v) => v ? Array.isArray(v) ? v : [v] : void 0).describe('Header to add in "Name: Value" format (repeatable)'),

package/lib/mcp/terminal/daemon.js

@@ -118,7 +118,8 @@ async function startMcpDaemonServer(config, contextFactory) {
         }
       } catch (e) {
         daemonDebug("command failed", e);
-        await connection.send({ id, error: e.message });
+        const error = process.env.PWDEBUGIMPL ? e.stack || e.message : e.message;
+        await connection.send({ id, error });
       }
     };
   });

package/lib/mcp/terminal/helpGenerator.js

@@ -23,6 +23,7 @@ __export(helpGenerator_exports, {
   generateReadme: () => generateReadme
 });
 module.exports = __toCommonJS(helpGenerator_exports);
+var import_mcpBundle = require("../../mcpBundle");
 var import_commands = require("./commands");
 function commandArgs(command) {
   const args = [];
@@ -134,12 +135,29 @@ function generateReadmeEntry(command) {
   const suffix = "# " + command.description.toLowerCase();
   return formatWithGap(prefix, suffix, 40);
 }
+function unwrapZodType(schema) {
+  if ("unwrap" in schema && typeof schema.unwrap === "function")
+    return unwrapZodType(schema.unwrap());
+  return schema;
+}
 function generateHelpJSON() {
+  const booleanOptions = /* @__PURE__ */ new Set();
+  for (const command of Object.values(import_commands.commands)) {
+    if (!command.options)
+      continue;
+    const optionsShape = command.options.shape;
+    for (const [name, schema] of Object.entries(optionsShape)) {
+      const innerSchema = unwrapZodType(schema);
+      if (innerSchema instanceof import_mcpBundle.z.ZodBoolean)
+        booleanOptions.add(name);
+    }
+  }
   const help = {
     global: generateHelp(),
     commands: Object.fromEntries(
       Object.entries(import_commands.commands).map(([name, command]) => [name, generateCommandHelp(command)])
-    )
+    ),
+    booleanOptions: [...booleanOptions]
   };
   return help;
 }

package/lib/mcp/terminal/program.js

@@ -56,9 +56,9 @@ class Session {
     if (!this.isCompatible()) {
       throw new Error(`Client is v${this._clientInfo.version}, session '${this.name}' is v${this._config.version}. Run
 
-  playwright-cli session-restart${this.name !== "default" ? ` ${this.name}` : ""}
+  playwright-cli${this.name !== "default" ? ` -s=${this.name}` : ""} open
 
-to restart the session daemon.`);
+to restart the browser session.`);
     }
   }
   async run(args) {
@@ -405,17 +405,28 @@ const booleanOptions = [
 ];
 async function program(packageLocation) {
   const clientInfo = createClientInfo(packageLocation);
+  const help = require("./help.json");
   const argv = process.argv.slice(2);
-  const args = require("minimist")(argv, { boolean: booleanOptions });
-  for (const option of booleanOptions) {
+  const boolean = [...help.booleanOptions, ...booleanOptions];
+  const args = require("minimist")(argv, { boolean });
+  for (const [key, value] of Object.entries(args)) {
+    if (key !== "_" && typeof value !== "boolean")
+      args[key] = String(value);
+  }
+  for (let index = 0; index < args._.length; index++)
+    args._[index] = String(args._[index]);
+  for (const option of boolean) {
     if (!argv.includes(`--${option}`) && !argv.includes(`--no-${option}`))
       delete args[option];
+    if (argv.some((arg) => arg.startsWith(`--${option}=`) || arg.startsWith(`--no-${option}=`))) {
+      console.error(`boolean option '--${option}' should not be passed with '=value', use '--${option}' or '--no-${option}' instead`);
+      process.exit(1);
+    }
   }
   if (args.s) {
     args.session = args.s;
     delete args.s;
   }
-  const help = require("./help.json");
   const commandName = args._?.[0];
   if (args.version || args.v) {
     console.log(clientInfo.version);
@@ -475,16 +486,16 @@ async function install(args) {
   const cwd = process.cwd();
   const playwrightDir = import_path.default.join(cwd, ".playwright");
   await import_fs.default.promises.mkdir(playwrightDir, { recursive: true });
-  console.log(`Workspace initialized at ${cwd}`);
+  console.log(`\u2705 Workspace initialized at \`${cwd}\`.`);
   if (args.skills) {
     const skillSourceDir = import_path.default.join(__dirname, "../../skill");
     const skillDestDir = import_path.default.join(cwd, ".claude", "skills", "playwright-cli");
     if (!import_fs.default.existsSync(skillSourceDir)) {
-      console.error("Skills source directory not found:", skillSourceDir);
+      console.error("\u274C Skills source directory not found:", skillSourceDir);
       process.exit(1);
     }
     await import_fs.default.promises.cp(skillSourceDir, skillDestDir, { recursive: true });
-    console.log(`Skills installed to ${import_path.default.relative(cwd, skillDestDir)}`);
+    console.log(`\u2705 Skills installed to \`${import_path.default.relative(cwd, skillDestDir)}\`.`);
   }
   if (!args.config)
     await ensureConfiguredBrowserInstalled();
@@ -516,7 +527,7 @@ async function createDefaultConfig(channel) {
     }
   };
   await import_fs.default.promises.writeFile(defaultConfigFile(), JSON.stringify(config, null, 2));
-  console.log(`Created default config for ${channel} at ${import_path.default.relative(process.cwd(), defaultConfigFile())}.`);
+  console.log(`\u2705 Created default config for ${channel} at ${import_path.default.relative(process.cwd(), defaultConfigFile())}.`);
 }
 async function findOrInstallDefaultBrowser() {
   const { registry } = await import("playwright-core/lib/server/registry/index");
@@ -525,7 +536,7 @@ async function findOrInstallDefaultBrowser() {
     const executable = registry.findExecutable(channel);
     if (!executable?.executablePath())
       continue;
-    console.log(`Found ${channel}, will use it as the default browser.`);
+    console.log(`\u2705 Found ${channel}, will use it as the default browser.`);
     return channel;
   }
   const chromiumExecutable = registry.findExecutable("chromium");
@@ -670,9 +681,10 @@ async function renderSessionStatus(session) {
   const text = [];
   const config = session.config();
   const canConnect = await session.canConnect();
-  const restartMarker = canConnect && !session.isCompatible() ? ` - v${config.version}, please reopen` : "";
   text.push(`- ${session.name}:`);
-  text.push(`  - status: ${canConnect ? "open" : "closed"}${restartMarker}`);
+  text.push(`  - status: ${canConnect ? "open" : "closed"}`);
+  if (canConnect && !session.isCompatible())
+    text.push(`  - version: v${config.version} [incompatible please re-open]`);
   if (config.resolvedConfig)
     text.push(...renderResolvedConfig(config.resolvedConfig));
   return text.join("\n");