bmad-plus 0.13.0 → 0.14.0

package/tools/cli/commands/doctor.js

@@ -1,205 +1,65 @@
-/**
- * BMAD+ Doctor Command
- * Checks installation integrity and reports issues
- *
- * Author: Laurent Rochetta
- */
-
-const path = require('node:path');
-const fs = require('node:fs');
+/** Human and JSON output share one report; runtime execution is explicitly opt-in. */
 const clack = require('@clack/prompts');
 const pc = require('picocolors');
-const { t } = require('../i18n');
-const { EXPECTED_AGENTS } = require('../lib/packs');
-const { readInstallManifest } = require('../lib/install-manifest');
+const { collectInstallationHealth } = require('../lib/installation-health');
 
 module.exports = {
   command: 'doctor',
-  description: 'Check BMAD+ installation integrity',
+  description: 'Check BMAD+ installation integrity and declared capabilities',
   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'],
+    ['--json', 'Print the diagnostic report as JSON (read-only unless a probe is requested)'],
+    ['--verify-python', 'Execute installed Python interpreters to verify versions and imports'],
   ],
-  action: async (options) => {
-    const projectDir = path.resolve(options.directory || process.cwd());
-    const packageJson = require('../../../package.json');
-
-    clack.intro(pc.bgBlue(pc.white(` BMAD+ Doctor v${packageJson.version} `)));
-
-    let checks = 0;
-    let passed = 0;
-    let warnings = 0;
-    let errors = 0;
-
-    // ── Check 1: Installation manifest ──
-    checks++;
-    const manifestPath = path.join(projectDir, '_bmad', '.bmad-plus-install.json');
-    if (!fs.existsSync(manifestPath)) {
-      clack.log.error('❌ BMAD+ is not installed in this directory');
-      clack.outro(pc.red('Run `npx bmad-plus install` first.'));
-      return;
-    }
-
-    let manifest;
-    try {
-      manifest = readInstallManifest(manifestPath);
-    } catch (err) {
-      const i = t(options.lang || 'en');
-      clack.log.error(i.manifest_invalid(err.message));
-      clack.outro(pc.red(i.manifest_repair));
-      process.exitCode = 1;
+  action: async (options = {}) => {
+    const report = collectInstallationHealth({
+      projectDir: options.directory,
+      verifyPython: Boolean(options.verifyPython),
+    });
+    process.exitCode = report.summary.errors ? 1 : 0;
+    if (options.json) {
+      console.log(JSON.stringify(report, null, 2));
       return;
     }
-    clack.log.success(`✅ Manifest found — v${manifest.version}`);
-    passed++;
-
-    // ── Check 2: Version comparison ──
-    checks++;
-    if (manifest.version === packageJson.version) {
-      clack.log.success(`✅ Version up to date (v${manifest.version})`);
-      passed++;
-    } else {
-      clack.log.warn(`⚠️  Version mismatch: installed v${manifest.version} → latest v${packageJson.version}`);
-      clack.log.info('   Run `npx bmad-plus update` to upgrade');
-      warnings++;
+    clack.intro(pc.bgBlue(pc.white(' BMAD+ Doctor v' + report.versions.cli + ' ')));
+    for (const entry of report.checks.filter(
+      (item) => item.status !== 'pass' || item.code === 'manifest.valid'
+    )) {
+      const log =
+        entry.status === 'error'
+          ? clack.log.error
+          : entry.status === 'warning'
+            ? clack.log.warn
+            : clack.log.success;
+      log(
+        entry.message +
+          (entry.path && !entry.message.includes(entry.path) ? ' (' + entry.path + ')' : '')
+      );
     }
-
-    // ── Check 3: Agent files ──
-    checks++;
-    const agentsDir = path.join(projectDir, '.agents', 'skills');
-    if (fs.existsSync(agentsDir)) {
-      const agentDirs = fs.readdirSync(agentsDir, { withFileTypes: true })
-        .filter(d => d.isDirectory())
-        .map(d => d.name);
-      clack.log.success(`✅ ${agentDirs.length} agent/skill directories found`);
-      passed++;
-
-      // Check each expected agent from manifest packs using shared EXPECTED_AGENTS module
-      for (const pack of (manifest.packs || ['core'])) {
-        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)) {
-            passed++;
-          } else {
-            clack.log.warn(`⚠️  Missing agent: ${agent} (pack: ${pack})`);
-            warnings++;
-          }
-        }
-
-        // Check pack directory if applicable
-        if (entry.packDir) {
-          checks++;
-          const packPath = path.join(agentsDir, entry.packDir);
-          if (fs.existsSync(packPath)) {
-            passed++;
-
-            // These packs bundle their agents as files inside packDir — verify
-            // them there (not as loose directories, which would falsely warn).
-            for (const agentFile of (entry.packAgents || [])) {
-              checks++;
-              if (fs.existsSync(path.join(packPath, agentFile))) {
-                passed++;
-              } else {
-                clack.log.warn(`⚠️  Missing agent: ${agentFile} (pack: ${pack})`);
-                warnings++;
-              }
-            }
-          } else {
-            clack.log.warn(`⚠️  Missing pack directory: ${entry.packDir} (pack: ${pack})`);
-            warnings++;
-          }
-        }
-      }
-    } else {
-      clack.log.error('❌ No .agents/skills/ directory found');
-      errors++;
+    if (report.versions.installed) {
+      clack.note(
+        [
+          report.summary.passed + '/' + report.checks.length + ' checks passed',
+          report.summary.errors + ' error(s), ' + report.summary.warnings + ' warning(s)',
+          'Installed: v' + report.versions.installed + ' | Executing CLI: v' + report.versions.cli,
+          'Ownership inventory: ' + report.inventory.status,
+          'Integrations use host-managed execution; host capabilities were not probed.',
+        ].join('\n'),
+        'BMAD+ Health Report'
+      );
     }
-
-    // ── Check 4: Config files ──
-    checks++;
-    const configPath = path.join(projectDir, '_bmad', 'config.yaml');
-    if (fs.existsSync(configPath)) {
-      clack.log.success('✅ config.yaml present');
-      passed++;
-    } else {
-      clack.log.error('❌ config.yaml missing');
-      errors++;
-    }
-
-    // ── Check 5: Module config ──
-    checks++;
-    const moduleYaml = path.join(projectDir, '_bmad', 'module.yaml');
-    if (fs.existsSync(moduleYaml)) {
-      clack.log.success('✅ module.yaml present');
-      passed++;
-    } else {
-      clack.log.warn('⚠️  module.yaml missing');
-      warnings++;
-    }
-
-    // ── Check 6: IDE configs ──
-    checks++;
-    const ideFiles = ['CLAUDE.md', 'GEMINI.md', 'AGENTS.md'].filter(f =>
-      fs.existsSync(path.join(projectDir, f))
+    clack.log.info(
+      'Published version is not checked here. Use `bmad-plus update-check --json` to check npm.'
+    );
+    clack.outro(
+      report.status === 'ok'
+        ? pc.green('Installation checks passed.')
+        : pc.yellow(
+            options.verifyPython
+              ? 'Review the reported evidence. Python import probes ran; no provisioning was requested.'
+              : 'Review the reported evidence before changing files. Doctor did not modify the project.'
+          )
     );
-    if (ideFiles.length > 0) {
-      clack.log.success(`✅ ${ideFiles.length} IDE config(s): ${ideFiles.join(', ')}`);
-      passed++;
-    } else {
-      clack.log.warn('⚠️  No IDE config files found');
-      warnings++;
-    }
-
-    // ── Check 7: Output directories ──
-    checks++;
-    const outputDir = path.join(projectDir, '_bmad-output');
-    if (fs.existsSync(outputDir)) {
-      clack.log.success('✅ _bmad-output/ directory exists');
-      passed++;
-    } else {
-      clack.log.warn('⚠️  _bmad-output/ not found');
-      warnings++;
-    }
-
-    // ── Check 8: Skills integrity (SKILL.md exists in pack dirs) ──
-    if (manifest.packs && manifest.packs.includes('seo')) {
-      checks++;
-      const seoSkill = path.join(agentsDir, 'pack-seo', 'SKILL.md');
-      if (fs.existsSync(seoSkill)) {
-        clack.log.success('✅ SEO Engine SKILL.md present');
-        passed++;
-      } else {
-        clack.log.error('❌ SEO Engine SKILL.md missing — /seo audit will not work');
-        errors++;
-      }
-    }
-
-    // ── Summary ──
-    const summaryColor = errors > 0 ? pc.red : warnings > 0 ? pc.yellow : pc.green;
-    const summaryIcon = errors > 0 ? '❌' : warnings > 0 ? '⚠️' : '✅';
-
-    clack.note([
-      `${summaryIcon} ${passed}/${checks} checks passed`,
-      errors > 0 ? `❌ ${errors} error(s)` : null,
-      warnings > 0 ? `⚠️  ${warnings} warning(s)` : null,
-      '',
-      `📦 Version: v${manifest.version}`,
-      `📅 Installed: ${manifest.installed.split('T')[0]}`,
-      `🗣️  Language: ${manifest.language || 'N/A'}`,
-      `📦 Packs: ${(manifest.packs || ['core']).join(', ')}`,
-    ].filter(Boolean).join('\n'), '🩺 BMAD+ Health Report');
-
-    clack.outro(summaryColor(
-      errors > 0
-        ? 'Issues found — run `npx bmad-plus install` to fix'
-        : warnings > 0
-          ? 'Minor issues found — consider running `npx bmad-plus update`'
-          : 'Everything looks great! 🎉'
-    ));
   },
 };

package/tools/cli/commands/install.js

@@ -35,7 +35,7 @@ module.exports = {
     ['-l, --lang <code>', 'Language code: en, fr, es, de, pt-br, ru, zh, he, ja, it (overrides auto-detection)'],
     ['--tools <tools>', 'Comma-separated IDE IDs, all, or none (default: auto-detect)'],
     ['--mode <mode>', 'Execution mode: manual, autopilot, hybrid (default: manual)'],
-    ['--provision-python', 'Provision an isolated Python env for packs that need one (seo, memory)'],
+    ['--provision-python', 'Provision isolated environments for selected Python packs'],
   ],
   action: async (options) => {
     // A closed/piped stdin cannot answer prompts. Require explicit defaults

package/tools/cli/commands/studio.js

@@ -0,0 +1,68 @@
+/** Inspect installed Dev Studio routes and prepare actual host instructions. */
+const path = require('node:path');
+const { installedPack, validatePack, prepare } = require('../lib/studio');
+
+module.exports = {
+  command: 'studio <action> [workflow]',
+  description: 'List Dev Studio workflows or prepare read-only execution context',
+  options: [
+    ['-d, --directory <path>', 'Installed project directory'],
+    ['--request <text>', 'Task scope for the host assistant'],
+    [
+      '--input <file>',
+      'Project-relative input file (repeatable)',
+      (file, files) => [...files, file],
+      [],
+    ],
+    ['--json', 'Print machine-readable context and evidence'],
+  ],
+  action: (action, workflow, options = {}) => {
+    try {
+      const projectDir = path.resolve(options.directory || process.cwd());
+      let result;
+      if (action === 'list' && !workflow) {
+        const { catalog, resources } = validatePack(installedPack(projectDir));
+        result = {
+          schemaVersion: 1,
+          execution: 'host-managed',
+          ...catalog,
+          validatedResources: resources.length,
+        };
+      } else if (action === 'prepare' && workflow) {
+        result = prepare({ projectDir, workflow, request: options.request, inputs: options.input });
+      } else throw new Error('Use studio list or studio prepare WORKFLOW.');
+      process.exitCode = result.status === 'needs-input' ? 2 : 0;
+      if (options.json) console.log(JSON.stringify(result, null, 2));
+      else if (action === 'list')
+        console.log(
+          result.workflows.map((w) => `${w.id} (${w.agent}; input: ${w.input})`).join('\n')
+        );
+      else
+        console.log(
+          [
+            `${result.workflow}: ${result.status} — prepared for the host, not executed.`,
+            `Project: ${result.projectDir}`,
+            `Request: ${result.request || '(none supplied)'}`,
+            `Resolved configuration:\n${JSON.stringify(result.config.values, null, 2)}`,
+            `Defaulted fields: ${result.config.defaultsUsed.join(', ') || '(none)'}`,
+            `Proposed report: ${result.outputPath}`,
+            ...result.missingInputs,
+            ...result.instructions.map(
+              (item) => `\n--- ${item.path} (${item.sha256}) ---\n${item.content}`
+            ),
+            ...result.inputs.map(
+              (item) => `\n--- Project input: ${item.path} (${item.sha256}) ---\n${item.content}`
+            ),
+            result.previousReport
+              ? `\n--- Continue existing report ---\n${result.previousReport.content}`
+              : '',
+          ].join('\n')
+        );
+    } catch (error) {
+      process.exitCode = 1;
+      if (options.json)
+        console.log(JSON.stringify({ schemaVersion: 1, status: 'error', error: error.message }));
+      else console.error('Dev Studio: ' + error.message);
+    }
+  },
+};