fix(hooks): Replace matcher追加 & deploy時のpermissions保持

[aloekun]

Summary

• PreToolUse / PostToolUse の matcher に Replace を追加し、ファイル全体置換ツール使用時もフックが発火するように修正
• deploy-hooks.js で既存 settings.local.json の hooks キーのみ上書きし、permissions 等の既存設定を保持するマージ方式に変更

Background

他プロジェクトへの hooks 配布・運用で判明した2つの課題を修正:

1. Replace ツールの抜け道: matcher に Replace がなく、拡張子が一致しないファイルを作成後に Replace で書き換えると linter フックをバイパスできた
2. permissions リセット問題: pnpm deploy:hooks 実行時に settings.local.json を丸ごと再生成していたため、ユーザーが許可したコマンド (permissions) が毎回消えていた

Changes

File	Change
.claude/settings.local.json.template	PreToolUse matcher: Bash|Write|Edit → Bash|Write|Edit|Replace PostToolUse matcher: Write|Edit → Write|Edit|Replace
scripts/deploy-hooks.js	既存ファイルがある場合は hooks キーのみ上書き、なければ新規生成

Test plan

• settings.local.json.template の matcher に Replace が含まれていること
• 既存 permissions を持つ対象プロジェクトに pnpm deploy:hooks を実行し、permissions が保持されること
• 既存 settings.local.json がない対象には従来通り新規生成されること

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

• 新機能

  • Hookのマッチングに新しい「Replace」タイプを追加しました。

• 改善

  • 設定デプロイ時に既存設定を保持し、フックのみを安全に更新するように改善しました（無効な既存設定は再生成）。
  • デプロイ時のログが「生成」「フック更新（権限保持）」「無効な設定のため再生成」を区別して出力するようになりました。

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 860d50d4-a07b-4ea2-af99-0160049a7542

📥 Commits

Reviewing files that changed from the base of the PR and between 76b0681 and 669aef3.

📒 Files selected for processing (1)

• scripts/deploy-hooks.js

🚧 Files skipped from review as they are similar to previous changes (1)

• scripts/deploy-hooks.js

---

📝 Walkthrough

ウォークスルー

Claude の hook マッチャーに Replace を追加し、デプロイスクリプトの deployTo() を修正してレンダリング済テンプレートの hooks を既存 settings.local.json があればそこにマージ、無効な既存ファイルは再生成するようにしました。

変更内容

コホート / ファイル	概要
Claude フック設定更新 ​.claude/settings.local.json.template	PreToolUse のマッチャーを Bash|Write|Edit|Replace に、PostToolUse を Write|Edit|Replace に更新し Replace をサポート。
デプロイスクリプトマージロジック scripts/deploy-hooks.js	deployTo() がレンダリング結果を JSON パースして hooks を検証。既存 settings.local.json が存在して有効な JSON の場合は既存内容を保持し hooks のみ newSettings.hooks に置換してファイルを書き戻す。既存が無効な JSON の場合は全体を再生成。ログメッセージを「generated」「updated (hooks merged, permissions preserved)」「WARN…regenerated due to invalid existing settings」に分ける。

シーケンス図

sequenceDiagram
    participant Deployer as Deployer (deployTo)
    participant Renderer as Template Renderer
    participant Parser as JSON Parser/Validator
    participant FS as File System (target settings.local.json)
    participant Logger as Logger

    Deployer->>Renderer: render settings.local.json.template -> renderedText
    Deployer->>Parser: parse renderedText -> newSettings
    Deployer->>FS: check if target file exists
    alt target exists
        FS->>Deployer: return fileContent
        Deployer->>Parser: try parse existing file
        alt parse succeeds
            Deployer->>Deployer: merge existing (preserve other keys) with newSettings.hooks
            Deployer->>FS: write merged JSON (preserve permissions)
            Deployer->>Logger: log "updated (hooks merged, permissions preserved)"
        else parse fails
            Deployer->>FS: write newSettings as full JSON
            Deployer->>Logger: log "WARN...regenerated due to invalid existing settings"
        end
    else target missing
        Deployer->>FS: write newSettings as full JSON
        Deployer->>Logger: log "generated"
    end

Loading

コード審査工数

🎯 3 (Moderate) | ⏱️ ~20 分

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	タイトルは日本語と英語が混在しており、記号や非標準的な表記を含んでいるため、可読性が低く曖昧である。	タイトルを明確で一貫性のある表記に統一してください。例：「fix: Replace matcher to hooks and preserve permissions on deploy」など、標準的な英語で記述することを推奨します。

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@scripts/deploy-hooks.js`:
- Around line 98-101: The code reads settingsDest into existing and blindly sets
existing.hooks = newSettings.hooks which breaks when existing is an array (e.g.,
[]) and prevents hooks from persisting; update the logic around the JSON.parse
result (existing) to perform a type guard: if existing is not a plain object,
replace it with an empty object before merging, then safely assign/merge
newSettings.hooks (preserving other properties/permissions) and write the
resulting object back to settingsDest; reference the variables existing,
newSettings.hooks and settingsDest when locating the change.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ec8b2b28-b3cb-4615-b52a-afe846dd9bd5

📥 Commits

Reviewing files that changed from the base of the PR and between a38e039 and 76b0681.

📒 Files selected for processing (2)

• .claude/settings.local.json.template
• scripts/deploy-hooks.js