@polymorphism-tech/morph-spec 2.2.0 → 2.4.0

package/src/commands/doctor.js

@@ -1,280 +1,356 @@
-import { join } from 'path';
-import { execSync } from 'child_process';
-import { platform } from 'os';
-import chalk from 'chalk';
-import { logger } from '../utils/logger.js';
-import { pathExists, readJson } from '../utils/file-copier.js';
-import {
-  checkCLIOutdated,
-  checkProjectOutdated,
-  getInstalledCLIVersion
-} from '../utils/version-checker.js';
-
-const isWindows = platform() === 'win32';
-
-/**
- * Check if morph-spec command is in PATH
- */
-function isMorphSpecInPath() {
-  try {
-    const command = isWindows ? 'where morph-spec' : 'which morph-spec';
-    execSync(command, { stdio: 'ignore' });
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-/**
- * Detect if nvm-windows is installed
- */
-function isUsingNvmWindows() {
-  if (!isWindows) return false;
-
-  try {
-    const path = process.env.PATH || '';
-    return path.includes('nvm4w') || path.includes('\\nvm\\');
-  } catch {
-    return false;
-  }
-}
-
-/**
- * Get npm global prefix
- */
-function getNpmGlobalPrefix() {
-  try {
-    const prefix = execSync('npm config get prefix', { encoding: 'utf8' }).trim();
-    return prefix;
-  } catch {
-    return null;
-  }
-}
-
-export async function doctorCommand() {
-  const targetPath = process.cwd();
-
-  logger.header('MORPH-SPEC Health Check');
-
-  const checks = [];
-  let hasErrors = false;
-  let hasWarnings = false;
-
-  // Check versions first
-  const cliCheck = await checkCLIOutdated();
-  const projectCheck = await checkProjectOutdated(targetPath);
-
-  // CLI Version
-  if (cliCheck.latest) {
-    if (cliCheck.isOutdated) {
-      checks.push({
-        name: `CLI version (${cliCheck.current})`,
-        status: 'warn',
-        msg: `outdated, ${cliCheck.latest} available`
-      });
-      hasWarnings = true;
-    } else {
-      checks.push({
-        name: `CLI version (${cliCheck.current})`,
-        status: 'ok',
-        msg: 'latest'
-      });
-    }
-  } else {
-    checks.push({
-      name: `CLI version (${cliCheck.current})`,
-      status: 'ok'
-    });
-  }
-
-  // Project MORPH version
-  if (projectCheck.current) {
-    if (projectCheck.isOutdated) {
-      checks.push({
-        name: `Project MORPH version (${projectCheck.current})`,
-        status: 'warn',
-        msg: `outdated, ${projectCheck.cliVersion} available`
-      });
-      hasWarnings = true;
-    } else {
-      checks.push({
-        name: `Project MORPH version (${projectCheck.current})`,
-        status: 'ok'
-      });
-    }
-  } else {
-    checks.push({
-      name: 'Project MORPH version',
-      status: 'warn',
-      msg: 'not found (legacy installation)'
-    });
-    hasWarnings = true;
-  }
-
-  // Check if morph-spec is in PATH
-  const inPath = isMorphSpecInPath();
-  if (inPath) {
-    checks.push({
-      name: 'morph-spec in PATH',
-      status: 'ok'
-    });
-  } else {
-    checks.push({
-      name: 'morph-spec in PATH',
-      status: 'warn',
-      msg: 'command not found'
-    });
-    hasWarnings = true;
-  }
-
-  // Check CLAUDE.md
-  const claudeMd = join(targetPath, 'CLAUDE.md');
-  if (await pathExists(claudeMd)) {
-    checks.push({ name: 'CLAUDE.md', status: 'ok' });
-  } else {
-    checks.push({ name: 'CLAUDE.md', status: 'missing' });
-    hasErrors = true;
-  }
-
-  // Check .morph folder
-  const morphPath = join(targetPath, '.morph');
-  if (await pathExists(morphPath)) {
-    checks.push({ name: '.morph/', status: 'ok' });
-  } else {
-    checks.push({ name: '.morph/', status: 'missing' });
-    hasErrors = true;
-  }
-
-  // Check config.json
-  const configPath = join(morphPath, 'config', 'config.json');
-  if (await pathExists(configPath)) {
-    try {
-      const config = await readJson(configPath);
-      if (config.project?.name) {
-        checks.push({ name: 'config.json', status: 'ok' });
-      } else {
-        checks.push({ name: 'config.json', status: 'warn', msg: 'project.name not set' });
-      }
-    } catch {
-      checks.push({ name: 'config.json', status: 'error', msg: 'invalid JSON' });
-      hasErrors = true;
-    }
-  } else {
-    checks.push({ name: 'config.json', status: 'missing' });
-    hasErrors = true;
-  }
-
-  // Check agents.json
-  const agentsPath = join(morphPath, 'config', 'agents.json');
-  if (await pathExists(agentsPath)) {
-    checks.push({ name: 'agents.json', status: 'ok' });
-  } else {
-    checks.push({ name: 'agents.json', status: 'missing' });
-    hasErrors = true;
-  }
-
-  // Check standards
-  const standardsPath = join(morphPath, 'standards');
-  if (await pathExists(standardsPath)) {
-    const codingStd = join(standardsPath, 'coding.md');
-    const archStd = join(standardsPath, 'architecture.md');
-    const azureStd = join(standardsPath, 'azure.md');
-
-    const hasAll = await Promise.all([
-      pathExists(codingStd),
-      pathExists(archStd),
-      pathExists(azureStd)
-    ]).then(results => results.every(Boolean));
-
-    if (hasAll) {
-      checks.push({ name: 'standards/', status: 'ok' });
-    } else {
-      checks.push({ name: 'standards/', status: 'warn', msg: 'some files missing' });
-    }
-  } else {
-    checks.push({ name: 'standards/', status: 'missing' });
-    hasErrors = true;
-  }
-
-  // Check templates
-  const templatesPath = join(morphPath, 'templates');
-  if (await pathExists(templatesPath)) {
-    checks.push({ name: 'templates/', status: 'ok' });
-  } else {
-    checks.push({ name: 'templates/', status: 'missing' });
-    hasErrors = true;
-  }
-
-  // Check .claude folder
-  const claudePath = join(targetPath, '.claude');
-  if (await pathExists(claudePath)) {
-    const commandsPath = join(claudePath, 'commands');
-    const skillsPath = join(claudePath, 'skills');
-
-    const hasCommands = await pathExists(commandsPath);
-    const hasSkills = await pathExists(skillsPath);
-
-    if (hasCommands && hasSkills) {
-      checks.push({ name: '.claude/', status: 'ok' });
-    } else {
-      checks.push({ name: '.claude/', status: 'warn', msg: 'incomplete structure' });
-    }
-  } else {
-    checks.push({ name: '.claude/', status: 'missing' });
-    hasErrors = true;
-  }
-
-  // Display results
-  for (const check of checks) {
-    if (check.status === 'ok') {
-      console.log(chalk.green(`  ✓ ${check.name}`));
-    } else if (check.status === 'warn') {
-      console.log(chalk.yellow(`  ⚠ ${check.name}`) + chalk.gray(` (${check.msg})`));
-    } else if (check.status === 'missing') {
-      console.log(chalk.red(`  ✗ ${check.name}`) + chalk.gray(' (missing)'));
-    } else if (check.status === 'error') {
-      console.log(chalk.red(`  ✗ ${check.name}`) + chalk.gray(` (${check.msg})`));
-    }
-  }
-
-  logger.blank();
-
-  if (hasErrors) {
-    logger.error('Some checks failed. Run "morph-spec init --force" to fix.');
-    process.exit(1);
-  } else if (hasWarnings) {
-    logger.warn('Some checks need attention.');
-    logger.blank();
-
-    if (!inPath) {
-      const usingNvm = isUsingNvmWindows();
-      const prefix = getNpmGlobalPrefix();
-
-      if (usingNvm && prefix) {
-        logger.info('PATH Issue Detected (nvm-windows):');
-        logger.dim(`  Add to PATH: ${prefix}`);
-        logger.blank();
-        logger.dim('  PowerShell (as Administrator):');
-        console.log(chalk.gray(`    [Environment]::SetEnvironmentVariable("Path", [Environment]::GetEnvironmentVariable("Path", "User") + ";${prefix}", "User")`));
-        logger.blank();
-        logger.dim('  Or use npx instead:');
-        logger.dim('    npx @polymorphism-tech/morph-spec init');
-        logger.blank();
-      } else if (!inPath) {
-        logger.info('morph-spec not in PATH:');
-        logger.dim('  Use npx instead:');
-        logger.dim('    npx @polymorphism-tech/morph-spec init');
-        logger.blank();
-      }
-    }
-
-    if (cliCheck.isOutdated || projectCheck.isOutdated) {
-      logger.info('To update:');
-      logger.dim('  1. Update the CLI: npm install -g @polymorphism-tech/morph-spec@latest');
-      logger.dim('  2. Update the project: morph-spec update');
-      logger.blank();
-    }
-  } else {
-    logger.success('All checks passed!');
-  }
-}
+import { join } from 'path';
+import { execSync } from 'child_process';
+import { platform } from 'os';
+import fs from 'fs-extra';
+import chalk from 'chalk';
+import { logger } from '../utils/logger.js';
+import { pathExists, readJson } from '../utils/file-copier.js';
+import {
+  checkCLIOutdated,
+  checkProjectOutdated,
+  getInstalledCLIVersion
+} from '../utils/version-checker.js';
+
+const isWindows = platform() === 'win32';
+
+/**
+ * Check if morph-spec command is in PATH
+ */
+function isMorphSpecInPath() {
+  try {
+    const command = isWindows ? 'where morph-spec' : 'which morph-spec';
+    execSync(command, { stdio: 'ignore' });
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+/**
+ * Detect if nvm-windows is installed
+ */
+function isUsingNvmWindows() {
+  if (!isWindows) return false;
+
+  try {
+    const path = process.env.PATH || '';
+    return path.includes('nvm4w') || path.includes('\\nvm\\');
+  } catch {
+    return false;
+  }
+}
+
+/**
+ * Get npm global prefix
+ */
+function getNpmGlobalPrefix() {
+  try {
+    const prefix = execSync('npm config get prefix', { encoding: 'utf8' }).trim();
+    return prefix;
+  } catch {
+    return null;
+  }
+}
+
+export async function doctorCommand() {
+  const targetPath = process.cwd();
+
+  logger.header('MORPH-SPEC Health Check');
+
+  const checks = [];
+  let hasErrors = false;
+  let hasWarnings = false;
+
+  // Check versions first
+  const cliCheck = await checkCLIOutdated();
+  const projectCheck = await checkProjectOutdated(targetPath);
+
+  // CLI Version
+  if (cliCheck.latest) {
+    if (cliCheck.isOutdated) {
+      checks.push({
+        name: `CLI version (${cliCheck.current})`,
+        status: 'warn',
+        msg: `outdated, ${cliCheck.latest} available`
+      });
+      hasWarnings = true;
+    } else {
+      checks.push({
+        name: `CLI version (${cliCheck.current})`,
+        status: 'ok',
+        msg: 'latest'
+      });
+    }
+  } else {
+    checks.push({
+      name: `CLI version (${cliCheck.current})`,
+      status: 'ok'
+    });
+  }
+
+  // Project MORPH version
+  if (projectCheck.current) {
+    if (projectCheck.isOutdated) {
+      checks.push({
+        name: `Project MORPH version (${projectCheck.current})`,
+        status: 'warn',
+        msg: `outdated, ${projectCheck.cliVersion} available`
+      });
+      hasWarnings = true;
+    } else {
+      checks.push({
+        name: `Project MORPH version (${projectCheck.current})`,
+        status: 'ok'
+      });
+    }
+  } else {
+    checks.push({
+      name: 'Project MORPH version',
+      status: 'warn',
+      msg: 'not found (legacy installation)'
+    });
+    hasWarnings = true;
+  }
+
+  // Check if morph-spec is in PATH
+  const inPath = isMorphSpecInPath();
+  if (inPath) {
+    checks.push({
+      name: 'morph-spec in PATH',
+      status: 'ok'
+    });
+  } else {
+    checks.push({
+      name: 'morph-spec in PATH',
+      status: 'warn',
+      msg: 'command not found'
+    });
+    hasWarnings = true;
+  }
+
+  // Check CLAUDE.md
+  const claudeMd = join(targetPath, 'CLAUDE.md');
+  if (await pathExists(claudeMd)) {
+    checks.push({ name: 'CLAUDE.md', status: 'ok' });
+  } else {
+    checks.push({ name: 'CLAUDE.md', status: 'missing' });
+    hasErrors = true;
+  }
+
+  // Check .morph folder
+  const morphPath = join(targetPath, '.morph');
+  if (await pathExists(morphPath)) {
+    checks.push({ name: '.morph/', status: 'ok' });
+  } else {
+    checks.push({ name: '.morph/', status: 'missing' });
+    hasErrors = true;
+  }
+
+  // Check config.json
+  const configPath = join(morphPath, 'config', 'config.json');
+  if (await pathExists(configPath)) {
+    try {
+      const config = await readJson(configPath);
+      if (config.project?.name) {
+        checks.push({ name: 'config.json', status: 'ok' });
+      } else {
+        checks.push({ name: 'config.json', status: 'warn', msg: 'project.name not set' });
+      }
+    } catch {
+      checks.push({ name: 'config.json', status: 'error', msg: 'invalid JSON' });
+      hasErrors = true;
+    }
+  } else {
+    checks.push({ name: 'config.json', status: 'missing' });
+    hasErrors = true;
+  }
+
+  // Check agents.json
+  const agentsPath = join(morphPath, 'config', 'agents.json');
+  if (await pathExists(agentsPath)) {
+    checks.push({ name: 'agents.json', status: 'ok' });
+  } else {
+    checks.push({ name: 'agents.json', status: 'missing' });
+    hasErrors = true;
+  }
+
+  // Check standards
+  const standardsPath = join(morphPath, 'standards');
+  if (await pathExists(standardsPath)) {
+    const codingStd = join(standardsPath, 'coding.md');
+    const archStd = join(standardsPath, 'architecture.md');
+    const azureStd = join(standardsPath, 'azure.md');
+
+    const hasAll = await Promise.all([
+      pathExists(codingStd),
+      pathExists(archStd),
+      pathExists(azureStd)
+    ]).then(results => results.every(Boolean));
+
+    if (hasAll) {
+      checks.push({ name: 'standards/', status: 'ok' });
+    } else {
+      checks.push({ name: 'standards/', status: 'warn', msg: 'some files missing' });
+    }
+  } else {
+    checks.push({ name: 'standards/', status: 'missing' });
+    hasErrors = true;
+  }
+
+  // Check templates
+  const templatesPath = join(morphPath, 'templates');
+  if (await pathExists(templatesPath)) {
+    checks.push({ name: 'templates/', status: 'ok' });
+  } else {
+    checks.push({ name: 'templates/', status: 'missing' });
+    hasErrors = true;
+  }
+
+  // Check .claude folder
+  const claudePath = join(targetPath, '.claude');
+  if (await pathExists(claudePath)) {
+    const commandsPath = join(claudePath, 'commands');
+    const skillsPath = join(claudePath, 'skills');
+
+    const hasCommands = await pathExists(commandsPath);
+    const hasSkills = await pathExists(skillsPath);
+
+    if (hasCommands && hasSkills) {
+      checks.push({ name: '.claude/', status: 'ok' });
+
+      // Check skills link status
+      const skillEntries = await fs.readdir(skillsPath, { withFileTypes: true });
+      const skillDirs = skillEntries.filter(e => e.isDirectory() || e.isSymbolicLink());
+      let linkedCount = 0;
+
+      for (const dir of skillDirs) {
+        try {
+          const stat = await fs.lstat(join(skillsPath, dir.name));
+          if (stat.isSymbolicLink()) linkedCount++;
+        } catch { /* ignore */ }
+      }
+
+      if (skillDirs.length > 0) {
+        if (linkedCount === skillDirs.length) {
+          checks.push({ name: '.claude/skills/ links', status: 'ok', msg: `${linkedCount} categories linked` });
+        } else if (linkedCount > 0) {
+          checks.push({ name: '.claude/skills/ links', status: 'warn', msg: `${linkedCount}/${skillDirs.length} linked, rest copied` });
+          hasWarnings = true;
+        } else {
+          checks.push({ name: '.claude/skills/ links', status: 'warn', msg: 'all categories copied (no auto-update)' });
+          hasWarnings = true;
+        }
+      }
+    } else {
+      checks.push({ name: '.claude/', status: 'warn', msg: 'incomplete structure' });
+    }
+  } else {
+    checks.push({ name: '.claude/', status: 'missing' });
+    hasErrors = true;
+  }
+
+  // Check state.json
+  const statePath = join(targetPath, '.morph', 'state.json');
+  if (await pathExists(statePath)) {
+    try {
+      const stateContent = await readJson(statePath);
+      if (stateContent.version && stateContent.features) {
+        const featureCount = Object.keys(stateContent.features).length;
+        checks.push({ name: 'state.json', status: 'ok', msg: `${featureCount} feature(s)` });
+      } else {
+        checks.push({ name: 'state.json', status: 'warn', msg: 'invalid schema' });
+        hasWarnings = true;
+      }
+    } catch {
+      checks.push({ name: 'state.json', status: 'error', msg: 'invalid JSON' });
+      hasErrors = true;
+    }
+  } else {
+    checks.push({ name: 'state.json', status: 'warn', msg: 'not initialized (run: morph-spec state init)' });
+    hasWarnings = true;
+  }
+
+  // Check key templates exist
+  const keyTemplates = ['proposal.md', 'spec.md', 'tasks.md', 'decisions.md'];
+  const missingTemplates = [];
+  for (const template of keyTemplates) {
+    const templatePath = join(templatesPath, template);
+    if (!(await pathExists(templatePath))) {
+      missingTemplates.push(template);
+    }
+  }
+
+  if (missingTemplates.length === 0) {
+    checks.push({ name: 'key templates', status: 'ok' });
+  } else if (missingTemplates.length < keyTemplates.length) {
+    checks.push({ name: 'key templates', status: 'warn', msg: `missing: ${missingTemplates.join(', ')}` });
+    hasWarnings = true;
+  } else {
+    checks.push({ name: 'key templates', status: 'missing' });
+    hasErrors = true;
+  }
+
+  // Check Node.js version
+  const nodeVersion = process.version;
+  const majorVersion = parseInt(nodeVersion.slice(1).split('.')[0], 10);
+  if (majorVersion >= 18) {
+    checks.push({ name: `Node.js ${nodeVersion}`, status: 'ok' });
+  } else {
+    checks.push({ name: `Node.js ${nodeVersion}`, status: 'warn', msg: 'v18+ recommended' });
+    hasWarnings = true;
+  }
+
+  // Display results
+  for (const check of checks) {
+    if (check.status === 'ok') {
+      console.log(chalk.green(`  ✓ ${check.name}`));
+    } else if (check.status === 'warn') {
+      console.log(chalk.yellow(`  ⚠ ${check.name}`) + chalk.gray(` (${check.msg})`));
+    } else if (check.status === 'missing') {
+      console.log(chalk.red(`  ✗ ${check.name}`) + chalk.gray(' (missing)'));
+    } else if (check.status === 'error') {
+      console.log(chalk.red(`  ✗ ${check.name}`) + chalk.gray(` (${check.msg})`));
+    }
+  }
+
+  logger.blank();
+
+  if (hasErrors) {
+    logger.error('Some checks failed. Run "morph-spec init --force" to fix.');
+    process.exit(1);
+  } else if (hasWarnings) {
+    logger.warn('Some checks need attention.');
+    logger.blank();
+
+    if (!inPath) {
+      const usingNvm = isUsingNvmWindows();
+      const prefix = getNpmGlobalPrefix();
+
+      if (usingNvm && prefix) {
+        logger.info('PATH Issue Detected (nvm-windows):');
+        logger.dim(`  Add to PATH: ${prefix}`);
+        logger.blank();
+        logger.dim('  PowerShell (as Administrator):');
+        console.log(chalk.gray(`    [Environment]::SetEnvironmentVariable("Path", [Environment]::GetEnvironmentVariable("Path", "User") + ";${prefix}", "User")`));
+        logger.blank();
+        logger.dim('  Or use npx instead:');
+        logger.dim('    npx @polymorphism-tech/morph-spec init');
+        logger.blank();
+      } else if (!inPath) {
+        logger.info('morph-spec not in PATH:');
+        logger.dim('  Use npx instead:');
+        logger.dim('    npx @polymorphism-tech/morph-spec init');
+        logger.blank();
+      }
+    }
+
+    if (cliCheck.isOutdated || projectCheck.isOutdated) {
+      logger.info('To update:');
+      logger.dim('  1. Update the CLI: npm install -g @polymorphism-tech/morph-spec@latest');
+      logger.dim('  2. Update the project: morph-spec update');
+      logger.blank();
+    }
+  } else {
+    logger.success('All checks passed!');
+  }
+}