From 27c498e0e783113b5641c6b1a7ce752ba0003bf6 Mon Sep 17 00:00:00 2001
From: Pavel Feldman <pavel.feldman@gmail.com>
Date: Wed, 4 Jun 2025 16:43:11 -0700
Subject: [PATCH] chore: rename browser agent to server (#521)

---
 package.json                                    | 1 +
 src/browserContextFactory.ts                    | 6 +++---
 src/{browserAgent.ts => browserServer.ts}       | 8 ++++----
 tests/{agent.spec.ts => browser-server.spec.ts} | 2 +-
 4 files changed, 9 insertions(+), 8 deletions(-)
 rename src/{browserAgent.ts => browserServer.ts} (97%)
 rename tests/{agent.spec.ts => browser-server.spec.ts} (97%)

diff --git a/package.json b/package.json
index 4492291..1bbc124 100644
--- a/package.json
+++ b/package.json
@@ -24,6 +24,7 @@
     "ctest": "playwright test --project=chrome",
     "ftest": "playwright test --project=firefox",
     "wtest": "playwright test --project=webkit",
+    "run-server": "node lib/browserServer.js",
     "clean": "rm -rf lib",
     "npm-publish": "npm run clean && npm run build && npm run test && npm publish"
   },
diff --git a/src/browserContextFactory.ts b/src/browserContextFactory.ts
index 92fbf18..f14cd7d 100644
--- a/src/browserContextFactory.ts
+++ b/src/browserContextFactory.ts
@@ -24,7 +24,7 @@ import * as playwright from 'playwright';
 import { userDataDir } from './fileUtils.js';
 
 import type { FullConfig } from './config.js';
-import type { BrowserInfo, LaunchBrowserRequest } from './browserAgent.js';
+import type { BrowserInfo, LaunchBrowserRequest } from './browserServer.js';
 
 const testDebug = debug('pw:mcp:test');
 
@@ -36,7 +36,7 @@ export function contextFactory(browserConfig: FullConfig['browser']): BrowserCon
   if (browserConfig.isolated)
     return new IsolatedContextFactory(browserConfig);
   if (browserConfig.browserAgent)
-    return new AgentContextFactory(browserConfig);
+    return new BrowserServerContextFactory(browserConfig);
   return new PersistentContextFactory(browserConfig);
 }
 
@@ -217,7 +217,7 @@ class PersistentContextFactory implements BrowserContextFactory {
   }
 }
 
-export class AgentContextFactory extends BaseContextFactory {
+export class BrowserServerContextFactory extends BaseContextFactory {
   constructor(browserConfig: FullConfig['browser']) {
     super('persistent', browserConfig);
   }
diff --git a/src/browserAgent.ts b/src/browserServer.ts
similarity index 97%
rename from src/browserAgent.ts
rename to src/browserServer.ts
index 67ec6f7..85c908d 100644
--- a/src/browserAgent.ts
+++ b/src/browserServer.ts
@@ -47,7 +47,7 @@ type BrowserEntry = {
   info: BrowserInfo;
 };
 
-class Agent {
+class BrowserServer {
   private _server = new HttpServer();
   private _entries: BrowserEntry[] = [];
 
@@ -121,7 +121,7 @@ class Agent {
   private _updateReport() {
     // Clear the current line and move cursor to top of screen
     process.stdout.write('\x1b[2J\x1b[H');
-    process.stdout.write(`Playwright Browser agent v${packageJSON.version}\n`);
+    process.stdout.write(`Playwright Browser Server v${packageJSON.version}\n`);
     process.stdout.write(`Listening on ${this._server.urlPrefix('human-readable')}\n\n`);
 
     if (this._entries.length === 0) {
@@ -173,8 +173,8 @@ program
 void program.parseAsync(process.argv);
 
 async function main(options: { port: string }) {
-  const agent = new Agent();
-  await agent.start(+options.port);
+  const server = new BrowserServer();
+  await server.start(+options.port);
 }
 
 function readBody<T>(req: http.IncomingMessage): Promise<T> {
diff --git a/tests/agent.spec.ts b/tests/browser-server.spec.ts
similarity index 97%
rename from tests/agent.spec.ts
rename to tests/browser-server.spec.ts
index ce2a8ee..60e8f60 100644
--- a/tests/agent.spec.ts
+++ b/tests/browser-server.spec.ts
@@ -31,7 +31,7 @@ const test = baseTest.extend<{ agentEndpoint: (options?: { args?: string[] }) =>
         throw new Error('Process already running');
 
       cp = spawn('node', [
-        path.join(path.dirname(__filename), '../lib/browserAgent.js'),
+        path.join(path.dirname(__filename), '../lib/browserServer.js'),
         ...(options?.args || []),
       ], {
         stdio: 'pipe',