agileflow 2.55.0 → 2.57.0

package/tools/cli/installers/ide/cursor.js

@@ -32,69 +32,39 @@ class CursorSetup extends BaseIdeSetup {
     // Clean up old installation first
     await this.cleanup(projectDir);
 
-    // Create .cursor/commands/agileflow directory
+    // Create .cursor/commands/AgileFlow directory
     const cursorDir = path.join(projectDir, this.configDir);
     const commandsDir = path.join(cursorDir, this.commandsDir);
     const agileflowCommandsDir = path.join(commandsDir, 'AgileFlow');
 
-    await this.ensureDir(agileflowCommandsDir);
-
-    // Get commands from AgileFlow installation
+    // Install commands using shared recursive method
     const commandsSource = path.join(agileflowDir, 'commands');
-    let commandCount = 0;
-
-    if (await this.exists(commandsSource)) {
-      const commands = await this.scanDirectory(commandsSource, '.md');
-
-      for (const command of commands) {
-        // Read the original command content
-        let content = await this.readFile(command.path);
-
-        // Inject dynamic content (agent lists, command lists)
-        content = this.injectDynamicContent(content, agileflowDir);
-
-        // Replace docs/ references with custom folder name
-        content = this.replaceDocsReferences(content);
-
-        const targetPath = path.join(agileflowCommandsDir, `${command.name}.md`);
-        await this.writeFile(targetPath, content);
-        commandCount++;
-      }
-    }
-
-    // Create agents subdirectory
-    const agileflowAgentsDir = path.join(agileflowCommandsDir, 'agents');
-    await this.ensureDir(agileflowAgentsDir);
-
-    // Get agents from AgileFlow installation
+    const commandResult = await this.installCommandsRecursive(
+      commandsSource,
+      agileflowCommandsDir,
+      agileflowDir,
+      true // Inject dynamic content
+    );
+
+    // Install agents as subdirectory
     const agentsSource = path.join(agileflowDir, 'agents');
-    let agentCount = 0;
-
-    if (await this.exists(agentsSource)) {
-      const agents = await this.scanDirectory(agentsSource, '.md');
-
-      for (const agent of agents) {
-        // Read the original agent content
-        let content = await this.readFile(agent.path);
-
-        // Replace docs/ references with custom folder name
-        content = this.replaceDocsReferences(content);
-
-        const targetPath = path.join(agileflowAgentsDir, `${agent.name}.md`);
-        await this.writeFile(targetPath, content);
-        agentCount++;
-      }
-    }
+    const agentsTargetDir = path.join(agileflowCommandsDir, 'agents');
+    const agentResult = await this.installCommandsRecursive(
+      agentsSource,
+      agentsTargetDir,
+      agileflowDir,
+      false // No dynamic content for agents
+    );
 
     console.log(chalk.green(`  ✓ ${this.displayName} configured:`));
-    console.log(chalk.dim(`    - ${commandCount} commands installed`));
-    console.log(chalk.dim(`    - ${agentCount} agents installed`));
+    console.log(chalk.dim(`    - ${commandResult.commands} commands installed`));
+    console.log(chalk.dim(`    - ${agentResult.commands} agents installed`));
     console.log(chalk.dim(`    - Path: ${path.relative(projectDir, agileflowCommandsDir)}`));
 
     return {
       success: true,
-      commands: commandCount,
-      agents: agentCount,
+      commands: commandResult.commands,
+      agents: agentResult.commands,
     };
   }
 

package/tools/cli/installers/ide/windsurf.js

@@ -37,64 +37,34 @@ class WindsurfSetup extends BaseIdeSetup {
     const workflowsDir = path.join(windsurfDir, this.workflowsDir);
     const agileflowWorkflowsDir = path.join(workflowsDir, 'agileflow');
 
-    await this.ensureDir(agileflowWorkflowsDir);
-
-    // Get commands from AgileFlow installation
+    // Install commands using shared recursive method
     const commandsSource = path.join(agileflowDir, 'commands');
-    let commandCount = 0;
-
-    if (await this.exists(commandsSource)) {
-      const commands = await this.scanDirectory(commandsSource, '.md');
-
-      for (const command of commands) {
-        // Read the original command content
-        let content = await this.readFile(command.path);
-
-        // Inject dynamic content (agent lists, command lists)
-        content = this.injectDynamicContent(content, agileflowDir);
-
-        // Replace docs/ references with custom folder name
-        content = this.replaceDocsReferences(content);
-
-        const targetPath = path.join(agileflowWorkflowsDir, `${command.name}.md`);
-        await this.writeFile(targetPath, content);
-        commandCount++;
-      }
-    }
-
-    // Create agents subdirectory
-    const agileflowAgentsDir = path.join(agileflowWorkflowsDir, 'agents');
-    await this.ensureDir(agileflowAgentsDir);
-
-    // Get agents from AgileFlow installation
+    const commandResult = await this.installCommandsRecursive(
+      commandsSource,
+      agileflowWorkflowsDir,
+      agileflowDir,
+      true // Inject dynamic content
+    );
+
+    // Install agents as subdirectory
     const agentsSource = path.join(agileflowDir, 'agents');
-    let agentCount = 0;
-
-    if (await this.exists(agentsSource)) {
-      const agents = await this.scanDirectory(agentsSource, '.md');
-
-      for (const agent of agents) {
-        // Read the original agent content
-        let content = await this.readFile(agent.path);
-
-        // Replace docs/ references with custom folder name
-        content = this.replaceDocsReferences(content);
-
-        const targetPath = path.join(agileflowAgentsDir, `${agent.name}.md`);
-        await this.writeFile(targetPath, content);
-        agentCount++;
-      }
-    }
+    const agentsTargetDir = path.join(agileflowWorkflowsDir, 'agents');
+    const agentResult = await this.installCommandsRecursive(
+      agentsSource,
+      agentsTargetDir,
+      agileflowDir,
+      false // No dynamic content for agents
+    );
 
     console.log(chalk.green(`  ✓ ${this.displayName} configured:`));
-    console.log(chalk.dim(`    - ${commandCount} workflows installed`));
-    console.log(chalk.dim(`    - ${agentCount} agent workflows installed`));
+    console.log(chalk.dim(`    - ${commandResult.commands} workflows installed`));
+    console.log(chalk.dim(`    - ${agentResult.commands} agent workflows installed`));
     console.log(chalk.dim(`    - Path: ${path.relative(projectDir, agileflowWorkflowsDir)}`));
 
     return {
       success: true,
-      commands: commandCount,
-      agents: agentCount,
+      commands: commandResult.commands,
+      agents: agentResult.commands,
     };
   }
 

package/tools/cli/lib/content-injector.js

@@ -7,7 +7,7 @@
 
 const fs = require('fs');
 const path = require('path');
-const yaml = require('js-yaml');
+const { parseFrontmatter, normalizeTools } = require('../../../scripts/lib/frontmatter-parser');
 
 /**
  * Scan agents directory and generate formatted agent list
@@ -22,34 +22,20 @@ function generateAgentList(agentsDir) {
     const filePath = path.join(agentsDir, file);
     const content = fs.readFileSync(filePath, 'utf8');
 
-    // Extract YAML frontmatter
-    const match = content.match(/^---\n([\s\S]*?)\n---/);
-    if (!match) continue;
-
-    try {
-      const frontmatter = yaml.load(match[1]);
-
-      // Skip if frontmatter is null or not an object
-      if (!frontmatter || typeof frontmatter !== 'object') {
-        continue;
-      }
-
-      // Handle tools field - can be array or string
-      let tools = frontmatter.tools || [];
-      if (typeof tools === 'string') {
-        tools = tools.split(',').map(t => t.trim());
-      }
-
-      agents.push({
-        name: frontmatter.name || path.basename(file, '.md'),
-        description: frontmatter.description || '',
-        tools: tools,
-        model: frontmatter.model || 'haiku',
-      });
-    } catch (err) {
-      // Silently skip files with parsing errors
+    // Parse frontmatter using shared parser
+    const frontmatter = parseFrontmatter(content);
+
+    // Skip if no frontmatter found
+    if (!frontmatter || Object.keys(frontmatter).length === 0) {
       continue;
     }
+
+    agents.push({
+      name: frontmatter.name || path.basename(file, '.md'),
+      description: frontmatter.description || '',
+      tools: normalizeTools(frontmatter.tools),
+      model: frontmatter.model || 'haiku',
+    });
   }
 
   // Sort alphabetically by name
@@ -82,29 +68,20 @@ function generateCommandList(commandsDir) {
     const filePath = path.join(commandsDir, file);
     const content = fs.readFileSync(filePath, 'utf8');
 
-    // Extract YAML frontmatter
-    const match = content.match(/^---\n([\s\S]*?)\n---/);
-    if (!match) continue;
-
-    try {
-      const frontmatter = yaml.load(match[1]);
-      const cmdName = path.basename(file, '.md');
-
-      // Skip if frontmatter is null or not an object
-      if (!frontmatter || typeof frontmatter !== 'object') {
-        continue;
-      }
-
-      commands.push({
-        name: cmdName,
-        description: frontmatter.description || '',
-        argumentHint: frontmatter['argument-hint'] || '',
-      });
-    } catch (err) {
-      // Silently skip files with parsing errors - they might be generated files
-      // with content that looks like frontmatter but isn't
+    // Parse frontmatter using shared parser
+    const frontmatter = parseFrontmatter(content);
+    const cmdName = path.basename(file, '.md');
+
+    // Skip if no frontmatter found
+    if (!frontmatter || Object.keys(frontmatter).length === 0) {
       continue;
     }
+
+    commands.push({
+      name: cmdName,
+      description: frontmatter.description || '',
+      argumentHint: frontmatter['argument-hint'] || '',
+    });
   }
 
   // Sort alphabetically by name

package/tools/cli/lib/npm-utils.js

@@ -2,14 +2,33 @@
  * AgileFlow CLI - npm Registry Utilities
  *
  * Utilities for interacting with the npm registry.
+ * Set DEBUG_NPM=1 environment variable for verbose error logging.
  */
 
 const https = require('https');
 
+// Debug mode: set DEBUG_NPM=1 to see error details
+const DEBUG = process.env.DEBUG_NPM === '1';
+
+/**
+ * Log debug messages when DEBUG_NPM=1
+ * @param {string} message - Message to log
+ * @param {*} data - Optional data to include
+ */
+function debugLog(message, data = null) {
+  if (DEBUG) {
+    console.error(`[npm-utils] ${message}`, data ? JSON.stringify(data) : '');
+  }
+}
+
 /**
  * Get the latest version of a package from npm registry
- * @param {string} packageName - Name of the package
- * @returns {Promise<string|null>} Latest version or null if error
+ *
+ * Returns null on any error (network, timeout, invalid response) since
+ * version checking should not block the CLI. Set DEBUG_NPM=1 to see errors.
+ *
+ * @param {string} packageName - Name of the package (e.g., 'agileflow' or '@scope/pkg')
+ * @returns {Promise<string|null>} Latest version string or null if unavailable
  */
 async function getLatestVersion(packageName) {
   return new Promise(resolve => {
@@ -23,6 +42,8 @@ async function getLatestVersion(packageName) {
       },
     };
 
+    debugLog('Fetching version', { package: packageName, path: options.path });
+
     const req = https.request(options, res => {
       let data = '';
 
@@ -31,24 +52,30 @@ async function getLatestVersion(packageName) {
       });
 
       res.on('end', () => {
+        if (res.statusCode !== 200) {
+          debugLog('Non-200 status', { statusCode: res.statusCode });
+          return resolve(null);
+        }
+
         try {
-          if (res.statusCode === 200) {
-            const json = JSON.parse(data);
-            resolve(json.version || null);
-          } else {
-            resolve(null);
-          }
+          const json = JSON.parse(data);
+          const version = json.version || null;
+          debugLog('Version found', { version });
+          resolve(version);
         } catch (err) {
+          debugLog('JSON parse error', { error: err.message });
           resolve(null);
         }
       });
     });
 
-    req.on('error', () => {
+    req.on('error', err => {
+      debugLog('Network error', { error: err.message });
       resolve(null);
     });
 
     req.setTimeout(5000, () => {
+      debugLog('Request timeout');
       req.destroy();
       resolve(null);
     });

package/tools/cli/lib/ui.js

@@ -8,6 +8,7 @@ const chalk = require('chalk');
 const inquirer = require('inquirer');
 const path = require('node:path');
 const fs = require('node:fs');
+const { IdeManager } = require('../installers/ide/manager');
 
 // Load package.json for version
 const packageJsonPath = path.join(__dirname, '..', '..', '..', 'package.json');
@@ -73,8 +74,27 @@ function info(message) {
   console.log(chalk.dim(`  ${message}`));
 }
 
+// IDE Manager instance for dynamic IDE discovery
+const ideManager = new IdeManager();
+
+/**
+ * Get available IDE choices dynamically from installed handlers
+ * @returns {Array} IDE choices formatted for inquirer
+ */
+function getIdeChoices() {
+  const ides = ideManager.getAvailableIdes();
+
+  return ides.map((ide, index) => ({
+    name: ide.name,
+    value: ide.value,
+    // First IDE (preferred) is checked by default
+    checked: ide.preferred || index === 0,
+  }));
+}
+
 /**
- * Available IDE configurations
+ * @deprecated Use getIdeChoices() instead - dynamically loaded from IDE handlers
+ * Legacy hardcoded IDE choices kept for backward compatibility
  */
 const IDE_CHOICES = [
   {
@@ -126,7 +146,7 @@ async function promptInstall() {
       type: 'checkbox',
       name: 'ides',
       message: 'Select your IDE(s):',
-      choices: IDE_CHOICES,
+      choices: getIdeChoices(),
       validate: input => {
         if (input.length === 0) {
           return 'Please select at least one IDE';
@@ -219,5 +239,6 @@ module.exports = {
   promptInstall,
   confirm,
   getIdeConfig,
-  IDE_CHOICES,
+  getIdeChoices,
+  IDE_CHOICES, // @deprecated - kept for backward compatibility
 };