Skip to content

cross-review: 投稿の後に監視の上限へ達した担当の指摘が記録されず、起動し直した担当が同じ指摘を重ねて投稿する #583

Description

@takemi-ohama

何を見つけたか

担当が投稿を終えた後、結果ファイルを書く前に監視の上限に達すると、投稿済みの指摘が状態ファイルに記録されない。

引き金は v10.13.0(#598 / #537)で緩んだが、無くなってはいない。 監視の上限はレビューの工程で
420 秒から 1200 秒になり(scripts/lib/limits.py)、下の再現例のように 420 秒で当たることは
なくなった。上限に達したときに記録が残らないという仕組みは変わっていない。 判定は「結果なし」として同じラウンドで担当を起動し直し、起動し直した担当が同じ論点を重ねて投稿する。

PR #578 の round 1(担当: agy / kiro)で起きた。

[agy] ⏰ agy TIMEOUT (420s) — hard timeout 420s reached (pid 30748)
{"agent": "agy", "status": "TIMEOUT", ... "progress_tail": "post: submit review", "result_exists": false}
→ 結果を残さなかったレビュアーがいる: agy。同じラウンドで 1 度だけ起動し直す。
[agy] ✅ agy OK (405s) — process exited; sentinel=False; result_exists=True
✅ agy: intent=REQUEST_CHANGES posted_as=COMMENT comments=1
✅ 新しい指摘が出なくなった。収束。
NEW_FINDINGS=0
実行 PR に投稿されたもの 状態ファイルの review_findings
1 回目(上限で停止) インライン 4 件(major 1 / minor 3)とレビュー本文 0 件
2 回目(起動し直し) インライン 1 件(1 回目の major と同じ論点)とレビュー本文 1 件(insufficient_evidence)

1 回目の 3 件(minor)は判定の入力に一度も入らない。 分類も反証も通らないまま、未解決のスレッドとして PR に残る。最終スイープが拾うため失われはしないが、ループの中の修正の工程は通らない。同じ論点が 2 つのスレッドに分かれ、修正の担当が両方へ返信する。

どこで見つけたか

なぜこの変更の範囲外なのか

#545 の受け入れ条件は、設計 Pull Request の本文と設計文書の決定の見出しを揃えることに閉じており、cross-review の手順は変えない(#545 の設計の決定 7)。

直さないと何が起きるか

証拠集約の印が付いたラウンドで、反証の無い単独の major が数えられず収束することの再現は #624 にある。

由来

PR #578(issue #545)

関連

次の課題は、同じ出口(結果なし → 同じラウンドで起動し直す → final=error)を持つ。
原因と直す箇所はそれぞれ別で、1 件を直しても他は残る。

未着手のもの:

v10.13.0 で解決したもの:

進行

モード: standard / 作業ツリー: .worktrees/feat/issue-730-writes-to-conductor

  • 要求と受け入れ条件 — 2026-09-15 13:45
  • 作業場所の用意 — 2026-09-15 12:45
  • 設計 — 2026-09-15 13:45
  • 素材の収集と出典の確定
  • ドキュメント再構成 — 2026-09-15 13:46
  • ドキュメントレビュー — 2026-09-15 13:46
  • 計画 — 2026-09-22 03:58
  • 実装 — 2026-09-22 04:07
  • 構造改善 — 2026-09-22 09:57
  • 実装レビュー — 2026-09-22 11:58
  • 完了判定 — 2026-09-22 13:38
  • Pull Request — 2026-09-22 13:42
  • 確定仕様化 — 2026-09-22 13:55
  • 後片付け — 2026-09-22 03:54
  • 配布 — 2026-09-22 14:17
  • 体裁レビュー
  • リリース後テスト — 2026-09-22 15:06
  • 振り返り — 2026-09-22 15:38

閉じた理由

PR #801 で直り、ndf 10.16.0(2026-09-22、main / タグ ndf--v10.16.0、PR #810)で配布した。リリース後テスト(#810 (comment) )でこの課題の受け入れ条件はすべて合格した。

振り返り: #810 (comment)

Activity

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

    area: ndf-skillNDF の Skill 本体bugSomething isn't workingpriority: high実害・安全機構の欠落など、優先して対応する

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions