@sienklogic/plan-build-run 2.33.1 → 2.37.0

package/plugins/pbr/scripts/pbr-tools.js

@@ -3,8 +3,16 @@
 /**
  * pbr-tools.js — Structured JSON state operations for Plan-Build-Run skills.
  *
- * Provides read-only commands that return JSON, replacing LLM-based text parsing
- * of STATE.md, ROADMAP.md, and config.json. Skills call this via:
+ * Thin dispatcher that imports from lib/ modules. All core logic lives in:
+ *   lib/core.js    — Foundation utilities (parsers, file ops, constants)
+ *   lib/config.js  — Config loading, validation, depth profiles
+ *   lib/state.js   — STATE.md operations (load, update, patch, advance)
+ *   lib/roadmap.js — ROADMAP.md operations (parse, update status/plans)
+ *   lib/phase.js   — Phase operations (add, remove, list, info, plan-index)
+ *   lib/init.js    — Compound init commands (execute-phase, plan-phase, etc.)
+ *   lib/history.js — HISTORY.md operations (append, load)
+ *
+ * Skills call this via:
  *   node ${CLAUDE_PLUGIN_ROOT}/scripts/pbr-tools.js <command> [args]
  *
  * Commands:
@@ -21,10 +29,88 @@
  *   history append <type> <title> [body] — Append record to HISTORY.md
  *   history load                         — Load all HISTORY.md records as JSON
  *   llm metrics [--session <ISO>]        — Lifetime or session-scoped LLM usage metrics
+ *   validate-project        — Comprehensive .planning/ integrity check
+ *   phase add <slug> [--after N] — Add a new phase directory (with renumbering)
+ *   phase remove <N>             — Remove an empty phase directory (with renumbering)
+ *   phase list                   — List all phase directories with status
+ *
+ * Environment: PBR_PROJECT_ROOT — Override project root directory (used when hooks fire from subagent cwd)
  */
 
 const fs = require('fs');
 const path = require('path');
+
+// --- Import lib modules ---
+const {
+  KNOWN_AGENTS,
+  VALID_STATUS_TRANSITIONS,
+  validateStatusTransition,
+  output,
+  error,
+  parseYamlFrontmatter,
+  parseMustHaves,
+  findFiles,
+  tailLines,
+  countMustHaves,
+  determinePhaseStatus,
+  atomicWrite,
+  lockedFileUpdate,
+  writeActiveSkill
+} = require('./lib/core');
+
+const {
+  configLoad: _configLoad,
+  configClearCache: _configClearCache,
+  configValidate: _configValidate,
+  resolveDepthProfile,
+  DEPTH_PROFILE_DEFAULTS
+} = require('./lib/config');
+
+const {
+  parseStateMd,
+  updateLegacyStateField,
+  updateFrontmatterField,
+  stateLoad: _stateLoad,
+  stateCheckProgress: _stateCheckProgress,
+  stateUpdate: _stateUpdate,
+  statePatch: _statePatch,
+  stateAdvancePlan: _stateAdvancePlan,
+  stateRecordMetric: _stateRecordMetric
+} = require('./lib/state');
+
+const {
+  parseRoadmapMd,
+  findRoadmapRow,
+  updateTableRow,
+  roadmapUpdateStatus: _roadmapUpdateStatus,
+  roadmapUpdatePlans: _roadmapUpdatePlans
+} = require('./lib/roadmap');
+
+const {
+  frontmatter: _frontmatter,
+  planIndex: _planIndex,
+  mustHavesCollect: _mustHavesCollect,
+  phaseInfo: _phaseInfo,
+  phaseAdd: _phaseAdd,
+  phaseRemove: _phaseRemove,
+  phaseList: _phaseList
+} = require('./lib/phase');
+
+const {
+  initExecutePhase: _initExecutePhase,
+  initPlanPhase: _initPlanPhase,
+  initQuick: _initQuick,
+  initVerifyWork: _initVerifyWork,
+  initResume: _initResume,
+  initProgress: _initProgress
+} = require('./lib/init');
+
+const {
+  historyAppend: _historyAppend,
+  historyLoad: _historyLoad
+} = require('./lib/history');
+
+// --- Local LLM imports (not extracted — separate module tree) ---
 const { resolveConfig, checkHealth } = require('./local-llm/health');
 const { classifyArtifact } = require('./local-llm/operations/classify-artifact');
 const { scoreSource } = require('./local-llm/operations/score-source');
@@ -33,188 +119,242 @@ const { summarizeContext } = require('./local-llm/operations/summarize-context')
 const { readSessionMetrics, summarizeMetrics, computeLifetimeMetrics } = require('./local-llm/metrics');
 const { computeThresholdAdjustments } = require('./local-llm/threshold-tuner');
 
-const cwd = process.cwd();
-const planningDir = path.join(cwd, '.planning');
+// --- Module-level state (for backwards compatibility) ---
 
-// --- Phase status transition state machine ---
+let cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
+let planningDir = path.join(cwd, '.planning');
 
-/**
- * Valid phase status transitions. Each key is a current status, and its value
- * is an array of statuses that are legal to transition to. This is advisory —
- * invalid transitions produce a stderr warning but are not blocked, to avoid
- * breaking existing workflows.
- *
- * State machine:
- *   pending -> planned, skipped
- *   planned -> building
- *   building -> built, partial, needs_fixes
- *   built -> verified, needs_fixes
- *   partial -> building, needs_fixes
- *   verified -> building (re-execution)
- *   needs_fixes -> planned, building
- *   skipped -> pending (unskip)
- */
-const VALID_STATUS_TRANSITIONS = {
-  pending:     ['planned', 'skipped'],
-  planned:     ['building'],
-  building:    ['built', 'partial', 'needs_fixes'],
-  built:       ['verified', 'needs_fixes'],
-  partial:     ['building', 'needs_fixes'],
-  verified:    ['building'],
-  needs_fixes: ['planned', 'building'],
-  skipped:     ['pending']
-};
+// --- Wrapper functions that pass planningDir to lib modules ---
+// These preserve the original function signatures (no planningDir param)
+// so existing callers (hook scripts, tests) continue to work.
 
-/**
- * Check whether a phase status transition is valid according to the state machine.
- * Returns { valid, warning? } — never blocks, only advises.
- *
- * @param {string} oldStatus - Current phase status
- * @param {string} newStatus - Desired phase status
- * @returns {{ valid: boolean, warning?: string }}
- */
-function validateStatusTransition(oldStatus, newStatus) {
-  const from = (oldStatus || '').trim().toLowerCase();
-  const to = (newStatus || '').trim().toLowerCase();
+function configLoad(dir) {
+  return _configLoad(dir || planningDir);
+}
 
-  // If the status isn't changing, that's always fine
-  if (from === to) {
-    return { valid: true };
-  }
+function configClearCache() {
+  _configClearCache();
+  cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
+  planningDir = path.join(cwd, '.planning');
+}
 
-  // If the old status is unknown to our map, we can't validate — allow it
-  if (!VALID_STATUS_TRANSITIONS[from]) {
-    return { valid: true };
-  }
+function configValidate(preloadedConfig) {
+  return _configValidate(preloadedConfig, planningDir);
+}
 
-  const allowed = VALID_STATUS_TRANSITIONS[from];
-  if (allowed.includes(to)) {
-    return { valid: true };
-  }
+function stateLoad() {
+  return _stateLoad(planningDir);
+}
 
-  return {
-    valid: false,
-    warning: `Suspicious status transition: "${from}" -> "${to}". Expected one of: [${allowed.join(', ')}]. Proceeding anyway (advisory).`
-  };
+function stateCheckProgress() {
+  return _stateCheckProgress(planningDir);
 }
 
-// --- Cached config loader ---
+function stateUpdate(field, value) {
+  return _stateUpdate(field, value, planningDir);
+}
 
-let _configCache = null;
-let _configMtime = 0;
-let _configPath = null;
+function statePatch(jsonStr) {
+  return _statePatch(jsonStr, planningDir);
+}
 
-/**
- * Load config.json with in-process mtime-based caching.
- * Returns the parsed config object, or null if not found / parse error.
- * Cache invalidates when file mtime changes or path differs.
- *
- * @param {string} [dir] - Path to .planning directory (defaults to cwd/.planning)
- * @returns {object|null} Parsed config or null
- */
-function configLoad(dir) {
-  const configPath = path.join(dir || planningDir, 'config.json');
-  try {
-    if (!fs.existsSync(configPath)) return null;
-    const stat = fs.statSync(configPath);
-    const mtime = stat.mtimeMs;
-    if (_configCache && mtime === _configMtime && configPath === _configPath) {
-      return _configCache;
-    }
-    _configCache = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-    _configMtime = mtime;
-    _configPath = configPath;
-    return _configCache;
-  } catch (_e) {
-    return null;
-  }
+function stateAdvancePlan() {
+  return _stateAdvancePlan(planningDir);
 }
 
-/**
- * Clear the configLoad() in-process cache.
- * Useful in tests where multiple temp directories are used in rapid succession.
- */
-function configClearCache() {
-  _configCache = null;
-  _configMtime = 0;
-  _configPath = null;
+function stateRecordMetric(metricArgs) {
+  return _stateRecordMetric(metricArgs, planningDir);
 }
 
-/**
- * Read the last N lines from a file efficiently.
- * Reads the entire file but only parses (JSON.parse) the trailing entries.
- * For JSONL files where full parsing is expensive, this avoids parsing
- * all lines when you only need recent entries.
- *
- * @param {string} filePath - Absolute path to the file
- * @param {number} n - Number of trailing lines to return
- * @returns {string[]} Array of raw line strings (last n lines)
- */
-function tailLines(filePath, n) {
-  try {
-    if (!fs.existsSync(filePath)) return [];
-    const content = fs.readFileSync(filePath, 'utf8').trim();
-    if (!content) return [];
-    const lines = content.split('\n');
-    if (lines.length <= n) return lines;
-    return lines.slice(lines.length - n);
-  } catch (_e) {
-    return [];
-  }
+function roadmapUpdateStatus(phaseNum, newStatus) {
+  return _roadmapUpdateStatus(phaseNum, newStatus, planningDir);
+}
+
+function roadmapUpdatePlans(phaseNum, complete, total) {
+  return _roadmapUpdatePlans(phaseNum, complete, total, planningDir);
+}
+
+function frontmatter(filePath) {
+  return _frontmatter(filePath);
+}
+
+function planIndex(phaseNum) {
+  return _planIndex(phaseNum, planningDir);
+}
+
+function mustHavesCollect(phaseNum) {
+  return _mustHavesCollect(phaseNum, planningDir);
+}
+
+function phaseInfo(phaseNum) {
+  return _phaseInfo(phaseNum, planningDir);
+}
+
+function phaseAdd(slug, afterPhase) {
+  return _phaseAdd(slug, afterPhase, planningDir);
+}
+
+function phaseRemove(phaseNum) {
+  return _phaseRemove(phaseNum, planningDir);
+}
+
+function phaseList() {
+  return _phaseList(planningDir);
+}
+
+function initExecutePhase(phaseNum) {
+  return _initExecutePhase(phaseNum, planningDir);
+}
+
+function initPlanPhase(phaseNum) {
+  return _initPlanPhase(phaseNum, planningDir);
+}
+
+function initQuick(description) {
+  return _initQuick(description, planningDir);
+}
+
+function initVerifyWork(phaseNum) {
+  return _initVerifyWork(phaseNum, planningDir);
+}
+
+function initResume() {
+  return _initResume(planningDir);
+}
+
+function initProgress() {
+  return _initProgress(planningDir);
+}
+
+function historyAppend(entry, dir) {
+  return _historyAppend(entry, dir || planningDir);
+}
+
+function historyLoad(dir) {
+  return _historyLoad(dir || planningDir);
 }
 
+// --- validateProject stays here (cross-cutting across modules) ---
+
 /**
- * Built-in depth profile defaults. These define the effective settings
- * for each depth level. User config.depth_profiles overrides these.
+ * Comprehensive .planning/ integrity check.
+ * Returns { valid, errors, warnings, checks } — errors mean workflow should not proceed.
  */
-const DEPTH_PROFILE_DEFAULTS = {
-  quick: {
-    'features.research_phase': false,
-    'features.plan_checking': false,
-    'features.goal_verification': false,
-    'features.inline_verify': false,
-    'scan.mapper_count': 2,
-    'scan.mapper_areas': ['tech', 'arch'],
-    'debug.max_hypothesis_rounds': 3
-  },
-  standard: {
-    'features.research_phase': true,
-    'features.plan_checking': true,
-    'features.goal_verification': true,
-    'features.inline_verify': false,
-    'scan.mapper_count': 4,
-    'scan.mapper_areas': ['tech', 'arch', 'quality', 'concerns'],
-    'debug.max_hypothesis_rounds': 5
-  },
-  comprehensive: {
-    'features.research_phase': true,
-    'features.plan_checking': true,
-    'features.goal_verification': true,
-    'features.inline_verify': true,
-    'scan.mapper_count': 4,
-    'scan.mapper_areas': ['tech', 'arch', 'quality', 'concerns'],
-    'debug.max_hypothesis_rounds': 10
+function validateProject() {
+  const checks = [];
+  const errors = [];
+  const warnings = [];
+
+  // 1. .planning/ directory exists
+  if (!fs.existsSync(planningDir)) {
+    return { valid: false, errors: ['.planning/ directory not found'], warnings: [], checks: ['directory_exists: FAIL'] };
   }
-};
+  checks.push('directory_exists: PASS');
 
-/**
- * Resolve the effective depth profile for the current config.
- * Merges built-in defaults with any user overrides from config.depth_profiles.
- *
- * @param {object|null} config - Parsed config.json (from configLoad). If null, returns 'standard' defaults.
- * @returns {{ depth: string, profile: object }} The resolved depth name and flattened profile settings.
- */
-function resolveDepthProfile(config) {
-  const depth = (config && config.depth) || 'standard';
-  const defaults = DEPTH_PROFILE_DEFAULTS[depth] || DEPTH_PROFILE_DEFAULTS.standard;
+  // 2. config.json exists and is valid
+  const config = configLoad();
+  if (!config) {
+    errors.push('config.json missing or invalid JSON');
+    checks.push('config_valid: FAIL');
+  } else {
+    const configResult = configValidate(config);
+    if (!configResult.valid) {
+      errors.push(...configResult.errors.map(e => 'config: ' + e));
+    }
+    warnings.push(...(configResult.warnings || []).map(w => 'config: ' + w));
+    checks.push('config_valid: ' + (configResult.valid ? 'PASS' : 'FAIL'));
+  }
+
+  // 3. STATE.md exists and has valid frontmatter
+  const statePath = path.join(planningDir, 'STATE.md');
+  if (!fs.existsSync(statePath)) {
+    errors.push('STATE.md not found');
+    checks.push('state_exists: FAIL');
+  } else {
+    try {
+      const stateContent = fs.readFileSync(statePath, 'utf8');
+      const fm = parseYamlFrontmatter(stateContent);
+      if (!fm || !fm.current_phase) {
+        warnings.push('STATE.md frontmatter missing current_phase');
+        checks.push('state_frontmatter: WARN');
+      } else {
+        checks.push('state_frontmatter: PASS');
+      }
+    } catch (e) {
+      errors.push('STATE.md unreadable: ' + e.message);
+      checks.push('state_readable: FAIL');
+    }
+  }
+
+  // 4. ROADMAP.md exists
+  const roadmapPath = path.join(planningDir, 'ROADMAP.md');
+  if (!fs.existsSync(roadmapPath)) {
+    warnings.push('ROADMAP.md not found (may be a new project)');
+    checks.push('roadmap_exists: WARN');
+  } else {
+    checks.push('roadmap_exists: PASS');
+  }
+
+  // 5. Phase directory matches STATE.md current_phase
+  try {
+    if (fs.existsSync(statePath)) {
+      const stateContent = fs.readFileSync(statePath, 'utf8');
+      const fm = parseYamlFrontmatter(stateContent);
+      if (fm && fm.current_phase) {
+        const phaseNum = String(fm.current_phase).padStart(2, '0');
+        const phasesDir = path.join(planningDir, 'phases');
+        if (fs.existsSync(phasesDir)) {
+          const dirs = fs.readdirSync(phasesDir).filter(d => d.startsWith(phaseNum + '-'));
+          if (dirs.length === 0) {
+            warnings.push(`Phase directory for current_phase ${fm.current_phase} not found in .planning/phases/`);
+            checks.push('phase_directory: WARN');
+          } else {
+            checks.push('phase_directory: PASS');
+          }
+        }
+      }
+    }
+  } catch (_e) { /* best effort */ }
+
+  // 6. No stale .active-skill (>2 hours old)
+  const activeSkillPath = path.join(planningDir, '.active-skill');
+  if (fs.existsSync(activeSkillPath)) {
+    try {
+      const stat = fs.statSync(activeSkillPath);
+      const ageMs = Date.now() - stat.mtimeMs;
+      if (ageMs > 2 * 60 * 60 * 1000) {
+        const ageHours = Math.round(ageMs / (60 * 60 * 1000));
+        warnings.push(`.active-skill is ${ageHours}h old — may be stale from a crashed session`);
+        checks.push('active_skill_fresh: WARN');
+      } else {
+        checks.push('active_skill_fresh: PASS');
+      }
+    } catch (_e) { checks.push('active_skill_fresh: SKIP'); }
+  } else {
+    checks.push('active_skill_fresh: SKIP');
+  }
 
-  // Merge user overrides if present
-  const userOverrides = (config && config.depth_profiles && config.depth_profiles[depth]) || {};
-  const profile = { ...defaults, ...userOverrides };
+  // 7. No .tmp files left from atomic writes
+  try {
+    const tmpFiles = fs.readdirSync(planningDir).filter(f => f.endsWith('.tmp.' + process.pid) || f.match(/\.tmp\.\d+$/));
+    if (tmpFiles.length > 0) {
+      warnings.push(`Found ${tmpFiles.length} leftover temp files in .planning/: ${tmpFiles.join(', ')}`);
+      checks.push('no_temp_files: WARN');
+    } else {
+      checks.push('no_temp_files: PASS');
+    }
+  } catch (_e) { /* best effort */ }
 
-  return { depth, profile };
+  return {
+    valid: errors.length === 0,
+    errors,
+    warnings,
+    checks
+  };
 }
 
+// --- CLI entry point ---
+
 async function main() {
   const args = process.argv.slice(2);
   const command = args[0];
@@ -409,1087 +549,58 @@ async function main() {
       output(suggestions.length > 0
         ? { suggestions }
         : { suggestions: [], message: 'Not enough shadow samples yet (need >= 20 per operation)' });
+    // --- Compound init commands ---
+    } else if (command === "init" && subcommand === "execute-phase") {
+      const phase = args[2];
+      if (!phase) error("Usage: pbr-tools.js init execute-phase <phase-number>");
+      output(initExecutePhase(phase));
+    } else if (command === "init" && subcommand === "plan-phase") {
+      const phase = args[2];
+      if (!phase) error("Usage: pbr-tools.js init plan-phase <phase-number>");
+      output(initPlanPhase(phase));
+    } else if (command === "init" && subcommand === "quick") {
+      const desc = args.slice(2).join(" ") || "";
+      output(initQuick(desc));
+    } else if (command === "init" && subcommand === "verify-work") {
+      const phase = args[2];
+      if (!phase) error("Usage: pbr-tools.js init verify-work <phase-number>");
+      output(initVerifyWork(phase));
+    } else if (command === "init" && subcommand === "resume") {
+      output(initResume());
+    } else if (command === "init" && subcommand === "progress") {
+      output(initProgress());
+    // --- State patch/advance/metric ---
+    } else if (command === "state" && subcommand === "patch") {
+      const jsonStr = args[2];
+      if (!jsonStr) error("Usage: pbr-tools.js state patch JSON");
+      output(statePatch(jsonStr));
+    } else if (command === "state" && subcommand === "advance-plan") {
+      output(stateAdvancePlan());
+    } else if (command === "state" && subcommand === "record-metric") {
+      output(stateRecordMetric(args.slice(2)));
+    } else if (command === 'phase' && subcommand === 'add') {
+      const slug = args[2];
+      if (!slug) { error('Usage: phase add <slug> [--after <phase_num>]'); }
+      const afterIdx = args.indexOf('--after');
+      const afterPhase = afterIdx !== -1 ? args[afterIdx + 1] : null;
+      output(phaseAdd(slug, afterPhase));
+    } else if (command === 'phase' && subcommand === 'remove') {
+      const phaseNum = args[2];
+      if (!phaseNum) { error('Usage: phase remove <phase_num>'); }
+      output(phaseRemove(phaseNum));
+    } else if (command === 'phase' && subcommand === 'list') {
+      output(phaseList());
+    } else if (command === 'validate-project') {
+      output(validateProject());
     } else {
-      error(`Unknown command: ${args.join(' ')}\nCommands: state load|check-progress|update, config validate, plan-index, frontmatter, must-haves, phase-info, roadmap update-status|update-plans, history append|load, event, llm health|status|classify|score-source|classify-error|summarize|metrics [--session <ISO>]|adjust-thresholds`);
+      error(`Unknown command: ${args.join(' ')}\nCommands: state load|check-progress|update|patch|advance-plan|record-metric, config validate, validate-project, init execute-phase|plan-phase|quick|verify-work|resume|progress, plan-index, frontmatter, must-haves, phase-info, phase add|remove|list, roadmap update-status|update-plans, history append|load, event, llm health|status|classify|score-source|classify-error|summarize|metrics [--session <ISO>]|adjust-thresholds`);
     }
   } catch (e) {
     error(e.message);
   }
 }
 
-// --- Commands ---
-
-function stateLoad() {
-  const result = {
-    exists: false,
-    config: null,
-    state: null,
-    roadmap: null,
-    phase_count: 0,
-    current_phase: null,
-    progress: null
-  };
-
-  if (!fs.existsSync(planningDir)) {
-    return result;
-  }
-  result.exists = true;
-
-  // Load config.json
-  const configPath = path.join(planningDir, 'config.json');
-  if (fs.existsSync(configPath)) {
-    try {
-      result.config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-    } catch (_) {
-      result.config = { _error: 'Failed to parse config.json' };
-    }
-  }
-
-  // Load STATE.md
-  const statePath = path.join(planningDir, 'STATE.md');
-  if (fs.existsSync(statePath)) {
-    const content = fs.readFileSync(statePath, 'utf8');
-    result.state = parseStateMd(content);
-  }
-
-  // Load ROADMAP.md
-  const roadmapPath = path.join(planningDir, 'ROADMAP.md');
-  if (fs.existsSync(roadmapPath)) {
-    const content = fs.readFileSync(roadmapPath, 'utf8');
-    result.roadmap = parseRoadmapMd(content);
-    result.phase_count = result.roadmap.phases.length;
-  }
-
-  // Extract current phase
-  if (result.state && result.state.current_phase) {
-    result.current_phase = result.state.current_phase;
-  }
-
-  // Calculate progress
-  result.progress = calculateProgress();
-
-  return result;
-}
-
-function stateCheckProgress() {
-  const phasesDir = path.join(planningDir, 'phases');
-  if (!fs.existsSync(phasesDir)) {
-    return { phases: [], total_plans: 0, completed_plans: 0, percentage: 0 };
-  }
-
-  const phases = [];
-  let totalPlans = 0;
-  let completedPlans = 0;
-
-  const entries = fs.readdirSync(phasesDir, { withFileTypes: true })
-    .filter(e => e.isDirectory())
-    .sort((a, b) => a.name.localeCompare(b.name));
-
-  for (const entry of entries) {
-    const phaseDir = path.join(phasesDir, entry.name);
-    const plans = findFiles(phaseDir, /-PLAN\.md$/);
-    const summaries = findFiles(phaseDir, /^SUMMARY-.*\.md$/);
-    const verification = fs.existsSync(path.join(phaseDir, 'VERIFICATION.md'));
-
-    const completedSummaries = summaries.filter(s => {
-      const content = fs.readFileSync(path.join(phaseDir, s), 'utf8');
-      return /status:\s*["']?complete/i.test(content);
-    });
-
-    const phaseInfo = {
-      directory: entry.name,
-      plans: plans.length,
-      summaries: summaries.length,
-      completed: completedSummaries.length,
-      has_verification: verification,
-      status: determinePhaseStatus(plans.length, completedSummaries.length, summaries.length, verification, phaseDir)
-    };
-
-    phases.push(phaseInfo);
-    totalPlans += plans.length;
-    completedPlans += completedSummaries.length;
-  }
-
-  return {
-    phases,
-    total_plans: totalPlans,
-    completed_plans: completedPlans,
-    percentage: totalPlans > 0 ? Math.round((completedPlans / totalPlans) * 100) : 0
-  };
-}
-
-function planIndex(phaseNum) {
-  const phasesDir = path.join(planningDir, 'phases');
-  if (!fs.existsSync(phasesDir)) {
-    return { error: 'No phases directory found' };
-  }
-
-  // Find phase directory matching the number
-  const entries = fs.readdirSync(phasesDir, { withFileTypes: true })
-    .filter(e => e.isDirectory());
-
-  const phaseDir = entries.find(e => e.name.startsWith(phaseNum.padStart(2, '0') + '-'));
-  if (!phaseDir) {
-    return { error: `No phase directory found matching phase ${phaseNum}` };
-  }
-
-  const fullDir = path.join(phasesDir, phaseDir.name);
-  const planFiles = findFiles(fullDir, /-PLAN\.md$/);
-
-  const plans = [];
-  const waves = {};
-
-  for (const file of planFiles) {
-    const content = fs.readFileSync(path.join(fullDir, file), 'utf8');
-    const frontmatter = parseYamlFrontmatter(content);
-
-    const plan = {
-      file,
-      plan_id: frontmatter.plan || file.replace(/-PLAN\.md$/, ''),
-      wave: parseInt(frontmatter.wave, 10) || 1,
-      type: frontmatter.type || 'unknown',
-      autonomous: frontmatter.autonomous !== false,
-      depends_on: frontmatter.depends_on || [],
-      gap_closure: frontmatter.gap_closure || false,
-      has_summary: fs.existsSync(path.join(fullDir, `SUMMARY-${frontmatter.plan || ''}.md`)),
-      must_haves_count: countMustHaves(frontmatter.must_haves)
-    };
-
-    plans.push(plan);
-
-    const waveKey = `wave_${plan.wave}`;
-    if (!waves[waveKey]) waves[waveKey] = [];
-    waves[waveKey].push(plan.plan_id);
-  }
-
-  return {
-    phase: phaseDir.name,
-    total_plans: plans.length,
-    plans,
-    waves
-  };
-}
-
-function configValidate(preloadedConfig) {
-  let config;
-  if (preloadedConfig) {
-    config = preloadedConfig;
-  } else {
-    const configPath = path.join(planningDir, 'config.json');
-    if (!fs.existsSync(configPath)) {
-      return { valid: false, errors: ['config.json not found'], warnings: [] };
-    }
-
-    try {
-      config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-    } catch (e) {
-      return { valid: false, errors: [`config.json is not valid JSON: ${e.message}`], warnings: [] };
-    }
-  }
-
-  const schema = JSON.parse(fs.readFileSync(path.join(__dirname, 'config-schema.json'), 'utf8'));
-  const warnings = [];
-  const errors = [];
-
-  validateObject(config, schema, '', errors, warnings);
-
-  // Semantic conflict detection — logical contradictions that pass schema validation
-  // Clear contradictions → errors; ambiguous/preference issues → warnings
-  if (config.mode === 'autonomous' && config.gates) {
-    const activeGates = Object.entries(config.gates || {}).filter(([, v]) => v === true).map(([k]) => k);
-    if (activeGates.length > 0) {
-      errors.push(`mode=autonomous with active gates (${activeGates.join(', ')}): gates are unreachable in autonomous mode`);
-    }
-  }
-  if (config.features && config.features.auto_continue && config.mode === 'interactive') {
-    warnings.push('features.auto_continue=true with mode=interactive: auto_continue only fires in autonomous mode');
-  }
-  if (config.parallelization) {
-    if (config.parallelization.enabled === false && config.parallelization.plan_level === true) {
-      warnings.push('parallelization.enabled=false with plan_level=true: plan_level is ignored when parallelization is disabled');
-    }
-    if (config.parallelization.max_concurrent_agents === 1 && config.teams && config.teams.coordination) {
-      errors.push('parallelization.max_concurrent_agents=1 with teams.coordination set: teams require concurrent agents to be useful');
-    }
-  }
-
-  return {
-    valid: errors.length === 0,
-    errors,
-    warnings
-  };
-}
-
-// --- New read-only commands ---
-
-/**
- * Parse a markdown file's YAML frontmatter and return as JSON.
- * Wraps parseYamlFrontmatter() + parseMustHaves().
- */
-function frontmatter(filePath) {
-  const resolved = path.resolve(filePath);
-  if (!fs.existsSync(resolved)) {
-    return { error: `File not found: ${resolved}` };
-  }
-  const content = fs.readFileSync(resolved, 'utf8');
-  return parseYamlFrontmatter(content);
-}
-
-/**
- * Collect all must-haves from all PLAN.md files in a phase.
- * Returns per-plan grouping + flat deduplicated list + total count.
- */
-function mustHavesCollect(phaseNum) {
-  const phasesDir = path.join(planningDir, 'phases');
-  if (!fs.existsSync(phasesDir)) {
-    return { error: 'No phases directory found' };
-  }
-
-  const entries = fs.readdirSync(phasesDir, { withFileTypes: true })
-    .filter(e => e.isDirectory());
-  const phaseDir = entries.find(e => e.name.startsWith(phaseNum.padStart(2, '0') + '-'));
-  if (!phaseDir) {
-    return { error: `No phase directory found matching phase ${phaseNum}` };
-  }
-
-  const fullDir = path.join(phasesDir, phaseDir.name);
-  const planFiles = findFiles(fullDir, /-PLAN\.md$/);
-
-  const perPlan = {};
-  const allTruths = new Set();
-  const allArtifacts = new Set();
-  const allKeyLinks = new Set();
-
-  for (const file of planFiles) {
-    const content = fs.readFileSync(path.join(fullDir, file), 'utf8');
-    const fm = parseYamlFrontmatter(content);
-    const planId = fm.plan || file.replace(/-PLAN\.md$/, '');
-    const mh = fm.must_haves || { truths: [], artifacts: [], key_links: [] };
-
-    perPlan[planId] = mh;
-    (mh.truths || []).forEach(t => allTruths.add(t));
-    (mh.artifacts || []).forEach(a => allArtifacts.add(a));
-    (mh.key_links || []).forEach(k => allKeyLinks.add(k));
-  }
-
-  const all = {
-    truths: [...allTruths],
-    artifacts: [...allArtifacts],
-    key_links: [...allKeyLinks]
-  };
-
-  return {
-    phase: phaseDir.name,
-    plans: perPlan,
-    all,
-    total: all.truths.length + all.artifacts.length + all.key_links.length
-  };
-}
-
-/**
- * Comprehensive single-phase status combining roadmap, filesystem, and plan data.
- */
-function phaseInfo(phaseNum) {
-  const phasesDir = path.join(planningDir, 'phases');
-  if (!fs.existsSync(phasesDir)) {
-    return { error: 'No phases directory found' };
-  }
-
-  const entries = fs.readdirSync(phasesDir, { withFileTypes: true })
-    .filter(e => e.isDirectory());
-  const phaseDir = entries.find(e => e.name.startsWith(phaseNum.padStart(2, '0') + '-'));
-  if (!phaseDir) {
-    return { error: `No phase directory found matching phase ${phaseNum}` };
-  }
-
-  const fullDir = path.join(phasesDir, phaseDir.name);
-
-  // Get roadmap info
-  let roadmapInfo = null;
-  const roadmapPath = path.join(planningDir, 'ROADMAP.md');
-  if (fs.existsSync(roadmapPath)) {
-    const roadmapContent = fs.readFileSync(roadmapPath, 'utf8');
-    const roadmap = parseRoadmapMd(roadmapContent);
-    roadmapInfo = roadmap.phases.find(p => p.number === phaseNum.padStart(2, '0')) || null;
-  }
-
-  // Get plan index
-  const plans = planIndex(phaseNum);
-
-  // Check for verification
-  const verificationPath = path.join(fullDir, 'VERIFICATION.md');
-  let verification = null;
-  if (fs.existsSync(verificationPath)) {
-    const vContent = fs.readFileSync(verificationPath, 'utf8');
-    verification = parseYamlFrontmatter(vContent);
-  }
-
-  // Check summaries
-  const summaryFiles = findFiles(fullDir, /^SUMMARY-.*\.md$/);
-  const summaries = summaryFiles.map(f => {
-    const content = fs.readFileSync(path.join(fullDir, f), 'utf8');
-    const fm = parseYamlFrontmatter(content);
-    return { file: f, plan: fm.plan || f.replace(/^SUMMARY-|\.md$/g, ''), status: fm.status || 'unknown' };
-  });
-
-  // Determine filesystem status
-  const planCount = plans.total_plans || 0;
-  const completedCount = summaries.filter(s => s.status === 'complete').length;
-  const hasVerification = fs.existsSync(verificationPath);
-  const fsStatus = determinePhaseStatus(planCount, completedCount, summaryFiles.length, hasVerification, fullDir);
-
-  return {
-    phase: phaseDir.name,
-    name: roadmapInfo ? roadmapInfo.name : phaseDir.name.replace(/^\d+-/, ''),
-    goal: roadmapInfo ? roadmapInfo.goal : null,
-    roadmap_status: roadmapInfo ? roadmapInfo.status : null,
-    filesystem_status: fsStatus,
-    plans: plans.plans || [],
-    plan_count: planCount,
-    summaries,
-    completed: completedCount,
-    verification,
-    has_context: fs.existsSync(path.join(fullDir, 'CONTEXT.md'))
-  };
-}
-
-// --- Mutation commands ---
-
-/**
- * Atomically update a field in STATE.md using lockedFileUpdate.
- * Supports both legacy and frontmatter (v2) formats.
- *
- * @param {string} field - One of: current_phase, status, plans_complete, last_activity
- * @param {string} value - New value (use 'now' for last_activity to auto-timestamp)
- */
-function stateUpdate(field, value) {
-  const statePath = path.join(planningDir, 'STATE.md');
-  if (!fs.existsSync(statePath)) {
-    return { success: false, error: 'STATE.md not found' };
-  }
-
-  const validFields = ['current_phase', 'status', 'plans_complete', 'last_activity'];
-  if (!validFields.includes(field)) {
-    return { success: false, error: `Invalid field: ${field}. Valid fields: ${validFields.join(', ')}` };
-  }
-
-  // Auto-timestamp
-  if (field === 'last_activity' && value === 'now') {
-    value = new Date().toISOString().slice(0, 19).replace('T', ' ');
-  }
-
-  const result = lockedFileUpdate(statePath, (content) => {
-    const fm = parseYamlFrontmatter(content);
-    if (fm.version === 2 || fm.current_phase !== undefined) {
-      return updateFrontmatterField(content, field, value);
-    }
-    return updateLegacyStateField(content, field, value);
-  });
-
-  if (result.success) {
-    return { success: true, field, value };
-  }
-  return { success: false, error: result.error };
-}
-
-/**
- * Append a record to HISTORY.md. Creates the file if it doesn't exist.
- * Each entry is a markdown section appended at the end.
- *
- * @param {object} entry - { type: 'milestone'|'phase', title: string, body: string }
- * @param {string} [dir] - Path to .planning directory (defaults to cwd/.planning)
- * @returns {{success: boolean, error?: string}}
- */
-function historyAppend(entry, dir) {
-  const historyPath = path.join(dir || planningDir, 'HISTORY.md');
-  const timestamp = new Date().toISOString().slice(0, 10);
-
-  let header = '';
-  if (!fs.existsSync(historyPath)) {
-    header = '# Project History\n\nCompleted milestones and phase records. This file is append-only.\n\n';
-  }
-
-  const section = `${header}## ${entry.type === 'milestone' ? 'Milestone' : 'Phase'}: ${entry.title}\n_Completed: ${timestamp}_\n\n${entry.body.trim()}\n\n---\n\n`;
-
-  try {
-    fs.appendFileSync(historyPath, section, 'utf8');
-    return { success: true };
-  } catch (e) {
-    return { success: false, error: e.message };
-  }
-}
-
-/**
- * Load HISTORY.md and parse it into structured records.
- * Returns null if HISTORY.md doesn't exist.
- *
- * @param {string} [dir] - Path to .planning directory
- * @returns {object|null} { records: [{type, title, date, body}], line_count }
- */
-function historyLoad(dir) {
-  const historyPath = path.join(dir || planningDir, 'HISTORY.md');
-  if (!fs.existsSync(historyPath)) return null;
-
-  const content = fs.readFileSync(historyPath, 'utf8');
-  const records = [];
-  const sectionRegex = /^## (Milestone|Phase): (.+)\n_Completed: (\d{4}-\d{2}-\d{2})_\n\n([\s\S]*?)(?=\n---|\s*$)/gm;
-
-  let match;
-  while ((match = sectionRegex.exec(content)) !== null) {
-    records.push({
-      type: match[1].toLowerCase(),
-      title: match[2].trim(),
-      date: match[3],
-      body: match[4].trim()
-    });
-  }
-
-  return {
-    records,
-    line_count: content.split('\n').length
-  };
-}
-
-/**
- * Update the Status column for a phase in ROADMAP.md's Phase Overview table.
- */
-function roadmapUpdateStatus(phaseNum, newStatus) {
-  const roadmapPath = path.join(planningDir, 'ROADMAP.md');
-  if (!fs.existsSync(roadmapPath)) {
-    return { success: false, error: 'ROADMAP.md not found' };
-  }
-
-  let oldStatus = null;
-
-  const result = lockedFileUpdate(roadmapPath, (content) => {
-    const lines = content.split('\n');
-    const rowIdx = findRoadmapRow(lines, phaseNum);
-    if (rowIdx === -1) {
-      return content; // No matching row found
-    }
-    const parts = lines[rowIdx].split('|');
-    oldStatus = parts[6] ? parts[6].trim() : 'unknown';
-    lines[rowIdx] = updateTableRow(lines[rowIdx], 5, newStatus);
-    return lines.join('\n');
-  });
-
-  if (!oldStatus) {
-    return { success: false, error: `Phase ${phaseNum} not found in ROADMAP.md table` };
-  }
-
-  // Advisory transition validation — warn on suspicious transitions but don't block
-  const transition = validateStatusTransition(oldStatus, newStatus);
-  if (!transition.valid && transition.warning) {
-    process.stderr.write(`[pbr-tools] WARNING: ${transition.warning}\n`);
-  }
-
-  if (result.success) {
-    const response = { success: true, old_status: oldStatus, new_status: newStatus };
-    if (!transition.valid) {
-      response.transition_warning = transition.warning;
-    }
-    return response;
-  }
-  return { success: false, error: result.error };
-}
-
-/**
- * Update the Plans column for a phase in ROADMAP.md's Phase Overview table.
- */
-function roadmapUpdatePlans(phaseNum, complete, total) {
-  const roadmapPath = path.join(planningDir, 'ROADMAP.md');
-  if (!fs.existsSync(roadmapPath)) {
-    return { success: false, error: 'ROADMAP.md not found' };
-  }
-
-  let oldPlans = null;
-  const newPlans = `${complete}/${total}`;
-
-  const result = lockedFileUpdate(roadmapPath, (content) => {
-    const lines = content.split('\n');
-    const rowIdx = findRoadmapRow(lines, phaseNum);
-    if (rowIdx === -1) {
-      return content;
-    }
-    const parts = lines[rowIdx].split('|');
-    oldPlans = parts[4] ? parts[4].trim() : 'unknown';
-    lines[rowIdx] = updateTableRow(lines[rowIdx], 3, newPlans);
-    return lines.join('\n');
-  });
-
-  if (!oldPlans) {
-    return { success: false, error: `Phase ${phaseNum} not found in ROADMAP.md table` };
-  }
-
-  if (result.success) {
-    return { success: true, old_plans: oldPlans, new_plans: newPlans };
-  }
-  return { success: false, error: result.error };
-}
-
-// --- Mutation helpers ---
-
-/**
- * Update a field in legacy (non-frontmatter) STATE.md content.
- * Pure function: content in, content out.
- */
-function updateLegacyStateField(content, field, value) {
-  const lines = content.split('\n');
-
-  switch (field) {
-    case 'current_phase': {
-      const idx = lines.findIndex(l => /Phase:\s*\d+\s+of\s+\d+/.test(l));
-      if (idx !== -1) {
-        lines[idx] = lines[idx].replace(/(Phase:\s*)\d+/, (_, prefix) => `${prefix}${value}`);
-      }
-      break;
-    }
-    case 'status': {
-      const idx = lines.findIndex(l => /^Status:/i.test(l));
-      if (idx !== -1) {
-        lines[idx] = `Status: ${value}`;
-      } else {
-        const phaseIdx = lines.findIndex(l => /Phase:/.test(l));
-        if (phaseIdx !== -1) {
-          lines.splice(phaseIdx + 1, 0, `Status: ${value}`);
-        } else {
-          lines.push(`Status: ${value}`);
-        }
-      }
-      break;
-    }
-    case 'plans_complete': {
-      const idx = lines.findIndex(l => /Plan:\s*\d+\s+of\s+\d+/.test(l));
-      if (idx !== -1) {
-        lines[idx] = lines[idx].replace(/(Plan:\s*)\d+/, (_, prefix) => `${prefix}${value}`);
-      }
-      break;
-    }
-    case 'last_activity': {
-      const idx = lines.findIndex(l => /^Last Activity:/i.test(l));
-      if (idx !== -1) {
-        lines[idx] = `Last Activity: ${value}`;
-      } else {
-        const statusIdx = lines.findIndex(l => /^Status:/i.test(l));
-        if (statusIdx !== -1) {
-          lines.splice(statusIdx + 1, 0, `Last Activity: ${value}`);
-        } else {
-          lines.push(`Last Activity: ${value}`);
-        }
-      }
-      break;
-    }
-  }
-
-  return lines.join('\n');
-}
-
-/**
- * Update a field in YAML frontmatter content.
- * Pure function: content in, content out.
- */
-function updateFrontmatterField(content, field, value) {
-  const match = content.match(/^(---\s*\n)([\s\S]*?)(\n---)/);
-  if (!match) return content;
-
-  const before = match[1];
-  let yaml = match[2];
-  const after = match[3];
-  const rest = content.slice(match[0].length);
-
-  // Format value: integers stay bare, strings get quotes
-  const isNum = /^\d+$/.test(String(value));
-  const formatted = isNum ? value : `"${value}"`;
-
-  const fieldRegex = new RegExp(`^(${field})\\s*:.*$`, 'm');
-  if (fieldRegex.test(yaml)) {
-    yaml = yaml.replace(fieldRegex, () => `${field}: ${formatted}`);
-  } else {
-    yaml = yaml + `\n${field}: ${formatted}`;
-  }
-
-  return before + yaml + after + rest;
-}
-
-/**
- * Find the row index of a phase in a ROADMAP.md table.
- * @returns {number} Line index or -1 if not found
- */
-function findRoadmapRow(lines, phaseNum) {
-  const paddedPhase = phaseNum.padStart(2, '0');
-  for (let i = 0; i < lines.length; i++) {
-    if (!lines[i].includes('|')) continue;
-    const parts = lines[i].split('|');
-    if (parts.length < 3) continue;
-    const phaseCol = parts[1] ? parts[1].trim() : '';
-    if (phaseCol === paddedPhase) {
-      return i;
-    }
-  }
-  return -1;
-}
-
-/**
- * Update a specific column in a markdown table row.
- * @param {string} row - The full table row string (e.g., "| 01 | Setup | ... |")
- * @param {number} columnIndex - 0-based column index (Phase=0, Name=1, ..., Status=5)
- * @param {string} newValue - New cell value
- * @returns {string} Updated row
- */
-function updateTableRow(row, columnIndex, newValue) {
-  const parts = row.split('|');
-  // parts[0] is empty (before first |), data starts at parts[1]
-  const partIndex = columnIndex + 1;
-  if (partIndex < parts.length) {
-    parts[partIndex] = ` ${newValue} `;
-  }
-  return parts.join('|');
-}
-
-/**
- * Lightweight JSON Schema validator — supports type, enum, properties,
- * additionalProperties, minimum, maximum for the config schema.
- */
-function validateObject(value, schema, prefix, errors, warnings) {
-  if (schema.type && typeof value !== schema.type) {
-    if (!(schema.type === 'integer' && typeof value === 'number' && Number.isInteger(value))) {
-      errors.push(`${prefix || 'root'}: expected ${schema.type}, got ${typeof value}`);
-      return;
-    }
-  }
-
-  if (schema.enum && !schema.enum.includes(value)) {
-    errors.push(`${prefix || 'root'}: value "${value}" not in allowed values [${schema.enum.join(', ')}]`);
-    return;
-  }
-
-  if (schema.minimum !== undefined && value < schema.minimum) {
-    errors.push(`${prefix || 'root'}: value ${value} is below minimum ${schema.minimum}`);
-  }
-  if (schema.maximum !== undefined && value > schema.maximum) {
-    errors.push(`${prefix || 'root'}: value ${value} is above maximum ${schema.maximum}`);
-  }
-
-  if (schema.type === 'object' && schema.properties) {
-    const knownKeys = new Set(Object.keys(schema.properties));
-
-    for (const key of Object.keys(value)) {
-      const fullKey = prefix ? `${prefix}.${key}` : key;
-      if (!knownKeys.has(key)) {
-        if (schema.additionalProperties === false) {
-          warnings.push(`${fullKey}: unrecognized key (possible typo?)`);
-        }
-        continue;
-      }
-      validateObject(value[key], schema.properties[key], fullKey, errors, warnings);
-    }
-  }
-}
-
-/**
- * Locked file update: read-modify-write with exclusive lockfile.
- * Prevents concurrent writes to STATE.md and ROADMAP.md.
- *
- * @param {string} filePath - Absolute path to the file to update
- * @param {function} updateFn - Receives current content, returns new content
- * @param {object} opts - Options: { retries: 3, retryDelayMs: 100, timeoutMs: 5000 }
- * @returns {object} { success, content?, error? }
- */
-function lockedFileUpdate(filePath, updateFn, opts = {}) {
-  const retries = opts.retries || 3;
-  const retryDelayMs = opts.retryDelayMs || 100;
-  const timeoutMs = opts.timeoutMs || 5000;
-  const lockPath = filePath + '.lock';
-
-  let lockFd = null;
-  let lockAcquired = false;
-
-  try {
-    // Acquire lock with retries
-    for (let attempt = 0; attempt < retries; attempt++) {
-      try {
-        lockFd = fs.openSync(lockPath, 'wx');
-        lockAcquired = true;
-        break;
-      } catch (e) {
-        if (e.code === 'EEXIST') {
-          // Lock exists — check if stale (older than timeoutMs)
-          try {
-            const stats = fs.statSync(lockPath);
-            if (Date.now() - stats.mtimeMs > timeoutMs) {
-              // Stale lock — remove and retry
-              fs.unlinkSync(lockPath);
-              continue;
-            }
-          } catch (_statErr) {
-            // Lock disappeared between check — retry
-            continue;
-          }
-
-          if (attempt < retries - 1) {
-            // Wait and retry
-            const waitMs = retryDelayMs * (attempt + 1);
-            const start = Date.now();
-            while (Date.now() - start < waitMs) {
-              // Busy wait (synchronous context)
-            }
-            continue;
-          }
-          return { success: false, error: `Could not acquire lock for ${path.basename(filePath)} after ${retries} attempts` };
-        }
-        throw e;
-      }
-    }
-
-    if (!lockAcquired) {
-      return { success: false, error: `Could not acquire lock for ${path.basename(filePath)}` };
-    }
-
-    // Write PID to lock file for debugging
-    fs.writeSync(lockFd, `${process.pid}`);
-    fs.closeSync(lockFd);
-    lockFd = null;
-
-    // Read current content
-    let content = '';
-    if (fs.existsSync(filePath)) {
-      content = fs.readFileSync(filePath, 'utf8');
-    }
-
-    // Apply update
-    const newContent = updateFn(content);
-
-    // Write back atomically
-    const writeResult = atomicWrite(filePath, newContent);
-    if (!writeResult.success) {
-      return { success: false, error: writeResult.error };
-    }
-
-    return { success: true, content: newContent };
-  } catch (e) {
-    return { success: false, error: e.message };
-  } finally {
-    // Close fd if still open
-    try {
-      if (lockFd !== null) fs.closeSync(lockFd);
-    } catch (_e) { /* ignore */ }
-    // Only release lock if we acquired it
-    if (lockAcquired) {
-      try {
-        fs.unlinkSync(lockPath);
-      } catch (_e) { /* ignore — may already be cleaned up */ }
-    }
-  }
-}
-
-// --- Parsers ---
-
-function parseStateMd(content) {
-  const result = {
-    current_phase: null,
-    phase_name: null,
-    progress: null,
-    status: null,
-    line_count: content.split('\n').length,
-    format: 'legacy' // 'legacy' or 'frontmatter'
-  };
-
-  // Check for YAML frontmatter (version 2 format)
-  const frontmatter = parseYamlFrontmatter(content);
-  if (frontmatter.version === 2 || frontmatter.current_phase !== undefined) {
-    result.format = 'frontmatter';
-    result.current_phase = frontmatter.current_phase || null;
-    result.total_phases = frontmatter.total_phases || null;
-    result.phase_name = frontmatter.phase_slug || frontmatter.phase_name || null;
-    result.status = frontmatter.status || null;
-    result.progress = frontmatter.progress_percent !== undefined ? frontmatter.progress_percent : null;
-    result.plans_total = frontmatter.plans_total || null;
-    result.plans_complete = frontmatter.plans_complete || null;
-    result.last_activity = frontmatter.last_activity || null;
-    result.last_command = frontmatter.last_command || null;
-    result.blockers = frontmatter.blockers || [];
-    return result;
-  }
-
-  // Legacy regex-based parsing (version 1 format, no frontmatter)
-  // DEPRECATED (2026-02): v1 STATE.md format (no YAML frontmatter) is deprecated.
-  // New projects should use v2 (frontmatter) format, generated by /pbr:setup.
-  // v1 support will be removed in a future major version.
-  process.stderr.write('[pbr] WARNING: STATE.md uses legacy v1 format. Run /pbr:setup to migrate to v2 format.\n');
-  // Extract "Phase: N of M"
-  const phaseMatch = content.match(/Phase:\s*(\d+)\s+of\s+(\d+)/);
-  if (phaseMatch) {
-    result.current_phase = parseInt(phaseMatch[1], 10);
-    result.total_phases = parseInt(phaseMatch[2], 10);
-  }
-
-  // Extract phase name (line after "Phase:")
-  const nameMatch = content.match(/--\s+(.+?)(?:\n|$)/);
-  if (nameMatch) {
-    result.phase_name = nameMatch[1].trim();
-  }
-
-  // Extract progress percentage
-  const progressMatch = content.match(/(\d+)%/);
-  if (progressMatch) {
-    result.progress = parseInt(progressMatch[1], 10);
-  }
-
-  // Extract plan status
-  const statusMatch = content.match(/Status:\s*(.+?)(?:\n|$)/i);
-  if (statusMatch) {
-    result.status = statusMatch[1].trim();
-  }
-
-  return result;
-}
-
-function parseRoadmapMd(content) {
-  const result = { phases: [], has_progress_table: false };
-
-  // Find Phase Overview table
-  const overviewMatch = content.match(/## Phase Overview[\s\S]*?\|[\s\S]*?(?=\n##|\s*$)/);
-  if (overviewMatch) {
-    const rows = overviewMatch[0].split('\n').filter(r => r.includes('|'));
-    // Skip header and separator rows
-    for (let i = 2; i < rows.length; i++) {
-      const cols = rows[i].split('|').map(c => c.trim()).filter(Boolean);
-      if (cols.length >= 3) {
-        result.phases.push({
-          number: cols[0],
-          name: cols[1],
-          goal: cols[2],
-          plans: cols[3] || '',
-          wave: cols[4] || '',
-          status: cols[5] || 'pending'
-        });
-      }
-    }
-  }
-
-  // Check for Progress table
-  result.has_progress_table = /## Progress/.test(content);
-
-  return result;
-}
-
-function parseYamlFrontmatter(content) {
-  const match = content.match(/^---\s*\n([\s\S]*?)\n---/);
-  if (!match) return {};
-
-  const yaml = match[1];
-  const result = {};
-
-  // Simple YAML parser for flat and basic nested values
-  const lines = yaml.split('\n');
-  let currentKey = null;
-
-  for (const line of lines) {
-    // Array item
-    if (/^\s+-\s+/.test(line) && currentKey) {
-      const val = line.replace(/^\s+-\s+/, '').trim().replace(/^["']|["']$/g, '');
-      if (!result[currentKey]) result[currentKey] = [];
-      if (Array.isArray(result[currentKey])) {
-        result[currentKey].push(val);
-      }
-      continue;
-    }
-
-    // Key-value pair
-    const kvMatch = line.match(/^(\w[\w_]*)\s*:\s*(.*)/);
-    if (kvMatch) {
-      currentKey = kvMatch[1];
-      let val = kvMatch[2].trim();
-
-      if (val === '' || val === '|') {
-        // Possible array or block follows
-        continue;
-      }
-
-      // Handle arrays on same line: [a, b, c]
-      if (val.startsWith('[') && val.endsWith(']')) {
-        result[currentKey] = val.slice(1, -1).split(',')
-          .map(v => v.trim().replace(/^["']|["']$/g, ''))
-          .filter(Boolean);
-        continue;
-      }
-
-      // Clean quotes
-      val = val.replace(/^["']|["']$/g, '');
-
-      // Type coercion
-      if (val === 'true') val = true;
-      else if (val === 'false') val = false;
-      else if (/^\d+$/.test(val)) val = parseInt(val, 10);
-
-      result[currentKey] = val;
-    }
-  }
-
-  // Handle must_haves as a nested object
-  if (yaml.includes('must_haves:')) {
-    result.must_haves = parseMustHaves(yaml);
-  }
-
-  return result;
-}
-
-function parseMustHaves(yaml) {
-  const result = { truths: [], artifacts: [], key_links: [] };
-  let section = null;
-
-  const inMustHaves = yaml.split('\n');
-  let collecting = false;
-
-  for (const line of inMustHaves) {
-    if (/^\s*must_haves:/.test(line)) {
-      collecting = true;
-      continue;
-    }
-    if (collecting) {
-      if (/^\s{2}truths:/.test(line)) { section = 'truths'; continue; }
-      if (/^\s{2}artifacts:/.test(line)) { section = 'artifacts'; continue; }
-      if (/^\s{2}key_links:/.test(line)) { section = 'key_links'; continue; }
-      if (/^\w/.test(line)) break; // New top-level key, stop
-
-      if (section && /^\s+-\s+/.test(line)) {
-        result[section].push(line.replace(/^\s+-\s+/, '').trim().replace(/^["']|["']$/g, ''));
-      }
-    }
-  }
-
-  return result;
-}
-
-// --- Helpers ---
-
-function findFiles(dir, pattern) {
-  try {
-    return fs.readdirSync(dir).filter(f => pattern.test(f)).sort();
-  } catch (_) {
-    return [];
-  }
-}
-
-function determinePhaseStatus(planCount, completedCount, summaryCount, hasVerification, phaseDir) {
-  if (planCount === 0) {
-    // Check for CONTEXT.md (discussed only)
-    if (fs.existsSync(path.join(phaseDir, 'CONTEXT.md'))) return 'discussed';
-    return 'not_started';
-  }
-  if (completedCount === 0 && summaryCount === 0) return 'planned';
-  if (completedCount < planCount) return 'building';
-  if (!hasVerification) return 'built';
-  // Check verification status
-  try {
-    const vContent = fs.readFileSync(path.join(phaseDir, 'VERIFICATION.md'), 'utf8');
-    if (/status:\s*["']?passed/i.test(vContent)) return 'verified';
-    if (/status:\s*["']?gaps_found/i.test(vContent)) return 'needs_fixes';
-    return 'reviewed';
-  } catch (_) {
-    return 'built';
-  }
-}
-
-function countMustHaves(mustHaves) {
-  if (!mustHaves) return 0;
-  return (mustHaves.truths || []).length +
-    (mustHaves.artifacts || []).length +
-    (mustHaves.key_links || []).length;
-}
-
-function calculateProgress() {
-  const phasesDir = path.join(planningDir, 'phases');
-  if (!fs.existsSync(phasesDir)) {
-    return { total: 0, completed: 0, percentage: 0 };
-  }
-
-  let total = 0;
-  let completed = 0;
-
-  const entries = fs.readdirSync(phasesDir, { withFileTypes: true })
-    .filter(e => e.isDirectory());
-
-  for (const entry of entries) {
-    const dir = path.join(phasesDir, entry.name);
-    const plans = findFiles(dir, /-PLAN\.md$/);
-    total += plans.length;
-
-    const summaries = findFiles(dir, /^SUMMARY-.*\.md$/);
-    for (const s of summaries) {
-      const content = fs.readFileSync(path.join(dir, s), 'utf8');
-      if (/status:\s*["']?complete/i.test(content)) completed++;
-    }
-  }
-
-  return {
-    total,
-    completed,
-    percentage: total > 0 ? Math.round((completed / total) * 100) : 0
-  };
-}
-
-function output(data) {
-  process.stdout.write(JSON.stringify(data, null, 2));
-  process.exit(0);
-}
-
-function error(msg) {
-  process.stdout.write(JSON.stringify({ error: msg }));
-  process.exit(1);
-}
-
-/**
- * Write content to a file atomically: write to .tmp, backup original to .bak,
- * rename .tmp over original. On failure, restore from .bak if available.
- *
- * @param {string} filePath - Target file path
- * @param {string} content - Content to write
- * @returns {{success: boolean, error?: string}} Result
- */
-function atomicWrite(filePath, content) {
-  const tmpPath = filePath + '.tmp';
-  const bakPath = filePath + '.bak';
-
-  try {
-    // 1. Write to temp file
-    fs.writeFileSync(tmpPath, content, 'utf8');
-
-    // 2. Backup original if it exists
-    if (fs.existsSync(filePath)) {
-      try {
-        fs.copyFileSync(filePath, bakPath);
-      } catch (_e) {
-        // Backup failure is non-fatal — proceed with rename
-      }
-    }
-
-    // 3. Rename temp over original (atomic on most filesystems)
-    fs.renameSync(tmpPath, filePath);
-
-    return { success: true };
-  } catch (e) {
-    // Rename failed — try to restore from backup
-    try {
-      if (fs.existsSync(bakPath)) {
-        fs.copyFileSync(bakPath, filePath);
-      }
-    } catch (_restoreErr) {
-      // Restore also failed — nothing more we can do
-    }
-
-    // Clean up temp file if it still exists
-    try {
-      if (fs.existsSync(tmpPath)) {
-        fs.unlinkSync(tmpPath);
-      }
-    } catch (_cleanupErr) {
-      // Best-effort cleanup
-    }
-
-    return { success: false, error: e.message };
-  }
-}
-
 if (require.main === module || process.argv[1] === __filename) { main().catch(err => { process.stderr.write(err.message + '\n'); process.exit(1); }); }
-module.exports = { parseStateMd, parseRoadmapMd, parseYamlFrontmatter, parseMustHaves, countMustHaves, stateLoad, stateCheckProgress, configLoad, configClearCache, configValidate, lockedFileUpdate, planIndex, determinePhaseStatus, findFiles, atomicWrite, tailLines, frontmatter, mustHavesCollect, phaseInfo, stateUpdate, roadmapUpdateStatus, roadmapUpdatePlans, updateLegacyStateField, updateFrontmatterField, updateTableRow, findRoadmapRow, resolveDepthProfile, DEPTH_PROFILE_DEFAULTS, historyAppend, historyLoad, VALID_STATUS_TRANSITIONS, validateStatusTransition };
+module.exports = { KNOWN_AGENTS, initExecutePhase, initPlanPhase, initQuick, initVerifyWork, initResume, initProgress, statePatch, stateAdvancePlan, stateRecordMetric, parseStateMd, parseRoadmapMd, parseYamlFrontmatter, parseMustHaves, countMustHaves, stateLoad, stateCheckProgress, configLoad, configClearCache, configValidate, lockedFileUpdate, planIndex, determinePhaseStatus, findFiles, atomicWrite, tailLines, frontmatter, mustHavesCollect, phaseInfo, stateUpdate, roadmapUpdateStatus, roadmapUpdatePlans, updateLegacyStateField, updateFrontmatterField, updateTableRow, findRoadmapRow, resolveDepthProfile, DEPTH_PROFILE_DEFAULTS, historyAppend, historyLoad, VALID_STATUS_TRANSITIONS, validateStatusTransition, writeActiveSkill, validateProject, phaseAdd, phaseRemove, phaseList };
+// NOTE: validateProject, phaseAdd, phaseRemove, phaseList were previously CLI-only (not exported).
+// They are now exported for testability. This is additive and backwards-compatible.