Skip to content

cross-review: 見送りの出直し・指摘 0 件の REQUEST_CHANGES・再開での数え直しでラウンドを回しすぎる #2468

Open

Description

@takemi-ohama

何が起きたか(2026-11-03〜04、#2436)

利用者の指摘: 「なんか役に立たないことやってない?」

直すこと(案)

  1. 前のラウンドで見送った指摘と同じもの(パスと要旨が一致)が出直したら、修正の対象にせず数えない。2 回出たら out-of-scope の起票へ回し、以後のラウンドの入力から外す
  2. 両者の指摘が 0 件のラウンドは、判定の文字(REQUEST_CHANGES)にかかわらず収束として扱う
  3. 収束した後に足したコミットがテストと CI だけで確かめられる種類(テスト・取り込み・行数の調整)なら、ラウンドを足さない
  4. --resume での再開は、前の実行のラウンド数を上限に含める(数え直さない)
  5. MVV の Value 10 の上限を数値にする。 .ndf/review.json に PR ごとの上限(費用・秒・ラウンド)を置き、cross-review(drive.py)は再開をまたいだ合計で止める。超えた分の指摘は自動で起票する。上限の既定は過去の実測から決める(Value 3)
  6. 流し直しに Value 4 を当てる。 supervise.py queue --resume は、同じプランの同じステップが 2 回止まったら流さず、「手当て 2 回目」として課題を起こす案内を返す

受け入れ条件

  1. 見送った指摘が出直しても、ラウンドが増えず、起票された Issue の URL が報告に載る
  2. 指摘 0 件のラウンドで final が converged になり、プランが止まらない
  3. 再開をまたいでも、ラウンドの合計が --max-rounds を越えない
  4. .ndf/review.json の上限(費用・秒・ラウンド)のどれかに当たると cross-review が止まり、残りの指摘の Issue の URL が報告に載る
  5. 同じステップが 2 回止まったプランを --resume すると、流さずに終了コードと案内で止まる

原因の見立て(MVV 以前の問題)

MVV(.ndf/mvv.md)の Value 10・1・4 は、今回の無駄を止める基準をすでに持つ。止まらなかったのは、(1) 上限が数値として設定に無い、(2) MVV の判定(mvv-gate.py)が承認ゲートでしか動かず、「もう 1 回回すか」「流し直すか」では誰も照らさない、(3) 費用は report.md に記録されるが上限と比べる手順が無い、の 3 つによる。MVV の文は変えない

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions