docs(todo): PR #86 post-merge-feedback の Tier 1/3 finding を採用

[aloekun]

Summary

PR #86 マージ後の post-merge-feedback (ADR-030) が抽出した Tier 1 + Tier 3 の計 3 件 を採用し、docs/todo.md / docs/todo2.md に反映する。

反映内容

新規エントリ (docs/todo2.md、3 件)

順位	Tier	タスク	工数
5	🚀 Tier 1	Polling anti-pattern 検出ルール (T1-1)	XS
17	💎 Tier 3	post-pr-monitor polling 禁止のグローバル明文化 (T3-2)	XS
18	💎 Tier 3	todo.md 採番管理の簡素化 ADR 起案 (T3-3)	S

docs/todo.md 更新

• 推奨実行順序サマリー table を 17 → 20 行 に拡張
• 戦略行: Tier 1 (1〜6) → Tier 2 (7〜10) → Tier 3 (11〜18) → Tier 4-5 (19〜20) に更新
• 既存 4 タスクの「実行優先度」+ cross-references を新採番 (/20) に更新

動機

post-merge-feedback (ADR-030) フィードバックループの 3 周回目。本セッションで顕在化した polling anti-pattern による rate limit 大量消費 (40% / 1 時間) への決定論的防止策を最優先で起案。

特筆すべき finding:

• Tier 1 feat(hooks): 設定駆動型アーキテクチャに移行し配布自動化を実装 #1 (polling 検出ルール): 同一セッションで 2 回発生した anti-pattern を機械的に防止
• Tier 3 fix(hooks): extra_protected_files でパス付き指定をサポート #3 (採番簡素化 ADR): 本 PR 自体が証明する構造的負債 (タスク数 20 超で cross-ref 維持コストが顕在化、本 PR の renumber で 11 件の Edit を要した) を解消するメタタスク

Test plan

• pre-push-review (simplicity + security) APPROVED (push 時に確認予定)
• CI (lint / test / build / rust-test) パス
• CodeRabbit レビュー異常なし
• table renumber の一貫性を目視確認 (順位 1-20、依存参照、cross-ref)
• 削除済みエントリ参照 (順位 N/10, 順位 N/13, 順位 N/17) の dangling 参照ゼロを grep で確認済 (push 前検証)

Summary by CodeRabbit

リリースノート

• Documentation
  • タスク管理文書の採番体系を更新し、推奨実行順序を拡張
  • ロードマップの優先度参照を同期化
  • 新規タスク項目を追加し、戦略的な実行順序を明確化

[coderabbitai]

📝 Walkthrough

Walkthrough

ドキュメント docs/todo.md と docs/todo2.md を更新し、タスク採番体系を 17 から 20 に拡張。PR #86 の関連タスク（Polling アンチパターン検出、post-pr-monitor ポーリング禁止、採番管理簡素化）を追加し、既存タスクの順位参照と依存前提を同期。

Changes

Cohort / File(s)	Summary
タスク採番体系の拡張とPR #86統合 docs/todo.md, docs/todo2.md	推奨実行順序テーブルを 17→20 に拡張。Tier 1-3 にわたり PR #86 関連の 3 つの新タスク（Polling アンチパターン検出、post-pr-monitor ポーリング禁止グローバル化、採番管理簡素化 ADR）を追加。ADR-030/031/032 など既存タスクの優先度順位と依存前提を更新。複数セクションの実行優先度ブロックで参照順位を同期（例：ADR-030 系が 16/17→19/20、ADR-031 が 6/17→7/20 へ修正）。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• PR #86: 本 PR で新規追加される Polling アンチパターン検出、post-pr-monitor ポーリング禁止、採番管理簡素化の 3 タスクの主体となる PR。
• PR #82: 本 PR で優先度調整される ADR-032 Phase 群や Tier 2 タスクを導入した PR。
• PR #85: 本 PR で採番体系更新の対象となる複数タスク（Tier 1-3）を含む 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	PRのタイトルは「docs(todo): PR #86 post-merge-feedback の Tier 1/3 finding を採用」であり、変更内容（PR #86の post-merge-feedback で抽出された Tier 1 と Tier 3 の合計 3 件を docs/todo.md と docs/todo2.md に反映する）を正確に要約している。
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.}

---

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]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@docs/todo2.md`:
- Around line 652-688: The design is invalid because PreToolUse
([pre_tool_validate]) cannot access prior tool-call context; update the proposal
to either (1) adopt the recommended PostToolUse approach by changing the
implementation plan to "PostToolUse + history buffering" and specify adding a
lightweight state file under .claude/ that records the last N tool calls
(including run_in_background flags) so the PostToolUse linter can detect a
subsequent until.*sleep polling pattern, or (2) propose adding a new hook type
that exposes session tool-call history; also revise the completion criteria to
"implementation approach decided and documented (PostToolUse + state buffering)
or new-hook spec completed" rather than immediate detection readiness.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2e81bb37-ec71-4665-9c50-8197010dbcb2

📥 Commits

Reviewing files that changed from the base of the PR and between 53d4cfc and 1e671c6.

📒 Files selected for processing (2)

• docs/todo.md
• docs/todo2.md