docs(todo): 論点 1-3 の対処 3 件を順位 393-395 へ登録し計画書の残作業へ反映 - #375
Merged
Conversation
📝 WalkthroughWalkthroughADR-072の判定を訂正し、夜間PRを通常PRへ変更した。週次レビューには、クローズ済みPRに残る浮きブランチの検出と削除提案を追加した。 Changes夜間ループ運用方針
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Contributor
🤖 PR Monitor 分析 (GitHub Actions バックストップ)
Applicable Findings (Critical / High / Major)該当なし（レビュー未着のため findings 自体が存在しない） Applicable Findings (Medium 以下)該当なし Filtered (not applicable)該当なし 次のアクション
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 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/harness-improvement-plan.md`:
- Line 174: Update the “採用率 2 週間測定” entry so its deadline matches the stated
2026-08-09 start date and two-week period by setting the judgment date to
2026-08-23. If retaining 2026-11-06, explicitly label it as the outer deadline
and add 2026-08-23 as the actual two-week judgment date.
- Around line 165-172: docs/harness-improvement-plan.md内のWP-17〜WP-19旧記述を、Phase
Bの訂正と順位393完了に合わせて更新または削除してください。特にLine 84-85および156-157の保留・不成立・Decision
11実測待ちの記述を現状へ同期し、Line 186周辺の--open-draft-prsとmax_open_draft_prsを、Line
172のautonomous系命名およびdraft廃止方針に合わせて置換または削除してください。
In `@docs/todo21.md`:
- Around line 287-296: Update the weekly-review documentation to explicitly
separate the data sources: nightly ranking exclusion must use only branch
existence from git ls-remote, while orphan-branch detection must combine closed
PR metadata with branch existence. State that detection produces deletion
proposals only and never performs automatic deletion.
🪄 Autofix
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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: a16d780a-611f-4dab-b100-08c468c9d4c4
📒 Files selected for processing (3)
docs/harness-improvement-plan.mddocs/todo-summary2.mddocs/todo21.md
Comment on lines
+165
to
+172
| **達成したこと**: 台帳（[claude-code-web-tasks.md](claude-code-web-tasks.md)）から Rust 分類関数が選んだ 1 タスクを、kill-switch（ADR-066）と背圧（ADR-071）の上で無人実装し PR 作成で止めるループが本番稼働に入った（2026-08-08 に schedule 初回実走で draft PR [#365](https://github.com/aloekun/claude-code-hook-test/pull/365) = `claude/nightly-203` を作成）。定常運用開始前に必須とした prompt injection 対策 4 件（順位 378-381 → ADR-072 決定 12-14 + [ADR-035](adr/adr-035-doc-evaluation-policy.md) / `lib-docs-policy` の同期）と外部設定の実体記録（順位 384 → ADR-072 § 外部設定の実体）も完了。実走スモーク 10 項目は **8 充足 / 1 訂正（Phase B 自動起動 → 経路は生存、順位 393 で記述訂正）/ 1 保留（トークン露出）**で、一覧は ADR-072 § 実走スモークが正。停止側実測は WP-17 の残課題（[ADR-066](adr/adr-066-autonomy-global-kill-switch.md) bounded lifetime の variable 側 3 状態）も同時に埋めた（ADR-066 § 実走観測 2）。 | ||
|
|
||
| #### 残作業 | ||
|
|
||
| | 内容 | 管理先 | 期限 / 条件 | | ||
| |---|---|---| | ||
| | **順位 393: ADR-072 決定 11 の撤回記録 + Phase B 判定の訂正** — App token 投稿は CodeRabbit に無視されると 2026-08-09 に実測確定（#373 で投稿者別の対照: bot は 10 時間無反応 / 人間は 4 秒で反応）。Phase B も「不成立」ではなく起動契機が無かっただけと判明 | [todo21.md](todo21.md) | 🚀 Tier 1、順位 394 の前提 | | ||
| | **順位 394: draft 廃止 + 背圧命名の `autonomous` 系への統一** — `--draft` 除去 / **背圧計数の `.isDraft` 除去（必須、落とすと ADR-052 原則 5 違反）** / 決定 11 step 撤去 / ADR-052 分類表の本体改訂 / ADR-019・071・072 の同期。12 ファイル 132 箇所、size gate 超過時は 2 本へ分割し [ADR-069](adr/adr-069-pr-chain-declaration.md) 宣言を付ける | [todo21.md](todo21.md) | 🚀 Tier 1 | |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
WP-17、WP-18、WP-19の旧記述を同期してください。
Line 165とLine 171は、Phase Bの訂正と順位393の完了を示しています。しかし、Line 84はPhase B起動を保留中と記載し、Line 85はDecision 11の実測を残課題としています。Lines 156-157も、Phase Bを「不成立」とし、Decision 11の実測待ちを残しています。
さらに、Line 172は背圧命名をautonomous系へ変更していますが、Line 186は--open-draft-prsとmax_open_draft_prsを使用しています。重複した状態記録とCLI名を、このPRで更新または削除してください。現状では、同じ計画書に矛盾する運用指示が残ります。
🤖 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/harness-improvement-plan.md` around lines 165 - 172,
docs/harness-improvement-plan.md内のWP-17〜WP-19旧記述を、Phase
Bの訂正と順位393完了に合わせて更新または削除してください。特にLine 84-85および156-157の保留・不成立・Decision
11実測待ちの記述を現状へ同期し、Line 186周辺の--open-draft-prsとmax_open_draft_prsを、Line
172のautonomous系命名およびdraft廃止方針に合わせて置換または削除してください。
| | スモーク残 1 項目: `cargo` サブプロセスへのトークン露出（順位 374 の残り。**意図的保留** — 初版 probe の設計欠陥を解消した安全な probe を設計してから 1 回で観測。ADR-072 決定 5 の Bash 再付与判断の材料でもある） | ADR-072 § 残課題 | 急がない（Bash 非付与が保守側） | | ||
| | 決定 11 の実測: CodeRabbit が bot（App）投稿の `@coderabbitai review` に反応するか。無反応なら 3 択で再判断。`coderabbitai[bot]` allowlist の要否も同時に再判定（WP-17 残課題と共通） | ADR-072 § 残課題 | 次回夜間 run で最優先 | | ||
| | **採用率 2 週間測定**（WP 全体の受け入れ基準。人間がマージした割合 50% 超で継続・拡大 — 参考値であり統計的意味は無い、ADR-072 § 欠点）。測定は weekly-review の自律アクション棚卸し（WP-19 ステップ 3）へ載せて仕組み化。**開始起点 = スモーク完走後（2026-08-09 ユーザー決定）**。完走の定義は決定 11 の実測完了までとし、意図的保留のトークン露出は完走条件に含めない | ADR-072 § 試験運用判断基準 | 2026-11-06 までに判定 | | ||
| | **採用率 2 週間測定**（WP 全体の受け入れ基準。人間がマージした割合 50% 超で継続・拡大 — 参考値であり統計的意味は無い、ADR-072 § 欠点）。測定は weekly-review の自律アクション棚卸し（WP-19 ステップ 3）へ載せて仕組み化。**開始起点 = スモーク完走後（2026-08-09 ユーザー決定）**。完走の定義を順位 394 の実走確認（PR に CodeRabbit の初回自動レビューが付くこと）までとし、意図的保留のトークン露出は完走条件に含めない | ADR-072 § 試験運用判断基準 | 2026-11-06 までに判定 | |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
2週間測定の期限を開始日と整合させてください。
Line 174は開始日を2026年8月9日、測定期間を2週間、判定期限を2026年11月6日としています。2026年8月9日から2週間後は2026年8月23日です。期限を2026年8月23日に変更するか、2026年11月6日を外側の期限と明記し、実際の2週間判定日を追加してください。
🤖 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/harness-improvement-plan.md` at line 174, Update the “採用率 2 週間測定” entry
so its deadline matches the stated 2026-08-09 start date and two-week period by
setting the judgment date to 2026-08-23. If retaining 2026-11-06, explicitly
label it as the outer deadline and add 2026-08-23 as the actual two-week
judgment date.
Comment on lines
+287
to
+296
| ### 週次レビューで浮きブランチを検出し削除を提案する | ||
|
|
||
| > **動機**: 2026-08-09 に、クローズ済み PR [#365](https://github.com/aloekun/claude-code-hook-test/pull/365) のブランチを手動削除したことで [ADR-072](adr/adr-072-nightly-todo-loop.md) 決定 3 の除外マーカーが消え、同じ順位 203 が再選択された (PR #373)。**決定 3 自体は設計どおり動作している** — ブランチの存在が着手済みマーカーであり、それが人手で消えたことが原因。 | ||
| > | ||
| > **方針 (ユーザー判断 2026-08-09)**: 判定は現行の `git ls-remote` による **branch 単一ソースのまま維持する**。「クローズ済み PR も見る」案は判定ソースが 2 つになり、fail-closed の単純さ (一致なしでも exit 0 + 空出力で「0 件」と「取得失敗」を取り違えない) を崩すため**採らない**。代わりに浮きブランチを週次で片付け、放置によるタスク滞留を解消する。 | ||
| > | ||
| > - **`claude/nightly-*` も削除提案の対象に含める** (除外しない)。ブランチが残る間そのタスクが選べない期間 (最大 7 日程度) は許容する — 自動実行できる todo はほとんどが改善タスクで急がず、重要な todo はメインセッションで消化するため | ||
| > - **削除後の再挑戦も許容する**。現状のクローズは夜間ループの機能不全に起因するもので、正常動作後は採用方向の選択が多くなる見込み | ||
| > | ||
| > **対処案**: weekly-review に「クローズ済み PR の残存ブランチ」検出を追加する。既存の観点⑤ (`review-todo-whole` facet) へ相乗りさせるか決定論的 scan として持つかは実装時判断 ([ADR-031](adr/adr-031-weekly-review-pipeline.md) の構成に従う)。**削除の実行は提案までとし、自動削除はしない** ([ADR-022](adr/adr-022-automation-responsibility-separation.md) / [ADR-028](adr/adr-028-pnpm-create-pr-gate.md))。 |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
順位除外と浮きブランチ検出のデータソースを分離して明記してください。
Line 291 は、夜間の判定を git ls-remote によるブランチ単一ソースで維持すると記載しています。Line 296 は、クローズ済み PR の残存ブランチを検出すると記載しています。git ls-remote だけでは、ブランチがクローズ済み PR に属するか判定できません。
次の2つの契約を明記してください。
- 夜間の順位除外: ブランチの存在だけを使用する。
- 週次の浮きブランチ検出: クローズ済み PR の情報とブランチの存在を組み合わせる。削除は承認後の提案に限定する。
🤖 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/todo21.md` around lines 287 - 296, Update the weekly-review
documentation to explicitly separate the data sources: nightly ranking exclusion
must use only branch existence from git ls-remote, while orphan-branch detection
must combine closed PR metadata with branch existence. State that detection
produces deletion proposals only and never performs automatic deletion.
Contributor
🤖 PR Monitor 分析 (GitHub Actions バックストップ)
Applicable Findings (Critical / High / Major)
Applicable Findings (Medium 以下)
Filtered (not applicable)該当なし 次のアクション
|
1 task
This was referenced Aug 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
autonomous系へ揃えるtodo-summary2.mdの順位 table に 3 行を追加 (ADR-033)Context
2026-08-09 に PR #373 (夜間ループが作成した draft PR) で 2 件を実測し、ユーザー判断を経て方針を確定したもの。
1 件目は CodeRabbit のトリガー。同一 PR・同一文言・同一設定で、投稿者だけが異なる 2 回の対照が取れた — App token (bot) の投稿は約 10 時間無反応、人間の投稿は 4 秒で応答しレビューが走った。ADR-072 決定 11 が未検証事項に挙げていた「bot 投稿を無視する実装」の仮説がそのまま実証された形になる。明示トリガーという方式自体は有効で、ADR-019 の fix push 後トリガーは現在も機能している (あちらは
cli-pr-monitorがユーザー資格情報で投稿しているため)。決定 11 は「ADR-019 と同型」と判断したが、同型だったのはコマンド文字列だけで投稿者の種別が違っていた。あわせて Phase B 自動起動の判定も誤りと判明した — CodeRabbit が動いた時点でissue_comment経路が発火し Phase A が起動しており、経路は生存している。2 件目はタスクの重複選択。クローズ済み PR #365 のブランチを手動削除したことで ADR-072 決定 3 の除外マーカーが消え、同じ順位 203 が再選択された。決定 3 自体は設計どおり動作しており、判定は現行の branch 単一ソース (
git ls-remote) を維持する。「クローズ済み PR も見る」案は判定ソースが 2 つになり fail-closed の単純さを崩すため採らない。代わりに浮きブランチを週次で片付け、最大 7 日程度の滞留と削除後の再挑戦は許容する。順位 394 は上記 1 件目への構造的な対処にあたる。draft をやめれば
auto_review.enabled: trueの初回レビューに自然に乗り、回避策 (決定 11) そのものが不要になる。トリガーがユーザー指示か自動採択かで扱いを区別せず、commitment 点をマージ 1 点に集約するというユーザー判断に基づく。背圧計数が.isDraftを条件にしているため、--draftの除去と同時に修正しないと背圧が完全に無効化される (ADR-052 原則 5 が禁じる状態) 点を、todo の作業計画と優先度の Adoption Risk 両方に明記した。本 PR は登録のみで、実装は含まない。順位 393 → 394 の順で着手し、395 は独立して並行可能。
Validation
pnpm pushpre-push review: verdict=APPROVE (simplicity / security とも、1 iteration・161s、fix step の発生なし)rust-lint-testは skip)pnpm lint:docs: OK (preamble + cross-ref + priority-inversion)References
Summary by CodeRabbit