test: add browser agent integration tests (#21151)

packages/core/src/agents/browser/browserManager.test.ts

@@ -147,7 +147,7 @@ describe('BrowserManager', () => {
       // Verify StdioClientTransport was created with correct args
       expect(StdioClientTransport).toHaveBeenCalledWith(
         expect.objectContaining({
-          command: 'npx',
+          command: process.platform === 'win32' ? 'npx.cmd' : 'npx',
           args: expect.arrayContaining([
             '-y',
             expect.stringMatching(/chrome-devtools-mcp@/),
@@ -185,7 +185,7 @@ describe('BrowserManager', () => {
 
       expect(StdioClientTransport).toHaveBeenCalledWith(
         expect.objectContaining({
-          command: 'npx',
+          command: process.platform === 'win32' ? 'npx.cmd' : 'npx',
           args: expect.arrayContaining(['--headless']),
         }),
       );
@@ -210,7 +210,7 @@ describe('BrowserManager', () => {
 
       expect(StdioClientTransport).toHaveBeenCalledWith(
         expect.objectContaining({
-          command: 'npx',
+          command: process.platform === 'win32' ? 'npx.cmd' : 'npx',
           args: expect.arrayContaining(['--userDataDir', '/path/to/profile']),
         }),
       );

packages/core/src/agents/browser/browserManager.ts

@@ -283,7 +283,7 @@ export class BrowserManager {
     // stderr is piped (not inherited) to prevent MCP server banners and
     // warnings from corrupting the UI in alternate buffer mode.
     this.mcpTransport = new StdioClientTransport({
-      command: 'npx',
+      command: process.platform === 'win32' ? 'npx.cmd' : 'npx',
       args: mcpArgs,
       stderr: 'pipe',
     });