@sienklogic/plan-build-run 2.19.1 → 2.19.2

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

@@ -84,119 +84,30 @@ function parseStateMd(content) {
     progress: null,
     status: null,
     line_count: normalized.split('\n').length,
-    format: 'legacy' // 'legacy' or 'frontmatter'
+    format: 'frontmatter'
   };
 
   // Check for YAML frontmatter (version 2 format)
   const frontmatter = parseYamlFrontmatter(normalized);
-  if (frontmatter.version === 2 || frontmatter.current_phase !== undefined) {
-    result.format = 'frontmatter';
-    result.current_phase = frontmatter.current_phase || 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 || [];
-    result.velocity = frontmatter.velocity || null;
-    result.session_last = frontmatter.session_last || null;
-    result.session_stopped_at = frontmatter.session_stopped_at || null;
-    result.session_resume = frontmatter.session_resume || null;
+  result.format = 'frontmatter';
+  result.current_phase = frontmatter.current_phase || 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 || [];
+  result.velocity = frontmatter.velocity || null;
+  result.session_last = frontmatter.session_last || null;
+  result.session_stopped_at = frontmatter.session_stopped_at || null;
+  result.session_resume = frontmatter.session_resume || null;
     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 = normalized.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 = normalized.match(/--\s+(.+?)(?:\n|$)/);
-  if (nameMatch) {
-    result.phase_name = nameMatch[1].trim();
-  }
-
-  // Extract progress percentage
-  const progressMatch = normalized.match(/(\d+)%/);
-  if (progressMatch) {
-    result.progress = parseInt(progressMatch[1], 10);
-  }
-
-  // Extract plan status
-  const statusMatch = normalized.match(/Status:\s*(.+?)(?:\n|$)/i);
-  if (statusMatch) {
-    result.status = statusMatch[1].trim();
-  }
-
-  return result;
 }
 
 // --- 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.replace(/\r\n/g, '\n').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.
@@ -428,7 +339,7 @@ function stateCheckProgress(planningDir) {
 
 /**
  * Atomically update a field in STATE.md using lockedFileUpdate.
- * Supports both legacy and frontmatter (v2) formats.
+ * Updates frontmatter (v2) format STATE.md.
  *
  * @param {string} field - One of: current_phase, status, plans_complete, last_activity,
  *   progress_percent, phase_slug, total_phases, last_command, blockers
@@ -468,13 +379,9 @@ function stateUpdate(field, value, planningDir) {
   }
 
   const result = lockedFileUpdate(statePath, (content) => {
-    const fm = parseYamlFrontmatter(content);
-    if (fm.version === 2 || fm.current_phase !== undefined) {
-      let updated = updateFrontmatterField(content, field, value);
-      updated = syncBodyLine(updated, field, value);
-      return updated;
-    }
-    return updateLegacyStateField(content, field, value);
+    let updated = updateFrontmatterField(content, field, value);
+    updated = syncBodyLine(updated, field, value);
+    return updated;
   });
 
   if (result.success) {
@@ -501,19 +408,13 @@ function statePatch(jsonStr, planningDir) {
 
   const result = lockedFileUpdate(statePath, (content) => {
     let updated = content;
-    const fm = parseYamlFrontmatter(updated);
-    const isFrontmatter = fm.version === 2 || fm.current_phase !== undefined;
     for (const [field, value] of Object.entries(fields)) {
       let val = String(value);
       if (field === 'last_activity' && val === 'now') {
         val = new Date().toISOString().slice(0, 19).replace('T', ' ');
       }
-      if (isFrontmatter) {
-        updated = updateFrontmatterField(updated, field, val);
-        updated = syncBodyLine(updated, field, val);
-      } else {
-        updated = updateLegacyStateField(updated, field, val);
-      }
+      updated = updateFrontmatterField(updated, field, val);
+      updated = syncBodyLine(updated, field, val);
     }
     return updated;
   });
@@ -546,15 +447,10 @@ function stateAdvancePlan(planningDir) {
     resultData = { previous_plan: current, current_plan: next, total_plans: total, progress_percent: progressPct };
 
     let updated = content;
-    const fm = parseYamlFrontmatter(content);
-    if (fm.version === 2 || fm.current_phase !== undefined) {
-      updated = updateFrontmatterField(updated, 'plans_complete', String(next));
-      updated = syncBodyLine(updated, 'plans_complete', String(next));
-      updated = updateFrontmatterField(updated, 'progress_percent', String(progressPct));
-      updated = syncBodyLine(updated, 'progress_percent', String(progressPct));
-    } else {
-      updated = updateLegacyStateField(updated, 'plans_complete', String(next));
-    }
+    updated = updateFrontmatterField(updated, 'plans_complete', String(next));
+    updated = syncBodyLine(updated, 'plans_complete', String(next));
+    updated = updateFrontmatterField(updated, 'progress_percent', String(progressPct));
+    updated = syncBodyLine(updated, 'progress_percent', String(progressPct));
     return updated;
   });
 
@@ -1158,7 +1054,6 @@ function stateBackup(planningDir) {
 
 module.exports = {
   parseStateMd,
-  updateLegacyStateField,
   updateFrontmatterField,
   syncBodyLine,
   buildProgressBar,

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

@@ -183,7 +183,7 @@ function cmdVerifyPhaseCompleteness(cwd, phase, raw) {
 
   // List plans and summaries
   let files;
-  try { files = fs.readdirSync(phaseDir); } catch { output({ error: 'Cannot read phase directory' }, raw); return; }
+  try { files = fs.readdirSync(phaseDir); } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Cannot read phase directory ${phaseDir}: ${err.message}`); output({ error: 'Cannot read phase directory' }, raw); return; }
 
   const plans = files.filter(f => f.match(/-PLAN\.md$/i));
   const summaries = files.filter(f => f.match(/-SUMMARY\.md$/i));
@@ -409,7 +409,8 @@ function cmdVerifyKeyLinks(cwd, planFilePath, raw) {
             check.detail = `Pattern "${link.pattern}" not found in source or target`;
           }
         }
-      } catch {
+      } catch (err) {
+        if (process.env.PBR_DEBUG) console.error(`[verify] Invalid regex pattern ${link.pattern}: ${err.message}`);
         check.detail = `Invalid regex pattern: ${link.pattern}`;
       }
     } else {
@@ -466,7 +467,7 @@ function cmdValidateConsistency(cwd, raw) {
       const dm = dir.match(/^(\d+[A-Z]?(?:\.\d+)*)/i);
       if (dm) diskPhases.add(dm[1]);
     }
-  } catch {}
+  } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to read phases directory: ${err.message}`); }
 
   // Check: phases in ROADMAP but not on disk
   for (const p of roadmapPhases) {
@@ -528,7 +529,7 @@ function cmdValidateConsistency(cwd, raw) {
         }
       }
     }
-  } catch {}
+  } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to check plan numbering: ${err.message}`); }
 
   // Check: frontmatter in plans has required fields
   try {
@@ -548,7 +549,7 @@ function cmdValidateConsistency(cwd, raw) {
         }
       }
     }
-  } catch {}
+  } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to check plan frontmatter: ${err.message}`); }
 
   const passed = errors.length === 0;
   output({ passed, errors, warnings, warning_count: warnings.length }, raw, passed ? 'passed' : 'failed');
@@ -678,7 +679,7 @@ function cmdValidateHealth(cwd, options, raw) {
           if (m) diskPhases.add(m[1]);
         }
       }
-    } catch {}
+    } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to read phases for state validation: ${err.message}`); }
     // Check for invalid references
     for (const ref of phaseRefs) {
       const normalizedRef = String(parseInt(ref, 10)).padStart(2, '0');
@@ -720,7 +721,7 @@ function cmdValidateHealth(cwd, options, raw) {
         addIssue('warning', 'W008', 'config.json: workflow.nyquist_validation absent (defaults to enabled but agents may skip)', 'Run /pbr:health --repair to add key', true);
         if (!repairs.includes('addNyquistKey')) repairs.push('addNyquistKey');
       }
-    } catch {}
+    } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to check nyquist key: ${err.message}`); }
   }
 
   // ─── Check 6: Phase directory naming (NN-name format) ─────────────────────
@@ -731,7 +732,7 @@ function cmdValidateHealth(cwd, options, raw) {
         addIssue('warning', 'W005', `Phase directory "${e.name}" doesn't follow NN-name format`, 'Rename to match pattern (e.g., 01-setup)');
       }
     }
-  } catch {}
+  } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to check phase directory naming: ${err.message}`); }
 
   // ─── Check 7: Orphaned plans (PLAN without SUMMARY) ───────────────────────
   try {
@@ -750,7 +751,7 @@ function cmdValidateHealth(cwd, options, raw) {
         }
       }
     }
-  } catch {}
+  } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to check orphaned plans: ${err.message}`); }
 
   // ─── Check 7b: Nyquist VALIDATION.md consistency ────────────────────────
   try {
@@ -768,7 +769,7 @@ function cmdValidateHealth(cwd, options, raw) {
         }
       }
     }
-  } catch {}
+  } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to check nyquist validation: ${err.message}`); }
 
   // ─── Check 8: Run existing consistency checks ─────────────────────────────
   // Inline subset of cmdValidateConsistency
@@ -790,7 +791,7 @@ function cmdValidateHealth(cwd, options, raw) {
           if (dm) diskPhases.add(dm[1]);
         }
       }
-    } catch {}
+    } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to read phases for roadmap check: ${err.message}`); }
 
     // Phases in ROADMAP but not on disk
     for (const p of roadmapPhases) {
@@ -909,20 +910,20 @@ function cmdValidateHealth(cwd, options, raw) {
       } else {
         feature_status.multi_phase_awareness = { enabled: false, status: 'disabled' };
       }
-    } catch (_e) { /* config parse errors handled in Check 5 */ }
+    } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Config parse error (handled in Check 5): ${err.message}`); }
   }
 
   // ─── Check 11: Phase 05 feature status ────────────────────────────────────
   {
     let p05Config = {};
-    try { p05Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (_) {}
+    try { p05Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse config for phase05: ${err.message}`); }
     var phase05_features = checkPhase05Features(planningDir, p05Config);
   }
 
   // ─── Check 12: Trust tracking health ──────────────────────────────────────
   {
     let config = {};
-    try { config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (_) {}
+    try { config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse config for trust: ${err.message}`); }
 
     if (config.features && config.features.trust_tracking === false) {
       addIssue('info', 'I-TRUST-DISABLED', 'trust_tracking is disabled in config', 'Enable features.trust_tracking in config.json if desired');
@@ -941,7 +942,8 @@ function cmdValidateHealth(cwd, options, raw) {
             }
           }
           addIssue('info', 'I-TRUST-HEALTHY', `trust_tracking: healthy — ${agents.length} agents, ${totalOutcomes} outcomes tracked`, '');
-        } catch (_) {
+        } catch (err) {
+          if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse agent-scores.json: ${err.message}`);
           addIssue('info', 'I-TRUST-DEGRADED', 'trust_tracking: degraded — agent-scores.json exists but is malformed', 'Delete .planning/trust/agent-scores.json to reset');
         }
       }
@@ -957,7 +959,8 @@ function cmdValidateHealth(cwd, options, raw) {
         try {
           JSON.parse(fs.readFileSync(trustFile, 'utf-8'));
           addIssue('info', 'I-CONFIDENCE-HEALTHY', 'confidence_calibration: healthy — trust data available for calibration', '');
-        } catch (_) {
+        } catch (err) {
+          if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse trust data for confidence: ${err.message}`);
           addIssue('info', 'I-CONFIDENCE-DEGRADED', 'confidence_calibration: degraded — trust data malformed', 'Delete .planning/trust/agent-scores.json to reset');
         }
       }
@@ -971,12 +974,12 @@ function cmdValidateHealth(cwd, options, raw) {
     const guardHealth = graph.guardHealthCheck(planningDir);
     feature_status.architecture_graph = graphHealth;
     feature_status.architecture_guard = guardHealth;
-  } catch (_e) { /* graph module not available — skip */ }
+  } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Graph module not available: ${err.message}`); }
 
   // ─── Check 14: Phase 15 DX feature health ────────────────────────────────
   {
     let p15Config = {};
-    try { p15Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (_) {}
+    try { p15Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse config for phase15: ${err.message}`); }
     const p15Features = (p15Config && p15Config.features) || {};
 
     const checkDxFeature = (featureName, modulePath, exportName) => {
@@ -1004,17 +1007,17 @@ function cmdValidateHealth(cwd, options, raw) {
 
     feature_status.progress_visualization = checkDxFeature(
       'progress_visualization',
-      path.join(__dirname, 'progress-visualization.cjs'),
+      path.join(__dirname, 'progress-visualization.js'),
       'getProgressData'
     );
     feature_status.contextual_help = checkDxFeature(
       'contextual_help',
-      path.join(__dirname, 'contextual-help.cjs'),
+      path.join(__dirname, 'contextual-help.js'),
       'getContextualHelp'
     );
     feature_status.team_onboarding = checkDxFeature(
       'team_onboarding',
-      path.join(__dirname, 'onboarding-generator.cjs'),
+      path.join(__dirname, 'onboarding-generator.js'),
       'generateOnboardingGuide'
     );
   }
@@ -1022,7 +1025,7 @@ function cmdValidateHealth(cwd, options, raw) {
   // ─── Check 15: Phase 14 Quality & Safety feature health ───────────────────
   {
     let p14Config = {};
-    try { p14Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (_) {}
+    try { p14Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse config for phase14: ${err.message}`); }
 
     const p14Features = p14Config.features || {};
 
@@ -1040,7 +1043,8 @@ function cmdValidateHealth(cwd, options, raw) {
         }
         addIssue('warning', `W-${featureName.toUpperCase()}-DEGRADED`, `${featureName}: degraded (module validation failed)`, `Check ${modulePath} exports`);
         return { enabled: true, status: 'degraded' };
-      } catch (_e) {
+      } catch (err) {
+        if (process.env.PBR_DEBUG) console.error(`[verify] Feature ${featureName} module load failed: ${err.message}`);
         addIssue('warning', `W-${featureName.toUpperCase()}-DEGRADED`, `${featureName}: degraded (module load failed)`, `Ensure ${modulePath} exists and is valid`);
         return { enabled: true, status: 'degraded' };
       }
@@ -1048,7 +1052,7 @@ function cmdValidateHealth(cwd, options, raw) {
 
     // regression_prevention: default true
     const rpEnabled = p14Features.regression_prevention !== false;
-    const rpModPath = path.join(__dirname, 'test-selection.cjs');
+    const rpModPath = path.join(__dirname, 'test-selection.js');
     const rpHealth = checkFeatureHealth(
       'regression_prevention',
       rpEnabled,
@@ -1062,7 +1066,7 @@ function cmdValidateHealth(cwd, options, raw) {
 
     // security_scanning: default true
     const ssEnabled = p14Features.security_scanning !== false;
-    const ssModPath = path.join(__dirname, 'security-scan.cjs');
+    const ssModPath = path.join(__dirname, 'security-scan.js');
     const ssHealth = checkFeatureHealth(
       'security_scanning',
       ssEnabled,
@@ -1074,7 +1078,7 @@ function cmdValidateHealth(cwd, options, raw) {
         const ssMod = require(ssModPath);
         const ruleCount = ssMod.SECURITY_RULES.length;
         addIssue('info', 'I-SS-HEALTHY', `security_scanning: healthy (${ruleCount} rules loaded)`, '');
-      } catch (_) {}
+      } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to load security rules: ${err.message}`); }
     }
     feature_status.security_scanning = ssHealth;
   }
@@ -1082,7 +1086,7 @@ function cmdValidateHealth(cwd, options, raw) {
   // ─── Check 16: Phase 11 Spec-Driven Development feature health ───────────
   {
     let p11Config = {};
-    try { p11Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (_) {}
+    try { p11Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse config for phase11: ${err.message}`); }
     const p11Features = p11Config.features || {};
 
     const checkSpecFeatureHealth = (featureName, defaultEnabled, modulePath, exportName) => {
@@ -1096,29 +1100,30 @@ function cmdValidateHealth(cwd, options, raw) {
           return { status: 'healthy', enabled: true, details: `${exportName} loaded` };
         }
         return { status: 'degraded', enabled: true, details: `${exportName} not a function` };
-      } catch (_e) {
-        return { status: 'degraded', enabled: true, details: `Cannot load module: ${_e.message}` };
+      } catch (err) {
+        if (process.env.PBR_DEBUG) console.error(`[verify] Cannot load spec module: ${err.message}`);
+        return { status: 'degraded', enabled: true, details: `Cannot load module: ${err.message}` };
       }
     };
 
     feature_status.machine_executable_plans = checkSpecFeatureHealth(
-      'machine_executable_plans', false, path.join(__dirname, 'spec-engine.cjs'), 'parsePlanToSpec'
+      'machine_executable_plans', false, path.join(__dirname, 'spec-engine.js'), 'parsePlanToSpec'
     );
     feature_status.spec_diffing = checkSpecFeatureHealth(
-      'spec_diffing', true, path.join(__dirname, 'spec-diff.cjs'), 'diffSpecs'
+      'spec_diffing', true, path.join(__dirname, 'spec-diff.js'), 'diffSpecs'
     );
     feature_status.reverse_spec = checkSpecFeatureHealth(
-      'reverse_spec', true, path.join(__dirname, 'reverse-spec.cjs'), 'generateReverseSpec'
+      'reverse_spec', true, path.join(__dirname, 'reverse-spec.js'), 'generateReverseSpec'
     );
     feature_status.predictive_impact = checkSpecFeatureHealth(
-      'predictive_impact', true, path.join(__dirname, 'impact-analysis.cjs'), 'analyzeImpact'
+      'predictive_impact', true, path.join(__dirname, 'impact-analysis.js'), 'analyzeImpact'
     );
   }
 
   // ─── Check 17: Phase 16 cross-project intelligence feature health ────────
   {
     let p16Config = {};
-    try { p16Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (_) {}
+    try { p16Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse config for phase16: ${err.message}`); }
     const p16Features = p16Config.features || {};
 
     // Helper: check if a feature is enabled (default true unless explicitly false)
@@ -1138,7 +1143,7 @@ function cmdValidateHealth(cwd, options, raw) {
           const patternsDir = require('path').join(require('os').homedir(), '.claude', 'patterns');
           return fs.existsSync(patternsDir) &&
             fs.readdirSync(patternsDir).some(f => f.endsWith('.json'));
-        } catch (_) { return false; }
+        } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to check patterns dir: ${err.message}`); return false; }
       }
     );
 
@@ -1155,7 +1160,7 @@ function cmdValidateHealth(cwd, options, raw) {
         try {
           const learningsPath = require('path').join(require('os').homedir(), '.claude', 'learnings.jsonl');
           return fs.existsSync(learningsPath);
-        } catch (_) { return false; }
+        } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to check learnings path: ${err.message}`); return false; }
       }
     );
   }
@@ -1163,7 +1168,7 @@ function cmdValidateHealth(cwd, options, raw) {
   // ─── Check 18: Phase 3 (zero-friction) feature status ────────────────────
   {
     let p3Config = {};
-    try { p3Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (_) {}
+    try { p3Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse config for phase3: ${err.message}`); }
     const p3Features = p3Config.features || {};
 
     const zfqEnabled = p3Features.zero_friction_quick !== false;
@@ -1176,7 +1181,7 @@ function cmdValidateHealth(cwd, options, raw) {
   // ─── Check 19: Phase 4 (NL routing) feature status ───────────────────────
   {
     let p4Config = {};
-    try { p4Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (_) {}
+    try { p4Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse config for phase4: ${err.message}`); }
     const p4Features = p4Config.features || {};
     const pluginRoot = path.resolve(__dirname, '..', '..', '..', 'plugins', 'pbr');
 
@@ -1188,7 +1193,8 @@ function cmdValidateHealth(cwd, options, raw) {
       try {
         require(path.join(pluginRoot, 'scripts', 'lib', 'alternatives.js'));
         feature_status.natural_language_routing = { enabled: true, status: 'healthy' };
-      } catch (_e) {
+      } catch (err) {
+        if (process.env.PBR_DEBUG) console.error(`[verify] Failed to load alternatives.js: ${err.message}`);
         feature_status.natural_language_routing = { enabled: true, status: 'degraded' };
       }
     }
@@ -1204,7 +1210,7 @@ function cmdValidateHealth(cwd, options, raw) {
   // ─── Check 20: Phase 6 (convention memory) feature status ────────────────
   {
     let p6Config = {};
-    try { p6Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (_) {}
+    try { p6Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse config for phase6: ${err.message}`); }
     const p6Features = p6Config.features || {};
     const pluginRoot = path.resolve(__dirname, '..', '..', '..', 'plugins', 'pbr');
 
@@ -1216,7 +1222,8 @@ function cmdValidateHealth(cwd, options, raw) {
       try {
         require(path.join(pluginRoot, 'scripts', 'lib', 'convention-detector.js'));
         feature_status.convention_memory = { enabled: true, status: 'healthy' };
-      } catch (_e) {
+      } catch (err) {
+        if (process.env.PBR_DEBUG) console.error(`[verify] Failed to load convention-detector.js: ${err.message}`);
         feature_status.convention_memory = { enabled: true, status: 'degraded' };
       }
     }
@@ -1229,7 +1236,8 @@ function cmdValidateHealth(cwd, options, raw) {
       try {
         require(path.join(pluginRoot, 'scripts', 'lib', 'snapshot-manager.js'));
         feature_status.mental_model_snapshots = { enabled: true, status: 'healthy' };
-      } catch (_e) {
+      } catch (err) {
+        if (process.env.PBR_DEBUG) console.error(`[verify] Failed to load snapshot-manager.js: ${err.message}`);
         feature_status.mental_model_snapshots = { enabled: true, status: 'degraded' };
       }
     }
@@ -1238,7 +1246,7 @@ function cmdValidateHealth(cwd, options, raw) {
   // ─── Check 21: Phase 9 (proactive intelligence) feature status ───────────
   {
     let p9Config = {};
-    try { p9Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (_) {}
+    try { p9Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse config for phase9: ${err.message}`); }
     const scriptsDir = path.resolve(__dirname, '..', '..', '..', 'plugins', 'pbr', 'scripts');
     const { checkFeatureHealth: checkPhase9FeatureHealth } = require('./health');
     const phase9Features = [
@@ -1274,7 +1282,8 @@ function cmdValidateHealth(cwd, options, raw) {
         enabled: metricsResult.enabled,
         status: metricsResult.status,
       };
-    } catch (_e) {
+    } catch (err) {
+      if (process.env.PBR_DEBUG) console.error(`[verify] Failed to load health-checks.js: ${err.message}`);
       // health-checks.js not available — mark all as degraded
       feature_status.post_hoc_artifacts = { enabled: true, status: 'degraded' };
       feature_status.agent_feedback_loop = { enabled: true, status: 'degraded' };
@@ -1285,7 +1294,7 @@ function cmdValidateHealth(cwd, options, raw) {
   // ─── Check 23: Phase 13 (multi-agent) feature status ─────────────────────
   {
     let p13Config = {};
-    try { p13Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (_) {}
+    try { p13Config = JSON.parse(fs.readFileSync(configPath, 'utf-8')); } catch (err) { if (process.env.PBR_DEBUG) console.error(`[verify] Failed to parse config for phase13: ${err.message}`); }
     const { checkMultiAgentHealth } = require('./health');
     const multiAgentResults = checkMultiAgentHealth(p13Config);
     for (const result of multiAgentResults) {
@@ -1407,7 +1416,8 @@ function checkFeatureModuleHealth(featureName, planningDir, pluginRoot, modulePa
     if (fs.existsSync(configPath)) {
       config = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
     }
-  } catch (_e) {
+  } catch (err) {
+    if (process.env.PBR_DEBUG) console.error(`[verify] Config unreadable for feature health: ${err.message}`);
     // Config unreadable — treat as defaults
   }