bmad-method 6.2.1-next.24 → 6.2.1-next.26

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
   "name": "bmad-method",
-  "version": "6.2.1-next.24",
+  "version": "6.2.1-next.26",
   "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

@@ -50,29 +50,6 @@ class ManifestGenerator {
     return getInstallToBmadShared(manifest, filename);
   }
 
-  /**
-   * Native SKILL.md entrypoints can be packaged as either skills or agents.
-   * Both need verbatim installation for skill-format IDEs.
-   * @param {string|null} artifactType - Manifest type resolved for SKILL.md
-   * @returns {boolean} True when the directory should be installed verbatim
-   */
-  isNativeSkillDirType(artifactType) {
-    return artifactType === 'skill' || artifactType === 'agent';
-  }
-
-  /**
-   * Check whether a loaded bmad-skill-manifest.yaml declares a native
-   * SKILL.md entrypoint, either as a single-entry manifest or a multi-entry map.
-   * @param {Object|null} manifest - Loaded manifest
-   * @returns {boolean} True when the manifest contains a native skill/agent entrypoint
-   */
-  hasNativeSkillManifest(manifest) {
-    if (!manifest) return false;
-    if (manifest.__single) return this.isNativeSkillDirType(manifest.__single.type);
-
-    return Object.values(manifest).some((entry) => this.isNativeSkillDirType(entry?.type));
-  }
-
   /**
    * Clean text for CSV output by normalizing whitespace.
    * Note: Quote escaping is handled by escapeCsv() at write time.
@@ -170,9 +147,9 @@ class ManifestGenerator {
 
   /**
    * Recursively walk a module directory tree, collecting native SKILL.md entrypoints.
-   * A native entrypoint directory is one that contains both a
-   * bmad-skill-manifest.yaml with type: skill or type: agent AND a SKILL.md file
-   * with name/description frontmatter.
+   * A directory is discovered as a skill when it contains a SKILL.md file with
+   * valid name/description frontmatter (name must match directory name).
+   * Manifest YAML is loaded only when present — for install_to_bmad and agent metadata.
    * Populates this.skills[] and this.skillClaimedDirs (Set of absolute paths).
    */
   async collectSkills() {
@@ -193,77 +170,55 @@ class ManifestGenerator {
           return;
         }
 
-        // Check this directory for skill manifest
-        const manifest = await this.loadSkillManifest(dir);
-
-        // Determine if this directory is a native SKILL.md entrypoint
+        // SKILL.md with valid frontmatter is the primary discovery gate
         const skillFile = 'SKILL.md';
-        const artifactType = this.getArtifactType(manifest, skillFile);
-
-        if (this.isNativeSkillDirType(artifactType)) {
-          const skillMdPath = path.join(dir, 'SKILL.md');
-          const dirName = path.basename(dir);
-
-          // Validate and parse SKILL.md
-          const skillMeta = await this.parseSkillMd(skillMdPath, dir, dirName, debug);
-
-          if (skillMeta) {
-            // Build path relative from module root (points to SKILL.md — the permanent entrypoint)
-            const relativePath = path.relative(modulePath, dir).split(path.sep).join('/');
-            const installPath = relativePath
-              ? `${this.bmadFolderName}/${moduleName}/${relativePath}/${skillFile}`
-              : `${this.bmadFolderName}/${moduleName}/${skillFile}`;
-
-            // Native SKILL.md entrypoints derive canonicalId from directory name.
-            // Agent entrypoints may keep canonicalId metadata for compatibility, so
-            // only warn for non-agent SKILL.md directories.
-            if (manifest && manifest.__single && manifest.__single.canonicalId && artifactType !== 'agent') {
-              console.warn(
-                `Warning: Native entrypoint manifest at ${dir}/bmad-skill-manifest.yaml contains canonicalId — this field is ignored for SKILL.md directories (directory name is the canonical ID)`,
-              );
-            }
-            const canonicalId = dirName;
-
-            this.skills.push({
-              name: skillMeta.name,
-              description: this.cleanForCSV(skillMeta.description),
-              module: moduleName,
-              path: installPath,
-              canonicalId,
-              install_to_bmad: this.getInstallToBmad(manifest, skillFile),
-            });
-
-            // Add to files list
-            this.files.push({
-              type: 'skill',
-              name: skillMeta.name,
-              module: moduleName,
-              path: installPath,
-            });
-
-            this.skillClaimedDirs.add(dir);
-
-            if (debug) {
-              console.log(`[DEBUG] collectSkills: claimed skill "${skillMeta.name}" as ${canonicalId} at ${dir}`);
-            }
+        const skillMdPath = path.join(dir, skillFile);
+        const dirName = path.basename(dir);
+
+        const skillMeta = await this.parseSkillMd(skillMdPath, dir, dirName, debug);
+
+        if (skillMeta) {
+          // Load manifest when present (for install_to_bmad and agent metadata)
+          const manifest = await this.loadSkillManifest(dir);
+          const artifactType = this.getArtifactType(manifest, skillFile);
+
+          // Build path relative from module root (points to SKILL.md — the permanent entrypoint)
+          const relativePath = path.relative(modulePath, dir).split(path.sep).join('/');
+          const installPath = relativePath
+            ? `${this.bmadFolderName}/${moduleName}/${relativePath}/${skillFile}`
+            : `${this.bmadFolderName}/${moduleName}/${skillFile}`;
+
+          // Native SKILL.md entrypoints derive canonicalId from directory name.
+          // Agent entrypoints may keep canonicalId metadata for compatibility, so
+          // only warn for non-agent SKILL.md directories.
+          if (manifest && manifest.__single && manifest.__single.canonicalId && artifactType !== 'agent') {
+            console.warn(
+              `Warning: Native entrypoint manifest at ${dir}/bmad-skill-manifest.yaml contains canonicalId — this field is ignored for SKILL.md directories (directory name is the canonical ID)`,
+            );
           }
-        }
+          const canonicalId = dirName;
 
-        // Warn if manifest says this is a native entrypoint but the directory was not claimed
-        if (manifest && !this.skillClaimedDirs.has(dir)) {
-          let hasNativeSkillType = false;
-          if (manifest.__single) {
-            hasNativeSkillType = this.isNativeSkillDirType(manifest.__single.type);
-          } else {
-            for (const key of Object.keys(manifest)) {
-              if (this.isNativeSkillDirType(manifest[key]?.type)) {
-                hasNativeSkillType = true;
-                break;
-              }
-            }
-          }
-          if (hasNativeSkillType && debug) {
-            console.log(`[DEBUG] collectSkills: dir has native SKILL.md manifest but failed validation: ${dir}`);
+          this.skills.push({
+            name: skillMeta.name,
+            description: this.cleanForCSV(skillMeta.description),
+            module: moduleName,
+            path: installPath,
+            canonicalId,
+            install_to_bmad: this.getInstallToBmad(manifest, skillFile),
+          });
+
+          // Add to files list
+          this.files.push({
+            type: 'skill',
+            name: skillMeta.name,
+            module: moduleName,
+            path: installPath,
+          });
+
+          this.skillClaimedDirs.add(dir);
+
+          if (debug) {
+            console.log(`[DEBUG] collectSkills: claimed skill "${skillMeta.name}" as ${canonicalId} at ${dir}`);
           }
         }
 
@@ -515,7 +470,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 +527,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)
@@ -1384,11 +1339,10 @@ class ManifestGenerator {
         const hasTasks = await fs.pathExists(path.join(modulePath, 'tasks'));
         const hasTools = await fs.pathExists(path.join(modulePath, 'tools'));
 
-        // Check for native-entrypoint-only modules: recursive scan for
-        // bmad-skill-manifest.yaml with type: skill or type: agent
+        // Check for native-entrypoint-only modules: recursive scan for SKILL.md
         let hasSkills = false;
         if (!hasAgents && !hasWorkflows && !hasTasks && !hasTools) {
-          hasSkills = await this._hasSkillManifestRecursive(modulePath);
+          hasSkills = await this._hasSkillMdRecursive(modulePath);
         }
 
         // If it has any of these directories or skill manifests, it's likely a module
@@ -1404,13 +1358,12 @@ class ManifestGenerator {
   }
 
   /**
-   * Recursively check if a directory tree contains a bmad-skill-manifest.yaml that
-   * declares a native SKILL.md entrypoint (type: skill or type: agent).
+   * Recursively check if a directory tree contains a SKILL.md file.
    * Skips directories starting with . or _.
    * @param {string} dir - Directory to search
-   * @returns {boolean} True if a skill manifest is found
+   * @returns {boolean} True if a SKILL.md is found
    */
-  async _hasSkillManifestRecursive(dir) {
+  async _hasSkillMdRecursive(dir) {
     let entries;
     try {
       entries = await fs.readdir(dir, { withFileTypes: true });
@@ -1418,15 +1371,14 @@ class ManifestGenerator {
       return false;
     }
 
-    // Check for manifest in this directory
-    const manifest = await this.loadSkillManifest(dir);
-    if (this.hasNativeSkillManifest(manifest)) return true;
+    // Check for SKILL.md in this directory
+    if (entries.some((e) => !e.isDirectory() && e.name === 'SKILL.md')) return true;
 
     // Recurse into subdirectories
     for (const entry of entries) {
       if (!entry.isDirectory()) continue;
       if (entry.name.startsWith('.') || entry.name.startsWith('_')) continue;
-      if (await this._hasSkillManifestRecursive(path.join(dir, entry.name))) return true;
+      if (await this._hasSkillMdRecursive(path.join(dir, entry.name))) return true;
     }
 
     return false;