agileflow 2.89.2 → 2.90.0

package/tools/cli/commands/doctor.js

@@ -21,6 +21,15 @@ const {
 const { IdeManager } = require('../installers/ide/manager');
 const { getCurrentVersion } = require('../lib/version-checker');
 const { ErrorHandler } = require('../lib/error-handler');
+const {
+  ErrorCodes,
+  getErrorCodeFromError,
+  getSuggestedFix,
+  isRecoverable,
+} = require('../../../lib/error-codes');
+const { safeDump } = require('../../../lib/yaml-utils');
+const { IdeRegistry } = require('../lib/ide-registry');
+const { formatKeyValue, formatList, isTTY } = require('../../../lib/table-formatter');
 
 const installer = new Installer();
 
@@ -88,6 +97,7 @@ module.exports = {
         issues++;
         repairs.push({
           type: 'missing-manifest',
+          errorCode: 'ENOENT',
           message: 'Recreate missing manifest.yaml',
           fix: async () => {
             info('Recreating manifest.yaml...');
@@ -95,7 +105,6 @@ module.exports = {
             const cfgDir = path.join(status.path, '_cfg');
             await fs.ensureDir(cfgDir);
 
-            const yaml = require('js-yaml');
             const manifest = {
               version: packageJson.version,
               installed_at: new Date().toISOString(),
@@ -107,7 +116,7 @@ module.exports = {
               docs_folder: 'docs',
             };
 
-            await fs.writeFile(manifestPath, yaml.dump(manifest), 'utf8');
+            await fs.writeFile(manifestPath, safeDump(manifest), 'utf8');
             success('Created manifest.yaml');
           },
         });
@@ -142,6 +151,7 @@ module.exports = {
           warnings++;
           repairs.push({
             type: 'invalid-file-index',
+            errorCode: 'EPARSE',
             message: 'Recreate files.json safe-update index',
             fix: async () => {
               await createProtectedFileIndex(status.path, fileIndexPath);
@@ -154,6 +164,7 @@ module.exports = {
         warnings++;
         repairs.push({
           type: 'missing-file-index',
+          errorCode: 'ENOENT',
           message: 'Create files.json safe-update index',
           fix: async () => {
             await createProtectedFileIndex(status.path, fileIndexPath);
@@ -196,6 +207,7 @@ module.exports = {
       if (missingCore) {
         repairs.push({
           type: 'missing-core',
+          errorCode: 'EEMPTYDIR',
           message: 'Reinstall missing core content',
           fix: async () => {
             info('Reinstalling core content...');
@@ -229,8 +241,8 @@ module.exports = {
         ideManager.setDocsFolder(status.docsFolder || 'docs');
 
         for (const ide of status.ides) {
-          const configPath = getIdeConfigPath(directory, ide);
-          const ideName = formatIdeName(ide);
+          const configPath = IdeRegistry.getConfigPath(ide, directory);
+          const ideName = IdeRegistry.getDisplayName(ide);
 
           if (await fs.pathExists(configPath)) {
             // Count files in config
@@ -241,6 +253,7 @@ module.exports = {
             warnings++;
             repairs.push({
               type: 'missing-ide-config',
+              errorCode: 'ENODIR',
               message: `Reinstall ${ideName} configuration`,
               fix: async () => {
                 info(`Reinstalling ${ideName} configuration...`);
@@ -254,19 +267,20 @@ module.exports = {
 
       // Check for orphaned configs
       console.log(chalk.bold('\nOrphan Check:'));
-      const allIdes = ['claude-code', 'cursor', 'windsurf'];
+      const allIdes = IdeRegistry.getAll();
       let orphansFound = false;
 
       for (const ide of allIdes) {
         if (!status.ides || !status.ides.includes(ide)) {
-          const configPath = getIdeConfigPath(directory, ide);
+          const configPath = IdeRegistry.getConfigPath(ide, directory);
           if (await fs.pathExists(configPath)) {
-            const ideName = formatIdeName(ide);
+            const ideName = IdeRegistry.getDisplayName(ide);
             warning(`${ideName}: Config exists but not in manifest`);
             orphansFound = true;
             warnings++;
             repairs.push({
               type: 'orphaned-config',
+              errorCode: 'ECONFLICT',
               message: `Remove orphaned ${ideName} configuration`,
               fix: async () => {
                 info(`Removing orphaned ${ideName} configuration...`);
@@ -291,7 +305,13 @@ module.exports = {
           try {
             await repair.fix();
           } catch (err) {
+            // Use error codes for better diagnosis
+            const codeData = getErrorCodeFromError(err);
             error(`Failed to ${repair.message.toLowerCase()}: ${err.message}`);
+            if (codeData.code !== 'EUNKNOWN') {
+              console.log(chalk.dim(`  Error code: ${codeData.code}`));
+              console.log(chalk.dim(`  Suggestion: ${codeData.suggestedFix}`));
+            }
           }
         }
 
@@ -300,6 +320,16 @@ module.exports = {
       } else if (repairs.length > 0 && !options.fix) {
         console.log();
         info(`Found ${repairs.length} fixable issue(s). Run with --fix to auto-repair.`);
+
+        // Show summary of fixable issues with error codes
+        console.log(chalk.bold('\nFixable Issues:'));
+        for (const repair of repairs) {
+          const errorCode = repair.errorCode || 'ECONFIG';
+          const codeData = ErrorCodes[errorCode] || ErrorCodes.ECONFIG;
+          console.log(
+            `  ${chalk.yellow('!')} ${repair.message} ${chalk.dim(`[${codeData.code}]`)}`
+          );
+        }
       }
 
       // Print summary
@@ -339,36 +369,6 @@ function compareVersions(a, b) {
   return 0;
 }
 
-/**
- * Get IDE config path
- * @param {string} projectDir - Project directory
- * @param {string} ide - IDE name
- * @returns {string}
- */
-function getIdeConfigPath(projectDir, ide) {
-  const paths = {
-    'claude-code': '.claude/commands/agileflow',
-    cursor: '.cursor/rules/agileflow',
-    windsurf: '.windsurf/workflows/agileflow',
-  };
-
-  return path.join(projectDir, paths[ide] || '');
-}
-
-/**
- * Format IDE name for display
- * @param {string} ide - IDE name
- * @returns {string}
- */
-function formatIdeName(ide) {
-  const names = {
-    'claude-code': 'Claude Code',
-    cursor: 'Cursor',
-    windsurf: 'Windsurf',
-  };
-
-  return names[ide] || ide;
-}
 
 /**
  * Count files in directory recursively
@@ -392,7 +392,7 @@ async function countFilesInDir(dirPath) {
 }
 
 /**
- * Print summary
+ * Print summary using formatKeyValue for consistent output
  * @param {number} issues - Issue count
  * @param {number} warnings - Warning count
  */
@@ -402,9 +402,17 @@ function printSummary(issues, warnings) {
   if (issues === 0 && warnings === 0) {
     console.log(chalk.green.bold('No issues found.\n'));
   } else if (issues === 0) {
-    console.log(chalk.yellow(`${warnings} warning(s), no critical issues.\n`));
+    console.log(formatKeyValue({
+      Warnings: chalk.yellow(warnings),
+      Issues: chalk.green('0'),
+    }, { separator: ':', alignValues: false }));
+    console.log();
   } else {
-    console.log(chalk.red(`${issues} issue(s), ${warnings} warning(s) found.\n`));
+    console.log(formatKeyValue({
+      Issues: chalk.red(issues),
+      Warnings: chalk.yellow(warnings),
+    }, { separator: ':', alignValues: false }));
+    console.log();
   }
 }
 

package/tools/cli/commands/list.js

@@ -7,12 +7,13 @@
 const chalk = require('chalk');
 const path = require('node:path');
 const fs = require('fs-extra');
-const yaml = require('js-yaml');
+const { safeLoad } = require('../../../lib/yaml-utils');
 const { Installer } = require('../installers/core/installer');
 const { displayLogo, displaySection, success, warning, info } = require('../lib/ui');
 const {
   parseFrontmatter: parseYamlFrontmatter,
 } = require('../../../scripts/lib/frontmatter-parser');
+const { formatList, formatKeyValue, formatHeader, isTTY } = require('../../../lib/table-formatter');
 
 const installer = new Installer();
 
@@ -233,7 +234,7 @@ async function listExperts(agileflowPath) {
 
     try {
       const content = await fs.readFile(expertiseFile, 'utf8');
-      const parsed = yaml.load(content);
+      const parsed = safeLoad(content);
 
       experts.push({
         name: entry.name,
@@ -300,73 +301,84 @@ function extractFirstLine(content) {
 }
 
 /**
- * Display compact output
+ * Display compact output using formatKeyValue
  */
 function displayCompact(result, showCommands, showAgents, showSkills, showExperts) {
+  const data = {};
+
   if (showCommands && result.commands?.length > 0) {
-    console.log(chalk.bold('Commands:'), result.commands.map(c => c.name).join(', '));
+    data.Commands = result.commands.map(c => c.name).join(', ');
   }
 
   if (showAgents && result.agents?.length > 0) {
-    console.log(chalk.bold('Agents:'), result.agents.map(a => a.name).join(', '));
+    data.Agents = result.agents.map(a => a.name).join(', ');
   }
 
   if (showSkills && result.skills?.length > 0) {
-    console.log(chalk.bold('Skills:'), result.skills.map(s => s.name).join(', '));
+    data.Skills = result.skills.map(s => s.name).join(', ');
   }
 
   if (showExperts && result.experts?.length > 0) {
-    console.log(chalk.bold('Experts:'), result.experts.map(e => e.name).join(', '));
+    data.Experts = result.experts.map(e => e.name).join(', ');
+  }
+
+  if (Object.keys(data).length > 0) {
+    console.log(formatKeyValue(data, { alignValues: false }));
   }
 }
 
 /**
- * Display full output with descriptions
+ * Display full output with descriptions using formatList
  */
 function displayFull(result, showCommands, showAgents, showSkills, showExperts) {
-  if (showCommands && result.commands?.length > 0) {
-    displaySection(`Commands (${result.commands.length})`);
+  const { BRAND_HEX } = require('../../../lib/colors');
 
-    for (const cmd of result.commands) {
-      console.log(chalk.hex('#e8683a')(`  ${cmd.name}`));
-      console.log(chalk.dim(`    ${cmd.description}`));
-    }
+  if (showCommands && result.commands?.length > 0) {
+    console.log(formatHeader(`Commands (${result.commands.length})`));
+    const items = result.commands.map(cmd => ({
+      text: `${chalk.hex(BRAND_HEX)(cmd.name)}\n      ${chalk.dim(cmd.description)}`,
+      status: 'active',
+    }));
+    console.log(formatList(items, { indent: '  ' }));
   }
 
   if (showAgents && result.agents?.length > 0) {
-    displaySection(`Agents (${result.agents.length})`);
-
-    for (const agent of result.agents) {
+    console.log(formatHeader(`Agents (${result.agents.length})`));
+    const items = result.agents.map(agent => {
       const modelBadge = agent.model !== 'default' ? chalk.dim(` [${agent.model}]`) : '';
-      console.log(chalk.hex('#e8683a')(`  ${agent.name}`) + modelBadge);
-      console.log(chalk.dim(`    ${agent.description}`));
-    }
+      return {
+        text: `${chalk.hex(BRAND_HEX)(agent.name)}${modelBadge}\n      ${chalk.dim(agent.description)}`,
+        status: 'active',
+      };
+    });
+    console.log(formatList(items, { indent: '  ' }));
   }
 
   if (showSkills && result.skills?.length > 0) {
-    displaySection(`Skills (${result.skills.length})`);
-
-    for (const skill of result.skills) {
-      console.log(chalk.hex('#e8683a')(`  ${skill.name}`));
-      console.log(chalk.dim(`    ${skill.description}`));
+    console.log(formatHeader(`Skills (${result.skills.length})`));
+    const items = result.skills.map(skill => {
+      let desc = chalk.dim(skill.description);
       if (skill.triggers?.length > 0) {
-        console.log(
-          chalk.dim(
-            `    Triggers: ${skill.triggers.slice(0, 3).join(', ')}${skill.triggers.length > 3 ? '...' : ''}`
-          )
-        );
+        desc += `\n      ${chalk.dim(`Triggers: ${skill.triggers.slice(0, 3).join(', ')}${skill.triggers.length > 3 ? '...' : ''}`)}`;
       }
-    }
+      return {
+        text: `${chalk.hex(BRAND_HEX)(skill.name)}\n      ${desc}`,
+        status: 'active',
+      };
+    });
+    console.log(formatList(items, { indent: '  ' }));
   }
 
   if (showExperts && result.experts?.length > 0) {
-    displaySection(`Experts (${result.experts.length})`);
-
-    for (const expert of result.experts) {
+    console.log(formatHeader(`Experts (${result.experts.length})`));
+    const items = result.experts.map(expert => {
       const versionBadge = expert.version !== 'unknown' ? chalk.dim(` v${expert.version}`) : '';
-      console.log(chalk.hex('#e8683a')(`  ${expert.name}`) + versionBadge);
-      console.log(chalk.dim(`    ${expert.description}`));
-    }
+      return {
+        text: `${chalk.hex(BRAND_HEX)(expert.name)}${versionBadge}\n      ${chalk.dim(expert.description)}`,
+        status: 'active',
+      };
+    });
+    console.log(formatList(items, { indent: '  ' }));
   }
 
   console.log(); // Final newline

package/tools/cli/commands/status.js

@@ -11,6 +11,8 @@ const ora = require('ora');
 const { Installer } = require('../installers/core/installer');
 const { displayLogo, displaySection, success, warning, info } = require('../lib/ui');
 const { checkForUpdate } = require('../lib/version-checker');
+const { IdeRegistry } = require('../lib/ide-registry');
+const { formatKeyValue, formatList, isTTY } = require('../../../lib/table-formatter');
 
 const installer = new Installer();
 
@@ -33,14 +35,18 @@ module.exports = {
         process.exit(0);
       }
 
-      // Show installation info
-      console.log(chalk.bold('Location:    '), status.path);
-      console.log(chalk.bold('Version:     '), status.version);
+      // Show installation info using formatKeyValue
+      console.log(formatKeyValue({
+        Location: status.path,
+        Version: status.version,
+      }));
 
       // Count installed items
       const counts = await installer.countInstalledItems(status.path);
 
-      console.log(chalk.bold('\nCore:        '), chalk.green('✓ Installed'));
+      console.log(formatKeyValue({
+        '\nCore': chalk.green('✓ Installed'),
+      }, { alignValues: false }));
       info(`${counts.agents} agents`);
       info(`${counts.commands} commands`);
       info(`${counts.skills} skills`);
@@ -50,13 +56,13 @@ module.exports = {
         console.log(chalk.bold('\nConfigured IDEs:'));
         for (const ide of status.ides) {
           // Check if IDE config exists
-          const ideConfigPath = getIdeConfigPath(directory, ide);
+          const ideConfigPath = IdeRegistry.getConfigPath(ide, directory);
           const exists = await fs.pathExists(ideConfigPath);
 
           if (exists) {
-            success(formatIdeName(ide));
+            success(IdeRegistry.getDisplayName(ide));
           } else {
-            warning(`${formatIdeName(ide)} (config missing)`);
+            warning(`${IdeRegistry.getDisplayName(ide)} (config missing)`);
           }
         }
       }
@@ -87,33 +93,3 @@ module.exports = {
   },
 };
 
-/**
- * Get IDE config path
- * @param {string} projectDir - Project directory
- * @param {string} ide - IDE name
- * @returns {string}
- */
-function getIdeConfigPath(projectDir, ide) {
-  const paths = {
-    'claude-code': '.claude/commands/agileflow',
-    cursor: '.cursor/rules/agileflow',
-    windsurf: '.windsurf/workflows/agileflow',
-  };
-
-  return path.join(projectDir, paths[ide] || '');
-}
-
-/**
- * Format IDE name for display
- * @param {string} ide - IDE name
- * @returns {string}
- */
-function formatIdeName(ide) {
-  const names = {
-    'claude-code': 'Claude Code',
-    cursor: 'Cursor',
-    windsurf: 'Windsurf',
-  };
-
-  return names[ide] || ide;
-}

package/tools/cli/commands/uninstall.js

@@ -11,6 +11,7 @@ const { Installer } = require('../installers/core/installer');
 const { IdeManager } = require('../installers/ide/manager');
 const { displayLogo, displaySection, success, warning, error, confirm } = require('../lib/ui');
 const { ErrorHandler } = require('../lib/error-handler');
+const { IdeRegistry } = require('../lib/ide-registry');
 
 const installer = new Installer();
 const ideManager = new IdeManager();
@@ -40,17 +41,17 @@ module.exports = {
       // Check if removing just one IDE
       if (options.ide) {
         const ideName = options.ide.toLowerCase();
-        displaySection('Removing IDE Configuration', `IDE: ${formatIdeName(ideName)}`);
+        displaySection('Removing IDE Configuration', `IDE: ${IdeRegistry.getDisplayName(ideName)}`);
 
         if (!status.ides || !status.ides.includes(ideName)) {
-          warning(`${formatIdeName(ideName)} is not configured in this installation`);
+          warning(`${IdeRegistry.getDisplayName(ideName)} is not configured in this installation`);
           console.log(chalk.dim(`Configured IDEs: ${(status.ides || []).join(', ') || 'none'}\n`));
           process.exit(0);
         }
 
         // Confirm removal
         if (!options.force) {
-          const proceed = await confirm(`Remove ${formatIdeName(ideName)} configuration?`, false);
+          const proceed = await confirm(`Remove ${IdeRegistry.getDisplayName(ideName)} configuration?`, false);
           if (!proceed) {
             console.log(chalk.dim('\nCancelled\n'));
             process.exit(0);
@@ -60,10 +61,10 @@ module.exports = {
         console.log();
 
         // Remove the IDE configuration
-        const configPath = getIdeConfigPath(directory, ideName);
+        const configPath = IdeRegistry.getConfigPath(ideName, directory);
         if (await fs.pathExists(configPath)) {
           await fs.remove(configPath);
-          success(`Removed ${formatIdeName(ideName)} configuration`);
+          success(`Removed ${IdeRegistry.getDisplayName(ideName)} configuration`);
         }
 
         // Also remove spawnable agents for claude-code
@@ -78,16 +79,16 @@ module.exports = {
         // Update the manifest to remove this IDE
         const manifestPath = path.join(status.path, '_cfg', 'manifest.yaml');
         if (await fs.pathExists(manifestPath)) {
-          const yaml = require('js-yaml');
+          const { safeLoad, safeDump } = require('../../../lib/yaml-utils');
           const manifestContent = await fs.readFile(manifestPath, 'utf8');
-          const manifest = yaml.load(manifestContent);
+          const manifest = safeLoad(manifestContent);
           manifest.ides = (manifest.ides || []).filter(ide => ide !== ideName);
           manifest.updated_at = new Date().toISOString();
-          await fs.writeFile(manifestPath, yaml.dump(manifest), 'utf8');
+          await fs.writeFile(manifestPath, safeDump(manifest), 'utf8');
           success('Updated manifest');
         }
 
-        console.log(chalk.green(`\n${formatIdeName(ideName)} has been removed.\n`));
+        console.log(chalk.green(`\n${IdeRegistry.getDisplayName(ideName)} has been removed.\n`));
         if (status.ides.length > 1) {
           console.log(
             chalk.dim(`Remaining IDEs: ${status.ides.filter(i => i !== ideName).join(', ')}\n`)
@@ -114,10 +115,10 @@ module.exports = {
       // Remove IDE configurations
       if (status.ides && status.ides.length > 0) {
         for (const ide of status.ides) {
-          const configPath = getIdeConfigPath(directory, ide);
+          const configPath = IdeRegistry.getConfigPath(ide, directory);
           if (await fs.pathExists(configPath)) {
             await fs.remove(configPath);
-            success(`Removed ${formatIdeName(ide)} configuration`);
+            success(`Removed ${IdeRegistry.getDisplayName(ide)} configuration`);
           }
           // Also remove spawnable agents for claude-code
           if (ide === 'claude-code') {
@@ -150,33 +151,3 @@ module.exports = {
   },
 };
 
-/**
- * Get IDE config path
- * @param {string} projectDir - Project directory
- * @param {string} ide - IDE name
- * @returns {string}
- */
-function getIdeConfigPath(projectDir, ide) {
-  const paths = {
-    'claude-code': '.claude/commands/agileflow',
-    cursor: '.cursor/rules/agileflow',
-    windsurf: '.windsurf/workflows/agileflow',
-  };
-
-  return path.join(projectDir, paths[ide] || '');
-}
-
-/**
- * Format IDE name for display
- * @param {string} ide - IDE name
- * @returns {string}
- */
-function formatIdeName(ide) {
-  const names = {
-    'claude-code': 'Claude Code',
-    cursor: 'Cursor',
-    windsurf: 'Windsurf',
-  };
-
-  return names[ide] || ide;
-}