openspec-playwright 0.1.61 → 0.1.63

package/src/commands/init.ts

@@ -1,252 +0,0 @@
-import { execSync } from 'child_process';
-import {
-  existsSync,
-  readFileSync,
-  writeFileSync,
-  mkdirSync,
-} from 'fs';
-import { join } from 'path';
-import { homedir } from 'os';
-import { fileURLToPath } from 'url';
-import chalk from 'chalk';
-import { readFile } from 'fs/promises';
-import { syncMcpTools } from './mcpSync.js';
-import { detectEditors, detectCodex, installForAllEditors, installSkill, installProjectClaudeMd, readEmployeeStandards, claudeAdapter } from './editors.js';
-
-const TEMPLATE_DIR = fileURLToPath(new URL('../../templates', import.meta.url));
-const SCHEMA_DIR = fileURLToPath(new URL('../../schemas', import.meta.url));
-const SKILL_SRC = fileURLToPath(new URL('../../.claude/skills/openspec-e2e/SKILL.md', import.meta.url));
-const CMD_BODY_SRC = fileURLToPath(new URL('../../.claude/commands/opsx/e2e-body.md', import.meta.url));
-const EMPLOYEE_STANDARDS_SRC = fileURLToPath(new URL('../../employee-standards.md', import.meta.url));
-
-export interface InitOptions {
-  change?: string;
-  mcp?: boolean;
-  seed?: boolean;
-}
-
-export async function init(options: InitOptions) {
-  console.log(chalk.blue('\n🔧 OpenSpec + Playwright E2E Setup\n'));
-
-  const projectRoot = process.cwd();
-
-  // 1. Check prerequisites
-  console.log(chalk.blue('─── Prerequisites ───'));
-
-  const hasNode = execCmd('node --version', 'Node.js', true);
-  const hasNpm = execCmd('npm --version', 'npm', true);
-  const hasOpenspec = execCmd('npx openspec --version 2>/dev/null || echo "not found"', 'OpenSpec', true);
-
-  if (!hasNode || !hasNpm) {
-    console.log(chalk.red('  ✗ Node.js/npm is required'));
-    process.exit(1);
-  }
-  console.log(chalk.green('  ✓ Node.js and npm found'));
-
-  // 2. Check OpenSpec
-  if (!existsSync(join(projectRoot, 'openspec'))) {
-    console.log(chalk.yellow('\n⚠ OpenSpec not initialized. Run these commands first:'));
-    console.log(chalk.gray('  npm install -g @fission-ai/openspec'));
-    console.log(chalk.gray('  openspec init'));
-    console.log(chalk.gray('  openspec config profile core'));
-    console.log(chalk.gray('  openspec update\n'));
-    console.log(chalk.gray('  Then run openspec-pw init again.\n'));
-    return;
-  }
-  console.log(chalk.green('  ✓ OpenSpec initialized'));
-
-  // 3. Install Playwright MCP (global)
-  if (options.mcp !== false) {
-    console.log(chalk.blue('\n─── Installing Playwright MCP ───'));
-
-    // Check if playwright MCP already exists in global config
-    const claudeJsonPath = join(homedir(), '.claude.json');
-    const claudeJson = existsSync(claudeJsonPath) ? JSON.parse(readFileSync(claudeJsonPath, 'utf-8')) : {};
-    const globalMcp = claudeJson?.mcpServers ?? {};
-    const localMcp = claudeJson?.projects?.[projectRoot]?.mcpServers ?? {};
-
-    if (globalMcp['playwright'] || localMcp['playwright']) {
-      console.log(chalk.green('  ✓ Playwright MCP already installed'));
-    } else {
-      try {
-        execSync('claude mcp add playwright npx @playwright/mcp@latest', {
-          cwd: projectRoot,
-          stdio: 'inherit',
-        });
-        console.log(chalk.green('  ✓ Playwright MCP installed globally'));
-        console.log(chalk.gray('  (Restart Claude Code to activate)'));
-      } catch {
-        console.log(chalk.yellow('  ⚠ Failed to run claude mcp add'));
-        console.log(chalk.gray('  Run manually: claude mcp add playwright npx @playwright/mcp@latest'));
-        console.log(chalk.gray('  (Restart Claude Code to activate the MCP server)'));
-      }
-    }
-  }
-
-  // 4. Install E2E commands for detected editors
-  console.log(chalk.blue('\n─── Installing E2E Commands ───'));
-  const detected = detectEditors(projectRoot);
-  const codex = detectCodex();
-  const adapters = codex ? [...detected, codex] : detected;
-  if (adapters.length > 0) {
-    const body = await readFile(CMD_BODY_SRC, 'utf-8');
-    installForAllEditors(body, adapters, projectRoot);
-  } else {
-    const body = await readFile(CMD_BODY_SRC, 'utf-8');
-    installForAllEditors(body, [claudeAdapter], projectRoot);
-  }
-
-  // Claude Code also gets the SKILL.md
-  if (existsSync(join(projectRoot, '.claude'))) {
-    const skillContent = await readFile(SKILL_SRC, 'utf-8');
-    installSkill(projectRoot, skillContent);
-  }
-
-  // 5. Sync Healer tools with latest @playwright/mcp (Claude Code only)
-  if (existsSync(join(projectRoot, '.claude'))) {
-    console.log(chalk.blue('\n─── Syncing Healer Tools ───'));
-    const skillDest = join(projectRoot, '.claude', 'skills', 'openspec-e2e', 'SKILL.md');
-    await syncMcpTools(skillDest, true);
-  } else {
-    console.log(chalk.blue('\n─── Syncing Healer Tools ───'));
-    console.log(chalk.gray('  - Claude Code not detected, skipping MCP sync'));
-  }
-
-  // 6. Install OpenSpec schema
-  console.log(chalk.blue('\n─── Installing OpenSpec Schema ───'));
-  await installSchema(projectRoot);
-
-  // 7. Generate seed test
-  if (options.seed !== false) {
-    console.log(chalk.blue('\n─── Generating Seed Test ───'));
-    await generateSeedTest(projectRoot);
-  }
-
-  // 8. Generate app-knowledge.md
-  console.log(chalk.blue('\n─── Generating App Knowledge ───'));
-  await generateAppKnowledge(projectRoot);
-
-  // 9. Install employee-grade CLAUDE.md
-  console.log(chalk.blue('\n─── Installing Employee Standards ───'));
-  const standards = readEmployeeStandards(EMPLOYEE_STANDARDS_SRC);
-  if (standards) {
-    installProjectClaudeMd(projectRoot, standards);
-  }
-
-  // 10. Summary
-  console.log(chalk.blue('\n─── Summary ───'));
-  console.log(chalk.green('  ✓ Setup complete!\n'));
-
-  console.log(chalk.bold('Next steps:'));
-  console.log(chalk.gray('  1. Install Playwright browsers: npx playwright install --with-deps'));
-  console.log(chalk.gray('  2. Customize tests/playwright/credentials.yaml with your test user'));
-  console.log(chalk.gray('  3. Set credentials: export E2E_USERNAME=xxx E2E_PASSWORD=yyy'));
-  console.log(chalk.gray('  4. Run auth setup: npx playwright test --project=setup'));
-  const hasClaude = existsSync(join(projectRoot, '.claude'));
-  if (hasClaude) {
-    console.log(chalk.gray('  5. In Claude Code, run: /opsx:e2e <change-name>'));
-  }
-  console.log(chalk.gray(`  ${hasClaude ? '6.' : '5.'} Or: openspec-pw run <change-name>`));
-  console.log(chalk.gray(`  ${hasClaude ? '7.' : '6.'} Or: openspec-pw doctor to verify setup\n`));
-
-  console.log(chalk.bold('How it works:'));
-  console.log(chalk.gray('  /opsx:e2e reads your OpenSpec specs and runs Playwright'));
-  console.log(chalk.gray('  E2E tests through a three-agent pipeline:'));
-  console.log(chalk.gray('  Planner → Generator → Healer\n'));
-}
-
-async function generateSeedTest(projectRoot: string) {
-  const testsDir = join(projectRoot, 'tests', 'playwright');
-  mkdirSync(testsDir, { recursive: true });
-
-  const seedPath = join(testsDir, 'seed.spec.ts');
-  if (existsSync(seedPath)) {
-    console.log(chalk.gray('  - seed.spec.ts already exists, skipping'));
-  } else {
-    const seedContent = await readFile(TEMPLATE_DIR + '/seed.spec.ts', 'utf-8');
-    writeFileSync(seedPath, seedContent);
-    console.log(chalk.green('  ✓ Generated: tests/playwright/seed.spec.ts'));
-  }
-
-  // Generate auth.setup.ts
-  const authSetupPath = join(testsDir, 'auth.setup.ts');
-  if (existsSync(authSetupPath)) {
-    console.log(chalk.gray('  - auth.setup.ts already exists, skipping'));
-  } else {
-    const authContent = await readFile(TEMPLATE_DIR + '/auth.setup.ts', 'utf-8');
-    writeFileSync(authSetupPath, authContent);
-    console.log(chalk.green('  ✓ Generated: tests/playwright/auth.setup.ts'));
-  }
-
-  // Generate credentials.yaml
-  const credsPath = join(testsDir, 'credentials.yaml');
-  if (existsSync(credsPath)) {
-    console.log(chalk.gray('  - credentials.yaml already exists, skipping'));
-  } else {
-    const credsContent = await readFile(TEMPLATE_DIR + '/credentials.yaml', 'utf-8');
-    writeFileSync(credsPath, credsContent);
-    console.log(chalk.green('  ✓ Generated: tests/playwright/credentials.yaml'));
-  }
-
-  console.log(chalk.gray('  (Customize BASE_URL and credentials for your app)'));
-}
-
-async function generateAppKnowledge(projectRoot: string) {
-  const src = join(SCHEMA_DIR, 'playwright-e2e', 'templates', 'app-knowledge.md');
-  const dest = join(projectRoot, 'tests', 'playwright', 'app-knowledge.md');
-
-  if (existsSync(dest)) {
-    console.log(chalk.gray('  - app-knowledge.md already exists, skipping'));
-    return;
-  }
-
-  if (existsSync(src)) {
-    writeFileSync(dest, readFileSync(src));
-    console.log(chalk.green('  ✓ Generated: tests/playwright/app-knowledge.md'));
-  }
-}
-
-async function installSchema(projectRoot: string) {
-  const schemaSrc = SCHEMA_DIR + '/playwright-e2e';
-  const schemaDest = join(projectRoot, 'openspec', 'schemas', 'playwright-e2e');
-  const schemaFiles = ['schema.yaml'];
-
-  mkdirSync(schemaDest, { recursive: true });
-  for (const file of schemaFiles) {
-    const src = join(schemaSrc, file);
-    const dest = join(schemaDest, file);
-    if (existsSync(src)) {
-      writeFileSync(dest, readFileSync(src));
-    }
-  }
-
-  // Copy templates
-  const templatesSrc = join(schemaSrc, 'templates');
-  const templatesDest = join(schemaDest, 'templates');
-  mkdirSync(templatesDest, { recursive: true });
-  const templateFiles = ['test-plan.md', 'report.md', 'e2e-test.ts', 'playwright.config.ts', 'app-knowledge.md'];
-  for (const file of templateFiles) {
-    const src = join(templatesSrc, file);
-    const dest = join(templatesDest, file);
-    if (existsSync(src)) {
-      writeFileSync(dest, readFileSync(src));
-    }
-  }
-
-  console.log(chalk.green('  ✓ Schema installed: openspec/schemas/playwright-e2e/'));
-}
-
-function execCmd(
-  cmd: string,
-  name: string,
-  silent = false
-): boolean {
-  try {
-    execSync(cmd, { stdio: 'pipe' });
-    if (!silent) console.log(chalk.green(`  ✓ ${name} found`));
-    return true;
-  } catch {
-    if (!silent) console.log(chalk.yellow(`  ⚠ ${name} not found`));
-    return false;
-  }
-}

package/src/commands/mcpSync.ts

@@ -1,160 +0,0 @@
-import { exec } from 'child_process';
-import { existsSync, readdirSync, readFileSync, writeFileSync, mkdirSync, rmSync } from 'fs';
-import { join } from 'path';
-import { tmpdir } from 'os';
-import { promisify } from 'util';
-import chalk from 'chalk';
-import * as tar from 'tar';
-
-export const MCP_VERSION_MARKER = '<!-- MCP_VERSION:';
-
-export const DEFAULT_HEALER_TOOLS = [
-  { name: 'browser_navigate', purpose: "Go to the failing test's page" },
-  { name: 'browser_snapshot', purpose: 'Get page structure to find equivalent selectors' },
-  { name: 'browser_console_messages', purpose: 'Diagnose JS errors that may cause failures' },
-  { name: 'browser_take_screenshot', purpose: 'Visually compare before/after fixes' },
-  { name: 'browser_run_code', purpose: 'Execute custom fix logic (optional)' },
-];
-
-/** Extract MCP version from SKILL.md marker */
-export function getStoredMcpVersion(skillContent: string): string | null {
-  const idx = skillContent.indexOf(MCP_VERSION_MARKER);
-  if (idx === -1) return null;
-  const end = skillContent.indexOf(' -->', idx);
-  return skillContent.slice(idx + MCP_VERSION_MARKER.length, end).trim();
-}
-
-/** Remove all existing MCP_VERSION comment lines from content */
-function removeMcpVersionMarkers(content: string): string {
-  return content
-    .split('\n')
-    .filter(line => !line.trim().startsWith(MCP_VERSION_MARKER))
-    .join('\n');
-}
-
-/** Build the Healer tools table markdown */
-function buildHealerTable(version: string, tools: Array<{ name: string; purpose: string }>): string {
-  const rows = tools.map(t => `| \`${t.name}\` | ${t.purpose} |`).join('\n');
-  return `${MCP_VERSION_MARKER} ${version} -->\n\n| Tool | Purpose |\n|------|---------|\n${rows}`;
-}
-
-/** Replace the Healer tools table in SKILL.md */
-export function updateHealerTable(
-  skillContent: string,
-  version: string,
-  tools: Array<{ name: string; purpose: string }>
-): string {
-  const noMarkers = removeMcpVersionMarkers(skillContent);
-  const start = noMarkers.indexOf('| Tool | Purpose |');
-  if (start === -1) return skillContent;
-  let end = noMarkers.indexOf('\n\n', start);
-  if (end === -1) end = noMarkers.length;
-
-  const before = noMarkers.slice(0, start);
-  const after = noMarkers.slice(end);
-  return before + buildHealerTable(version, tools) + after;
-}
-
-/** Fetch latest @playwright/mcp version from npm registry */
-export function getLatestMcpVersion(): Promise<string | null> {
-  return new Promise((resolve) => {
-    exec('npm show @playwright/mcp version --json', { timeout: 15000 }, (err, stdout) => {
-      if (err) { resolve(null); return; }
-      try { resolve(JSON.parse(stdout.trim())); } catch { resolve(null); }
-    });
-  });
-}
-
-const execAsync = promisify(exec);
-
-/** Extract a .tgz tarball to a destination directory (cross-platform) */
-async function extractTarball(tarballPath: string, destDir: string): Promise<void> {
-  rmSync(destDir, { recursive: true, force: true });
-  mkdirSync(destDir, { recursive: true });
-  await tar.extract({ file: tarballPath, cwd: destDir, strip: 1 });
-}
-
-/** Parse README markdown to extract browser_* tool entries */
-function parseMcpReadme(content: string): Array<{ name: string; purpose: string }> {
-  const tools: Array<{ name: string; purpose: string }> = [];
-  const re = /-\s+\*\*`?([^`*\n]+)`?\*\*\s*-\s*Title:\s*([^\n]+)/g;
-  let m;
-  while ((m = re.exec(content)) !== null) {
-    const name = m[1].trim();
-    if (name.startsWith('browser_')) {
-      const purpose = m[2].trim().replace(/\.$/, '');
-      tools.push({ name, purpose });
-    }
-  }
-  return tools;
-}
-
-/**
- * Fetch @playwright/mcp tools from npm package.
- * Downloads the tarball, extracts README, parses tool names.
- */
-export async function fetchMcpTools(version: string): Promise<Array<{ name: string; purpose: string }>> {
-  const tmpDir = join(tmpdir(), `openspec-pw-mcp-${version}`);
-  try {
-    await execAsync(
-      `npm pack @playwright/mcp@${version} --pack-destination ${tmpDir}`,
-      { timeout: 30000 }
-    );
-    const tgzFiles = readdirSync(tmpDir).filter(f => f.startsWith('playwright-mcp-') && f.endsWith('.tgz'));
-    if (tgzFiles.length === 0) return [];
-    const tarballPath = join(tmpDir, tgzFiles[0]);
-    const extractDir = join(tmpDir, 'pkg');
-    await extractTarball(tarballPath, extractDir);
-    const readmePath = join(extractDir, 'README.md');
-    const content = existsSync(readmePath) ? readFileSync(readmePath, 'utf-8') : '';
-    rmSync(tmpDir, { recursive: true, force: true });
-    return parseMcpReadme(content);
-  } catch {
-    return [];
-  }
-}
-
-/**
- * Sync Healer tools table in SKILL.md with latest @playwright/mcp.
- * Returns true if updated, false if already current or failed.
- */
-export async function syncMcpTools(
-  skillDest: string,
-  verbose = false
-): Promise<boolean> {
-  const latestVersion = await getLatestMcpVersion();
-  if (!latestVersion) {
-    if (verbose) console.log(chalk.yellow('  ⚠ Could not fetch latest @playwright/mcp version'));
-    return false;
-  }
-
-  if (!existsSync(skillDest)) {
-    if (verbose) console.log(chalk.gray('  - SKILL.md not found, skipping MCP sync'));
-    return false;
-  }
-
-  const skillContent = readFileSync(skillDest, 'utf-8');
-  const storedVersion = getStoredMcpVersion(skillContent);
-
-  if (storedVersion === latestVersion) {
-    if (verbose) console.log(chalk.gray(`  - Healer tools current (${latestVersion})`));
-    return false;
-  }
-
-  if (verbose) console.log(chalk.blue(`  - Updating from ${storedVersion ?? 'unknown'} → ${latestVersion}`));
-
-  const tools = await fetchMcpTools(latestVersion);
-  const toolSet = tools.length > 0 ? tools : DEFAULT_HEALER_TOOLS;
-
-  const updated = updateHealerTable(skillContent, latestVersion, toolSet);
-  writeFileSync(skillDest, updated);
-
-  if (verbose) {
-    if (tools.length > 0) {
-      console.log(chalk.green(`  ✓ Healer tools synced to ${latestVersion} (${tools.length} tools)`));
-    } else {
-      console.log(chalk.green(`  ✓ Healer tools synced to ${latestVersion} (default set)`));
-    }
-  }
-  return true;
-}

package/src/commands/run.ts

@@ -1,172 +0,0 @@
-import { execSync } from 'child_process';
-import { existsSync, mkdirSync, writeFileSync } from 'fs';
-import { join } from 'path';
-import chalk from 'chalk';
-
-export interface RunOptions {
-  project?: string;
-  timeout?: number;
-}
-
-const REPORTS_DIR = 'openspec/reports';
-
-export async function run(changeName: string, options: RunOptions) {
-  console.log(chalk.blue(`\n🔍 OpenSpec Playwright E2E: ${changeName}\n`));
-
-  const projectRoot = process.cwd();
-
-  // 1. Verify test file exists
-  const testFile = join(projectRoot, 'tests', 'playwright', `${changeName}.spec.ts`);
-  if (!existsSync(testFile)) {
-    console.log(chalk.red(`  ✗ Test file not found: tests/playwright/${changeName}.spec.ts`));
-    console.log(chalk.gray('  Run /opsx:e2e first to generate tests.\n'));
-    process.exit(1);
-  }
-
-  // 2. Setup reports dir
-  mkdirSync(join(projectRoot, REPORTS_DIR), { recursive: true });
-
-  // 3. Detect auth credentials
-  const credsPath = join(projectRoot, 'tests', 'playwright', 'credentials.yaml');
-  const hasCredentials = existsSync(credsPath);
-  if (!hasCredentials) {
-    console.log(chalk.yellow('  ⚠ No credentials.yaml found — tests may fail if auth required'));
-  }
-
-  // 4. Run Playwright tests with output capture
-  console.log(chalk.blue('─── Running Tests ───'));
-
-  const args = ['npx', 'playwright', 'test', testFile, '--reporter=list'];
-  if (options.project) {
-    args.push('--project=' + options.project);
-  }
-
-  let testOutput = '';
-  let exitCode = 0;
-
-  try {
-    // Capture stdout to detect port mismatch
-    const result = execSync(args.join(' '), {
-      cwd: projectRoot,
-      encoding: 'utf-8',
-      stdio: ['pipe', 'pipe', 'pipe'],
-      timeout: (options.timeout ?? 300) * 1000,
-    });
-    testOutput = result;
-  } catch (err: unknown) {
-    exitCode = 1;
-    const error = err as { stdout?: string; stderr?: string; status?: number };
-    testOutput = (error.stdout ?? '') + (error.stderr ?? '');
-  }
-
-  // 5. Parse results from Playwright output
-  const results = parsePlaywrightOutput(testOutput);
-
-  // 6. Detect port mismatch
-  if (testOutput.includes('net::ERR_CONNECTION_REFUSED') ||
-      testOutput.includes('listen EADDRINUSE') ||
-      testOutput.includes('0.0.0.0:')) {
-    console.log(chalk.yellow('  ⚠ Port mismatch detected. Check BASE_URL and webServer port.'));
-  }
-
-  // 7. Generate markdown report
-  const timestamp = new Date().toISOString().replace(/[:.]/g, '-').slice(0, 19);
-  const reportPath = join(projectRoot, REPORTS_DIR, `playwright-e2e-${changeName}-${timestamp}.md`);
-
-  const reportContent = generateReport(changeName, timestamp, results);
-  writeFileSync(reportPath, reportContent);
-
-  // 8. Summary
-  console.log(chalk.blue('\n─── Results ───'));
-  console.log(`  Tests: ${results.total}  ` +
-    chalk.green(`✓ ${results.passed}`) + '  ' +
-    (results.failed > 0 ? chalk.red(`✗ ${results.failed}`) : chalk.gray(`✗ ${results.failed}`)) +
-    `  Duration: ${results.duration}`);
-
-  console.log(chalk.blue('\n─── Report ───'));
-  console.log(chalk.green(`  ✓ ${reportPath}\n`));
-
-  if (results.failed > 0) {
-    console.log(chalk.red(`✗ E2E verification FAILED (${results.failed} tests)`));
-    process.exit(1);
-  } else {
-    console.log(chalk.green('✓ E2E verification PASSED'));
-  }
-}
-
-interface TestResults {
-  total: number;
-  passed: number;
-  failed: number;
-  duration: string;
-  tests: Array<{ name: string; status: 'passed' | 'failed' }>;
-}
-
-function parsePlaywrightOutput(output: string): TestResults {
-  const results: TestResults = { total: 0, passed: 0, failed: 0, duration: '0s', tests: [] };
-
-  // Parse: "✓ my-test (1.2s)" or "✗ my-test (0.5s)"
-  const testLineRegex = /([✓✗x]) (.+?) \((\d+(?:\.\d+)?[a-z]+)\)/g;
-  let match;
-  while ((match = testLineRegex.exec(output)) !== null) {
-    const status = match[1] === '✓' ? 'passed' : 'failed';
-    results.tests.push({ name: match[2], status });
-    results.total++;
-    if (status === 'passed') results.passed++;
-    else results.failed++;
-  }
-
-  // Parse duration: "N tests ran (1m 30s)" or "1 test ran (5s)"
-  const durationMatch = output.match(/\d+ tests? ran \((\d+(?:m\s*\d+)?s?)\)/);
-  if (durationMatch) results.duration = durationMatch[1];
-
-  return results;
-}
-
-function generateReport(changeName: string, timestamp: string, results: TestResults): string {
-  const lines: string[] = [
-    `# E2E Verify Report: ${changeName}`,
-    '',
-    `**Change**: \`${changeName}\``,
-    `**Generated**: ${timestamp.replace('T', ' ').slice(0, 16)} UTC`,
-    '',
-    '## Summary',
-    '',
-    '| Check | Status |',
-    '|-------|--------|',
-    `| Tests Run | ${results.total} |`,
-    `| Passed | ${results.passed} |`,
-    `| Failed | ${results.failed} |`,
-    `| Duration | ${results.duration} |`,
-    `| Final Status | ${results.failed === 0 ? '✅ PASS' : '❌ FAIL'} |`,
-    '',
-    '## Test Results',
-    '',
-  ];
-
-  if (results.tests.length === 0) {
-    lines.push('_(No test output captured — check Playwright configuration)_', '');
-  } else {
-    for (const test of results.tests) {
-      const icon = test.status === 'passed' ? '✅' : '❌';
-      lines.push(`- ${test.name}: ${icon} ${test.status}`);
-    }
-    lines.push('');
-  }
-
-  lines.push('## Recommendations', '');
-  if (results.failed > 0) {
-    lines.push(
-      'Review failed tests above. Common fixes:',
-      '- Update selectors if UI changed (use `data-testid` attributes)',
-      '- Adjust BASE_URL in seed.spec.ts if port differs',
-      '- Set E2E_USERNAME/E2E_PASSWORD if auth is required',
-      '- Check `npx playwright show-report` for screenshots',
-      ''
-    );
-  } else {
-    lines.push('All tests passed. No action needed.', '');
-  }
-
-  return lines.join('\n');
-}