feat: Bundle 既存ルール仕組み化 — file_length lint + pr_size_check stage (順位 147 + 151)

[aloekun]

Summary

Bundle "既存ルール仕組み化" — ~/.claude/rules/common/ に書かれているが mechanical enforcement されていなかった 2 つの size limit を hook + push 両層の機械強制に昇格する。

順位 147: file_length lint 追加 (hooks-post-tool-comment-lint-rust)

• coding-style.md § File Organization の 800 行 max ガイドラインを決定論的に維持
• Touch-trigger ratchet (whole-file flag): 既存 800+ ファイルは触られるまで grandfather、任意の Edit/Write で feedback (additionalContext)
• soft nag のため override env は持たない (block しない nag に override は無意味)
• AST 不要 (line count のみ) で ~30 行追加 + 14 tests

順位 151: pr_size_check stage 追加 (cli-push-runner)

• git-workflow.md § Multi-PR chaining の 「1 PR あたり 250-800 lines」 目安を決定論的に維持
• jj diff --stat -r '<default_branch>..@' で insertions + deletions を計測
• 2 段階閾値: warning_threshold = 800 (警告のみ) / block_threshold = 1500 (push 停止)
• Kill-switch env: PR_SIZE_CHECK_OVERRIDE=1 (大型 refactoring 時のバイパス)
• ADR-039 三点セット完全準拠 (scratch_file_warning template ベース)
• default_branch 引数化で rule⑫ no-hardcoded-jj-revset-range 適用、alternative branch ("main" 等) 対応

Self-dogfood の結果

• jj diff --stat -r 'master..@' = 921 行 → warning_threshold (800) 超過、block_threshold (1500) 未満で warning のみ
• file_length lint deploy 後、1606 行に成長した hooks-post-tool-comment-lint-rust/src/main.rs で RUST_FILE_TOO_LONG 正常発火を確認

Semantic 注記 (147)

todo 文言「順位 48 と同 touch-trigger ratchet」を whole-file flag で実装。ファイル全体は常に「触られている」ので関数長の body overlap 検査がそのまま translate せず、aggressive (任意 Edit で flag) を採用。soft nag なので開発フローを妨げない (block しない)。

Test plan

• cargo test --workspace 全 PASS (cli-push-runner 169 / comment-lint-rust 100 / その他)
• cargo clippy --workspace -- -D warnings 0 warnings
• pnpm build:all 成功
• file_length lint dogfood (1606 行ファイルで RUST_FILE_TOO_LONG 発火)
• pr_size_check self-test (PR 921 行で warning のみ、block_threshold 未満で push 続行)
• takt pre-push-review APPROVE (N-1 minor: 単数形 "file changed" 未対応は non-blocking、次 PR で fix)
• (post-merge dogfood) 3-5 PR 後に false positive / 検出効果 / override 使用頻度を観測し default-ON 昇格 or 却下を判定 (ADR-039 bounded lifetime)

関連 ADR

• ADR-039 (Experimental feature 標準パターン): 両機能とも config opt-in + kill-switch + bounded lifetime 準拠
• ADR-007 (custom linter layer boundary): 147 = PostToolUse hook 層、151 = pre-push 層、責務分離
• rule⑫ (no-hardcoded-jj-revset-range): 151 で適用、format!("{}..@", default_branch) で組立

Summary by CodeRabbit

リリースノート

• New Features

  • プッシュ時にPRの変更行数をチェック。800行で警告、1500行を超えるとプッシュをブロックします。PR_SIZE_CHECK_OVERRIDE環境変数でバイパス可能。
  • Rustファイルの行数が800行を超える場合、エラーとして検出します。

• Documentation

  • タスク追跡ドキュメントを更新し、GitHubトークン検出テストと設計方針の追記タスクを追加。

[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: b523340a-beb3-454f-9d8b-89dc66ebed41

📥 Commits

Reviewing files that changed from the base of the PR and between d5ca4f5 and db6fa6c.

📒 Files selected for processing (8)

• docs/todo-summary.md
• docs/todo10.md
• push-runner-config.toml
• src/cli-push-runner/src/config.rs
• src/cli-push-runner/src/main.rs
• src/cli-push-runner/src/stages/mod.rs
• src/cli-push-runner/src/stages/pr_size_check.rs
• src/hooks-post-tool-comment-lint-rust/src/main.rs

---

📝 Walkthrough

Walkthrough

PR #202 は、PR diff サイズを検査する push-runner ステージと、Rust ソースファイルのファイル長違反を検出する lint ルールを新たに実装します。設定・CLI 統合・コア実装の段階を経て PR サイズチェックを完成させ、並行して file length detection を hooks-lint に追加します。

Changes

PR Size Check Pipeline

Layer / File(s)	Summary
Configuration schema and defaults push-runner-config.toml, src/cli-push-runner/src/config.rs	[pr_size_check] TOML セクション、PrSizeCheckConfig 構造体、警告閾値 800・ブロック閾値 1500・基準ブランチ "master" のデフォルト定数、および TOML パース・設定テストを追加します。
CLI pipeline integration and exit codes src/cli-push-runner/src/main.rs	終了コード EXIT_PR_SIZE_EXCEEDED（8）を追加し、run_pre_checks 内で run_pr_size_check を呼び出す分岐を実装します。ブロック時にログ出力後パイプライン中断、説明文を更新します。
Module declaration and re-export src/cli-push-runner/src/stages/mod.rs	pr_size_check モジュール追加と run_pr_size_check の pub(crate) 再エクスポートにより、main.rs からの呼び出しを可能にします。
Core size check implementation and parsing src/cli-push-runner/src/stages/pr_size_check.rs	jj diff --stat -r default_branch..@ で PR diff サイズを取得し、insertions/deletions を合算して警告・ブロック閾値と判定します。fail-open デザイン（jj 失敗時も push 継続）、環境変数 PR_SIZE_CHECK_OVERRIDE によるバイパス、summary 行パース・ヘルパー関数・境界値テストを含みます。

File Length Violation Detection

Layer / File(s)	Summary
File length violation detection and integration src/hooks-post-tool-comment-lint-rust/src/main.rs	MAX_FILE_LINES（800行）を定数化し、find_file_length_violations でファイル全体の行数をカウント、超過時に RUST_FILE_TOO_LONG（error）を生成します。純削除ファイルはスキップ、collect_all_violations に統合して既存違反と集約します。行数カウント・判定・フィルタ・統合テストを含みます。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#171: 主 PR の hooks-lint 作業と同じく collect_all_violations の集約フローを拡張して長さ系 violation を追加する点で関連します。
• aloekun/claude-code-hook-test#104: ファイル長違反検出の line_filter・pure-deletion スキップ処理が、取得 PR の changed-lines scoping ロジックを踏襲しています。
• aloekun/claude-code-hook-test#105: 同じく collect_all_violations の集約フローを拡張して長さ系 violation（関数長）を追加する実装パターンで関連します。

🚥 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 のタイトルは、主要な変更（ファイル長リント + pr_size_check ステージの追加）を明確に要約しており、優先順位（147 + 151）も含めて具体的かつ正確です。
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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.}

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