Skip to content

Commit 0373a79

Browse files
committed
fix: correct argv index in denial block for node -e scripts
For `node -e "..." arg`, process.argv[1] is empty — the actual argument lands at process.argv[2]. This caused the denial reason to be undefined when blocking commits with dead exports.
1 parent 876e52a commit 0373a79

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

.claude/hooks/check-dead-exports.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ if [ -n "$DEAD_EXPORTS" ]; then
137137
hookSpecificOutput: {
138138
hookEventName: 'PreToolUse',
139139
permissionDecision: 'deny',
140-
permissionDecisionReason: process.argv[1]
140+
permissionDecisionReason: process.argv[2]
141141
}
142142
}));
143143
" "$REASON"

docs/examples/claude-code-hooks/check-dead-exports.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ if [ -n "$DEAD_EXPORTS" ]; then
137137
hookSpecificOutput: {
138138
hookEventName: 'PreToolUse',
139139
permissionDecision: 'deny',
140-
permissionDecisionReason: process.argv[1]
140+
permissionDecisionReason: process.argv[2]
141141
}
142142
}));
143143
" "$REASON"

0 commit comments

Comments
 (0)