@wordbricks/playwright-mcp 0.1.19 → 0.1.22

package/lib/browserServerBackend.js

@@ -13,16 +13,16 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import { fileURLToPath } from 'url';
-import { Context } from './context.js';
-import { logUnhandledError } from './utils/log.js';
-import { Response } from './response.js';
-import { SessionLog } from './sessionLog.js';
-import { filteredTools } from './tools.js';
-import { packageJSON } from './utils/package.js';
-import { toMcpTool } from './mcp/tool.js';
+import { fileURLToPath } from "url";
+import { Context } from "./context.js";
+import { toMcpTool } from "./mcp/tool.js";
+import { Response } from "./response.js";
+import { SessionLog } from "./sessionLog.js";
+import { filteredTools } from "./tools.js";
+import { logUnhandledError } from "./utils/log.js";
+import { packageJSON } from "./utils/package.js";
 export class BrowserServerBackend {
-    name = 'Playwright';
+    name = "Playwright";
     version = packageJSON.version;
     _tools;
     _context;
@@ -43,7 +43,9 @@ export class BrowserServerBackend {
             const url = firstRootUri ? new URL(firstRootUri) : undefined;
             rootPath = url ? fileURLToPath(url) : undefined;
         }
-        this._sessionLog = this._config.saveSession ? await SessionLog.create(this._config, rootPath) : undefined;
+        this._sessionLog = this._config.saveSession
+            ? await SessionLog.create(this._config, rootPath)
+            : undefined;
         this._context = new Context({
             tools: this._tools,
             config: this._config,
@@ -55,11 +57,11 @@ export class BrowserServerBackend {
             await this._context.ensureTab();
     }
     async listTools() {
-        return this._tools.map(tool => toMcpTool(tool.schema));
+        return this._tools.map((tool) => toMcpTool(tool.schema));
     }
     async callTool(name, rawArguments) {
         const context = this._context;
-        const tool = context.tools.find(tool => tool.schema.name === name);
+        const tool = context.tools.find((tool) => tool.schema.name === name);
         if (!tool)
             throw new Error(`Tool "${name}" not found`);
         const parsedArguments = tool.schema.inputSchema.parse(rawArguments || {});

package/lib/config.js

@@ -13,17 +13,17 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import fs from 'fs';
-import os from 'os';
-import path from 'path';
-import { devices } from 'playwright-core';
-import { sanitizeForFilePath } from './utils/fileUtils.js';
+import fs from "fs";
+import os from "os";
+import path from "path";
+import { devices } from "playwright-core";
+import { sanitizeForFilePath } from "./utils/fileUtils.js";
 const defaultConfig = {
     browser: {
-        browserName: 'chromium',
+        browserName: "chromium",
         launchOptions: {
-            channel: 'chrome',
-            headless: os.platform() === 'linux' && !process.env.DISPLAY,
+            channel: "chrome",
+            headless: os.platform() === "linux" && !process.env.DISPLAY,
             chromiumSandbox: true,
         },
         contextOptions: {
@@ -54,23 +54,23 @@ export function configFromCLIOptions(cliOptions) {
     let browserName;
     let channel;
     switch (cliOptions.browser) {
-        case 'chrome':
-        case 'chrome-beta':
-        case 'chrome-canary':
-        case 'chrome-dev':
-        case 'chromium':
-        case 'msedge':
-        case 'msedge-beta':
-        case 'msedge-canary':
-        case 'msedge-dev':
-            browserName = 'chromium';
+        case "chrome":
+        case "chrome-beta":
+        case "chrome-canary":
+        case "chrome-dev":
+        case "chromium":
+        case "msedge":
+        case "msedge-beta":
+        case "msedge-canary":
+        case "msedge-dev":
+            browserName = "chromium";
             channel = cliOptions.browser;
             break;
-        case 'firefox':
-            browserName = 'firefox';
+        case "firefox":
+            browserName = "firefox";
             break;
-        case 'webkit':
-            browserName = 'webkit';
+        case "webkit":
+            browserName = "webkit";
             break;
     }
     // Launch options
@@ -78,13 +78,25 @@ export function configFromCLIOptions(cliOptions) {
         channel,
         executablePath: cliOptions.executablePath,
         headless: cliOptions.headless,
+        // Ignore default automation flag to reduce bot detection
+        ignoreDefaultArgs: ["--enable-automation"],
     };
     // --no-sandbox was passed, disable the sandbox
     if (cliOptions.sandbox === false)
         launchOptions.chromiumSandbox = false;
-    // Add default Chrome args to avoid automation detection
+    // Add default Chrome args to avoid automation detection and suppress popups
     launchOptions.args = launchOptions.args || [];
-    launchOptions.args.push('--disable-infobars');
+    launchOptions.args.push("--disable-infobars", 
+    // Password & credential popups
+    "--disable-save-password-bubble", "--password-store=basic", "--use-mock-keychain", 
+    // Crash restore popup
+    "--hide-crash-restore-bubble", 
+    // First run & default browser
+    "--no-first-run", "--no-default-browser-check", 
+    // Disable various Chrome features (Translate, PasswordManager, Autofill, Sync)
+    "--disable-features=Translate,PasswordManager,PasswordManagerEnabled,PasswordManagerOnboarding,AutofillServerCommunication,CredentialManagerOnboarding", "--disable-sync", 
+    // Disable password manager via experimental options
+    "--enable-features=DisablePasswordManager");
     // --app was passed, add app mode argument
     if (cliOptions.app) {
         launchOptions.args = launchOptions.args || [];
@@ -93,9 +105,9 @@ export function configFromCLIOptions(cliOptions) {
     // --window-position was passed, add window position argument
     if (cliOptions.windowPosition) {
         try {
-            const [x, y] = cliOptions.windowPosition.split(',').map(n => +n);
+            const [x, y] = cliOptions.windowPosition.split(",").map((n) => +n);
             if (isNaN(x) || isNaN(y))
-                throw new Error('bad values');
+                throw new Error("bad values");
             launchOptions.args = launchOptions.args || [];
             launchOptions.args.push(`--window-position=${x},${y}`);
         }
@@ -106,9 +118,9 @@ export function configFromCLIOptions(cliOptions) {
     // --window-size was passed, add window size argument
     if (cliOptions.windowSize) {
         try {
-            const [width, height] = cliOptions.windowSize.split(',').map(n => +n);
+            const [width, height] = cliOptions.windowSize.split(",").map((n) => +n);
             if (isNaN(width) || isNaN(height))
-                throw new Error('bad values');
+                throw new Error("bad values");
             launchOptions.args = launchOptions.args || [];
             launchOptions.args.push(`--window-size=${width},${height}`);
         }
@@ -118,24 +130,26 @@ export function configFromCLIOptions(cliOptions) {
     }
     if (cliOptions.proxyServer) {
         launchOptions.proxy = {
-            server: cliOptions.proxyServer
+            server: cliOptions.proxyServer,
         };
         if (cliOptions.proxyBypass)
             launchOptions.proxy.bypass = cliOptions.proxyBypass;
     }
     if (cliOptions.device && cliOptions.cdpEndpoint)
-        throw new Error('Device emulation is not supported with cdpEndpoint.');
+        throw new Error("Device emulation is not supported with cdpEndpoint.");
     // Context options
-    const contextOptions = cliOptions.device ? devices[cliOptions.device] : {};
+    const contextOptions = cliOptions.device
+        ? devices[cliOptions.device]
+        : {};
     if (cliOptions.storageState)
         contextOptions.storageState = cliOptions.storageState;
     if (cliOptions.userAgent)
         contextOptions.userAgent = cliOptions.userAgent;
     if (cliOptions.viewportSize) {
         try {
-            const [width, height] = cliOptions.viewportSize.split(',').map(n => +n);
+            const [width, height] = cliOptions.viewportSize.split(",").map((n) => +n);
             if (isNaN(width) || isNaN(height))
-                throw new Error('bad values');
+                throw new Error("bad values");
             contextOptions.viewport = { width, height };
         }
         catch (e) {
@@ -145,7 +159,7 @@ export function configFromCLIOptions(cliOptions) {
     if (cliOptions.ignoreHttpsErrors)
         contextOptions.ignoreHTTPSErrors = true;
     if (cliOptions.blockServiceWorkers)
-        contextOptions.serviceWorkers = 'block';
+        contextOptions.serviceWorkers = "block";
     const result = {
         browser: {
             browserName,
@@ -190,8 +204,8 @@ function configFromEnv() {
     options.ignoreHttpsErrors = envToBoolean(process.env.PLAYWRIGHT_MCP_IGNORE_HTTPS_ERRORS);
     options.initScript = envToString(process.env.PLAYWRIGHT_MCP_INIT_SCRIPT);
     options.isolated = envToBoolean(process.env.PLAYWRIGHT_MCP_ISOLATED);
-    if (process.env.PLAYWRIGHT_MCP_IMAGE_RESPONSES === 'omit')
-        options.imageResponses = 'omit';
+    if (process.env.PLAYWRIGHT_MCP_IMAGE_RESPONSES === "omit")
+        options.imageResponses = "omit";
     options.sandbox = envToBoolean(process.env.PLAYWRIGHT_MCP_SANDBOX);
     options.outputDir = envToString(process.env.PLAYWRIGHT_MCP_OUTPUT_DIR);
     options.port = envToNumber(process.env.PLAYWRIGHT_MCP_PORT);
@@ -210,16 +224,16 @@ async function loadConfig(configFile) {
     if (!configFile)
         return {};
     try {
-        return JSON.parse(await fs.promises.readFile(configFile, 'utf8'));
+        return JSON.parse(await fs.promises.readFile(configFile, "utf8"));
     }
     catch (error) {
         throw new Error(`Failed to load config file: ${configFile}, ${error}`);
     }
 }
 export async function outputFile(config, rootPath, name) {
-    const outputDir = config.outputDir
-        ?? (rootPath ? path.join(rootPath, '.playwright-mcp') : undefined)
-        ?? path.join(os.tmpdir(), 'playwright-mcp-output', sanitizeForFilePath(new Date().toISOString()));
+    const outputDir = config.outputDir ??
+        (rootPath ? path.join(rootPath, ".playwright-mcp") : undefined) ??
+        path.join(os.tmpdir(), "playwright-mcp-output", sanitizeForFilePath(new Date().toISOString()));
     await fs.promises.mkdir(outputDir, { recursive: true });
     const fileName = sanitizeForFilePath(name);
     return path.join(outputDir, fileName);
@@ -231,7 +245,7 @@ function mergeConfig(base, overrides) {
     const browser = {
         ...pickDefined(base.browser),
         ...pickDefined(overrides.browser),
-        browserName: overrides.browser?.browserName ?? base.browser?.browserName ?? 'chromium',
+        browserName: overrides.browser?.browserName ?? base.browser?.browserName ?? "chromium",
         isolated: overrides.browser?.isolated ?? base.browser?.isolated ?? false,
         launchOptions: {
             ...pickDefined(base.browser?.launchOptions),
@@ -243,7 +257,7 @@ function mergeConfig(base, overrides) {
             ...pickDefined(overrides.browser?.contextOptions),
         },
     };
-    if (browser.browserName !== 'chromium' && browser.launchOptions)
+    if (browser.browserName !== "chromium" && browser.launchOptions)
         delete browser.launchOptions.channel;
     return {
         ...pickDefined(base),
@@ -262,12 +276,12 @@ function mergeConfig(base, overrides) {
 export function semicolonSeparatedList(value) {
     if (!value)
         return undefined;
-    return value.split(';').map(v => v.trim());
+    return value.split(";").map((v) => v.trim());
 }
 export function commaSeparatedList(value) {
     if (!value)
         return undefined;
-    return value.split(',').map(v => v.trim());
+    return value.split(",").map((v) => v.trim());
 }
 function envToNumber(value) {
     if (!value)
@@ -275,9 +289,9 @@ function envToNumber(value) {
     return +value;
 }
 function envToBoolean(value) {
-    if (value === 'true' || value === '1')
+    if (value === "true" || value === "1")
         return true;
-    if (value === 'false' || value === '0')
+    if (value === "false" || value === "0")
         return false;
     return undefined;
 }

package/lib/context.js

@@ -13,24 +13,24 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import debug from 'debug';
-import { logUnhandledError } from './utils/log.js';
-import { Tab } from './tab.js';
-import { applyHooksToTools, hookRegistryMap } from './hooks/core.js';
-import { buildHookRegistry } from './hooks/registry.js';
-import { setEventStore, createEventStore } from './hooks/events.js';
-import { setupNetworkTracking } from './hooks/networkSetup.js';
-import { outputFile } from './config.js';
-import * as codegen from './utils/codegen.js';
-import { shouldBlockRequest } from './utils/adBlockFilter.js';
-const testDebug = debug('pw:mcp:test');
+import debug from "debug";
+import { outputFile } from "./config.js";
+import { applyHooksToTools, hookRegistryMap } from "./hooks/core.js";
+import { createEventStore, setEventStore } from "./hooks/events.js";
+import { setupNetworkTracking } from "./hooks/networkSetup.js";
+import { buildHookRegistry } from "./hooks/registry.js";
+import { Tab } from "./tab.js";
+import { shouldBlockRequest } from "./utils/adBlockFilter.js";
+import * as codegen from "./utils/codegen.js";
+import { logUnhandledError } from "./utils/log.js";
+const testDebug = debug("pw:mcp:test");
 const protocolPattern = /^[a-zA-Z][a-zA-Z0-9+.-]*:\/\//;
 const defaultPortForProtocol = (protocol) => {
-    if (protocol === 'http:')
-        return '80';
-    if (protocol === 'https:')
-        return '443';
-    return '';
+    if (protocol === "http:")
+        return "80";
+    if (protocol === "https:")
+        return "443";
+    return "";
 };
 const matchesOriginHost = (requestUrl, candidate) => {
     const normalized = candidate.trim();
@@ -47,15 +47,16 @@ const matchesOriginHost = (requestUrl, candidate) => {
     })();
     if (!parsed)
         return false;
-    const candidateHost = parsed.hostname.toLowerCase().replace(/\.+$/, '');
+    const candidateHost = parsed.hostname.toLowerCase().replace(/\.+$/, "");
     if (!candidateHost)
         return false;
     const requestHost = requestUrl.hostname.toLowerCase();
-    if (requestHost !== candidateHost && !requestHost.endsWith(`.${candidateHost}`))
+    if (requestHost !== candidateHost &&
+        !requestHost.endsWith(`.${candidateHost}`))
         return false;
     if (hasProtocol && parsed.protocol !== requestUrl.protocol)
         return false;
-    if (parsed.port === '')
+    if (parsed.port === "")
         return true;
     const candidatePort = parsed.port || defaultPortForProtocol(parsed.protocol);
     if (!candidatePort)
@@ -88,11 +89,11 @@ export class Context {
         this.options = options;
         this._browserContextFactory = options.browserContextFactory;
         this._clientInfo = options.clientInfo;
-        testDebug('create context');
+        testDebug("create context");
         Context._allContexts.add(this);
     }
     static async disposeAll() {
-        await Promise.all([...Context._allContexts].map(context => context.dispose()));
+        await Promise.all([...Context._allContexts].map((context) => context.dispose()));
     }
     tabs() {
         return this._tabs;
@@ -108,7 +109,7 @@ export class Context {
     async newTab() {
         const { browserContext } = await this._ensureBrowserContext();
         const page = await browserContext.newPage();
-        this._currentTab = this._tabs.find(t => t.page === page);
+        this._currentTab = this._tabs.find((t) => t.page === page);
         return this._currentTab;
     }
     async selectTab(index) {
@@ -137,7 +138,7 @@ export class Context {
         return outputFile(this.config, this._clientInfo.rootPath, name);
     }
     _onPageCreated(page) {
-        const tab = new Tab(this, page, tab => this._onPageClosed(tab));
+        const tab = new Tab(this, page, (tab) => this._onPageClosed(tab));
         this._tabs.push(tab);
         if (!this._currentTab)
             this._currentTab = tab;
@@ -156,7 +157,8 @@ export class Context {
     }
     async closeBrowserContext() {
         if (!this._closeBrowserContextPromise)
-            this._closeBrowserContextPromise = this._closeBrowserContextImpl().catch(logUnhandledError);
+            this._closeBrowserContextPromise =
+                this._closeBrowserContextImpl().catch(logUnhandledError);
         await this._closeBrowserContextPromise;
         this._closeBrowserContextPromise = undefined;
     }
@@ -169,7 +171,7 @@ export class Context {
     async _closeBrowserContextImpl() {
         if (!this._browserContextPromise)
             return;
-        testDebug('close context');
+        testDebug("close context");
         const promise = this._browserContextPromise;
         this._browserContextPromise = undefined;
         await promise.then(async ({ browserContext, close }) => {
@@ -179,12 +181,12 @@ export class Context {
         });
     }
     async dispose() {
-        this._abortController.abort('MCP context disposed');
+        this._abortController.abort("MCP context disposed");
         await this.closeBrowserContext();
         Context._allContexts.delete(this);
     }
     async _setupRequestInterception(context) {
-        await context.route('**', route => {
+        await context.route("**", (route) => {
             const request = route.request();
             const url = request.url();
             const resourceType = request.resourceType();
@@ -198,20 +200,20 @@ export class Context {
             }
             const domain = urlObj.hostname;
             if (shouldBlockRequest(url, domain, resourceType)) {
-                void route.abort('blockedbyclient');
+                void route.abort("blockedbyclient");
                 return;
             }
             if (this.config.network?.allowedOrigins?.length) {
-                const isAllowed = this.config.network.allowedOrigins.some(allowed => matchesOriginHost(urlObj, allowed));
+                const isAllowed = this.config.network.allowedOrigins.some((allowed) => matchesOriginHost(urlObj, allowed));
                 if (!isAllowed) {
-                    void route.abort('blockedbyclient');
+                    void route.abort("blockedbyclient");
                     return;
                 }
             }
             if (this.config.network?.blockedOrigins?.length) {
-                const isBlocked = this.config.network.blockedOrigins.some(blocked => matchesOriginHost(urlObj, blocked));
+                const isBlocked = this.config.network.blockedOrigins.some((blocked) => matchesOriginHost(urlObj, blocked));
                 if (isBlocked) {
-                    void route.abort('blockedbyclient');
+                    void route.abort("blockedbyclient");
                     return;
                 }
             }
@@ -229,7 +231,7 @@ export class Context {
     }
     async _setupBrowserContext() {
         if (this._closeBrowserContextPromise)
-            throw new Error('Another browser context is being closed.');
+            throw new Error("Another browser context is being closed.");
         // TODO: move to the browser context factory to make it based on isolation mode.
         const result = await this._browserContextFactory.createContext(this._clientInfo, this._abortController.signal);
         const { browserContext } = result;
@@ -238,10 +240,10 @@ export class Context {
             await InputRecorder.create(this, browserContext);
         for (const page of browserContext.pages())
             this._onPageCreated(page);
-        browserContext.on('page', page => this._onPageCreated(page));
+        browserContext.on("page", (page) => this._onPageCreated(page));
         if (this.config.saveTrace) {
             await browserContext.tracing.start({
-                name: 'trace',
+                name: "trace",
                 screenshots: false,
                 snapshots: true,
                 sources: false,
@@ -273,8 +275,8 @@ export class InputRecorder {
     async _initialize() {
         const sessionLog = this._context.sessionLog;
         await this._browserContext._enableRecorder({
-            mode: 'recording',
-            recorderMode: 'api',
+            mode: "recording",
+            recorderMode: "api",
         }, {
             actionAdded: (page, data, code) => {
                 if (this._context.isRunningTool())
@@ -293,11 +295,11 @@ export class InputRecorder {
             signalAdded: (page, data) => {
                 if (this._context.isRunningTool())
                     return;
-                if (data.signal.name !== 'navigation')
+                if (data.signal.name !== "navigation")
                     return;
                 const tab = Tab.forPage(page);
                 const navigateAction = {
-                    name: 'navigate',
+                    name: "navigate",
                     url: data.signal.url,
                     signals: [],
                 };