@empirical-gauzza/cli 1.0.11 → 1.0.12

package/lib/installers/hooks.js

@@ -217,16 +217,13 @@ function main() {
 main();
 `;
 
-// Claude-only variant of the Stop reminder above. Claude Code's Stop hook contract is the one
-// platform of the three we've actually confirmed supports safe forced-continuation: exit code 2
-// feeds stderr back to Claude as the reason and blocks the stop, and the stop_hook_active field on
-// the next invocation's stdin payload tells us a forced-continuation already happened for this
-// turn (Claude sets it specifically so a hook can avoid re-blocking and trapping the session in a
-// loop). EMPIRICAL_HOOK_STOP_SOURCE above already reads stop_hook_active defensively but never
-// blocks, so on Claude the reminder was silently logged and never reached the model - this variant
-// is the same script with the loop guard actually load-bearing: block once per turn, never twice.
-// Codex/Copilot keep using the silent EMPIRICAL_HOOK_STOP_SOURCE until their own Stop/agentStop
-// blocking semantics are confirmed against source the way Claude's was here.
+// Claude-only variant of the Stop reminder above. hookSpecificOutput.additionalContext (stdout,
+// exit 0) is Claude Code's non-blocking Stop-hook feedback channel: the text is fed into context
+// and the conversation continues, same outcome the earlier exit-2 approach achieved, but the
+// transcript UI renders it as a plain system reminder instead of a red "Stop hook error" line.
+// EMPIRICAL_HOOK_STOP_SOURCE above still uses plain console.log + exit 0, which Claude Code treats
+// as non-blocking output that's logged but never reaches the model - that was this hook's original
+// bug (fixed for Claude here, unconfirmed/unfixed for Codex/Copilot's own Stop/agentStop semantics).
 const CLAUDE_HOOK_STOP_SOURCE = `#!/usr/bin/env node
 'use strict';
 
@@ -264,22 +261,25 @@ function main() {
   var payload = readStdinPayload();
 
   // stop_hook_active === true means Claude is re-invoking this hook because a PRIOR Stop hook
-  // (possibly this same one) already forced a continuation this turn. Blocking again here would
-  // trap the session in a forced-continuation loop, so this is the one case that must always exit
-  // 0 with no reminder, unconditionally.
+  // (possibly this same one) already surfaced additionalContext and forced a continuation this
+  // turn. additionalContext causes the same forced-continuation behavior decision:block does -
+  // confirmed empirically: without this guard the hook re-fired every ~2-3s in an infinite loop,
+  // since each additionalContext output triggers another Stop event. Must exit 0 with no output
+  // here, unconditionally, or the loop continues.
   if (payload && payload.stop_hook_active === true) {
-    appendLog('stop_hook_active=true, skipping reminder (already blocked once this turn)');
+    appendLog('stop_hook_active=true, skipping reminder (already surfaced once this turn)');
     process.exit(0);
   }
 
-  appendLog('stop hook fired (pid=' + process.pid + ', platform=' + process.platform + '), blocking once to surface reminder');
+  appendLog('stop hook fired (pid=' + process.pid + ', platform=' + process.platform + '), surfacing reminder via additionalContext');
 
-  // Exit 2 is Claude Code's documented Stop-hook blocking signal: stderr is fed back to Claude as
-  // context and the turn is forced to continue instead of ending, which is what actually gets this
-  // reminder read (a plain stdout console.log + exit 0 is only logged, never seen by the model -
-  // this was this hook's exact original bug).
-  process.stderr.write('[Empirical reminder] Before finishing: if this turn included a completed decision, a corrected mistake, or meaningful work worth remembering, record it now via the empirical-memory skill or \`empirical memory record\` - do not wait to be asked.\\n');
-  process.exit(2);
+  process.stdout.write(JSON.stringify({
+    hookSpecificOutput: {
+      hookEventName: 'Stop',
+      additionalContext: '[Empirical] Record memory-worthy work now via empirical-memory skill.'
+    }
+  }) + '\\n');
+  process.exit(0);
 }
 
 main();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@empirical-gauzza/cli",
-  "version": "1.0.11",
+  "version": "1.0.12",
   "description": "Empirical CLI installer and local MCP wiring helper",
   "type": "commonjs",
   "license": "MIT",