feat(hooks): Phase D D-5 — comment-lint test 拡充 + MAX cap break scope test

[aloekun]

Summary

Phase D Round 2 の D-5 (順位 56 + 119 bundle) 実装 + dogfood 観測。

コード変更 (commit 5cbed3c)

• hooks-post-tool-comment-lint-rust: UTF-8 multi-byte test 5 パターン + block comment boundary test 6 パターン + direct unit test 1 件追加 (計 12 tests)
• hooks-post-tool-linter: MAX_CUSTOM_VIOLATIONS outer/inner loop break scope explicit test 2 件追加
• 副産物 production bug fix: byte_offset_to_line の char-boundary panic を解消 (UTF-8 multi-byte 文字で終わる needle で panic していた production bug)

Test

• comment-lint-rust: 85 passed / linter: 111 passed = 全 196 tests pass

Phase D D-5 dogfood 観測 (3 push events で 3 data points)

• Push 1 (Rust diff 649 行): auto_fix + 1 FP (file 混同型 hallucinate)
• Push 2 (docs-only 67 行): auto_fix + 1 FP (同 FP 再現 = mistral:7b が diff 外 hook source を見ている根拠)
• Push 3 (docs-only 50 行): auto_fix + 1 FP (file 名を hallucinate、reviewer が "the report documents its own FP" と評価)
• 累積 6 data points / 3 PR で ADR-038 採用条件先行充足、fallback 0/5 = 0%

詳細は docs/local-llm-offload-analysis.md の D-5 outcome 節を参照。

Test plan

• cargo test 全 pass (196 tests)
• cargo fmt / clippy clean
• pre-push-review APPROVE (3 push events 全て 1-iter 収束)
• lint_screen dogfood 観測完了 (3 data points)
• CodeRabbit review (post-PR)

Summary by CodeRabbit

リリースノート

• ドキュメント

  • 分析進捗と実装データを更新
  • タスク管理・推奨実行順序を最新状況に反映

• テスト

  • 多言語・特殊文字に対応したテストカバレッジを拡充
  • カスタムルール動作の検証テストを追加

• 機能拡張

  • TOML ファイルに対応する規則を追加

[coderabbitai]

📝 Walkthrough

Walkthrough

PR #151 は Phase D Round 2 の lint_screen 実測結果をドキュメントに記録し、D-5 で発見された UTF-8/byte_offset のバグを Comment Lint Rust hook で修正・テスト拡張、Linter hook で MAX_CUSTOM_VIOLATIONS キャップのテストカバレッジを強化し、no-ephemeral-todo-reference rule を toml ファイルへ拡張する変更をまとめたもの。

Changes

Phase D Round 2 進捗とUTF-8バグ修正

Layer / File(s)	Summary
Phase D Round 2 進捗ドキュメント更新 docs/local-llm-offload-analysis.md, docs/todo-summary.md, docs/todo7.md, docs/todo8.md	local-llm-offload-analysis.md に Phase D Round 2 の state セクション、D-4（PR #150 merged）の dogfood outcome（2 回の push による informational/auto_fix のスクリーンショット結果）、D-5 の dogfood outcome（両方 auto_fix、false positive 再現、production bug fix 副産物）を追記。todo-summary.md・todo7.md から comment-lint hook テスト拡充タスク削除、todo8.md に PR #150 関連の 3 つの新規タスク（takt-workflow-persona コメント拡張＋ADR-007 case study、permission_mode doc 修正＋fixture テスト、development-workflow Step 0 チェック）を追加。
Comment Lint Rust の UTF-8/Multibyte バグ修正とテスト拡張 src/hooks-post-tool-comment-lint-rust/src/main.rs	D-5 で観測された byte_offset/UTF-8 境界バグを修正。byte_offset_to_line を as_bytes() スライスベースの実装に再構築し multibyte 文字の正確な行計算を実現。function_metric と collect_all_violations をリフォーマット。新たに 4 つのテストグループを追加：mixed ASCII+Kanji・emoji・combining characters での locate_string_line_ranges 検証、mid-multibyte offset での byte_offset_to_line 検証、multiline/inline block comment の range overlap（start/middle/end only 等）での find_violations 検証。

Linter ルール拡張とテストカバレッジ

Layer / File(s)	Summary
MAX_CUSTOM_VIOLATIONS 上限キャップのテスト src/hooks-post-tool-linter/src/main.rs	複数ルール間での MAX_CUSTOM_VIOLATIONS キャップ動作の 2 つの新規テストを追加。Outer break テストで最初のルール実行後のキャップ使い尽くしと後続ルール実行ブロック、inner cap テストで最初のルール部分消費後の次ルール実行を検証。
Linter ルール拡張とコード整形 src/hooks-post-tool-linter/src/main.rs	compile_paths_glob エラーマッピングをシングルライン化、build_violation_json の関数シグネチャと JSON 構築をマルチラインチェーン形式に再整形。複数の test helper 呼び出し（make_test_rule、write_file）を整形。no-ephemeral-todo-reference rule の extensions に "toml" を追加して toml ファイルへのマッチング対象を拡張。deployed custom rules テストを可読性向上のため再整形。

推奨される関連PR

• aloekun/claude-code-hook-test#104: comment-lint-rust の byte_offset_to_line/find_violations range-filter リファクタリング・UTF-8/offset テスト拡張が PR #104 で導入された line_filter 仕組みの上に直接構築。
• aloekun/claude-code-hook-test#148: Linter hook の compile_paths_glob/build_violation_json 変更が PR #148 の paths-filter 整備の直後に追加実装。
• aloekun/claude-code-hook-test#141: no-ephemeral-todo-reference rule の extensions に "toml" 追加と PR #141 の同ルール self-exclusion 回帰テスト（deployed TOML が自分自身で発火しない）が相互補完。

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 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タイトルは「feat(hooks): Phase D D-5 — comment-lint test 拡充 + MAX cap break scope test」で、PR目標に完全に一致し、変更内容の主要部分（comment-lint テスト拡充と MAX_CUSTOM_VIOLATIONS の break scope テスト追加）を正確に反映している。
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.}

[coderabbitai]

🧹 Nitpick comments (1)

src/hooks-post-tool-linter/src/main.rs (1)

1917-1918: ⚡ Quick win

toml 追加に対する正方向の検出テストを1本追加しておくと堅いです。

extensions に "toml" を追加したので、config.toml で実際に違反が検出されるケースも明示しておくと、この拡張の回帰検知がより確実になります。

📌 追加例（テスト）

+    #[test]
+    fn no_ephemeral_todo_detects_concrete_digit_reference_in_toml() {
+        let dir = tempfile::tempdir().unwrap();
+        let file = write_file(
+            dir.path(),
+            "config.toml",
+            "note = \"see docs/todo3.md\"\n",
+        );
+        let rules = compile_test_rules(vec![no_ephemeral_todo_reference_rule()]);
+        let violations = run_custom_rules(file.to_str().unwrap(), &rules);
+        assert_eq!(violations.len(), 1);
+    }

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/hooks-post-tool-linter/src/main.rs` around lines 1917 - 1918, extensions
配列に "toml" を追加したので、対応する正方向の検出テストを追加してください: 新しいユニット/統合テストを作り、config.toml を入力として
linter 実行（関数/バイナリ呼び出しを担当するテストヘルパー）し、期待通り違反が検出されることをアサートします。該当コード箇所は extensions
配列（"rs","toml",...）と linter 実行を行うテストヘルパー（既存のテストで使っている run_lint/execute_tool
相当の関数名）を使って、config.toml に違反となるサンプルを書き込み、違反メッセージが返ることを検証するように実装してください。

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@src/hooks-post-tool-linter/src/main.rs`:
- Around line 1917-1918: extensions 配列に "toml" を追加したので、対応する正方向の検出テストを追加してください:
新しいユニット/統合テストを作り、config.toml を入力として linter
実行（関数/バイナリ呼び出しを担当するテストヘルパー）し、期待通り違反が検出されることをアサートします。該当コード箇所は extensions
配列（"rs","toml",...）と linter 実行を行うテストヘルパー（既存のテストで使っている run_lint/execute_tool
相当の関数名）を使って、config.toml に違反となるサンプルを書き込み、違反メッセージが返ることを検証するように実装してください。

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7413deb6-f185-4dbc-b287-34a6902d592b

📥 Commits

Reviewing files that changed from the base of the PR and between 39ae2cd and 1a69cc6.

📒 Files selected for processing (6)

• docs/local-llm-offload-analysis.md
• docs/todo-summary.md
• docs/todo7.md
• docs/todo8.md
• src/hooks-post-tool-comment-lint-rust/src/main.rs
• src/hooks-post-tool-linter/src/main.rs

💤 Files with no reviewable changes (1)

• docs/todo7.md