oh-my-customcode 2.0.7 → 2.0.9

package/templates/.claude/hooks/scripts/bash-cd-advisor.sh

@@ -0,0 +1,172 @@
+#!/usr/bin/env bash
+# bash-cd-advisor.sh — PreToolUse(Bash): 서브셸 밖 최상위 `cd` 감지 (advisory)
+#
+# 배경 (#1834 / #1816 H2, R005 "Bash `cd`는 세션 cwd로 남습니다"):
+#   Bash 도구에서 괄호 밖에 놓인 `cd`는 세션 cwd로 남아(CwdChanged 발화로 확인) 이후 명령의
+#   상대 경로를 어긋나게 한다. `cd X && ( … )`도 `cd`가 서브셸 밖이므로 지속 `cd`다.
+#   안전한 형태는 절대 경로 또는 `( cd <dir> && … )`.
+#
+# 역할:
+#   .tool_input.command 에서 괄호 깊이 0, 따옴표/`$( … )`/백틱/히어독 본문 밖의 `cd`/`pushd`/`popd`
+#   명령어 단어를 찾으면 한국어 advisory 한 단락을 전달한다. 그 외에는 아무것도 출력하지 않는다.
+#   명령 시작 위치 = 문장 처음, `&&` `||` `;` 개행 `|` `&` 뒤, 그리고 접두 키워드(`then` `do` `else`
+#   `elif` `if` `while` `until` `{` `!` `time` `builtin` `command`)와 선행 `VAR=value` 할당 뒤.
+#   파이프의 마지막이 아닌 구간(`cd x | b`)과 백그라운드(`cd x &`)의 cd 는 서브셸에서 실행되어 cwd가
+#   남지 않으므로 감지하지 않는다. 파이프의 마지막 구간(`a | cd x`)은 감지한다: 이 규칙은 zsh 의미론을
+#   따른다 — Bash 도구의 셸이 zsh(측정: $ZSH_VERSION=5.9)이고 zsh 는 마지막 파이프 요소를 현재 셸에서
+#   실행한다(`zsh -fc 'cd /; echo x | cd /tmp; pwd'` → /tmp; bash 는 `/` 라 남지 않는다).
+#
+# 구현: 스캐너는 awk 한 번의 선형 패스(LC_ALL=C, 바이트 단위)다. 측정(macOS, awk 20200816): 30K자 0.03s.
+#   `cd`/`pushd`/`popd` 부분문자열이 없는 명령은 스캐너를 건너뛴다.
+#
+# 전달 경로 (Measured):
+#   CC 2.1.310, `claude -p`, haiku(claude-haiku-5-5), 1회 실행, PreToolUse(Bash) 훅.
+#   - exit 0 + stderr  : 모델에 도달하지 않음 (마커 미관측)
+#   - exit 0 + stdout JSON hookSpecificOutput.additionalContext : 모델에 도달
+#     ("PreToolUse:Bash hook additional context: <text>" 형태로 모델이 인용)
+#   따라서 additionalContext 로만 전달한다. 미측정: 대화형 세션, 다른 모델.
+#
+# 차단하지 않는다 (R021 advisory-first):
+#   항상 exit 0. permissionDecision/decision 필드를 출력하지 않고 tool input도 수정하지 않는다.
+#   fail-open: stdin이 객체가 아니거나 command 가 없거나 jq 가 없으면 조용히 exit 0.
+#
+# 환경변수 override:
+#   OMCUSTOM_CD_ADVISOR=off — advisory 비활성화
+#
+# 알려진 한계 (documents … 테스트로 고정):
+#   - `eval "cd x"`, `bash -c 'cd x'` 등 문자열 안의 cd 는 따옴표 안이라 감지하지 않는다.
+#   - 셸 함수 본문 `f() { cd x; }` 의 cd 는 `)` 뒤라 명령 시작이 아니어서 감지하지 않는다
+#     (`{ cd x; }` 그룹 자체는 감지한다).
+#   - `case $x in a) cd x ;; esac` 의 case arm 안 cd 는 감지하지 않는다(`)` 뒤).
+#   - 히어독 구분자 줄은 `<<[-]WORD` 의 WORD(따옴표 제거)와 일치하는 줄로 판정한다. 히어독 본문은
+#     건너뛰지만, 큰따옴표 문자열 안의 `$( … <<EOF … )` 형태는 큰따옴표 구간을 통째로 불투명하게 취급한다.
+#   - bash 로 실행되는 환경에서는 `ls | cd x` 가 cwd 를 바꾸지 않으나 zsh 기준으로 감지한다(위 참조).
+#   - `command -p cd x` 처럼 옵션이 낀 접두 형태는 감지하지 않는다.
+
+set -uo pipefail
+
+[ "${OMCUSTOM_CD_ADVISOR:-}" = "off" ] && exit 0
+command -v jq >/dev/null 2>&1 || exit 0
+
+input=$(cat) || exit 0
+cmd=$(printf '%s' "$input" | jq -r 'if type == "object" then (.tool_input.command // "" | if type == "string" then . else "" end) else "" end' 2>/dev/null) || exit 0
+[ -n "$cmd" ] || exit 0
+
+# 사전 필터: cwd 변경 명령어 부분문자열이 없으면 스캔하지 않는다.
+case "$cmd" in
+  *cd*|*pushd*|*popd*) ;;
+  *) exit 0 ;;
+esac
+
+# 선형 awk 스캐너: 따옴표 / 괄호 깊이(`(`, `$(` 포함) / 백틱 / 히어독 / 명령 시작 위치 / 구간 종결자 추적.
+# 종료코드 0 = 지속 cd 발견. 입력은 stdin 단일 레코드(RS=\001)로 받는다.
+# shellcheck disable=SC2016
+AWK_PROG='
+BEGIN { RS = "\001" }
+{ s = s $0 }
+function isdelim(c) { return (c == " " || c == "\t" || c == "\n" || c == ";" || c == "&" || c == "|" || c == "(" || c == ")" || c == "<" || c == ">" || c == "\"" || c == "\047" || c == "\\" || c == "`") }
+function scan(   n, ch, i, c, nx, pv, q, depth, bt, start, assign, cand, arith, akind, sep, adv, j, w, hn, hd, hdash, k, line, dq, d, dash) {
+  n = split(s, ch, "")
+  q = ""; depth = 0; bt = 0; start = 1; assign = 0; cand = 0; arith = 0; hn = 0
+  i = 1
+  while (i <= n) {
+    c = ch[i]
+    if (q == "\047") { if (c == "\047") q = ""; i++; continue }
+    if (q == "\"") {
+      if (c == "\\") { i += 2; continue }
+      if (c == "\"") q = ""
+      i++; continue
+    }
+    if (c == "\\") { i += 2; start = 0; continue }
+    if (c == "\047") { q = "\047"; start = 0; i++; continue }
+    if (c == "\"") { q = "\""; start = 0; i++; continue }
+    if (c == "`") { bt = 1 - bt; start = bt;  assign = 0; i++; continue }
+    if (c == "(") { depth++; if (i > 1 && ch[i - 1] == "(") { akind[depth] = 1; arith++ } start = 1; assign = 0; i++; continue }
+    if (c == ")") { if (akind[depth]) { akind[depth] = 0; arith-- } if (depth > 0) depth--; start = 0; assign = 0; i++; continue }
+    if (c == " " || c == "\t") { if (assign) { start = 1; assign = 0 } i++; continue }
+    if (c == "<") {
+      nx = ch[i + 1]
+      if (nx == "<") {
+        if (ch[i + 2] == "<") { i += 3; start = 0; continue }
+        if (arith > 0) { i += 2; start = 0; continue }
+        i += 2; dash = 0
+        if (ch[i] == "-") { dash = 1; i++ }
+        while (i <= n && (ch[i] == " " || ch[i] == "\t")) i++
+        dq = ""
+        while (i <= n) {
+          d = ch[i]
+          if (d == "\047" || d == "\"" || d == "\\") { i++; continue }
+          if (d == " " || d == "\t" || d == "\n" || d == ";" || d == "&" || d == "|" || d == "(" || d == ")" || d == "<" || d == ">") break
+          dq = dq d; i++
+        }
+        hn++; hd[hn] = dq; hdash[hn] = dash
+        start = 0; continue
+      }
+      start = 0; i++; continue
+    }
+    sep = ""; adv = 1
+    if (c == ";" || c == "\n") sep = "term"
+    else if (c == "|") {
+      nx = ch[i + 1]
+      if (nx == "|") { sep = "term"; adv = 2 }
+      else if (nx == "&") { sep = "pipe"; adv = 2 }
+      else sep = "pipe"
+    } else if (c == "&") {
+      nx = ch[i + 1]; pv = (i > 1) ? ch[i - 1] : ""
+      if (nx == "&") { sep = "term"; adv = 2 }
+      else if (pv == ">" || pv == "<" || nx == ">") { start = 0; i++; continue }
+      else sep = "bg"
+    }
+    if (sep != "") {
+      if (cand) { if (sep == "term") return 1; cand = 0 }
+      start = 1; assign = 0
+      i += adv
+      if (c == "\n" && hn > 0) {
+        for (k = 1; k <= hn; k++) {
+          while (i <= n) {
+            line = ""
+            while (i <= n && ch[i] != "\n") { line = line ch[i]; i++ }
+            i++
+            if (hdash[k]) sub(/^\t+/, "", line)
+            if (line == hd[k]) break
+          }
+        }
+        hn = 0
+      }
+      continue
+    }
+    if (c == "#" && start == 1) {
+      while (i <= n && ch[i] != "\n") i++
+      continue
+    }
+    if (start == 1) {
+      j = i; w = ""
+      while (j <= n && !isdelim(ch[j])) { w = w ch[j]; j++ }
+      if (j == i) { start = 0; i++; continue }
+      if ((w == "cd" || w == "pushd" || w == "popd") && depth == 0 && bt == 0 && !cand) {
+        cand = 1; start = 0; i = j; continue
+      }
+      if (w == "then" || w == "do" || w == "else" || w == "elif" || w == "if" || w == "while" || w == "until" || w == "time" || w == "builtin" || w == "command" || w == "!" || w == "{") {
+        i = j; continue
+      }
+      if (w ~ /^[A-Za-z_][A-Za-z0-9_]*=/) { assign = 1; start = 0; i = j; continue }
+      start = 0; i = j; continue
+    }
+    i++
+  }
+  return cand ? 1 : 0
+}
+END { exit (scan() ? 0 : 1) }
+'
+
+has_toplevel_cd() {
+  printf '%s' "$1" | LC_ALL=C awk "$AWK_PROG" 2>/dev/null
+}
+
+if has_toplevel_cd "$cmd"; then
+  advisory_text='[CD Advisory] 이 Bash 명령에는 서브셸 밖에 놓인 최상위 `cd`(또는 `pushd`/`popd`)가 있습니다. Bash 도구의 `cd`는 세션 cwd로 남아 이후 명령의 상대 경로를 어긋나게 합니다(`cd X && ( … )` 형태도 `cd`가 괄호 밖이므로 동일합니다). 절대 경로를 쓰거나 `( cd <dir> && … )`처럼 서브셸 안에서만 이동하십시오(R005). 이미 의도한 지속 이동이라면 이후 명령의 경로가 이를 전제하는지 확인하십시오.'
+  jq -cn --arg ctx "$advisory_text" \
+    '{hookSpecificOutput: {hookEventName: "PreToolUse", additionalContext: $ctx}}'
+fi
+
+exit 0

package/templates/.claude/hooks/scripts/content-hash-validator.sh

@@ -1,11 +1,22 @@
 #!/bin/bash
 # Content-Hash Validator — Detect stale file state for Edit operations
-# Trigger: PostToolUse on Read (stores hashes), PreToolUse on Edit (validates)
-# Purpose: Advisory warning when file content changed between Read and Edit
-# Protocol: stdin JSON -> validate -> stdout pass-through
+# Trigger: PostToolUse on Read/Edit/Write (stores or refreshes the hash), PreToolUse on Edit (validates)
+# Purpose: Advisory warning when file content changed outside the model's own Read/Edit/Write
+# Protocol (#1885, per event):
+#   PostToolUse (Read|Edit|Write): store/refresh hash of .tool_input.file_path, stdin pass-through to stdout.
+#   PreToolUse (Edit): validate; stdout is one JSON line (hookSpecificOutput.additionalContext) or nothing.
+#     No stderr, no passthrough (exit-0 stderr never reaches the model; measured CC 2.1.310, #1775).
+#   Missing hook_event_name: Read and Write store, Edit VALIDATES (Edit was historically PreToolUse-only).
+# Store: per-path atomic files in /tmp/.claude-content-hashes-${PPID}.d/<cksum of path>
+#   (line 1 = hash, line 2 = path). Each entry is written to a unique temp file in the same dir, then
+#   mv -f'd onto the entry, so parallel hook calls never share a read-modify-write file. No ring buffer:
+#   the directory is session-scoped (PPID) and entries are tiny. Every state operation is wrapped so a
+#   failure never aborts or leaks stderr.
+# Known limitation: a file rewritten after its hash was stored (a parallel PostToolUse formatter hook such
+#   as prettier/gofmt/ruff, or a Bash edit) can still trigger the warning on the next Edit.
 # Always exits 0 (advisory only, never blocks)
 
-set -euo pipefail
+set -uo pipefail
 
 # Dependency check: exit silently if jq not available
 command -v jq >/dev/null 2>&1 || exit 0
@@ -13,67 +24,73 @@ command -v jq >/dev/null 2>&1 || exit 0
 input=$(cat)
 
 # Guard: non-object stdin (non-JSON / JSON array / empty) — swallow and exit 0.
-# Hooks must never crash (R021); jq parse errors would otherwise abort under `set -e`. (#1650)
+# Hooks must never crash (R021). (#1650)
 printf '%s' "$input" | jq -e 'type=="object"' >/dev/null 2>&1 || exit 0
 
-# Hash store (PPID-scoped, session-only)
-HASH_STORE="/tmp/.claude-content-hashes-${PPID}"
-
-tool_name=$(printf '%s\n' "$input" | jq -r '.tool_name // "unknown"')
+STORE_DIR="/tmp/.claude-content-hashes-${PPID}.d"
+
+ctx=""
+add() { if [ -z "$ctx" ]; then ctx="$1"; else ctx="${ctx}"$'\n'"$1"; fi; }
+
+tool_name=$(printf '%s\n' "$input" | jq -r '.tool_name // "unknown"' 2>/dev/null || echo unknown)
+event=$(printf '%s\n' "$input" | jq -r '.hook_event_name // ""' 2>/dev/null || echo "")
+file_path=$(printf '%s\n' "$input" | jq -r '.tool_input.file_path? // ""' 2>/dev/null || echo "")
+
+mode="other"
+if [ "$event" = "PreToolUse" ]; then
+  [ "$tool_name" = "Edit" ] && mode="validate"
+elif [ "$event" = "" ]; then
+  case "$tool_name" in
+    Edit) mode="validate" ;;
+    Read|Write) mode="store" ;;
+  esac
+else
+  case "$tool_name" in Read|Edit|Write) mode="store" ;; esac
+fi
 
-case "$tool_name" in
-  "Read")
-    # Store content hash for the file that was just read
-    file_path=$(printf '%s\n' "$input" | jq -r '.tool_input.file_path? // ""')
+file_hash() { md5 -q "$1" 2>/dev/null || md5sum "$1" 2>/dev/null | cut -d' ' -f1 || echo "unknown"; }
+entry_for() { printf '%s' "$1" | cksum | cut -d' ' -f1; }
 
+case "$mode" in
+  "store")
     if [ -n "$file_path" ] && [ -f "$file_path" ]; then
-      content_hash=$(md5 -q "$file_path" 2>/dev/null || md5sum "$file_path" 2>/dev/null | cut -d' ' -f1 || echo "unknown")
-      timestamp=$(date -u +%Y-%m-%dT%H:%M:%SZ)
-
-      # Store hash entry (overwrite previous for same file)
-      if [ -f "$HASH_STORE" ]; then
-        # Remove old entry for this file
-        grep -v "\"path\":\"${file_path}\"" "$HASH_STORE" > "${HASH_STORE}.tmp" 2>/dev/null || true
-        mv "${HASH_STORE}.tmp" "$HASH_STORE" 2>/dev/null || true
-      fi
-
-      jq -cn \
-        --arg path "$file_path" \
-        --arg hash "$content_hash" \
-        --arg ts "$timestamp" \
-        '{path: $path, hash: $hash, stored_at: $ts}' >> "$HASH_STORE" 2>/dev/null || true
+      {
+        content_hash=$(file_hash "$file_path")
+        mkdir -p "$STORE_DIR"
+        tmp=$(mktemp "$STORE_DIR/.tmp.XXXXXX")
+        printf '%s\n%s\n' "$content_hash" "$file_path" > "$tmp"
+        mv -f "$tmp" "$STORE_DIR/$(entry_for "$file_path")"
+      } 2>/dev/null || true
     fi
     ;;
 
-  "Edit")
-    # Validate that file hasn't changed since last Read
-    file_path=$(printf '%s\n' "$input" | jq -r '.tool_input.file_path? // ""')
-
-    if [ -n "$file_path" ] && [ -f "$HASH_STORE" ] && [ -f "$file_path" ]; then
-      stored_hash=$(grep "\"path\":\"${file_path}\"" "$HASH_STORE" 2>/dev/null | tail -1 | jq -r '.hash // ""' 2>/dev/null || echo "")
+  "validate")
+    if [ -n "$file_path" ] && [ -f "$file_path" ]; then
+      entry="$STORE_DIR/$(entry_for "$file_path")"
+      stored_hash=""
+      stored_path=""
+      { stored_hash=$(sed -n 1p "$entry"); stored_path=$(sed -n 2p "$entry"); } 2>/dev/null || true
 
-      if [ -n "$stored_hash" ] && [ "$stored_hash" != "unknown" ]; then
-        current_hash=$(md5 -q "$file_path" 2>/dev/null || md5sum "$file_path" 2>/dev/null | cut -d' ' -f1 || echo "unknown")
+      if [ -n "$stored_hash" ] && [ "$stored_hash" != "unknown" ] && [ "$stored_path" = "$file_path" ]; then
+        current_hash=$(file_hash "$file_path")
 
         if [ "$current_hash" != "unknown" ] && [ "$stored_hash" != "$current_hash" ]; then
-          echo "[Content-Hash] WARNING: $(basename "$file_path") may have changed since last Read" >&2
-          echo "[Content-Hash] Stored hash: ${stored_hash:0:8}... Current: ${current_hash:0:8}..." >&2
-          echo "[Content-Hash] Advisory: re-read the file before editing if unsure" >&2
+          add "[Content-Hash] WARNING: $(basename "$file_path") may have changed since last Read"
+          add "[Content-Hash] Stored hash: ${stored_hash:0:8}... Current: ${current_hash:0:8}..."
+          add "[Content-Hash] Advisory: re-read the file before editing if unsure"
         fi
       fi
     fi
     ;;
 esac
 
-# Ring buffer: keep last 200 entries
-if [ -f "$HASH_STORE" ]; then
-  line_count=$(wc -l < "$HASH_STORE" 2>/dev/null || echo "0")
-  if [ "$line_count" -gt 200 ]; then
-    tail -200 "$HASH_STORE" > "${HASH_STORE}.tmp"
-    mv "${HASH_STORE}.tmp" "$HASH_STORE"
+if [ "$mode" = "validate" ]; then
+  # Emit advisory to the model (#1885); silent when nothing to advise.
+  if [ -n "$ctx" ]; then
+    jq -cn --arg ctx "$ctx" '{hookSpecificOutput:{hookEventName:"PreToolUse",additionalContext:$ctx}}'
   fi
+else
+  # PostToolUse paths and others: unchanged pass-through
+  printf '%s\n' "$input"
 fi
-
-# Always pass through
-printf '%s\n' "$input"
 exit 0

package/templates/.claude/hooks/scripts/destructive-git-guard.sh

@@ -1,7 +1,9 @@
 #!/bin/bash
 # R001/R021 destructive git guard hook
 # Warns before commands that can discard working tree, untracked files, or branches.
-# Advisory tier only: emits stderr guidance, records PPID-scoped warnings, exits 0.
+# Advisory tier only: emits guidance, records PPID-scoped warnings, exits 0.
+# Channel (#1885): exit-0 stderr never reaches the model; advice is delivered via stdout JSON
+# hookSpecificOutput.additionalContext (measured CC 2.1.310, #1775). No stdin passthrough, no stderr.
 #
 # PPID Scoping Convention:
 #   Session-scoped temp files MUST use $PPID (Claude Code parent PID), not $$.
@@ -26,35 +28,48 @@ fi
 
 VIOLATION_FILE="/tmp/.claude-destructive-git-guard-${PPID}"
 matched=""
+ctx=""
+# Optional `-C <path>` / `-c <k=v>` option pairs between `git` and the subcommand (L10).
+GIT_PFX='(^|[;&|[:space:]])git([[:space:]]+(-C[[:space:]]+("[^"]*"|'"'"'[^'"'"']*'"'"'|[^[:space:]]+)|-c[[:space:]]+[^[:space:]]+))*[[:space:]]+'
+add() { if [ -z "$ctx" ]; then ctx="$1"; else ctx="${ctx}"$'\n'"$1"; fi; }
 
-if printf '%s\n' "$command_text" | grep -Eq '(^|[;&|[:space:]])git[[:space:]]+reset[[:space:]]+--hard([[:space:]]|$)'; then
+if printf '%s\n' "$command_text" | grep -Eq "${GIT_PFX}reset[[:space:]]+--hard([[:space:]]|$)"; then
   matched="git reset --hard"
-elif printf '%s\n' "$command_text" | grep -Eq '(^|[;&|[:space:]])git[[:space:]]+clean[[:space:]]+([^;&|]*[[:space:]])?-f(dx|xd|d|x)?([[:space:]]|$)'; then
+elif printf '%s\n' "$command_text" | grep -Eq "${GIT_PFX}clean[[:space:]]+([^;&|]*[[:space:]])?(-[A-Za-z]*f[A-Za-z]*|--force)([[:space:]]|$)"; then
   matched="git clean"
-elif printf '%s\n' "$command_text" | grep -Eq '(^|[;&|[:space:]])git[[:space:]]+restore([[:space:]]+--[^;&|[:space:]]+)*([[:space:]]+(\.|\*|/|--worktree|--staged))*([[:space:]]|$)'; then
+elif printf '%s\n' "$command_text" | grep -Eq "${GIT_PFX}restore([[:space:]]+--[^;&|[:space:]]+)*([[:space:]]+(\.|\*|/|--worktree|--staged))*([[:space:]]|$)"; then
   matched="git restore"
-elif printf '%s\n' "$command_text" | grep -Eq '(^|[;&|[:space:]])git[[:space:]]+checkout[[:space:]]+--[[:space:]]+(\.|\*|/)([[:space:]]|$)'; then
+elif printf '%s\n' "$command_text" | grep -Eq "${GIT_PFX}checkout[[:space:]]+--[[:space:]]+(\.|\*|/)([[:space:]]|$)"; then
   matched="git checkout -- ."
-elif printf '%s\n' "$command_text" | grep -Eq '(^|[;&|[:space:]])git[[:space:]]+branch[[:space:]]+-D([[:space:]]|$)'; then
+elif printf '%s\n' "$command_text" | grep -Eq "${GIT_PFX}branch[[:space:]]+-D([[:space:]]|$)"; then
   matched="git branch -D"
 fi
 
 if [ -n "$matched" ]; then
-  echo "[Hook] DESTRUCTIVE GIT WARNING: detected '$matched' in Bash command" >&2
-  echo "[Hook] Advisory only (R021): command is not blocked, but it may discard local work." >&2
-  echo "[Hook] Before proceeding, verify git status and preserve work with git stash or a WIP commit." >&2
-  echo "[Hook] Recovery: use 'git reflog' to find prior HEADs; for deleted branches, check 'git reflog --all'." >&2
+  add "[Hook] DESTRUCTIVE GIT WARNING: detected '$matched' in Bash command"
+  add "[Hook] Advisory only (R021): command is not blocked, but it may discard local work."
+  add "[Hook] Before proceeding, verify git status and preserve work with git stash or a WIP commit."
+  add "[Hook] Recovery: use 'git reflog' to find prior HEADs; for deleted branches, check 'git reflog --all'."
 
-  if printf '%s\n' "$command_text" | grep -Eq '(^|[;&|[:space:]])git[[:space:]]+branch[[:space:]]+-D([[:space:]]|$)'; then
-    echo "[Hook] Branch deletion note: prefer 'git branch -d' first so Git checks for unmerged commits." >&2
+  if printf '%s\n' "$command_text" | grep -Eq "${GIT_PFX}branch[[:space:]]+-D([[:space:]]|$)"; then
+    add "[Hook] Branch deletion note: prefer 'git branch -d' first so Git checks for unmerged commits."
   fi
 
-  printf '%s %s\n' "$(date -u +%Y-%m-%dT%H:%M:%SZ)" "$matched" >> "$VIOLATION_FILE"
+  # State writes are silent and non-fatal (no stderr, no abort).
+  { printf '%s %s\n' "$(date -u +%Y-%m-%dT%H:%M:%SZ)" "$matched" >> "$VIOLATION_FILE"; } 2>/dev/null || true
   violation_count=$(wc -l < "$VIOLATION_FILE" 2>/dev/null | tr -d ' ')
-  if [ "${violation_count:-0}" -ge 3 ]; then
-    echo "[Hook] Destructive git warnings: ${violation_count} this session — R021 promotion threshold reached" >&2
+  # Count line once per session: count >= 3 AND the atomic marker mkdir wins (parallel-safe).
+  if [ "${violation_count:-0}" -ge 3 ] && { mkdir "/tmp/.claude-destructive-git-threshold-${PPID}"; } 2>/dev/null; then
+    add "[Hook] Destructive git warnings: ${violation_count} this session (advisory telemetry; R021 promotion needs user approval)"
   fi
 fi
 
-# Always pass through - this hook is advisory only.
-printf '%s\n' "$input"
+# Emit advisory to the model (#1885); silent when nothing matched.
+if [ -n "$ctx" ]; then
+  if command -v jq >/dev/null 2>&1; then
+    jq -cn --arg ctx "$ctx" '{hookSpecificOutput:{hookEventName:"PreToolUse",additionalContext:$ctx}}'
+  elif command -v python3 >/dev/null 2>&1; then
+    CTX="$ctx" python3 -c 'import json,os; print(json.dumps({"hookSpecificOutput":{"hookEventName":"PreToolUse","additionalContext":os.environ["CTX"]}}))'
+  fi
+fi
+exit 0

package/templates/.claude/hooks/scripts/fail-axis-cause-advisor.sh

@@ -9,7 +9,7 @@
 #
 # 역할:
 #   (1) 짧은 진행 지시이고 (2) 원인 언급이 없으며 (3) 이 세션에 기록된 도구 실패가 있을 때,
-#   Claude에게 "진행 전에 사용자에게 원인 가설 한 줄을 되물어라"는 advisory를 전달한다.
+#   Claude에게 "재시도 전에 원인 가설 한 줄을 명시하고, 사용자가 묻지 말라고 지시하지 않았을 때만 확인하라"는 advisory를 전달한다 (#1834: 훅은 세션 지시를 볼 수 없다).
 #
 # 왜 사용자가 아니라 Claude에게 전달하는가:
 #   hookSpecificOutput.additionalContext는 모델 컨텍스트로만 들어간다(사용자에게 표시되지 않음).
@@ -121,7 +121,7 @@ fail_tools=$(tail -n 300 "$LEDGER" 2>/dev/null \
   | sort | uniq -c | sort -rn | head -3 \
   | awk '{printf "%s(%s) ", $2, $1}') || fail_tools=""
 
-advisory_text=$(printf '[FAIL Advisory] 이 세션에 도구 실패 %s건이 기록되어 있습니다 (%s). 방금 입력은 원인 언급이 없는 진행 지시입니다. 곧바로 재시도하지 말고, 먼저 직전 실패의 원인 가설을 한 줄로 제시한 뒤 사용자에게 "이 진단이 맞는지 / 다른 원인이 짚이는지" 짧게 한 번만 확인하십시오. 사용자가 답하면 그대로 진행합니다. 이 확인은 한 턴을 넘기지 마십시오.' \
+advisory_text=$(printf '[FAIL Advisory] 이 세션에 도구 실패 %s건이 기록되어 있습니다 (%s). 방금 입력은 원인 언급이 없는 진행 지시입니다. 곧바로 재시도하지 말고, 먼저 직전 실패의 원인 가설을 한 줄로 명시하십시오. 사용자가 묻지 말고 진행하라고 지시하지 않은 경우에만 "이 진단이 맞는지" 짧게 한 번 확인하고, 그런 지시가 있었다면 확인 없이 가설만 기록하고 진행하십시오.' \
   "$fail_count" "${fail_tools:-unknown}")
 
 # 사람이 보는 감사 추적용 (exit 0에서 stderr는 모델에 전달되지 않음)

package/templates/.claude/hooks/scripts/failure-ledger.sh

@@ -18,6 +18,15 @@
 #   - 어떤 실패에도 exit 0 — 원장 기록 실패가 본 작업을 막아서는 안 된다 (R021 advisory-first).
 #   - 명령/에러 문자열은 절단하여 기록한다 (원장 비대화 방지).
 #
+# exit_code 필드 (#1834 / #1814 R2):
+#   PostToolUseFailure에는 숫자 exit code 필드가 없고 `error` 문자열("Exit code 3")만 온다
+#   (실측: CC 2.1.310, `claude -p`, haiku, 1회, `sh -c 'exit 3'`). 그래서 `^Exit code ([0-9]+)`
+#   일 때만 정수를 파싱해 exit_code로 기록하고, 아니면 null이다.
+#   기대 exit 화이트리스트는 두지 않는다 — 동기였던 `git grep` no-match(exit 1)는 같은
+#   조건에서 PostToolUseFailure를 발생시키지 않고, returnCodeInterpretation "No matches found"를
+#   가진 PostToolUse로만 도달했다(실측). `false`, 일반 `grep` 등 다른 exit-1 명령, 대화형 세션,
+#   다른 모델/버전은 미측정이다.
+#
 # 환경변수 override:
 #   OMCUSTOM_FAILURE_LEDGER=off   — 기록 완전 비활성화
 #   OMCUSTOM_ERROR_LEDGER=<path>  — 원장 경로 override (기본: ~/.claude/error-ledger.jsonl)
@@ -36,6 +45,14 @@ if ! command -v jq >/dev/null 2>&1; then
   exit 0
 fi
 
+# HOME guard: with no override, skip when HOME is unset, empty or not absolute
+# (otherwise `set -u` exits 1 on an unset HOME, or the ledger lands in a relative path).
+if [ -z "${OMCUSTOM_ERROR_LEDGER:-}" ]; then
+  case "${HOME:-}" in
+    /*) ;;
+    *) exit 0 ;;
+  esac
+fi
 LEDGER="${OMCUSTOM_ERROR_LEDGER:-${HOME}/.claude/error-ledger.jsonl}"
 
 if ! mkdir -p "$(dirname "$LEDGER")" 2>/dev/null; then
@@ -68,6 +85,7 @@ printf '%s' "$input" \
                     then (.tool_input.command // .tool_input.file_path // "")
                     else "" end) | tostring | .[0:160]),
         interrupt: (.is_interrupt == true),
+        exit_code: ((.error | if type == "string" then (capture("^Exit code (?<c>[0-9]+)") | .c | tonumber) else null end) // null),
         err:     ((.error
                    // .tool_error
                    // (if (.tool_response | type) == "object"

package/templates/.claude/hooks/scripts/file-change-validator.sh

@@ -4,7 +4,7 @@
 
 input=$(cat)
 file_path=$(printf '%s\n' "$input" | jq -r '.file_path // ""' 2>/dev/null)
-change_type=$(printf '%s\n' "$input" | jq -r '.change_type // "modified"' 2>/dev/null)
+change_type=$(printf '%s\n' "$input" | jq -r '.event // .change_type // "modified"' 2>/dev/null)
 
 if [ -z "$file_path" ]; then
   printf '%s\n' "$input"
@@ -14,11 +14,14 @@ fi
 echo "[Hook] External file change detected: ${change_type} ${file_path}" >&2
 
 # Warn about important files
+# Arms mirror the hooks.json FileChanged matcher (literal working-directory filenames).
+# Nested paths such as .claude/hooks/hooks.json or .claude/settings*.json cannot be matched
+# (the matcher only registers literal filenames in the working directory), so no arm for them.
 case "$file_path" in
-  */CLAUDE.md|*/hooks.json|*/settings*.json)
+  CLAUDE.md|*/CLAUDE.md)
     echo "[Hook] WARNING: Configuration file changed externally — re-read recommended" >&2
     ;;
-  *.lock|*lockfile*)
+  package-lock.json|*/package-lock.json|pnpm-lock.yaml|*/pnpm-lock.yaml|*.lock)
     echo "[Hook] Lock file changed — dependency state may have shifted" >&2
     ;;
 esac