@sienklogic/plan-build-run 2.61.0 → 2.61.1

package/CHANGELOG.md

@@ -5,6 +5,13 @@ 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.61.1](https://github.com/SienkLogic/plan-build-run/compare/plan-build-run-v2.61.0...plan-build-run-v2.61.1) (2026-03-06)
+
+
+### Bug Fixes
+
+* **hooks:** resolve script path before .planning/ early-exit check ([06487f7](https://github.com/SienkLogic/plan-build-run/commit/06487f78799a521f6d9041dc1323d36bfdf4a1d8))
+
 ## [2.61.0](https://github.com/SienkLogic/plan-build-run/compare/plan-build-run-v2.60.0...plan-build-run-v2.61.0) (2026-03-06)
 
 

package/package.json

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

package/plugins/copilot-pbr/hooks/hooks.json

@@ -79,18 +79,6 @@
           }
         ]
       },
-      {
-        "matcher": "Write|Edit",
-        "hooks": [
-          {
-            "type": "command",
-            "bash": "node \"$(cd \"$(dirname \"$0\")\" && pwd)/../../pbr/scripts/run-hook.js\" suggest-compact.js",
-            "powershell": "node (Join-Path (Split-Path -Parent $PSScriptRoot) 'pbr\\scripts\\run-hook.js') suggest-compact.js",
-            "cwd": ".",
-            "timeoutSec": 15
-          }
-        ]
-      },
       {
         "matcher": "Bash",
         "hooks": [
@@ -166,18 +154,6 @@
           }
         ]
       },
-      {
-        "matcher": "Bash",
-        "hooks": [
-          {
-            "type": "command",
-            "bash": "node \"$(cd \"$(dirname \"$0\")\" && pwd)/../../pbr/scripts/run-hook.js\" check-cross-plugin-sync.js",
-            "powershell": "node (Join-Path (Split-Path -Parent $PSScriptRoot) 'pbr\\scripts\\run-hook.js') check-cross-plugin-sync.js",
-            "cwd": ".",
-            "timeoutSec": 15
-          }
-        ]
-      },
       {
         "matcher": "Write|Edit",
         "hooks": [

package/plugins/copilot-pbr/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "pbr",
   "displayName": "Plan-Build-Run",
-  "version": "2.61.0",
+  "version": "2.61.1",
   "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.61.0",
+  "version": "2.61.1",
   "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/cursor-pbr/hooks/hooks.json

@@ -67,16 +67,6 @@
           }
         ]
       },
-      {
-        "matcher": "Write|Edit",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node -e \"var r=process.env.CLAUDE_PLUGIN_ROOT||'',m=r.match(/^\\/([a-zA-Z])\\/(.*)/);if(m)r=m[1]+String.fromCharCode(58)+String.fromCharCode(92)+m[2];require(require('path').resolve(r,'..','pbr','scripts','run-hook.js'))\" suggest-compact.js",
-            "statusMessage": "Checking context budget..."
-          }
-        ]
-      },
       {
         "matcher": "Bash",
         "hooks": [
@@ -140,16 +130,6 @@
           }
         ]
       },
-      {
-        "matcher": "Bash",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node -e \"var r=process.env.CLAUDE_PLUGIN_ROOT||'',m=r.match(/^\\/([a-zA-Z])\\/(.*)/);if(m)r=m[1]+String.fromCharCode(58)+String.fromCharCode(92)+m[2];require(require('path').resolve(r,'..','pbr','scripts','run-hook.js'))\" check-cross-plugin-sync.js",
-            "statusMessage": "Checking cross-plugin sync..."
-          }
-        ]
-      },
       {
         "matcher": "Write|Edit",
         "hooks": [

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

@@ -1,6 +1,6 @@
 {
   "name": "pbr",
-  "version": "2.61.0",
+  "version": "2.61.1",
   "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/hooks/hooks.json

@@ -72,16 +72,6 @@
           }
         ]
       },
-      {
-        "matcher": "Write|Edit",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node -e \"var r=process.env.CLAUDE_PLUGIN_ROOT||'',m=r.match(/^\\/([a-zA-Z])\\/(.*)/);if(m)r=m[1]+String.fromCharCode(58)+String.fromCharCode(92)+m[2];require(require('path').resolve(r,'scripts','run-hook.js'))\" suggest-compact.js",
-            "statusMessage": "Checking context budget..."
-          }
-        ]
-      },
       {
         "matcher": "Bash",
         "hooks": [
@@ -145,16 +135,6 @@
           }
         ]
       },
-      {
-        "matcher": "Bash",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node -e \"var r=process.env.CLAUDE_PLUGIN_ROOT||'',m=r.match(/^\\/([a-zA-Z])\\/(.*)/);if(m)r=m[1]+String.fromCharCode(58)+String.fromCharCode(92)+m[2];require(require('path').resolve(r,'scripts','run-hook.js'))\" check-cross-plugin-sync.js",
-            "statusMessage": "Checking cross-plugin sync..."
-          }
-        ]
-      },
       {
         "matcher": "Write|Edit",
         "hooks": [

package/plugins/pbr/scripts/context-bridge.js

@@ -239,6 +239,21 @@ function main() {
           source: 'bridge'
         });
         process.stdout.write(JSON.stringify(output));
+        process.exit(0);
+      }
+
+      // For Write|Edit tools, also run suggest-compact check in-process
+      // (eliminates a separate Node process spawn per Write/Edit).
+      // Detect Write|Edit by presence of file_path in tool_input.
+      const toolInput = data.tool_input || {};
+      if (toolInput.file_path || toolInput.path) {
+        try {
+          const { checkCompaction } = require('./suggest-compact');
+          const compactResult = checkCompaction(planningDir, cwd);
+          if (compactResult) {
+            process.stdout.write(JSON.stringify(compactResult));
+          }
+        } catch (_e) { /* best-effort — never block on compact check */ }
       }
 
       process.exit(0);

package/plugins/pbr/scripts/enforce-pbr-workflow.js

@@ -21,7 +21,6 @@
 const fs = require('fs');
 const path = require('path');
 const { logHook } = require('./hook-logger');
-const { sessionLoad } = require('./pbr-tools');
 
 /**
  * Load the enforcement configuration from .planning/config.json.
@@ -75,6 +74,7 @@ function checkUnmanagedSourceWrite(data) {
   if (!fs.existsSync(planningDir)) return null;
 
   // Skip if a PBR skill is active — try .session.json first, fall back to legacy .active-skill
+  const { sessionLoad } = require('./pbr-tools');
   let activeSkill = sessionLoad(planningDir).activeSkill || '';
   if (!activeSkill) {
     try { activeSkill = fs.readFileSync(path.join(planningDir, '.active-skill'), 'utf8').trim(); } catch (_) { /* legacy file missing */ }
@@ -240,6 +240,7 @@ function checkUnmanagedCommit(data) {
   if (!fs.existsSync(planningDir)) return null;
 
   // Skip if a PBR skill is active — try .session.json first, fall back to legacy .active-skill
+  const { sessionLoad } = require('./pbr-tools');
   let activeSkillCommit = sessionLoad(planningDir).activeSkill || '';
   if (!activeSkillCommit) {
     try { activeSkillCommit = fs.readFileSync(path.join(planningDir, '.active-skill'), 'utf8').trim(); } catch (_) { /* legacy file missing */ }

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

@@ -50,6 +50,7 @@ const { logHook } = require('./hook-logger');
 const { checkDangerous } = require('./check-dangerous-commands');
 const { checkCommit, enrichCommitLlm } = require('./validate-commit');
 const { checkUnmanagedCommit } = require('./enforce-pbr-workflow');
+const { checkCrossPluginSync } = require('./check-cross-plugin-sync');
 
 function main() {
   let input = '';
@@ -104,6 +105,12 @@ function main() {
         warnings.push(unmanagedCommitResult.output.additionalContext);
       }
 
+      // Cross-plugin sync advisory — warn when pbr changes lack cursor/copilot counterparts
+      const syncResult = checkCrossPluginSync(data);
+      if (syncResult) {
+        warnings.push(syncResult.additionalContext);
+      }
+
       // LLM commit semantic classification — advisory only
       const llmAdvisory = await enrichCommitLlm(data);
       if (llmAdvisory) {

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

@@ -21,8 +21,22 @@
 
 'use strict';
 
+const fs = require('fs');
 const path = require('path');
 
+// Scripts that must run even without .planning/ (lifecycle hooks).
+// All per-tool-call hooks (PreToolUse, PostToolUse, etc.) are PBR-specific
+// and can early-exit when .planning/ doesn't exist, skipping their entire
+// require() chain (~10-20ms saved per hook invocation on non-PBR projects).
+const ALWAYS_RUN = new Set([
+  'progress-tracker.js',   // SessionStart — reports non-PBR status
+  'session-cleanup.js',    // SessionEnd — cleanup
+  'worktree-create.js',    // WorktreeCreate — creates .planning/ in worktrees
+  'worktree-remove.js',    // WorktreeRemove — cleans up worktrees
+  'instructions-loaded.js', // InstructionsLoaded — detects instruction changes
+  'check-config-change.js', // ConfigChange — monitors config
+]);
+
 /**
  * Fix MSYS-style paths on Windows.
  * Converts /d/Repos/... to D:\Repos\...
@@ -73,23 +87,46 @@ if (scriptName) {
 
 function runScript(name, args) {
   args = args || [];
+
   // Try __dirname first, then pluginRoot
   const candidates = [
     path.resolve(__dirname, name),
     pluginRoot ? path.resolve(pluginRoot, 'scripts', name) : null
   ].filter(Boolean);
 
+  // Phase 1: Resolve the script path (cheap — just fs.statSync, no require).
+  // This must happen before the .planning/ check so missing scripts still
+  // get proper error messages regardless of project type.
+  let resolvedPath = null;
   for (const candidate of candidates) {
     try {
-      process.argv = [process.argv[0], candidate, ...args];
-      require(candidate);
-      return;
-    } catch (err) {
-      if (err.code !== 'MODULE_NOT_FOUND') throw err;
+      fs.statSync(candidate);
+      resolvedPath = candidate;
+      break;
+    } catch (_e) {
+      // Not found at this path
+    }
+  }
+
+  if (!resolvedPath) {
+    process.stderr.write(`run-hook: cannot find script: ${name}\n`);
+    process.stderr.write(`  searched: ${candidates.join(', ')}\n`);
+    process.exit(1);
+  }
+
+  // Phase 2: Early-exit for non-PBR projects. Script exists but .planning/
+  // doesn't — skip loading the module entirely. This avoids the full
+  // require() chain (~10-20ms) for every hook invocation in non-PBR projects.
+  if (!ALWAYS_RUN.has(name)) {
+    const cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
+    try {
+      fs.statSync(path.join(cwd, '.planning'));
+    } catch (_e) {
+      process.exit(0);
     }
   }
 
-  process.stderr.write(`run-hook: cannot find script: ${name}\n`);
-  process.stderr.write(`  searched: ${candidates.join(', ')}\n`);
-  process.exit(1);
+  // Phase 3: Load and execute the script.
+  process.argv = [process.argv[0], resolvedPath, ...args];
+  require(resolvedPath);
 }

package/plugins/pbr/scripts/suggest-compact.js

@@ -20,7 +20,6 @@
 const fs = require('fs');
 const path = require('path');
 const { logHook } = require('./hook-logger');
-const { configLoad, sessionSave } = require('./pbr-tools');
 const { loadBridge, TIER_MESSAGES } = require('./context-bridge');
 
 const DEFAULT_THRESHOLD = 50;
@@ -159,12 +158,14 @@ function saveCounter(counterPath, counter) {
   // Derive planningDir from counterPath (counterPath is planningDir/.compact-counter)
   try {
     const planningDir = path.dirname(counterPath);
+    const { sessionSave } = require('./pbr-tools');
     sessionSave(planningDir, { compactCounter: counter });
   } catch (_e) { /* non-fatal mirror */ }
 }
 
 function getThreshold(cwd) {
   const planningDir = path.join(cwd, '.planning');
+  const { configLoad } = require('./pbr-tools');
   const config = configLoad(planningDir);
   if (!config) return DEFAULT_THRESHOLD;
   return config.hooks?.compactThreshold || DEFAULT_THRESHOLD;
@@ -173,6 +174,7 @@ function getThreshold(cwd) {
 function resetCounter(planningDir) {
   // Primary: reset compactCounter in .session.json to 0
   try {
+    const { sessionSave } = require('./pbr-tools');
     sessionSave(planningDir, { compactCounter: { count: 0, lastSuggested: 0 } });
   } catch (_e) { /* best-effort */ }
 

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

@@ -194,8 +194,8 @@ function loadTracker(trackerPath) {
 function checkBridge(planningDir) {
   const bridgePath = path.join(planningDir, '.context-budget.json');
   try {
-    if (!fs.existsSync(bridgePath)) return null;
-
+    // Single statSync replaces existsSync + statSync (2 syscalls → 1).
+    // If the file doesn't exist, statSync throws and we catch below.
     const stats = fs.statSync(bridgePath);
     const ageMs = Date.now() - stats.mtimeMs;
     if (ageMs > BRIDGE_STALENESS_MS) return null;