harness-everything 0.16.0 → 0.16.1

package/hooks/hooks.json

@@ -34,7 +34,7 @@
             "command": "node \"${CLAUDE_PLUGIN_ROOT}/hooks/scripts/workflow-gate.js\""
           }
         ],
-        "description": "Enforce mandatory workflow entry and Tier 3 Git-worktree mutation isolation",
+        "description": "Emit advisory workflow/worktree reminders before mutation",
         "id": "harness:pre:workflow-gate"
       },
       {
@@ -49,7 +49,7 @@
         "id": "harness:pre:action-gate"
       },
       {
-        "matcher": "Bash|PowerShell|Edit|Write",
+        "matcher": "Bash|PowerShell|Edit|Write|apply_patch",
         "hooks": [
           {
             "type": "command",
@@ -67,7 +67,7 @@
             "command": "node \"${CLAUDE_PLUGIN_ROOT}/hooks/scripts/boundary-guard.js\""
           }
         ],
-        "description": "Physical boundary enforcement for searches and large file reads",
+        "description": "Warn about expensive broad searches and large file reads",
         "id": "harness:pre:boundary-guard"
       },
       {
@@ -78,7 +78,7 @@
             "command": "node \"${CLAUDE_PLUGIN_ROOT}/hooks/scripts/depth-guard.js\""
           }
         ],
-        "description": "Information depth guard - Enforce impact assessment before writing code",
+        "description": "Remind to inspect existing files before editing",
         "id": "harness:pre:depth-guard"
       },
       {
@@ -133,7 +133,7 @@
             "command": "node \"${CLAUDE_PLUGIN_ROOT}/hooks/scripts/workflow-stop-gate.js\""
           }
         ],
-        "description": "Prevent completion while a selected Fable workflow has not reached verified stage resolution",
+        "description": "Remind when selected workflow evidence is incomplete",
         "id": "harness:stop:workflow-completion-gate"
       },
       {
@@ -143,7 +143,7 @@
             "command": "node \"${CLAUDE_PLUGIN_ROOT}/hooks/scripts/stop-gate.js\""
           }
         ],
-        "description": "Pre-delivery gate - bounce the end of a turn once per edit batch when edits were never verified",
+        "description": "Remind when edits have no observed successful verification",
         "id": "harness:stop:stop-gate"
       },
       {
@@ -157,19 +157,6 @@
         "id": "harness:stop:skill-telemetry"
       }
     ],
-    "PermissionDenied": [
-      {
-        "matcher": "Bash|PowerShell",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node \"${CLAUDE_PLUGIN_ROOT}/hooks/scripts/workflow-mutation-denied.js\""
-          }
-        ],
-        "description": "Release reserved mutation capacity when Claude auto mode denies a shell tool call",
-        "id": "harness:permission-denied:mutation-probe"
-      }
-    ],
     "PostToolUse": [
       {
         "matcher": "Bash|PowerShell|mcp__.*",
@@ -183,7 +170,7 @@
         "id": "harness:post:action-gate-audit"
       },
       {
-        "matcher": "Bash|PowerShell",
+        "matcher": "Bash|PowerShell|apply_patch",
         "hooks": [
           {
             "type": "command",
@@ -194,7 +181,7 @@
         "id": "harness:post:rule-of-3-tracker"
       },
       {
-        "matcher": "Edit|Write|Bash|PowerShell",
+        "matcher": "Edit|Write|Bash|PowerShell|apply_patch",
         "hooks": [
           {
             "type": "command",
@@ -262,7 +249,7 @@
         "id": "harness:post-failure:action-gate-audit"
       },
       {
-        "matcher": "Bash|PowerShell",
+        "matcher": "Bash|PowerShell|apply_patch",
         "hooks": [
           {
             "type": "command",
@@ -273,7 +260,7 @@
         "id": "harness:post-failure:rule-of-3-tracker"
       },
       {
-        "matcher": "Bash|PowerShell",
+        "matcher": "Bash|PowerShell|apply_patch",
         "hooks": [
           {
             "type": "command",

package/hooks/scripts/atomic-commit-check.js

@@ -143,7 +143,7 @@ function main(payload) {
       const changedFiles = statusRaw.split('\n').length;
       console.error(`[Atomic Commit Check] ${state.editCount} edits since the last commit (${changedFiles} files currently changed).`);
       console.error('If a logically complete chunk of work is done, commit it now before continuing - large uncommitted diffs are harder to review and harder to recover from.');
-      process.exit(2);
+      process.exit(0);
     }
 
     process.exit(0);

package/hooks/scripts/boundary-guard.js

@@ -25,10 +25,10 @@ process.stdin.on('end', () => {
       if (filePath && !input.limit && fs.existsSync(filePath)) {
         const stat = fs.statSync(filePath);
         if (stat.isFile() && stat.size > MAX_BYTES_WITHOUT_LIMIT) {
-          console.error(`[Boundary Guard] BLOCKED: "${filePath}" is ${(stat.size / 1024).toFixed(0)}KB.`);
+          console.error(`[Boundary Reminder]: "${filePath}" is ${(stat.size / 1024).toFixed(0)}KB.`);
           console.error(`Reading a file this large in one call risks context bloat and "lost in the middle" degradation.`);
           console.error(`Re-issue the Read with an explicit "offset"/"limit" to pull a targeted slice, or use Grep to locate the relevant section first.`);
-          process.exit(2);
+          process.exit(0);
         }
       }
     }
@@ -41,10 +41,10 @@ process.stdin.on('end', () => {
         return asPathSegment.test(searchPath) || asPathSegment.test(pattern);
       });
       if (hitsNoiseDir) {
-        console.error(`[Boundary Guard] BLOCKED: search targets a noise directory (${NOISE_DIRS.join(', ')}).`);
+        console.error(`[Boundary Reminder]: search targets a noise directory (${NOISE_DIRS.join(', ')}).`);
         console.error(`These produce huge, low-signal result sets and rarely contain what you're actually looking for.`);
         console.error(`Scope the search to source directories instead, or state explicitly why this directory must be searched.`);
-        process.exit(2);
+        process.exit(0);
       }
     }
 

package/hooks/scripts/contract-test.js

@@ -22,7 +22,7 @@ const {
   listRunContracts,
   getWorkerId,
 } = require('./lib/fable-contracts');
-const { loadWorkflow, matchingRun, recordBudgetEvent } = require('./lib/workflow-runtime');
+const { loadWorkflow, matchingRun } = require('./lib/workflow-runtime');
 const { createLearningOpportunity } = require('./lib/learning-opportunity');
 
 function resultState(payload) {
@@ -119,7 +119,7 @@ process.stdin.on('end', () => {
       if (resolution.ambiguous || !resolution.match) {
         console.error(`[Contract Test] Ambiguous Fable check command matched ${runMatches.length} active run contract(s): ${command}`);
         console.error('No contract was mutated. Correlate the worker/session to one run before accepting this check.');
-        process.exit(2);
+        process.exit(0);
       }
 
       const entry = resolution.match;
@@ -127,14 +127,14 @@ process.stdin.on('end', () => {
         const worker = getWorkerId(payload);
         if (!worker || (entry.contract.workerId && worker !== entry.contract.workerId)) {
           console.error('[Contract Test] Active workflow check lacks a matching observed worker identity.');
-          process.exit(2);
+          process.exit(0);
         }
         for (const id of entry.contract.dependsOn || []) {
           const dependency = JSON.parse(fs.readFileSync(path.join(entry.runRoot, 'contracts', `${id}.json`), 'utf8'));
           const escaped = workflowContext.workflow.escapes?.some(item => item.stageId === id && item.runId === entry.contract.runId);
           if (dependency.status !== 'pass' && !escaped) {
             console.error('[Contract Test] Dependency has not passed: ' + id);
-            process.exit(2);
+            process.exit(0);
           }
         }
       }
@@ -164,12 +164,9 @@ process.stdin.on('end', () => {
       }
       console.log(`[Contract Test] ${contract.planId}/${contract.runId}/${contract.stageId}: ${contract.status.toUpperCase()} ${contract.checkCommand}`);
       if (contract.status === 'fail') {
-        if (workflowContext.workflow?.workflowId) {
-          recordBudgetEvent(workflowContext, 'revision', { evidence: `${contract.stageId}:${contract.evidence || 'verification-failed'}` });
-        }
         console.error(`Evidence: ${contract.evidence || '(no output captured)'}`);
-        console.error(`Do not build on this stage's output until it passes. Artifact: ${contract.outputPath || '(no path recorded)'}`);
-        process.exit(2);
+        console.error(`Review this failed stage before relying on its output. Artifact: ${contract.outputPath || '(no path recorded)'}`);
+        process.exit(0);
       }
       process.exit(0);
     }
@@ -179,12 +176,12 @@ process.stdin.on('end', () => {
     const legacy = resolveLegacyContract(stateRoot, command, payload);
     if (legacy && legacy.ambiguous) {
       console.error(`[Contract Test] Ambiguous legacy stage contracts matched: ${command}`);
-      process.exit(2);
+      process.exit(0);
     }
     if (legacy && legacy.failed) {
       console.error(`[Contract Test] Legacy stage "${legacy.contract.stageId}" FAILED its named check: ${legacy.contract.checkCommand}`);
       console.error(`Evidence: ${legacy.contract.evidence || '(no output captured)'}`);
-      process.exit(2);
+      process.exit(0);
     }
     process.exit(0);
   } catch (_) {

package/hooks/scripts/depth-guard.js

@@ -52,10 +52,10 @@ process.stdin.on('end', () => {
     });
 
     if (!wasRead) {
-      console.error(`[Depth Guard] BLOCKED: "${filePath}" already exists but hasn't been Read in this session.`);
+      console.error(`[Depth Reminder]: "${filePath}" already exists but hasn't been Read in this session.`);
       console.error(`Write() overwrites the whole file - doing that blind risks destroying content you never looked at.`);
       console.error(`Read the file first (even a quick pass), then retry the Write, or use Edit for a targeted change.`);
-      process.exit(2);
+      process.exit(0);
     }
 
     process.exit(0);