fix(hooks): file_size_check を default ON 化 + ADR-039 mechanical lint exception codify (PR #203 follow-up)

[aloekun]

Summary

PR #203 post-merge-feedback で発見した ADR-039 over-application を是正する PR。順位 177 file_size_check が ADR-039 § 1 機械適用で enabled = false (opt-in) になっていたが、決定論的 mechanical lint としては default ON で配布すべきだった (順位 147 file_length lint との asymmetry が判明)。ADR-039 に § 1.b mechanical lint 例外 を codify し、対象 lint を default ON に切替、本 PR の dogfood として todo10.md split も同時実施。

PR #203 の post-merge-feedback (順位 198 T3-1) も同 PR に bundle (ユーザー承認、イテレーション速度優先)。

変更内容 (5 commits、thematically coherent)

Commit	内容	Files
1	docs(todo) T3-1 採用反映	todo10.md, todo-summary.md
2	docs(adr-039) mechanical lint exception § 1.b 追加 + checklist 上位判定	adr-039-*.md
3	docs(adr-007) Layer 0.5 file_size_check 追記削除	adr-007-*.md
4	fix(hooks-config) file_size_check default ON + コメント表記修正	hooks-config.toml
5	docs(todo) todo10.md を todo12.md に分割 (50KB threshold 内に収める)	todo10.md (-377), todo12.md (new)
5b	docs(todo) 8 todo*.md preamble の「十二つ→十三つ」drift fix	todo3-9.md, todo11.md (cli-docs-lint 検出)

設計判断

ADR-039 § 1.b 例外条項 (commit 2)

「behavior の妥当性が不確定な experimental feature」と「決定論的 mechanical lint」を区別:

• § 1 対象: 挙動が dogfood で「失敗 / 却下 / 方向転換」されうるもの (post-merge-feedback / weekly-review / local-llm 等)
• § 1.b 対象 (default ON 許容): 4 条件を満たす機能
  1. 失敗 mode が non-blocking (additionalContext warning のみ)
  2. 判定が決定論的 (閾値 / regex / metadata)
  3. 影響範囲が宣言的に限定 (paths glob / extension)
  4. recovery hint が明確

順位 147 file_length lint / 順位 177 file_size_check は § 1.b 該当 (default ON で配布)。

checklist § 0 上位判定追加 (commit 2)

「そもそも § 1 適用対象か?」を最初に判定するゲートを追加。skip すると order-application 発生 (PR #197 で実観測)。

ADR-007 Layer 0.5 削除 (commit 3)

「Layer 0.5」概念は file_size_check 1 件のみのために導入したが、independent layer を設ける積極的理由がない。順位 177 を通常 custom linter の一つとして扱う方針 (ユーザー判断 2026-06-12)。今後 metadata-only check 追加は通常 ADR-007 判断フローに従う。

todo10.md split (commit 5、本 PR の dogfood)

commit 4 で file_size_check default ON 化したため、touched で 50KB 超の todo10.md (57KB) に warning が出る状態に。順位 177 設計の recovery flow (= 新 todo<N+1>.md 新設 + entry 移管) を実行し、todo10.md を 27KB まで縮小。8 entries (PR #185 〜 PR #196 era) を新規 docs/todo12.md に分離。

順位 198 への影響 (commit 1)

PR #203 post-merge-feedback Tier 3 #1 (Timestamp invariant safety ADR) を採用。analyzer は新規 entry 提案だったが、順位 198 (PR #199 T3-2) と同 ADR 提案のため、既存 entry の data point 強化として merge。

• 動機: 2 件観測 (Medium) → 3 件観測 (High) に Frequency 昇格
• 派生プロジェクト適用: "順位 197 で実装予定" → "PR feat(hooks-session-start): PastTime newtype + proptest 移植 (順位 197 / Bundle W 派生) #203 で実装済"
• 順位 194 (task 着手前 grep 確認 rule、PR Bundle 195-FB: jj revset hardcode lint rule + helper parameterize #196 採用) の初実践例

テスト結果

cargo test -p hooks-post-tool-linter file_size
test result: ok. 8 passed; 0 failed (既存 file_size_check tests 全 pass、regression なし)

cargo build --workspace
OK

takt pre-push-review (simplicity + security):
APPROVE (1 iteration、9m 10s)

File-length / file-size lint 連動効果

• main.rs の file-length lint (順位 147、Rust source 800 行) は本 PR 着手前から発火している既存 grandfather 状態 → 本 PR では touch なし
• file_size_check (順位 177、docs/src 50KB) は本 PR の commit 5 で default ON 化 + 同時に todo10.md split で初回 dogfood (= 設計通り threshold 内に収まる recovery flow が機能)

pr_size_check 1693 行 > 1500 block の override 根拠

PR_SIZE_CHECK_OVERRIDE=1 を適用 (順位 151 = PR #202 で land した PR diff lines check の規定 escape 経路):

• 5 commits = ADR rationale (§ 1.b) → ADR cleanup (§ Layer 0.5) → config enforcement (default ON) → todo dogfood (split) が thematically coherent
• ユーザー指示「同一 PR に含めることを許容、イテレーションを早く進める意図」(2026-06-12)
• 個別 PR 分割は scope creep + review iteration cost が増大

参照

• PR feat(hooks-session-start): PastTime newtype + proptest 移植 (順位 197 / Bundle W 派生) #203 (hooks-session-start PastTime + proptest 移植、本 PR のソース)
• .claude/feedback-reports/203.md Tier 3 feat(hooks): 設定駆動型アーキテクチャに移行し配布自動化を実装 #1 (順位 198 3 観測目昇格の出典)
• ADR-039 § 1.b — 本 PR で追加
• ADR-007 — Layer 0.5 追記を本 PR で削除
• 順位 147 file_length lint (src/hooks-post-tool-comment-lint-rust/src/main.rs:100) — § 1.b 該当 default ON の同類例

Test plan

• cargo test -p hooks-post-tool-linter file_size で 8 tests pass
• cargo build --workspace OK
• takt pre-push-review (simplicity + security) で APPROVE
• cli-docs-lint preamble drift 検出 → 8 todo*.md 修正で resolve
• CodeRabbit review で systemic 問題なしを確認
• file_size_check enable 後の next session で「新 Edit が >50KB ファイルに対し warning を出す」を実観測 (recovery flow dogfood)

Summary by CodeRabbit

• Documentation

  • アーキテクチャ決定記録（ADR）を更新し、ガイドラインと実装パターンを明確化しました。
  • プロジェクト管理ドキュメントを整理し、参照構造を統合しました。

• Chores

  • 内部設定を更新し、自動チェック機能の動作を調整しました。

[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: 4a09f7ce-4390-426f-b1b6-7e460b076975

📥 Commits

Reviewing files that changed from the base of the PR and between e2ae73e and f47e9ad.

📒 Files selected for processing (14)

• .claude/hooks-config.toml
• docs/adr/adr-007-custom-linter-layer-boundary.md
• docs/adr/adr-039-experimental-feature-standard-pattern.md
• docs/todo-summary.md
• docs/todo10.md
• docs/todo11.md
• docs/todo12.md
• docs/todo3.md
• docs/todo4.md
• docs/todo5.md
• docs/todo6.md
• docs/todo7.md
• docs/todo8.md
• docs/todo9.md

💤 Files with no reviewable changes (1)

• docs/adr/adr-007-custom-linter-layer-boundary.md

---

📝 Walkthrough

Walkthrough

ファイルサイズ機械チェックを opt-in（disabled）から default ON（enabled）に変更し、その判断根拠を ADR-007・ADR-039 で codify する。並行して、ファイルサイズ制限回避のため todo10.md を todo12.md に分割し、全 todo ファイルの参照を一括更新する。

Changes

File Size Check の有効化と ADR 整備

Layer / File(s)	Summary
file_size_check 設定の有効化 .claude/hooks-config.toml	[post_tool_use.file_size_check] の enabled = false を enabled = true に変更。非ブロッキング・決定論的・scope 限定・recovery hint 明確といった機械 lint 例外の性質をコメントで明記。
ADR-007, ADR-039 による policy 整備 docs/adr/adr-007-custom-linter-layer-boundary.md, docs/adr/adr-039-experimental-feature-standard-pattern.md	ADR-007 に「Layer 0.5」として、メタデータのみで行う file_size_check を正規表現層未満に位置付け。ADR-039 に §1.b 例外として「決定論的 mechanical lint は default ON 許容」を追加し、満たすべき 4 条件と self-review checklist の分岐ロジックを明文化。

Todo ドキュメント再構成と参照更新

Layer / File(s)	Summary
todo12 新規作成と todo10 の再構成 docs/todo12.md, docs/todo10.md, docs/todo-summary.md	todo12.md を新規作成し、PR #185-196 era の 8 つの active entries（fixture、invariant test、boundary test、markdown escape、raw JSON、重複検出、signature check、grep 追記）を記録。todo10.md は確認対象数を十二つから十三つに更新し、新セッションでの参照範囲を拡張。todo-summary.md はファイル位置付けに todo12 の收容順位レンジを追記。
todo3-9, todo11 の参照統一更新 docs/todo3.md, docs/todo4.md, docs/todo5.md, docs/todo6.md, docs/todo7.md, docs/todo8.md, docs/todo9.md, docs/todo11.md	各ファイルの冒頭「本ファイルの位置付け」で参照先を todo2-11.md → todo2-12.md へ統一し、確認対象数を「十二つ」→「十三つ」に一括更新。実務タスク内容は維持。

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#198: file_size_check を enabled=false から enabled=true へ切り替える本 PR に先行して、同一フックを enabled=false で導入した PR。
• aloekun/claude-code-hook-test#195: ADR-039 の experimental feature standard-pattern ルールと self-review checklist に関連する過去の変更。
• aloekun/claude-code-hook-test#161: ADR-007 の semantic self-limitation と minimal lint test-checklist 条件の基礎を確立した PR。

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PRのタイトルは、file_size_check をデフォルト ON 化し ADR-039 の mechanical lint 例外を codify するという主要な変更を明確に要約している。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

---

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.}