docs: Bundle g registration + Phase 5 dogfood plan (post-PR #121 follow-up)

[aloekun]

Summary

PR #121 post-merge-feedback の採用 4 件 (Bundle g) 登録 + §8.A-2 (Phase 5 dogfood 計測計画) 策定を 1 PR に集約 (docs-only):

1. Bundle g 採用 4 件登録 (順位 85-88、PR docs: ADR-038 textual fix + Bundle f task registration (post-PR #120 follow-up) #121 post-merge-feedback)
2. §8.A-2 Phase 5 dogfood 計画策定 (docs/local-llm-offload-analysis.md)
3. 整合更新: §7 / §8.E

Bundle g (順位 85-88) 詳細

順位	Tier	タスク	Effort
85	🚀 Tier 1	cli-pr-monitor monitor state machine guard 強化 (review_state: not_found && findings: [] を pending 据置)	S
86	🔧 Tier 2	state transition test 網羅 (順位 85 の回帰テスト)	S
87	💎 Tier 3	Multi-PR chaining ベストプラクティス codify	XS
88	💎 Tier 3	edge case 観測頻度 3 = Tier 1 昇格基準 codify	XS

頻度評価: 順位 85 は PR #119/#120/#121 で 3 PR 連続観測 により Frequency Medium 閾値到達済み = Tier 1 妥当性確定。順位 86 は dependent。87/88 は本セッションの運用知見を rule 化する meta-task。

Bundle f との関係: f = retry logic (rate-limit + 投稿エラー)、g = verdict logic (review_state 評価) で別軸。両者 land で post-pr-monitor が retry / verdict / state 全方向に堅牢化。

§8.A-2 Phase 5 dogfood 計画 (約 130 行)

ADR-038 §試験運用→本採用の 未達 2 条件 を達成する計画:

• 条件 1: 5 PR 以上の実 review サイクルで dogfood (現状 0 PR)
• 条件 3: Claude session の入力 token 削減効果体感 (現状未測定)

構成: P-0 (config opt-in) + P-1〜P-5 (5 PR)

PR	タスク	順位	Effort
P-0	pr-monitor-config.toml で [classifier] enabled = true opt-in	—	XS
P-1	Bundle g-1 (monitor state guard + test)	85+86	S+S
P-2	> vs >= boundary inconsistency lint rule	47	S
P-3	PowerShell (?i) フラグ自動検証 lint rule	7	S
P-4	overflow 統合テスト + 境界値 matrix (Bb-3)	76+77	M+S
P-5	Bundle f-1 (retry logic + ADR)	80+81+82	M+M+S

選定基準: Rust 実装系を中心 (CR の findings 数が多い)、docs-only / config-only タスクは除外、頻度確認済 Tier 1 を先行。

Self-contained な記述

別セッションで再開可能な構造 (本 doc 1 ファイルで全手順把握可):

• Setup 手順 (チェックリスト 4 項目、コマンド付き)
• 計測手順 (state.classified_findings 取得 jq クエリ等、6 項目の指標取得)
• 判定基準 (5 指標の目標値 + ADR-038 昇格条件 mapping)
• 既知の注意事項 5 件 (本セッションで観測した dogfood 阻害要因)
• session 跨ぎ運用ガイド (中断・再開・完了処理)
• 計測ログ template (P-0〜P-5 の結果を後から埋める)

設計のポイント

• docs-only PR (ADR-035 適用): コード変更なし
• §8.A-2 を docs/todo.md priority table に重複登録しない: dogfood 計画 = 既存 task の集合 + 計測 wrapper という性質。priority table = 単一 task trace、A-2 = 集合 task orchestration で責務分離
• 頻度確認方針との整合: Bundle g 順位 85 は 3 PR 連続観測で Tier 1 確定、Bundle f 順位 80-82 は本セッション 1 観測で頻度未確定のため §8.A-2 の最終段 P-5 に配置 (P-1〜P-4 中の dogfood で再観測あれば頻度確認達成)

検証結果

• quality_gate (lint/test/build/rust-test): PASS
• takt pre-push-review: APPROVE (1 iter, 4m 14s、ADR-035 docs-only 適用基準クリア)

scope 外 (本 PR では未対応)

• Bundle g 自体の実装 (順位 85-88) は別 PR (本 PR は登録のみ)
• §8.A-2 dogfood 実施は別セッション (本 PR は計画策定のみ)

Test plan

• cargo test --workspace --lib --bins 全 pass (前回 PR docs: ADR-038 textual fix + Bundle f task registration (post-PR #120 follow-up) #121 から変化なし、docs-only)
• markdownlint 全 pass (3 ファイル全て)
• takt pre-push-review APPROVE
• CR review で textual consistency 確認

関連

• docs/local-llm-offload-analysis.md: §8.A-2 を新規追加、§7 / §8.E 整合更新
• docs/todo.md: priority table 4 行追加 (順位 85-88) + Bundle g commentary
• docs/todo5.md: 4 件詳細エントリ追加
• PR docs: ADR-038 textual fix + Bundle f task registration (post-PR #120 follow-up) #121: 本 PR の origin (post-merge-feedback)

Summary by CodeRabbit

リリースノート

• Documentation
  • ADR-038仕様の明確化（confidence パラメータの統一）
  • Phase 5 検証計画と測定基準の詳細化
  • 監視システムの回帰防止タスク（状態遷移テスト、ガード条件強化）の追加
  • マルチPRチェーンと運用基準のワークフロー ガイドラインへの統合

[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: 00b32446-435e-48b0-854f-eff41d955d4a

📥 Commits

Reviewing files that changed from the base of the PR and between 6640dc7 and 6fa6558.

📒 Files selected for processing (3)

• docs/local-llm-offload-analysis.md
• docs/todo.md
• docs/todo5.md

---

📝 Walkthrough

Walkthrough

3つのドキュメントファイル（docs/local-llm-offload-analysis.md、docs/todo.md、docs/todo5.md）を更新し、PR #121 の post-merge フィードバック、ADR-038 の表記統一、Phase 5 dogfood の計測計画（P-0〜P-5：config opt-in と5PR dogfood、計測手順、判定基準、運用ガイド、ログテンプレ）、提案1 の依存条件の具体化（agreement rate ≥80% かつ session token 削減 ≥10%）、および Bundle g 由来の新規タスク4件（順位85〜88：verdict guard、state transition テスト、global rule codify）を反映。

Changes

Phase 5 Dogfood 計測計画と依存条件の具体化

Layer / File(s)	Summary
現状確認と前提条件の統一 docs/local-llm-offload-analysis.md	ADR-038 の confidence を action_confidence に統一、Bundle f のタスク登録完了、PR #121 のマージ状況、post-merge での Bundle g 追加、classifier の default OFF 状態を確認。
Dogfood 計測計画の定義 docs/local-llm-offload-analysis.md	§8.A-2 として P-0（config opt-in）〜P-5（5PR dogfood）の具体的な構成、各 PR ごとの Bundle・ルール・順位・期待 findings・想定難度を表形式で追加。計測手順（classified_findings 取得、agreement rate 評価、token・latency・fallback・言語制約違反率の算出）、判定基準、既知の注意事項、セッション跨ぎ運用ガイド、完了基準、計測ログテンプレを包含。
提案1 の依存条件具体化 docs/local-llm-offload-analysis.md	提案1（ollama-lint-screen）の依存条件を「§8.A-2 dogfood 完了後かつ agreement rate ≥80% かつ session token 削減 ≥10%」に具体化し、計測計画に紐づけた判定基準として明記。
新規タスク登録と実行順序更新 docs/todo.md, docs/todo5.md	推奨実行順序表に PR #121（Bundle g）由来の新規タスク4件（順位85〜88）を登録：cli-pr-monitor の verdict guard（review_state: not_found && findings: [] を pending として扱う）、state transition 回帰テスト、Multi-PR chaining の global rule codify、edge case 観測3回閾値の Tier 1 昇格基準。Bundle g セクションを追加し、Sub-PR 分割方針と Bundle f との責務分離を記述。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#121: PR #121 の post-merge フィードバックを直接反映する内容。ADR-038 の統一と Bundle f/g のタスク登録が連続。
• aloekun/claude-code-hook-test#112: Bundle f、ADR、todo/task 更新の post-merge フィードバックと governance に関わる重複するドキュメント変更。
• aloekun/claude-code-hook-test#89: docs/todo.md のタスク順序変更と bundle 関係性の更新、todo ワークフロー全般に関わる重複する変更。

🚥 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	プルリクエストのタイトルは、変更セットの主な内容である「Bundle g登録とPhase 5 dogfood計画」を正確かつ具体的に要約しており、ドキュメント更新の関連性を明確に示しています。
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.}

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

---

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