bmad-plus 0.8.0 → 0.9.1

package/tools/cli/commands/autoconfig.js

@@ -8,67 +8,12 @@
 
 const path = require('node:path');
 const fs = require('node:fs');
-const os = require('node:os');
-const fsExtra = require('fs-extra');
 const clack = require('@clack/prompts');
 const pc = require('picocolors');
+const { detectStack } = require('../lib/stack-detect');
 
 // ── Project Analysis Engine ──
 
-function detectStack(dir) {
-  const result = {
-    language: null,
-    framework: null,
-    runtime: null,
-    packageManager: null,
-    hasTypeScript: false,
-  };
-
-  // Package.json analysis
-  const pkgPath = path.join(dir, 'package.json');
-  if (fs.existsSync(pkgPath)) {
-    try {
-      const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
-      const deps = { ...pkg.dependencies, ...pkg.devDependencies };
-      result.runtime = 'Node.js';
-      result.language = deps['typescript'] || fs.existsSync(path.join(dir, 'tsconfig.json')) ? 'TypeScript' : 'JavaScript';
-      result.hasTypeScript = result.language === 'TypeScript';
-
-      // Framework detection
-      if (deps['next']) result.framework = 'Next.js';
-      else if (deps['nuxt']) result.framework = 'Nuxt';
-      else if (deps['@angular/core']) result.framework = 'Angular';
-      else if (deps['react']) result.framework = 'React';
-      else if (deps['vue']) result.framework = 'Vue.js';
-      else if (deps['svelte']) result.framework = 'Svelte';
-      else if (deps['express']) result.framework = 'Express';
-      else if (deps['fastify']) result.framework = 'Fastify';
-      else if (deps['hono']) result.framework = 'Hono';
-      else if (deps['electron']) result.framework = 'Electron';
-      else if (deps['tauri']) result.framework = 'Tauri';
-      else if (deps['react-native']) result.framework = 'React Native';
-
-      // Package manager
-      if (fs.existsSync(path.join(dir, 'pnpm-lock.yaml'))) result.packageManager = 'pnpm';
-      else if (fs.existsSync(path.join(dir, 'yarn.lock'))) result.packageManager = 'yarn';
-      else if (fs.existsSync(path.join(dir, 'bun.lockb'))) result.packageManager = 'bun';
-      else result.packageManager = 'npm';
-    } catch {}
-  }
-
-  // Other languages
-  if (!result.runtime) {
-    if (fs.existsSync(path.join(dir, 'Cargo.toml'))) { result.language = 'Rust'; result.runtime = 'Rust'; }
-    else if (fs.existsSync(path.join(dir, 'pyproject.toml')) || fs.existsSync(path.join(dir, 'requirements.txt'))) { result.language = 'Python'; result.runtime = 'Python'; }
-    else if (fs.existsSync(path.join(dir, 'go.mod'))) { result.language = 'Go'; result.runtime = 'Go'; }
-    else if (fs.existsSync(path.join(dir, 'composer.json'))) { result.language = 'PHP'; result.runtime = 'PHP'; }
-    else if (fs.existsSync(path.join(dir, 'Gemfile'))) { result.language = 'Ruby'; result.runtime = 'Ruby'; }
-    else if (fs.existsSync(path.join(dir, 'pom.xml')) || fs.existsSync(path.join(dir, 'build.gradle'))) { result.language = 'Java'; result.runtime = 'JVM'; }
-  }
-
-  return result;
-}
-
 function analyzeStructure(dir) {
   const structure = {
     hasSrc: false,
@@ -115,7 +60,9 @@ function analyzeStructure(dir) {
 
     // Check for .git directory
     if (fs.existsSync(path.join(dir, '.git'))) structure.hasGit = true;
-  } catch {}
+  } catch (e) {
+    console.warn('autoconfig: Failed to analyze directory structure', e.message);
+  }
 
   return structure;
 }
@@ -203,7 +150,9 @@ function getProjectName(dir) {
       const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
       if (pkg.name) return pkg.name;
     }
-  } catch {}
+  } catch (e) {
+    console.warn('autoconfig: Failed to read package.json in getProjectName', e.message);
+  }
   return path.basename(dir);
 }
 
@@ -241,6 +190,7 @@ module.exports = {
   description: 'Smart project bootstrap — auto-detect, install, and configure',
   options: [
     ['-d, --directory <path>', 'Project directory (default: current directory)'],
+    ['-l, --lang <code>', 'Language code: en, fr, es, de, pt-br, ru, zh, he, ja, it'],
     ['-y, --yes', 'Accept all recommendations without prompting'],
   ],
   action: async (options) => {
@@ -250,7 +200,7 @@ module.exports = {
 
     // Check if directory has content
     let entries = [];
-    try { entries = fs.readdirSync(projectDir).filter(e => !e.startsWith('.')); } catch {}
+    try { entries = fs.readdirSync(projectDir).filter(e => !e.startsWith('.')); } catch (e) { console.warn('autoconfig: Failed to list directory', e.message); }
 
     const isExistingProject = entries.length > 0;
 
@@ -341,9 +291,11 @@ module.exports = {
           packs: packs.join(','),
           yes: true,
           tools: toolsArg,
+          lang: options.lang,
         });
       } catch (e) {
-        // Install may have its own output, that's fine
+        clack.log.error(`Autoconfig install failed: ${e.message}`);
+        throw e;
       }
 
       spinner2.stop('Installation complete');
@@ -441,8 +393,12 @@ module.exports = {
           directory: projectDir,
           packs: packs.join(','),
           yes: true,
+          lang: options.lang,
         });
-      } catch {}
+      } catch (e) {
+        clack.log.error(`Autoconfig install failed: ${e.message}`);
+        throw e;
+      }
 
       // Write initial context
       const contextPath = path.join(projectDir, '.agents', 'memory', 'context.md');
@@ -494,5 +450,6 @@ module.exports = {
     recommendPacks,
     generateRecommendations,
     getProjectName,
+    newProjectWizard,
   },
 };

package/tools/cli/commands/doctor.js

@@ -10,12 +10,14 @@ const fs = require('node:fs');
 const clack = require('@clack/prompts');
 const pc = require('picocolors');
 const { t } = require('../i18n');
+const { PACKS, EXPECTED_AGENTS } = require('../lib/packs');
 
 module.exports = {
   command: 'doctor',
   description: 'Check BMAD+ installation integrity',
   options: [
     ['-d, --directory <path>', 'Project directory (default: current directory)'],
+    ['-l, --lang <code>', 'Language code: en, fr, es, de, pt-br, ru, zh, he, ja, it'],
   ],
   action: async (options) => {
     const projectDir = path.resolve(options.directory || process.cwd());
@@ -38,6 +40,7 @@ module.exports = {
     }
 
     const manifest = JSON.parse(fs.readFileSync(manifestPath, 'utf8'));
+    const lang = options.lang || manifest.uiLanguage || 'en';
     clack.log.success(`✅ Manifest found — v${manifest.version}`);
     passed++;
 
@@ -62,19 +65,13 @@ module.exports = {
       clack.log.success(`✅ ${agentDirs.length} agent/skill directories found`);
       passed++;
 
-      // Check each expected agent from manifest packs
-      const expectedAgents = {
-        core: ['agent-strategist', 'agent-architect-dev', 'agent-quality', 'agent-orchestrator'],
-        osint: ['agent-shadow'],
-        maker: ['agent-maker'],
-        seo: ['pack-seo'],
-        backup: ['pack-backup'],
-        animated: ['pack-animated'],
-      };
-
+      // Check each expected agent from manifest packs using shared EXPECTED_AGENTS module
       for (const pack of (manifest.packs || ['core'])) {
-        const expected = expectedAgents[pack] || [];
-        for (const agent of expected) {
+        const entry = EXPECTED_AGENTS[pack];
+        if (!entry) continue;
+
+        // Check individual agent directories
+        for (const agent of (entry.agents || [])) {
           checks++;
           const agentPath = path.join(agentsDir, agent);
           if (fs.existsSync(agentPath)) {
@@ -84,6 +81,18 @@ module.exports = {
             warnings++;
           }
         }
+
+        // Check pack directory if applicable
+        if (entry.packDir) {
+          checks++;
+          const packPath = path.join(agentsDir, entry.packDir);
+          if (fs.existsSync(packPath)) {
+            passed++;
+          } else {
+            clack.log.warn(`⚠️  Missing pack directory: ${entry.packDir} (pack: ${pack})`);
+            warnings++;
+          }
+        }
       }
     } else {
       clack.log.error('❌ No .agents/skills/ directory found');
@@ -153,37 +162,27 @@ module.exports = {
     checks++;
     try {
       const yaml = require('js-yaml');
-      const installModule = require('./install');
-      const installSrc = fs.readFileSync(path.join(__dirname, 'install.js'), 'utf8');
-      const packsMatch = installSrc.match(/const PACKS\s*=\s*\{/);
-
       const moduleYamlSrc = path.join(__dirname, '..', '..', '..', 'src', 'bmad-plus', 'module.yaml');
-      if (packsMatch && fs.existsSync(moduleYamlSrc)) {
+      if (fs.existsSync(moduleYamlSrc)) {
         const moduleContent = yaml.load(fs.readFileSync(moduleYamlSrc, 'utf8'));
         const modulePackIds = Object.keys(moduleContent.packs || {});
 
-        // Extract PACKS keys from install.js via require
-        // The PACKS keys are the pack IDs available in the CLI menu
-        // We compare against module.yaml packs
-        const installContent = fs.readFileSync(path.join(__dirname, 'install.js'), 'utf8');
-        const packKeyMatches = installContent.match(/^\s+'?([a-z][-a-z]*)'?\s*:\s*\{/gm);
-        const installPackIds = packKeyMatches
-          ? packKeyMatches.map(m => m.trim().replace(/[':{ ]/g, '').replace(/-/g, '-'))
-          : [];
+        // Use shared PACKS module instead of fragile regex on install.js source
+        const cliPackIds = Object.keys(PACKS);
 
         // Find mismatches
-        const missingInInstall = modulePackIds.filter(p => !installPackIds.includes(p));
-        const missingInModule = installPackIds.filter(p => !modulePackIds.includes(p));
+        const missingInCLI = modulePackIds.filter(p => !cliPackIds.includes(p));
+        const missingInModule = cliPackIds.filter(p => !modulePackIds.includes(p));
 
-        if (missingInInstall.length === 0 && missingInModule.length === 0) {
+        if (missingInCLI.length === 0 && missingInModule.length === 0) {
           clack.log.success(`✅ PACKS ↔ module.yaml in sync (${modulePackIds.length} packs)`);
           passed++;
         } else {
-          if (missingInInstall.length > 0) {
-            clack.log.warn(`⚠️  Packs in module.yaml but missing from install.js PACKS: ${missingInInstall.join(', ')}`);
+          if (missingInCLI.length > 0) {
+            clack.log.warn(`⚠️  Packs in module.yaml but missing from CLI PACKS: ${missingInCLI.join(', ')}`);
           }
           if (missingInModule.length > 0) {
-            clack.log.warn(`⚠️  Packs in install.js PACKS but missing from module.yaml: ${missingInModule.join(', ')}`);
+            clack.log.warn(`⚠️  Packs in CLI PACKS but missing from module.yaml: ${missingInModule.join(', ')}`);
           }
           warnings++;
         }