Revert "feat(mcp): add headers capability (#37583)" (#37970)

Author: yury-s

packages/playwright/src/mcp/browser/context.ts

@@ -52,7 +52,6 @@ export class Context {
   private _tabs: Tab[] = [];
   private _currentTab: Tab | undefined;
   private _clientInfo: ClientInfo;
-  private _extraHTTPHeaders: Record<string, string> | undefined;
 
   private static _allContexts: Set<Context> = new Set();
   private _closeBrowserContextPromise: Promise<void> | undefined;
@@ -221,20 +220,6 @@ export class Context {
     return browserContext;
   }
 
-  async setExtraHTTPHeaders(headers: Record<string, string>) {
-    if (!Object.keys(headers).length)
-      throw new Error('Please provide at least one header to set.');
-
-    for (const name of Object.keys(headers)) {
-      if (!name.trim())
-        throw new Error('Header names must be non-empty strings.');
-    }
-
-    this._extraHTTPHeaders = { ...headers };
-    const { browserContext } = await this._ensureBrowserContext();
-    await browserContext.setExtraHTTPHeaders(this._extraHTTPHeaders);
-  }
-
   private _ensureBrowserContext() {
     if (!this._browserContextPromise) {
       this._browserContextPromise = this._setupBrowserContext();
@@ -255,8 +240,6 @@ export class Context {
     const result = await this._browserContextFactory.createContext(this._clientInfo, this._abortController.signal, this._runningToolName);
     const { browserContext } = result;
     await this._setupRequestInterception(browserContext);
-    if (this._extraHTTPHeaders)
-      await browserContext.setExtraHTTPHeaders(this._extraHTTPHeaders);
     if (this.sessionLog)
       await InputRecorder.create(this, browserContext);
     for (const page of browserContext.pages())

packages/playwright/src/mcp/browser/tools.ts

@@ -26,7 +26,6 @@ import mouse from './tools/mouse';
 import navigate from './tools/navigate';
 import network from './tools/network';
 import pdf from './tools/pdf';
-import headers from './tools/headers';
 import snapshot from './tools/snapshot';
 import screenshot from './tools/screenshot';
 import tabs from './tools/tabs';
@@ -48,7 +47,6 @@ export const browserTools: Tool<any>[] = [
   ...keyboard,
   ...navigate,
   ...network,
-  ...headers,
   ...mouse,
   ...pdf,
   ...screenshot,

packages/playwright/src/mcp/browser/tools/headers.ts

@@ -16,7 +16,7 @@
 
 import type * as playwright from 'playwright-core';
 
-export type ToolCapability = 'core' | 'core-tabs' | 'core-install' | 'vision' | 'pdf' | 'testing' | 'tracing' | 'headers';
+export type ToolCapability = 'core' | 'core-tabs' | 'core-install' | 'vision' | 'pdf' | 'testing' | 'tracing';
 
 export type Config = {
   /**
@@ -99,7 +99,6 @@ export type Config = {
    *   - 'core': Core browser automation features.
    *   - 'pdf': PDF generation and manipulation.
    *   - 'vision': Coordinate-based interactions.
-   *   - 'headers': Manage persistent custom HTTP headers.
    */
   capabilities?: ToolCapability[];
 
@@ -172,3 +171,4 @@ export type Config = {
    */
   imageResponses?: 'allow' | 'omit';
 };
+

packages/playwright/src/mcp/config.d.ts

@@ -33,7 +33,7 @@ export function decorateCommand(command: Command, version: string) {
       .option('--blocked-origins <origins>', 'semicolon-separated list of origins to block the browser from requesting. Blocklist is evaluated before allowlist. If used without the allowlist, requests not matching the blocklist are still allowed.', semicolonSeparatedList)
       .option('--block-service-workers', 'block service workers')
       .option('--browser <browser>', 'browser or chrome channel to use, possible values: chrome, firefox, webkit, msedge.')
-      .option('--caps <caps>', 'comma-separated list of additional capabilities to enable, possible values: vision, pdf, headers.', commaSeparatedList)
+      .option('--caps <caps>', 'comma-separated list of additional capabilities to enable, possible values: vision, pdf.', commaSeparatedList)
       .option('--cdp-endpoint <endpoint>', 'CDP endpoint to connect to.')
       .option('--cdp-header <headers...>', 'CDP headers to send with the connect request, multiple can be specified.', headerParser)
       .option('--config <path>', 'path to the configuration file.')

packages/playwright/src/mcp/program.ts

@@ -41,7 +41,6 @@ test('generator tools intent', async ({ startClient }) => {
     'browser_type',
     'browser_navigate',
     'browser_navigate_back',
-    'browser_set_headers',
     'browser_mouse_move_xy',
     'browser_mouse_click_xy',
     'browser_mouse_drag_xy',