docs(linter): extensions 拡張時の test 追加 pattern を明文化 (順位 127)

[aloekun]

概要

docs/claude-code-web-tasks.md の採用タスクから 順位 127 (T3, XS) を実行しました。

採用タスク 3 件を調査した結果:

• 順位 120 — TOML rule⑨ の field 拡張手順 + ADR-007 case study が既に実装済み
• 順位 134 — ADR-035 の「❌ 適用しない criteria」表が既に実装済み
• 順位 127 — 唯一未実装だったため選択

変更内容

src/hooks-post-tool-linter/src/main.rs の CustomRuleTestCoverage struct doc comment に、既存 rule の extensions を拡張した際の test 追加 pattern を明文化しました（純コメント追記のため Rust build / Windows hook 実行は成功条件にならず、Claude Code Web scope の判定基準を満たす）。

要点（test location を正確に参照）:

1. 追加 ext が主要拡張子 (rs / toml / yaml / yml) → [rules.test_coverage.main_ext_tests.<ext>] に test 宣言が必須
2. 非主要拡張子のみ targets する場合 → other_ext_tests に 1+ 宣言が必須
3. 宣言した test 名は main.rs 走査で実在確認されるため、typo / 削除した test は orphan として検出される

参照した enforcing test / helper（いずれも実在確認済み）:

• tests::rule_test_coverage_check
• tests::classify_rule_extensions
• tests::check_main_ext_coverage / tests::check_other_ext_coverage
• tests::extract_existing_test_fn_names
• tests::MAIN_EXTENSIONS

着手フロー対応

着手フローに従い docs/claude-code-web-tasks.md の該当行を削除しました（todo-summary.md / todoN.md には ADR-033 の採番簡素化により当該エントリは既に存在せず）。

https://claude.ai/code/session_0143vzZuBE6aRw1fXfkujzFD

---

Generated by Claude Code

Summary by CodeRabbit

リリースノート

• Documentation
  • ドキュメンテーション内の採用タスク一覧を更新
  • コード内コメントを拡張し、テスト仕様に関する説明を明文化

[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: 6bddfebd-29b7-4582-9752-faece4fd72c1

📥 Commits

Reviewing files that changed from the base of the PR and between c9bf349 and 731959c.

📒 Files selected for processing (2)

• docs/claude-code-web-tasks.md
• src/hooks-post-tool-linter/src/main.rs

💤 Files with no reviewable changes (1)

• docs/claude-code-web-tasks.md

---

📝 Walkthrough

Walkthrough

採用タスクテーブルから PR #151（順位127相当）を削除し、PR #150/#156 関連の新タスク（順位120/134相当）に置き換え。同時に、カスタムルール拡張時のテスト宣言パターンを Rust コメントで明文化。

Changes

ドキュメント整備と仕様説明

Layer / File(s)	Summary
採用タスク優先順位更新 docs/claude-code-web-tasks.md	採用タスクテーブルの該当行が差し替わり、PR #151 のタスク（順位127相当）が削除され、PR #150 と PR #156 のタスク（順位120/134相当）が新規追加。
test_coverage 拡張時テスト宣言パターン仕様 src/hooks-post-tool-linter/src/main.rs	CustomRuleTestCoverage メタフィールドに関するコメントブロックを追記。extensions 拡張時に main_ext_tests/other_ext_tests で宣言すべきテストパターン、および TOML 宣言したテスト名の実在確認による orphan 検出方法を明文化。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#161: PR #161 の ADR-007 amendment と同表タスク削除が、メイン PR の採用タスク差し替え（ADR-007 case study 反映）に直接対応している。
• aloekun/claude-code-hook-test#165: メイン PR の CustomRuleTestCoverage（[rules.test_coverage]）拡張時テスト宣言パターン明文化は、PR #165 が導入した同メタフィールドのテスト検証ロジックと同じ領域に関連。
• aloekun/claude-code-hook-test#150: メイン PR が明文化したテスト追加ルール（拡張時のテスト宣言パターン）は、PR #150 が takt-workflow-persona-without-model ルール実装で実際に適用したテスト作成方針と直結。

🚥 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タイトルは、main.rsのコメント明文化という実際の変更内容と一致しており、「extensions拡張時のtest追加patternを明文化」という主要な変更を適切に要約している。
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.}

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