fix: detect kavex source repo in activate to use correct hook paths

ChiFungHillmanChan · claude · ChiFungHillmanChan · commit f7d1a89bc1ab · 2026-03-20T20:19:22.000Z
kavex activate was hardcoding .claude/hooks/ paths, which breaks when
running inside the kavex source repo where hooks live at hooks/. Now
detects the repo by checking for hooks at the project root and uses
"$CLAUDE_PROJECT_DIR"/hooks/ paths accordingly.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/scripts/kavex b/scripts/kavex
@@ -361,34 +361,46 @@ cmd_activate() {
     fi
   fi
 
-  # Define hooks config &mdash; references scripts installed by install.sh into .claude/hooks/
-  # Legacy install path; plugin mode uses hooks/hooks.json instead
+  # Detect hook path prefix:
+  # - Kavex source repo: hooks live at $PROJECT_ROOT/hooks/ -> use "$CLAUDE_PROJECT_DIR"/hooks/
+  # - Installed project: hooks live at $PROJECT_ROOT/.claude/hooks/ -> use .claude/hooks/
+  local hook_prefix
+  if [ -f "$PROJECT_ROOT/hooks/format.sh" ] && [ -f "$PROJECT_ROOT/hooks/block-dangerous.sh" ]; then
+    # We're inside the Kavex source repo &mdash; use $CLAUDE_PROJECT_DIR paths
+    # Escaped quotes for JSON embedding
+    hook_prefix='\"$CLAUDE_PROJECT_DIR\"/hooks'
+  else
+    # Legacy install &mdash; hooks copied into .claude/hooks/
+    hook_prefix='.claude/hooks'
+  fi
+
+  # Define hooks config using detected path prefix
   local hooks_json
-  hooks_json=$(cat <<'HOOKS'
+  hooks_json=$(cat <<HOOKS
 {
   "PreToolUse": [
     {
       "matcher": "Bash",
       "hooks": [
-        {"type": "command", "command": "cat | .claude/hooks/block-dangerous.sh", "timeout": 5},
-        {"type": "command", "command": "cat | .claude/hooks/kavex-commit-gate.sh", "timeout": 5}
+        {"type": "command", "command": "cat | ${hook_prefix}/block-dangerous.sh", "timeout": 5},
+        {"type": "command", "command": "cat | ${hook_prefix}/kavex-commit-gate.sh", "timeout": 5}
       ]
     },
     {
       "matcher": "Write|Edit|MultiEdit",
-      "hooks": [{"type": "command", "command": "cat | .claude/hooks/protect-files.sh", "timeout": 5}]
+      "hooks": [{"type": "command", "command": "cat | ${hook_prefix}/protect-files.sh", "timeout": 5}]
     }
   ],
   "PostToolUse": [
     {
       "matcher": "Write|Edit|MultiEdit",
-      "hooks": [{"type": "command", "command": "cat | .claude/hooks/format.sh", "timeout": 30}]
+      "hooks": [{"type": "command", "command": "cat | ${hook_prefix}/format.sh", "timeout": 30}]
     }
   ],
   "Stop": [
     {
       "matcher": "",
-      "hooks": [{"type": "command", "command": "cat | .claude/hooks/verify-on-stop.sh", "timeout": 60}]
+      "hooks": [{"type": "command", "command": "cat | ${hook_prefix}/verify-on-stop.sh", "timeout": 60}]
     }
   ]
 }

Original file line number	Diff line number	Diff line change
`@@ -361,34 +361,46 @@ cmd_activate() {`
`361`	`361`	`fi`
`362`	`362`	`fi`
`363`	`363`
`364`		`- # Define hooks config — references scripts installed by install.sh into .claude/hooks/`
`365`		`- # Legacy install path; plugin mode uses hooks/hooks.json instead`
	`364`	`+ # Detect hook path prefix:`
	`365`	`+ # - Kavex source repo: hooks live at $PROJECT_ROOT/hooks/ -> use "$CLAUDE_PROJECT_DIR"/hooks/`
	`366`	`+ # - Installed project: hooks live at $PROJECT_ROOT/.claude/hooks/ -> use .claude/hooks/`
	`367`	`+ local hook_prefix`
	`368`	`+ if [ -f "$PROJECT_ROOT/hooks/format.sh" ] && [ -f "$PROJECT_ROOT/hooks/block-dangerous.sh" ]; then`
	`369`	`+ # We're inside the Kavex source repo — use $CLAUDE_PROJECT_DIR paths`
	`370`	`+ # Escaped quotes for JSON embedding`
	`371`	`+ hook_prefix='\"$CLAUDE_PROJECT_DIR\"/hooks'`
	`372`	`+ else`
	`373`	`+ # Legacy install — hooks copied into .claude/hooks/`
	`374`	`+ hook_prefix='.claude/hooks'`
	`375`	`+ fi`
	`376`	`+`
	`377`	`+ # Define hooks config using detected path prefix`
`366`	`378`	`local hooks_json`
`367`		`- hooks_json=$(cat <<'HOOKS'`
	`379`	`+ hooks_json=$(cat <<HOOKS`
`368`	`380`	`{`
`369`	`381`	`"PreToolUse": [`
`370`	`382`	`{`
`371`	`383`	`"matcher": "Bash",`
`372`	`384`	`"hooks": [`
`373`		`- {"type": "command", "command": "cat \| .claude/hooks/block-dangerous.sh", "timeout": 5},`
`374`		`- {"type": "command", "command": "cat \| .claude/hooks/kavex-commit-gate.sh", "timeout": 5}`
	`385`	`+ {"type": "command", "command": "cat \| ${hook_prefix}/block-dangerous.sh", "timeout": 5},`
	`386`	`+ {"type": "command", "command": "cat \| ${hook_prefix}/kavex-commit-gate.sh", "timeout": 5}`
`375`	`387`	`]`
`376`	`388`	`},`
`377`	`389`	`{`
`378`	`390`	`"matcher": "Write\|Edit\|MultiEdit",`
`379`		`- "hooks": [{"type": "command", "command": "cat \| .claude/hooks/protect-files.sh", "timeout": 5}]`
	`391`	`+ "hooks": [{"type": "command", "command": "cat \| ${hook_prefix}/protect-files.sh", "timeout": 5}]`
`380`	`392`	`}`
`381`	`393`	`],`
`382`	`394`	`"PostToolUse": [`
`383`	`395`	`{`
`384`	`396`	`"matcher": "Write\|Edit\|MultiEdit",`
`385`		`- "hooks": [{"type": "command", "command": "cat \| .claude/hooks/format.sh", "timeout": 30}]`
	`397`	`+ "hooks": [{"type": "command", "command": "cat \| ${hook_prefix}/format.sh", "timeout": 30}]`
`386`	`398`	`}`
`387`	`399`	`],`
`388`	`400`	`"Stop": [`
`389`	`401`	`{`
`390`	`402`	`"matcher": "",`
`391`		`- "hooks": [{"type": "command", "command": "cat \| .claude/hooks/verify-on-stop.sh", "timeout": 60}]`
	`403`	`+ "hooks": [{"type": "command", "command": "cat \| ${hook_prefix}/verify-on-stop.sh", "timeout": 60}]`
`392`	`404`	`}`
`393`	`405`	`]`
`394`	`406`	`}`