@nerviq/cli 1.18.0 → 1.20.0

package/src/doctor.js

@@ -5,13 +5,13 @@
  * Checks: Node version, dependencies, platform detection, freshness gates.
  */
 
-'use strict';
-
-const fs = require('fs');
-const path = require('path');
-const { version } = require('../package.json');
-const { validateDeclaredMcpServers } = require('./mcp-validation');
-const { validateDeclaredHooks } = require('./hook-validation');
+'use strict';
+
+const fs = require('fs');
+const path = require('path');
+const { version } = require('../package.json');
+const { validateDeclaredMcpServers } = require('./mcp-validation');
+const { validateDeclaredHooks } = require('./hook-validation');
 
 const COLORS = {
   reset: '\x1b[0m',
@@ -27,11 +27,11 @@ function c(text, color) {
   return `${COLORS[color] || ''}${text}${COLORS.reset}`;
 }
 
-const PLATFORM_SIGNALS = {
-  claude:    ['CLAUDE.md', '.claude/CLAUDE.md', '.claude/settings.json', '.mcp.json'],
-  codex:     ['AGENTS.md', '.codex/', '.codex/config.toml'],
-  cursor:    ['.cursor/rules/', '.cursor/mcp.json', '.cursorrules'],
-  copilot:   ['.github/copilot-instructions.md', '.github/', '.vscode/mcp.json'],
+const PLATFORM_SIGNALS = {
+  claude:    ['CLAUDE.md', '.claude/CLAUDE.md', '.claude/settings.json', '.mcp.json'],
+  codex:     ['AGENTS.md', '.codex/', '.codex/config.toml'],
+  cursor:    ['.cursor/rules/', '.cursor/mcp.json', '.cursorrules'],
+  copilot:   ['.github/copilot-instructions.md', '.github/', '.vscode/mcp.json'],
   gemini:    ['GEMINI.md', '.gemini/', '.gemini/settings.json'],
   windsurf:  ['.windsurf/', '.windsurfrules', '.windsurf/rules/'],
   aider:     ['.aider.conf.yml', '.aider.model.settings.yml'],
@@ -164,59 +164,59 @@ function checkGitRepo(dir) {
 
 // ─── Main doctor function ────────────────────────────────────────────────────
 
-async function runDoctor({ dir = process.cwd(), json = false, verbose = false } = {}) {
-  const startMs = Date.now();
-
-  const checks = [
+async function runDoctor({ dir = process.cwd(), json = false, verbose = false } = {}) {
+  const startMs = Date.now();
+
+  const checks = [
     checkNodeVersion(),
     checkDeps(),
     checkJestInstalled(),
     checkCliPermissions(),
     checkGitRepo(dir),
     checkPlatformDetection(dir),
-  ];
-
-  const detectedPlatforms = (checks.find(c => c.detected) || {}).detected || [];
-  const freshnessChecks = checkFreshnessGates();
-  const mcpSummary = await validateDeclaredMcpServers({ dir, detectedPlatforms });
-  const hookSummary = validateDeclaredHooks({ dir, detectedPlatforms });
-
-  const totalPass = checks.filter(c => c.status === 'pass').length;
-  const totalWarn = checks.filter(c => c.status === 'warn').length;
-  const totalFail = checks.filter(c => c.status === 'fail').length;
-
-  const freshPass = freshnessChecks.filter(f => f.status === 'pass').length;
-  const freshWarn = freshnessChecks.filter(f => f.status !== 'pass').length;
-
-  const overallOk = totalFail === 0 && mcpSummary.fail === 0 && hookSummary.fail === 0;
-  const elapsed = Date.now() - startMs;
-
-  if (json) {
-    return JSON.stringify({
-      nerviq: version,
+  ];
+
+  const detectedPlatforms = (checks.find(c => c.detected) || {}).detected || [];
+  const freshnessChecks = checkFreshnessGates();
+  const mcpSummary = await validateDeclaredMcpServers({ dir, detectedPlatforms });
+  const hookSummary = validateDeclaredHooks({ dir, detectedPlatforms });
+
+  const totalPass = checks.filter(c => c.status === 'pass').length;
+  const totalWarn = checks.filter(c => c.status === 'warn').length;
+  const totalFail = checks.filter(c => c.status === 'fail').length;
+
+  const freshPass = freshnessChecks.filter(f => f.status === 'pass').length;
+  const freshWarn = freshnessChecks.filter(f => f.status !== 'pass').length;
+
+  const overallOk = totalFail === 0 && mcpSummary.fail === 0 && hookSummary.fail === 0;
+  const elapsed = Date.now() - startMs;
+
+  if (json) {
+    return JSON.stringify({
+      nerviq: version,
       node: process.version,
       dir,
       overallOk,
       checks,
-      freshnessChecks,
-      mcpChecks: mcpSummary.checks,
-      hookChecks: hookSummary.checks,
-      totalPass,
-      totalWarn,
-      totalFail,
-      freshPass,
-      freshWarn,
-      mcpDeclared: mcpSummary.declared,
-      mcpPass: mcpSummary.pass,
-      mcpWarn: mcpSummary.warn,
-      mcpFail: mcpSummary.fail,
-      hookDeclared: hookSummary.declared,
-      hookPass: hookSummary.pass,
-      hookWarn: hookSummary.warn,
-      hookFail: hookSummary.fail,
-      elapsed,
-    }, null, 2);
-  }
+      freshnessChecks,
+      mcpChecks: mcpSummary.checks,
+      hookChecks: hookSummary.checks,
+      totalPass,
+      totalWarn,
+      totalFail,
+      freshPass,
+      freshWarn,
+      mcpDeclared: mcpSummary.declared,
+      mcpPass: mcpSummary.pass,
+      mcpWarn: mcpSummary.warn,
+      mcpFail: mcpSummary.fail,
+      hookDeclared: hookSummary.declared,
+      hookPass: hookSummary.pass,
+      hookWarn: hookSummary.warn,
+      hookFail: hookSummary.fail,
+      elapsed,
+    }, null, 2);
+  }
 
   const lines = [''];
   lines.push(c(`  nerviq doctor  v${version}`, 'bold'));
@@ -234,9 +234,9 @@ async function runDoctor({ dir = process.cwd(), json = false, verbose = false }
   }
 
   // Platform detection detail
-  if (detectedPlatforms.length > 0) {
-    lines.push('');
-    lines.push(c('  Detected Platforms', 'bold'));
+  if (detectedPlatforms.length > 0) {
+    lines.push('');
+    lines.push(c('  Detected Platforms', 'bold'));
     for (const p of detectedPlatforms) {
       lines.push(`    ${c('✓', 'green')}  ${p}`);
     }
@@ -245,67 +245,67 @@ async function runDoctor({ dir = process.cwd(), json = false, verbose = false }
   // Freshness
   lines.push('');
   lines.push(c('  Freshness Gates', 'bold'));
-  for (const f of freshnessChecks) {
-    const icon = f.status === 'pass' ? c('✓', 'green') : c('⚠', 'yellow');
-    const label = f.platform.padEnd(12);
-    lines.push(`    ${icon}  ${label}  ${c(f.detail || f.status, f.status === 'pass' ? 'dim' : 'yellow')}`);
-  }
-
-  lines.push('');
-  lines.push(c('  MCP Servers', 'bold'));
-  if (mcpSummary.checks.length === 0) {
-    lines.push(c('    No declared MCP servers found in the detected project surfaces.', 'dim'));
-  } else {
-    for (const item of mcpSummary.checks) {
-      const icon = item.status === 'pass'
-        ? c('✓', 'green')
-        : item.status === 'warn'
-          ? c('⚠', 'yellow')
-          : c('✗', 'red');
-      const label = `${item.platform}/${item.scope}`.padEnd(16);
-      lines.push(`    ${icon}  ${label} ${item.serverName}  ${c(item.detail, item.status === 'pass' ? 'dim' : item.status === 'warn' ? 'yellow' : 'red')}`);
-      if (verbose && item.source) {
-        lines.push(c(`         Source: ${item.source}`, 'dim'));
-      }
-      if (item.fix && (verbose || item.status === 'fail')) {
-        lines.push(c(`         Fix: ${item.fix}`, item.status === 'fail' ? 'yellow' : 'dim'));
-      }
-    }
-  }
-
-  lines.push('');
-  lines.push(c('  Hook Runtime', 'bold'));
-  if (hookSummary.checks.length === 0) {
-    lines.push(c('    No declared hooks found in the detected project surfaces.', 'dim'));
-  } else {
-    for (const item of hookSummary.checks) {
-      const icon = item.status === 'pass'
-        ? c('✓', 'green')
-        : item.status === 'warn'
-          ? c('⚠', 'yellow')
-          : c('✗', 'red');
-      const label = `${item.platform}/${item.validationMode}`.padEnd(16);
-      lines.push(`    ${icon}  ${label} ${item.label}  ${c(item.detail, item.status === 'pass' ? 'dim' : item.status === 'warn' ? 'yellow' : 'red')}`);
-      if (verbose && item.script) {
-        lines.push(c(`         Script: ${item.script}`, 'dim'));
-      }
-      if (verbose && item.executable) {
-        lines.push(c(`         Runtime: ${item.executable}`, 'dim'));
-      }
-      if (item.fix && (verbose || item.status === 'fail')) {
-        lines.push(c(`         Fix: ${item.fix}`, item.status === 'fail' ? 'yellow' : 'dim'));
-      }
-    }
-  }
-
-  lines.push('');
-  lines.push(c('  Summary', 'bold'));
-  lines.push(`    Checks:    ${c(String(totalPass), 'green')} pass  ${totalWarn > 0 ? c(String(totalWarn), 'yellow') + ' warn  ' : ''}${totalFail > 0 ? c(String(totalFail), 'red') + ' fail' : ''}`);
-  lines.push(`    Freshness: ${c(String(freshPass), 'green')} fresh  ${freshWarn > 0 ? c(String(freshWarn), 'yellow') + ' stale/unverified' : ''}`);
-  lines.push(`    MCP:       ${c(String(mcpSummary.pass), 'green')} pass  ${mcpSummary.warn > 0 ? c(String(mcpSummary.warn), 'yellow') + ' warn  ' : ''}${mcpSummary.fail > 0 ? c(String(mcpSummary.fail), 'red') + ' fail' : ''}${c(`(${mcpSummary.declared} declared)`, 'dim')}`);
-  lines.push(`    Hooks:     ${c(String(hookSummary.pass), 'green')} pass  ${hookSummary.warn > 0 ? c(String(hookSummary.warn), 'yellow') + ' warn  ' : ''}${hookSummary.fail > 0 ? c(String(hookSummary.fail), 'red') + ' fail' : ''}${c(`(${hookSummary.declared} declared)`, 'dim')}`);
-  lines.push(`    Status:    ${overallOk ? c('✓ Healthy', 'green') : c('✗ Issues found', 'red')}`);
-  lines.push(`    Duration:  ${elapsed}ms`);
+  for (const f of freshnessChecks) {
+    const icon = f.status === 'pass' ? c('✓', 'green') : c('⚠', 'yellow');
+    const label = f.platform.padEnd(12);
+    lines.push(`    ${icon}  ${label}  ${c(f.detail || f.status, f.status === 'pass' ? 'dim' : 'yellow')}`);
+  }
+
+  lines.push('');
+  lines.push(c('  MCP Servers', 'bold'));
+  if (mcpSummary.checks.length === 0) {
+    lines.push(c('    No declared MCP servers found in the detected project surfaces.', 'dim'));
+  } else {
+    for (const item of mcpSummary.checks) {
+      const icon = item.status === 'pass'
+        ? c('✓', 'green')
+        : item.status === 'warn'
+          ? c('⚠', 'yellow')
+          : c('✗', 'red');
+      const label = `${item.platform}/${item.scope}`.padEnd(16);
+      lines.push(`    ${icon}  ${label} ${item.serverName}  ${c(item.detail, item.status === 'pass' ? 'dim' : item.status === 'warn' ? 'yellow' : 'red')}`);
+      if (verbose && item.source) {
+        lines.push(c(`         Source: ${item.source}`, 'dim'));
+      }
+      if (item.fix && (verbose || item.status === 'fail')) {
+        lines.push(c(`         Fix: ${item.fix}`, item.status === 'fail' ? 'yellow' : 'dim'));
+      }
+    }
+  }
+
+  lines.push('');
+  lines.push(c('  Hook Runtime', 'bold'));
+  if (hookSummary.checks.length === 0) {
+    lines.push(c('    No declared hooks found in the detected project surfaces.', 'dim'));
+  } else {
+    for (const item of hookSummary.checks) {
+      const icon = item.status === 'pass'
+        ? c('✓', 'green')
+        : item.status === 'warn'
+          ? c('⚠', 'yellow')
+          : c('✗', 'red');
+      const label = `${item.platform}/${item.validationMode}`.padEnd(16);
+      lines.push(`    ${icon}  ${label} ${item.label}  ${c(item.detail, item.status === 'pass' ? 'dim' : item.status === 'warn' ? 'yellow' : 'red')}`);
+      if (verbose && item.script) {
+        lines.push(c(`         Script: ${item.script}`, 'dim'));
+      }
+      if (verbose && item.executable) {
+        lines.push(c(`         Runtime: ${item.executable}`, 'dim'));
+      }
+      if (item.fix && (verbose || item.status === 'fail')) {
+        lines.push(c(`         Fix: ${item.fix}`, item.status === 'fail' ? 'yellow' : 'dim'));
+      }
+    }
+  }
+
+  lines.push('');
+  lines.push(c('  Summary', 'bold'));
+  lines.push(`    Checks:    ${c(String(totalPass), 'green')} pass  ${totalWarn > 0 ? c(String(totalWarn), 'yellow') + ' warn  ' : ''}${totalFail > 0 ? c(String(totalFail), 'red') + ' fail' : ''}`);
+  lines.push(`    Freshness: ${c(String(freshPass), 'green')} fresh  ${freshWarn > 0 ? c(String(freshWarn), 'yellow') + ' stale/unverified' : ''}`);
+  lines.push(`    MCP:       ${c(String(mcpSummary.pass), 'green')} pass  ${mcpSummary.warn > 0 ? c(String(mcpSummary.warn), 'yellow') + ' warn  ' : ''}${mcpSummary.fail > 0 ? c(String(mcpSummary.fail), 'red') + ' fail' : ''}${c(`(${mcpSummary.declared} declared)`, 'dim')}`);
+  lines.push(`    Hooks:     ${c(String(hookSummary.pass), 'green')} pass  ${hookSummary.warn > 0 ? c(String(hookSummary.warn), 'yellow') + ' warn  ' : ''}${hookSummary.fail > 0 ? c(String(hookSummary.fail), 'red') + ' fail' : ''}${c(`(${hookSummary.declared} declared)`, 'dim')}`);
+  lines.push(`    Status:    ${overallOk ? c('✓ Healthy', 'green') : c('✗ Issues found', 'red')}`);
+  lines.push(`    Duration:  ${elapsed}ms`);
   lines.push('');
 
   if (!overallOk) {