@sienklogic/plan-build-run 2.41.0 → 2.42.0

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

@@ -25,6 +25,16 @@ const { validateTask: llmValidateTask } = require('./local-llm/operations/valida
 const { checkNonPbrAgent } = require('./enforce-pbr-workflow');
 const { KNOWN_AGENTS } = require('./pbr-tools');
 
+// Gate modules
+const { checkQuickExecutorGate } = require('./lib/gates/quick-executor');
+const { checkBuildExecutorGate } = require('./lib/gates/build-executor');
+const { checkPlanExecutorGate } = require('./lib/gates/plan-executor');
+const { checkReviewPlannerGate } = require('./lib/gates/review-planner');
+const { checkReviewVerifierGate } = require('./lib/gates/review-verifier');
+const { checkMilestoneCompleteGate, getVerificationStatus } = require('./lib/gates/milestone-complete');
+const { checkBuildDependencyGate } = require('./lib/gates/build-dependency');
+const { checkDebuggerAdvisory, checkCheckpointManifest, checkActiveSkillIntegrity } = require('./lib/gates/advisories');
+
 /**
  * Load and resolve the local_llm config block from .planning/config.json.
  * Returns a resolved config (always safe to use — disabled by default on error).
@@ -88,611 +98,6 @@ function checkTask(data) {
   return warnings;
 }
 
-/**
- * Blocking check: when the active skill is "quick" and an executor is being
- * spawned, verify that at least one .planning/quick/{NNN}-{slug}/PLAN.md exists.
- * Returns { block: true, reason: "..." } if the executor should be blocked,
- * or null if it's OK to proceed.
- */
-function checkQuickExecutorGate(data) {
-  const toolInput = data.tool_input || {};
-  const subagentType = toolInput.subagent_type || '';
-
-  // Only gate pbr:executor
-  if (subagentType !== 'pbr:executor') return null;
-
-  const cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
-  const planningDir = path.join(cwd, '.planning');
-  const activeSkillFile = path.join(planningDir, '.active-skill');
-
-  // Only gate when active skill is "quick"
-  try {
-    const activeSkill = fs.readFileSync(activeSkillFile, 'utf8').trim();
-    if (activeSkill !== 'quick') return null;
-  } catch (_e) {
-    // No .active-skill file — not in a quick task flow
-    return null;
-  }
-
-  // Check for any PLAN.md in .planning/quick/*/
-  const quickDir = path.join(planningDir, 'quick');
-  if (!fs.existsSync(quickDir)) {
-    return {
-      block: true,
-      reason: 'Cannot spawn executor: .planning/quick/ directory does not exist.\n\nThe quick skill must create the task directory and PLAN.md before an executor can run (Steps 4-6).\n\nRe-run /pbr:quick to create the quick task directory and PLAN.md.'
-    };
-  }
-
-  try {
-    const dirs = fs.readdirSync(quickDir).filter(d => {
-      return /^\d{3}-/.test(d) && fs.statSync(path.join(quickDir, d)).isDirectory();
-    });
-
-    // Look for the most recent quick task dir that has a PLAN.md
-    const hasPlan = dirs.some(d => {
-      const planFile = path.join(quickDir, d, 'PLAN.md');
-      try {
-        const stat = fs.statSync(planFile);
-        return stat.size > 0;
-      } catch (_e) {
-        return false;
-      }
-    });
-
-    if (!hasPlan) {
-      return {
-        block: true,
-        reason: 'Cannot spawn executor: no PLAN.md found in any .planning/quick/*/ directory.\n\nThe quick skill must write a non-empty PLAN.md inside .planning/quick/{NNN}-{slug}/ before an executor can run (Steps 4-6).\n\nRe-run /pbr:quick to create the quick task directory and PLAN.md.'
-      };
-    }
-  } catch (_e) {
-    return {
-      block: true,
-      reason: 'Cannot spawn executor: failed to read .planning/quick/ directory.\n\nThe directory exists but could not be read, possibly due to a permissions issue or filesystem error.\n\nRe-run /pbr:quick to recreate the quick task directory and PLAN.md.'
-    };
-  }
-
-  return null;
-}
-
-/**
- * Blocking check: when the active skill is "build" and an executor is being
- * spawned, verify that a PLAN*.md exists in the current phase directory.
- * Returns { block: true, reason: "..." } if blocked, or null if OK.
- */
-function checkBuildExecutorGate(data) {
-  const toolInput = data.tool_input || {};
-  const subagentType = toolInput.subagent_type || '';
-
-  // Only gate pbr:executor
-  if (subagentType !== 'pbr:executor') return null;
-
-  const cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
-  const planningDir = path.join(cwd, '.planning');
-  const activeSkillFile = path.join(planningDir, '.active-skill');
-
-  // Only gate when active skill is "build"
-  try {
-    const activeSkill = fs.readFileSync(activeSkillFile, 'utf8').trim();
-    if (activeSkill !== 'build') return null;
-  } catch (_e) {
-    return null;
-  }
-
-  // Read STATE.md for current phase
-  const stateFile = path.join(planningDir, 'STATE.md');
-  try {
-    const state = fs.readFileSync(stateFile, 'utf8');
-    const phaseMatch = state.match(/Phase:\s*(\d+)\s+of\s+\d+/);
-    if (!phaseMatch) return null;
-
-    const currentPhase = phaseMatch[1].padStart(2, '0');
-    const phasesDir = path.join(planningDir, 'phases');
-    if (!fs.existsSync(phasesDir)) {
-      return {
-        block: true,
-        reason: 'Cannot spawn executor: .planning/phases/ directory does not exist.\n\nThe build skill requires a phases directory with at least one PLAN.md before an executor can run.\n\nRun /pbr:plan {N} to create plans first.'
-      };
-    }
-
-    const dirs = fs.readdirSync(phasesDir).filter(d => d.startsWith(currentPhase + '-'));
-    if (dirs.length === 0) {
-      return {
-        block: true,
-        reason: `Cannot spawn executor: no phase directory found for phase ${currentPhase}.\n\nThe build skill needs a phase directory (e.g., .planning/phases/${currentPhase}-slug/) containing PLAN.md files.\n\nRun /pbr:plan ${currentPhase} to create plans first.`
-      };
-    }
-
-    const phaseDir = path.join(phasesDir, dirs[0]);
-    const files = fs.readdirSync(phaseDir);
-    const hasPlan = files.some(f => {
-      if (!/^PLAN.*\.md$/i.test(f)) return false;
-      try {
-        return fs.statSync(path.join(phaseDir, f)).size > 0;
-      } catch (_e) {
-        return false;
-      }
-    });
-
-    if (!hasPlan) {
-      return {
-        block: true,
-        reason: `Cannot spawn executor: no PLAN.md found in .planning/phases/${dirs[0]}/.\n\nThe phase directory exists but contains no PLAN.md files. The executor needs at least one non-empty PLAN.md to work from.\n\nRun /pbr:plan ${currentPhase} to create plans first.`
-      };
-    }
-  } catch (_e) {
-    return null;
-  }
-
-  return null;
-}
-
-/**
- * Blocking check: when the active skill is "plan", block executor spawning.
- * The plan skill should never spawn executors.
- * Returns { block: true, reason: "..." } if blocked, or null if OK.
- */
-function checkPlanExecutorGate(data) {
-  const toolInput = data.tool_input || {};
-  const subagentType = toolInput.subagent_type || '';
-
-  // Only gate pbr:executor
-  if (subagentType !== 'pbr:executor') return null;
-
-  const cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
-  const planningDir = path.join(cwd, '.planning');
-  const activeSkillFile = path.join(planningDir, '.active-skill');
-
-  try {
-    const activeSkill = fs.readFileSync(activeSkillFile, 'utf8').trim();
-    if (activeSkill !== 'plan') return null;
-  } catch (_e) {
-    return null;
-  }
-
-  return {
-    block: true,
-    reason: 'Plan skill cannot spawn executors.\n\nThe plan skill creates plans; the build skill executes them. Spawning an executor from the plan skill violates the separation of concerns.\n\nRun /pbr:build to execute plans instead.'
-  };
-}
-
-/**
- * Blocking check: when the active skill is "review" and a planner is being
- * spawned, verify that a VERIFICATION.md exists in the current phase directory.
- * Returns { block: true, reason: "..." } if blocked, or null if OK.
- */
-function checkReviewPlannerGate(data) {
-  const toolInput = data.tool_input || {};
-  const subagentType = toolInput.subagent_type || '';
-
-  // Only gate pbr:planner
-  if (subagentType !== 'pbr:planner') return null;
-
-  const cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
-  const planningDir = path.join(cwd, '.planning');
-  const activeSkillFile = path.join(planningDir, '.active-skill');
-
-  // Only gate when active skill is "review"
-  try {
-    const activeSkill = fs.readFileSync(activeSkillFile, 'utf8').trim();
-    if (activeSkill !== 'review') return null;
-  } catch (_e) {
-    return null;
-  }
-
-  // Read STATE.md for current phase
-  const stateFile = path.join(planningDir, 'STATE.md');
-  try {
-    const state = fs.readFileSync(stateFile, 'utf8');
-    const phaseMatch = state.match(/Phase:\s*(\d+)/);
-    if (!phaseMatch) return null;
-
-    const currentPhase = phaseMatch[1].padStart(2, '0');
-    const phasesDir = path.join(planningDir, 'phases');
-    if (!fs.existsSync(phasesDir)) return null;
-
-    const dirs = fs.readdirSync(phasesDir).filter(d => d.startsWith(currentPhase + '-'));
-    if (dirs.length === 0) return null;
-
-    const phaseDir = path.join(phasesDir, dirs[0]);
-    const hasVerification = fs.existsSync(path.join(phaseDir, 'VERIFICATION.md'));
-
-    if (!hasVerification) {
-      return {
-        block: true,
-        reason: 'Review planner gate: cannot spawn planner without VERIFICATION.md.\n\nGap closure requires an existing VERIFICATION.md to identify which gaps need closing. Without it, the planner has no input.\n\nRun /pbr:review {N} to create VERIFICATION.md first.'
-      };
-    }
-  } catch (_e) {
-    return null;
-  }
-
-  return null;
-}
-
-/**
- * Blocking check: when the active skill is "review" and a verifier is being
- * spawned, verify that a SUMMARY*.md exists in the current phase directory.
- * Returns { block: true, reason: "..." } if blocked, or null if OK.
- */
-function checkReviewVerifierGate(data) {
-  const toolInput = data.tool_input || {};
-  const subagentType = toolInput.subagent_type || '';
-
-  // Only gate pbr:verifier
-  if (subagentType !== 'pbr:verifier') return null;
-
-  const cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
-  const planningDir = path.join(cwd, '.planning');
-  const activeSkillFile = path.join(planningDir, '.active-skill');
-
-  // Only gate when active skill is "review"
-  try {
-    const activeSkill = fs.readFileSync(activeSkillFile, 'utf8').trim();
-    if (activeSkill !== 'review') return null;
-  } catch (_e) {
-    return null;
-  }
-
-  // Read STATE.md for current phase
-  const stateFile = path.join(planningDir, 'STATE.md');
-  try {
-    const state = fs.readFileSync(stateFile, 'utf8');
-    const phaseMatch = state.match(/Phase:\s*(\d+)/);
-    if (!phaseMatch) return null;
-
-    const currentPhase = phaseMatch[1].padStart(2, '0');
-    const phasesDir = path.join(planningDir, 'phases');
-    if (!fs.existsSync(phasesDir)) return null;
-
-    const dirs = fs.readdirSync(phasesDir).filter(d => d.startsWith(currentPhase + '-'));
-    if (dirs.length === 0) return null;
-
-    const phaseDir = path.join(phasesDir, dirs[0]);
-    const files = fs.readdirSync(phaseDir);
-    const hasSummary = files.some(f => {
-      if (!/^SUMMARY/i.test(f)) return false;
-      try {
-        return fs.statSync(path.join(phaseDir, f)).size > 0;
-      } catch (_e) {
-        return false;
-      }
-    });
-
-    if (!hasSummary) {
-      return {
-        block: true,
-        reason: 'Review verifier gate: cannot spawn verifier without SUMMARY.md.\n\nThe verifier checks executor output against the plan. Without a SUMMARY.md, there is nothing to verify.\n\nRun /pbr:build {N} to create SUMMARY.md first.'
-      };
-    }
-  } catch (_e) {
-    return null;
-  }
-
-  return null;
-}
-
-/**
- * Blocking check: when the active skill is "milestone" and a general/planner agent
- * is being spawned for a "complete" operation, verify all milestone phases have VERIFICATION.md.
- * Returns { block: true, reason: "..." } if blocked, or null if OK.
- */
-function checkMilestoneCompleteGate(data) {
-  const toolInput = data.tool_input || {};
-  const subagentType = toolInput.subagent_type || '';
-  const description = toolInput.description || '';
-
-  const cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
-  const planningDir = path.join(cwd, '.planning');
-  const activeSkillFile = path.join(planningDir, '.active-skill');
-
-  // Only gate when active skill is "milestone"
-  try {
-    const activeSkill = fs.readFileSync(activeSkillFile, 'utf8').trim();
-    if (activeSkill !== 'milestone') return null;
-  } catch (_e) {
-    return null;
-  }
-
-  // Only gate pbr:general and pbr:planner
-  if (subagentType !== 'pbr:general' && subagentType !== 'pbr:planner') return null;
-
-  // Only gate "complete" operations
-  if (!/complete/i.test(description)) return null;
-
-  // Read STATE.md for current phase
-  const stateFile = path.join(planningDir, 'STATE.md');
-  let currentPhase;
-  try {
-    const state = fs.readFileSync(stateFile, 'utf8');
-    const fmMatch = state.match(/current_phase:\s*(\d+)/);
-    if (fmMatch) {
-      currentPhase = parseInt(fmMatch[1], 10);
-    } else {
-      const bodyMatch = state.match(/Phase:\s*(\d+)/);
-      if (bodyMatch) currentPhase = parseInt(bodyMatch[1], 10);
-    }
-    if (!currentPhase) return null;
-  } catch (_e) {
-    return null;
-  }
-
-  // Read ROADMAP.md and find the milestone containing the current phase
-  const roadmapFile = path.join(planningDir, 'ROADMAP.md');
-  try {
-    const roadmap = fs.readFileSync(roadmapFile, 'utf8');
-
-    // Split into milestone sections
-    const milestoneSections = roadmap.split(/^## Milestone:/m).slice(1);
-
-    for (const section of milestoneSections) {
-      // Parse phase numbers from table rows
-      const phaseNumbers = [];
-      const tableRowRegex = /^\|\s*(\d+)\s*\|/gm;
-      let match;
-      while ((match = tableRowRegex.exec(section)) !== null) {
-        phaseNumbers.push(parseInt(match[1], 10));
-      }
-
-      // Check if current phase is in this milestone
-      if (!phaseNumbers.includes(currentPhase)) continue;
-
-      // Found the right milestone — check all phases have VERIFICATION.md
-      const phasesDir = path.join(planningDir, 'phases');
-      if (!fs.existsSync(phasesDir)) return null;
-
-      for (const phaseNum of phaseNumbers) {
-        const paddedPhase = String(phaseNum).padStart(2, '0');
-        const pDirs = fs.readdirSync(phasesDir).filter(d => d.startsWith(paddedPhase + '-'));
-        if (pDirs.length === 0) {
-          return {
-            block: true,
-            reason: `Milestone complete gate: phase ${paddedPhase} directory not found.\n\nAll milestone phases must exist and have a passing VERIFICATION.md before the milestone can be completed.\n\nRun /pbr:review ${paddedPhase} to verify the phase (it must reach status: passed).`
-          };
-        }
-        const verificationFile = path.join(phasesDir, pDirs[0], 'VERIFICATION.md');
-        const hasVerification = fs.existsSync(verificationFile);
-        if (!hasVerification) {
-          return {
-            block: true,
-            reason: `Milestone complete gate: phase ${paddedPhase} (${pDirs[0]}) lacks VERIFICATION.md.\n\nAll milestone phases must have a passing VERIFICATION.md before the milestone can be completed.\n\nRun /pbr:review ${paddedPhase} to verify the phase (it must reach status: passed).`
-          };
-        }
-        const verStatus = getVerificationStatus(verificationFile);
-        if (verStatus === 'gaps_found') {
-          return {
-            block: true,
-            reason: `Milestone complete gate: phase ${paddedPhase} VERIFICATION.md has status: gaps_found.\n\nAll gaps must be closed before the milestone can be completed. The verifier found issues that need resolution.\n\nRun /pbr:review ${paddedPhase} to close gaps (phase must reach status: passed).`
-          };
-        }
-      }
-
-      // All phases verified
-      return null;
-    }
-  } catch (_e) {
-    return null;
-  }
-
-  return null;
-}
-
-/**
- * Blocking check: when the active skill is "build" and an executor is being
- * spawned, verify that dependent phases (from ROADMAP.md) have VERIFICATION.md.
- * Returns { block: true, reason: "..." } if blocked, or null if OK.
- */
-function checkBuildDependencyGate(data) {
-  const toolInput = data.tool_input || {};
-  const subagentType = toolInput.subagent_type || '';
-
-  // Only gate pbr:executor
-  if (subagentType !== 'pbr:executor') return null;
-
-  const cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
-  const planningDir = path.join(cwd, '.planning');
-  const activeSkillFile = path.join(planningDir, '.active-skill');
-
-  // Only gate when active skill is "build"
-  try {
-    const activeSkill = fs.readFileSync(activeSkillFile, 'utf8').trim();
-    if (activeSkill !== 'build') return null;
-  } catch (_e) {
-    return null;
-  }
-
-  // Read STATE.md for current phase
-  const stateFile = path.join(planningDir, 'STATE.md');
-  let currentPhase;
-  try {
-    const state = fs.readFileSync(stateFile, 'utf8');
-    const phaseMatch = state.match(/Phase:\s*(\d+)/);
-    if (!phaseMatch) return null;
-    currentPhase = parseInt(phaseMatch[1], 10);
-  } catch (_e) {
-    return null;
-  }
-
-  // Read ROADMAP.md, find current phase section, check dependencies
-  const roadmapFile = path.join(planningDir, 'ROADMAP.md');
-  try {
-    const roadmap = fs.readFileSync(roadmapFile, 'utf8');
-
-    // Find ### Phase N: section
-    const phaseRegex = new RegExp(`### Phase ${currentPhase}:[\\s\\S]*?(?=### Phase \\d|$)`);
-    const phaseSection = roadmap.match(phaseRegex);
-    if (!phaseSection) return null;
-
-    // Look for **Depends on:** line
-    const depMatch = phaseSection[0].match(/\*\*Depends on:\*\*\s*(.*)/);
-    if (!depMatch) return null;
-
-    const depLine = depMatch[1].trim();
-    if (!depLine || /^none$/i.test(depLine)) return null;
-
-    // Parse phase numbers from "Phase 1", "Phase 1, Phase 2", etc.
-    const depPhases = [];
-    const depRegex = /Phase\s+(\d+)/gi;
-    let match;
-    while ((match = depRegex.exec(depLine)) !== null) {
-      depPhases.push(parseInt(match[1], 10));
-    }
-
-    if (depPhases.length === 0) return null;
-
-    // Check each dependent phase has VERIFICATION.md
-    const phasesDir = path.join(planningDir, 'phases');
-    if (!fs.existsSync(phasesDir)) return null;
-
-    for (const depPhase of depPhases) {
-      const paddedPhase = String(depPhase).padStart(2, '0');
-      const pDirs = fs.readdirSync(phasesDir).filter(d => d.startsWith(paddedPhase + '-'));
-      if (pDirs.length === 0) {
-        return {
-          block: true,
-          reason: `Build dependency gate: dependent phase ${paddedPhase} lacks VERIFICATION.md.\n\nPhase ${currentPhase} depends on phase ${paddedPhase}, which must be verified before building can proceed.\n\nRun /pbr:review ${paddedPhase} to verify the dependency phase first.`
-        };
-      }
-      const hasVerification = fs.existsSync(path.join(phasesDir, pDirs[0], 'VERIFICATION.md'));
-      if (!hasVerification) {
-        return {
-          block: true,
-          reason: `Build dependency gate: dependent phase ${paddedPhase} lacks VERIFICATION.md.\n\nPhase ${currentPhase} depends on phase ${paddedPhase}, which must be verified before building can proceed.\n\nRun /pbr:review ${paddedPhase} to verify the dependency phase first.`
-        };
-      }
-    }
-  } catch (_e) {
-    return null;
-  }
-
-  return null;
-}
-
-/**
- * Parse VERIFICATION.md frontmatter to extract status field.
- * Returns the status string or 'unknown' if not parseable.
- */
-function getVerificationStatus(filePath) {
-  try {
-    const content = fs.readFileSync(filePath, 'utf8');
-    const fmMatch = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
-    if (!fmMatch) return 'unknown';
-    const statusMatch = fmMatch[1].match(/^status:\s*(\S+)/m);
-    return statusMatch ? statusMatch[1] : 'unknown';
-  } catch (_e) {
-    return 'unknown';
-  }
-}
-
-/**
- * Advisory check: when pbr:debugger is spawned and .active-skill is 'debug',
- * warn if .planning/debug/ directory does not exist.
- * Returns a warning string or null.
- */
-function checkDebuggerAdvisory(data) {
-  const subagentType = data.tool_input?.subagent_type || '';
-  if (subagentType !== 'pbr:debugger') return null;
-  // Only advise when spawned from the debug skill
-  const debugCwd = process.env.PBR_PROJECT_ROOT || process.cwd();
-  const activeSkillPath = path.join(debugCwd, '.planning', '.active-skill');
-  try {
-    const activeSkill = fs.readFileSync(activeSkillPath, 'utf8').trim();
-    if (activeSkill !== 'debug') return null;
-  } catch (_e) {
-    return null; // No .active-skill file — skip advisory
-  }
-  const debugDir = path.join(debugCwd, '.planning', 'debug');
-  if (!fs.existsSync(debugDir)) {
-    return 'Debugger advisory: .planning/debug/ does not exist. Create it before spawning the debugger so output has a target location.';
-  }
-  return null;
-}
-
-function checkCheckpointManifest(data) {
-  const toolInput = data.tool_input || {};
-  const subagentType = toolInput.subagent_type || '';
-
-  if (subagentType !== 'pbr:executor') return null;
-
-  const cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
-  const planningDir = path.join(cwd, '.planning');
-  const activeSkillFile = path.join(planningDir, '.active-skill');
-
-  try {
-    const activeSkill = fs.readFileSync(activeSkillFile, 'utf8').trim();
-    if (activeSkill !== 'build') return null;
-  } catch (_e) {
-    return null;
-  }
-
-  // Find current phase dir
-  const stateFile = path.join(planningDir, 'STATE.md');
-  try {
-    const state = fs.readFileSync(stateFile, 'utf8');
-    const phaseMatch = state.match(/Phase:\s*(\d+)\s+of\s+\d+/);
-    if (!phaseMatch) return null;
-
-    const currentPhase = phaseMatch[1].padStart(2, '0');
-    const phasesDir = path.join(planningDir, 'phases');
-    if (!fs.existsSync(phasesDir)) return null;
-
-    const dirs = fs.readdirSync(phasesDir).filter(d => d.startsWith(currentPhase + '-'));
-    if (dirs.length === 0) return null;
-
-    const phaseDir = path.join(phasesDir, dirs[0]);
-    const manifestFile = path.join(phaseDir, '.checkpoint-manifest.json');
-    if (!fs.existsSync(manifestFile)) {
-      return 'Build advisory: .checkpoint-manifest.json not found in phase directory. The build skill should write this before spawning executors. To fix: Run /pbr:health to regenerate checkpoint manifest.';
-    }
-  } catch (_e) {
-    return null;
-  }
-
-  return null;
-}
-
-/**
- * Advisory check: when any pbr:* agent is being spawned, warn if
- * .planning/.active-skill doesn't exist. Without this file, all
- * skill-specific enforcement is silently disabled.
- * Returns a warning string or null.
- */
-function checkActiveSkillIntegrity(data) {
-  const toolInput = data.tool_input || {};
-  const subagentType = toolInput.subagent_type || '';
-
-  if (typeof subagentType !== 'string' || !subagentType.startsWith('pbr:')) return null;
-
-  // Advisory agents that run without an active skill context — exempt from .active-skill checks
-  const EXEMPT_AGENTS = ['pbr:researcher', 'pbr:synthesizer', 'pbr:audit', 'pbr:dev-sync', 'pbr:general'];
-  if (EXEMPT_AGENTS.includes(subagentType)) return null;
-
-  const cwd = process.env.PBR_PROJECT_ROOT || process.cwd();
-  const planningDir = path.join(cwd, '.planning');
-
-  // Only check if .planning/ exists (PBR project)
-  if (!fs.existsSync(planningDir)) return null;
-
-  const activeSkillFile = path.join(planningDir, '.active-skill');
-  if (!fs.existsSync(activeSkillFile)) {
-    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;
-}
-
 function main() {
   let input = '';