@sienklogic/plan-build-run 2.19.1 → 2.19.2

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

@@ -43,8 +43,6 @@
  *   auto-cleanup --phase N | --milestone vN — Auto-close todos and archive notes matching phase/milestone deliverables
  *   state reconcile          — Detect and repair STATE.md/ROADMAP.md desync
  *   state backup             — Create timestamped backup of STATE.md and ROADMAP.md
- *   llm <subcommand>                     — DEPRECATED: local_llm removed (no-op)
- *   validate-project        — Comprehensive .planning/ integrity check
  *   phase add <slug> [--after N] [--goal "..."] [--depends-on N] — Add phase with ROADMAP.md integration
  *   phase remove <N>             — Remove an empty phase directory (with renumbering)
  *   phase list [--status S] [--before N] — List phase directories with optional status/before filters
@@ -62,14 +60,9 @@
  *   learnings query-global [--tags X] [--project P] — Query global knowledge files
  *   data status                      — Freshness report for research/, intel/, codebase/ directories
  *   data prune --before <ISO-date> [--dry-run] — Archive stale research/codebase files
- *   incidents list [--limit N]       — List recent incidents as JSON array
- *   incidents summary               — Aggregate incident stats by type/severity/source
- *   incidents query [--type T] [--severity S] [--source S] [--last Nd|Nh] — Filter incidents
  *   nk record --title "..." --category "..." --files "f1,f2" --tried "..." --failed "..." — Record negative knowledge entry
  *   nk list [--category X] [--phase X] [--status X] — List negative knowledge entries
  *   nk resolve <slug>               — Mark a negative knowledge entry as resolved
- *   insights import <html-path> [--project <name>] — Parse insights HTML report into learnings
- *   audit plan-checks [--last N]     — List all plan-check results from hooks.jsonl (default: last 30 days)
  *   hooks perf [--last N] [--json]  — Show P50/P95/P99 hook performance report from hooks-*.jsonl
  *   spot-check <phaseSlug> <planId>  — Verify SUMMARY, key_files, and commits exist for a plan
  *   staleness-check <phase-slug>  — Check if phase plans are stale vs dependencies
@@ -143,7 +136,6 @@ const {
 
 const {
   parseStateMd,
-  updateLegacyStateField,
   updateFrontmatterField,
   stateLoad: _stateLoad,
   stateCheckProgress: _stateCheckProgress,
@@ -231,7 +223,6 @@ const {
 } = require('./lib/migrate');
 
 const {
-  spotCheck: _spotCheck,
   verifySpotCheck: _verifySpotCheck
 } = require('./lib/spot-check');
 
@@ -253,9 +244,7 @@ const {
   queryGlobal: _queryGlobal
 } = require('./lib/learnings');
 
-const {
-  insightsImport: _insightsImport
-} = require('./lib/insights-parser');
+// insights-parser.js CLI subcommand removed as dead (plan 140-01)
 
 const {
   referenceGet: _referenceGet
@@ -318,18 +307,13 @@ const {
   quickStatus: _quickStatus
 } = require('./quick-status');
 
-// --- Local LLM imports removed (feature archived in phase 53) ---
-
 const {
   dataStatus: _dataStatus,
   dataPrune: _dataPrune
 } = require('./lib/data-hygiene');
 
-const {
-  list: _incidentsList,
-  query: _incidentsQuery,
-  summary: _incidentsSummary
-} = require('./lib/incidents');
+// incidents.js lib retained (used by error-analysis.js, record-incident.js, dashboard)
+// but CLI subcommands removed as dead (plan 140-01)
 
 // --- Module-level state (for backwards compatibility) ---
 
@@ -360,10 +344,6 @@ function configValidate(preloadedConfig) {
   return _configValidate(preloadedConfig, planningDir);
 }
 
-function insightsImport(htmlPath, projectName) {
-  return _insightsImport(htmlPath, projectName, planningDir);
-}
-
 function stateLoad() {
   return _stateLoad(planningDir);
 }
@@ -556,16 +536,6 @@ function autoArchiveNotes(context) {
   return _autoArchiveNotes(planningDir, context);
 }
 
-function incidentsList(opts) {
-  return _incidentsList({ ...opts, planningDir });
-}
-function incidentsQuery(filter, opts) {
-  return _incidentsQuery(filter, { ...opts, planningDir });
-}
-function incidentsSummary(opts) {
-  return _incidentsSummary({ ...opts, planningDir });
-}
-
 function intelQuery(term) {
   const { intelQuery: _intelQuery } = require('./lib/intel');
   return _intelQuery(term, planningDir);
@@ -602,9 +572,6 @@ function migrate(options) {
   return _applyMigrations(planningDir, options);
 }
 
-function spotCheck(phaseDir, planId) {
-  return _spotCheck(planningDir, phaseDir, planId);
-}
 
 function verifySpotCheck(type, dirPath) {
   return _verifySpotCheck(type, dirPath);
@@ -1285,9 +1252,6 @@ async function main() {
       const { logEvent } = require('./event-logger');
       logEvent(category, event, details);
       output({ logged: true, category, event });
-    } else if (command === 'llm') {
-      // local_llm feature has been removed (phase 53). All subcommands are no-ops.
-      output({ deprecated: true, message: 'local_llm feature has been removed. This subcommand is a no-op.' });
     // --- Compound init commands ---
     } else if (command === "init" && subcommand === "execute-phase") {
       const phase = args[2];
@@ -1533,21 +1497,6 @@ async function main() {
         error('Usage: learnings <ingest|query|check-thresholds|copy-global|query-global>');
         process.exit(1);
       }
-    } else if (command === 'insights') {
-      const subCmd = args[1];
-      if (subCmd === 'import') {
-        const htmlPath = args[2];
-        if (!htmlPath) { error('Usage: insights import <html-file-path> [--project <name>]'); process.exit(1); }
-        let projectName;
-        for (let i = 3; i < args.length; i++) {
-          if (args[i] === '--project' && args[i + 1]) { projectName = args[++i]; }
-        }
-        const result = insightsImport(htmlPath, projectName);
-        output(result);
-      } else {
-        error('Usage: insights <import>');
-        process.exit(1);
-      }
     } else if (command === 'verify' && subcommand === 'spot-check') {
       const scType = args[2];
       const scPath = args[3];
@@ -1598,10 +1547,9 @@ async function main() {
       // Returns JSON: { ok, summary_exists, key_files_checked, commits_present, detail }
       const phaseSlug = args[1];
       const planId = args[2];
-      if (!phaseSlug || !planId) {
-        error('Usage: spot-check <phaseSlug> <planId>');
-      }
-      output(spotCheck(phaseSlug, planId));
+      if (!phaseSlug || !planId) { error('Usage: spot-check <phase-slug> <plan-id>'); process.exit(1); }
+      const { verifySpotCheck } = require('./lib/spot-check');
+      output(verifySpotCheck(phaseSlug, planId));
     } else if (command === 'staleness-check') {
       const slug = args[1];
       if (!slug) { error('Usage: staleness-check <phase-slug>'); process.exit(1); }
@@ -1715,8 +1663,6 @@ async function main() {
       const version = args[1];
       if (!version) error('Usage: pbr-tools.js milestone-stats <version>');
       output(milestoneStats(version));
-    } else if (command === 'validate-project') {
-      output(validateProject());
     } else if (command === 'skill-section') {
       // skill-section --list <skill>  — list all headings
       if (args[1] === '--list') {
@@ -1894,28 +1840,6 @@ async function main() {
       const maxIterations = maxIterIdx !== -1 ? parseInt(args[maxIterIdx + 1], 10) : 3;
       output(ciFix({ dryRun, maxIterations }));
 
-    // ─── Incidents ───────────────────────────────────────────────────────────
-    } else if (command === 'incidents') {
-      const sub = args[1];
-      if (sub === 'list') {
-        const limitIdx = args.indexOf('--limit');
-        const limit = limitIdx !== -1 ? parseInt(args[limitIdx + 1], 10) : 50;
-        output(incidentsList({ limit }));
-      } else if (sub === 'summary') {
-        output(incidentsSummary());
-      } else if (sub === 'query') {
-        const filter = {};
-        for (let i = 2; i < args.length; i++) {
-          if (args[i] === '--type' && args[i + 1]) { filter.type = args[++i]; }
-          else if (args[i] === '--severity' && args[i + 1]) { filter.severity = args[++i]; }
-          else if (args[i] === '--source' && args[i + 1]) { filter.source = args[++i]; }
-          else if (args[i] === '--last' && args[i + 1]) { filter.last = args[++i]; }
-        }
-        output(incidentsQuery(filter));
-      } else {
-        error('Usage: incidents <list|summary|query> [--type T] [--severity S] [--source S] [--last Nd|Nh] [--limit N]');
-      }
-
     } else if (command === 'nk') {
       // Lazy require to avoid loading for unrelated commands
       const nkLib = require(path.join(__dirname, 'lib', 'negative-knowledge'));
@@ -1992,13 +1916,6 @@ async function main() {
       const graphCli = require('./lib/graph-cli');
       graphCli.handleGraphCommand(subcommand, args, planningDir, cwd, output, error);
 
-    // --- Audit Operations ---
-    } else if (command === 'audit' && subcommand === 'plan-checks') {
-      const { auditPlanChecks } = require('./lib/audit');
-      const lastIdx = args.indexOf('--last');
-      const last = lastIdx !== -1 ? parseInt(args[lastIdx + 1], 10) : 30;
-      output(auditPlanChecks({ last }));
-
     // --- Hooks Perf Report ---
     } else if (command === 'hooks' && subcommand === 'perf') {
       const { summarizeHookPerf, formatPerfTable, loadPerfEntries } = require('./lib/perf');
@@ -2031,7 +1948,7 @@ async function main() {
       if (result.error) process.exit(1);
 
     } else {
-      error(`Unknown command: ${args.join(' ')}\nCommands: state load|check-progress|update|patch|advance-plan|record-metric, config validate|load-defaults|save-defaults|resolve-depth, validate health, validate-project, verify summary|plan-structure|phase-completeness|artifacts|key-links|commits|references|spot-check, migrate [--dry-run] [--force], init execute-phase|plan-phase|quick|verify-work|resume|progress|map-codebase, state-bundle <phase>, plan-index, frontmatter, must-haves, phase-info, phase add|remove|list|complete, roadmap update-status|update-plans, history append|load, todo list|get|add|done, auto-cleanup --phase N|--milestone vN, event, llm health|status|classify|score-source|classify-error|summarize|metrics [--session <ISO>]|adjust-thresholds, learnings ingest|query|check-thresholds, incidents list|summary|query, nk record|list|resolve, data status|prune, graph build|query|impact|stats, hooks perf [--last N] [--json], spec parse|diff|reverse|impact, milestone-stats <version>, context-triage [--agents-done N] [--plans-total N] [--step NAME], ci-poll <run-id> [--timeout <seconds>], ci-fix [--dry-run] [--max-iterations N], rollback <manifest-path>, session get|set|clear|dump, claim acquire|release|list, skill-section <skill> <section>|--list <skill>, step-verify <skill> <step> <checklist-json>, suggest-alternatives phase-not-found|missing-prereq|config-invalid [args], tmux detect, quick init, generate-slug|slug-generate, parse-args plan|quick, status fingerprint, quick-status, help, skill-metadata <name>`);
+      error(`Unknown command: ${args.join(' ')}\nCommands: state load|check-progress|update|patch|advance-plan|record-metric, config validate|load-defaults|save-defaults|resolve-depth, validate health, verify summary|plan-structure|phase-completeness|artifacts|key-links|commits|references|spot-check, migrate [--dry-run] [--force], init execute-phase|plan-phase|quick|verify-work|resume|progress|map-codebase, state-bundle <phase>, plan-index, frontmatter, must-haves, phase-info, phase add|remove|list|complete, roadmap update-status|update-plans, history append|load, todo list|get|add|done, auto-cleanup --phase N|--milestone vN, event, llm health|status|classify|score-source|classify-error|summarize|metrics [--session <ISO>]|adjust-thresholds, learnings ingest|query|check-thresholds, nk record|list|resolve, data status|prune, graph build|query|impact|stats, hooks perf [--last N] [--json], spec parse|diff|reverse|impact, milestone-stats <version>, context-triage [--agents-done N] [--plans-total N] [--step NAME], ci-poll <run-id> [--timeout <seconds>], ci-fix [--dry-run] [--max-iterations N], rollback <manifest-path>, session get|set|clear|dump, claim acquire|release|list, skill-section <skill> <section>|--list <skill>, step-verify <skill> <step> <checklist-json>, suggest-alternatives phase-not-found|missing-prereq|config-invalid [args], tmux detect, quick init, generate-slug|slug-generate, parse-args plan|quick, status fingerprint, quick-status, help, skill-metadata <name>`);
     }
   } catch (e) {
     error(e.message);
@@ -2039,6 +1956,6 @@ async function main() {
 }
 
 if (require.main === module || process.argv[1] === __filename) { main().catch(err => { process.stderr.write(err.message + '\n'); process.exit(1); }); }
-module.exports = { KNOWN_AGENTS, initExecutePhase, initPlanPhase, initQuick, initVerifyWork, initResume, initProgress, initStateBundle: stateBundle, stateBundle, statePatch, stateAdvancePlan, stateRecordMetric, stateRecordActivity, stateUpdateProgress, parseStateMd, parseRoadmapMd, parseYamlFrontmatter, parseMustHaves, countMustHaves, stateLoad, stateCheckProgress, configLoad, configClearCache, configValidate, lockedFileUpdate, planIndex, determinePhaseStatus, findFiles, atomicWrite, tailLines, frontmatter, mustHavesCollect, phaseInfo, stateUpdate, roadmapUpdateStatus, roadmapUpdatePlans, roadmapAnalyze, updateLegacyStateField, updateFrontmatterField, updateTableRow, findRoadmapRow, resolveDepthProfile, DEPTH_PROFILE_DEFAULTS, historyAppend, historyLoad, VALID_STATUS_TRANSITIONS, validateStatusTransition, writeActiveSkill, validateProject, phaseAdd, phaseRemove, phaseList, loadUserDefaults, saveUserDefaults, mergeUserDefaults, USER_DEFAULTS_PATH, todoList, todoGet, todoAdd, todoDone, migrate, spotCheck, referenceGet, milestoneStats, contextTriage, stalenessCheck, summaryGate, checkpointInit, checkpointUpdate, seedsMatch, ciPoll, ciFix, parseJestOutput: _parseJestOutput, parseLintOutput: _parseLintOutput, autoFixLint: _autoFixLint, runCiFixLoop: _runCiFixLoop, rollbackPlan, sessionLoad, sessionSave, SESSION_ALLOWED_KEYS, claimAcquire, claimRelease, claimList, skillSectionGet, listSkillHeadings, stepVerify: _stepVerify, phaseAlternatives: _phaseAlternatives, prerequisiteAlternatives: _prereqAlternatives, configAlternatives: _configAlternatives, phaseComplete, phaseInsert, quickStatus, autoCloseTodos, autoArchiveNotes, buildCleanupContext, incidentsList, incidentsQuery, incidentsSummary, helpListCmd, skillMetadataCmd, initMapCodebase };
+module.exports = { KNOWN_AGENTS, initExecutePhase, initPlanPhase, initQuick, initVerifyWork, initResume, initProgress, initStateBundle: stateBundle, stateBundle, statePatch, stateAdvancePlan, stateRecordMetric, stateRecordActivity, stateUpdateProgress, parseStateMd, parseRoadmapMd, parseYamlFrontmatter, parseMustHaves, countMustHaves, stateLoad, stateCheckProgress, configLoad, configClearCache, configValidate, lockedFileUpdate, planIndex, determinePhaseStatus, findFiles, atomicWrite, tailLines, frontmatter, mustHavesCollect, phaseInfo, stateUpdate, roadmapUpdateStatus, roadmapUpdatePlans, roadmapAnalyze, updateFrontmatterField, updateTableRow, findRoadmapRow, resolveDepthProfile, DEPTH_PROFILE_DEFAULTS, historyAppend, historyLoad, VALID_STATUS_TRANSITIONS, validateStatusTransition, writeActiveSkill, validateProject, phaseAdd, phaseRemove, phaseList, loadUserDefaults, saveUserDefaults, mergeUserDefaults, USER_DEFAULTS_PATH, todoList, todoGet, todoAdd, todoDone, migrate, verifySpotCheck, referenceGet, milestoneStats, contextTriage, stalenessCheck, summaryGate, checkpointInit, checkpointUpdate, seedsMatch, ciPoll, ciFix, parseJestOutput: _parseJestOutput, parseLintOutput: _parseLintOutput, autoFixLint: _autoFixLint, runCiFixLoop: _runCiFixLoop, rollbackPlan, sessionLoad, sessionSave, SESSION_ALLOWED_KEYS, claimAcquire, claimRelease, claimList, skillSectionGet, listSkillHeadings, stepVerify: _stepVerify, phaseAlternatives: _phaseAlternatives, prerequisiteAlternatives: _prereqAlternatives, configAlternatives: _configAlternatives, phaseComplete, phaseInsert, quickStatus, autoCloseTodos, autoArchiveNotes, buildCleanupContext, helpListCmd, skillMetadataCmd, initMapCodebase };
 // NOTE: validateProject, phaseAdd, phaseRemove, phaseList were previously CLI-only (not exported).
 // They are now exported for testability. This is additive and backwards-compatible.

package/plugins/pbr/dist/post-bash-triage.js

@@ -1,12 +1,8 @@
 #!/usr/bin/env node
 
 /**
- * PostToolUse hook for Bash: Triages test failure output using the local LLM.
- *
- * When a Bash command that looks like a test invocation exits with a non-zero
- * exit code, this hook sends the output to the local LLM for classification.
- * The triage result is returned as advisory context to help the frontier model
- * focus on the right failure category without re-parsing raw test output.
+ * PostToolUse hook for Bash: Previously triaged test failures via local LLM.
+ * Now a no-op stub — local LLM infrastructure has been removed.
  *
  * Exit codes:
  *   0 = always (PostToolUse hook, never blocks)
@@ -14,11 +10,7 @@
 
 'use strict';
 
-const fs = require('fs');
-const path = require('path');
 const { logHook } = require('./hook-logger');
-const { resolveConfig } = require('./lib/local-llm/health');
-const { triageTestOutput } = require('./lib/local-llm/operations/triage-test-output');
 
 const TEST_COMMAND_PATTERNS = [
   /\bnpm\s+test\b/,
@@ -48,63 +40,13 @@ function detectTestRunner(command) {
   return null;
 }
 
-/**
- * Load and resolve the local_llm config block from .planning/config.json.
- */
-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);
-  }
-}
-
 /**
  * Check if Bash output contains test failure and triage it.
- * @param {object} data - parsed hook data
+ * Previously used local LLM for classification — now a no-op stub.
+ * @param {object} _data - parsed hook data
  * @returns {Promise<{output: object}|null>}
  */
-async function checkTestTriage(data) {
-  const command = data.tool_input?.command || '';
-  const toolOutput = data.tool_output || '';
-  const exitCode = data.tool_exit_code;
-
-  // Only triage test commands that failed
-  if (exitCode === 0 || exitCode === undefined) return null;
-  if (!TEST_COMMAND_PATTERNS.some(p => p.test(command))) return null;
-  if (!toolOutput || toolOutput.length < 20) return null;
-
-  const cwd = process.cwd();
-  const llmConfig = loadLocalLlmConfig(cwd);
-  const planningDir = path.join(cwd, '.planning');
-  const testRunner = detectTestRunner(command);
-
-  // Truncate to last 2000 chars — test failures are usually at the end
-  const tail = toolOutput.length > 2000 ? toolOutput.slice(-2000) : toolOutput;
-
-  try {
-    const llmResult = await triageTestOutput(llmConfig, planningDir, tail, testRunner, data.session_id);
-    if (llmResult && llmResult.category && llmResult.category !== 'unknown') {
-      logHook('post-bash-triage', 'PostToolUse', 'triage', {
-        category: llmResult.category,
-        file_hint: llmResult.file_hint,
-        runner: testRunner
-      });
-
-      let msg = `[pbr] Test failure triage: ${llmResult.category}`;
-      if (llmResult.file_hint) {
-        msg += ` (likely: ${llmResult.file_hint})`;
-      }
-      msg += ` (confidence: ${(llmResult.confidence * 100).toFixed(0)}%)`;
-
-      return { output: { additionalContext: msg } };
-    }
-  } catch (_llmErr) {
-    // Never propagate LLM errors
-  }
-
+async function checkTestTriage(_data) {
   return null;
 }
 

package/plugins/pbr/dist/post-hoc.js

@@ -12,56 +12,7 @@
 const fs = require('fs');
 const path = require('path');
 const { execSync } = require('child_process');
-
-/**
- * Minimal YAML frontmatter parser. Returns key-value pairs from --- blocks.
- * Handles arrays (lines starting with "  - ") and nested objects (indented keys).
- */
-function parseFrontmatter(content) {
-  if (!content) return {};
-  const match = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
-  if (!match) return {};
-
-  const lines = match[1].split(/\r?\n/);
-  const result = {};
-  let currentKey = null;
-  let currentNested = null;
-
-  for (const line of lines) {
-    // Nested array item: "  - value"
-    const arrayMatch = line.match(/^(\s{2,})- "?([^"]*)"?$/);
-    if (arrayMatch && currentKey) {
-      if (currentNested) {
-        if (!result[currentKey]) result[currentKey] = {};
-        if (!result[currentKey][currentNested]) result[currentKey][currentNested] = [];
-        result[currentKey][currentNested].push(arrayMatch[2]);
-      } else {
-        if (!Array.isArray(result[currentKey])) result[currentKey] = [];
-        result[currentKey].push(arrayMatch[2]);
-      }
-      continue;
-    }
-
-    // Nested key: "  key:"
-    const nestedKeyMatch = line.match(/^\s{2,}(\w+):\s*$/);
-    if (nestedKeyMatch && currentKey) {
-      currentNested = nestedKeyMatch[1];
-      if (!result[currentKey]) result[currentKey] = {};
-      continue;
-    }
-
-    // Top-level key-value: "key: value" or "key:"
-    const kvMatch = line.match(/^(\w[\w_]*):\s*(.*)$/);
-    if (kvMatch) {
-      currentKey = kvMatch[1];
-      currentNested = null;
-      const val = kvMatch[2].replace(/^"(.*)"$/, '$1').trim();
-      result[currentKey] = val || null;
-    }
-  }
-
-  return result;
-}
+const { extractFrontmatter } = require('./lib/frontmatter');
 
 /**
  * Read all PLAN-*.md files from phaseDir and parse their frontmatter.
@@ -72,7 +23,7 @@ function readPlanFiles(phaseDir) {
     const files = fs.readdirSync(phaseDir).filter(f => /^PLAN.*\.md$/i.test(f));
     for (const file of files) {
       const content = fs.readFileSync(path.join(phaseDir, file), 'utf-8');
-      const fm = parseFrontmatter(content);
+      const fm = extractFrontmatter(content);
       plans.push(fm);
     }
   } catch (_e) {
@@ -207,7 +158,7 @@ function generateLearnings(phaseDir, _planningDir) {
 
   try {
     const verContent = fs.readFileSync(verPath, 'utf-8');
-    const fm = parseFrontmatter(verContent);
+    const fm = extractFrontmatter(verContent);
     verStatus = fm.status || '';
     if (Array.isArray(fm.gaps)) {
       gaps = fm.gaps;

package/plugins/pbr/dist/post-write-dispatch.js

@@ -185,42 +185,6 @@ async function processEvent(data, planningDir) {
     logHook('post-write-dispatch', 'PostToolUse', 'error', { check: 'checkReadFirst', error: e.message });
   }
 
-  // LLM file intent classification — advisory enrichment for non-planning files
-  const filePath = data.tool_input?.file_path || data.tool_input?.path || '';
-  const normalizedPath = filePath.replace(/\\/g, '/');
-  if (filePath && !normalizedPath.includes('.planning/') && !normalizedPath.includes('.planning\\')) {
-    try {
-      const { resolveConfig } = require('./lib/local-llm/health');
-      const { classifyFileIntent } = require('./lib/local-llm/operations/classify-file-intent');
-      const llmConfig = (() => {
-        try {
-          const configPath = path.join(planningDir, 'config.json');
-          const parsed = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-          return resolveConfig(parsed.local_llm);
-        } catch (_e) {
-          return resolveConfig(undefined);
-        }
-      })();
-
-      let contentSnippet = '';
-      try {
-        const content = data.tool_input?.content || data.tool_input?.new_string || '';
-        contentSnippet = content.slice(0, 400);
-      } catch (_e) {
-        // No content available
-      }
-
-      if (contentSnippet) {
-        const llmResult = await classifyFileIntent(llmConfig, planningDir, filePath, contentSnippet, data.session_id);
-        if (llmResult && llmResult.file_type) {
-          results.push(`[pbr] File classified: ${llmResult.file_type}/${llmResult.intent} (confidence: ${(llmResult.confidence * 100).toFixed(0)}%)`);
-        }
-      }
-    } catch (_llmErr) {
-      // Never propagate LLM errors
-    }
-  }
-
   // Merge all results into a single response
   if (results.length > 0) {
     return { additionalContext: results.join('\n') };

package/plugins/pbr/dist/pre-bash-dispatch.js

@@ -48,7 +48,7 @@
 
 const { logHook } = require('./hook-logger');
 const { checkDangerous } = require('./check-dangerous-commands');
-const { checkCommit, enrichCommitLlm } = require('./validate-commit');
+const { checkCommit } = require('./validate-commit');
 const { checkUnmanagedCommit } = require('./enforce-pbr-workflow');
 const { checkRequirePaths, checkMirrorSync, checkLintErrors } = require('./lib/pre-commit-checks');
 // Cross-plugin sync disabled -- derivative plugins updated separately
@@ -138,12 +138,6 @@ async function processEvent(data) {
     //   warnings.push(syncResult.additionalContext);
     // }
 
-    // LLM commit semantic classification -- advisory only
-    const llmAdvisory = await enrichCommitLlm(data);
-    if (llmAdvisory) {
-      warnings.push(llmAdvisory);
-    }
-
     if (warnings.length > 0) {
       logHook('pre-bash-dispatch', 'PreToolUse', 'warn', { warnings: warnings.length, cmd: command.substring(0, 80) });
       return {

package/plugins/pbr/dist/pre-task-dispatch.js

@@ -31,8 +31,6 @@ const fs = require('fs');
 const path = require('path');
 const { logHook } = require('./hook-logger');
 const { checkBudget } = require('./enforce-context-budget');
-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');
 
@@ -52,19 +50,6 @@ const { checkUserConfirmationGate } = require('./lib/gates/user-confirmation');
 
 const MAX_DESCRIPTION_LENGTH = 100;
 
-/**
- * Load and resolve the local_llm config block from .planning/config.json.
- */
-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);
-  }
-}
-
 /**
  * Check a parsed hook data object for Task() validation issues.
  * Returns an array of warning strings (empty if all good).
@@ -223,19 +208,6 @@ async function processEvent(data) {
   if (nonPbrAgentResult) warnings.push(nonPbrAgentResult.output.additionalContext);
   if (planValGate && planValGate.warning) warnings.push(planValGate.warning);
 
-  // LLM task coherence check — advisory only
-  try {
-    const llmCwd = data.cwd || process.env.PBR_PROJECT_ROOT || process.cwd();
-    const llmConfig = loadLocalLlmConfig(llmCwd);
-    const llmPlanningDir = path.join(llmCwd, '.planning');
-    const llmResult = await llmValidateTask(llmConfig, llmPlanningDir, 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('pre-task-dispatch', 'PreToolUse', 'warn', { warning });

package/plugins/pbr/dist/progress-tracker.js

@@ -18,7 +18,6 @@ const { logHook } = require('./hook-logger');
 const { logEvent } = require('./event-logger');
 const { configLoad, sessionSave } = require('./pbr-tools');
 const { ensureSessionDir, cleanStaleSessions } = require('./lib/core');
-const { resolveConfig, checkHealth, warmUp } = require('./lib/local-llm/health');
 
 // Re-export from extracted modules for backward compatibility
 const {
@@ -134,34 +133,10 @@ async function main() {
     tryLaunchHookServer(config, planningDir);
   }
 
-  // Write session-start timestamp for local-llm metrics correlation
-  // Primary: write to .session.json (unified session state)
-  // Legacy: also write .session-start file for session-cleanup.js backward compat
+  // Write session-start timestamp to .session.json (unified session state)
   const sessionStart = new Date().toISOString();
   try { sessionSave(planningDir, { sessionStart }, sessionId); } catch (_e) { /* non-fatal */ }
-  const sessionStartFile = path.join(planningDir, '.session-start');
-  try {
-    fs.writeFileSync(sessionStartFile, sessionStart, 'utf8');
-  } catch (_e) { /* non-fatal */ }
 
-  // Local LLM health check (advisory only -- never blocks SessionStart)
-  let llmContext = '';
-  try {
-    const rawLlmConfig = config && config.local_llm;
-    const llmConfig = resolveConfig(rawLlmConfig);
-    if (llmConfig.enabled) {
-      const health = await checkHealth(llmConfig);
-      if (health.available) {
-        llmContext = `\nLocal LLM: ${llmConfig.model} (${health.warm ? 'warm' : 'cold start'})`;
-        if (!health.warm) {
-          // Fire warm-up without awaiting -- 23s cold start must not block hook
-          warmUp(llmConfig);
-        }
-      } else if (health.reason !== 'disabled') {
-        llmContext = `\nLocal LLM: unavailable -- ${health.detail || health.reason}`;
-      }
-    }
-  } catch (_e) { /* graceful degradation -- never surface to user */ }
 
   // Enrich context with recent session activity from hook server (advisory, fail-open)
   let enrichedContext = '';
@@ -179,7 +154,7 @@ async function main() {
 
   if (context) {
     const output = {
-      additionalContext: context + sessionWarning + llmContext + enrichedContext
+      additionalContext: context + sessionWarning + enrichedContext
     };
     process.stdout.write(JSON.stringify(output));
     logHook('progress-tracker', 'SessionStart', 'injected', { hasState: true });

package/plugins/pbr/dist/session-cleanup.js

@@ -24,7 +24,6 @@ const { logHook, getLogFilename: getHooksFilename, cleanOldHookLogs } = require(
 const { getLogFilename: getEventsFilename, cleanOldEventLogs } = require('./event-logger');
 const { tailLines, configLoad } = require('./pbr-tools');
 const { removeSessionDir, releaseSessionClaims } = require('./lib/core');
-const { readSessionMetrics, summarizeMetrics, formatSessionSummary } = require('./lib/local-llm/metrics');
 const { writeSnapshot } = require('./lib/snapshot-manager');
 
 function readStdin() {
@@ -492,35 +491,6 @@ function main() {
     }
   } catch (_e) { /* metrics display is best-effort */ }
 
-  // Local LLM metrics summary (SessionEnd — sync reads only, never throws)
-  let llmAdditionalContext = null;
-  try {
-    const sessionStartFile = path.join(planningDir, '.session-start');
-    if (fs.existsSync(sessionStartFile)) {
-      const sessionStartTime = fs.readFileSync(sessionStartFile, 'utf8').trim();
-      const entries = readSessionMetrics(planningDir, sessionStartTime);
-      if (entries.length > 0) {
-        const summary = summarizeMetrics(entries);
-        logHook('session-cleanup', 'SessionEnd', 'llm-metrics', {
-          total_calls: summary.total_calls,
-          fallback_count: summary.fallback_count,
-          avg_latency_ms: summary.avg_latency_ms,
-          tokens_saved: summary.tokens_saved,
-          cost_saved_usd: summary.cost_saved_usd
-        });
-        if (summary.total_calls > 0) {
-          let modelName = null;
-          try {
-            const rawConfig = configLoad(planningDir) || {};
-            modelName = (rawConfig.local_llm && rawConfig.local_llm.model) || null;
-          } catch (_e) { /* config read failure is non-fatal */ }
-          llmAdditionalContext = formatSessionSummary(summary, modelName);
-        }
-      }
-      // Clean up session-start file
-      try { fs.unlinkSync(sessionStartFile); } catch (_e) { /* non-fatal */ }
-    }
-  } catch (_e) { /* metrics never crash the hook */ }
 
   // Surface compliance violations from this session
   let complianceContext = null;
@@ -568,7 +538,7 @@ function main() {
     // Snapshot failure must never crash SessionEnd
   }
 
-  const combinedContext = [metricsContext, llmAdditionalContext, complianceContext].filter(Boolean).join('\n');
+  const combinedContext = [metricsContext, complianceContext].filter(Boolean).join('\n');
   if (combinedContext) {
     process.stdout.write(JSON.stringify({ additionalContext: combinedContext }) + '\n');
   }

package/plugins/pbr/dist/status-line.js

@@ -16,6 +16,7 @@ const cp = require('child_process');
 const { logHook } = require('./hook-logger');
 const { configLoad } = require('./pbr-tools');
 const { resolveSessionPath } = require('./lib/core');
+const { extractFrontmatter } = require('./lib/frontmatter');
 // ANSI color codes
 const c = {
   reset: '\x1b[0m',
@@ -542,26 +543,27 @@ function main() {
 
 /**
  * Parse YAML frontmatter from STATE.md content.
- * Returns an object with frontmatter fields, or null if no frontmatter.
+ * Thin adapter around canonical extractFrontmatter preserving null-on-missing
+ * and filtering out null/undefined string values for status-line display.
  */
-function parseFrontmatter(content) {
+const parseFrontmatter = (content) => {
   if (!content.startsWith('---')) return null;
   const endIdx = content.indexOf('---', 3);
   if (endIdx === -1) return null;
-  const fm = content.substring(3, endIdx);
+  const raw = extractFrontmatter(content);
   const result = {};
-  for (const line of fm.split(/\r?\n/)) {
-    const m = line.match(/^(\w[\w_]*):\s*"?([^"]*)"?\s*$/);
-    if (m) {
-      const val = m[2].trim();
-      // Skip YAML nulls and empty values — they'd render as literal "null"
-      if (val && val !== 'null' && val !== 'undefined') {
-        result[m[1]] = val;
+  for (const [key, val] of Object.entries(raw)) {
+    if (typeof val === 'object') {
+      // Keep objects/arrays (non-empty)
+      if (Array.isArray(val) ? val.length > 0 : Object.keys(val).length > 0) {
+        result[key] = val;
       }
+    } else if (typeof val === 'string' && val && val !== 'null' && val !== 'undefined') {
+      result[key] = val;
     }
   }
   return result;
-}
+};
 
 function buildStatusLine(content, ctxPercent, cfg, stdinData, planningDir) {
   const config = cfg || DEFAULTS;