@sienklogic/plan-build-run 2.19.1 → 2.19.2

package/plugins/pbr/scripts/check-subagent-output.js

@@ -21,8 +21,6 @@ const fs = require('fs');
 const path = require('path');
 const { logHook } = require('./hook-logger');
 const { KNOWN_AGENTS, sessionLoad } = require('./pbr-tools');
-const { resolveConfig } = require('./lib/local-llm/health');
-const { classifyError } = require('./lib/local-llm/operations/classify-error');
 const { resolveSessionPath } = require('./lib/core');
 const { logEvent } = require('./event-logger');
 const { recordOutcome } = require('./trust-tracker');
@@ -102,15 +100,6 @@ function readStdin() {
   return {};
 }
 
-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 (_) {
-    return resolveConfig(undefined);
-  }
-}
 
 async function main() {
   const data = readStdin();
@@ -248,22 +237,8 @@ async function main() {
       agent_type: agentType,
       warnings: skillWarnings
     });
-    // LLM error classification -- advisory enrichment
-    let llmCategoryNote = '';
-    try {
-      const llmConfig = loadLocalLlmConfig(cwd);
-      const errorText = (data.tool_output || '').substring(0, 500);
-      if (errorText) {
-        const llmResult = await classifyError(llmConfig, planningDir, errorText, agentType, data.session_id);
-        if (llmResult && llmResult.category) {
-          llmCategoryNote = `\nLLM error category: ${llmResult.category} (confidence: ${(llmResult.confidence * 100).toFixed(0)}%)`;
-        }
-      }
-    } catch (_llmErr) {
-      // Never propagate
-    }
     const msg = `Warning: Agent ${agentType} completed but no ${outputSpec.description} was found.\nSkill-specific warnings:\n` +
-      skillWarnings.map(w => `- ${w}`).join('\n') + llmCategoryNote;
+      skillWarnings.map(w => `- ${w}`).join('\n');
     process.stdout.write(JSON.stringify({ additionalContext: msg }));
   } else if (genericMissing) {
     logHook('check-subagent-output', 'PostToolUse', 'warning', {
@@ -271,22 +246,8 @@ async function main() {
       expected: outputSpec.description,
       found: 'none'
     });
-    // LLM error classification -- advisory enrichment
-    let llmCategoryNote = '';
-    try {
-      const llmConfig = loadLocalLlmConfig(cwd);
-      const errorText = (data.tool_output || '').substring(0, 500);
-      if (errorText) {
-        const llmResult = await classifyError(llmConfig, planningDir, errorText, agentType, data.session_id);
-        if (llmResult && llmResult.category) {
-          llmCategoryNote = `\nLLM error category: ${llmResult.category} (confidence: ${(llmResult.confidence * 100).toFixed(0)}%)`;
-        }
-      }
-    } catch (_llmErr) {
-      // Never propagate
-    }
     const output = {
-      additionalContext: `[WARN] Agent ${agentType} completed but no ${outputSpec.description} was found. Likely causes: (1) agent hit an error mid-run, (2) wrong working directory. To fix: re-run the parent skill — the executor gate will block until the output is present. Check the Task() output above for error details.` + llmCategoryNote
+      additionalContext: `[WARN] Agent ${agentType} completed but no ${outputSpec.description} was found. Likely causes: (1) agent hit an error mid-run, (2) wrong working directory. To fix: re-run the parent skill — the executor gate will block until the output is present. Check the Task() output above for error details.`
     };
     process.stdout.write(JSON.stringify(output));
   } else if (skillWarnings.length > 0) {
@@ -412,32 +373,15 @@ async function handleHttp(reqBody) {
     }
   }
 
-  // LLM classification helper (advisory, never throws)
-  async function getLlmNote() {
-    try {
-      const cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
-      const llmConfig = loadLocalLlmConfig(cwd);
-      const errorText = (data.tool_output || '').substring(0, 500);
-      if (!errorText) return '';
-      const llmResult = await classifyError(llmConfig, planningDir, errorText, agentType, data.session_id);
-      if (llmResult && llmResult.category) {
-        return `\nLLM error category: ${llmResult.category} (confidence: ${(llmResult.confidence * 100).toFixed(0)}%)`;
-      }
-    } catch (_e) { /* never propagate */ }
-    return '';
-  }
-
   if (genericMissing && skillWarnings.length > 0) {
     logHook('check-subagent-output', 'PostToolUse', 'skill-warning', { skill: activeSkill, agent_type: agentType, warnings: skillWarnings });
-    const llmCategoryNote = await getLlmNote();
     const msg = `Warning: Agent ${agentType} completed but no ${outputSpec.description} was found.\nSkill-specific warnings:\n` +
-      skillWarnings.map(w => `- ${w}`).join('\n') + llmCategoryNote;
+      skillWarnings.map(w => `- ${w}`).join('\n');
     return { additionalContext: msg };
   } else if (genericMissing) {
     logHook('check-subagent-output', 'PostToolUse', 'warning', { agent_type: agentType, expected: outputSpec.description, found: 'none' });
-    const llmCategoryNote = await getLlmNote();
     return {
-      additionalContext: `[WARN] Agent ${agentType} completed but no ${outputSpec.description} was found. Likely causes: (1) agent hit an error mid-run, (2) wrong working directory. To fix: re-run the parent skill — the executor gate will block until the output is present. Check the Task() output above for error details.` + llmCategoryNote
+      additionalContext: `[WARN] Agent ${agentType} completed but no ${outputSpec.description} was found. Likely causes: (1) agent hit an error mid-run, (2) wrong working directory. To fix: re-run the parent skill — the executor gate will block until the output is present. Check the Task() output above for error details.`
     };
   } else if (skillWarnings.length > 0) {
     logHook('check-subagent-output', 'PostToolUse', 'skill-warning', { skill: activeSkill, agent_type: agentType, warnings: skillWarnings });

package/plugins/pbr/scripts/check-summary-gate.js

@@ -27,24 +27,13 @@
 const fs = require('fs');
 const path = require('path');
 const { logHook } = require('./hook-logger');
+const { extractFrontmatter } = require('./lib/frontmatter');
 
 // Statuses that indicate a phase has been executed
 const ADVANCED_STATUSES = ['built', 'verified', 'complete'];
 
-/**
- * Extract YAML frontmatter values from markdown content.
- * Returns an object with parsed key-value pairs.
- */
-function parseFrontmatter(content) {
-  const match = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
-  if (!match) return {};
-  const result = {};
-  for (const line of match[1].split(/\r?\n/)) {
-    const kv = line.match(/^(\w[\w_]*):\s*"?([^"\r\n]*)"?$/);
-    if (kv) result[kv[1]] = kv[2].trim();
-  }
-  return result;
-}
+// Re-export extractFrontmatter as parseFrontmatter for backward compat (tests import it)
+const parseFrontmatter = extractFrontmatter;
 
 /**
  * Check if a SUMMARY file exists for the given phase directory.

package/plugins/pbr/scripts/config-schema.json

@@ -288,7 +288,7 @@
         },
         "reverse_spec": {
           "type": "boolean",
-          "default": true,
+          "default": false,
           "description": "Enable reverse spec generation from existing code."
         },
         "predictive_impact": {
@@ -330,6 +330,21 @@
           "type": "boolean",
           "default": true,
           "description": "Enable architecture consistency guard that warns on dependency violations."
+        },
+        "cross_project_patterns": {
+          "type": "boolean",
+          "default": false,
+          "description": "Enable cross-project pattern detection and reuse across repositories."
+        },
+        "spec_templates": {
+          "type": "boolean",
+          "default": false,
+          "description": "Enable spec template generation from existing code patterns."
+        },
+        "global_learnings": {
+          "type": "boolean",
+          "default": false,
+          "description": "Enable global learnings aggregation across phases into KNOWLEDGE.md."
         }
       },
       "additionalProperties": false
@@ -1077,134 +1092,6 @@
       },
       "additionalProperties": false
     },
-    "local_llm": {
-      "type": "object",
-      "deprecated": true,
-      "description": "DEPRECATED. Local LLM infrastructure removed in v14.0. Key retained for backward compatibility.",
-      "properties": {
-        "enabled": {
-          "type": "boolean"
-        },
-        "provider": {
-          "type": "string",
-          "enum": [
-            "ollama"
-          ]
-        },
-        "endpoint": {
-          "type": "string",
-          "format": "uri"
-        },
-        "model": {
-          "type": "string"
-        },
-        "timeout_ms": {
-          "type": "integer",
-          "minimum": 500
-        },
-        "max_retries": {
-          "type": "integer",
-          "minimum": 0,
-          "maximum": 3
-        },
-        "fallback": {
-          "type": "string",
-          "enum": [
-            "frontier",
-            "skip"
-          ]
-        },
-        "routing_strategy": {
-          "type": "string",
-          "enum": [
-            "local_first",
-            "frontier_first"
-          ]
-        },
-        "features": {
-          "type": "object",
-          "properties": {
-            "artifact_classification": {
-              "type": "boolean"
-            },
-            "task_validation": {
-              "type": "boolean"
-            },
-            "plan_adequacy": {
-              "type": "boolean"
-            },
-            "gap_detection": {
-              "type": "boolean"
-            },
-            "context_summarization": {
-              "type": "boolean"
-            },
-            "source_scoring": {
-              "type": "boolean"
-            },
-            "commit_classification": {
-              "type": "boolean"
-            },
-            "test_triage": {
-              "type": "boolean"
-            },
-            "file_intent_classification": {
-              "type": "boolean"
-            }
-          },
-          "additionalProperties": false
-        },
-        "metrics": {
-          "type": "object",
-          "properties": {
-            "enabled": {
-              "type": "boolean"
-            },
-            "log_file": {
-              "type": "string"
-            },
-            "show_session_summary": {
-              "type": "boolean"
-            },
-            "frontier_token_rate": {
-              "type": "number",
-              "minimum": 0
-            }
-          },
-          "additionalProperties": false
-        },
-        "advanced": {
-          "type": "object",
-          "properties": {
-            "confidence_threshold": {
-              "type": "number",
-              "minimum": 0,
-              "maximum": 1
-            },
-            "max_input_tokens": {
-              "type": "integer",
-              "minimum": 100
-            },
-            "keep_alive": {
-              "type": "string"
-            },
-            "num_ctx": {
-              "type": "integer",
-              "minimum": 512
-            },
-            "disable_after_failures": {
-              "type": "integer",
-              "minimum": 1
-            },
-            "shadow_mode": {
-              "type": "boolean"
-            }
-          },
-          "additionalProperties": false
-        }
-      },
-      "additionalProperties": false
-    },
     "intel": {
       "type": "object",
       "properties": {

package/plugins/pbr/scripts/feedback-loop.js

@@ -20,36 +20,19 @@
 
 const fs = require('fs');
 const path = require('path');
+const { extractFrontmatter } = require('./lib/frontmatter');
 
 /**
- * Parse YAML-ish frontmatter from markdown content.
- * Lightweight parser — no dependency on pbr-tools to avoid circular deps.
+ * Parse frontmatter and body from markdown content.
+ * Delegates YAML parsing to canonical extractFrontmatter.
  * @param {string} content - Full file content
  * @returns {{ frontmatter: object, body: string }}
  */
-function parseFrontmatter(content) {
-  const match = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
-  if (!match) return { frontmatter: {}, body: content };
-
-  const fm = {};
-  const lines = match[1].split(/\r?\n/);
-  for (const line of lines) {
-    const kvMatch = line.match(/^(\w[\w_]*):\s*(.+)/);
-    if (kvMatch) {
-      const key = kvMatch[1];
-      let val = kvMatch[2].trim();
-      // Parse numbers
-      if (/^\d+$/.test(val)) val = parseInt(val, 10);
-      // Strip quotes
-      if (typeof val === 'string' && val.startsWith('"') && val.endsWith('"')) {
-        val = val.slice(1, -1);
-      }
-      fm[key] = val;
-    }
-  }
-
-  const body = content.slice(match[0].length).trim();
-  return { frontmatter: fm, body };
+function extractFrontmatterWithBody(content) {
+  const frontmatter = extractFrontmatter(content);
+  const match = content.match(/^---\r?\n[\s\S]*?\r?\n---/);
+  const body = match ? content.slice(match[0].length).trim() : content;
+  return { frontmatter, body };
 }
 
 /**
@@ -69,7 +52,7 @@ function extractFeedback(phaseDir) {
     return null;
   }
 
-  const { frontmatter, body } = parseFrontmatter(content);
+  const { frontmatter, body } = extractFrontmatterWithBody(content);
 
   const status = frontmatter.status;
   if (!status) return null;
@@ -77,9 +60,9 @@ function extractFeedback(phaseDir) {
   // No feedback needed if verification passed
   if (status === 'passed' || status === 'all_passed') return null;
 
-  const attempt = typeof frontmatter.attempt === 'number' ? frontmatter.attempt : 1;
-  const passed = typeof frontmatter.must_haves_passed === 'number' ? frontmatter.must_haves_passed : 0;
-  const total = typeof frontmatter.must_haves_total === 'number' ? frontmatter.must_haves_total : 1;
+  const attempt = frontmatter.attempt ? parseInt(frontmatter.attempt, 10) || 1 : 1;
+  const passed = frontmatter.must_haves_passed ? parseInt(frontmatter.must_haves_passed, 10) || 0 : 0;
+  const total = frontmatter.must_haves_total ? parseInt(frontmatter.must_haves_total, 10) || 1 : 1;
   const pass_rate = total > 0 ? passed / total : 0;
 
   // Parse gap sections from body

package/plugins/pbr/scripts/hook-server.js

@@ -89,15 +89,23 @@ function readEventLogTail(logFile, maxLines) {
   if (maxLines === undefined) maxLines = 500;
   try {
     if (!fs.existsSync(logFile)) return [];
-    const content = fs.readFileSync(logFile, 'utf8');
-    const lines = content.split('\n').filter(l => l.trim().length > 0);
+    const stat = fs.statSync(logFile);
+    if (stat.size === 0) return [];
+    // Read only the last chunk instead of the entire file
+    const CHUNK_SIZE = 64 * 1024; // 64KB should hold 500+ lines
+    const start = Math.max(0, stat.size - CHUNK_SIZE);
+    const fd = fs.openSync(logFile, 'r');
+    const buffer = Buffer.alloc(Math.min(CHUNK_SIZE, stat.size));
+    fs.readSync(fd, buffer, 0, buffer.length, start);
+    fs.closeSync(fd);
+    const lines = buffer.toString('utf8').split('\n').filter(l => l.trim().length > 0);
     const tail = lines.slice(-maxLines);
     const events = [];
     for (const line of tail) {
       try {
         events.push(JSON.parse(line));
       } catch (_e) {
-        // Skip malformed lines
+        // Skip malformed lines (including partial first line from mid-file read)
       }
     }
     return events;
@@ -106,6 +114,23 @@ function readEventLogTail(logFile, maxLines) {
   }
 }
 
+/**
+ * Rotate the event log if it exceeds 2MB — truncate to last 1000 lines.
+ * Called once at server startup to prevent unbounded log growth.
+ */
+function rotateEventLog(planningDir) {
+  if (!planningDir) return;
+  const logFile = path.join(planningDir, '.hook-events.jsonl');
+  try {
+    const stat = fs.statSync(logFile);
+    if (stat.size > 2 * 1024 * 1024) { // 2MB
+      const content = fs.readFileSync(logFile, 'utf8');
+      const lines = content.split('\n').filter(l => l.trim().length > 0);
+      fs.writeFileSync(logFile, lines.slice(-1000).join('\n') + '\n');
+    }
+  } catch (_e) { /* best-effort — file may not exist yet */ }
+}
+
 /** Append a JSON event object as a single line to .planning/.hook-events.jsonl */
 function appendEvent(planningDir, eventObj) {
   if (!planningDir) return;
@@ -176,6 +201,30 @@ function mergeContext(...fns) {
   };
 }
 
+// ---------------------------------------------------------------------------
+// PreToolUse response translation
+// ---------------------------------------------------------------------------
+
+/**
+ * Translate PreToolUse block responses into Claude Code's hookSpecificOutput format.
+ * Handlers return { decision: 'block', reason: '...' } but Claude Code expects
+ * { hookSpecificOutput: { hookEventName, permissionDecision, permissionDecisionReason } }.
+ */
+function translatePreToolUseResponse(event, result) {
+  if (!result || event !== 'PreToolUse') return result;
+  if (result.decision === 'block' && result.reason) {
+    return {
+      hookSpecificOutput: {
+        hookEventName: 'PreToolUse',
+        permissionDecision: 'deny',
+        permissionDecisionReason: result.reason
+      }
+    };
+  }
+  // Allow decisions or non-blocking results pass through unchanged
+  return result;
+}
+
 // ---------------------------------------------------------------------------
 // Handler routing table (dynamic Map populated by register/initRoutes)
 // ---------------------------------------------------------------------------
@@ -377,7 +426,7 @@ function createServer(planningDir) {
       }
       const duration_ms = Date.now() - dispatchStart;
       appendEvent(planningDir, { ts: new Date().toISOString(), type: 'dispatch', event, tool, hook: `${event}:${tool}`, duration_ms, transport: 'http' });
-      let finalResult = dispatchResult || {};
+      let finalResult = translatePreToolUseResponse(event, dispatchResult) || {};
       if (duration_ms >= 100) {
         const alertMsg = `HOOK PERFORMANCE ALERT: ${event}:${tool} took ${duration_ms}ms (threshold: 100ms). Check handler for blocking I/O.`;
         if (finalResult.additionalContext) {
@@ -431,7 +480,7 @@ function createServer(planningDir) {
       }
       const legacyDurationMs = Date.now() - legacyDispatchStart;
       appendEvent(planningDir, { ts: new Date().toISOString(), type: 'dispatch', event, tool, hook: `${event}:${tool}`, duration_ms: legacyDurationMs, transport: 'http' });
-      let legacyFinalResult = legacyDispatchResult || {};
+      let legacyFinalResult = translatePreToolUseResponse(event, legacyDispatchResult) || {};
       if (legacyDurationMs >= 100) {
         const alertMsg = `HOOK PERFORMANCE ALERT: ${event}:${tool} took ${legacyDurationMs}ms (threshold: 100ms). Check handler for blocking I/O.`;
         if (legacyFinalResult.additionalContext) {
@@ -573,6 +622,9 @@ function main() {
     cache.config = configLoad(planningDir);
   } catch (_e) { /* best-effort */ }
 
+  // Rotate event log if oversized (before any new events are appended)
+  rotateEventLog(planningDir);
+
   initRoutes();
 
   const server = createServer(planningDir);
@@ -601,6 +653,6 @@ function main() {
   process.on('SIGINT', shutdown);
 }
 
-module.exports = { createServer, appendEvent, readEventLogTail, mergeContext, lazyHandler, resolveHandler, register, initRoutes, triggerShutdown, tryNextPort, normalizeMsysPath, DEFAULT_PORT };
+module.exports = { createServer, appendEvent, readEventLogTail, rotateEventLog, mergeContext, lazyHandler, resolveHandler, register, initRoutes, triggerShutdown, tryNextPort, normalizeMsysPath, translatePreToolUseResponse, DEFAULT_PORT };
 
 if (require.main === module || process.argv[1] === __filename) { main(); }

package/plugins/pbr/scripts/lib/config.js

@@ -116,11 +116,6 @@ function configValidate(configOrDir, planningDir) {
     warnings.push(`config.json schema_version (${config.schema_version}) is behind current (${CURRENT_SCHEMA_VERSION}) — run "pbr-tools migrate" to update`);
   }
 
-  // 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.');
-  }
-
   // Semantic conflict detection
   if (config.mode === 'autonomous' && config.gates) {
     const activeGates = Object.entries(config.gates || {}).filter(([, v]) => v === true).map(([k]) => k);
@@ -290,7 +285,7 @@ function configLoadDefaults(planningDir) {
   // No config found — return hardcoded defaults
   return {
     version: 2,
-    schema_version: 1,
+    schema_version: 3,
     mode: 'interactive',
     depth: 'standard',
     features: {
@@ -300,14 +295,14 @@ function configLoadDefaults(planningDir) {
       plan_checking: true,
     },
     planning: {
-      commit_docs: true,
+      commit_docs: false,
       search_gitignored: false,
     },
     git: {
       branching: 'none',
     },
     parallelization: {
-      enabled: true,
+      enabled: false,
     },
   };
 }
@@ -903,7 +898,6 @@ const CONFIG_DEFAULTS = {
     max_phases_in_context: 3
   },
   hook_server: { enabled: false, port: 19836, event_log: true },
-  local_llm: { enabled: false }, // DEPRECATED: local_llm infrastructure removed in v14.0. Key retained for backward compat.
   intel: { enabled: false, auto_update: false, inject_on_start: false },
   context_ledger: { enabled: false, stale_after_minutes: 60 },
   learnings: { enabled: false, read_depth: 3, cross_project_knowledge: false },
@@ -1153,7 +1147,6 @@ const CONFIG_SECTIONS = [
       'debug.max_hypothesis_rounds: 1-20 — max hypothesis cycles for /pbr:debug',
       'depth_profiles: override built-in quick/standard/comprehensive defaults',
       'developer_profile: behavioral profiling from session history + prompt injection',
-      '(DEPRECATED) local_llm: offload classification tasks to local Ollama instance — removed in v14.0',
       'prd.auto_extract: skip confirmation gate during PRD import',
       'spinner_tips: custom messages shown during agent execution',
       'status_line: status bar appearance (sections, branding, context bar)',
@@ -1161,7 +1154,7 @@ const CONFIG_SECTIONS = [
       'ui.enabled: enable UI design pipeline (/pbr:ui-phase, /pbr:ui-review)',
       'worktree.sparse_paths: glob patterns for sparse checkout in agent worktrees'
     ],
-    keys: ['dashboard', 'debug', 'depth_profiles', 'developer_profile', 'local_llm', 'prd', 'spinner_tips', 'status_line', 'timeouts', 'ui', 'worktree']
+    keys: ['dashboard', 'debug', 'depth_profiles', 'developer_profile', 'prd', 'spinner_tips', 'status_line', 'timeouts', 'ui', 'worktree']
   }
 ];
 

package/plugins/pbr/scripts/lib/contextual-help.js

@@ -12,6 +12,7 @@
 
 const fs = require('fs');
 const path = require('path');
+const { extractFrontmatter } = require('./frontmatter');
 
 // ─── Help templates ────────────────────────────────────────────────────────────
 
@@ -79,39 +80,14 @@ const HELP_TEMPLATES = {
  * @param {string} statePath - Path to STATE.md
  * @returns {{ status: string, blockers: string[] }}
  */
-function parseStateFrontmatter(statePath) {
+const parseStateFrontmatter = (statePath) => {
   const defaults = { status: 'unknown', blockers: [] };
   try {
     if (!fs.existsSync(statePath)) return defaults;
     const content = fs.readFileSync(statePath, 'utf8');
-    // Extract frontmatter between --- delimiters
-    const fmMatch = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
-    if (!fmMatch) return defaults;
-    const fm = fmMatch[1];
-
-    // Parse status
-    const statusMatch = fm.match(/^status:\s*["']?([^"'\r\n]+)["']?/m);
-    const status = statusMatch ? statusMatch[1].trim() : 'unknown';
-
-    // Parse blockers array
-    const blockers = [];
-    const blockersMatch = fm.match(/^blockers:\s*\[(.*?)\]/ms);
-    if (blockersMatch) {
-      // Inline array format: blockers: ["item1", "item2"]
-      const items = blockersMatch[1].match(/"([^"]+)"/g) || [];
-      blockers.push(...items.map(i => i.replace(/"/g, '')));
-    } else {
-      // Multi-line array format
-      const blockersSection = fm.match(/^blockers:\s*\r?\n((?:\s+- .*\r?\n?)*)/m);
-      if (blockersSection) {
-        const lines = blockersSection[1].split(/\r?\n/);
-        for (const line of lines) {
-          const item = line.match(/^\s+- ["']?(.+?)["']?\s*$/);
-          if (item) blockers.push(item[1]);
-        }
-      }
-    }
-
+    const fm = extractFrontmatter(content);
+    const status = fm.status || 'unknown';
+    const blockers = Array.isArray(fm.blockers) ? fm.blockers : [];
     return { status, blockers };
   } catch (_e) {
     return defaults;

package/plugins/pbr/scripts/lib/format-validators.js

@@ -372,31 +372,6 @@ async function checkPlanWrite(data) {
                 ? validateContext(content, filePath)
                 : validateSummary(content, filePath);
 
-  // LLM advisory enrichment -- advisory only, never blocks
-  if ((isPlan || isSummary) && result.errors.length === 0) {
-    try {
-      const { resolveConfig } = require('../local-llm/health');
-      const { classifyArtifact } = require('../local-llm/operations/classify-artifact');
-      let llmConfig;
-      try {
-        const configPath = path.join(process.cwd(), '.planning', 'config.json');
-        const parsed = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-        llmConfig = resolveConfig(parsed.local_llm);
-      } catch (_e) {
-        llmConfig = resolveConfig(undefined);
-      }
-      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';
 
   if (result.errors.length > 0) {
@@ -934,7 +909,7 @@ function validateConfig(content, _filePath) {
     }
   }
 
-  const knownKeys = ['version', 'schema_version', 'context_strategy', 'mode', 'depth', 'session_phase_limit', 'session_cycling', 'context_window_tokens', 'agent_checkpoint_pct', 'features', 'validation_passes', 'autonomy', 'models', 'model_profiles', 'parallelization', 'teams', 'planning', 'git', 'gates', 'safety', 'timeouts', 'hooks', 'prd', 'depth_profiles', 'debug', 'developer_profile', 'spinner_tips', 'dashboard', 'status_line', 'workflow', 'hook_server', 'local_llm', 'intel', 'context_ledger', 'learnings', 'verification', 'context_budget', 'ui', 'worktree', 'ceremony_level', 'skip_rag_max_lines', 'orchestrator_budget_pct'];
+  const knownKeys = ['version', 'schema_version', 'context_strategy', 'mode', 'depth', 'session_phase_limit', 'session_cycling', 'context_window_tokens', 'agent_checkpoint_pct', 'features', 'autonomy', 'models', 'model_profiles', 'parallelization', 'teams', 'planning', 'git', 'gates', 'safety', 'timeouts', 'hooks', 'prd', 'depth_profiles', 'debug', 'developer_profile', 'spinner_tips', 'dashboard', 'status_line', 'workflow', 'hook_server', 'intel', 'context_ledger', 'learnings', 'verification', 'context_budget', 'ui', 'worktree', 'ceremony_level', 'skip_rag_max_lines', 'orchestrator_budget_pct'];
   for (const key of Object.keys(parsed)) {
     if (!knownKeys.includes(key)) {
       warnings.push(`Unknown top-level key: "${key}" (known: ${knownKeys.join(', ')})`);