@sienklogic/plan-build-run 2.11.0 → 2.12.0

package/CHANGELOG.md

@@ -5,6 +5,20 @@ All notable changes to Plan-Build-Run will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.12.0](https://github.com/SienkLogic/plan-build-run/compare/plan-build-run-v2.11.0...plan-build-run-v2.12.0) (2026-02-21)
+
+
+### Features
+
+* **tools:** add review verifier post-check and stale active-skill detection ([dbd2eb8](https://github.com/SienkLogic/plan-build-run/commit/dbd2eb899c2b15cc2ac61913b29ca1aaab6b0b1f))
+* **tools:** add scan mapper area validation and stale Building status detection ([8d8a438](https://github.com/SienkLogic/plan-build-run/commit/8d8a43809095722ae9d00242f90f29d32bef4d1f))
+
+
+### Bug Fixes
+
+* **tools:** extend executor commit check to quick skill and add .catch() to log-tool-failure ([197efc7](https://github.com/SienkLogic/plan-build-run/commit/197efc70cb5e2a36f014ee2c5c7d653b4b1898f4))
+* **tools:** warn on context budget tracker reset and roadmap sync parse failures ([f5aef28](https://github.com/SienkLogic/plan-build-run/commit/f5aef2804e42a934d3e7a480e3045ac6c0e0fc9b))
+
 ## [2.11.0](https://github.com/SienkLogic/plan-build-run/compare/plan-build-run-v2.10.0...plan-build-run-v2.11.0) (2026-02-21)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sienklogic/plan-build-run",
-  "version": "2.11.0",
+  "version": "2.12.0",
   "description": "Plan it, Build it, Run it — structured development workflow for Claude Code",
   "keywords": [
     "claude-code",

package/plugins/copilot-pbr/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "pbr",
   "displayName": "Plan-Build-Run",
-  "version": "2.11.0",
+  "version": "2.12.0",
   "description": "Plan-Build-Run — Structured development workflow for GitHub Copilot CLI. Solves context rot through disciplined agent delegation, structured planning, atomic execution, and goal-backward verification.",
   "author": {
     "name": "SienkLogic",

package/plugins/cursor-pbr/.cursor-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "pbr",
   "displayName": "Plan-Build-Run",
-  "version": "2.11.0",
+  "version": "2.12.0",
   "description": "Plan-Build-Run — Structured development workflow for Cursor. Solves context rot through disciplined subagent delegation, structured planning, atomic execution, and goal-backward verification.",
   "author": {
     "name": "SienkLogic",

package/plugins/pbr/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "pbr",
-  "version": "2.11.0",
+  "version": "2.12.0",
   "description": "Plan-Build-Run — Structured development workflow for Claude Code. Solves context rot through disciplined subagent delegation, structured planning, atomic execution, and goal-backward verification.",
   "author": {
     "name": "SienkLogic",

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

@@ -48,6 +48,10 @@ function main() {
       const stateInfo = parseState(stateContent);
       if (!stateInfo || !stateInfo.phase || !stateInfo.status) {
         logHook('check-roadmap-sync', 'PostToolUse', 'skip', { reason: 'could not parse STATE.md' });
+        const output = {
+          additionalContext: '[Roadmap Sync] Could not parse phase/status from STATE.md — ensure it contains **Phase**: and **Status**: fields.'
+        };
+        process.stdout.write(JSON.stringify(output));
         process.exit(0);
       }
 
@@ -220,7 +224,11 @@ function checkSync(data) {
   const stateInfo = parseState(stateContent);
   if (!stateInfo || !stateInfo.phase || !stateInfo.status) {
     logHook('check-roadmap-sync', 'PostToolUse', 'skip', { reason: 'could not parse STATE.md' });
-    return null;
+    return {
+      output: {
+        additionalContext: '[Roadmap Sync] Could not parse phase/status from STATE.md — ensure it contains **Phase**: and **Status**: fields.'
+      }
+    };
   }
 
   if (!LIFECYCLE_STATUSES.includes(stateInfo.status)) {

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

@@ -312,8 +312,38 @@ function main() {
     }
   }
 
-  // GAP-06: Build executor SUMMARY should have commits
-  if (activeSkill === 'build' && agentType === 'pbr:executor') {
+  // GAP-08: Scan codebase-mapper should produce all 4 focus areas
+  if (activeSkill === 'scan' && agentType === 'pbr:codebase-mapper') {
+    const expectedAreas = ['tech', 'arch', 'quality', 'concerns'];
+    const codebaseDir = path.join(planningDir, 'codebase');
+    if (fs.existsSync(codebaseDir)) {
+      try {
+        const files = fs.readdirSync(codebaseDir).map(f => f.toLowerCase());
+        for (const area of expectedAreas) {
+          if (!files.some(f => f.includes(area))) {
+            skillWarnings.push(`Scan mapper: No output file containing "${area}" found in .planning/codebase/. One of the 4 mappers may have failed.`);
+          }
+        }
+      } catch (_e) { /* best-effort */ }
+    }
+  }
+
+  // GAP-07: Review verifier should produce meaningful VERIFICATION.md status
+  if (activeSkill === 'review' && agentType === 'pbr:verifier') {
+    const verFiles = findInPhaseDir(planningDir, /^VERIFICATION\.md$/i);
+    for (const vf of verFiles) {
+      try {
+        const content = fs.readFileSync(vf, 'utf8');
+        const statusMatch = content.match(/^status:\s*(\S+)/mi);
+        if (statusMatch && statusMatch[1] === 'gaps_found') {
+          skillWarnings.push('Review verifier: VERIFICATION.md has status "gaps_found" — ensure gaps are surfaced to the user.');
+        }
+      } catch (_e) { /* best-effort */ }
+    }
+  }
+
+  // GAP-06: Build/quick executor SUMMARY should have commits
+  if ((activeSkill === 'build' || activeSkill === 'quick') && agentType === 'pbr:executor') {
     checkSummaryCommits(planningDir, found, skillWarnings);
   }
 

package/plugins/pbr/scripts/log-tool-failure.js

@@ -85,4 +85,4 @@ function summarizeInput(toolName, toolInput) {
   }
 }
 
-main();
+main().catch(() => {});

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

@@ -91,6 +91,20 @@ function buildContext(planningDir, stateFile) {
     if (continuity) {
       parts.push(`\nLast Session:\n${continuity}`);
     }
+
+    // Detect stale "Building" status — likely a crashed executor
+    const statusMatch = state.match(/\*{0,2}(?:Phase\s+)?Status\*{0,2}:\s*["']?(\w+)["']?/i);
+    if (statusMatch && statusMatch[1].toLowerCase() === 'building') {
+      try {
+        const stateStat = fs.statSync(stateFile);
+        const ageMs = Date.now() - stateStat.mtimeMs;
+        const ageMinutes = Math.round(ageMs / 60000);
+        if (ageMinutes > 30) {
+          parts.push(`\nWarning: STATE.md shows status "Building" but was last modified ${ageMinutes} minutes ago. This may indicate a crashed executor. Run /pbr:health to diagnose.`);
+          logHook('progress-tracker', 'SessionStart', 'stale-building', { ageMinutes });
+        }
+      } catch (_e) { /* best-effort */ }
+    }
   } else {
     parts.push('\nNo STATE.md found. Run /pbr:begin to initialize or /pbr:status to check.');
   }

package/plugins/pbr/scripts/track-context-budget.js

@@ -67,8 +67,17 @@ function main() {
       const currentSkill = readFileSafe(skillPath);
       let tracker = loadTracker(trackerPath);
 
-      if (tracker.skill !== currentSkill || tracker.files.length > 200) {
+      if (tracker.skill !== currentSkill) {
         tracker = { skill: currentSkill, reads: 0, total_chars: 0, files: [] };
+      } else if (tracker.files.length > 200) {
+        logHook('track-context-budget', 'PostToolUse', 'warn', {
+          reason: 'tracker reset at 200 files',
+          reads: tracker.reads,
+          total_chars: tracker.total_chars,
+          unique_files: tracker.files.length,
+        });
+        const prevCharsTotal = tracker.total_chars;
+        tracker = { skill: currentSkill, reads: 0, total_chars: prevCharsTotal, files: [] };
       }
 
       // Update tracker

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

@@ -677,6 +677,18 @@ function checkActiveSkillIntegrity(data) {
     return 'Active-skill integrity: .planning/.active-skill not found. Skill-specific enforcement is disabled. The invoking skill should write this file. To fix: Wait for the current skill to finish, or delete .planning/.active-skill if stale.';
   }
 
+  // Stale lock detection: warn if .active-skill is older than 2 hours
+  try {
+    const stat = fs.statSync(activeSkillFile);
+    const ageMs = Date.now() - stat.mtimeMs;
+    const TWO_HOURS = 2 * 60 * 60 * 1000;
+    if (ageMs > TWO_HOURS) {
+      const ageHours = Math.round(ageMs / (60 * 60 * 1000));
+      const skill = fs.readFileSync(activeSkillFile, 'utf8').trim();
+      return `Active-skill integrity: .planning/.active-skill is ${ageHours}h old (skill: "${skill}"). This may be a stale lock from a crashed session. Run /pbr:health to diagnose, or delete .planning/.active-skill if the previous session is no longer running.`;
+    }
+  } catch (_e) { /* best-effort */ }
+
   return null;
 }