Skip to content

docs(todo): CodeRabbit の auto レビュー再開に review-request 等を追従させる計画を順位 520 として起票する - #503

Open
aloekun wants to merge 1 commit into
masterfrom
docs/coderabbit-auto-followup
Open

aloekun wants to merge 1 commit into
masterfrom
docs/coderabbit-auto-followup

Conversation

@aloekun

@aloekun aloekun commented Sep 16, 2026

Copy link
Copy Markdown
Owner

概要

CodeRabbit の auto レビュー再開に review-request / 再レビュー経路を追従させる 順位 520 (Tier 3) を起票する。docs のみ (todo26.md エントリ + todo-summary3.md 行)。実装は別 PR (workflow は Guard 禁止パス、cli-pr-monitor は判定共有の設計を伴うため human lane)。

背景

CodeRabbit が 2026-09-10 頃から、star 10 未満・bot 作成 PR でも PR 作成直後 (7〜8 秒) に自発 walkthrough を出すようになった。これは .coderabbit.yaml と review-request.yml が土台にしてきた「star 10 未満 → auto レビューされない → 人間 PAT で @coderabbitai review を要求する」(2026-09-08 実測、ADR-019 amendment) という前提の崩壊である。

昨夜の夜間 PR #502 で review-request CI が誤 red になったのがきっかけ。レビュー自体は成功しており (指摘 0)、失敗は補助ジョブの二重依頼 + 時系列競合による誤検知だった。

実測 (bot PR の初動タイムライン)

PR 日付 CodeRabbit の初動 review-request
#477 09-04 要求 → ack → walkthrough success (旧挙動)
#483 09-06 RATE-LIMITED (「already reviewed commits は再レビューしない」)
#487 09-07 RATE-LIMITED
#494 09-10 自発 walkthrough (要求より前) failure
#502 09-15 自発 walkthrough (要求より前) failure

境界は 09-06〜09-10。誤 red の直接原因は (a) auto walkthrough の後に review-request が要求を重ねる二重依頼、(b) 判定が comment_id > SINCE_ID で「要求より後のレビュー」しか見ず、要求の数秒前の walkthrough を構造的に見落とすこと。

起票した内容 (順位 520)

方針は fallback 化 (2026-09-16 ヒアリングで決定)。3 経路すべてが「auto が効かない」前提に立つため、auto が効くかを 1 つの共有判定関数で確かめ、効いていれば各経路は自分の起動を skip する。即撤去はしない — auto の継続性は n=2 の観測で、CodeRabbit 挙動は短期間で変わりうるため。判定の集約は ADR-081 の分散解消でもある。

経路 追従後
review-request.yml 要求前に現 head の auto walkthrough を確認 → あれば要求せず success
cli-pr-monitor trigger_review.rs (順位 321) Trigger review チェックボックスが無ければ (= auto 済み) skip
cli-pr-monitor review_trigger / rate_limit fix push 後の再レビューが auto で来るなら明示要求を skip

完了基準は「auto があれば二重依頼せず success / auto が無ければ従来どおり PAT 要求 (両挙動に耐える) / 3 経路が同一判定関数を通る (写経なし)」。

検証

  • pnpm lint:docs 7 検査 OK / markdownlint 0 error
  • cargo test -p lib-ledger 195 passed (エントリ ⇄ summary の 1:1 対応を含む)
  • pre-push takt: simplicity / security とも APPROVE

🤖 Generated with Claude Code

Summary by CodeRabbit

  • ドキュメント
    • CodeRabbit の自動レビュー再開に伴う review-request および再レビュー経路の対応方針を追加しました。
    • 自動レビューの有無を共有判定し、重複起動を避けるフォールバック設計、作業計画、完了基準を文書化しました。
    • 推薦実行順序テーブルに関連タスクを追加しました。

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 16, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 649942fc-6057-4fab-9ed6-cdcccd63fc15

📥 Commits

Reviewing files that changed from the base of the PR and between e5bf94c and 0326dd0.

📒 Files selected for processing (2)
  • docs/todo-summary3.md
  • docs/todo26.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

CodeRabbit の auto レビュー再開に対応する順位 520 のタスクを追加した。review-request.ymlcli-pr-monitor の fallback 化方針、作業計画、完了基準、Guard 禁止パスを文書化した。

Changes

CodeRabbit autoレビュー対応

Layer / File(s) Summary
対応タスクと実行順序の追加
docs/todo26.md, docs/todo-summary3.md
順位 520 のタスクを追加した。CodeRabbit の auto レビュー有無を共有判定し、auto が有効な場合に各経路の起動を skip する fallback 化方針を記載した。作業計画、完了基準、Guard 禁止パスの注記も追加した。推薦実行順序テーブルに同タスクを登録した。

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 0326d

This documentation-only change does not alter production behavior and is mergeable.

🚥 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 タイトルは、CodeRabbit の auto レビュー再開に review-request などの経路を追従させる計画を順位 520 として文書化するという、PR の主な変更を正確かつ具体的に要約しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/coderabbit-auto-followup

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.

@github-actions

Copy link
Copy Markdown
Contributor

🤖 PR Monitor 分析 (GitHub Actions バックストップ)

  • トリガー: issue_comment (created) / 実行 run
  • CI: rust (ubuntu-latest) pending, rust (windows-latest) pending, request skipping, CodeRabbit pending (Review in progress)
  • レビュー状況: 未実施 (陽性証拠なし) — pulls/503/reviews は空、インライン指摘も 0 件。会話コメントにある CodeRabbit の投稿は "Currently processing new changes... please wait" という進行中プレースホルダで、head 0326dd0c に対する完了済みレビューではない
  • Verdict: user_decision

Applicable Findings (Critical / High / Major)

(該当なし)

Applicable Findings (Medium 以下)

(該当なし)

Filtered (not applicable)

(該当なし — レビュー指摘自体がまだ 0 件)

diff 概要

  • 変更ファイル: 2件 (docs/todo-summary3.md +1 行、docs/todo26.md +55 行)、追加のみ・削除なし
  • 内容: todo 台帳に順位 520 (CodeRabbit auto レビュー再開に review-request 等の判定ロジックを追従させる設計方針) を新規起票するドキュメントのみの変更。コード変更なし

次のアクション

  • CodeRabbit のレビュー完了 (walkthrough 確定) と rust CI (ubuntu/windows) の結果を待ってから再判定する
  • 現時点では人間・他 bot による実質レビューが 0 件のため、マージ判断は保留 (user_decision)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant