ma-agents 3.6.2 → 3.8.0

package/test/bmad-extension.test.js

@@ -1,283 +0,0 @@
-#!/usr/bin/env node
-/**
- * Tests for BMAD Extension Module and Built-in Agent Customizations
- *
- * Story 8.3: Extension module structure (module.yaml, module-help.csv, workflows)
- * Story 15.6: Built-in agent customizations separated to lib/bmad-customize/
- *
- * Validates extension module files, directory structure, built-in customize.yaml
- * contents, and deployment logic in bmad.js.
- */
-'use strict';
-
-const assert = require('assert');
-const fs = require('fs');
-const path = require('path');
-
-let passed = 0;
-let failed = 0;
-const errors = [];
-
-function test(name, fn) {
-  try {
-    fn();
-    console.log(`  \u2713 ${name}`);
-    passed++;
-  } catch (err) {
-    console.error(`  \u2717 ${name}: ${err.message}`);
-    failed++;
-    errors.push({ name, error: err.message });
-  }
-}
-
-const extDir = path.join(__dirname, '..', 'lib', 'bmad-extension');
-const customizeDir = path.join(__dirname, '..', 'lib', 'bmad-customize');
-
-// ─── AC #1: Extension module structure (module.yaml, module-help.csv, workflows) ──
-
-console.log('\nAC #1 — Extension module structure');
-
-test('1.1: lib/bmad-extension/ directory exists', () => {
-  assert.ok(fs.existsSync(extDir), 'lib/bmad-extension/ must exist');
-});
-
-test('1.2: lib/bmad-extension/agents/ has no .customize.yaml files (Story 15.6)', () => {
-  const agentsDir = path.join(extDir, 'agents');
-  // bmad-method may recreate an empty agents/ dir when processing --custom-content,
-  // so we check for absence of customize files, not absence of the directory itself.
-  if (fs.existsSync(agentsDir)) {
-    const files = fs.readdirSync(agentsDir).filter(f => f.endsWith('.customize.yaml'));
-    assert.strictEqual(files.length, 0, `agents/ must have 0 .customize.yaml files, found ${files.length}`);
-  }
-  // If directory doesn't exist, that's also correct
-});
-
-test('1.3: module.yaml exists with extends-module: bmm', () => {
-  const filePath = path.join(extDir, 'module.yaml');
-  assert.ok(fs.existsSync(filePath), 'module.yaml must exist');
-  const content = fs.readFileSync(filePath, 'utf-8');
-  assert.ok(content.includes('extends-module: bmm'), 'module.yaml must declare extends-module: bmm');
-  assert.ok(content.includes('name: "MA-Agents Skills & Agents"'), 'module.yaml must have name: "MA-Agents Skills & Agents"');
-});
-
-test('1.4: module-help.csv exists', () => {
-  const filePath = path.join(extDir, 'module-help.csv');
-  assert.ok(fs.existsSync(filePath), 'module-help.csv must exist');
-});
-
-test('1.5: workflows/ directory exists in extension module', () => {
-  const workflowsDir = path.join(extDir, 'workflows');
-  assert.ok(fs.existsSync(workflowsDir), 'lib/bmad-extension/workflows/ must exist');
-});
-
-// ─── AC #2: 8 built-in agents in lib/bmad-customize/ with critical_actions only ──
-
-console.log('\nAC #2 — Built-in agent customizations in lib/bmad-customize/');
-
-test('2.1: lib/bmad-customize/ directory exists', () => {
-  assert.ok(fs.existsSync(customizeDir), 'lib/bmad-customize/ must exist');
-});
-
-const builtinAgents = [
-  'bmm-pm', 'bmm-architect', 'bmm-dev',
-  'bmm-sm', 'bmm-tech-writer', 'bmm-ux-designer', 'bmm-bmad-master'
-];
-
-test('2.2: lib/bmad-customize/ contains exactly 10 .customize.yaml files', () => {
-  const files = fs.readdirSync(customizeDir).filter(f => f.endsWith('.customize.yaml'));
-  assert.strictEqual(files.length, 10, `Expected 10 .customize.yaml files, found ${files.length}`);
-});
-
-for (const agentId of builtinAgents) {
-  test(`2.3-${agentId}: ${agentId}.customize.yaml exists with correct content`, () => {
-    const filePath = path.join(customizeDir, `${agentId}.customize.yaml`);
-    assert.ok(fs.existsSync(filePath), `${agentId}.customize.yaml must exist`);
-    const content = fs.readFileSync(filePath, 'utf-8');
-
-    // Must have critical_actions
-    assert.ok(content.includes('critical_actions:'), `${agentId} must have critical_actions`);
-
-    // Must use 6.2.0 array syntax (- "...") not numbered keys (1: "...")
-    assert.ok(content.includes('  - "'), `${agentId} must use array syntax (- "...")`);
-    assert.ok(!content.match(/^\s+\d+:/m), `${agentId} must NOT use numbered key syntax`);
-
-    // Must NOT have persona or menu
-    assert.ok(!content.includes('persona:'), `${agentId} must NOT have persona section`);
-    assert.ok(!content.includes('menu:'), `${agentId} must NOT have menu section`);
-
-    // Must reference project-level skills/MANIFEST.yaml
-    assert.ok(content.includes('skills/MANIFEST.yaml'), `${agentId} must reference project-level skills/MANIFEST.yaml`);
-
-    // Must have all 4 critical actions
-    assert.ok(content.includes('Read the skills MANIFEST'), `${agentId} must have MANIFEST read action`);
-    assert.ok(content.includes('always_load: true'), `${agentId} must reference always_load`);
-    assert.ok(content.includes('project-context.md'), `${agentId} must reference project-context.md`);
-    assert.ok(content.includes('Follow all skill directives'), `${agentId} must have follow directives action`);
-
-    // Must have agent-specific comment header
-    assert.ok(content.includes(`MA-AGENTS: skill enforcement for ${agentId}`),
-      `${agentId} must have agent-specific comment header`);
-  });
-}
-
-// ─── AC #3: bmad.js has extension and customize deployment stages ─────────
-
-console.log('\nAC #3 — bmad.js deployment logic');
-
-test('3.1: bmad.js contains extension deployment code', () => {
-  const bmadSrc = fs.readFileSync(path.join(__dirname, '..', 'lib', 'bmad.js'), 'utf-8');
-  assert.ok(bmadSrc.includes("'bmad-extension'"), 'bmad.js must reference bmad-extension source dir');
-  assert.ok(bmadSrc.includes("'extensions'"), 'bmad.js must reference extensions target dir');
-  assert.ok(bmadSrc.includes("'ma-agents-skills'"), 'bmad.js must reference ma-agents-skills');
-});
-
-test('3.2: bmad.js contains customize deployment code', () => {
-  const bmadSrc = fs.readFileSync(path.join(__dirname, '..', 'lib', 'bmad.js'), 'utf-8');
-  assert.ok(bmadSrc.includes("'bmad-customize'"), 'bmad.js must reference bmad-customize source dir');
-  assert.ok(bmadSrc.includes('Applied built-in customization'), 'bmad.js must log built-in customization deployment');
-});
-
-test('3.3: extension deployment uses path.join for all paths', () => {
-  const bmadSrc = fs.readFileSync(path.join(__dirname, '..', 'lib', 'bmad.js'), 'utf-8');
-  assert.ok(bmadSrc.includes("path.join(__dirname, 'bmad-extension')"),
-    'extensionSource must use path.join');
-  assert.ok(bmadSrc.includes("path.join(projectRoot, BMAD_DIR, 'extensions', 'ma-agents-skills')"),
-    'extensionTarget must use path.join');
-});
-
-test('3.4: customize deployment uses path.join for source', () => {
-  const bmadSrc = fs.readFileSync(path.join(__dirname, '..', 'lib', 'bmad.js'), 'utf-8');
-  assert.ok(bmadSrc.includes("path.join(__dirname, 'bmad-customize')"),
-    'customizeSource must use path.join');
-});
-
-test('3.5: deployment order is RECOMPILE > EXTENSION > CUSTOMIZE > WORKFLOWS', () => {
-  const bmadSrc = fs.readFileSync(path.join(__dirname, '..', 'lib', 'bmad.js'), 'utf-8');
-  const recompileIdx = bmadSrc.indexOf('STAGE:RECOMPILE');
-  const extensionIdx = bmadSrc.indexOf('STAGE:EXTENSION');
-  const customizeIdx = bmadSrc.indexOf('STAGE:CUSTOMIZE');
-  const workflowIdx = bmadSrc.indexOf('STAGE:WORKFLOWS');
-
-  assert.ok(recompileIdx > 0, 'STAGE:RECOMPILE marker must exist');
-  assert.ok(extensionIdx > 0, 'STAGE:EXTENSION marker must exist');
-  assert.ok(customizeIdx > 0, 'STAGE:CUSTOMIZE marker must exist');
-  assert.ok(workflowIdx > 0, 'STAGE:WORKFLOWS marker must exist');
-  assert.ok(extensionIdx > recompileIdx, 'extension deployment must come after recompile');
-  assert.ok(customizeIdx > extensionIdx, 'customize deployment must come after extension');
-  assert.ok(customizeIdx < workflowIdx, 'customize deployment must come before workflows');
-});
-
-test('3.6: extension deployment logs with chalk.cyan', () => {
-  const bmadSrc = fs.readFileSync(path.join(__dirname, '..', 'lib', 'bmad.js'), 'utf-8');
-  assert.ok(bmadSrc.includes("chalk.cyan('    + Deployed BMAD extension module: ma-agents-skills')"),
-    'must log deployment with chalk.cyan');
-});
-
-test('3.7: bmad-customizations retains platform and generic files (agent files removed in Epic 15)', () => {
-  const customDir = path.join(__dirname, '..', 'lib', 'bmad-customizations');
-  // Platform-specific and generic files must still exist
-  const expectedFiles = ['claude-code', 'cursor', 'cline', 'gemini', 'generic', 'antigravity'];
-  for (const name of expectedFiles) {
-    const filePath = path.join(customDir, `${name}.customize.yaml`);
-    assert.ok(fs.existsSync(filePath), `${name}.customize.yaml must still exist in bmad-customizations/`);
-  }
-  // Custom agent files (bmm-sre, bmm-devops, etc.) were removed — now in skill folders
-  const removedAgents = ['bmm-sre', 'bmm-devops', 'bmm-cyber', 'bmm-qa'];
-  for (const id of removedAgents) {
-    const filePath = path.join(customDir, `${id}.customize.yaml`);
-    assert.ok(!fs.existsSync(filePath), `${id}.customize.yaml should NOT exist in bmad-customizations/ (moved to skill folders)`);
-  }
-});
-
-// ─── Integration tests — actual fs.copy deployment ───────────────────────
-
-console.log('\nIntegration — applyCustomizations() deployment');
-
-const fsExtra = require('fs-extra');
-const os = require('os');
-const { applyCustomizations } = require('../lib/bmad');
-
-async function asyncTest(name, fn) {
-  try {
-    await fn();
-    console.log(`  \u2713 ${name}`);
-    passed++;
-  } catch (err) {
-    console.error(`  \u2717 ${name}: ${err.message}`);
-    failed++;
-    errors.push({ name, error: err.message });
-  }
-}
-
-async function runIntegrationTests() {
-  const tmpDir = path.join(os.tmpdir(), `bmad-ext-test-${Date.now()}`);
-
-  await asyncTest('INT-1: applyCustomizations deploys extension module to target directory', async () => {
-    // Set up minimal BMAD structure so applyCustomizations doesn't fail on missing dirs
-    await fsExtra.ensureDir(path.join(tmpDir, '_bmad', '_config', 'agents'));
-    await fsExtra.ensureDir(path.join(tmpDir, '_bmad', 'custom', 'agents'));
-    await fsExtra.ensureDir(path.join(tmpDir, '_bmad', 'bmm', 'agents'));
-
-    // Run applyCustomizations — recompile will fail (no BMAD installed in temp) but
-    // extension deployment happens after recompile regardless of recompile outcome
-    await applyCustomizations(tmpDir, ['bmm'], [], []);
-
-    // Verify extension was deployed
-    const deployedModule = path.join(tmpDir, '_bmad', 'extensions', 'ma-agents-skills', 'module.yaml');
-    assert.ok(fsExtra.existsSync(deployedModule),
-      `module.yaml must exist at ${deployedModule}`);
-
-    const content = fsExtra.readFileSync(deployedModule, 'utf-8');
-    assert.ok(content.includes('extends-module: bmm'),
-      'deployed module.yaml must contain extends-module: bmm');
-  });
-
-  await asyncTest('INT-2: applyCustomizations deploys built-in customizations to _bmad/_config/agents/', async () => {
-    // Verify built-in customizations were deployed to config dir
-    const configAgentsDir = path.join(tmpDir, '_bmad', '_config', 'agents');
-
-    for (const agentId of builtinAgents) {
-      const deployedFile = path.join(configAgentsDir, `${agentId}.customize.yaml`);
-      assert.ok(fsExtra.existsSync(deployedFile),
-        `${agentId}.customize.yaml must be deployed to _bmad/_config/agents/`);
-
-      const content = fsExtra.readFileSync(deployedFile, 'utf-8');
-      assert.ok(content.includes('critical_actions:'),
-        `deployed ${agentId}.customize.yaml must contain critical_actions`);
-      assert.ok(content.includes('skills/MANIFEST.yaml'),
-        `deployed ${agentId}.customize.yaml must reference skills/MANIFEST.yaml`);
-    }
-  });
-
-  await asyncTest('INT-3: deployed customizations use array syntax not numbered keys', async () => {
-    const configAgentsDir = path.join(tmpDir, '_bmad', '_config', 'agents');
-
-    for (const agentId of builtinAgents) {
-      const deployedFile = path.join(configAgentsDir, `${agentId}.customize.yaml`);
-      const content = fsExtra.readFileSync(deployedFile, 'utf-8');
-      assert.ok(content.includes('  - "'), `deployed ${agentId} must use array syntax`);
-      assert.ok(!content.match(/^\s+\d+:/m), `deployed ${agentId} must NOT use numbered keys`);
-    }
-  });
-
-  // Cleanup
-  await fsExtra.remove(tmpDir);
-
-  printSummary();
-}
-
-function printSummary() {
-// ─── Summary ────────────────────────────────────────────────────────────────
-
-  console.log(`\n${'─'.repeat(60)}`);
-  console.log(`Results: ${passed} passed, ${failed} failed`);
-  if (errors.length > 0) {
-    console.log('\nFailures:');
-    for (const { name, error } of errors) {
-      console.log(`  ${name}: ${error}`);
-    }
-  }
-  process.exit(failed > 0 ? 1 : 0);
-}
-
-runIntegrationTests();

package/test/bmad-output-policy.test.js

@@ -1,119 +0,0 @@
-const assert = require('assert');
-const { test } = require('node:test');
-const path = require('path');
-const fs = require('fs');
-const os = require('os');
-
-// Helper: create a temp project dir with a .gitignore
-function makeProject(gitignoreContent) {
-  const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'bmad-test-'));
-  if (gitignoreContent !== null) {
-    fs.writeFileSync(path.join(dir, '.gitignore'), gitignoreContent, 'utf-8');
-  }
-  return dir;
-}
-
-const { ensureBmadOutputTracked } = require('../lib/installer');
-
-const BMAD_OUTPUT_PATTERNS = ['_bmad-output', '_bmad-output/', '/_bmad-output', '/_bmad-output/'];
-
-// ─── Story 10.1 — ensureBmadOutputTracked() unit tests ───────────────────────
-
-test('AC1 — removes _bmad-output/ and prints message', () => {
-  const dir = makeProject('node_modules/\n_bmad-output/\ndist/\n');
-  const logs = [];
-  const orig = console.log;
-  console.log = (...args) => logs.push(args.join(' '));
-  ensureBmadOutputTracked(dir);
-  console.log = orig;
-  const written = fs.readFileSync(path.join(dir, '.gitignore'), 'utf-8');
-  assert.ok(!written.includes('_bmad-output'), 'entry must be removed');
-  assert.ok(
-    logs.some(l => l.includes('_bmad-output is now tracked as project knowledge (not gitignored)')),
-    'must print the tracking message'
-  );
-});
-
-test('AC2 — no write when .gitignore has no _bmad-output entry', () => {
-  const dir = makeProject('node_modules/\ndist/\n');
-  const before = fs.statSync(path.join(dir, '.gitignore')).mtimeMs;
-  ensureBmadOutputTracked(dir);
-  const after = fs.statSync(path.join(dir, '.gitignore')).mtimeMs;
-  assert.strictEqual(before, after, 'file must not be written');
-});
-
-test('AC3 — no error and no file created when .gitignore absent', () => {
-  const dir = makeProject(null);
-  assert.doesNotThrow(() => ensureBmadOutputTracked(dir));
-  assert.ok(!fs.existsSync(path.join(dir, '.gitignore')), '.gitignore must not be created');
-});
-
-test('Windows CRLF — detects and removes _bmad-output/\\r\\n', () => {
-  const dir = makeProject('node_modules/\r\n_bmad-output/\r\ndist/\r\n');
-  ensureBmadOutputTracked(dir);
-  const written = fs.readFileSync(path.join(dir, '.gitignore'), 'utf-8');
-  assert.ok(!written.includes('_bmad-output'), 'CRLF entry must be removed');
-});
-
-test('Idempotency — second run does not write', () => {
-  const dir = makeProject('node_modules/\n_bmad-output/\n');
-  ensureBmadOutputTracked(dir);                          // first run — removes it
-  const after1 = fs.statSync(path.join(dir, '.gitignore')).mtimeMs;
-  ensureBmadOutputTracked(dir);                          // second run — no-op
-  const after2 = fs.statSync(path.join(dir, '.gitignore')).mtimeMs;
-  assert.strictEqual(after1, after2, 'second run must not write');
-});
-
-test('Own .gitignore contains no _bmad-output entry', () => {
-  const gitignore = fs.readFileSync(path.join(__dirname, '..', '.gitignore'), 'utf-8');
-  const active = gitignore.split(/\r?\n/).filter(l => !l.trim().startsWith('#'));
-  assert.ok(
-    !active.some(l => BMAD_OUTPUT_PATTERNS.includes(l.trim())),
-    'own .gitignore must not contain _bmad-output'
-  );
-});
-
-// ─── Story 10.2 — README _bmad-output/ policy documentation tests ─────────────
-
-const README_PATH = path.join(__dirname, '..', 'README.md');
-const readmeContent = fs.readFileSync(README_PATH, 'utf-8');
-
-test('README contains _bmad-output/ policy section heading', () => {
-  assert.ok(
-    readmeContent.includes('## Project Knowledge: _bmad-output/'),
-    'README must contain heading "## Project Knowledge: _bmad-output/"'
-  );
-});
-
-test('README contains explicit do-not-gitignore directive', () => {
-  assert.ok(
-    readmeContent.includes('Do not add `_bmad-output/` to your `.gitignore`'),
-    'README must contain the phrase "Do not add `_bmad-output/` to your `.gitignore`"'
-  );
-});
-
-test('README explains why _bmad-output/ is tracked (team alignment)', () => {
-  const mentionsReason =
-    readmeContent.includes('team alignment') ||
-    readmeContent.includes('AI context') ||
-    readmeContent.includes('planning artifacts');
-  assert.ok(
-    mentionsReason,
-    'README must mention at least one reason for tracking (team alignment, AI context, planning artifacts)'
-  );
-});
-
-test('README mentions installer removes _bmad-output/ from .gitignore', () => {
-  assert.ok(
-    readmeContent.includes('installer') && readmeContent.includes('.gitignore') && readmeContent.includes('_bmad-output'),
-    'README must mention installer behaviour around .gitignore and _bmad-output/'
-  );
-});
-
-test('_bmad-output/ section appears within first two major sections after Installation', () => {
-  const sectionIdx = readmeContent.indexOf('## Project Knowledge: _bmad-output/');
-  // Count H2 headings that appear before this section
-  const before = readmeContent.slice(0, sectionIdx);
-  const h2sBefore = (before.match(/^## /gm) || []).length;
-  assert.ok(h2sBefore <= 3, `Section must be within the first few major sections, found ${h2sBefore} H2s before it`);
-});

package/test/bmad-persona-phase-prefix.test.js

@@ -1,271 +0,0 @@
-#!/usr/bin/env node
-/**
- * Story 21.7 — Tests for BMAD persona phase-aware prompt prefix.
- *
- * AC coverage (12 required):
- *   7.1  bmm-pm source contains phase: planning and non-empty on_prem_phase_prefix
- *   7.2  bmm-architect source contains phase: planning and non-empty on_prem_phase_prefix
- *   7.3  bmm-dev source contains phase: implementation and non-empty on_prem_phase_prefix
- *   7.4  Every planning persona's on_prem_phase_prefix contains /no_think literal
- *   7.5  No implementation persona's on_prem_phase_prefix contains /no_think
- *   7.6  Standard-profile deploy: bmm-pm has NO phase:, NO on_prem_phase_prefix:, NO /no_think
- *   7.7  On-prem deploy: bmm-pm critical_actions[0] contains planning prefix with /no_think
- *   7.8  On-prem deploy: bmm-pm has NO phase: or on_prem_phase_prefix: top-level keys
- *   7.9  On-prem deploy: ALL pre-existing critical_actions entries preserved in order
- *   7.10 Idempotency: two consecutive on-prem deploys produce byte-identical deployed bmm-pm
- *   7.11 bmm-analyst.customize.yaml exists and has phase: planning
- *   7.12 bmm-quick-flow-solo-dev.customize.yaml exists and has phase: implementation
- */
-'use strict';
-
-const assert = require('assert');
-const path = require('path');
-const fs = require('fs');
-const os = require('os');
-const jsYaml = require('js-yaml');
-
-let passed = 0;
-let failed = 0;
-const errors = [];
-
-async function test(name, fn) {
-  try {
-    await fn();
-    console.log(`  \u2713 ${name}`);
-    passed++;
-  } catch (err) {
-    console.error(`  \u2717 ${name}: ${err.stack || err.message}`);
-    failed++;
-    errors.push({ name, error: err.message });
-  }
-}
-
-const CUSTOMIZE_SOURCE = path.join(__dirname, '..', 'lib', 'bmad-customize');
-const { applyPersonaPhasePrefix } = require('../lib/bmad');
-const { setProfile } = require('../lib/profile');
-
-function mktemp() {
-  return fs.mkdtempSync(path.join(os.tmpdir(), 'ma-21-7-'));
-}
-
-function cleanup(dir) {
-  try { fs.rmSync(dir, { recursive: true, force: true }); } catch {}
-}
-
-/**
- * Copy all customize yaml files from CUSTOMIZE_SOURCE into tmpDir,
- * then call applyPersonaPhasePrefix with the given profile.
- * Returns the parsed YAML of the specified file from tmpDir.
- */
-async function deployAndParse(tmpDir, profile, filename) {
-  // Copy all source customize files into tmpDir
-  const files = fs.readdirSync(CUSTOMIZE_SOURCE).filter(f => f.endsWith('.customize.yaml'));
-  for (const file of files) {
-    fs.copyFileSync(
-      path.join(CUSTOMIZE_SOURCE, file),
-      path.join(tmpDir, file)
-    );
-  }
-  await applyPersonaPhasePrefix(CUSTOMIZE_SOURCE, tmpDir, profile);
-  const raw = fs.readFileSync(path.join(tmpDir, filename), 'utf8');
-  return jsYaml.load(raw);
-}
-
-console.log('\n story 21.7 — BMAD persona phase-aware prompt prefix\n');
-
-async function runAll() {
-
-  // 7.1 — bmm-pm source contains phase: planning and non-empty on_prem_phase_prefix
-  await test('7.1 bmm-pm.customize.yaml source has phase: planning and non-empty on_prem_phase_prefix', () => {
-    const doc = jsYaml.load(fs.readFileSync(path.join(CUSTOMIZE_SOURCE, 'bmm-pm.customize.yaml'), 'utf8'));
-    assert.strictEqual(doc.phase, 'planning', 'phase should be "planning"');
-    assert.ok(doc.on_prem_phase_prefix && doc.on_prem_phase_prefix.trim().length > 0,
-      'on_prem_phase_prefix should be non-empty');
-  });
-
-  // 7.2 — bmm-architect source contains phase: planning and non-empty on_prem_phase_prefix
-  await test('7.2 bmm-architect.customize.yaml source has phase: planning and non-empty on_prem_phase_prefix', () => {
-    const doc = jsYaml.load(fs.readFileSync(path.join(CUSTOMIZE_SOURCE, 'bmm-architect.customize.yaml'), 'utf8'));
-    assert.strictEqual(doc.phase, 'planning', 'phase should be "planning"');
-    assert.ok(doc.on_prem_phase_prefix && doc.on_prem_phase_prefix.trim().length > 0,
-      'on_prem_phase_prefix should be non-empty');
-  });
-
-  // 7.3 — bmm-dev source contains phase: implementation and non-empty on_prem_phase_prefix
-  await test('7.3 bmm-dev.customize.yaml source has phase: implementation and non-empty on_prem_phase_prefix', () => {
-    const doc = jsYaml.load(fs.readFileSync(path.join(CUSTOMIZE_SOURCE, 'bmm-dev.customize.yaml'), 'utf8'));
-    assert.strictEqual(doc.phase, 'implementation', 'phase should be "implementation"');
-    assert.ok(doc.on_prem_phase_prefix && doc.on_prem_phase_prefix.trim().length > 0,
-      'on_prem_phase_prefix should be non-empty');
-  });
-
-  // 7.4 — Every planning persona's on_prem_phase_prefix contains /no_think literal
-  await test('7.4 All planning personas have /no_think in on_prem_phase_prefix', () => {
-    const files = fs.readdirSync(CUSTOMIZE_SOURCE).filter(f => f.endsWith('.customize.yaml'));
-    for (const file of files) {
-      const doc = jsYaml.load(fs.readFileSync(path.join(CUSTOMIZE_SOURCE, file), 'utf8'));
-      if (doc && doc.phase === 'planning') {
-        assert.ok(
-          typeof doc.on_prem_phase_prefix === 'string' && doc.on_prem_phase_prefix.includes('/no_think'),
-          `${file}: planning persona on_prem_phase_prefix must contain /no_think, got: ${doc.on_prem_phase_prefix}`
-        );
-      }
-    }
-  });
-
-  // 7.5 — No implementation persona's on_prem_phase_prefix contains /no_think
-  await test('7.5 No implementation persona has /no_think in on_prem_phase_prefix', () => {
-    const files = fs.readdirSync(CUSTOMIZE_SOURCE).filter(f => f.endsWith('.customize.yaml'));
-    for (const file of files) {
-      const doc = jsYaml.load(fs.readFileSync(path.join(CUSTOMIZE_SOURCE, file), 'utf8'));
-      if (doc && doc.phase === 'implementation') {
-        const prefix = doc.on_prem_phase_prefix || '';
-        assert.ok(
-          !prefix.includes('/no_think'),
-          `${file}: implementation persona on_prem_phase_prefix must NOT contain /no_think`
-        );
-      }
-    }
-  });
-
-  // 7.6 — Standard-profile deploy: bmm-pm has NO phase:, NO on_prem_phase_prefix:, NO /no_think
-  await test('7.6 Standard-profile deploy: bmm-pm has no phase, no on_prem_phase_prefix, no /no_think', async () => {
-    const tmpDir = mktemp();
-    try {
-      const doc = await deployAndParse(tmpDir, 'standard', 'bmm-pm.customize.yaml');
-      assert.ok(!Object.prototype.hasOwnProperty.call(doc, 'phase'),
-        'deployed file must not have "phase" key');
-      assert.ok(!Object.prototype.hasOwnProperty.call(doc, 'on_prem_phase_prefix'),
-        'deployed file must not have "on_prem_phase_prefix" key');
-      const raw = fs.readFileSync(path.join(tmpDir, 'bmm-pm.customize.yaml'), 'utf8');
-      assert.ok(!raw.includes('/no_think'),
-        'deployed file must not contain /no_think string');
-    } finally {
-      cleanup(tmpDir);
-    }
-  });
-
-  // 7.7 — On-prem deploy: bmm-pm critical_actions[0] contains planning prefix with /no_think
-  await test('7.7 On-prem deploy: bmm-pm critical_actions[0] contains /no_think prefix', async () => {
-    const tmpDir = mktemp();
-    try {
-      const doc = await deployAndParse(tmpDir, 'on-prem', 'bmm-pm.customize.yaml');
-      assert.ok(Array.isArray(doc.critical_actions) && doc.critical_actions.length > 0,
-        'critical_actions must be a non-empty array');
-      const first = doc.critical_actions[0];
-      assert.ok(typeof first === 'string' && first.includes('/no_think'),
-        `critical_actions[0] must contain /no_think, got: ${first}`
-      );
-    } finally {
-      cleanup(tmpDir);
-    }
-  });
-
-  // 7.8 — On-prem deploy: bmm-pm has NO phase: or on_prem_phase_prefix: top-level keys
-  await test('7.8 On-prem deploy: bmm-pm has no phase or on_prem_phase_prefix keys after deploy', async () => {
-    const tmpDir = mktemp();
-    try {
-      const doc = await deployAndParse(tmpDir, 'on-prem', 'bmm-pm.customize.yaml');
-      assert.ok(!Object.prototype.hasOwnProperty.call(doc, 'phase'),
-        'deployed on-prem file must not have "phase" key');
-      assert.ok(!Object.prototype.hasOwnProperty.call(doc, 'on_prem_phase_prefix'),
-        'deployed on-prem file must not have "on_prem_phase_prefix" key');
-    } finally {
-      cleanup(tmpDir);
-    }
-  });
-
-  // 7.9 — On-prem deploy: ALL pre-existing critical_actions entries preserved in order
-  await test('7.9 On-prem deploy: all original critical_actions entries preserved', async () => {
-    const tmpDir = mktemp();
-    try {
-      // Get original critical_actions from source
-      const sourceDoc = jsYaml.load(
-        fs.readFileSync(path.join(CUSTOMIZE_SOURCE, 'bmm-pm.customize.yaml'), 'utf8')
-      );
-      const originalActions = Array.isArray(sourceDoc.critical_actions)
-        ? sourceDoc.critical_actions : [];
-
-      const deployedDoc = await deployAndParse(tmpDir, 'on-prem', 'bmm-pm.customize.yaml');
-      assert.ok(Array.isArray(deployedDoc.critical_actions),
-        'critical_actions must be an array');
-      // The deployed array should be [prefix, ...originalActions]
-      assert.strictEqual(
-        deployedDoc.critical_actions.length,
-        originalActions.length + 1,
-        `deployed critical_actions length should be original + 1, got ${deployedDoc.critical_actions.length}`
-      );
-      for (let i = 0; i < originalActions.length; i++) {
-        assert.strictEqual(
-          deployedDoc.critical_actions[i + 1],
-          originalActions[i],
-          `critical_actions[${i + 1}] should match original[${i}]`
-        );
-      }
-    } finally {
-      cleanup(tmpDir);
-    }
-  });
-
-  // 7.10 — Idempotency: two consecutive on-prem deploys produce byte-identical deployed bmm-pm
-  await test('7.10 Idempotency: two on-prem deploys produce byte-identical bmm-pm.customize.yaml', async () => {
-    const tmpDir = mktemp();
-    try {
-      // First deploy
-      const files = fs.readdirSync(CUSTOMIZE_SOURCE).filter(f => f.endsWith('.customize.yaml'));
-      for (const file of files) {
-        fs.copyFileSync(
-          path.join(CUSTOMIZE_SOURCE, file),
-          path.join(tmpDir, file)
-        );
-      }
-      await applyPersonaPhasePrefix(CUSTOMIZE_SOURCE, tmpDir, 'on-prem');
-      const after1 = fs.readFileSync(path.join(tmpDir, 'bmm-pm.customize.yaml'), 'utf8');
-
-      // Second deploy: re-copy source files and apply again
-      for (const file of files) {
-        fs.copyFileSync(
-          path.join(CUSTOMIZE_SOURCE, file),
-          path.join(tmpDir, file)
-        );
-      }
-      await applyPersonaPhasePrefix(CUSTOMIZE_SOURCE, tmpDir, 'on-prem');
-      const after2 = fs.readFileSync(path.join(tmpDir, 'bmm-pm.customize.yaml'), 'utf8');
-
-      assert.strictEqual(after1, after2, 'Two consecutive on-prem deploys must produce byte-identical files');
-    } finally {
-      cleanup(tmpDir);
-    }
-  });
-
-  // 7.11 — bmm-analyst.customize.yaml exists and has phase: planning
-  await test('7.11 bmm-analyst.customize.yaml exists and has phase: planning', () => {
-    const filePath = path.join(CUSTOMIZE_SOURCE, 'bmm-analyst.customize.yaml');
-    assert.ok(fs.existsSync(filePath), 'bmm-analyst.customize.yaml must exist in lib/bmad-customize/');
-    const doc = jsYaml.load(fs.readFileSync(filePath, 'utf8'));
-    assert.strictEqual(doc.phase, 'planning', 'bmm-analyst must have phase: planning');
-  });
-
-  // 7.12 — bmm-quick-flow-solo-dev.customize.yaml exists and has phase: implementation
-  await test('7.12 bmm-quick-flow-solo-dev.customize.yaml exists and has phase: implementation', () => {
-    const filePath = path.join(CUSTOMIZE_SOURCE, 'bmm-quick-flow-solo-dev.customize.yaml');
-    assert.ok(fs.existsSync(filePath), 'bmm-quick-flow-solo-dev.customize.yaml must exist in lib/bmad-customize/');
-    const doc = jsYaml.load(fs.readFileSync(filePath, 'utf8'));
-    assert.strictEqual(doc.phase, 'implementation', 'bmm-quick-flow-solo-dev must have phase: implementation');
-  });
-
-  // Summary
-  console.log(`\n  ${passed} passed, ${failed} failed\n`);
-  if (errors.length > 0) {
-    console.error('Failures:');
-    for (const e of errors) {
-      console.error(`  - ${e.name}: ${e.error}`);
-    }
-    process.exit(1);
-  }
-}
-
-runAll().catch(err => {
-  console.error('Unexpected error:', err);
-  process.exit(1);
-});