@sienklogic/plan-build-run 2.19.1 → 2.19.2

package/plugins/pbr/dist/suggest-compact.js

@@ -9,7 +9,7 @@
  * fresh (<60s old). CRITICAL tier always emits; others use REMINDER_INTERVAL debounce.
  *
  * Fallback: when bridge is absent or stale, uses call-count threshold.
- * Counter stored in .planning/.compact-counter (JSON).
+ * Counter stored in .planning/.compact-counter (JSON) and .session.json.
  * Threshold configurable via config.json hooks.compactThreshold (default: 50).
  * Counter resets on SessionStart (via progress-tracker.js).
  *
@@ -281,15 +281,7 @@ function resetCounter(planningDir, sessionId) {
     sessionSave(planningDir, { compactCounter: { count: 0, lastSuggested: 0 } }, sessionId);
   } catch (_e) { /* best-effort */ }
 
-  // Legacy: also delete .compact-counter file if present
-  const counterPath = path.join(planningDir, '.compact-counter');
-  try {
-    if (fs.existsSync(counterPath)) {
-      fs.unlinkSync(counterPath);
-    }
-  } catch (_e) {
-    // Best-effort
-  }
+  // .compact-counter cleanup removed -- legacy file no longer used
 }
 
 /**

package/plugins/pbr/dist/validate-commit.js

@@ -26,8 +26,6 @@ const path = require('path');
 const { execSync } = require('child_process');
 const { logHook } = require('./hook-logger');
 const { logEvent } = require('./event-logger');
-const { resolveConfig } = require('./lib/local-llm/health');
-const { classifyCommit } = require('./lib/local-llm/operations/classify-commit');
 
 const VALID_TYPES = ['feat', 'fix', 'refactor', 'test', 'docs', 'chore', 'wip', 'revert', 'perf', 'ci', 'build'];
 
@@ -75,7 +73,13 @@ function checkAiCoAuthorResult(command, sessionId) {
 
 function checkSensitiveFilesResult(sessionId) {
   try {
-    const output = execSync('git diff --cached --name-only', { encoding: 'utf8' });
+    let output;
+    try {
+      output = execSync('git diff --cached --name-only', { encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
+    } catch (_e) {
+      // git diff --cached fails during git tag or in --no-index contexts
+      return null;
+    }
     const files = output.trim().split('\n').filter(Boolean);
 
     const matched = files.filter((file) => {
@@ -157,58 +161,6 @@ function checkCommit(data) {
   return null;
 }
 
-/**
- * Load and resolve the local_llm config block from .planning/config.json.
- * Returns a resolved config (always safe to use — disabled by default on error).
- */
-function loadLocalLlmConfig(cwd) {
-  try {
-    const configPath = path.join(cwd || process.cwd(), '.planning', 'config.json');
-    const parsed = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-    return resolveConfig(parsed.local_llm);
-  } catch (_e) {
-    return resolveConfig(undefined);
-  }
-}
-
-/**
- * Async LLM enrichment for commit messages. Returns an advisory string or null.
- * Called after checkCommit passes (valid format) to provide semantic classification.
- *
- * @param {object} data - parsed hook data
- * @returns {Promise<string|null>}
- */
-async function enrichCommitLlm(data) {
-  try {
-    const command = data.tool_input?.command || '';
-    const message = extractCommitMessage(command);
-    if (!message) return null;
-
-    const cwd = process.cwd();
-    const llmConfig = loadLocalLlmConfig(cwd);
-    const planningDir = path.join(cwd, '.planning');
-
-    // Get staged files for scope validation
-    let stagedFiles = [];
-    try {
-      const output = execSync('git diff --cached --name-only', { encoding: 'utf8' });
-      stagedFiles = output.trim().split('\n').filter(Boolean);
-    } catch (_e) {
-      // Not in a git repo — skip staged files context
-    }
-
-    const llmResult = await classifyCommit(llmConfig, planningDir, message, stagedFiles, data.session_id);
-    if (llmResult && llmResult.classification !== 'correct') {
-      return 'LLM commit advisory: ' + llmResult.classification +
-        ' (confidence: ' + (llmResult.confidence * 100).toFixed(0) + '%)';
-    }
-    return null;
-  } catch (_llmErr) {
-    // Never propagate LLM errors
-    return null;
-  }
-}
-
 function main() {
   let input = '';
 
@@ -223,14 +175,6 @@ function main() {
         process.exit(result.exitCode);
       }
 
-      // LLM semantic classification — advisory only (after format validation passes)
-      const llmAdvisory = await enrichCommitLlm(data);
-      if (llmAdvisory) {
-        process.stdout.write(JSON.stringify({
-          additionalContext: '[pbr] ' + llmAdvisory
-        }));
-      }
-
       process.exit(0);
     } catch (_e) {
       // Parse error - don't block
@@ -267,5 +211,5 @@ function extractCommitMessage(command) {
   return null;
 }
 
-module.exports = { checkCommit, enrichCommitLlm };
+module.exports = { checkCommit };
 if (require.main === module || process.argv[1] === __filename) { main(); }

package/plugins/pbr/dist/validate-task.js

@@ -20,8 +20,6 @@
 const fs = require('fs');
 const path = require('path');
 const { logHook } = require('./hook-logger');
-const { resolveConfig } = require('./lib/local-llm/health');
-const { validateTask: llmValidateTask } = require('./lib/local-llm/operations/validate-task');
 const { checkNonPbrAgent } = require('./enforce-pbr-workflow');
 const { KNOWN_AGENTS } = require('./lib/core');
 
@@ -39,21 +37,6 @@ const { checkDebuggerAdvisory, checkCheckpointManifest, checkActiveSkillIntegrit
 const { checkDocExistence } = require('./lib/gates/doc-existence');
 const { checkUserConfirmationGate } = require('./lib/gates/user-confirmation');
 
-/**
- * Load and resolve the local_llm config block from .planning/config.json.
- * Returns a resolved config (always safe to use — disabled by default on error).
- * @param {string} cwd - working directory to resolve .planning/config.json from
- */
-function loadLocalLlmConfig(cwd) {
-  try {
-    const configPath = path.join(cwd, '.planning', 'config.json');
-    const parsed = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-    return resolveConfig(parsed.local_llm);
-  } catch (_e) {
-    return resolveConfig(undefined);
-  }
-}
-
 const MAX_DESCRIPTION_LENGTH = 100;
 
 /**
@@ -264,19 +247,6 @@ function main() {
       if (nonPbrAgentResult) warnings.push(nonPbrAgentResult.output.additionalContext);
       if (planValGate && planValGate.warning) warnings.push(planValGate.warning);
 
-      // LLM task coherence check — advisory only
-      try {
-        const llmCwd = process.env.PBR_PROJECT_ROOT || process.cwd();
-        const llmConfig = loadLocalLlmConfig(llmCwd);
-        const planningDir = path.join(llmCwd, '.planning');
-        const llmResult = await llmValidateTask(llmConfig, planningDir, data.tool_input || {}, data.session_id);
-        if (llmResult && !llmResult.coherent) {
-          warnings.push('LLM task coherence advisory: ' + (llmResult.issue || 'Task description may not match intended operation.') + ' (confidence: ' + (llmResult.confidence * 100).toFixed(0) + '%)');
-        }
-      } catch (_llmErr) {
-        // Never propagate LLM errors
-      }
-
       if (warnings.length > 0) {
         for (const warning of warnings) {
           logHook('validate-task', 'PreToolUse', 'warn', { warning });

package/plugins/pbr/references/config-reference.md

@@ -118,7 +118,6 @@ Boolean toggles that enable or disable specific workflow capabilities. All defau
 | `auto_advance` | `false` | Chain build, review, and plan automatically (requires `mode: autonomous`) |
 | `team_discussions` | `false` | Enable team-based discussion workflows (never used for execution) |
 | `inline_verify` | `false` | Per-task verification after each executor commit; adds ~10-20s latency per plan |
-| `multi_layer_validation` | `false` | **(DEPRECATED)** Parallel BugBot-style review passes. Never integrated; removed in plan 100-01. Key retained for backward compatibility. |
 | `extended_context` | `false` | Enable aggressive 1M context optimizations: higher concurrency (5 agents vs 3), default team review, always-parallel scan, pre-load build steps. Auto-set by quality profile. Safe optimizations (parallel research, full SUMMARY reads) use `context_window_tokens >= 500000` instead. |
 
 **Notable interactions:**
@@ -468,14 +467,6 @@ When `confidence_gate` is `true` and the executor SUMMARY reports all must-haves
 
 ---
 
-## validation_passes (DEPRECATED)
-
-> **DEPRECATED**: The `multi_layer_validation` feature was never integrated and was removed in plan 100-01. This config key is retained for backward compatibility. The array has no effect.
-
-Defined which review passes would run with the multi-layer validation feature. Options included: `correctness`, `security`, `performance`, `style`, `tests`, `accessibility`, `docs`, `deps`.
-
----
-
 ## context_budget
 
 Controls context budget warning thresholds.
@@ -814,90 +805,3 @@ Run validation with: `node ${CLAUDE_PLUGIN_ROOT}/scripts/pbr-tools.js config val
 | `tdd_mode: true` + `depth: quick` | quick depth skips verification, which conflicts with TDD's verify-first approach |
 | `git.mode: disabled` + `atomic_commits: true` | atomic_commits has no effect when git is disabled |
 | `git.branching: phase` + `git.mode: disabled` | Branching settings are ignored when git is disabled |
-
----
-
-## local_llm (DEPRECATED)
-
-> **DEPRECATED**: This feature has been removed as of v14.0. The config key is retained for backward compatibility. Setting `enabled: true` has no effect.
-
-Offloads selected PBR inference tasks to a locally running Ollama instance, reducing frontier model usage and latency for fast classification calls. The key `enabled` defaults to `false`, so users without Ollama see no change — all LLM calls continue routing to Claude as normal. When enabled, PBR uses a `local_first` routing strategy: fast tasks (artifact classification, task validation) go to the local model; complex tasks (planning, execution) stay on the frontier model.
-
-### Quick setup
-
-1. Install Ollama:
-   - **Linux/macOS**: `curl -fsSL https://ollama.com/install.sh | sh`
-   - **Windows**: Download from [ollama.com/download](https://ollama.com/download) and run the installer
-2. Pull the recommended model: `ollama pull qwen2.5-coder:7b`
-3. Add to `.planning/config.json`:
-
-   ```json
-   "local_llm": {
-     "enabled": true,
-     "model": "qwen2.5-coder:7b"
-   }
-   ```
-
-4. Verify connectivity: `node ${CLAUDE_PLUGIN_ROOT}/scripts/pbr-tools.js llm health` (DEPRECATED -- returns `{ deprecated: true }` as a no-op)
-
-### Field reference
-
-| Property | Type | Default | Description |
-|----------|------|---------|-------------|
-| `local_llm.enabled` | boolean | `false` | Enable local LLM offloading; `false` = all calls use frontier |
-| `local_llm.provider` | string | `"ollama"` | Backend provider; only `"ollama"` is supported |
-| `local_llm.endpoint` | string | `"http://localhost:11434"` | Ollama API base URL |
-| `local_llm.model` | string | `"qwen2.5-coder:7b"` | Model tag to use for local inference |
-| `local_llm.timeout_ms` | integer | `3000` | Per-request timeout in milliseconds; >= 500 |
-| `local_llm.max_retries` | integer | `1` | Number of retry attempts on failure before falling back |
-| `local_llm.fallback` | string | `"frontier"` | What to use when local LLM fails: `"frontier"` or `"skip"` |
-| `local_llm.routing_strategy` | string | `"local_first"` | `"local_first"` sends fast tasks local; `"always_local"` routes everything |
-
-### features sub-table
-
-Controls which PBR tasks are eligible for local LLM offloading.
-
-| Property | Default | Description |
-|----------|---------|-------------|
-| `artifact_classification` | `true` | Classify artifact types (PLAN, SUMMARY, VERIFICATION) locally |
-| `task_validation` | `true` | Validate task scope and completeness locally |
-| `context_summarization` | `false` | Summarize context windows locally (higher token demand) |
-| `source_scoring` | `false` | Score source files by relevance locally |
-
-### advanced sub-table
-
-| Property | Default | Description |
-|----------|---------|-------------|
-| `confidence_threshold` | `0.9` | Minimum confidence (0–1) for local output to be accepted; below this, falls back to frontier |
-| `shadow_mode` | `false` | Run local LLM in parallel with frontier but discard local results — useful for tuning confidence thresholds without affecting output |
-| `max_input_tokens` | `2000` | Truncate inputs longer than this before sending to local model |
-| `keep_alive` | `"30m"` | How long Ollama keeps the model loaded between requests (Ollama format: `"5m"`, `"1h"`) |
-| `num_ctx` | `4096` | Context window size passed to Ollama; **must be 4096 on Windows** (see Windows gotchas) |
-| `disable_after_failures` | `3` | Automatically disable local LLM for the session after this many consecutive failures |
-
-### Hardware requirements
-
-| Tier | Hardware | Notes |
-|------|----------|-------|
-| Recommended | RTX 3060+ with 8 GB VRAM | Full GPU acceleration; qwen2.5-coder:7b loads entirely in VRAM |
-| Functional | GTX 1660+ with 6 GB VRAM | GPU acceleration with slight layer offload to RAM |
-| Marginal | CPU only, 32 GB RAM | Works but adds 5-20s latency per call; disable context-heavy features |
-
-For GPU acceleration, ensure NVIDIA drivers are 520+ and CUDA 11.8+ is installed. AMD GPU support is available via ROCm on Linux only.
-
-### Windows gotchas
-
-- **Smart App Control**: May block `ollama_llama_server.exe` on first run. Allow it via Security settings or disable Smart App Control.
-- **Windows Defender**: Add an exclusion for `%LOCALAPPDATA%\Programs\Ollama\ollama_llama_server.exe` to prevent Defender from scanning inference calls in real time.
-- **`num_ctx` must be 4096**: Higher values cause GPU memory fragmentation on Windows and result in OOM errors mid-session. Always set `advanced.num_ctx: 4096` in your config.
-- **Firewall**: Ollama listens on `localhost:11434` by default. If you see connection refused errors, check that Windows Firewall is not blocking loopback connections.
-
-### Viewing metrics (DEPRECATED)
-
-The local LLM feature was removed in v14.0. The `llm metrics` and `llm adjust-thresholds` subcommands are no-ops that return `{ deprecated: true }`. No metrics are collected.
-
-```bash
-# These commands return { deprecated: true } and perform no action:
-node ${CLAUDE_PLUGIN_ROOT}/scripts/pbr-tools.js llm metrics
-node ${CLAUDE_PLUGIN_ROOT}/scripts/pbr-tools.js llm adjust-thresholds
-```

package/plugins/pbr/scripts/audit-checks/si-agent-hook-config-checks.js

@@ -9,82 +9,12 @@
 
 const fs = require('fs');
 const path = require('path');
+const { extractFrontmatter } = require('../lib/frontmatter');
 
 // ---------------------------------------------------------------------------
 // Helpers
 // ---------------------------------------------------------------------------
 
-/**
- * Parse YAML frontmatter from a markdown file.
- * Returns an object with extracted key-value pairs (simple flat parsing).
- */
-function parseFrontmatter(content) {
-  const lines = content.replace(/\r\n/g, '\n').split('\n');
-  if (lines[0] !== '---') return {};
-
-  const endIdx = lines.indexOf('---', 1);
-  if (endIdx === -1) return {};
-
-  const fmLines = lines.slice(1, endIdx);
-  const result = {};
-  let currentKey = null;
-  let inArray = false;
-
-  for (const line of fmLines) {
-    // Array item
-    if (inArray && /^\s+-\s+/.test(line)) {
-      const val = line.replace(/^\s+-\s+/, '').trim();
-      result[currentKey].push(val);
-      continue;
-    }
-
-    // Key-value pair
-    const kvMatch = line.match(/^(\w[\w-]*)\s*:\s*(.*)/);
-    if (kvMatch) {
-      const key = kvMatch[1];
-      const rawVal = kvMatch[2].trim();
-
-      if (rawVal === '' || rawVal === '[]') {
-        // Could be start of array or empty value
-        result[key] = [];
-        currentKey = key;
-        inArray = true;
-        continue;
-      }
-
-      // Inline array like [Read, Write, Edit]
-      if (rawVal.startsWith('[') && rawVal.endsWith(']')) {
-        result[key] = rawVal.slice(1, -1).split(',').map(s => s.trim().replace(/^["']|["']$/g, ''));
-        currentKey = key;
-        inArray = false;
-        continue;
-      }
-
-      // Scalar value
-      result[key] = rawVal.replace(/^["']|["']$/g, '');
-      currentKey = key;
-      inArray = false;
-      continue;
-    }
-
-    // Array item under current key (indented with -)
-    if (currentKey && /^\s+-\s/.test(line)) {
-      if (!Array.isArray(result[currentKey])) {
-        result[currentKey] = [];
-      }
-      const val = line.replace(/^\s+-\s+/, '').trim().replace(/^["']|["']$/g, '');
-      result[currentKey].push(val);
-      inArray = true;
-      continue;
-    }
-
-    // Not a recognized line — end array mode
-    inArray = false;
-  }
-
-  return result;
-}
-
 /**
  * Read all agent .md files and return array of { file, frontmatter, content }.
  */
@@ -97,7 +27,7 @@ function readAgentFiles(pluginRoot) {
     .map(f => {
       const filePath = path.join(agentsDir, f);
       const content = fs.readFileSync(filePath, 'utf8');
-      return { file: f, frontmatter: parseFrontmatter(content), content };
+      return { file: f, frontmatter: extractFrontmatter(content), content };
     });
 }
 

package/plugins/pbr/scripts/audit-checks/workflow-compliance.js

@@ -25,6 +25,7 @@
 const fs = require('fs');
 const path = require('path');
 const { execSync } = require('child_process');
+const { extractFrontmatter } = require('../lib/frontmatter');
 
 // ---------------------------------------------------------------------------
 // Result helper
@@ -47,43 +48,6 @@ function result(dimCode, status, message, evidence) {
   };
 }
 
-// ---------------------------------------------------------------------------
-// YAML frontmatter parser (lightweight, no external deps)
-// ---------------------------------------------------------------------------
-
-/**
- * Extract YAML frontmatter from markdown content as key-value pairs.
- * Handles both \n and \r\n line endings.
- * @param {string} content
- * @returns {object} Parsed frontmatter fields
- */
-function parseFrontmatter(content) {
-  const lines = content.replace(/\r\n/g, '\n').split('\n');
-  if (lines[0] !== '---') return {};
-
-  const fields = {};
-  for (let i = 1; i < lines.length; i++) {
-    if (lines[i] === '---') break;
-    const match = lines[i].match(/^(\w[\w_-]*):\s*(.*)$/);
-    if (match) {
-      let value = match[2].trim();
-      // Strip surrounding quotes
-      if ((value.startsWith('"') && value.endsWith('"')) ||
-          (value.startsWith("'") && value.endsWith("'"))) {
-        value = value.slice(1, -1);
-      }
-      // Parse arrays (simple single-line format)
-      if (value === '[]') {
-        value = [];
-      } else if (value === 'null' || value === '') {
-        value = value === 'null' ? null : value;
-      }
-      fields[match[1]] = value;
-    }
-  }
-  return fields;
-}
-
 // ---------------------------------------------------------------------------
 // Session log helper (shared by WC-01, WC-07, WC-09, WC-10)
 // ---------------------------------------------------------------------------
@@ -285,7 +249,7 @@ function checkCompactionQuality(planningDir, _config) {
   const statePath = path.join(planningDir, 'STATE.md');
   try {
     const stateContent = fs.readFileSync(statePath, 'utf8');
-    const fm = parseFrontmatter(stateContent);
+    const fm = extractFrontmatter(stateContent);
     const criticalFields = ['current_phase', 'status', 'last_activity'];
     const missing = criticalFields.filter(f => fm[f] === undefined || fm[f] === '');
     if (missing.length > 0) {
@@ -330,7 +294,7 @@ function checkStateFileIntegrity(planningDir, _config) {
     ]);
   }
 
-  const fm = parseFrontmatter(content);
+  const fm = extractFrontmatter(content);
   const evidence = [];
 
   // Get actual phase directories
@@ -429,7 +393,7 @@ function checkStateFrontmatterIntegrity(planningDir, _config) {
     ]);
   }
 
-  const fm = parseFrontmatter(content);
+  const fm = extractFrontmatter(content);
   const evidence = [];
 
   // Required fields
@@ -799,7 +763,7 @@ function checkArtifactFormatValidation(planningDir, _config) {
     for (const sf of summaryFiles) {
       try {
         const content = fs.readFileSync(path.join(dirPath, sf), 'utf8');
-        const fm = parseFrontmatter(content);
+        const fm = extractFrontmatter(content);
         const missingFields = [];
 
         if (fm.requires === undefined) missingFields.push('requires');

package/plugins/pbr/scripts/check-config-change.js

@@ -64,8 +64,6 @@ function validateConfig(configPath) {
     }
   }
 
-  // local_llm is deprecated — no advisory for missing key
-
   // Check version
   if (config.version && config.version < 2) {
     warnings.push(`Config version ${config.version} is outdated — expected version 2+`);
@@ -92,11 +90,6 @@ function validateConfig(configPath) {
     }
   }
 
-  // DEPRECATED: local_llm infrastructure removed in v14.0
-  if (config.local_llm && config.local_llm.enabled === true) {
-    warnings.push('local_llm feature is deprecated and has no effect. Set enabled: false to suppress this warning.');
-  }
-
   return warnings;
 }
 

package/plugins/pbr/scripts/check-cross-plugin-sync.js

@@ -29,7 +29,7 @@ function checkCrossPluginSync(data) {
 
   let stagedFiles;
   try {
-    stagedFiles = execSync('git diff --cached --name-only', { encoding: 'utf8' }).trim().split('\n').filter(Boolean);
+    stagedFiles = execSync('git diff --cached --name-only', { encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] }).trim().split('\n').filter(Boolean);
   } catch (_e) {
     return null;
   }

package/plugins/pbr/scripts/check-plan-format.js

@@ -25,8 +25,6 @@ const fs = require('fs');
 const path = require('path');
 const { logHook } = require('./hook-logger');
 const { logEvent } = require('./event-logger');
-const { resolveConfig } = require('./lib/local-llm/health');
-const { classifyArtifact } = require('./lib/local-llm/operations/classify-artifact');
 
 // Import all validators from extracted module
 const {
@@ -49,20 +47,6 @@ const {
   validateContext
 } = require('./lib/format-validators');
 
-/**
- * Load and resolve the local_llm config block from .planning/config.json.
- * Returns a resolved config (always safe to use -- disabled by default on error).
- */
-function loadLocalLlmConfig() {
-  try {
-    const configPath = path.join(process.cwd(), '.planning', 'config.json');
-    const parsed = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-    return resolveConfig(parsed.local_llm);
-  } catch (_e) {
-    return resolveConfig(undefined);
-  }
-}
-
 async function main() {
   let input = '';
 
@@ -119,22 +103,6 @@ async function main() {
         result.warnings.push(`Plan file "${basename}" uses non-standard naming. Expected format: PLAN-{NN}.md or PLAN.md. Phase-prefixed names (e.g., "10-02-PLAN.md") bypass some validation checks.`);
       }
 
-      // LLM advisory enrichment -- advisory only, never blocks
-      if ((isPlan || isSummary) && result.errors.length === 0) {
-        try {
-          const llmConfig = loadLocalLlmConfig();
-          const planningDir = path.join(process.cwd(), '.planning');
-          const fileType = isPlan ? 'PLAN' : 'SUMMARY';
-          const llmResult = await classifyArtifact(llmConfig, planningDir, content, fileType, data.session_id);
-          if (llmResult && llmResult.classification) {
-            const llmNote = `Local LLM: ${fileType} classified as "${llmResult.classification}" (confidence: ${(llmResult.confidence * 100).toFixed(0)}%)${llmResult.reason ? ' — ' + llmResult.reason : ''}`;
-            result.warnings.push(llmNote);
-          }
-        } catch (_llmErr) {
-          // Never propagate LLM errors
-        }
-      }
-
       const eventType = isPlan ? 'plan-validated' : isVerification ? 'verification-validated' : isRoadmap ? 'roadmap-validated' : isLearnings ? 'learnings-validated' : isConfig ? 'config-validated' : isResearch ? 'research-validated' : isContext ? 'context-validated' : 'summary-validated';
 
       // Detect Write vs Edit: Write = full creation/overwrite (likely first attempt)

package/plugins/pbr/scripts/check-roadmap-sync.js

@@ -17,6 +17,7 @@ const fs = require('fs');
 const path = require('path');
 const { logHook } = require('./hook-logger');
 const { logEvent } = require('./event-logger');
+const { extractFrontmatter } = require('./lib/frontmatter');
 
 const LIFECYCLE_STATUSES = ['planned', 'built', 'partial', 'verified'];
 
@@ -197,29 +198,32 @@ function main() {
 
 /**
  * Extract current phase number and status from STATE.md.
- * Handles common formats:
- *   "**Phase**: 03 - slug-name"
- *   "Phase: 3"
- *   "Current phase: 03-slug-name"
- *   "**Status**: planned"
- *   "Phase status: built"
+ * Tries frontmatter first (v2), falls back to body regex (legacy).
+ * Delegates frontmatter parsing to canonical extractFrontmatter.
+ * Returns { phase, status } or null if unparseable.
  */
-function parseState(content) {
+const parseState = (content) => {
+  // Try frontmatter first (v2 format)
+  const fm = extractFrontmatter(content);
+  if (fm.current_phase && fm.status) {
+    return {
+      phase: normalizePhaseNum(String(fm.current_phase)),
+      status: fm.status.toLowerCase()
+    };
+  }
+  // Fallback: body regex for legacy formats
   const phaseMatch = content.match(
     /\*{0,2}(?:Current\s+)?Phase\*{0,2}:\s*(\d+(?:\.\d+)?)/i
   );
-
   const statusMatch = content.match(
     /\*{0,2}(?:Phase\s+)?Status\*{0,2}:\s*["']?(\w+)["']?/i
   );
-
   if (!phaseMatch || !statusMatch) return null;
-
   return {
     phase: normalizePhaseNum(phaseMatch[1]),
     status: statusMatch[1].toLowerCase()
   };
-}
+};
 
 /**
  * Find the status for a given phase in ROADMAP.md's Phase Overview table.