@sienklogic/plan-build-run 2.11.0 → 2.13.0

package/CHANGELOG.md

@@ -5,6 +5,27 @@ 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.13.0](https://github.com/SienkLogic/plan-build-run/compare/plan-build-run-v2.12.0...plan-build-run-v2.13.0) (2026-02-22)
+
+
+### Features
+
+* **tools:** add stale active-skill session-start warning and copilot hook limitation docs ([158a78d](https://github.com/SienkLogic/plan-build-run/commit/158a78d03b482f56ab6f09e89bf9ef67b81fb409))
+
+## [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.13.0",
   "description": "Plan it, Build it, Run it — structured development workflow for Claude Code",
   "keywords": [
     "claude-code",

package/plugins/copilot-pbr/README.md

@@ -119,6 +119,13 @@ Copilot CLI supports 6 hook events vs Claude Code's full set. The following hook
 
 **Not available** in Copilot CLI (present in Claude Code/Cursor ports): `SubagentStart`, `SubagentStop`, `TaskCompleted`, `PostToolUseFailure`, `PreCompact`, `Stop`.
 
+**Impact of missing hooks:**
+
+- No auto-continue between skills (`Stop` hook) — you must manually run the next command
+- No tool failure logging (`PostToolUseFailure`) — silent failures won't be recorded to `.planning/logs/`
+- No context budget preservation on compaction (`PreCompact`) — STATE.md won't be auto-preserved when context is compressed
+- No subagent lifecycle tracking (`SubagentStart`/`SubagentStop`/`TaskCompleted`) — agent spawn/completion events aren't logged
+
 ## Cross-Plugin Compatibility
 
 This plugin works alongside the Claude Code and Cursor versions of Plan-Build-Run. All three plugins share the same `.planning/` directory and file formats, so you can switch between tools without losing state. Hook scripts under `plugins/pbr/scripts/` are shared between all plugins via relative paths.

package/plugins/copilot-pbr/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "pbr",
   "displayName": "Plan-Build-Run",
-  "version": "2.11.0",
+  "version": "2.13.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.13.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.13.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.');
   }
@@ -168,6 +182,23 @@ function buildContext(planningDir, stateFile) {
     }
   }
 
+  // Check for stale .active-skill (multi-session conflict detection)
+  const activeSkillFile = path.join(planningDir, '.active-skill');
+  if (fs.existsSync(activeSkillFile)) {
+    try {
+      const stats = fs.statSync(activeSkillFile);
+      const ageMs = Date.now() - stats.mtimeMs;
+      const ageMinutes = Math.floor(ageMs / 60000);
+      if (ageMinutes > 60) {
+        const skill = fs.readFileSync(activeSkillFile, 'utf8').trim();
+        parts.push(`\nWarning: .active-skill is ${ageMinutes} minutes old (skill: "${skill}"). This may be a stale lock from a crashed session or concurrent session conflict. Run /pbr:health to auto-fix, or delete .planning/.active-skill manually.`);
+        logHook('progress-tracker', 'SessionStart', 'stale-active-skill', { ageMinutes, skill });
+      }
+    } catch (_e) {
+      // Ignore errors
+    }
+  }
+
   // Check for stale .auto-next signal (S>M-9)
   const autoNextFile = path.join(planningDir, '.auto-next');
   if (fs.existsSync(autoNextFile)) {

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;
 }