feat(hooks): polling-anti-pattern preset で polling loop をブロック (Phase 1)

[aloekun]

Summary

PreToolUse hook に polling-anti-pattern preset を追加し、until COND; do BODY; sleep N; done / while ! COND; do ... sleep N 形式の polling loop をブロックします。Claude Code が反射的に書きがちな polling pattern を決定論的に検出することで、PR #86 で実証された rate-limit 大量浪費 (1 セッションで 40%) の再発を防止します。

Why

PR #86 のセッション中、run_in_background: true で起動した背景タスクの完了確認用に until grep -q done; do sleep N; done 形式の polling が反射的に書かれ、Claude Code Max (5x) のレートリミットを 1 時間で 40% 消費する事故が発生しました。背景タスクは task-notification 経由で自動的に完了通知される設計のため、polling は本来不要です。ガイドライン (順位 26) と組み合わせ、決定論的検出 + ドキュメント補完の二層防御を構築する第一段階。

Changes

• src/hooks-pre-tool-validate/src/main.rs
  • preset_polling_anti_pattern() を追加 (既存 preset 関数群と同一パターン)
  • 検出 regex: (?is)\buntil\b.*?\bdo\b.*?\bsleep\s+\d / (?is)\bwhile\s+!\s.*?\bdo\b.*?\bsleep\s+\d
  • \bdo\b 制約により bash loop syntax のみマッチ (echo 文字列・--until=DATE フラグ等の false positive を排除)
  • ブロック時メッセージに代替手段 (Monitor tool / run_in_background + 通知 / 単発取得) を提示
• .claude/hooks-config.toml
  • 本プロジェクト用に "polling-anti-pattern" を blocked_patterns に追加
  • デフォルトフォールバックには非追加 (後方互換維持、派生プロジェクトは明示 opt-in)
• テスト 12 件追加 (block 6 / allow 6 / 後方互換 1 = 計 13 件、cargo test で全 pass)
  • block: 1-line / multi-line / [ ] / while ! / state file polling / chain
  • allow (false positive 排除確認): for loop / 単純 sleep / sleep なし / while true / echo string / --until= flag
• docs/todo.md 順位 3 と docs/todo2.md 該当エントリを削除し、参照を整合化

Design notes

• 当初設計案にあった "run_in_background → 直後の polling" の cross-tool-call 検出は YAGNI で見送り (transcript 読み取りが必要、複雑度に対する効果が低い)。in-command 検出だけで PR docs(todo): PR #85 post-merge-feedback の Tier 1-3 finding を全採用 #86 の canonical case を捕捉できる。
• ExitCode 2 でブロックする選択 (spec は "warning" だが、PreToolUse hook framework の唯一のフィードバック手段が exit 2 のため、強制的な行動修正が目標と整合)。
• 後方互換: opt-in 方式により既存の派生プロジェクトには影響しない。

Test plan

• cargo test (122 件、polling 関連 13 件含め全 pass)
• cargo fmt --check / cargo clippy --all-targets -- -D warnings 全 pass
• 実機検証: real exe に JSON stdin で 6 ケース投入し、block/allow が期待通り動作することを確認
• dogfood: コミットメッセージ中の polling pattern 文字列にも実際に hook が発火することを確認 (機能していることの直接的な証跡)
• takt pre-push-review で APPROVE (3 iterations → 1 iteration、最終的に aggregate=approved)

Phase

todo.md 推奨実行順序 Phase 1 (順位 3, Tier 1) 完了。次は Bundle T (順位 9 + 14) を計画。

Summary by CodeRabbit

リリースノート

• 新機能

  • ポーリング検出パターンを検証機構に追加しました。

• ドキュメント

  • 完了した検出ルール実装の参照を更新しました。
  • 実施済みのタスク項目を削除しました。

[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: 5a8516d1-6b28-4cf6-b8b4-3733327f2f7f

📥 Commits

Reviewing files that changed from the base of the PR and between f8591a0 and abe498f.

📒 Files selected for processing (4)

• .claude/hooks-config.toml
• docs/todo.md
• docs/todo2.md
• src/hooks-pre-tool-validate/src/main.rs

💤 Files with no reviewable changes (1)

• docs/todo2.md

---

📝 Walkthrough

Walkthrough

ポーリング・アンチパターン（until/while ! ... sleep ループ）を検出するプリセット「polling-anti-pattern」を pre-tool-validate 機能に追加し、設定ファイルに登録して有効化するとともに、対応する実装テストを整備し、関連ドキュメントを更新する変更です。

Changes

Cohort / File(s)	Summary
Configuration .claude/hooks-config.toml	pre_tool_validate.blocked_patterns に新規プリセット "polling-anti-pattern" を追加
Documentation docs/todo.md, docs/todo2.md	Tier 1 ポーリング検出ルールの完了に伴い、関連 TODO エントリを削除・更新し、依存関係を明確化
Implementation src/hooks-pre-tool-validate/src/main.rs	polling-anti-pattern プリセット関数を実装し、マルチラインポーリングパターンと複数の偽陽性除外ケースを含む包括的なテストスイートを追加

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Possibly related PRs

• feat(cli): merge-pipeline 導入 (ADR-013) #27: 同じ src/hooks-pre-tool-validate モジュールの build_blocked_patterns と blocked-pattern プリセット関数を修正している関連 PR
• refactor(hooks): post-pr-monitor を PostToolUse hook からスタンドアロン CLI に移行 #13: 同様に pre-tool-validate 機能の blocked-pattern プリセット追加とモジュール更新を行う関連 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	プルリクエストのタイトルは、ポーリング anti-pattern プリセットを使用してポーリングループをブロックする主要な変更を正確に反映しており、簡潔で明確です。
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

_{Review rate limit: 0/1 reviews remaining, refill in 60 minutes.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}