@polymorphism-tech/morph-spec 8.37.1 → 8.38.0

package/src/utils/hooks-installer.js

@@ -35,8 +35,25 @@ import { realEffects } from '../lib/installers/effects.js';
  * (junctions, port block, primary-root landing) and its removal routed
  * through the safe `worktree remove --if-clean` — and appends
  * block-raw-worktree-add to the PreToolUse:Bash chain.
+ *
+ * 3.9.0 binds telemetry-log to PostToolUseFailure (a Bash command that exits
+ * non-zero fires only that event, so red runs used to vanish from the log),
+ * PostToolBatch (once per model turn that used tools) and SubagentStop (the
+ * sub-agent's own token usage). The three only record, so they run with
+ * `async: true`: a node process per model turn never holds the turn up.
+ *
+ * 3.10.0 appends graph-refresh-kick to the all-tools PostToolUse chain: after a
+ * Bash or PowerShell command that runs git/gh/morph-spec it launches the
+ * detached refresh of the ROOT's codebase graph
+ * (framework/hooks/shared/graph-refresh.js). `update` re-runs
+ * this installer, so installed projects get the refresh without re-running
+ * `setup-infra` — the step that never reached them.
+ *
+ * 3.11.0 writes every `timeout` in seconds, the unit Claude Code reads: the
+ * values were milliseconds, so a hung hook held the session for hours (120000
+ * on WorktreeCreate is 33 h) instead of the seconds they meant.
  */
-const HOOKS_VERSION = '3.8.0';
+const HOOKS_VERSION = '3.11.0';
 
 /** Marker for old dispatch.js (v1) */
 const OLD_DISPATCH_COMMAND = 'node framework/hooks/agent-teams/dispatch.js';
@@ -119,7 +136,7 @@ const MORPH_HOOKS = [
     hooks: [{
       type: 'command',
       command: 'node framework/hooks/claude-code/core/worktree-context.js',
-      timeout: 5000
+      timeout: 5
     }]
   },
 
@@ -173,7 +190,7 @@ const MORPH_HOOKS = [
     hooks: [{
       type: 'command',
       command: 'node framework/hooks/claude-code/core/worktree-create.js',
-      timeout: 120000
+      timeout: 120
     }]
   },
 
@@ -188,7 +205,7 @@ const MORPH_HOOKS = [
     hooks: [{
       type: 'command',
       command: 'node framework/hooks/claude-code/core/worktree-remove.js',
-      timeout: 60000
+      timeout: 60
     }]
   },
 
@@ -227,24 +244,66 @@ const MORPH_HOOKS = [
       type: 'command',
       command: 'node framework/hooks/claude-code/core/chain.js',
       args: ['state-sync', 'gate-guard'],
-      timeout: 6000
+      timeout: 6
     }]
   },
 
-  // === PostToolUse: All — telemetry capture (silent) + loop detection.
-  // telemetry-log is the single generic vantage point of an NLH: it appends
-  // every tool call (skill, sub-agent dispatch, knowledge-base read, …) to
-  // .morph/logs/events.jsonl. Silent and fail-open — never blocks, never
-  // injects. `morph-spec telemetry` reads the stream back. Both links
-  // declared 5000ms; chain declares the sum (sequential worst case). ===
+  // === PostToolUse: All — telemetry capture (silent) + loop detection +
+  // graph refresh kick. telemetry-log is the single generic vantage point of an
+  // NLH: it appends every tool call (skill, sub-agent dispatch, knowledge-base
+  // read, …) to .morph/logs/events.jsonl. Silent and fail-open — never blocks,
+  // never injects. `morph-spec telemetry` reads the stream back.
+  // graph-refresh-kick only spawns a DETACHED worker (never waits on graphify),
+  // so it rides this chain instead of costing a process of its own per Bash
+  // call. Timeout stays the sum of the first two links (sequential worst case);
+  // the kick returns in milliseconds. ===
   {
     event: 'PostToolUse',
     matcher: null,
     hooks: [{
       type: 'command',
       command: 'node framework/hooks/claude-code/core/chain.js',
-      args: ['telemetry-log', 'loop-detect'],
-      timeout: 10000
+      args: ['telemetry-log', 'loop-detect', 'graph-refresh-kick'],
+      timeout: 10
+    }]
+  },
+
+  // === PostToolUseFailure: All — telemetry of the failed call (exit code,
+  // duration). loop-detect stays out: its warning is shaped for PostToolUse.
+  {
+    event: 'PostToolUseFailure',
+    matcher: null,
+    hooks: [{
+      type: 'command',
+      command: 'node framework/hooks/claude-code/core/telemetry-log.js',
+      timeout: 5,
+      async: true
+    }]
+  },
+
+  // === PostToolBatch — one event per model turn that used tools. The event
+  // takes no matcher. ===
+  {
+    event: 'PostToolBatch',
+    matcher: null,
+    hooks: [{
+      type: 'command',
+      command: 'node framework/hooks/claude-code/core/telemetry-log.js',
+      timeout: 5,
+      async: true
+    }]
+  },
+
+  // === SubagentStop — the sub-agent's token usage, read from its own
+  // transcript (the session transcript does not include it). ===
+  {
+    event: 'SubagentStop',
+    matcher: null,
+    hooks: [{
+      type: 'command',
+      command: 'node framework/hooks/claude-code/core/telemetry-log.js',
+      timeout: 10,
+      async: true
     }]
   },
 
@@ -258,7 +317,7 @@ const MORPH_HOOKS = [
     hooks: [{
       type: 'command',
       command: 'node framework/hooks/claude-code/core/telemetry-log.js',
-      timeout: 5000
+      timeout: 5
     }]
   },
 
@@ -338,7 +397,8 @@ export async function installClaudeHooks(targetPath, { fx = realEffects() } = {}
       return {
         type: h.type,
         command,
-        ...(h.timeout !== undefined ? { timeout: h.timeout } : {})
+        ...(h.timeout !== undefined ? { timeout: h.timeout } : {}),
+        ...(h.async ? { async: true } : {})
       };
     });