docs(adr): coderabbit-monitoring-efficiency.md retire — knowledge を ADR-018/034 に集約

[aloekun]

Summary

docs/coderabbit-monitoring-efficiency.md を retire する docs-only PR。Bundle b (Bb-1/Bb-2/Bb-3、PR #113-115 + PR #116) の land で計画書の使命「bundle が消化されたら役割を終える」を達成。docs-governance.md の retirement workflow (4 ステップ) に従って permanent value を ADR に集約してから物理削除。

削除に至った経緯

• 計画書の役割: Bundle b (CR auto-monitoring core) の概要・並行進行設計を集約する index
• 完了状況: 順位 53 (Bb-1) / 54 (Bb-2) / 55 (Bb-3) / 75 (T2-2 Bb-1 follow-up) / 76-78 (Bb-3 post-merge-feedback) を全て land 完了
• 計画書自体の自己宣言: 試験運用 flag 付きで「bundle が消化されたら役割を終える」と明記済

変更内容

Step 1: knowledge migration (permanent value 退避)

• ADR-018 に追記 (2026-05-06 セクション):
  • Bundle b で CronCreate を park モデルとして再導入した経緯 (廃止した「daemon + CronCreate 4 段間接連携」とは別アーキテクチャ)
  • CronCreate の重要事実 (60 分上限なし、ScheduleWakeup との違い、durable: true で session 跨ぎ、recurring 7 日 auto-expire)
  • 廃止された 4 段間接連携 vs Bundle b park モデルの比較表
  • Bb-2 で observer モード (2026-04-22 追記) を撤廃 (二重 polling 解消)
• ADR-034 に追記 (Bundle b との関係セクション):
  • Sub-PR 2 の scope 縮小: 旧 4 component → 残 3 component (順位 42 = Bb-1 で実質達成、順位 43 = ADR-018 追記で部分達成、残 順位 46/49 のみ)
  • Bundle 並行進行設計の教訓 (touched modules ベースで重複可能性確認の必要性)

Step 2: residual tasks 整理

• 順位 15 (cli-pr-monitor 通知 Recovery 経路 / SessionStart hook 拡張) を Bb-3 SessionStart catch-up nudge で吸収済と判定
  • todo3.md / todo.md priority table から削除
  • Bb-3 catch-up nudge ([PR_MONITOR_CATCHUP] envelope、parked_* action gate) が順位 15 と同種の silent loss 防止機能を提供しているため重複
• 順位 11 (cli-pr-monitor プロセス正常終了 integration test) は本 doc のスコープ外 (auxiliary、todo2.md に維持)
• 順位 42/43/46/49 (Bundle a Sub-PR 2) は ADR-034 に集約済、todo4.md/todo5.md の詳細 entry はそのまま維持

Step 3: reference cleanup

• ADR-033 line 124 の領域特化計画書 example を docs/coderabbit-monitoring-efficiency.md 名指しから「試験運用 doc / Bundle 脚注」に一般化

Step 4: file deletion

• docs/coderabbit-monitoring-efficiency.md を物理削除 (上記 1-3 と同 commit)

pre-push-review 結果

all("approved") (1 iteration、4m 19s)。ADR-035 docs-only 評価ポリシーが適用され、reviewer から:

File deletion: permanent value visibly migrated; no dead navigation links created

の positive 評価あり。

Test Plan

• pnpm lint:md (54 ファイル / 0 error、削除前 55 ファイル → 54 に減少)
• grep -rn coderabbit-monitoring-efficiency で残存参照を確認 (ADR-034 内の retire 経緯 historical note 2 件のみ = 意図通り)
• pre-push-review all("approved")
• (post-merge) 削除されたファイル名を別 doc / commit message で参照しても broken link にならないことを確認
• (post-merge) 順位 76/77 着手時に ADR-034 の Sub-PR 2 残 scope 記述が指針として機能するか確認

Out of Scope

• Sub-PR 2 残 task (順位 43 ADR-009 navigation 注記 / 順位 46 integration test / 順位 49 parse_findings error-path test infra) の実装 — 本 PR は retirement のみ
• 順位 11 (cli-pr-monitor プロセス正常終了 integration test) の評価 — 本 doc スコープ外なので変更なし

Summary by CodeRabbit

ドキュメント

• アーキテクチャ決定記録を更新し、設計方針の明確化を進めました
• ドキュメント構成を整理し、参照ガイダンスと移行手順を追加しました
• 番号付けルールとガバナンスガイドラインを統一し、今後の拡張に対応しやすくしました
• 関連する計画資料を整理・統合しました

[coderabbitai]

📝 Walkthrough

Walkthrough

複数のドキュメント・ADR（Architecture Decision Records）を更新するPRです。Bundle bによるCronCreateの park モデル再導入、タスク番号付けの簡略化ガイドラインの詳細化、関連する設計決定の記述を追加し、古いドキュメント1ファイルを削除しています。

Changes

ADR・ドキュメント戦略更新

Layer / File(s)	Summary
Bundle b 実装背景 docs/adr/adr-018-pr-monitor-takt-migration.md	CronCreateの park モデル再導入、observer モード撤廃の背景と決定理由を追記。4段階連携との比較表、関連PR#113–#115の参照を追加。
タスク番号付けガイドラインの詳細化 docs/adr/adr-033-todo-numbering-simplification.md	優先度テーブルでの絶対番号使用と本文でのタスク名参照への移行ルール、新規タスク追加テンプレート、2層依存管理（技術的・戦略的）ガイダンスを追加。
Bundle B関連性の文書化 docs/adr/adr-034-coderabbit-auto-monitoring.md	Bundle BとBundle Aの関係、Sub-PR 2スコープの縮減（4コンポーネント→3コンポーネント）、並行進行上の考慮事項を追加。
タスク管理表の整理 docs/todo.md, docs/todo3.md	推奨実行順序テーブルから Tier 2 タスク1件を削除。新しい takt ハーネス terminal verdict REJECT-ESCALATE の設計ドキュメントを docs/todo3.md に追加。
古いドキュメント削除 docs/coderabbit-monitoring-efficiency.md	CodeRabbit 監視改善の計画・概要ドキュメント全体を削除。内容は関連ADRに統合済み。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#113: PR内で参照されている Bundle b 実装（state.next_wakeup_at_unix、park シグナル関連）とADR-018の追記内容が直結。
• aloekun/claude-code-hook-test#112: 同じADR-033、docs/coderabbit-monitoring-efficiency、todo ファイルを修正しており、ドキュメント体系の関連更新。
• aloekun/claude-code-hook-test#108: ADR-034とcoderabbit-monitoring-efficiency.mdの同じファイルを修正しており、docs削除・統合の流れで関連。

🚥 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	プルリクエストのタイトルは、docs/coderabbit-monitoring-efficiency.mdの廃止と知識をADR-018/034に統合するという主要な変更を正確かつ簡潔に説明しており、チェンジセット全体の主目的を明確に表現しています。
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 current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/adr/adr-018-pr-monitor-takt-migration.md`:
- Around line 134-135: The heading "CronCreate 廃止" is misleading now that the
note documents reintroducing CronCreate; update the title to reflect it as the
initial decision (e.g., change "CronCreate 廃止" to "当初廃止→parkモデルで再導入" or
equivalent) so the ADR clearly shows the original abolishment and subsequent
reintroduction; locate and edit the heading string "CronCreate 廃止" (and nearby
summary line "追記 (2026-05-06): Bundle b で CronCreate を park モデルとして再導入 + observer
モード撤廃") to use the revised phrasing.

🪄 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: 7d237e59-884a-471b-a355-463f1141e8fd

📥 Commits

Reviewing files that changed from the base of the PR and between a0f0e27 and 6e6954e.

📒 Files selected for processing (6)

• docs/adr/adr-018-pr-monitor-takt-migration.md
• docs/adr/adr-033-todo-numbering-simplification.md
• docs/adr/adr-034-coderabbit-auto-monitoring.md
• docs/coderabbit-monitoring-efficiency.md
• docs/todo.md
• docs/todo3.md

💤 Files with no reviewable changes (3)

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

[coderabbitai]

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

見出しが現行判断を誤解させる可能性があります

追記で CronCreate の再導入を正式に記録しているため、タイトルの「CronCreate 廃止」は現時点の意思決定状態としてはミスリードです。
CronCreate 廃止 を「当初判断」に限定する表現（例: 「当初廃止→parkモデルで再導入」）に更新すると、ADR 単体読解時の誤読を防げます。

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/adr/adr-018-pr-monitor-takt-migration.md` around lines 134 - 135, The
heading "CronCreate 廃止" is misleading now that the note documents reintroducing
CronCreate; update the title to reflect it as the initial decision (e.g., change
"CronCreate 廃止" to "当初廃止→parkモデルで再導入" or equivalent) so the ADR clearly shows
the original abolishment and subsequent reintroduction; locate and edit the
heading string "CronCreate 廃止" (and nearby summary line "追記 (2026-05-06): Bundle
b で CronCreate を park モデルとして再導入 + observer モード撤廃") to use the revised phrasing.