bmad-method 4.24.0 → 4.24.2

package/tools/installer/lib/installer.js

@@ -2,6 +2,7 @@ const path = require("node:path");
 const fileManager = require("./file-manager");
 const configLoader = require("./config-loader");
 const ideSetup = require("./ide-setup");
+const { extractYamlFromAgent } = require("../../lib/yaml-utils");
 
 // Dynamic imports for ES modules
 let chalk, ora, inquirer;
@@ -19,13 +20,13 @@ class Installer {
   async getCoreVersion() {
     const yaml = require("js-yaml");
     const fs = require("fs-extra");
-    const coreConfigPath = path.join(__dirname, "../../../bmad-core/core-config.yml");
+    const coreConfigPath = path.join(__dirname, "../../../bmad-core/core-config.yaml");
     try {
       const coreConfigContent = await fs.readFile(coreConfigPath, "utf8");
       const coreConfig = yaml.load(coreConfigContent);
       return coreConfig.version || "unknown";
     } catch (error) {
-      console.warn("Could not read version from core-config.yml, using 'unknown'");
+      console.warn("Could not read version from core-config.yaml, using 'unknown'");
       return "unknown";
     }
   }
@@ -185,7 +186,7 @@ class Installer {
 
     // Check for V4 installation (has .bmad-core with manifest)
     const bmadCorePath = path.join(installDir, ".bmad-core");
-    const manifestPath = path.join(bmadCorePath, "install-manifest.yml");
+    const manifestPath = path.join(bmadCorePath, "install-manifest.yaml");
 
     if (await fileManager.pathExists(manifestPath)) {
       state.type = "v4_existing";
@@ -713,7 +714,7 @@ class Installer {
       
       for (const file of filesToRestore) {
         // Skip the manifest file itself
-        if (file.endsWith('install-manifest.yml')) continue;
+        if (file.endsWith('install-manifest.yaml')) continue;
         
         const relativePath = file.replace('.bmad-core/', '');
         const destPath = path.join(installDir, file);
@@ -1009,7 +1010,7 @@ class Installer {
         
         // Check if expansion pack already exists
         let expansionDotFolder = path.join(installDir, `.${packId}`);
-        const existingManifestPath = path.join(expansionDotFolder, 'install-manifest.yml');
+        const existingManifestPath = path.join(expansionDotFolder, 'install-manifest.yaml');
         
         if (await fileManager.pathExists(existingManifestPath)) {
           spinner.stop();
@@ -1151,12 +1152,12 @@ class Installer {
           }
         }
 
-        // Copy config.yml
-        const configPath = path.join(expansionPackDir, 'config.yml');
+        // Copy config.yaml
+        const configPath = path.join(expansionPackDir, 'config.yaml');
         if (await fileManager.pathExists(configPath)) {
-          const configDestPath = path.join(expansionDotFolder, 'config.yml');
+          const configDestPath = path.join(expansionDotFolder, 'config.yaml');
           if (await fileManager.copyFile(configPath, configDestPath)) {
-            installedFiles.push(path.join(`.${packId}`, 'config.yml'));
+            installedFiles.push(path.join(`.${packId}`, 'config.yaml'));
           }
         }
         
@@ -1222,10 +1223,10 @@ class Installer {
       const agentContent = await fs.readFile(agentPath, 'utf8');
       
       // Extract YAML frontmatter to check dependencies
-      const yamlMatch = agentContent.match(/```yaml\n([\s\S]*?)```/);
-      if (yamlMatch) {
+      const yamlContent = extractYamlFromAgent(agentContent);
+      if (yamlContent) {
         try {
-          const agentConfig = yaml.parse(yamlMatch[1]);
+          const agentConfig = yaml.parse(yamlContent);
           const dependencies = agentConfig.dependencies || {};
           
           // Check for core dependencies (those that don't exist in the expansion pack)
@@ -1268,7 +1269,7 @@ class Installer {
     const fs = require('fs').promises;
     
     // Find all team files in the expansion pack
-    const teamFiles = glob.sync('agent-teams/*.yml', {
+    const teamFiles = glob.sync('agent-teams/*.yaml', {
       cwd: expansionDotFolder
     });
 
@@ -1314,13 +1315,10 @@ class Installer {
               
               // Now resolve this agent's dependencies too
               const agentContent = await fs.readFile(coreAgentPath, 'utf8');
-              const yamlMatch = agentContent.match(/```ya?ml\n([\s\S]*?)```/);
+              const yamlContent = extractYamlFromAgent(agentContent, true);
               
-              if (yamlMatch) {
+              if (yamlContent) {
                 try {
-                  // Clean up the YAML to handle command descriptions
-                  let yamlContent = yamlMatch[1];
-                  yamlContent = yamlContent.replace(/^(\s*-)(\s*"[^"]+")(\s*-\s*.*)$/gm, '$1$2');
                   
                   const agentConfig = yaml.parse(yamlContent);
                   const dependencies = agentConfig.dependencies || {};
@@ -1330,7 +1328,7 @@ class Installer {
                     const deps = dependencies[depType] || [];
                     
                     for (const dep of deps) {
-                      const depFileName = dep.endsWith('.md') || dep.endsWith('.yml') ? dep : `${dep}.md`;
+                      const depFileName = dep.endsWith('.md') || dep.endsWith('.yaml') ? dep : `${dep}.md`;
                       const expansionDepPath = path.join(expansionDotFolder, depType, depFileName);
                       
                       // Check if dependency exists in expansion pack
@@ -1360,7 +1358,7 @@ class Installer {
                 }
               }
             } else {
-              console.warn(chalk.yellow(`  Warning: Core agent ${agentId} not found for team ${path.basename(teamFile, '.yml')}`));
+              console.warn(chalk.yellow(`  Warning: Core agent ${agentId} not found for team ${path.basename(teamFile, '.yaml')}`));
             }
           }
         }
@@ -1528,7 +1526,7 @@ class Installer {
       
       if (stats) {
         // Check if it has a manifest
-        const manifestPath = path.join(folderPath, "install-manifest.yml");
+        const manifestPath = path.join(folderPath, "install-manifest.yaml");
         if (await fileManager.pathExists(manifestPath)) {
           const manifest = await fileManager.readExpansionPackManifest(installDir, folder.substring(1));
           if (manifest) {
@@ -1539,8 +1537,8 @@ class Installer {
             };
           }
         } else {
-          // Check if it has a config.yml (expansion pack without manifest)
-          const configPath = path.join(folderPath, "config.yml");
+          // Check if it has a config.yaml (expansion pack without manifest)
+          const configPath = path.join(folderPath, "config.yaml");
           if (await fileManager.pathExists(configPath)) {
             expansionPacks[folder.substring(1)] = {
               path: folderPath,
@@ -1579,7 +1577,7 @@ class Installer {
       
       for (const file of filesToRestore) {
         // Skip the manifest file itself
-        if (file.endsWith('install-manifest.yml')) continue;
+        if (file.endsWith('install-manifest.yaml')) continue;
         
         const relativePath = file.replace(`.${packId}/`, '');
         const sourcePath = path.join(pack.packPath, relativePath);
@@ -1645,7 +1643,7 @@ class Installer {
 
     while (currentDir !== path.dirname(currentDir)) {
       const bmadDir = path.join(currentDir, ".bmad-core");
-      const manifestPath = path.join(bmadDir, "install-manifest.yml");
+      const manifestPath = path.join(bmadDir, "install-manifest.yaml");
 
       if (await fileManager.pathExists(manifestPath)) {
         return bmadDir;
@@ -1656,7 +1654,7 @@ class Installer {
 
     // Also check if we're inside a .bmad-core directory
     if (path.basename(process.cwd()) === ".bmad-core") {
-      const manifestPath = path.join(process.cwd(), "install-manifest.yml");
+      const manifestPath = path.join(process.cwd(), "install-manifest.yaml");
       if (await fileManager.pathExists(manifestPath)) {
         return process.cwd();
       }

package/tools/installer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bmad-method",
-  "version": "4.24.0",
+  "version": "4.24.2",
   "description": "BMAD Method installer - AI-powered Agile development framework",
   "main": "lib/installer.js",
   "bin": {

package/tools/lib/dependency-resolver.js

@@ -1,6 +1,7 @@
 const fs = require('fs').promises;
 const path = require('path');
 const yaml = require('js-yaml');
+const { extractYamlFromAgent } = require('./yaml-utils');
 
 class DependencyResolver {
   constructor(rootDir) {
@@ -14,17 +15,12 @@ class DependencyResolver {
     const agentPath = path.join(this.bmadCore, 'agents', `${agentId}.md`);
     const agentContent = await fs.readFile(agentPath, 'utf8');
     
-    // Extract YAML from markdown content
-    const yamlMatch = agentContent.replace(/\r/g, "").match(/```ya?ml\n([\s\S]*?)\n```/);
-    if (!yamlMatch) {
+    // Extract YAML from markdown content with command cleaning
+    const yamlContent = extractYamlFromAgent(agentContent, true);
+    if (!yamlContent) {
       throw new Error(`No YAML configuration found in agent ${agentId}`);
     }
     
-    // Clean up the YAML to handle command descriptions after dashes
-    let yamlContent = yamlMatch[1];
-    // Fix commands section: convert "- command - description" to just "- command"
-    yamlContent = yamlContent.replace(/^(\s*-)(\s*"[^"]+")(\s*-\s*.*)$/gm, '$1$2');
-    
     const agentConfig = yaml.load(yamlContent);
     
     const dependencies = {
@@ -53,7 +49,7 @@ class DependencyResolver {
   }
 
   async resolveTeamDependencies(teamId) {
-    const teamPath = path.join(this.bmadCore, 'agent-teams', `${teamId}.yml`);
+    const teamPath = path.join(this.bmadCore, 'agent-teams', `${teamId}.yaml`);
     const teamContent = await fs.readFile(teamPath, 'utf8');
     const teamConfig = yaml.load(teamContent);
     
@@ -120,7 +116,7 @@ class DependencyResolver {
     }
 
     try {
-      const extensions = ['.md', '.yml', '.yaml'];
+      const extensions = ['.md', '.yaml'];
       let content = null;
       let filePath = null;
 
@@ -183,8 +179,8 @@ class DependencyResolver {
     try {
       const files = await fs.readdir(path.join(this.bmadCore, 'agent-teams'));
       return files
-        .filter(f => f.endsWith('.yml'))
-        .map(f => f.replace('.yml', ''));
+        .filter(f => f.endsWith('.yaml'))
+        .map(f => f.replace('.yaml', ''));
     } catch (error) {
       return [];
     }

package/tools/lib/yaml-utils.js

@@ -0,0 +1,29 @@
+/**
+ * Utility functions for YAML extraction from agent files
+ */
+
+/**
+ * Extract YAML content from agent markdown files
+ * @param {string} agentContent - The full content of the agent file
+ * @param {boolean} cleanCommands - Whether to clean command descriptions (default: false)
+ * @returns {string|null} - The extracted YAML content or null if not found
+ */
+function extractYamlFromAgent(agentContent, cleanCommands = false) {
+  // Remove carriage returns and match YAML block
+  const yamlMatch = agentContent.replace(/\r/g, "").match(/```ya?ml\n([\s\S]*?)\n```/);
+  if (!yamlMatch) return null;
+  
+  let yamlContent = yamlMatch[1].trim();
+  
+  // Clean up command descriptions if requested
+  // Converts "- command - description" to just "- command"
+  if (cleanCommands) {
+    yamlContent = yamlContent.replace(/^(\s*-)(\s*"[^"]+")(\s*-\s*.*)$/gm, '$1$2');
+  }
+  
+  return yamlContent;
+}
+
+module.exports = {
+  extractYamlFromAgent
+};

package/tools/update-expansion-version.js

@@ -22,8 +22,8 @@ if (!/^\d+\.\d+\.\d+$/.test(newVersion)) {
 
 async function updateVersion() {
   try {
-    // Update in config.yml
-    const configPath = path.join(__dirname, '..', 'expansion-packs', packId, 'config.yml');
+    // Update in config.yaml
+    const configPath = path.join(__dirname, '..', 'expansion-packs', packId, 'config.yaml');
     
     if (!fs.existsSync(configPath)) {
       console.error(`Error: Expansion pack '${packId}' not found`);
@@ -39,7 +39,7 @@ async function updateVersion() {
     const updatedYaml = yaml.dump(config, { indent: 2 });
     fs.writeFileSync(configPath, updatedYaml);
     
-    console.log(`✓ Updated ${packId}/config.yml: ${oldVersion} → ${newVersion}`);
+    console.log(`✓ Updated ${packId}/config.yaml: ${oldVersion} → ${newVersion}`);
     console.log(`\n✓ Successfully updated ${packId} to version ${newVersion}`);
     console.log('\nNext steps:');
     console.log('1. Test the changes');

package/tools/yaml-format.js

@@ -197,7 +197,7 @@ async function main() {
       let changed = false;
       if (ext === '.md') {
         changed = await processMarkdownFile(filePath);
-      } else if (ext === '.yml' || ext === '.yaml' || basename.includes('roomodes') || basename.includes('.yml') || basename.includes('.yaml')) {
+      } else if (ext === '.yaml' || ext === '.yml' || basename.includes('roomodes') || basename.includes('.yaml') || basename.includes('.yml')) {
         // Handle YAML files and special cases like .roomodes
         changed = await processYamlFile(filePath);