test(hooks): Bundle k-2 — TOML positive/negative test + extensions reminder

[aloekun]

Summary

Bundle k-2: PR #155 (Bundle k-1) の続編として、no-ephemeral-todo-reference rule の test gap 補強 + extensions 変更時 reminder の 2 タスクを bundled で land。

• 順位 124 (🚀 Tier 1, S): TOML positive + negative test 追加 — 既存 self-exclusion 不変条件 test が検出力 test を含んでおらず、TOML 拡張子で rule が機能することの explicit な seal が不在だった gap を塞ぐ
• 順位 127 (💎 Tier 3, XS): extensions 拡張時 test 追加義務を .claude/custom-lint-rules.toml の rule 上に reminder comment として明文化 — point-of-edit reminder

背景

no-ephemeral-todo-reference rule の extensions に対する test gap が複数 PR で反復観測 されてきた pattern (Frequency Medium / 4 PR 観測):

• PR feat(hooks): Bundle "docs quality pre-write" — Stop hook lint:md + ephemeral todo ref detection (順位 4/29) #110: yaml/yml 追加時に対応 test 不在
• PR feat(hooks): Phase D D-5 — comment-lint test 拡充 + MAX cap break scope test #151: toml 拡張子で検出力 test 不在 (latent)
• PR feat(takt-facet): D-6 — refresh review-diff between fix/review iterations + Bundle k post-merge-feedback #152: docs-only PR で .md への FP が再現 — 同根 root cause
• PR feat(lint-screen): Bundle k-1 (順位 123) — Markdown ハンク除外フィルター + docs Phase D Round 2 完遂反映 #155 (Bundle k-1): MD 拡張子フィルター実装で structural fix

本 Bundle k-2 は test gap を closing。Bundle k-1 (実装層) + k-2 (test 層) + k-3 (UTF-8 横展開、todo 順位 125、別 PR 予定) の 3 層分割の中段。

変更内容

ファイル	変更	内容
.claude/custom-lint-rules.toml	+9	順位 127: extensions 変更時 test 追加 reminder block
src/hooks-post-tool-linter/src/main.rs	+41	順位 124: no_ephemeral_todo_detects_toml_ephemeral_reference + no_ephemeral_todo_toml_skips_permanent_adr_reference
docs/todo-summary.md	-2	順位 124 + 127 table 行削除
docs/todo8.md	-44	順位 124 + 127 詳細 entry 削除

Net: +4 lines (test code 41 増、docs 46 減、TOML 9 増)

検証

cargo test -p hooks-post-tool-linter → 121 passed / 0 failed

no_ephemeral_todo 系列 8 tests 全通過:

• 新規: no_ephemeral_todo_detects_toml_ephemeral_reference (positive)
• 新規: no_ephemeral_todo_toml_skips_permanent_adr_reference (negative)
• 既存 6 tests に regression なし

設計判断

Test fixture の self-trigger 回避

最初 positive test の fixture に literal docs/todo3.md を書いて、まさに rule が想定する meta-case (永続成果物 = Rust source → ephemeral 参照) を踏んだ。
解決: 既存 helper build_concrete_digit_fixture(3) を再利用 — format! interpolation で literal pattern が source に出現しない設計。本リポジトリの self-exclusion 標準パターン (.claude/custom-lint-rules.toml rule⑥ 冒頭コメントに記載)。

build_concrete_digit_fixture(3) の 3 は digit 番号 (= docs/todo3.md の 3) であって violation count ではない。violations.len() == 1 のアサートは 1 件の ephemeral 参照から 1 violation が返ることを check する意図で整合。

Positive と Negative の fixture 構築非対称性

positive test は ephemeral 参照生成 helper を使い、negative test (docs/adr/...) はインライン構築。意図的に非対称:

• helper は ephemeral 参照生成専用 (self-trigger 回避の責務)
• permanent 参照 fixture は literal で書いても rule は fire しないので self-trigger 懸念なし → インラインの方が test 単独で意図が読みやすい

memory rule feedback_test_dry_antipattern.md (テストの DRY は適用しない、独立性優先) に沿った判断。

Non-doc comment 削除

Bundle Z #B-α (非 doc コメント禁止) の lint 層により、test 内の // comment は block される。最終的に既存 test と同じ「コメントなし、build_* helper 名で意図を表現」スタイルに収束。

takt pre-push-review

• APPROVED (1 iteration / 3m20s)
• Blocking anomalies: 0
• Non-blocking notes: 0
• reviewer が positive/negative test の非対称性、build_concrete_digit_fixture(3) の digit 解釈、todo cleanup の atomicity を独立 validate

Test plan

• cargo test -p hooks-post-tool-linter: 121 passed
• takt pre-push-review: APPROVED
• CI green (post-push)
• CodeRabbit に actionable comments がない

Summary by CodeRabbit

• Documentation

  • ドキュメントのタスク追跡情報を更新しました。

• Tests

  • ルール検証用のテストケースを追加しました。

• 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: b51376b1-9721-4341-8548-a71ac73c4bcf

📥 Commits

Reviewing files that changed from the base of the PR and between 9942e46 and 55ceed9.

📒 Files selected for processing (4)

• .claude/custom-lint-rules.toml
• docs/todo-summary.md
• docs/todo8.md
• src/hooks-post-tool-linter/src/main.rs

💤 Files with no reviewable changes (2)

• docs/todo8.md
• docs/todo-summary.md

---

📝 Walkthrough

Walkthrough

PR #163 は no_ephemeral_todo_reference_rule に対して TOML ファイル拡張子のサポートテストを追加し、extensions 変更時の テスト追加規約をカスタムリント設定にコメント化し、対応済み TODO エントリを削除します。

Changes

TOML カスタムルール拡張テストと規約

Layer / File(s)	Summary
no_ephemeral_todo_reference_rule の TOML positive/negative テスト src/hooks-post-tool-linter/src/main.rs	config.toml に含まれるエフェメラル TODO 参照（docs/todo3.md）を 1 件検出する positive テストと、permanent な ADR 参照（docs/adr/adr-...）が検出されない negative テストを追加。既存の検出ロジックは変更なし。
Extensions 変更時のテスト追加規約をコメント化 .claude/custom-lint-rules.toml	no_ephemeral_todo_reference_rule 定義に、extensions 追加/削除時は同一 PR で positive/negative テスト系列を追加すべき旨の注意書きコメントを挿入し、サイレント検出力回帰のリスクを明文化。
対応済み TODO エントリの削除 docs/todo-summary.md, docs/todo8.md	PR #151 で実装・完了した「TOML positive test 追加」と「extensions 拡張時テスト追加パターン明文化」タスクを削除。推奨実行順序サマリーテーブルは順位 125 を保持。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#151: 当 PR の TOML positive/negative テスト追加は、retrieved PR #151 で実装された no-ephemeral-todo-reference ルール（.claude/custom-lint-rules.toml 内 extensions 設定）に対する直接的なテストカバレッジ拡張のため。
• aloekun/claude-code-hook-test#141: 当 PR の no_ephemeral_todo_reference_rule TOML テスト追加が、retrieved PR #141 で導入された no-ephemeral-todo-reference self-exclusion invariant テスト領域と同じルール内でテストを拡張するため。
• aloekun/claude-code-hook-test#110: Retrieved PR #110 で初期導入された no-ephemeral-todo-reference ルール定義に対して、当 PR が TOML 拡張子対応のテストカバレッジを追加するため。

🚥 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	プルリクエストのタイトルは、TOML 形式のポジティブ/ネガティブテストの追加と extensions 拡張時のリマインダーコメント追加という 2 つの主要な変更内容を正確に反映しており、変更セットの核心を明確に要約しています。
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.}