docs(monitoring): Bundle b 完成記録 + Bb-3 post-merge-feedback の採用 3 件登録 (PR #115 follow-up)

[aloekun]

Summary

PR #115 (Bb-3) マージ後の post-merge-feedback で採用された 3 件を docs/todo*.md 系列に登録する docs-only PR。実装は今回スコープ外で、後続の Tier 2 / Tier 3 PR で進める。

今回の運用テスト: これまでは post-merge-feedback の採用案を次の実装 PR に fold-in していたが、本 PR から docs PR で採用案を独立に登録 → 実装は別 PR という運用に切り替える。docs PR 向けの開発環境 (ADR-035 docs-only 評価ポリシー / pre-push-review の docs-only 経路) の使用感確認も兼ねる。

変更内容

1. coderabbit-monitoring-efficiency.md Bb-3 完了記録

• Bb-3 エントリを「マージ済 (PR feat(cli-pr-monitor,hooks-session-start): config 拡張 + SessionStart catch-up + T2-2 follow-up (Bundle b PR-3, Bb-3 順位 55) #115)」に更新
• CR Major feat(hooks): 設定駆動型アーキテクチャに移行し配布自動化を実装 #1+fix(hooks): stop-quality のパイプデッドロックを修正 #2 の fold-in 修正経緯 (ReviewRecheckConfig::sanitize() + MAX_SAFE_WAIT_SECS = 1 年 上限化) を記録
• Bundle b 全機能 (Bb-1 rate-limit park / Bb-2 wakeup 経路 / Bb-3 sanitize) が PR feat(cli-pr-monitor,hooks-session-start): config 拡張 + SessionStart catch-up + T2-2 follow-up (Bundle b PR-3, Bb-3 順位 55) #115 自身で連携動作した dogfood 実証を明記
• Bundle b 完結 (Bb-1 + Bb-2 + Bb-3 + 順位 75 + 76-78) を宣言

2. todo.md priority table 更新

• 完了済の 順位 55 (Bb-3) を削除
• 採用 3 件を新規登録:
  • 順位 76 🔧 Tier 2 (M): cross-module overflow 統合テスト (config sanitize → poll arithmetic)
  • 順位 77 🔧 Tier 2 (S): Unix timestamp baseline 境界値テスト matrix
  • 順位 78 💎 Tier 3 (S): ADR-038 (Rust timestamp arithmetic safety) + CLAUDE.md security 拡充

3. todo5.md 詳細セクション追加

順位 76/77/78 の詳細を既存フォーマット (動機 / 設計決定 / 作業計画 / 完了基準 / 詰まっている箇所) で記述。

実装順の推奨:

• 順位 76 + 77 は同一 PR で land (両方とも wait_secs boundary を扱う test 層)
• 順位 78 は test 層 land 後の codification PR (ADR + CLAUDE.md で「test layer の根拠」として参照)

post-merge-feedback サマリー (PR #115)

.claude/feedback-reports/115.md で生成済。10 件中:

Recommendation	件数	主な提案
✅ 採用 (本 PR で登録)	3	T2-1 (統合テスト) / T2-2 (境界値テスト) / T3-1 (ADR + CLAUDE.md)
🤔 様子見	4	cast lint / Clippy integer_arithmetic / 後方互換 CI / magic string test
❌ 却下	3	SAFETY コメント rule / hardcoded constants block / magic string block

pre-push-review 結果

all("approved") (1 iteration、2m 23s)。ADR-035 docs-only 評価ポリシーが正しく適用され、findings なし (APPROVE)。

Test Plan

• pnpm lint:md (55 ファイル、0 error)
• pre-push-review all("approved")
• markdownlint-cli2 Stop hook (0 error)
• (post-merge) 本 PR 自身が docs-only PR の運用感サンプルとなる。次回 docs PR で改善点を検討
• (post-merge) 順位 76/77 を Bundle として実装着手 (本 PR land 後)

[coderabbitai]

Warning

Rate limit exceeded

@aloekun has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 17 minutes and 5 seconds before requesting another review.

To continue reviewing without waiting, purchase usage credits in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c0ffc774-941e-421e-a0db-033d9b771268

📥 Commits

Reviewing files that changed from the base of the PR and between d582f70 and 3a6e64f.

📒 Files selected for processing (3)

• docs/coderabbit-monitoring-efficiency.md
• docs/todo.md
• docs/todo5.md

---

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