fix(careful): hook 発火経路と output format を Claude Code 仕様に合わせる + 日本語化 + BSD sed 互換性

ToraDady · claude · ToraDady · commit bc67c8dedd1c · 2026-04-26T23:19:32.000+09:00
PR #24（step 28）の実機 integration テストで判明した問題を 4 件修正： 1. hook command path：`${CLAUDE_SKILL_DIR}` は Claude Code 公式 env var ではなく gstack 独自拡張だった（Claude Code は提供しない）。Claude Code 公式 `$CLAUDE_PROJECT_DIR` ベースの絶対 path に置換 → `$CLAUDE_PROJECT_DIR/.claude/skills/careful/bin/check-careful.sh`。これにより hook がそもそも script を見つけられない問題（`bash: /bin/check-careful.sh: No such file or directory`）を解消 2. hook output JSON format：gstack の `{"permissionDecision":"ask","message":"..."}` は旧 format。現行 Claude Code は `hookSpecificOutput` で wrap した format を要求 → `{"hookSpecificOutput":{"hookEventName":"PreToolUse","permissionDecision":"ask","permissionDecisionReason":"..."}}`。これにより hook は呼ばれるが警告が UI に表示されない問題を解消 3. WARN メッセージ 8 種を日本語化：「Destructive: ...」→「危険: ...」（uzustack の言語感に整合、SKILL.md.tmpl は日本語、statusMessage も日本語、warning だけ英語は不整合） 4. BSD sed 互換性：sed regex の `\s+` を `[[:space:]]+` に置換。macOS の BSD sed は `\s` 未対応で safe exception の RM_ARGS strip が失敗、結果として `rm -rf node_modules` が安全例外として通らなかった。POSIX の `[[:space:]]+` に置換して macOS / Linux 両対応に これらの finding は Phase 4（freeze + investigate hook 復活）翻訳でも再利用される設計知見として [[未決-16 hook 機構の発動経路の検証]] に記録予定。 実機検証済み（Mode A、uzustack repo 内 Claude Code）： - destructive コマンド → `[careful] 危険: 再帰削除 (rm -r)。ファイルを永続的に削除します。` 警告発火 - 安全例外（`./fake_dir/node_modules`） → `{}` 返却で標準 prompt - git reset --hard → `[careful] 危険: git reset --hard は未コミットの変更をすべて破棄します。` 警告 - 非 destructive → `{}` 返却 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
diff --git a/careful/SKILL.md b/careful/SKILL.md
@@ -22,7 +22,7 @@ hooks:
     - matcher: "Bash"
       hooks:
         - type: command
-          command: "bash ${CLAUDE_SKILL_DIR}/bin/check-careful.sh"
+          command: "bash $CLAUDE_PROJECT_DIR/.claude/skills/careful/bin/check-careful.sh"
           statusMessage: "危険コマンドをチェック中..."
 sensitive: true
 ---
diff --git a/careful/SKILL.md.tmpl b/careful/SKILL.md.tmpl
@@ -22,7 +22,7 @@ hooks:
     - matcher: "Bash"
       hooks:
         - type: command
-          command: "bash ${CLAUDE_SKILL_DIR}/bin/check-careful.sh"
+          command: "bash $CLAUDE_PROJECT_DIR/.claude/skills/careful/bin/check-careful.sh"
           statusMessage: "危険コマンドをチェック中..."
 sensitive: true
 ---
diff --git a/careful/bin/check-careful.sh b/careful/bin/check-careful.sh
@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 # check-careful.sh — PreToolUse hook for /careful skill
 # Reads JSON from stdin, checks Bash command for destructive patterns.
-# Returns {"permissionDecision":"ask","message":"..."} to warn, or {} to allow.
+# Returns hookSpecificOutput permissionDecision: "ask" to warn, or {} to allow.
 set -euo pipefail
 
 # Read stdin (JSON with tool_input)
@@ -28,7 +28,7 @@ CMD_LOWER=$(printf '%s' "$CMD" | tr '[:upper:]' '[:lower:]')
 # --- Check for safe exceptions (rm -rf of build artifacts) ---
 if printf '%s' "$CMD" | grep -qE 'rm\s+(-[a-zA-Z]*r[a-zA-Z]*\s+|--recursive\s+)' 2>/dev/null; then
   SAFE_ONLY=true
-  RM_ARGS=$(printf '%s' "$CMD" | sed -E 's/.*rm\s+(-[a-zA-Z]+\s+)*//;s/--recursive\s*//')
+  RM_ARGS=$(printf '%s' "$CMD" | sed -E 's/.*rm[[:space:]]+(-[a-zA-Z]+[[:space:]]+)*//;s/--recursive[[:space:]]*//')
   for target in $RM_ARGS; do
     case "$target" in
       */node_modules|node_modules|*/\.next|\.next|*/dist|dist|*/__pycache__|__pycache__|*/\.cache|\.cache|*/build|build|*/\.turbo|\.turbo|*/coverage|coverage)
@@ -53,56 +53,56 @@ PATTERN=""
 
 # rm -rf / rm -r / rm --recursive
 if printf '%s' "$CMD" | grep -qE 'rm\s+(-[a-zA-Z]*r|--recursive)' 2>/dev/null; then
-  WARN="Destructive: recursive delete (rm -r). This permanently removes files."
+  WARN="危険: 再帰削除 (rm -r)。ファイルを永続的に削除します。"
   PATTERN="rm_recursive"
 fi
 
 # DROP TABLE / DROP DATABASE
 if [ -z "$WARN" ] && printf '%s' "$CMD_LOWER" | grep -qE 'drop\s+(table|database)' 2>/dev/null; then
-  WARN="Destructive: SQL DROP detected. This permanently deletes database objects."
+  WARN="危険: SQL DROP を検出。データベースオブジェクトを永続的に削除します。"
   PATTERN="drop_table"
 fi
 
 # TRUNCATE
 if [ -z "$WARN" ] && printf '%s' "$CMD_LOWER" | grep -qE '\btruncate\b' 2>/dev/null; then
-  WARN="Destructive: SQL TRUNCATE detected. This deletes all rows from a table."
+  WARN="危険: SQL TRUNCATE を検出。テーブルの全行を削除します。"
   PATTERN="truncate"
 fi
 
 # git push --force / git push -f
 if [ -z "$WARN" ] && printf '%s' "$CMD" | grep -qE 'git\s+push\s+.*(-f\b|--force)' 2>/dev/null; then
-  WARN="Destructive: git force-push rewrites remote history. Other contributors may lose work."
+  WARN="危険: git force-push は remote の履歴を書き換えます。他の contributor の作業が失われる可能性があります。"
   PATTERN="git_force_push"
 fi
 
 # git reset --hard
 if [ -z "$WARN" ] && printf '%s' "$CMD" | grep -qE 'git\s+reset\s+--hard' 2>/dev/null; then
-  WARN="Destructive: git reset --hard discards all uncommitted changes."
+  WARN="危険: git reset --hard は未コミットの変更をすべて破棄します。"
   PATTERN="git_reset_hard"
 fi
 
 # git checkout . / git restore .
 if [ -z "$WARN" ] && printf '%s' "$CMD" | grep -qE 'git\s+(checkout|restore)\s+\.' 2>/dev/null; then
-  WARN="Destructive: discards all uncommitted changes in the working tree."
+  WARN="危険: ワーキングツリーの未コミット変更をすべて破棄します。"
   PATTERN="git_discard"
 fi
 
 # kubectl delete
 if [ -z "$WARN" ] && printf '%s' "$CMD" | grep -qE 'kubectl\s+delete' 2>/dev/null; then
-  WARN="Destructive: kubectl delete removes Kubernetes resources. May impact production."
+  WARN="危険: kubectl delete は Kubernetes リソースを削除します。プロダクションへの影響可能性あり。"
   PATTERN="kubectl_delete"
 fi
 
 # docker rm -f / docker system prune
 if [ -z "$WARN" ] && printf '%s' "$CMD" | grep -qE 'docker\s+(rm\s+-f|system\s+prune)' 2>/dev/null; then
-  WARN="Destructive: Docker force-remove or prune. May delete running containers or cached images."
+  WARN="危険: Docker force-remove または prune。実行中のコンテナや cached image を削除する可能性があります。"
   PATTERN="docker_destructive"
 fi
 
 # --- Output ---
 if [ -n "$WARN" ]; then
   WARN_ESCAPED=$(printf '%s' "$WARN" | sed 's/"/\\"/g')
-  printf '{"permissionDecision":"ask","message":"[careful] %s"}\n' "$WARN_ESCAPED"
+  printf '{"hookSpecificOutput":{"hookEventName":"PreToolUse","permissionDecision":"ask","permissionDecisionReason":"[careful] %s"}}\n' "$WARN_ESCAPED"
 else
   echo '{}'
 fi
