bmad-method 6.2.1-next.23 → 6.2.1-next.25

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
   "name": "bmad-method",
-  "version": "6.2.1-next.23",
+  "version": "6.2.1-next.25",
   "description": "Breakthrough Method of Agile AI-driven Development",
   "keywords": [
     "agile",

package/tools/cli/commands/install.js

@@ -18,7 +18,7 @@ module.exports = {
       'Comma-separated list of tool/IDE IDs to configure (e.g., "claude-code,cursor"). Use "none" to skip tool configuration.',
     ],
     ['--custom-content <paths>', 'Comma-separated list of paths to custom modules/agents/workflows'],
-    ['--action <type>', 'Action type for existing installations: install, update, quick-update, or compile-agents'],
+    ['--action <type>', 'Action type for existing installations: install, update, or quick-update'],
     ['--user-name <name>', 'Name for agents to use (default: system username)'],
     ['--communication-language <lang>', 'Language for agent communication (default: English)'],
     ['--document-output-language <lang>', 'Language for document output (default: English)'],
@@ -49,13 +49,6 @@ module.exports = {
         process.exit(0);
       }
 
-      // Handle compile agents separately
-      if (config.actionType === 'compile-agents') {
-        const result = await installer.compileAgents(config);
-        await prompts.log.info(`Recompiled ${result.agentCount} agents with customizations applied`);
-        process.exit(0);
-      }
-
       // Regular install/update flow
       const result = await installer.install(config);
 

package/tools/cli/installers/lib/core/installer.js

@@ -6,7 +6,6 @@ const { ModuleManager } = require('../modules/manager');
 const { IdeManager } = require('../ide/manager');
 const { FileOps } = require('../../../lib/file-ops');
 const { Config } = require('../../../lib/config');
-const { XmlHandler } = require('../../../lib/xml-handler');
 const { DependencyResolver } = require('./dependency-resolver');
 const { ConfigCollector } = require('./config-collector');
 const { getProjectRoot, getSourcePath, getModulePath } = require('../../../lib/project-root');
@@ -25,7 +24,6 @@ class Installer {
     this.ideManager = new IdeManager();
     this.fileOps = new FileOps();
     this.config = new Config();
-    this.xmlHandler = new XmlHandler();
     this.dependencyResolver = new DependencyResolver();
     this.configCollector = new ConfigCollector();
     this.ideConfigManager = new IdeConfigManager();
@@ -2114,10 +2112,6 @@ class Installer {
       },
     );
 
-    // Process agent files to build YAML agents and create customize templates
-    const modulePath = path.join(bmadDir, moduleName);
-    await this.processAgentFiles(modulePath, moduleName);
-
     // Dependencies are already included in full module install
   }
 
@@ -2227,16 +2221,8 @@ class Installer {
     const sourcePath = getModulePath('core');
     const targetPath = path.join(bmadDir, 'core');
 
-    // Copy core files (skip .agent.yaml files like modules do)
+    // Copy core files
     await this.copyCoreFiles(sourcePath, targetPath);
-
-    // Compile agents using the same compiler as modules
-    const { ModuleManager } = require('../modules/manager');
-    const moduleManager = new ModuleManager();
-    await moduleManager.compileModuleAgents(sourcePath, targetPath, 'core', bmadDir, this);
-
-    // Process agent files to inject activation block
-    await this.processAgentFiles(targetPath, 'core');
   }
 
   /**
@@ -2254,16 +2240,6 @@ class Installer {
         continue;
       }
 
-      // Skip sidecar directories - they are handled separately during agent compilation
-      if (
-        path
-          .dirname(file)
-          .split('/')
-          .some((dir) => dir.toLowerCase().includes('sidecar'))
-      ) {
-        continue;
-      }
-
       // Skip module.yaml at root - it's only needed at install time
       if (file === 'module.yaml') {
         continue;
@@ -2274,27 +2250,9 @@ class Installer {
         continue;
       }
 
-      // Skip .agent.yaml files - they will be compiled separately
-      if (file.endsWith('.agent.yaml')) {
-        continue;
-      }
-
       const sourceFile = path.join(sourcePath, file);
       const targetFile = path.join(targetPath, file);
 
-      // Check if this is an agent file
-      if (file.startsWith('agents/') && file.endsWith('.md')) {
-        // Read the file to check for localskip
-        const content = await fs.readFile(sourceFile, 'utf8');
-
-        // Check for localskip="true" in the agent tag
-        const agentMatch = content.match(/<agent[^>]*\slocalskip="true"[^>]*>/);
-        if (agentMatch) {
-          await prompts.log.message(`  Skipping web-only agent: ${path.basename(file)}`);
-          continue; // Skip this agent
-        }
-      }
-
       // Copy the file with placeholder replacement
       await fs.ensureDir(path.dirname(targetFile));
       await this.copyFileWithPlaceholderReplacement(sourceFile, targetFile);
@@ -2328,58 +2286,6 @@ class Installer {
     return files;
   }
 
-  /**
-   * Process agent files to build YAML agents and inject activation blocks
-   * @param {string} modulePath - Path to module in bmad/ installation
-   * @param {string} moduleName - Module name
-   */
-  async processAgentFiles(modulePath, moduleName) {
-    const agentsPath = path.join(modulePath, 'agents');
-
-    // Check if agents directory exists
-    if (!(await fs.pathExists(agentsPath))) {
-      return; // No agents to process
-    }
-
-    // Determine project directory (parent of bmad/ directory)
-    const bmadDir = path.dirname(modulePath);
-    const cfgAgentsDir = path.join(bmadDir, '_config', 'agents');
-
-    // Ensure _config/agents directory exists
-    await fs.ensureDir(cfgAgentsDir);
-
-    // Get all agent files
-    const agentFiles = await fs.readdir(agentsPath);
-
-    for (const agentFile of agentFiles) {
-      // Skip .agent.yaml files - they should already be compiled by compileModuleAgents
-      if (agentFile.endsWith('.agent.yaml')) {
-        continue;
-      }
-
-      // Only process .md files (already compiled from YAML)
-      if (!agentFile.endsWith('.md')) {
-        continue;
-      }
-
-      const agentName = agentFile.replace('.md', '');
-      const mdPath = path.join(agentsPath, agentFile);
-      const customizePath = path.join(cfgAgentsDir, `${moduleName}-${agentName}.customize.yaml`);
-
-      // For .md files that are already compiled, we don't need to do much
-      // Just ensure the customize template exists
-      if (!(await fs.pathExists(customizePath))) {
-        const genericTemplatePath = getSourcePath('utility', 'agent-components', 'agent.customize.template.yaml');
-        if (await fs.pathExists(genericTemplatePath)) {
-          await this.copyFileWithPlaceholderReplacement(genericTemplatePath, customizePath);
-          if (process.env.BMAD_VERBOSE_INSTALL === 'true') {
-            await prompts.log.message(`  Created customize: ${moduleName}-${agentName}.customize.yaml`);
-          }
-        }
-      }
-    }
-  }
-
   /**
    * Private: Update core
    */
@@ -2393,12 +2299,6 @@ class Installer {
     } else {
       // Selective update - preserve user modifications
       await this.fileOps.syncDirectory(sourcePath, targetPath);
-
-      // Recompile agents (#1133)
-      const { ModuleManager } = require('../modules/manager');
-      const moduleManager = new ModuleManager();
-      await moduleManager.compileModuleAgents(sourcePath, targetPath, 'core', bmadDir, this);
-      await this.processAgentFiles(targetPath, 'core');
     }
   }
 
@@ -2643,114 +2543,6 @@ class Installer {
     }
   }
 
-  /**
-   * Compile agents with customizations only
-   * @param {Object} config - Configuration with directory
-   * @returns {Object} Compilation result
-   */
-  async compileAgents(config) {
-    // Using @clack prompts
-    const { ModuleManager } = require('../modules/manager');
-    const { getSourcePath } = require('../../../lib/project-root');
-
-    const spinner = await prompts.spinner();
-    spinner.start('Recompiling agents with customizations...');
-
-    try {
-      const projectDir = path.resolve(config.directory);
-      const { bmadDir } = await this.findBmadDir(projectDir);
-
-      // Check if bmad directory exists
-      if (!(await fs.pathExists(bmadDir))) {
-        spinner.stop('No BMAD installation found');
-        throw new Error(`BMAD not installed at ${bmadDir}. Use regular install for first-time setup.`);
-      }
-
-      // Detect existing installation
-      const existingInstall = await this.detector.detect(bmadDir);
-      const installedModules = existingInstall.modules.map((m) => m.id);
-
-      // Initialize module manager
-      const moduleManager = new ModuleManager();
-      moduleManager.setBmadFolderName(path.basename(bmadDir));
-
-      let totalAgentCount = 0;
-
-      // Get custom module sources from cache
-      const customModuleSources = new Map();
-      const cacheDir = path.join(bmadDir, '_config', 'custom');
-      if (await fs.pathExists(cacheDir)) {
-        const cachedModules = await fs.readdir(cacheDir, { withFileTypes: true });
-
-        for (const cachedModule of cachedModules) {
-          if (cachedModule.isDirectory()) {
-            const moduleId = cachedModule.name;
-            const cachedPath = path.join(cacheDir, moduleId);
-            const moduleYamlPath = path.join(cachedPath, 'module.yaml');
-
-            // Check if this is actually a custom module
-            if (await fs.pathExists(moduleYamlPath)) {
-              // Check if this is an external official module - skip cache for those
-              const isExternal = await this.moduleManager.isExternalModule(moduleId);
-              if (isExternal) {
-                // External modules are handled via cloneExternalModule, not from cache
-                continue;
-              }
-              customModuleSources.set(moduleId, cachedPath);
-            }
-          }
-        }
-      }
-
-      // Process each installed module
-      for (const moduleId of installedModules) {
-        spinner.message(`Recompiling agents in ${moduleId}...`);
-
-        // Get source path
-        let sourcePath;
-        if (moduleId === 'core') {
-          sourcePath = getSourcePath('core-skills');
-        } else {
-          // First check if it's in the custom cache
-          if (customModuleSources.has(moduleId)) {
-            sourcePath = customModuleSources.get(moduleId);
-          } else {
-            sourcePath = await moduleManager.findModuleSource(moduleId);
-          }
-        }
-
-        if (!sourcePath) {
-          await prompts.log.warn(`Source not found for module ${moduleId}, skipping...`);
-          continue;
-        }
-
-        const targetPath = path.join(bmadDir, moduleId);
-
-        // Compile agents for this module
-        await moduleManager.compileModuleAgents(sourcePath, targetPath, moduleId, bmadDir, this);
-
-        // Count agents (rough estimate based on files)
-        const agentsPath = path.join(targetPath, 'agents');
-        if (await fs.pathExists(agentsPath)) {
-          const agentFiles = await fs.readdir(agentsPath);
-          const agentCount = agentFiles.filter((f) => f.endsWith('.md')).length;
-          totalAgentCount += agentCount;
-        }
-      }
-
-      spinner.stop('Agent recompilation complete!');
-
-      return {
-        success: true,
-        agentCount: totalAgentCount,
-        modules: installedModules,
-      };
-    } catch (error) {
-      spinner.error('Agent recompilation failed');
-      throw error;
-    }
-  }
-
   /**
    * Private: Prompt for update action
    */

package/tools/cli/installers/lib/core/manifest-generator.js

@@ -515,7 +515,7 @@ class ManifestGenerator {
 
   /**
    * Get agents from a directory recursively
-   * Only includes compiled .md files (not .agent.yaml source files)
+   * Only includes .md files with agent content
    */
   async getAgentsFromDir(dirPath, moduleName, relativePath = '') {
     // Skip directories claimed by collectSkills
@@ -572,7 +572,7 @@ class ManifestGenerator {
         const newRelativePath = relativePath ? `${relativePath}/${entry.name}` : entry.name;
         const subDirAgents = await this.getAgentsFromDir(fullPath, moduleName, newRelativePath);
         agents.push(...subDirAgents);
-      } else if (entry.name.endsWith('.md') && !entry.name.endsWith('.agent.yaml') && entry.name.toLowerCase() !== 'readme.md') {
+      } else if (entry.name.endsWith('.md') && entry.name.toLowerCase() !== 'readme.md') {
         const content = await fs.readFile(fullPath, 'utf8');
 
         // Skip files that don't contain <agent> tag (e.g., README files)

package/tools/cli/installers/lib/custom/handler.js

@@ -2,19 +2,11 @@ const path = require('node:path');
 const fs = require('fs-extra');
 const yaml = require('yaml');
 const prompts = require('../../../lib/prompts');
-const { FileOps } = require('../../../lib/file-ops');
-const { XmlHandler } = require('../../../lib/xml-handler');
-
 /**
  * Handler for custom content (custom.yaml)
- * Installs custom agents and workflows without requiring a full module structure
+ * Discovers custom agents and workflows in the project
  */
 class CustomHandler {
-  constructor() {
-    this.fileOps = new FileOps();
-    this.xmlHandler = new XmlHandler();
-  }
-
   /**
    * Find all custom.yaml files in the project
    * @param {string} projectRoot - Project root directory
@@ -115,244 +107,6 @@ class CustomHandler {
       return null;
     }
   }
-
-  /**
-   * Install custom content
-   * @param {string} customPath - Path to custom content directory
-   * @param {string} bmadDir - Target bmad directory
-   * @param {Object} config - Configuration from custom.yaml
-   * @param {Function} fileTrackingCallback - Optional callback to track installed files
-   * @returns {Object} Installation result
-   */
-  async install(customPath, bmadDir, config, fileTrackingCallback = null) {
-    const results = {
-      agentsInstalled: 0,
-      workflowsInstalled: 0,
-      filesCopied: 0,
-      preserved: 0,
-      errors: [],
-    };
-
-    try {
-      // Create custom directories in bmad
-      const bmadCustomDir = path.join(bmadDir, 'custom');
-      const bmadAgentsDir = path.join(bmadCustomDir, 'agents');
-      const bmadWorkflowsDir = path.join(bmadCustomDir, 'workflows');
-
-      await fs.ensureDir(bmadCustomDir);
-      await fs.ensureDir(bmadAgentsDir);
-      await fs.ensureDir(bmadWorkflowsDir);
-
-      // Process agents - compile and copy agents
-      const agentsDir = path.join(customPath, 'agents');
-      if (await fs.pathExists(agentsDir)) {
-        await this.compileAndCopyAgents(agentsDir, bmadAgentsDir, bmadDir, config, fileTrackingCallback, results);
-
-        // Count agent files
-        const agentFiles = await this.findFilesRecursively(agentsDir, ['.agent.yaml', '.md']);
-        results.agentsInstalled = agentFiles.length;
-      }
-
-      // Process workflows - copy entire workflows directory structure
-      const workflowsDir = path.join(customPath, 'workflows');
-      if (await fs.pathExists(workflowsDir)) {
-        await this.copyDirectory(workflowsDir, bmadWorkflowsDir, results, fileTrackingCallback, config);
-
-        // Count workflow files
-        const workflowFiles = await this.findFilesRecursively(workflowsDir, ['.md']);
-        results.workflowsInstalled = workflowFiles.length;
-      }
-
-      // Process any additional files at root
-      const entries = await fs.readdir(customPath, { withFileTypes: true });
-      for (const entry of entries) {
-        if (entry.isFile() && entry.name !== 'custom.yaml' && !entry.name.startsWith('.') && !entry.name.endsWith('.md')) {
-          // Skip .md files at root as they're likely docs
-          const sourcePath = path.join(customPath, entry.name);
-          const targetPath = path.join(bmadCustomDir, entry.name);
-
-          try {
-            // Check if file already exists
-            if (await fs.pathExists(targetPath)) {
-              // File already exists, preserve it
-              results.preserved = (results.preserved || 0) + 1;
-            } else {
-              await fs.copy(sourcePath, targetPath);
-              results.filesCopied++;
-
-              if (fileTrackingCallback) {
-                fileTrackingCallback(targetPath);
-              }
-            }
-          } catch (error) {
-            results.errors.push(`Failed to copy file ${entry.name}: ${error.message}`);
-          }
-        }
-      }
-    } catch (error) {
-      results.errors.push(`Installation failed: ${error.message}`);
-    }
-
-    return results;
-  }
-
-  /**
-   * Find all files with specific extensions recursively
-   * @param {string} dir - Directory to search
-   * @param {Array} extensions - File extensions to match
-   * @returns {Array} List of matching files
-   */
-  async findFilesRecursively(dir, extensions) {
-    const files = [];
-
-    async function search(currentDir) {
-      const entries = await fs.readdir(currentDir, { withFileTypes: true });
-
-      for (const entry of entries) {
-        const fullPath = path.join(currentDir, entry.name);
-
-        if (entry.isDirectory()) {
-          await search(fullPath);
-        } else if (extensions.some((ext) => entry.name.endsWith(ext))) {
-          files.push(fullPath);
-        }
-      }
-    }
-
-    await search(dir);
-    return files;
-  }
-
-  /**
-   * Recursively copy a directory
-   * @param {string} sourceDir - Source directory
-   * @param {string} targetDir - Target directory
-   * @param {Object} results - Results object to update
-   * @param {Function} fileTrackingCallback - Optional callback
-   * @param {Object} config - Configuration for placeholder replacement
-   */
-  async copyDirectory(sourceDir, targetDir, results, fileTrackingCallback, config) {
-    await fs.ensureDir(targetDir);
-    const entries = await fs.readdir(sourceDir, { withFileTypes: true });
-
-    for (const entry of entries) {
-      const sourcePath = path.join(sourceDir, entry.name);
-      const targetPath = path.join(targetDir, entry.name);
-
-      if (entry.isDirectory()) {
-        await this.copyDirectory(sourcePath, targetPath, results, fileTrackingCallback, config);
-      } else {
-        try {
-          // Check if file already exists
-          if (await fs.pathExists(targetPath)) {
-            // File already exists, preserve it
-            results.preserved = (results.preserved || 0) + 1;
-          } else {
-            // Copy with placeholder replacement for text files
-            const textExtensions = ['.md', '.yaml', '.yml', '.txt', '.json'];
-            if (textExtensions.some((ext) => entry.name.endsWith(ext))) {
-              // Read source content
-              let content = await fs.readFile(sourcePath, 'utf8');
-
-              // Replace placeholders
-              content = content.replaceAll('{user_name}', config.user_name || 'User');
-              content = content.replaceAll('{communication_language}', config.communication_language || 'English');
-              content = content.replaceAll('{output_folder}', config.output_folder || 'docs');
-
-              // Write to target
-              await fs.ensureDir(path.dirname(targetPath));
-              await fs.writeFile(targetPath, content, 'utf8');
-            } else {
-              // Copy binary files as-is
-              await fs.copy(sourcePath, targetPath);
-            }
-
-            results.filesCopied++;
-            if (entry.name.endsWith('.md')) {
-              results.workflowsInstalled++;
-            }
-            if (fileTrackingCallback) {
-              fileTrackingCallback(targetPath);
-            }
-          }
-        } catch (error) {
-          results.errors.push(`Failed to copy ${entry.name}: ${error.message}`);
-        }
-      }
-    }
-  }
-
-  /**
-   * Compile .agent.yaml files to .md format and handle sidecars
-   * @param {string} sourceAgentsPath - Source agents directory
-   * @param {string} targetAgentsPath - Target agents directory
-   * @param {string} bmadDir - BMAD installation directory
-   * @param {Object} config - Configuration for placeholder replacement
-   * @param {Function} fileTrackingCallback - Optional callback to track installed files
-   * @param {Object} results - Results object to update
-   */
-  async compileAndCopyAgents(sourceAgentsPath, targetAgentsPath, bmadDir, config, fileTrackingCallback, results) {
-    // Get all .agent.yaml files recursively
-    const agentFiles = await this.findFilesRecursively(sourceAgentsPath, ['.agent.yaml']);
-
-    for (const agentFile of agentFiles) {
-      const relativePath = path.relative(sourceAgentsPath, agentFile).split(path.sep).join('/');
-      const targetDir = path.join(targetAgentsPath, path.dirname(relativePath));
-
-      await fs.ensureDir(targetDir);
-
-      const agentName = path.basename(agentFile, '.agent.yaml');
-      const targetMdPath = path.join(targetDir, `${agentName}.md`);
-      // Use the actual bmadDir if available (for when installing to temp dir)
-      const actualBmadDir = config._bmadDir || bmadDir;
-      const customizePath = path.join(actualBmadDir, '_config', 'agents', `custom-${agentName}.customize.yaml`);
-
-      // Read and compile the YAML
-      try {
-        const yamlContent = await fs.readFile(agentFile, 'utf8');
-        const { compileAgent } = require('../../../lib/agent/compiler');
-
-        // Create customize template if it doesn't exist
-        if (!(await fs.pathExists(customizePath))) {
-          const { getSourcePath } = require('../../../lib/project-root');
-          const genericTemplatePath = getSourcePath('utility', 'agent-components', 'agent.customize.template.yaml');
-          if (await fs.pathExists(genericTemplatePath)) {
-            let templateContent = await fs.readFile(genericTemplatePath, 'utf8');
-            await fs.writeFile(customizePath, templateContent, 'utf8');
-            // Only show customize creation in verbose mode
-            if (process.env.BMAD_VERBOSE_INSTALL === 'true') {
-              await prompts.log.message('  Created customize: custom-' + agentName + '.customize.yaml');
-            }
-          }
-        }
-
-        // Compile the agent
-        const { xml } = compileAgent(yamlContent, {}, agentName, relativePath, { config });
-
-        // Replace placeholders in the compiled content
-        let processedXml = xml;
-        processedXml = processedXml.replaceAll('{user_name}', config.user_name || 'User');
-        processedXml = processedXml.replaceAll('{communication_language}', config.communication_language || 'English');
-        processedXml = processedXml.replaceAll('{output_folder}', config.output_folder || 'docs');
-
-        // Write the compiled MD file
-        await fs.writeFile(targetMdPath, processedXml, 'utf8');
-
-        // Track the file
-        if (fileTrackingCallback) {
-          fileTrackingCallback(targetMdPath);
-        }
-
-        // Only show compilation details in verbose mode
-        if (process.env.BMAD_VERBOSE_INSTALL === 'true') {
-          await prompts.log.message('    Compiled agent: ' + agentName + ' -> ' + path.relative(targetAgentsPath, targetMdPath));
-        }
-      } catch (error) {
-        await prompts.log.warn('    Failed to compile agent ' + agentName + ': ' + error.message);
-        results.errors.push(`Failed to compile agent ${agentName}: ${error.message}`);
-      }
-    }
-  }
 }
 
 module.exports = { CustomHandler };

package/tools/cli/installers/lib/ide/_base-ide.js

@@ -1,6 +1,5 @@
 const path = require('node:path');
 const fs = require('fs-extra');
-const { XmlHandler } = require('../../../lib/xml-handler');
 const prompts = require('../../../lib/prompts');
 const { getSourcePath } = require('../../../lib/project-root');
 const { BMAD_FOLDER_NAME } = require('./shared/path-utils');
@@ -18,7 +17,6 @@ class BaseIdeSetup {
     this.rulesDir = null; // Override in subclasses
     this.configFile = null; // Override in subclasses when detection is file-based
     this.detectionPaths = []; // Additional paths that indicate the IDE is configured
-    this.xmlHandler = new XmlHandler();
     this.bmadFolderName = BMAD_FOLDER_NAME; // Default, can be overridden
   }
 
@@ -30,15 +28,6 @@ class BaseIdeSetup {
     this.bmadFolderName = bmadFolderName;
   }
 
-  /**
-   * Get the agent command activation header from the central template
-   * @returns {string} The activation header text
-   */
-  async getAgentCommandHeader() {
-    const headerPath = getSourcePath('utility', 'agent-components', 'agent-command-header.md');
-    return await fs.readFile(headerPath, 'utf8');
-  }
-
   /**
    * Main setup method - must be implemented by subclasses
    * @param {string} projectDir - Project directory
@@ -511,11 +500,6 @@ class BaseIdeSetup {
     // Replace placeholders
     let processed = content;
 
-    // Inject activation block for agent files FIRST (before replacements)
-    if (metadata.name && content.includes('<agent')) {
-      processed = this.xmlHandler.injectActivationSimple(processed, metadata);
-    }
-
     // Only replace {project-root} if a specific projectDir is provided
     // Otherwise leave the placeholder intact
     // Note: Don't add trailing slash - paths in source include leading slash