feat(weekly-review): file_length scan を pre-LLM step として追加 (PR-W0) + 順位 222 採用

[aloekun]

Summary

docs/file-length-enforcement-plan.md (PR #218 で land 済) で計画した PR-W0 (Weekly audit / E) を実装する。ADR-031 weekly-review pipeline に deterministic な file_length scan を pre-LLM step として追加 し、800 行超 file の現状を毎週 1 回機械的に可視化する。

順位 147 (file_length lint、PR #202) は hooks-post-tool-comment-lint-rust の PostToolUse hook として実装されており、触られた file の編集時のみ additionalContext で 800 行超を警告する設計 (soft-nag、touch-trigger ratchet)。このため未触り state の violation は警告されず、PR-3a (#217) 時点で 7 件の累積を許す結果となった。

本 PR はその構造的盲点を「週次 audit」で補う最初の機構。順位 147 の touch-trigger と本 step の whole-tree scan が補完関係で 2 層防御を形成する。

変更内容

新 facet file-length-watchlist

.takt/workflows/weekly-review.yaml の reviewers parallel step に追加 (4 番目の parallel facet)。

• type: LLM facet (takt は command-type step を持たないため LLM facet を deterministic instruction で wrap)
• persona: simplicity-reviewer (file length は SRP / simplicity 観点と整合)
• model: haiku (cheap、command 実行 + 整形のみ)
• policy: review (read-only、既存 reviewer と一貫)
• allowed_tools: Bash + Read (find/wc/awk 実行用)
• instruction: file-length-watchlist (新規)
• output: file-length-watchlist.md (Report Directory)
• completion: analysis complete (step-level all("analysis complete") rule と整合)

新 instruction .takt/facets/instructions/file-length-watchlist.md

決定論的 shell command を実行し、出力を markdown table に整形する deterministic instruction:

find src -name '*.rs' -not -path '*/target/*' -exec wc -l {} + 2>/dev/null \
  | awk '$1 > 800 && $2 != "total" { print $0 }' \
  | sort -rn

出力 markdown は以下の構造:

• 800 行超 file 一覧 (件数表示 + table)
• 計画書 docs/file-length-enforcement-plan.md Phase 1 (PR-W1〜W4) との関係を明示
• 0 件 (clean state) case でも file を生成 (aggregate-weekly が常に Read 可能な前提を満たす)

LLM 判断の余地なし。命令通りに Bash を実行し、出力を整形するだけ。

aggregate-weekly instruction の更新

.takt/facets/instructions/aggregate-weekly.md を 3 reports → 4 reports に拡張:

• Input source list に file-length-watchlist.md を追加
• 機械的観測は Phase 1 統合 (LLM findings) には含めず、Phase 4 出力の専用 section "File Length Watchlist (機械的観測)" として weekly report に転載
• findings ゼロ case の出力 template にも同 section を追加

計画書 status 更新

docs/file-length-enforcement-plan.md の進捗追跡 table で PR-W0 を [in progress] にマーク。merge 後に [x] + PR 番号に更新する想定 (本 file 削除条件 1 / 3)。

設計上のポイント

• takt の step type 制約: takt workflow は LLM facet のみサポート (command-type step なし)。Rust 製の deterministic 機構を入れるには「LLM facet を limit instruction で deterministic wrapping」する形を採用 (analyzer 推奨の "pre-LLM Rust step" との差分)
• 観点番号 ⑦: 既存 3 facet は ADR-031 の review scope 軸で観点 ① (architecture) / ④ (security) / ⑥ (simplicity) を扱う。本 facet は観点 ⑦ として deterministic な機械観測層を補完
• aggregate-weekly での扱い: LLM findings (severity / category / location 必須) とは別系統の「機械的観測」section として転載。findings JSON には含めない (Phase C skill /weekly-review での AskUserQuestion 採否 flow は LLM findings のみが対象、watchlist は単なる health check)
• 計画書との整合: planning doc PR-W0 section の設計案通り (find command / aggregate-weekly 注入)

Test plan

• markdownlint: 0 errors (PostToolUse hook で各 Edit 確認)
• cli-docs-lint --check cross-ref: OK (planning doc 内 link は ../src/ 形式)
• takt pre-push-review approved (Result: Success)
• YAML 構文確認 (既存 3 facet と structurally identical)
• 次回 /weekly-review 実行で実観測 (本 PR merge 後、weekly schedule に合わせて発火)

dogfood は本 PR land 後の次の weekly run (= 2026-06-30 周辺、ADR-031 § Phase E トリガー方式 per "前回実行から 7 日経過で promote" reminder) で行う。

PR 計画における位置

master (PR #218 land 済、planning doc 確立)
  ↓
本 PR (PR-W0、weekly audit step 追加)  ← 本 PR
  ↓
PR-W1 (hooks-post-tool-comment-lint-rust 分割)
  ↓
PR-W2 (cli-pr-monitor 分割)
  ↓
PR-W3 (cli-merge-pipeline 分割)
  ↓
PR-W4 (cli-push-runner 分割)
  ↓
PR-W5 (Stop hook gate 追加、Phase 2 完了)
  ↓
[将来] PR-W6 (C → B 移行検討)

PR-W1 〜 W4 が land するごとに、本 watchlist の件数が 7 → 6 → 5 → 4 → ... → 0 と減少する想定。0 件到達後は恒久 audit として継続。

関連 PR / commit

• PR docs: PR-3a follow-up (順位 220/221) + ファイルサイズチェックフロー改善計画 (W0-W5 plan) #218 (docs: PR-3a follow-up + ファイルサイズチェックフロー改善計画、merged at 2026-06-23T19:31:19Z): 本作業の起点となる planning doc を land
• PR refactor(hooks): 3 hook の main.rs を module 分割 (PR-3a、file_length lint 違反解消) #217 (refactor(hooks): 3 hook の main.rs を module 分割, merged): file_length lint 違反が 7 file 累積している事実を顕在化
• 順位 222 (本 PR の commit 1、docs(todo): 順位 222 採用): PR docs: PR-3a follow-up (順位 220/221) + ファイルサイズチェックフロー改善計画 (W0-W5 plan) #218 post-merge-feedback refactor(deploy): 出力先へのテンプレート配布を廃止 #5 採用 entry (独立 commit で含まれる)

bonus content

本 PR には 2 つの commit が含まれる:

1. docs(todo): 順位 222 採用 (PR #218 post-merge-feedback #5) — ~/.claude/CLAUDE.md に「AI 先走り防止」rule 追加の todo entry
2. feat(weekly-review): file_length scan を pre-LLM step として追加 (PR-W0) — 本 PR の主目的

Summary by CodeRabbit

• New Features

  • 週次レポートに、長すぎるファイルの一覧を自動で含めるようになりました。
  • しきい値超過ファイルがある場合も、ない場合も結果が出力されます。

• Documentation

  • 週次レビューの手順と出力形式を更新しました。
  • 実行ルールや確認フローの案内を追記しました。

• 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: 1da0c3a9-5e44-4c20-ae16-c3e50ebc05ef

📥 Commits

Reviewing files that changed from the base of the PR and between b87174c and 591d108.

📒 Files selected for processing (6)

• .takt/facets/instructions/aggregate-weekly.md
• .takt/facets/instructions/file-length-watchlist.md
• .takt/workflows/weekly-review.yaml
• docs/file-length-enforcement-plan.md
• docs/todo-summary.md
• docs/todo10.md

---

📝 Walkthrough

Walkthrough

週次レビューワークフローに file-length-watchlist facetを新規追加した。800行超のRustファイルを機械的にスキャンするinstruction・ワークフロー定義・集約ルールを追加し、PR-W0進捗をin progressに更新した。

Changes

file-length-watchlist facet追加

Layer / File(s)	Summary
file-length-watchlist facet instruction定義 .takt/facets/instructions/file-length-watchlist.md	800行超Rustファイルの週次スキャン手順を新規定義。find/wc/awk/sortによるBashコマンド、markdownテンプレート（1件以上/0件）、出力契約、完了条件、読み取り専用・LLM判断排除の原則を規定。
weekly-reviewワークフローへの組み込み .takt/workflows/weekly-review.yaml	reviewers並列ステップにfile-length-watchlistエントリを追加し、許可ツール（Bash/Read）・instruction参照・出力ファイル名・完了条件を定義。
aggregate-weeklyへの集約ルール追加 .takt/facets/instructions/aggregate-weekly.md	入力レポート一覧にfile-length-watchlist.mdを追加。findingsに含めず「File Length Watchlist (機械的観測)」sectionとして転載するルール、findingsゼロ時テンプレへの同section追記を反映。
ドキュメント進捗・タスク更新 docs/file-length-enforcement-plan.md, docs/todo-summary.md, docs/todo10.md	PR-W0ステータスをin progressへ更新。todo-summary.mdに順位222タスク行を追加。todo10.mdにplanning doc完了時GO/NO-GO確認ルール追加タスクを新規追記。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#182: aggregate-weekly.mdおよびweekly-review.yamlの週次Phase B集約・ワークフロー基盤を確立したPRで、本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-W0のfile length scanをweekly-reviewのpre-LLM stepとして追加した主旨が伝わっており、順位222の採用も含めて変更内容に概ね一致しています。
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.}