docs(adr-031): Phase E land — ADR-031 本採用昇格 + handoff doc retire + 2026-06-01 dogfood 採用 3 件 by aloekun · Pull Request #192 · aloekun/claude-code-hook-test

aloekun · 2026-06-02T05:12:19Z

概要

ADR-031 (週次プロジェクト全体レビューパイプライン) Phase E land — 試験運用 → 承認済みへ昇格。Phase D + Phase E の 2 回 dogfood で本採用条件を充足したため本採用化、handoff doc retire、関連 todo entry close。

Phase E dogfood (2026-06-01) 結果

Metric	観測値	評価
Wall-clock	takt 全体約 6 分	✅
Findings	6 件 (high=1, medium=4, low=1)	✅
採用率	50% (3/6)	✅
累計採用 (Phase D + E)	8/18 = 44%	✅
Context window 圧迫	なし	✅
False positive	なし	✅

Phase E 採用 findings (3 件)

WR-2026-06-01-S01 [High, simplicity] subprocess utils (drain_pipe / wait_with_timeout / run_cmd / MAX_LINES) 4 crate 横断重複 → 既存 todo9.md combine_output extract entry に augment (scope 拡大、lib-subprocess-utils へ命名変更)
WR-2026-06-01-C02 [Medium, security] cli-merge-pipeline/feedback.rs の owner_repo 検証非対称 (defense-in-depth 欠如) → docs/todo.md 「週次レビュー採用 (2026-06-01)」に新規追加
WR-2026-06-01-A01 [Medium, architecture] CLAUDE.md ADR-032 dead-pointer (CLAUDE.md ADR index 跳び + ADR-033 内 4 箇所参照) → docs/todo.md 新規追加

Phase E 保留 + 却下

保留: S02 (hook バイナリ 800 行超過) / C01 (PR_MONITOR_STATE_FILE_OVERRIDE)
却下: C03 (env var override 監査ログ、ADR-039 設計通り)

ADR-031 本採用化 + ADR-039 retirement workflow

ADR-031 ステータス: 試験運用 (2026-04-27) → 承認済み (2026-06-01)
### 採用判定の根拠 (Phase E dogfood 観測結果) section を追加 (観測メトリクス表 + 判定根拠)
実装タスク section に Phase B/C/D/E land 履歴を追記
将来の展望 section から「Phase E dogfood 安定後の本採用化」を打消し済み表記に
ADR-039 retirement workflow: 採用経路 = status update のみで十分 (3 点セット = config opt-in / kill-switch / bounded lifetime は本採用後も guard として維持)

handoff doc retire (docs-governance § Retirement Workflow 4 step 適合)

docs/handoff-weekly-review-phase-c-onwards.md (157 行) 物理削除
永続価値は ADR-031 § 採用判定の根拠に migrate 済
残存参照 0 件 (ADR-031 内の 1 参照は path-free 過去形に書換)

todo / cross-ref cleanup

削除: docs/todo.md ADR-031 entry (200 行) + docs/todo-summary.md 順位 8 (ADR-031 Phase B 実装)
置換: 順位 21 / 38 / 153 / 154 (todo-summary.md) + todo9.md 153/154 entry + todo4.md L3 weekly entry の「順位 8 / Phase B follow-up」参照を「ADR-031 本採用 2026-06-01」参照に統一
feedback_todo_no_history.md 運用ルール (完了タスクは ADR/仕組みに反映後 todo.md から削除) を実適用

pre-push pipeline

takt pre-push-review (simplicity facet) = APPROVED (1 iteration, 6m 2s)
Reviewer 確認項目: docs-governance compliance / todo.md hygiene / ADR-031 promotion rationale / cross-ref coherence — 全て OK と verdict

Test plan

CI green
CodeRabbit walkthrough で adoption rationale + retire workflow が伝わるか確認
handoff doc 削除に伴う dead-link 発生がないこと (grep -rn 'handoff-weekly-review-phase-c-onwards' 0 件確認済)
ADR-031 § 採用判定の根拠 table の数値整合 (Phase D + E 観測値) を再確認

Summary by CodeRabbit

リリースノート

Chores
- 内部ドキュメントおよびプロジェクト管理ファイルを更新しました。ユーザーに対する機能変更や影響はありません。

…26-06-01 dogfood 採用 3 件 ## 採用判定 (ADR-031 試験運用 → 承認済み) Phase D (2026-05-30) + Phase E (2026-06-01) 計 2 回の dogfood で計 18 findings / 採用 8 件 / 却下 8 件 / 保留 2 件を観測。wall-clock (約 5 分以内) / context window 圧迫なし / finding 品質 (採用率 44%、false positive なし) / systemic finding 検出力 (cross-PR drift / dead-pointer / partial overlap を実検出) など、scope 軸の空白埋めとして機能を確認。本採用化判定済。 ## Phase E dogfood (2026-06-01) 採用 3 件 + 保留 2 件 + 却下 1 件 - WR-2026-06-01-S01 (High, simplicity): subprocess utils 4-5 crate 横断重複 → 既存 todo9.md combine_output extract entry に augment (scope 拡大、lib-subprocess-utils へ命名変更) - WR-2026-06-01-C02 (Medium, security): cli-merge-pipeline/feedback.rs の owner_repo 検証非対称 → docs/todo.md 「週次レビュー採用 (2026-06-01)」に新規追加 - WR-2026-06-01-A01 (Medium, architecture): CLAUDE.md ADR-032 dead-pointer (4 箇所) → docs/todo.md に新規追加 - 保留: WR-2026-06-01-S02 (hook バイナリ 800 行超過) + WR-2026-06-01-C01 (PR_MONITOR_STATE_FILE_OVERRIDE) - 却下: WR-2026-06-01-C03 (env var override 監査ログ、ADR-039 設計通り) ## handoff doc retire (Cross-File Reference Lifecycle 適合) docs/handoff-weekly-review-phase-c-onwards.md を物理削除。永続価値は ADR-031 § 採用判定の根拠に migrate 済 (dogfood 観測メトリクス + 本採用判断根拠)。残存参照は ADR-031 内 1 箇所のみ (path-free な過去形記述に書き換え済)。 ## todo.md / todo-summary.md cross-ref 清掃 - todo-summary.md 順位 8 (週次レビュー Phase B) 削除 - todo-summary.md 順位 21/38/153/154 の「順位 8」参照を ADR-031 (本採用 2026-06-01) 参照に置換 - todo.md ADR-031 entry (200 行) 削除 (feedback_todo_no_history.md 適用) - todo9.md 153/154 entry の「順位 8 follow-up」を「ADR-031 weekly-review 拡張」に置換 - todo4.md L3 weekly mutants entry の Phase B 依存記述を本採用後の独立着手記述に更新 ## ADR-039 retirement workflow 採用経路 = 試験運用 → 本採用 (ADR status update のみで十分、新規 ADR 不要)。config opt-in (`WeeklyReviewReminderConfig::enabled`) / kill-switch (revert path) / bounded lifetime (dogfood 採用条件) の 3 点セットは本採用後も guard として維持。

coderabbitai · 2026-06-02T05:12:30Z

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: e3df3096-dac2-465f-9c9e-5f2b0f9bd598

📥 Commits

Reviewing files that changed from the base of the PR and between 12be361 and bef30e7.

📒 Files selected for processing (9)

.claude/weekly-review-deferred.json
.claude/weekly-review-last-run.json
.claude/weekly-reviews/2026-06-01.md
docs/adr/adr-031-weekly-review-pipeline.md
docs/handoff-weekly-review-phase-c-onwards.md
docs/todo-summary.md
docs/todo.md
docs/todo4.md
docs/todo9.md

💤 Files with no reviewable changes (1)

docs/handoff-weekly-review-phase-c-onwards.md

📝 Walkthrough

概要

ADR-031 週次レビューパイプラインが 2026-06-01 に本採用承認され、Phase E dogfood の観測結果と判定がドキュメントに記録されました。同時にハンドオフドキュメントが削除され、関連タスク実行順序と今後の拡張計画が再整理されました。

変更内容

ADR-031 本採用と後続計画

レイヤー / ファイル	概要
本採用メタデータと判定の記録 `docs/adr/adr-031-weekly-review-pipeline.md`, `.claude/weekly-review-last-run.json`, `.claude/weekly-reviews/2026-06-01.md`, `.claude/weekly-review-deferred.json`	ADR-031 のステータスを「試験運用」から「承認済み（2026-06-01）」へ更新し、Phase E dogfood の観測メトリクス（採用/却下/保留の内訳、検出品質、採否対話 UX、重複検出 MVP+1 実装状況）を記録。週次レビュー実行メタデータ（最終実行日時、レポート保存先、findings 統計）を 2026-06-01 版に更新し、報告書と延期 findings を新規作成。
ハンドオフドキュメントの削除 `docs/handoff-weekly-review-phase-c-onwards.md`	週次レビューパイプラインの Phase C 以降に関する試験運用計画・実装工程・retirement 条件を記載していたハンドオフドキュメント（157 行）をリポジトリから削除。
タスク実行順序テーブルの再整理 `docs/todo-summary.md`	推奨実行順序テーブルで ADR-031 Phase B エントリ（順位 8）を削除し、前後の Tier 2/3 エントリを再配置。ADR-032 PR-γ（順位 21）の依存を「順位 8 dogfood」から「ADR-031 dogfood（2026-06-01 採用済）+ 順位 20」へ更新。review-harness-whole・review-todo-whole（順位 153/154）の前提を「ADR-031 本採用後の Phase B+1 拡張」へ変更。
本採用判定と今後タスクの追加 `docs/todo.md`	「現在進行中」先頭に 2026-06-01 付の週次レビュー採用タスク（owner_repo 検証追加、ADR-032 reserved スタブ追加、adr-033 dead-pointer 参照修正）を新規追加。従来「現在進行中」にあった大規模な ADR-031 計画エントリ一式を削除。
関連タスク計画の参照・位置づけ更新 `docs/todo4.md`, `docs/todo9.md`	docs/todo4.md では ADR-031 本採用を前提にした動機・背景を更新し、facet 拡張/aggregate 前 pre-step としての独立着手可能性を明記。docs/todo9.md では review-harness-whole の依存を Phase B+1 に変更、review-todo-whole facet（観点 ⑤ ⑦、cross-file 重複検出）と file size pre-step を新規追加、subprocess utils 重複 extract スコープを拡大（combine_output のみ → drain/wait/run 系ユーティリティと MAX_LINES 定数不整合まで対象化）。

評価

🎯 2 (Simple) | ⏱️ ~12 分

関連する可能性のある PR

aloekun/claude-code-hook-test#81: ADR-031 週次レビューパイプラインの Phase A 計画を最初に導入した PR で、本 PR の ADR-031 本採用判定の直接的な前提。
aloekun/claude-code-hook-test#183: docs/adr/adr-031-weekly-review-pipeline.md の Phase A～E 参照と実装タスク欄を同じ方針で更新した PR。
aloekun/claude-code-hook-test#89: docs/todo.md と docs/todo-summary.md の推奨実行順序テーブルとタスク優先度を更新した同種の 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タイトルは、ADR-031の本採用昇格、handoff docの削除、2026-06-01の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.}

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

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

…nup (#193) * docs(adr-031): § Adoption Criteria threshold 追加 + ADR-039 cross-ref (PR #192 T3-#5) PR #192 post-merge-feedback Tier 3 #5 採用。Phase E land 時 § 採用判定の根拠は観測値の記録のみで「閾値」が暗黙だった。5 閾値 (採用率 ≥ 40% / wall-clock ≤ 10 分 / FP ≤ 5% / context 圧迫なし / systemic 検出力) を ADR-031 inline で永続記録、将来 trial ADR の採用判定で参照可能化。 ADR-039 § 関連にも back-link を追加し双方向 link 形成、§ Bounded lifetime の 3 値判定 (採用 / 却下 / 継続) の具体化例として参照可能。Tier 3 #6 (ADR-039 audit) の価値も部分吸収。 * docs(todo): Bundle CR-RL stale entry cleanup (順位 167/168/169 — PR #185 で land 済) PR #185 (commit 7f8b613) で Bundle CR-RL の実装 3 件は全て land 済: - 順位 167: RATE_LIMIT_MARKERS multi-variant 配列化 (main.rs:261) - 順位 168: 新 format fixture 3 variant (full / minutes-only / mixed) - 順位 169: ADR-018 lines 185-186 multi-variant 表記 + ADR-034 § 既知 format 一覧 + § 検出 logic 更新手順 todo9.md / todo-summary.md の stale entry を削除して in-progress を反映。 memory feedback_verify_task_not_already_done の本来用途 (= 既 land 済タスクを stale entry 削除に再目的化) を実適用。

本 PR は以下 4 つの作業を 1 コミットに統合: ## A. PR #196 post-merge-feedback 採用 2 件 (順位 193, 194 登録) PR #196 (Bundle 195-FB) post-merge-feedback 8 件のうち 2 件採用、6 件却下/様子見。採用 (todo10.md に entry 追加、todo-summary.md table に行追加): - 順位 193 (T2): Companion helper group 署名整合 compile-time validation test - 順位 194 (T3): development-workflow.md \"1. Plan First\" に Codification 重複確認 step 追記却下/様子見の詳細は .claude/feedback-reports/196.md 参照。 ## B. queue 棚卸し (順位 ≤ 100 の 32 件を audit) 8 件削除 + 7 件改訂で queue の signal/noise 改善: 削除 8 件 (既存 land 確認、または動機失効): - 順位 41 (Bundle Y2 効果定量計測): 動機の主軸失効 (Bundle Z 完成 + Z2 不採用) - 順位 42, 43, 46 (rate-limit auto-retry 系): PR #97/#113/#129/#185 段階 land 完了 - 順位 45 (--list-findings Rust モード): PR #101 で land 済 - 順位 57 (Aggregation cap integration test): PR #171 で land 済 - 順位 93 (coding-style.md partial fix 例追加): ~/.claude/rules/common/ coding-style.md に既に section 存在 - 順位 97 (with_num_ctx serialization test): lib.rs:494 で mockito test 実体存在改訂 7 件 (Status update 2026-06-06 を front-matter に追加、現状反映): - 順位 11: ADR-018 park / ADR-030 短命プロセス移行後の再現確認が前段 - 順位 19: ADR-037/043/PR #194 land 後の残余 case baseline 観測が前段 - 順位 27: Phase D-7 = PR #154 land 済を反映 - 順位 38: ADR-031 採用昇格済 (PR #192) → Bundle W/X land のみ残依存 - 順位 40: PR #175 push-runner bookmark_check 実装済 → skill 側は二重防御に縮小 - 順位 51: 採用案 C (fix.md instruction 追加) は land 済、残作業 = dogfood 観測のみ - 順位 92: ADR-038 採用昇格済 (PR #156) で Phase d 運用入り、動機書き換え cross-reference 修復: - 順位 49: 旧依存 Bundle a Sub-PR 2 (順位 42/43/46) 消滅を反映 - 順位 61: 旧依存順位 45 land 済を反映 ## C. todo9.md → todo11.md 分割 todo9.md が 75KB / 890 行に到達し読み取り安定性閾値 (50KB) 超過のため分割: - todo9.md (37KB / 454 行): 既存ルール仕組み化バンドル (順位 146-151) + 週次レビュー拡張 (順位 152-154) を保持 - todo11.md (41KB / 453 行、新規): PR-specific follow-up entries 10 件 (順位 157, 160, 161, 162, 163, 165, 170, 171, 172, 173) theme-based split で意味的分離 + 両ファイルとも 50KB 閾値以下に収まる。 todo-summary.md table の file 参照を Python script で一括更新 (10 件)、 todo-summary.md 冒頭の \"追加先ファイル\" 説明を todo10.md に更新。 ## D. 順位 177 優先度引上げ PostToolUse hook ファイルサイズ検出 task が 4 回目の同型観測に到達 (PR #133 + #172 + #186 + 本セッション = Very High frequency)。CLAUDE.md code-review.md \"同型 finding の閾値判定\" (3 観測 = Tier 1 昇格) を超え systemic risk 閾値到達。 3 箇所同期更新: - todo10.md entry 本体に Status update 2026-06-06 blockquote 追加、優先度を \"Tier 1\" → \"Tier 1 (優先実装)\" に格上げ - todo-summary.md table 行で Tier 列を太字 + 注記、dependency 列に urgency note - todo-summary.md 末尾の戦略 note に \"直近優先 (2026-06-06 ユーザー指示)\" 段落を新設、Bundle 195-FB-Followup (順位 193 + 194) の次の PR で消化推奨と specific call-out ## 統計 - 10 ファイル変更 (1 新規) - ~670 lines insertions, ~910 lines deletions (net ~240 行削減) - 全 todo*.md が 50KB 閾値以下に収まる (todo9: 37KB, todo10: 36KB, todo11: 41KB) ## 参照 - .claude/feedback-reports/196.md (採否確定 commit、PR #196 由来) - memory feedback_post_merge_feedback_adoption_requires_user_approval per ユーザー承認済 - memory feedback_todo_no_history (削除は痕跡なし、コメントマーカー不使用) - ADR-035 (docs-only PR 評価ポリシー) - ADR-033 (採番管理簡素化、順位 renumber は避けて semantic markers で表現)

aloekun merged commit 8c71029 into master Jun 2, 2026
1 check passed

aloekun deleted the weekly-review-phase-e-land branch June 2, 2026 11:20

aloekun mentioned this pull request Jun 3, 2026

docs: ADR-031 § Adoption Criteria threshold + Bundle CR-RL stale cleanup #193

Merged

6 tasks

aloekun mentioned this pull request Jun 6, 2026

docs(todo): queue 棚卸し + todo9 → todo11 分割 + 順位 177 優先度引上げ #197

Merged

5 tasks

This was referenced Jun 8, 2026

feat(cli-docs-lint): priority_inversion validator + 順位 19 retire + T2-1/T3-2 採用 #200

Merged

docs(adr): ADR-044 subprocess utility extraction の境界判定 — 順位 173 family close #209

Merged

coderabbitai Bot mentioned this pull request Aug 13, 2026

docs(weekly-review): 台帳昇格候補チェックを必須出力化し週次レビュー採用 6 件を todo 登録する #395

Merged

Navigation Menu

docs(adr-031): Phase E land — ADR-031 本採用昇格 + handoff doc retire + 2026-06-01 dogfood 採用 3 件 - #192

docs(adr-031): Phase E land — ADR-031 本採用昇格 + handoff doc retire + 2026-06-01 dogfood 採用 3 件#192
aloekun merged 1 commit into
masterfrom
weekly-review-phase-e-land

aloekun commented Jun 2, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Jun 2, 2026 •

edited

Loading

概要

変更内容

評価

関連する可能性のある PR

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

aloekun commented Jun 2, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

概要

Phase E dogfood (2026-06-01) 結果

Phase E 採用 findings (3 件)

Phase E 保留 + 却下

ADR-031 本採用化 + ADR-039 retirement workflow

handoff doc retire (docs-governance § Retirement Workflow 4 step 適合)

todo / cross-ref cleanup

pre-push pipeline

Test plan

関連

Summary by CodeRabbit

リリースノート

Uh oh!

coderabbitai Bot commented Jun 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

概要

変更内容

評価

関連する可能性のある PR

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

aloekun commented Jun 2, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Jun 2, 2026 •

edited

Loading