agileflow 2.51.0 → 2.56.0

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
@@ -149,5 +126,5 @@ function injectContent(templateContent, agentsDir, commandsDir) {
 module.exports = {
   generateAgentList,
   generateCommandList,
-  injectContent
+  injectContent,
 };

package/tools/cli/lib/docs-setup.js

@@ -25,7 +25,8 @@ try {
   );
 } catch (e) {
   // Fallback if README not found
-  packageReadme = '# AgileFlow\n\nSee https://github.com/projectquestorg/AgileFlow for documentation.';
+  packageReadme =
+    '# AgileFlow\n\nSee https://github.com/projectquestorg/AgileFlow for documentation.';
 }
 
 /**
@@ -386,7 +387,7 @@ Document your CI/CD workflows and configuration here.
 
       if (fs.existsSync(gitignorePath)) {
         const existingGitignore = await fs.readFile(gitignorePath, 'utf8');
-        const newEntries = gitignoreEntries.filter((entry) => !existingGitignore.includes(entry));
+        const newEntries = gitignoreEntries.filter(entry => !existingGitignore.includes(entry));
         if (newEntries.length > 0) {
           await fs.appendFile(gitignorePath, '\n' + newEntries.join('\n') + '\n', 'utf8');
           console.log(chalk.yellow(`  ↻ Updated .gitignore with ${newEntries.length} entries`));

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

@@ -2,17 +2,36 @@
  * 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) => {
+  return new Promise(resolve => {
     const options = {
       hostname: 'registry.npmjs.org',
       port: 443,
@@ -23,32 +42,40 @@ async function getLatestVersion(packageName) {
       },
     };
 
-    const req = https.request(options, (res) => {
+    debugLog('Fetching version', { package: packageName, path: options.path });
+
+    const req = https.request(options, res => {
       let data = '';
 
-      res.on('data', (chunk) => {
+      res.on('data', chunk => {
         data += chunk;
       });
 
       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 = [
   {
@@ -82,7 +102,7 @@ const IDE_CHOICES = [
     value: 'claude-code',
     checked: true,
     configDir: '.claude/commands',
-    description: 'Anthropic\'s Claude Code IDE',
+    description: "Anthropic's Claude Code IDE",
   },
   {
     name: 'Cursor',
@@ -96,7 +116,7 @@ const IDE_CHOICES = [
     value: 'windsurf',
     checked: false,
     configDir: '.windsurf/workflows',
-    description: 'Codeium\'s AI IDE',
+    description: "Codeium's AI IDE",
   },
 ];
 
@@ -113,7 +133,7 @@ async function promptInstall() {
       name: 'directory',
       message: 'Where would you like to install AgileFlow?',
       default: '.',
-      validate: (input) => {
+      validate: input => {
         const resolved = path.resolve(input);
         const parent = path.dirname(resolved);
         if (!fs.existsSync(parent)) {
@@ -126,8 +146,8 @@ async function promptInstall() {
       type: 'checkbox',
       name: 'ides',
       message: 'Select your IDE(s):',
-      choices: IDE_CHOICES,
-      validate: (input) => {
+      choices: getIdeChoices(),
+      validate: input => {
         if (input.length === 0) {
           return 'Please select at least one IDE';
         }
@@ -145,7 +165,7 @@ async function promptInstall() {
       name: 'agileflowFolder',
       message: 'AgileFlow installation folder name:',
       default: '.agileflow',
-      validate: (input) => {
+      validate: input => {
         if (!/^[a-zA-Z0-9._-]+$/.test(input)) {
           return 'Folder name can only contain letters, numbers, dots, underscores, and hyphens';
         }
@@ -157,7 +177,7 @@ async function promptInstall() {
       name: 'docsFolder',
       message: 'Documentation folder name:',
       default: 'docs',
-      validate: (input) => {
+      validate: input => {
         if (!/^[a-zA-Z0-9._-]+$/.test(input)) {
           return 'Folder name can only contain letters, numbers, dots, underscores, and hyphens';
         }
@@ -206,7 +226,7 @@ async function confirm(message, defaultValue = true) {
  * @returns {Object|null}
  */
 function getIdeConfig(ideName) {
-  return IDE_CHOICES.find((ide) => ide.value === ideName) || null;
+  return IDE_CHOICES.find(ide => ide.value === ideName) || null;
 }
 
 module.exports = {
@@ -219,5 +239,6 @@ module.exports = {
   promptInstall,
   confirm,
   getIdeConfig,
-  IDE_CHOICES,
+  getIdeChoices,
+  IDE_CHOICES, // @deprecated - kept for backward compatibility
 };

package/tools/cli/lib/version-checker.js

@@ -18,17 +18,17 @@ const packageJson = require(packageJsonPath);
  * @returns {Promise<string|null>} Latest version or null
  */
 async function getLatestVersion(packageName = 'agileflow') {
-  return new Promise((resolve) => {
+  return new Promise(resolve => {
     const url = `https://registry.npmjs.org/${packageName}/latest`;
 
-    const req = https.get(url, { timeout: 5000 }, (res) => {
+    const req = https.get(url, { timeout: 5000 }, res => {
       if (res.statusCode !== 200) {
         resolve(null);
         return;
       }
 
       let data = '';
-      res.on('data', (chunk) => {
+      res.on('data', chunk => {
         data += chunk;
       });
 

package/tools/postinstall.js

@@ -137,8 +137,8 @@ async function runAutoInstall() {
         output: process.stdout,
       });
 
-      const answer = await new Promise((resolve) => {
-        readline.question('Choose an option (1 or 2): ', (ans) => {
+      const answer = await new Promise(resolve => {
+        readline.question('Choose an option (1 or 2): ', ans => {
           readline.close();
           resolve(ans.trim());
         });
@@ -176,7 +176,6 @@ async function runAutoInstall() {
     console.log('');
     log('To skip auto-setup in the future, set: AGILEFLOW_SKIP_INSTALL=true', 'dim');
     console.log('');
-
   } catch (error) {
     console.log('');
     log('⚠️  Auto-setup encountered an issue.', 'yellow');