@matware/e2e-runner 1.0.1 → 1.0.3

package/README.md

@@ -318,13 +318,22 @@ import {
 
 ## Claude Code Integration (MCP)
 
-The package includes a built-in [MCP server](https://modelcontextprotocol.io/) that gives Claude Code native access to the test runner. Install once and it's available in every project:
+The package includes a built-in [MCP server](https://modelcontextprotocol.io/) that gives Claude Code native access to the test runner. Install once and it's available in every project.
+
+**Via npm** (requires Node.js):
 
 ```bash
 claude mcp add --transport stdio --scope user e2e-runner \
   -- npx -y -p @matware/e2e-runner e2e-runner-mcp
 ```
 
+**Via Docker** (no Node.js required):
+
+```bash
+claude mcp add --transport stdio --scope user e2e-runner \
+  -- docker run -i --rm fastslack/e2e-runner-mcp
+```
+
 ### MCP Tools
 
 | Tool | Description |
@@ -336,6 +345,8 @@ claude mcp add --transport stdio --scope user e2e-runner \
 | `e2e_pool_start` | Start the Chrome pool Docker container |
 | `e2e_pool_stop` | Stop the Chrome pool |
 
+All tools accept an optional `cwd` parameter (absolute path to the project root). Claude Code passes its current working directory so the MCP server resolves `e2e/tests/`, `e2e.config.js`, and `.e2e-pool/` relative to the correct project — even when switching between multiple projects in the same session.
+
 Once installed, Claude Code can run tests, analyze failures, create new test files, and manage the Chrome pool as part of its normal workflow. Just ask:
 
 > "Run all E2E tests"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@matware/e2e-runner",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "mcpName": "io.github.fastslack/e2e-runner",
   "description": "E2E test runner using Chrome Pool (browserless/chrome) with parallel execution",
   "type": "module",

package/src/config.js

@@ -68,8 +68,8 @@ async function loadConfigFile(cwd) {
   return {};
 }
 
-export async function loadConfig(cliArgs = {}) {
-  const cwd = process.cwd();
+export async function loadConfig(cliArgs = {}, cwd = null) {
+  cwd = cwd || process.cwd();
   const fileConfig = await loadConfigFile(cwd);
   const envConfig = loadEnvVars();
 

package/src/mcp-server.js

@@ -61,6 +61,10 @@ const TOOLS = [
           type: 'number',
           description: 'Number of retries for failed tests',
         },
+        cwd: {
+          type: 'string',
+          description: 'Absolute path to the project root directory. Claude Code should pass its current working directory.',
+        },
       },
     },
   },
@@ -70,7 +74,12 @@ const TOOLS = [
       'List all available E2E test suites with their test names and counts.',
     inputSchema: {
       type: 'object',
-      properties: {},
+      properties: {
+        cwd: {
+          type: 'string',
+          description: 'Absolute path to the project root directory. Claude Code should pass its current working directory.',
+        },
+      },
     },
   },
   {
@@ -122,6 +131,10 @@ const TOOLS = [
             afterEach: { type: 'array', items: { type: 'object' } },
           },
         },
+        cwd: {
+          type: 'string',
+          description: 'Absolute path to the project root directory. Claude Code should pass its current working directory.',
+        },
       },
       required: ['name', 'tests'],
     },
@@ -132,7 +145,12 @@ const TOOLS = [
       'Get the status of the Chrome pool (browserless/chrome). Shows availability, running sessions, capacity, and queued requests.',
     inputSchema: {
       type: 'object',
-      properties: {},
+      properties: {
+        cwd: {
+          type: 'string',
+          description: 'Absolute path to the project root directory. Claude Code should pass its current working directory.',
+        },
+      },
     },
   },
   {
@@ -150,6 +168,10 @@ const TOOLS = [
           type: 'number',
           description: 'Max concurrent Chrome sessions (default 10)',
         },
+        cwd: {
+          type: 'string',
+          description: 'Absolute path to the project root directory. Claude Code should pass its current working directory.',
+        },
       },
     },
   },
@@ -158,7 +180,12 @@ const TOOLS = [
     description: 'Stop the Chrome pool Docker container.',
     inputSchema: {
       type: 'object',
-      properties: {},
+      properties: {
+        cwd: {
+          type: 'string',
+          description: 'Absolute path to the project root directory. Claude Code should pass its current working directory.',
+        },
+      },
     },
   },
 ];
@@ -171,7 +198,7 @@ async function handleRun(args) {
   if (args.baseUrl) configOverrides.baseUrl = args.baseUrl;
   if (args.retries !== undefined) configOverrides.retries = args.retries;
 
-  const config = await loadConfig(configOverrides);
+  const config = await loadConfig(configOverrides, args.cwd);
 
   await waitForPool(config.poolUrl);
 
@@ -182,7 +209,8 @@ async function handleRun(args) {
   } else if (args.suite) {
     ({ tests, hooks } = loadTestSuite(args.suite, config.testsDir));
   } else if (args.file) {
-    const filePath = path.isAbsolute(args.file) ? args.file : path.resolve(args.file);
+    const cwd = args.cwd || process.cwd();
+    const filePath = path.isAbsolute(args.file) ? args.file : path.resolve(cwd, args.file);
     ({ tests, hooks } = loadTestFile(filePath));
   } else {
     return errorResult('Provide one of: all (true), suite (name), or file (path)');
@@ -213,14 +241,23 @@ async function handleRun(args) {
     reportPath: path.join(config.screenshotsDir, 'report.json'),
   };
 
+  const consoleErrors = report.results
+    .filter(r => r.consoleLogs?.some(l => l.type === 'error' || l.type === 'warning'))
+    .map(r => ({ name: r.name, logs: r.consoleLogs.filter(l => l.type === 'error' || l.type === 'warning') }));
+  const networkErrors = report.results
+    .filter(r => r.networkErrors?.length > 0)
+    .map(r => ({ name: r.name, errors: r.networkErrors }));
+
   if (flaky.length > 0) summary.flaky = flaky;
   if (failures.length > 0) summary.failures = failures;
+  if (consoleErrors.length > 0) summary.consoleErrors = consoleErrors;
+  if (networkErrors.length > 0) summary.networkErrors = networkErrors;
 
   return textResult(JSON.stringify(summary, null, 2));
 }
 
-async function handleList() {
-  const config = await loadConfig({});
+async function handleList(args) {
+  const config = await loadConfig({}, args.cwd);
   const suites = listSuites(config.testsDir);
 
   if (suites.length === 0) {
@@ -235,7 +272,7 @@ async function handleList() {
 }
 
 async function handleCreateTest(args) {
-  const config = await loadConfig({});
+  const config = await loadConfig({}, args.cwd);
 
   if (!fs.existsSync(config.testsDir)) {
     fs.mkdirSync(config.testsDir, { recursive: true });
@@ -259,8 +296,8 @@ async function handleCreateTest(args) {
   return textResult(`Created test file: ${filePath}\n\n${args.tests.length} test(s) defined.`);
 }
 
-async function handlePoolStatus() {
-  const config = await loadConfig({});
+async function handlePoolStatus(args) {
+  const config = await loadConfig({}, args.cwd);
   const status = await getPoolStatus(config.poolUrl);
 
   const lines = [
@@ -282,14 +319,14 @@ async function handlePoolStart(args) {
   if (args.port) overrides.poolPort = args.port;
   if (args.maxSessions) overrides.maxSessions = args.maxSessions;
 
-  const config = await loadConfig(overrides);
-  startPool(config);
+  const config = await loadConfig(overrides, args.cwd);
+  startPool(config, args.cwd);
   return textResult(`Chrome pool started on port ${config.poolPort}`);
 }
 
-async function handlePoolStop() {
-  const config = await loadConfig({});
-  stopPool(config);
+async function handlePoolStop(args) {
+  const config = await loadConfig({}, args.cwd);
+  stopPool(config, args.cwd);
   return textResult('Chrome pool stopped');
 }
 
@@ -323,15 +360,15 @@ export async function startMcpServer() {
         case 'e2e_run':
           return await handleRun(args);
         case 'e2e_list':
-          return await handleList();
+          return await handleList(args);
         case 'e2e_create_test':
           return await handleCreateTest(args);
         case 'e2e_pool_status':
-          return await handlePoolStatus();
+          return await handlePoolStatus(args);
         case 'e2e_pool_start':
           return await handlePoolStart(args);
         case 'e2e_pool_stop':
-          return await handlePoolStop();
+          return await handlePoolStop(args);
         default:
           return errorResult(`Unknown tool: ${name}`);
       }

package/src/pool.js

@@ -61,8 +61,8 @@ export async function connectToPool(poolUrl, retries = 3, delay = 2000) {
 }
 
 /** Generates docker-compose.yml and starts the pool */
-export function startPool(config) {
-  const cwd = process.cwd();
+export function startPool(config, cwd = null) {
+  cwd = cwd || process.cwd();
   const poolDir = path.join(cwd, '.e2e-pool');
 
   if (!fs.existsSync(poolDir)) {
@@ -93,8 +93,9 @@ export function startPool(config) {
 }
 
 /** Stops the pool */
-export function stopPool(config) {
-  const composePath = path.join(process.cwd(), '.e2e-pool', 'docker-compose.yml');
+export function stopPool(config, cwd = null) {
+  cwd = cwd || process.cwd();
+  const composePath = path.join(cwd, '.e2e-pool', 'docker-compose.yml');
   if (!fs.existsSync(composePath)) {
     log('⚠️', '.e2e-pool/docker-compose.yml not found');
     return;
@@ -106,9 +107,9 @@ export function stopPool(config) {
 }
 
 /** Restarts the pool */
-export function restartPool(config) {
-  stopPool(config);
-  startPool(config);
+export function restartPool(config, cwd = null) {
+  stopPool(config, cwd);
+  startPool(config, cwd);
 }
 
 /** Gets pool status */

package/src/reporter.js

@@ -55,6 +55,11 @@ export function generateJUnitXML(report) {
       xml += `      <system-out><![CDATA[${logs}]]></system-out>\n`;
     }
 
+    const netErrors = (result.networkErrors || []).map(e => `[${e.error || 'unknown'}] ${e.url}`).join('\n');
+    if (netErrors) {
+      xml += `      <system-err><![CDATA[${netErrors}]]></system-err>\n`;
+    }
+
     xml += '    </testcase>\n';
   }
 
@@ -109,6 +114,31 @@ export function printReport(report, screenshotsDir) {
     });
   }
 
+  const consoleIssues = report.results.filter(r =>
+    r.consoleLogs?.some(l => l.type === 'error' || l.type === 'warning')
+  );
+  if (consoleIssues.length > 0) {
+    console.log(`\n${C.yellow}${C.bold}BROWSER CONSOLE ISSUES:${C.reset}`);
+    consoleIssues.forEach(r => {
+      const logs = r.consoleLogs.filter(l => l.type === 'error' || l.type === 'warning');
+      console.log(`  ${C.yellow}⚠${C.reset} ${r.name}:`);
+      logs.forEach(l => {
+        console.log(`    ${C.dim}[${l.type}]${C.reset} ${l.text}`);
+      });
+    });
+  }
+
+  const networkIssues = report.results.filter(r => r.networkErrors?.length > 0);
+  if (networkIssues.length > 0) {
+    console.log(`\n${C.yellow}${C.bold}NETWORK ERRORS:${C.reset}`);
+    networkIssues.forEach(r => {
+      console.log(`  ${C.yellow}⚠${C.reset} ${r.name}:`);
+      r.networkErrors.forEach(e => {
+        console.log(`    ${C.dim}[${e.error || 'unknown'}]${C.reset} ${e.url}`);
+      });
+    });
+  }
+
   if (screenshotsDir) {
     console.log(`\n${C.dim}Report: ${path.join(screenshotsDir, 'report.json')}${C.reset}`);
     console.log(`${C.dim}Screenshots: ${screenshotsDir}${C.reset}\n`);

package/src/runner.js

@@ -208,6 +208,14 @@ export async function runTestsParallel(tests, config, suiteHooks = {}) {
         const attempts = result.maxAttempts > 1 ? ` (${result.maxAttempts} attempts)` : '';
         log('❌', `${C.red}${test.name}${C.reset}: ${result.error}${attempts}`);
       }
+
+      const consoleIssues = result.consoleLogs?.filter(l => l.type === 'error' || l.type === 'warning').length || 0;
+      if (consoleIssues > 0) {
+        log('⚠️', `${C.yellow}${test.name}: ${consoleIssues} console ${consoleIssues === 1 ? 'issue' : 'issues'}${C.reset}`);
+      }
+      if (result.networkErrors?.length > 0) {
+        log('⚠️', `${C.yellow}${test.name}: ${result.networkErrors.length} network ${result.networkErrors.length === 1 ? 'error' : 'errors'}${C.reset}`);
+      }
     }
   };