ma-agents 3.6.2 → 3.8.0

package/test/reconfigure.test.js

@@ -1,436 +0,0 @@
-#!/usr/bin/env node
-/**
- * Story 21.10 — Tests for the `reconfigure` subcommand orchestrator.
- *
- * AC coverage (one-to-one with acceptance criteria 1–11):
- *   8.1  --yes rejected with pinned message (AC #6)
- *   8.2  missing .ma-agents.json → named error (AC #1)
- *   8.3  same-value → "Profile unchanged" and no writes (AC #4)
- *   8.4  profile flip standard → on-prem re-stamps injection files + .roomodes (AC #5)
- *   8.5  two-step "Continue?" → no aborts with zero writes (AC #7)
- *   8.6  backup files written at <target>.backup-<ISO> (AC #8)
- *   8.7  slug divergence raises RoomodesSlugDivergenceError; --force bypasses (AC #9)
- *   8.8  dual-file drift raises ClinerulesDualFileDriftError (AC #10)
- *   8.9  profileHistory appends; cap at 20; first call creates field (AC #11)
- *   8.10 full round-trip standard → on-prem → standard (NFR46)
- *   8.11 Current profile appears in the prompt message (AC #2)
- */
-'use strict';
-
-const assert = require('assert');
-const path = require('path');
-const fs = require('fs');
-const os = require('os');
-
-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 {
-  reconfigure,
-  RoomodesSlugDivergenceError,
-  ManifestNotFoundError,
-  ReconfigureYesRejectedError,
-  YES_REJECT_MESSAGE,
-  PROFILE_HISTORY_CAP,
-  listTouchedFiles,
-  appendProfileHistory
-} = require('../lib/reconfigure');
-const { setProfile, getProfile } = require('../lib/profile');
-const installer = require('../lib/installer');
-const { getAgent } = require('../lib/agents');
-
-function mktemp() {
-  return fs.mkdtempSync(path.join(os.tmpdir(), 'ma-agents-reconfigure-test-'));
-}
-function cleanup(dir) { try { fs.rmSync(dir, { recursive: true, force: true }); } catch {} }
-
-function silenceLogs(fn) {
-  return async (...args) => {
-    const origLog = console.log;
-    const origErr = console.error;
-    const origWarn = console.warn;
-    console.log = () => {};
-    console.error = () => {};
-    console.warn = () => {};
-    try { return await fn(...args); }
-    finally {
-      console.log = origLog;
-      console.error = origErr;
-      console.warn = origWarn;
-    }
-  };
-}
-
-/**
- * Build a prompts() stub that returns the given sequence of responses,
- * one per prompt call. Throws if the caller asks for more prompts than
- * responses were scripted.
- */
-function scriptedPrompts(responses) {
-  let i = 0;
-  const stub = async () => {
-    if (i >= responses.length) {
-      throw new Error(`scriptedPrompts exhausted after ${responses.length} calls`);
-    }
-    return responses[i++];
-  };
-  stub.calls = () => i;
-  stub.messages = [];
-  return stub;
-}
-
-/**
- * Install a minimal project scaffold: .ma-agents.json with a given profile and
- * agent list. Does NOT run the real installer (tests isolate the reconfigure
- * orchestration). Returns the tmp dir.
- */
-function setupProject({ profile = 'standard', agents = ['claude-code'] } = {}) {
-  const dir = mktemp();
-  const manifest = {
-    manifestVersion: '1.2.0',
-    agent: agents[0],
-    agents,
-    scope: 'project',
-    profile,
-    skills: {}
-  };
-  fs.writeFileSync(
-    path.join(dir, '.ma-agents.json'),
-    JSON.stringify(manifest, null, 2) + '\n',
-    'utf-8'
-  );
-  return dir;
-}
-
-console.log('\n reconfigure tests (Story 21.10)\n');
-
-async function runAll() {
-  // 8.1 --yes rejected
-  await test('8.1 --yes rejected with pinned message and ReconfigureYesRejectedError (AC #6)', silenceLogs(async () => {
-    const dir = setupProject();
-    try {
-      await assert.rejects(
-        () => reconfigure({ projectRoot: dir, argv: ['--yes'], promptsLib: scriptedPrompts([]) }),
-        (err) => err instanceof ReconfigureYesRejectedError && err.message === YES_REJECT_MESSAGE
-      );
-      // Manifest should be untouched.
-      const json = JSON.parse(fs.readFileSync(path.join(dir, '.ma-agents.json'), 'utf-8'));
-      assert.strictEqual(json.profile, 'standard');
-      assert.strictEqual(json.profileHistory, undefined);
-    } finally { cleanup(dir); }
-  }));
-
-  // 8.2 missing manifest
-  await test('8.2 missing .ma-agents.json → ManifestNotFoundError (AC #1)', silenceLogs(async () => {
-    const dir = mktemp();
-    try {
-      await assert.rejects(
-        () => reconfigure({ projectRoot: dir, argv: [], promptsLib: scriptedPrompts([]) }),
-        (err) => err instanceof ManifestNotFoundError
-      );
-    } finally { cleanup(dir); }
-  }));
-
-  // 8.3 same-value short-circuit
-  await test('8.3 same-value exits with "Profile unchanged:" and no writes (AC #4)', silenceLogs(async () => {
-    const dir = setupProject({ profile: 'standard' });
-    try {
-      const manifestPath = path.join(dir, '.ma-agents.json');
-      const originalBytes = fs.readFileSync(manifestPath);
-      const result = await reconfigure({
-        projectRoot: dir,
-        argv: [],
-        promptsLib: scriptedPrompts([{ chosenProfile: 'standard' }])
-      });
-      assert.strictEqual(result.status, 'unchanged');
-      // No writes → byte-identical manifest.
-      assert.deepStrictEqual(fs.readFileSync(manifestPath), originalBytes);
-    } finally { cleanup(dir); }
-  }));
-
-  // 8.4 profile flip re-stamps injection files + .roomodes
-  await test('8.4 profile flip standard→on-prem re-stamps injection files (AC #5)', silenceLogs(async () => {
-    const dir = setupProject({ profile: 'standard', agents: ['claude-code'] });
-    try {
-      // Pre-stamp the .claude/CLAUDE.md with standard content via the installer
-      // helper so we have a realistic starting state.
-      const agent = getAgent('claude-code');
-      await installer._testUpdateAgentInstructions(agent, dir, { yesMode: true });
-      const claudePath = path.join(dir, '.claude', 'CLAUDE.md');
-      const before = fs.readFileSync(claudePath, 'utf-8');
-
-      const result = await reconfigure({
-        projectRoot: dir,
-        argv: [],
-        promptsLib: scriptedPrompts([
-          { chosenProfile: 'on-prem' }, // profile choice
-          { proceed: true }              // Continue?
-        ])
-      });
-      assert.strictEqual(result.status, 'reconfigured');
-      const after = fs.readFileSync(claudePath, 'utf-8');
-      assert.notStrictEqual(after, before, 'file must change');
-      // On-prem content ships onprem-specific strings (e.g. /no_think).
-      assert.match(after, /\/no_think|str_replace_editor|~\/\.claude\/|Local-LLM|Nemotron|on-prem/i,
-        'on-prem profile output should mention a local-LLM-specific token');
-      assert.strictEqual(getProfile(dir), 'on-prem');
-    } finally { cleanup(dir); }
-  }));
-
-  // 8.5 two-step confirmation decline
-  await test('8.5 declining "Continue?" aborts with zero writes (AC #7)', silenceLogs(async () => {
-    const dir = setupProject({ profile: 'standard', agents: ['claude-code'] });
-    try {
-      // Seed .claude/CLAUDE.md with standard content so a real re-stamp would
-      // leave a visible diff.
-      const agent = getAgent('claude-code');
-      await installer._testUpdateAgentInstructions(agent, dir, { yesMode: true });
-      const claudePath = path.join(dir, '.claude', 'CLAUDE.md');
-      const before = fs.readFileSync(claudePath);
-      const manifestBefore = fs.readFileSync(path.join(dir, '.ma-agents.json'));
-
-      const result = await reconfigure({
-        projectRoot: dir,
-        argv: [],
-        promptsLib: scriptedPrompts([
-          { chosenProfile: 'on-prem' },
-          { proceed: false }
-        ])
-      });
-      assert.strictEqual(result.status, 'aborted');
-      assert.deepStrictEqual(fs.readFileSync(claudePath), before, 'CLAUDE.md must be untouched');
-      assert.deepStrictEqual(
-        fs.readFileSync(path.join(dir, '.ma-agents.json')),
-        manifestBefore,
-        'manifest must be untouched when user declines'
-      );
-    } finally { cleanup(dir); }
-  }));
-
-  // 8.6 backup files are written
-  await test('8.6 backup files created at <target>.backup-<timestamp> (AC #8)', silenceLogs(async () => {
-    const dir = setupProject({ profile: 'standard', agents: ['claude-code'] });
-    try {
-      const agent = getAgent('claude-code');
-      await installer._testUpdateAgentInstructions(agent, dir, { yesMode: true });
-      await reconfigure({
-        projectRoot: dir,
-        argv: [],
-        promptsLib: scriptedPrompts([
-          { chosenProfile: 'on-prem' },
-          { proceed: true }
-        ])
-      });
-      const claudeDir = path.join(dir, '.claude');
-      const backups = fs.readdirSync(claudeDir).filter(f => /CLAUDE\.md\.backup-/.test(f));
-      assert.ok(backups.length >= 1, `expected at least one backup file, got ${backups.join(',')}`);
-      const backupName = backups[0];
-      // Canonical format: <target>.backup-<YYYY-MM-DDTHH-mm-ssZ>
-      assert.match(backupName, /^CLAUDE\.md\.backup-\d{4}-\d{2}-\d{2}T\d{2}-\d{2}-\d{2}Z(\.\d+)?$/);
-    } finally { cleanup(dir); }
-  }));
-
-  // 8.7 slug divergence
-  await test('8.7 RoomodesSlugDivergenceError raised when user edited owned slug; --force bypasses (AC #9)', silenceLogs(async () => {
-    const dir = setupProject({ profile: 'standard', agents: ['roo-code'] });
-    try {
-      // Write a .roomodes whose `bmad-architect` entry has a fabricated
-      // `whenToUse` that doesn't match the shipped template.
-      const yaml = require('js-yaml');
-      const divergent = {
-        customModes: [
-          { slug: 'bmad-architect', name: 'Hacked', whenToUse: 'different', roleDefinition: 'x', groups: ['read'], customInstructions: 'x' }
-        ]
-      };
-      fs.writeFileSync(path.join(dir, '.roomodes'), yaml.dump(divergent), 'utf-8');
-
-      await assert.rejects(
-        () => reconfigure({
-          projectRoot: dir,
-          argv: [],
-          promptsLib: scriptedPrompts([{ chosenProfile: 'on-prem' }])
-        }),
-        (err) => err instanceof RoomodesSlugDivergenceError && err.divergentSlugs.includes('bmad-architect')
-      );
-
-      // With --force-roomodes-overwrite, proceed through (confirm continue).
-      const result = await reconfigure({
-        projectRoot: dir,
-        argv: ['--force-roomodes-overwrite'],
-        promptsLib: scriptedPrompts([
-          { chosenProfile: 'on-prem' },
-          { proceed: true }
-        ])
-      });
-      assert.strictEqual(result.status, 'reconfigured');
-    } finally { cleanup(dir); }
-  }));
-
-  // 8.8 dual-file drift (Cline)
-  await test('8.8 ClinerulesDualFileDriftError raised; no --force override (AC #10)', silenceLogs(async () => {
-    const dir = setupProject({ profile: 'standard', agents: ['cline'] });
-    try {
-      // Seed with identical content first.
-      const agent = getAgent('cline');
-      await installer._testUpdateAgentInstructions(agent, dir, { yesMode: true });
-      // Mutate ONE of the two files inside the markers to induce drift.
-      const fileA = path.join(dir, '.cline', 'clinerules.md');
-      const content = fs.readFileSync(fileA, 'utf-8');
-      const mutated = content.replace(
-        /<!-- MA-AGENTS-START -->[\s\S]*?<!-- MA-AGENTS-END -->/,
-        '<!-- MA-AGENTS-START -->\nCLI-NO-MATCHY\n<!-- MA-AGENTS-END -->'
-      );
-      fs.writeFileSync(fileA, mutated, 'utf-8');
-
-      await assert.rejects(
-        () => reconfigure({
-          projectRoot: dir,
-          argv: [],
-          promptsLib: scriptedPrompts([{ chosenProfile: 'on-prem' }])
-        }),
-        (err) => err && err.name === 'ClinerulesDualFileDriftError'
-      );
-    } finally { cleanup(dir); }
-  }));
-
-  // 8.9 profileHistory append + cap
-  await test('8.9 profileHistory appends; missing-field creation; 20-cap evicts oldest (AC #11)', silenceLogs(async () => {
-    const dir = setupProject({ profile: 'standard', agents: ['claude-code'] });
-    try {
-      // First reconfigure creates the field.
-      await reconfigure({
-        projectRoot: dir,
-        argv: [],
-        promptsLib: scriptedPrompts([
-          { chosenProfile: 'on-prem' },
-          { proceed: true }
-        ])
-      });
-      let manifest = JSON.parse(fs.readFileSync(path.join(dir, '.ma-agents.json'), 'utf-8'));
-      assert.ok(Array.isArray(manifest.profileHistory));
-      assert.strictEqual(manifest.profileHistory.length, 1);
-      assert.deepStrictEqual(
-        { from: manifest.profileHistory[0].from, to: manifest.profileHistory[0].to, source: manifest.profileHistory[0].source },
-        { from: 'standard', to: 'on-prem', source: 'reconfigure' }
-      );
-      assert.ok(typeof manifest.profileHistory[0].date === 'string');
-
-      // Now directly invoke appendProfileHistory 25 more times to exercise the cap.
-      for (let i = 0; i < 25; i++) {
-        appendProfileHistory(dir, {
-          date: new Date(2027, 0, i + 1).toISOString(),
-          from: 'standard',
-          to: 'on-prem',
-          source: 'reconfigure'
-        });
-      }
-      manifest = JSON.parse(fs.readFileSync(path.join(dir, '.ma-agents.json'), 'utf-8'));
-      assert.strictEqual(
-        manifest.profileHistory.length,
-        PROFILE_HISTORY_CAP,
-        `history should be capped at ${PROFILE_HISTORY_CAP}`
-      );
-      // Oldest-first eviction: the first entry we wrote (the real reconfigure
-      // result) must be gone. The newest entry's date must survive.
-      const dates = manifest.profileHistory.map(e => e.date);
-      assert.ok(!dates.includes(manifest.profileHistory[0].date === manifest.profileHistory[0].date && undefined));
-      const newest = new Date(2027, 0, 25).toISOString();
-      assert.ok(dates.includes(newest), 'newest entry must be preserved');
-    } finally { cleanup(dir); }
-  }));
-
-  // 8.10 round-trip standard → on-prem → standard
-  await test('8.10 round-trip standard → on-prem → standard lands on standard content (NFR46)', silenceLogs(async () => {
-    const dir = setupProject({ profile: 'standard', agents: ['claude-code'] });
-    try {
-      const agent = getAgent('claude-code');
-      await installer._testUpdateAgentInstructions(agent, dir, { yesMode: true });
-      const baselineStandard = fs.readFileSync(path.join(dir, '.claude', 'CLAUDE.md'), 'utf-8');
-
-      await reconfigure({
-        projectRoot: dir,
-        argv: [],
-        promptsLib: scriptedPrompts([
-          { chosenProfile: 'on-prem' },
-          { proceed: true }
-        ])
-      });
-      assert.strictEqual(getProfile(dir), 'on-prem');
-
-      await reconfigure({
-        projectRoot: dir,
-        argv: [],
-        promptsLib: scriptedPrompts([
-          { chosenProfile: 'standard' },
-          { proceed: true }
-        ])
-      });
-      assert.strictEqual(getProfile(dir), 'standard');
-      const after = fs.readFileSync(path.join(dir, '.claude', 'CLAUDE.md'), 'utf-8');
-      // Compare MARKER block content only, since outside-markers content may
-      // accumulate nothing here but we want a robust assertion.
-      const extract = (s) => {
-        const m = s.match(/<!-- MA-AGENTS-START -->([\s\S]*?)<!-- MA-AGENTS-END -->/);
-        return m ? m[1] : null;
-      };
-      assert.strictEqual(extract(after), extract(baselineStandard),
-        'post round-trip marker block must equal the original standard-profile stamp');
-
-      // Two history entries.
-      const manifest = JSON.parse(fs.readFileSync(path.join(dir, '.ma-agents.json'), 'utf-8'));
-      assert.strictEqual(manifest.profileHistory.length, 2);
-      assert.strictEqual(manifest.profileHistory[0].to, 'on-prem');
-      assert.strictEqual(manifest.profileHistory[1].to, 'standard');
-    } finally { cleanup(dir); }
-  }));
-
-  // 8.11 prompt text references current value
-  await test('8.11 prompt message references current profile value (AC #2)', silenceLogs(async () => {
-    const dir = setupProject({ profile: 'on-prem', agents: ['claude-code'] });
-    try {
-      let seenMessage = null;
-      let seenInitial = null;
-      const stub = async (opts) => {
-        if (opts && opts.type === 'select' && !seenMessage) {
-          seenMessage = opts.message;
-          seenInitial = opts.initial;
-          return { chosenProfile: 'on-prem' }; // same-value → early exit
-        }
-        return { proceed: false };
-      };
-      await reconfigure({ projectRoot: dir, argv: [], promptsLib: stub });
-      assert.match(seenMessage || '', /Current profile: on-prem\. Change to\?/);
-      // AC: default-highlighted option is the persisted value → initial index 0 (on-prem row).
-      assert.strictEqual(seenInitial, 0);
-    } finally { cleanup(dir); }
-  }));
-
-  // listTouchedFiles smoke
-  await test('listTouchedFiles returns instructionFiles + extraInstructionTemplates, sorted', () => {
-    const files = listTouchedFiles('/tmp/x', [getAgent('roo-code')]);
-    assert.ok(files.includes('.roomodes'));
-    assert.ok(files.includes('.roo/rules/00-ma-agents.md'));
-  });
-}
-
-runAll().then(() => {
-  console.log(`\n ${passed} passed, ${failed} failed\n`);
-  if (failed > 0) {
-    console.error('FAILURES:');
-    errors.forEach(e => console.error(`  - ${e.name}: ${e.error}`));
-    process.exit(1);
-  }
-});

package/test/repo-layout.test.js

@@ -1,246 +0,0 @@
-#!/usr/bin/env node
-/**
- * Tests for Story 16.1: Repository Layout Wizard
- * Tests collectRepoLayout() function and its sub-flows
- */
-'use strict';
-
-const assert = require('assert');
-const path = require('path');
-const fs = require('fs');
-const os = require('os');
-
-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 });
-  }
-}
-
-async function testAsync(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 });
-  }
-}
-
-// ─── Load module ─────────────────────────────────────────────────────────────
-let collectRepoLayout;
-try {
-  ({ collectRepoLayout } = require('../bin/cli.js'));
-  if (typeof collectRepoLayout !== 'function') {
-    throw new Error('collectRepoLayout is not exported from cli.js');
-  }
-} catch (e) {
-  console.error('\n  FATAL: Cannot load collectRepoLayout from cli.js');
-  console.error(' ', e.message);
-  process.exit(1);
-}
-
-// ─── Task 5.1: --yes flag returns both as same mode ─────────────────────────
-console.log('\nTask 5.1 — collectRepoLayout with --yes flag (no env vars)');
-
-async function runTests() {
-  await testAsync('--yes returns both concerns as same mode', async () => {
-    // Clear env vars to ensure clean test
-    const origKb = process.env.MA_KNOWLEDGEBASE_PATH;
-    const origSp = process.env.MA_SPRINT_PATH;
-    delete process.env.MA_KNOWLEDGEBASE_PATH;
-    delete process.env.MA_SPRINT_PATH;
-
-    try {
-      const result = await collectRepoLayout({ yes: true });
-      assert.deepStrictEqual(result.knowledgebase, { mode: 'same', path: '.' });
-      assert.deepStrictEqual(result.sprintManagement, { mode: 'same', path: '.' });
-    } finally {
-      if (origKb !== undefined) process.env.MA_KNOWLEDGEBASE_PATH = origKb;
-      if (origSp !== undefined) process.env.MA_SPRINT_PATH = origSp;
-    }
-  });
-
-  // ─── Task 5.2: --yes + env vars returns local mode with resolved paths ────
-  console.log('\nTask 5.2 — collectRepoLayout with --yes + env vars');
-
-  await testAsync('--yes + MA_KNOWLEDGEBASE_PATH returns local mode with resolved path', async () => {
-    const origKb = process.env.MA_KNOWLEDGEBASE_PATH;
-    const origSp = process.env.MA_SPRINT_PATH;
-    process.env.MA_KNOWLEDGEBASE_PATH = '../kb-repo';
-    delete process.env.MA_SPRINT_PATH;
-
-    try {
-      const result = await collectRepoLayout({ yes: true });
-      assert.strictEqual(result.knowledgebase.mode, 'local');
-      assert.strictEqual(result.knowledgebase.path, path.resolve('../kb-repo'));
-      assert.deepStrictEqual(result.sprintManagement, { mode: 'same', path: '.' });
-    } finally {
-      if (origKb !== undefined) process.env.MA_KNOWLEDGEBASE_PATH = origKb;
-      else delete process.env.MA_KNOWLEDGEBASE_PATH;
-      if (origSp !== undefined) process.env.MA_SPRINT_PATH = origSp;
-    }
-  });
-
-  await testAsync('--yes + MA_SPRINT_PATH returns local mode with resolved path', async () => {
-    const origKb = process.env.MA_KNOWLEDGEBASE_PATH;
-    const origSp = process.env.MA_SPRINT_PATH;
-    delete process.env.MA_KNOWLEDGEBASE_PATH;
-    process.env.MA_SPRINT_PATH = '/absolute/sprint/path';
-
-    try {
-      const result = await collectRepoLayout({ yes: true });
-      assert.deepStrictEqual(result.knowledgebase, { mode: 'same', path: '.' });
-      assert.strictEqual(result.sprintManagement.mode, 'local');
-      assert.strictEqual(result.sprintManagement.path, path.resolve('/absolute/sprint/path'));
-    } finally {
-      if (origKb !== undefined) process.env.MA_KNOWLEDGEBASE_PATH = origKb;
-      if (origSp !== undefined) process.env.MA_SPRINT_PATH = origSp;
-      else delete process.env.MA_SPRINT_PATH;
-    }
-  });
-
-  await testAsync('--yes + both env vars returns local mode for both', async () => {
-    const origKb = process.env.MA_KNOWLEDGEBASE_PATH;
-    const origSp = process.env.MA_SPRINT_PATH;
-    process.env.MA_KNOWLEDGEBASE_PATH = '../shared-docs';
-    process.env.MA_SPRINT_PATH = '../shared-sprint';
-
-    try {
-      const result = await collectRepoLayout({ yes: true });
-      assert.strictEqual(result.knowledgebase.mode, 'local');
-      assert.strictEqual(result.knowledgebase.path, path.resolve('../shared-docs'));
-      assert.strictEqual(result.sprintManagement.mode, 'local');
-      assert.strictEqual(result.sprintManagement.path, path.resolve('../shared-sprint'));
-    } finally {
-      if (origKb !== undefined) process.env.MA_KNOWLEDGEBASE_PATH = origKb;
-      else delete process.env.MA_KNOWLEDGEBASE_PATH;
-      if (origSp !== undefined) process.env.MA_SPRINT_PATH = origSp;
-      else delete process.env.MA_SPRINT_PATH;
-    }
-  });
-
-  // ─── Task 5.3: Local path validation ──────────────────────────────────────
-  console.log('\nTask 5.3 — Local path validation (integration via spawnSync)');
-
-  const { spawnSync } = require('child_process');
-  const CLI_PATH = path.join(__dirname, '..', 'bin', 'cli.js');
-
-  // These tests verify behavior by checking the collectRepoLayout function directly
-  // The local path and remote flows require interactive prompts, so we test them
-  // through the --yes code path and validate the logic structurally.
-
-  test('collectRepoLayout exists and is a function', () => {
-    assert.strictEqual(typeof collectRepoLayout, 'function');
-  });
-
-  test('collectRepoLayout returns object with knowledgebase and sprintManagement keys', async () => {
-    delete process.env.MA_KNOWLEDGEBASE_PATH;
-    delete process.env.MA_SPRINT_PATH;
-    const result = await collectRepoLayout({ yes: true });
-    assert.ok(result.knowledgebase, 'should have knowledgebase key');
-    assert.ok(result.sprintManagement, 'should have sprintManagement key');
-    assert.ok(result.knowledgebase.mode, 'knowledgebase should have mode');
-    assert.ok(result.knowledgebase.path !== undefined, 'knowledgebase should have path');
-    assert.ok(result.sprintManagement.mode, 'sprintManagement should have mode');
-    assert.ok(result.sprintManagement.path !== undefined, 'sprintManagement should have path');
-  });
-
-  // ─── Task 5.4: execFileSync called with correct args (structural check) ───
-  console.log('\nTask 5.4 — Remote flow uses execFileSync with array args');
-
-  test('collectRemotePath function exists in cli.js source', () => {
-    const src = fs.readFileSync(path.join(__dirname, '..', 'bin', 'cli.js'), 'utf-8');
-    assert.ok(src.includes("execFileSync('git', ['clone',"), 'Should use execFileSync with array args for git clone');
-    assert.ok(!src.includes('execSync(`git clone'), 'Should NOT use execSync with template literal for git clone');
-    assert.ok(!src.includes("execSync('git clone"), 'Should NOT use execSync with string concat for git clone');
-  });
-
-  // ─── Task 5.5: Clone failure cleanup (structural check) ──────────────────
-  console.log('\nTask 5.5 — Clone failure: cleanup logic present');
-
-  test('clone failure cleanup removes partial directory', () => {
-    const src = fs.readFileSync(path.join(__dirname, '..', 'bin', 'cli.js'), 'utf-8');
-    assert.ok(src.includes('rmSync(destPath, { recursive: true, force: true })'),
-      'Should clean up partial clone directory on failure');
-    assert.ok(src.includes('existedBefore'),
-      'Should check if directory existed before clone attempt');
-  });
-
-  // ─── Task 5.6: Empty/whitespace input rejection (structural check) ────────
-  console.log('\nTask 5.6 — Empty/whitespace input rejection');
-
-  test('local path flow rejects empty input', () => {
-    const src = fs.readFileSync(path.join(__dirname, '..', 'bin', 'cli.js'), 'utf-8');
-    assert.ok(src.includes('Path cannot be empty'),
-      'Should have empty path rejection message');
-  });
-
-  test('remote flow rejects empty git URL', () => {
-    const src = fs.readFileSync(path.join(__dirname, '..', 'bin', 'cli.js'), 'utf-8');
-    assert.ok(src.includes('Git URL cannot be empty'),
-      'Should have empty git URL rejection message');
-  });
-
-  test('remote flow rejects empty destination path', () => {
-    const src = fs.readFileSync(path.join(__dirname, '..', 'bin', 'cli.js'), 'utf-8');
-    assert.ok(src.includes('Destination path cannot be empty'),
-      'Should have empty destination path rejection message');
-  });
-
-  // ─── Task 5.7: Nested-repo warning (structural check) ────────────────────
-  console.log('\nTask 5.7 — Nested-repo warning');
-
-  test('warns when path is inside cwd', () => {
-    const src = fs.readFileSync(path.join(__dirname, '..', 'bin', 'cli.js'), 'utf-8');
-    assert.ok(src.includes('Nested repos may cause git confusion'),
-      'Should warn about nested repos');
-  });
-
-  // ─── Integration: --yes mode with env vars via spawnSync ──────────────────
-  console.log('\nIntegration — --yes mode processes repo layout without prompts');
-
-  test('--yes mode completes without hanging (repo layout integrated)', () => {
-    // This test verifies the wizard completes with --yes flag
-    // which exercises the collectRepoLayout CI/CD path
-    const result = spawnSync(process.execPath, [CLI_PATH, 'install', '--yes'], {
-      timeout: 600000,
-      encoding: 'utf-8',
-      input: '',
-      stdio: ['pipe', 'pipe', 'pipe'],
-      cwd: path.join(__dirname, '..')
-    });
-
-    if (result.status === null) {
-      console.log('    SKIP: process timed out');
-      return;
-    }
-
-    assert.strictEqual(
-      result.status,
-      0,
-      `Expected exit 0, got ${result.status}.\nstdout: ${(result.stdout || '').slice(0, 500)}\nstderr: ${(result.stderr || '').slice(0, 500)}`
-    );
-  });
-
-  // ─── Summary ──────────────────────────────────────────────────────────────
-  console.log(`\n${passed} passed, ${failed} failed`);
-  if (errors.length > 0) {
-    console.log('\nFailed tests:');
-    errors.forEach(e => console.log(`  - ${e.name}: ${e.error}`));
-  }
-  if (failed > 0) process.exit(1);
-}
-
-runTests();