bmad-method 6.2.2 → 6.2.3-next.1

package/tools/{cli/installers/lib → installer}/core/manifest-generator.js

@@ -3,8 +3,8 @@ const fs = require('fs-extra');
 const yaml = require('yaml');
 const crypto = require('node:crypto');
 const csv = require('csv-parse/sync');
-const { getSourcePath, getModulePath } = require('../../../lib/project-root');
-const prompts = require('../../../lib/prompts');
+const { getSourcePath, getModulePath } = require('../project-root');
+const prompts = require('../prompts');
 const {
   loadSkillManifest: loadSkillManifestShared,
   getCanonicalId: getCanonicalIdShared,
@@ -13,7 +13,7 @@ const {
 } = require('../ide/shared/skill-manifest');
 
 // Load package.json for version info
-const packageJson = require('../../../../../package.json');
+const packageJson = require('../../../package.json');
 
 /**
  * Generates manifest files for installed skills and agents

package/tools/{cli/installers/lib → installer}/core/manifest.js

@@ -1,8 +1,8 @@
 const path = require('node:path');
 const fs = require('fs-extra');
 const crypto = require('node:crypto');
-const { getProjectRoot } = require('../../../lib/project-root');
-const prompts = require('../../../lib/prompts');
+const { getProjectRoot } = require('../project-root');
+const prompts = require('../prompts');
 
 class Manifest {
   /**

package/tools/{cli/installers/lib/custom/handler.js → installer/custom-handler.js}

@@ -1,7 +1,7 @@
 const path = require('node:path');
 const fs = require('fs-extra');
 const yaml = require('yaml');
-const prompts = require('../../../lib/prompts');
+const prompts = require('./prompts');
 /**
  * Handler for custom content (custom.yaml)
  * Discovers custom agents and workflows in the project

package/tools/{cli/installers/lib → installer}/ide/_config-driven.js

@@ -2,9 +2,9 @@ const os = require('node:os');
 const path = require('node:path');
 const fs = require('fs-extra');
 const yaml = require('yaml');
-const { BaseIdeSetup } = require('./_base-ide');
-const prompts = require('../../../lib/prompts');
+const prompts = require('../prompts');
 const csv = require('csv-parse/sync');
+const { BMAD_FOLDER_NAME } = require('./shared/path-utils');
 
 /**
  * Config-driven IDE setup handler
@@ -15,43 +15,45 @@ const csv = require('csv-parse/sync');
  *
  * Features:
  * - Config-driven from platform-codes.yaml
- * - Template-based content generation
- * - Multi-target installation support (e.g., GitHub Copilot)
- * - Artifact type filtering (agents, workflows, tasks, tools)
+ * - Verbatim skill installation from skill-manifest.csv
+ * - Legacy directory cleanup and IDE-specific marker removal
  */
-class ConfigDrivenIdeSetup extends BaseIdeSetup {
+class ConfigDrivenIdeSetup {
   constructor(platformCode, platformConfig) {
-    super(platformCode, platformConfig.name, platformConfig.preferred);
+    this.name = platformCode;
+    this.displayName = platformConfig.name || platformCode;
+    this.preferred = platformConfig.preferred || false;
     this.platformConfig = platformConfig;
     this.installerConfig = platformConfig.installer || null;
+    this.bmadFolderName = BMAD_FOLDER_NAME;
 
-    // Set configDir from target_dir so base-class detect() works
-    if (this.installerConfig?.target_dir) {
-      this.configDir = this.installerConfig.target_dir;
-    }
+    // Set configDir from target_dir so detect() works
+    this.configDir = this.installerConfig?.target_dir || null;
+  }
+
+  setBmadFolderName(bmadFolderName) {
+    this.bmadFolderName = bmadFolderName;
   }
 
   /**
    * Detect whether this IDE already has configuration in the project.
-   * For skill_format platforms, checks for bmad-prefixed entries in target_dir
-   * (matching old codex.js behavior) instead of just checking directory existence.
+   * Checks for bmad-prefixed entries in target_dir.
    * @param {string} projectDir - Project directory
    * @returns {Promise<boolean>}
    */
   async detect(projectDir) {
-    if (this.installerConfig?.skill_format && this.configDir) {
-      const dir = path.join(projectDir || process.cwd(), this.configDir);
-      if (await fs.pathExists(dir)) {
-        try {
-          const entries = await fs.readdir(dir);
-          return entries.some((e) => typeof e === 'string' && e.startsWith('bmad'));
-        } catch {
-          return false;
-        }
+    if (!this.configDir) return false;
+
+    const dir = path.join(projectDir || process.cwd(), this.configDir);
+    if (await fs.pathExists(dir)) {
+      try {
+        const entries = await fs.readdir(dir);
+        return entries.some((e) => typeof e === 'string' && e.startsWith('bmad'));
+      } catch {
+        return false;
       }
-      return false;
     }
-    return super.detect(projectDir);
+    return false;
   }
 
   /**
@@ -90,12 +92,6 @@ class ConfigDrivenIdeSetup extends BaseIdeSetup {
       return { success: false, reason: 'no-config' };
     }
 
-    // Handle multi-target installations (e.g., GitHub Copilot)
-    if (this.installerConfig.targets) {
-      return this.installToMultipleTargets(projectDir, bmadDir, this.installerConfig.targets, options);
-    }
-
-    // Handle single-target installations
     if (this.installerConfig.target_dir) {
       return this.installToTarget(projectDir, bmadDir, this.installerConfig, options);
     }
@@ -113,13 +109,8 @@ class ConfigDrivenIdeSetup extends BaseIdeSetup {
    */
   async installToTarget(projectDir, bmadDir, config, options) {
     const { target_dir } = config;
-
-    if (!config.skill_format) {
-      return { success: false, reason: 'missing-skill-format', error: 'Installer config missing skill_format — cannot install skills' };
-    }
-
     const targetPath = path.join(projectDir, target_dir);
-    await this.ensureDir(targetPath);
+    await fs.ensureDir(targetPath);
 
     this.skillWriteTracker = new Set();
     const results = { skills: 0 };
@@ -132,351 +123,6 @@ class ConfigDrivenIdeSetup extends BaseIdeSetup {
     return { success: true, results };
   }
 
-  /**
-   * Install to multiple target directories
-   * @param {string} projectDir - Project directory
-   * @param {string} bmadDir - BMAD installation directory
-   * @param {Array} targets - Array of target configurations
-   * @param {Object} options - Setup options
-   * @returns {Promise<Object>} Installation result
-   */
-  async installToMultipleTargets(projectDir, bmadDir, targets, options) {
-    const allResults = { skills: 0 };
-
-    for (const target of targets) {
-      const result = await this.installToTarget(projectDir, bmadDir, target, options);
-      if (result.success) {
-        allResults.skills += result.results.skills || 0;
-      }
-    }
-
-    return { success: true, results: allResults };
-  }
-
-  /**
-   * Load template based on type and configuration
-   * @param {string} templateType - Template type (claude, windsurf, etc.)
-   * @param {string} artifactType - Artifact type (agent, workflow, task, tool)
-   * @param {Object} config - Installation configuration
-   * @param {string} fallbackTemplateType - Fallback template type if requested template not found
-   * @returns {Promise<{content: string, extension: string}>} Template content and extension
-   */
-  async loadTemplate(templateType, artifactType, config = {}, fallbackTemplateType = null) {
-    const { header_template, body_template } = config;
-
-    // Check for separate header/body templates
-    if (header_template || body_template) {
-      const content = await this.loadSplitTemplates(templateType, artifactType, header_template, body_template);
-      // Allow config to override extension, default to .md
-      const ext = config.extension || '.md';
-      const normalizedExt = ext.startsWith('.') ? ext : `.${ext}`;
-      return { content, extension: normalizedExt };
-    }
-
-    // Load combined template - try multiple extensions
-    // If artifactType is empty, templateType already contains full name (e.g., 'gemini-workflow-yaml')
-    const templateBaseName = artifactType ? `${templateType}-${artifactType}` : templateType;
-    const templateDir = path.join(__dirname, 'templates', 'combined');
-    const extensions = ['.md', '.toml', '.yaml', '.yml'];
-
-    for (const ext of extensions) {
-      const templatePath = path.join(templateDir, templateBaseName + ext);
-      if (await fs.pathExists(templatePath)) {
-        const content = await fs.readFile(templatePath, 'utf8');
-        return { content, extension: ext };
-      }
-    }
-
-    // Fall back to default template (if provided)
-    if (fallbackTemplateType) {
-      for (const ext of extensions) {
-        const fallbackPath = path.join(templateDir, `${fallbackTemplateType}${ext}`);
-        if (await fs.pathExists(fallbackPath)) {
-          const content = await fs.readFile(fallbackPath, 'utf8');
-          return { content, extension: ext };
-        }
-      }
-    }
-
-    // Ultimate fallback - minimal template
-    return { content: this.getDefaultTemplate(artifactType), extension: '.md' };
-  }
-
-  /**
-   * Load split templates (header + body)
-   * @param {string} templateType - Template type
-   * @param {string} artifactType - Artifact type
-   * @param {string} headerTpl - Header template name
-   * @param {string} bodyTpl - Body template name
-   * @returns {Promise<string>} Combined template content
-   */
-  async loadSplitTemplates(templateType, artifactType, headerTpl, bodyTpl) {
-    let header = '';
-    let body = '';
-
-    // Load header template
-    if (headerTpl) {
-      const headerPath = path.join(__dirname, 'templates', 'split', headerTpl);
-      if (await fs.pathExists(headerPath)) {
-        header = await fs.readFile(headerPath, 'utf8');
-      }
-    } else {
-      // Use default header for template type
-      const defaultHeaderPath = path.join(__dirname, 'templates', 'split', templateType, 'header.md');
-      if (await fs.pathExists(defaultHeaderPath)) {
-        header = await fs.readFile(defaultHeaderPath, 'utf8');
-      }
-    }
-
-    // Load body template
-    if (bodyTpl) {
-      const bodyPath = path.join(__dirname, 'templates', 'split', bodyTpl);
-      if (await fs.pathExists(bodyPath)) {
-        body = await fs.readFile(bodyPath, 'utf8');
-      }
-    } else {
-      // Use default body for template type
-      const defaultBodyPath = path.join(__dirname, 'templates', 'split', templateType, 'body.md');
-      if (await fs.pathExists(defaultBodyPath)) {
-        body = await fs.readFile(defaultBodyPath, 'utf8');
-      }
-    }
-
-    // Combine header and body
-    return `${header}\n${body}`;
-  }
-
-  /**
-   * Get default minimal template
-   * @param {string} artifactType - Artifact type
-   * @returns {string} Default template
-   */
-  getDefaultTemplate(artifactType) {
-    if (artifactType === 'agent') {
-      return `---
-name: '{{name}}'
-description: '{{description}}'
-disable-model-invocation: true
----
-
-You must fully embody this agent's persona and follow all activation instructions exactly as specified.
-
-<agent-activation CRITICAL="TRUE">
-1. LOAD the FULL agent file from {project-root}/{{bmadFolderName}}/{{path}}
-2. READ its entire contents - this contains the complete agent persona, menu, and instructions
-3. FOLLOW every step in the <activation> section precisely
-</agent-activation>
-`;
-    }
-    return `---
-name: '{{name}}'
-description: '{{description}}'
----
-
-# {{name}}
-
-LOAD and execute from: {project-root}/{{bmadFolderName}}/{{path}}
-`;
-  }
-
-  /**
-   * Render template with artifact data
-   * @param {string} template - Template content
-   * @param {Object} artifact - Artifact data
-   * @returns {string} Rendered content
-   */
-  renderTemplate(template, artifact) {
-    // Use the appropriate path property based on artifact type
-    let pathToUse = artifact.relativePath || '';
-    switch (artifact.type) {
-      case 'agent-launcher': {
-        pathToUse = artifact.agentPath || artifact.relativePath || '';
-
-        break;
-      }
-      case 'workflow-command': {
-        pathToUse = artifact.workflowPath || artifact.relativePath || '';
-
-        break;
-      }
-      case 'task':
-      case 'tool': {
-        pathToUse = artifact.path || artifact.relativePath || '';
-
-        break;
-      }
-      // No default
-    }
-
-    // Replace _bmad placeholder with actual folder name BEFORE inserting paths,
-    // so that paths containing '_bmad' are not corrupted by the blanket replacement.
-    let rendered = template.replaceAll('_bmad', this.bmadFolderName);
-
-    // Replace {{bmadFolderName}} placeholder if present
-    rendered = rendered.replaceAll('{{bmadFolderName}}', this.bmadFolderName);
-
-    rendered = rendered
-      .replaceAll('{{name}}', artifact.name || '')
-      .replaceAll('{{module}}', artifact.module || 'core')
-      .replaceAll('{{path}}', pathToUse)
-      .replaceAll('{{description}}', artifact.description || `${artifact.name} ${artifact.type || ''}`)
-      .replaceAll('{{workflow_path}}', pathToUse);
-
-    return rendered;
-  }
-
-  /**
-   * Write artifact as a skill directory with SKILL.md inside.
-   * Writes artifact as a skill directory with SKILL.md inside.
-   * @param {string} targetPath - Base skills directory
-   * @param {Object} artifact - Artifact data
-   * @param {string} content - Rendered template content
-   */
-  async writeSkillFile(targetPath, artifact, content) {
-    const { resolveSkillName } = require('./shared/path-utils');
-
-    // Get the skill name (prefers canonicalId, falls back to path-derived) and remove .md
-    const flatName = resolveSkillName(artifact);
-    const skillName = path.basename(flatName.replace(/\.md$/, ''));
-
-    if (!skillName) {
-      throw new Error(`Cannot derive skill name for artifact: ${artifact.relativePath || JSON.stringify(artifact)}`);
-    }
-
-    // Create skill directory
-    const skillDir = path.join(targetPath, skillName);
-    await this.ensureDir(skillDir);
-    this.skillWriteTracker?.add(skillName);
-
-    // Transform content: rewrite frontmatter for skills format
-    const skillContent = this.transformToSkillFormat(content, skillName);
-
-    await this.writeFile(path.join(skillDir, 'SKILL.md'), skillContent);
-  }
-
-  /**
-   * Transform artifact content to Agent Skills format.
-   * Rewrites frontmatter to contain only unquoted name and description.
-   * @param {string} content - Original content with YAML frontmatter
-   * @param {string} skillName - Skill name (must match directory name)
-   * @returns {string} Transformed content
-   */
-  transformToSkillFormat(content, skillName) {
-    // Normalize line endings
-    content = content.replaceAll('\r\n', '\n').replaceAll('\r', '\n');
-
-    // Parse frontmatter
-    const fmMatch = content.match(/^---\n([\s\S]*?)\n---\n?([\s\S]*)$/);
-    if (!fmMatch) {
-      // No frontmatter -- wrap with minimal frontmatter
-      const fm = yaml.stringify({ name: skillName, description: skillName }).trimEnd();
-      return `---\n${fm}\n---\n\n${content}`;
-    }
-
-    const frontmatter = fmMatch[1];
-    const body = fmMatch[2];
-
-    // Parse frontmatter with yaml library to extract description
-    let description;
-    try {
-      const parsed = yaml.parse(frontmatter);
-      const rawDesc = parsed?.description;
-      description = typeof rawDesc === 'string' && rawDesc ? rawDesc : `${skillName} skill`;
-    } catch {
-      description = `${skillName} skill`;
-    }
-
-    // Build new frontmatter with only name and description, unquoted
-    const newFrontmatter = yaml.stringify({ name: skillName, description: String(description) }, { lineWidth: 0 }).trimEnd();
-    return `---\n${newFrontmatter}\n---\n${body}`;
-  }
-
-  /**
-   * Install a custom agent launcher.
-   * For skill_format platforms, produces <skillDir>/SKILL.md.
-   * For flat platforms, produces a single file in target_dir.
-   * @param {string} projectDir - Project directory
-   * @param {string} agentName - Agent name (e.g., "fred-commit-poet")
-   * @param {string} agentPath - Path to compiled agent (relative to project root)
-   * @param {Object} metadata - Agent metadata
-   * @returns {Object|null} Info about created file/skill
-   */
-  async installCustomAgentLauncher(projectDir, agentName, agentPath, metadata) {
-    if (!this.installerConfig?.target_dir) return null;
-
-    const { customAgentDashName } = require('./shared/path-utils');
-    const targetPath = path.join(projectDir, this.installerConfig.target_dir);
-    await this.ensureDir(targetPath);
-
-    // Build artifact to reuse existing template rendering.
-    // The default-agent template already includes the _bmad/ prefix before {{path}},
-    // but agentPath is relative to project root (e.g. "_bmad/custom/agents/fred.md").
-    // Strip the bmadFolderName prefix so the template doesn't produce a double path.
-    const bmadPrefix = this.bmadFolderName + '/';
-    const normalizedPath = agentPath.startsWith(bmadPrefix) ? agentPath.slice(bmadPrefix.length) : agentPath;
-
-    const artifact = {
-      type: 'agent-launcher',
-      name: agentName,
-      description: metadata?.description || `${agentName} agent`,
-      agentPath: normalizedPath,
-      relativePath: normalizedPath,
-      module: 'custom',
-    };
-
-    const { content: template } = await this.loadTemplate(
-      this.installerConfig.template_type || 'default',
-      'agent',
-      this.installerConfig,
-      'default-agent',
-    );
-    const content = this.renderTemplate(template, artifact);
-
-    if (this.installerConfig.skill_format) {
-      const skillName = customAgentDashName(agentName).replace(/\.md$/, '');
-      const skillDir = path.join(targetPath, skillName);
-      await this.ensureDir(skillDir);
-      const skillContent = this.transformToSkillFormat(content, skillName);
-      const skillPath = path.join(skillDir, 'SKILL.md');
-      await this.writeFile(skillPath, skillContent);
-      return { path: path.relative(projectDir, skillPath), command: `$${skillName}` };
-    }
-
-    // Flat file output
-    const filename = customAgentDashName(agentName);
-    const filePath = path.join(targetPath, filename);
-    await this.writeFile(filePath, content);
-    return { path: path.relative(projectDir, filePath), command: agentName };
-  }
-
-  /**
-   * Generate filename for artifact
-   * @param {Object} artifact - Artifact data
-   * @param {string} artifactType - Artifact type (agent, workflow, task, tool)
-   * @param {string} extension - File extension to use (e.g., '.md', '.toml')
-   * @returns {string} Generated filename
-   */
-  generateFilename(artifact, artifactType, extension = '.md') {
-    const { resolveSkillName } = require('./shared/path-utils');
-
-    // Reuse central logic to ensure consistent naming conventions
-    // Prefers canonicalId from manifest when available, falls back to path-derived name
-    const standardName = resolveSkillName(artifact);
-
-    // Clean up potential double extensions from source files (e.g. .yaml.md, .xml.md -> .md)
-    // This handles any extensions that might slip through toDashPath()
-    const baseName = standardName.replace(/\.(md|yaml|yml|json|xml|toml)\.md$/i, '.md');
-
-    // If using default markdown, preserve the bmad-agent- prefix for agents
-    if (extension === '.md') {
-      return baseName;
-    }
-
-    // For other extensions (e.g., .toml), replace .md extension
-    // Note: agent prefix is preserved even with non-markdown extensions
-    return baseName.replace(/\.md$/, extension);
-  }
-
   /**
    * Install verbatim native SKILL.md directories from skill-manifest.csv.
    * Copies the entire source directory as-is into the IDE skill directory.
@@ -598,22 +244,8 @@ LOAD and execute from: {project-root}/{{bmadFolderName}}/{{path}}
       await this.cleanupRovoDevPrompts(projectDir, options);
     }
 
-    // Clean all target directories
-    if (this.installerConfig?.targets) {
-      const parentDirs = new Set();
-      for (const target of this.installerConfig.targets) {
-        await this.cleanupTarget(projectDir, target.target_dir, options);
-        // Track parent directories for empty-dir cleanup
-        const parentDir = path.dirname(target.target_dir);
-        if (parentDir && parentDir !== '.') {
-          parentDirs.add(parentDir);
-        }
-      }
-      // After all targets cleaned, remove empty parent directories (recursive up to projectDir)
-      for (const parentDir of parentDirs) {
-        await this.removeEmptyParents(projectDir, parentDir);
-      }
-    } else if (this.installerConfig?.target_dir) {
+    // Clean target directory
+    if (this.installerConfig?.target_dir) {
       await this.cleanupTarget(projectDir, this.installerConfig.target_dir, options);
     }
   }
@@ -711,6 +343,7 @@ LOAD and execute from: {project-root}/{{bmadFolderName}}/{{path}}
       }
     }
   }
+
   /**
    * Strip BMAD-owned content from .github/copilot-instructions.md.
    * The old custom installer injected content between <!-- BMAD:START --> and <!-- BMAD:END --> markers.

package/tools/{cli/installers/lib → installer}/ide/manager.js

@@ -1,5 +1,5 @@
 const { BMAD_FOLDER_NAME } = require('./shared/path-utils');
-const prompts = require('../../../lib/prompts');
+const prompts = require('../prompts');
 
 /**
  * IDE Manager - handles IDE-specific setup
@@ -226,23 +226,6 @@ class IdeManager {
     return results;
   }
 
-  /**
-   * Get list of supported IDEs
-   * @returns {Array} List of supported IDE names
-   */
-  getSupportedIdes() {
-    return [...this.handlers.keys()];
-  }
-
-  /**
-   * Check if an IDE is supported
-   * @param {string} ideName - Name of the IDE
-   * @returns {boolean} True if IDE is supported
-   */
-  isSupported(ideName) {
-    return this.handlers.has(ideName.toLowerCase());
-  }
-
   /**
    * Detect installed IDEs
    * @param {string} projectDir - Project directory
@@ -259,41 +242,6 @@ class IdeManager {
 
     return detected;
   }
-
-  /**
-   * Install custom agent launchers for specified IDEs
-   * @param {Array} ides - List of IDE names to install for
-   * @param {string} projectDir - Project directory
-   * @param {string} agentName - Agent name (e.g., "fred-commit-poet")
-   * @param {string} agentPath - Path to compiled agent (relative to project root)
-   * @param {Object} metadata - Agent metadata
-   * @returns {Object} Results for each IDE
-   */
-  async installCustomAgentLaunchers(ides, projectDir, agentName, agentPath, metadata) {
-    const results = {};
-
-    for (const ideName of ides) {
-      const handler = this.handlers.get(ideName.toLowerCase());
-
-      if (!handler) {
-        await prompts.log.warn(`IDE '${ideName}' is not yet supported for custom agent installation`);
-        continue;
-      }
-
-      try {
-        if (typeof handler.installCustomAgentLauncher === 'function') {
-          const result = await handler.installCustomAgentLauncher(projectDir, agentName, agentPath, metadata);
-          if (result) {
-            results[ideName] = result;
-          }
-        }
-      } catch (error) {
-        await prompts.log.warn(`Failed to install ${ideName} launcher: ${error.message}`);
-      }
-    }
-
-    return results;
-  }
 }
 
 module.exports = { IdeManager };

package/tools/installer/ide/platform-codes.js

@@ -0,0 +1,37 @@
+const fs = require('fs-extra');
+const path = require('node:path');
+const yaml = require('yaml');
+
+const PLATFORM_CODES_PATH = path.join(__dirname, 'platform-codes.yaml');
+
+let _cachedPlatformCodes = null;
+
+/**
+ * Load the platform codes configuration from YAML
+ * @returns {Object} Platform codes configuration
+ */
+async function loadPlatformCodes() {
+  if (_cachedPlatformCodes) {
+    return _cachedPlatformCodes;
+  }
+
+  if (!(await fs.pathExists(PLATFORM_CODES_PATH))) {
+    throw new Error(`Platform codes configuration not found at: ${PLATFORM_CODES_PATH}`);
+  }
+
+  const content = await fs.readFile(PLATFORM_CODES_PATH, 'utf8');
+  _cachedPlatformCodes = yaml.parse(content);
+  return _cachedPlatformCodes;
+}
+
+/**
+ * Clear the cached platform codes (useful for testing)
+ */
+function clearCache() {
+  _cachedPlatformCodes = null;
+}
+
+module.exports = {
+  loadPlatformCodes,
+  clearCache,
+};