feat(mcp): remove remoteHeaders in favor of remoteEndpoint headers

Drops the remoteHeaders config field, the --remote-header CLI flag, and
the PLAYWRIGHT_MCP_REMOTE_HEADERS env var. Headers for a remote connect
are now supplied through the ConnectOptions object form of remoteEndpoint.

Author: adityasingh2400

packages/playwright-core/src/tools/mcp/browserFactory.ts

@@ -121,7 +121,7 @@ async function createRemoteBrowser(config: FullConfig): Promise<BrowserWithInfo>
   // shape.
   const remote = config.browser.remoteEndpoint!;
   const remoteOptions = typeof remote === 'string'
-    ? { endpoint: remote, headers: config.browser.remoteHeaders }
+    ? { endpoint: remote }
     : remote;
 
   const descriptor = await serverRegistry.find(remoteOptions.endpoint);

packages/playwright-core/src/tools/mcp/config.d.ts

@@ -90,13 +90,6 @@ export type Config = {
      */
     remoteEndpoint?: string | playwright.ConnectOptions & { endpoint: string };
 
-    /**
-     * Headers to send with the remote endpoint connect request. Ignored when
-     * `remoteEndpoint` is provided as a [ConnectOptions] object; supply
-     * `headers` on that object instead.
-     */
-    remoteHeaders?: Record<string, string>;
-
     /**
      * Paths to TypeScript files to add as initialization scripts for Playwright page.
      */

packages/playwright-core/src/tools/mcp/config.ts

@@ -63,7 +63,6 @@ export type CLIOptions = {
   port?: number;
   proxyBypass?: string;
   proxyServer?: string;
-  remoteHeader?: Record<string, string>;
   saveSession?: boolean;
   secrets?: Record<string, string>;
   sharedBrowserContext?: boolean;
@@ -333,7 +332,6 @@ function configFromCLIOptions(cliOptions: CLIOptions): Config & { configFile?: s
       initPage: cliOptions.initPage,
       initScript: cliOptions.initScript,
       remoteEndpoint: cliOptions.endpoint,
-      remoteHeaders: cliOptions.remoteHeader,
     },
     extension: cliOptions.extension,
     server: {
@@ -404,7 +402,6 @@ export function configFromEnv(env?: NodeJS.ProcessEnv): Config & { configFile?:
   options.port = numberParser(e.PLAYWRIGHT_MCP_PORT);
   options.proxyBypass = envToString(e.PLAYWRIGHT_MCP_PROXY_BYPASS);
   options.proxyServer = envToString(e.PLAYWRIGHT_MCP_PROXY_SERVER);
-  options.remoteHeader = headerParser(envToString(e.PLAYWRIGHT_MCP_REMOTE_HEADERS));
   options.secrets = dotenvFileLoader(e.PLAYWRIGHT_MCP_SECRETS_FILE);
   options.storageState = envToString(e.PLAYWRIGHT_MCP_STORAGE_STATE);
   options.testIdAttribute = envToString(e.PLAYWRIGHT_MCP_TEST_ID_ATTRIBUTE);

packages/playwright-core/src/tools/mcp/program.ts

@@ -63,7 +63,6 @@ export function decorateMCPCommand(command: Command) {
       .option('--port <port>', 'port to listen on for SSE transport.')
       .option('--proxy-bypass <bypass>', 'comma-separated domains to bypass proxy, for example ".com,chromium.org,.domain.com"')
       .option('--proxy-server <proxy>', 'specify proxy server, for example "http://myproxy:3128" or "socks5://myproxy:8080"')
-      .option('--remote-header <headers...>', 'headers to send with the remote endpoint connect request, multiple can be specified.', headerParser)
       .option('--sandbox', 'enable the sandbox for all process types that are normally not sandboxed.')
       .option('--save-session', 'Whether to save the Playwright MCP session into the output directory.')
       .option('--secrets <path>', 'path to a file containing secrets in the dotenv format', dotenvFileLoader)

tests/mcp/cli-remote.spec.ts

@@ -19,12 +19,14 @@ import { test, expect } from './cli-fixtures';
 
 test.skip(({ mcpBrowser }) => mcpBrowser !== 'chromium', 'Run only on the chromium project; the remote server connection is browser-agnostic.');
 
-test('attach to run-server endpoint with remoteHeaders from config', async ({ cli, runServerEndpoint, server }, testInfo) => {
+test('attach to run-server endpoint with headers from config', async ({ cli, runServerEndpoint, server }, testInfo) => {
   const configPath = testInfo.outputPath('config.json');
   await fs.promises.writeFile(configPath, JSON.stringify({
     browser: {
-      remoteEndpoint: runServerEndpoint,
-      remoteHeaders: { 'x-playwright-browser': 'chromium' },
+      remoteEndpoint: {
+        endpoint: runServerEndpoint,
+        headers: { 'x-playwright-browser': 'chromium' },
+      },
       isolated: true,
     },
   }, null, 2));

tests/mcp/remote-endpoint.spec.ts

@@ -18,27 +18,7 @@ import { test, expect } from './fixtures';
 
 test.skip(({ mcpBrowser }) => mcpBrowser !== 'chromium', 'Run only on the chromium project; the remote server connection is browser-agnostic.');
 
-test('remoteHeaders selects the browser on run-server endpoint', async ({ startClient, server, runServerEndpoint }) => {
-  const { client } = await startClient({
-    config: {
-      browser: {
-        remoteEndpoint: runServerEndpoint,
-        remoteHeaders: { 'x-playwright-browser': 'chromium' },
-        isolated: true,
-      },
-    },
-  });
-
-  const response = await client.callTool({
-    name: 'browser_navigate',
-    arguments: { url: server.HELLO_WORLD },
-  });
-  expect(response).toHaveResponse({
-    page: expect.stringContaining('Page Title: Title'),
-  });
-});
-
-test('connect without remoteHeaders fails on run-server endpoint', async ({ startClient, server, runServerEndpoint }) => {
+test('connect without headers fails on run-server endpoint', async ({ startClient, server, runServerEndpoint }) => {
   const { client } = await startClient({
     config: {
       browser: {