Skip to content

リリース記録の PR を、まとめのプランを組む時点でなく実行の時点で解決する #2230

Closed
2 of 2 issues completed

Description

@takemi-ohama

依頼(原文)

この課題は、起票者(AI)が子の 2 件(#2044・#2121)をまとめて起こした親の課題である。利用者の指示の原文は課題にも記録にも見当たらない(2026-10-31 の利用者の方針 5 点〔--loop を残す・codex / agy / kiro への配布を cross 系に絞る・全工程で統一した JSON レポート・並列に強く・テストを削る〕には、リリース記録の PR を決める時点の項目が無い)。マイルストーン「26 自律実行の信頼性」の説明の主題 4 と、起票時の本文を写す。

リリース記録の PR を、プランを組む時点でなく実行の時点で決める(親 D)

(書き手: 起票者〔マイルストーンの説明に AI が書いた主題。利用者の発言の原文ではない〕、日付: 2026-10-31)

起票時の本文(題「リリース記録の PR を、まとめのプランを組む時点でなく実行の時点で解決する」):

何が困るか

まとめ(sprint-close)のプランが、リリース記録の PR をプランを組む時点で決める。merge・手動の経路やループの経路では、その時点に記録の PR がまだ無い。

子 経路
#2044 merge・手動の経路。close_waves が最終の検査・リリース・まとめを 1 つのキューに組み、record_pr_ref が unknown を返す
#2121 開発ループの承認ゲート 2 の後のまとめが、配布の記録の PR を読まない

修正レイヤー

supervise_lib/sprint_routes.record_pr_ref と close_plan。 記録の PR を、プランを組む時点ではなく実行の時点で解決する(supervise_lib/queue.py の {queue_pr:<名>} の解決を広げる)。

受け入れ条件

  • どの経路でも、まとめのステップが実行の時点で記録の PR を解決する
  • 解決できないときは unknown で進めず、止まる(親 A の規則に従う)

(書き手: 起票者、日付: 2026-10-31。利用者の指示の原文として扱わない)

子の課題:

この要求は、人へ問わずに進める起動(AI の起動指示、2026-11-01: 「人へ問わずに進め、決められない点は前提か未決として課題の本文へ書く」)で書いた。決められない点は「前提」と「未決」に置く。

起票者の推定

利用者の指示の原文が無いため、題・本文の条件はどれも起票者の推定である。そのうち要求の形を決める条件を並べる。承認ゲート 1 で利用者が確かめるまで、利用者の決定として扱わない。

# 条件 推した出所 この要求での扱い
S1 記録の PR は、まとめのプランを組む時点でなく、まとめのステップを流す時点で決める 本文の「何が困るか」「受け入れ条件」1 点目 受け入れ条件 1・2・3
S2 解決できないときは unknown で進めず止まる(親 A の「欠け・未記録を ok で返さない」に従う) 本文の「受け入れ条件」2 点目、#2227 の「直し方の方針」 受け入れ条件 4・5
S3 直す場所は sprint_routes.record_pr_ref と close_plan で、解決は queue の {queue_pr:<名>} の解決を広げて行う 本文の「修正レイヤー」、#2121 の直し方の候補 B 前提 1。解決の手段(差し込みを広げるか、ほかの手段か)は未決 1 として設計で決める
S4 子の 2 件(#2044・#2121 の問題 1)は同じ層の直しで解ける 本文の子の表、#2044 の「現状と修正レイヤー」 受け入れ条件 1〜3
S5 #2121 の問題 2(manual ステージを含む経路で、まとめが手作業の前に走りうる)はこの課題の範囲に入らない 本文の子の表が #2121 を問題 1 だけで書いていること 含まない。ただし記録の PR が無い時点でまとめの close が走れば止まる(受け入れ条件 4)ため、手作業の前に課題が閉じる害はこの課題で防ぐ
S6 merge・手で届ける経路でも、記録の PR が解決できたならリリース後テストのステップをまとめに置く(close は --with-verification を渡すため、置かないと課題が閉じない) #2044 の「残ること」(本番まで出て受け入れ条件を満たしたスプリントでも毎回手で閉じる)と、_close_step が常に --with-verification を渡すこと 受け入れ条件 3

目的

  • 本番まで届けて受け入れ条件を満たしたスプリントで、リリースの経路(雛形・昇格・merge・手で届ける・検証の環境だけ・開発ループの周回)にかかわらず、まとめの close がリリース記録を読んで課題を閉じられるようにする(今は merge・手で届ける経路で毎回手で閉じ、開発ループでは検査のスプリント PR を記録として読む)
  • 記録の PR が見つからないとき、まとめが「閉じずに ok」で先へ進まず、止まって理由を返すようにする(親 A の規則)

前提

  • 前提 1: 直す層は、まとめのプランが記録の PR を決める箇所(supervise_lib/sprint_routes.record_pr_ref・supervise_lib/sprint.close_plan)と、ステージを流す時点の差し込みの解決(supervise_lib/queue_fill.py)、開発ループのまとめの組み立て(lib/sprint_loop.gate2_pipeline・lib/sprint_loop_gate.build_close)である。sprint-close.py の閉じる条件(--with-verification・最終工程の記録)は変えない
  • 前提 2: merge・手で届ける経路でも、release の手順はリリース記録(## 配布の記録)を Pull Request に残す(plugins/ndf/skills/release/SKILL.md の「出力物」: リリースの PR の本文、PR を作らない形ではスプリントの最後にマージした PR へコメント)。この課題はその記録を読む側を直し、記録を書く手順の形は変えない(記録の PR の番号を置く場所を足すかは未決 1)
  • 前提 3: 「実行の時点」は、まとめのステージ(close の queue)がそのプランを流し始める時点を指す。まとめのプランを組む時点(supervise.py new close・new sprint の組み立て、開発ループの承認ゲート 2 の時点)に記録の PR が無くてもよい
  • 前提 4: 届けない経路だけのスプリント(記録の PR が 0。sprint_routes.RECORD_NONE)は、記録を読まずに閉じる今の扱いを変えない
  • 前提 5: 呼ぶ側が --record-pr <番号> を明示して new close を打ったときは、その番号を使う今の扱いを変えない

対象範囲

含む:

含まない:

ドメインイベント

# イベント 引き金 失敗したとき 順序の前提
E1 まとめのプランを組んだ(記録の PR は未解決の参照として持つ) supervise.py new sprint / new close、開発ループの承認ゲート 2 の後の gate2_pipeline 組み立てが断る(今と同じ) —
E2 本番のリリースを終え、リリース記録を Pull Request に残した 雛形・昇格のプラン、merge の経路の導入の確認、手で行う /ndf:release 記録が残らない → E4 で解決できない E1 の後でも前でもよい(並びは前提 3)
E3 まとめのステージを流し始めた queue がまとめのステージへ進んだ(--then・単独の queue) — E2 の後であることを並びが保証しない経路がある(#2121 の問題 2。含まない)
E4 記録の PR を解決した E3 見つからない → E5 E2
E5 記録の PR が見つからず、まとめが止まった E4 の失敗 — E3
E6 リリース後テストのステップが記録の PR を受け取った E4 の成功 今と同じ(リリース後テストの失敗) E4
E7 まとめの close が記録の PR を --record-pr に受けて課題を閉じた E6 の後の close のステップ 閉じる条件を満たさない課題は開いたまま(今と同じ) E4・E6

用語

用語 意味
リリース記録 release が Pull Request へ残す ## 配布の記録 のブロック。段階・版・スプリントの Pull Request を持つ(用語集の語)
記録の置き場が分からない まとめの close が読むリリース記録の Pull Request を決められない状態。今は --record-pr unknown で渡し、課題を開いたまま ok で返す(用語集の語。この課題の後は、まとめのプランがこの値を組まない)

受け入れ条件

非機能の条件

大項目 条件
運用・保守性 記録の PR の解決は 1 か所で行い、経路ごと(通常の起動・開発ループ)に別の解決の関数を持たない(Value 6)
移行性 実行中のスプリントで、この版より前に組んだまとめのプラン(--record-pr unknown を持つもの)は、今の振る舞い(閉じずに ok)のまま流れてよい。組み直しは求めない

影響

対象 影響
公開インタフェース supervise.py new sprint / new close が書くまとめのプランの close の引数が、merge・手で届ける経路で変わる。sprint-close.py --record-pr unknown の受け付けを残すかは未決 2。queue の差し込みの規則(supervise_lib/plan.py の説明)が広がる場合は、その説明も変わる
データ 無し(リリース記録の形は変えない)
既存の振る舞い merge・手で届ける経路のまとめが、記録が見つからないときに「閉じずに ok」から「止まる」へ変わる。開発ループの承認ゲート 2 の後のまとめが、本番のリリース記録を読む

検証手段

項目 手段
テスト uv run --frozen --project . --all-extras pytest plugins/ndf/scripts/tests -q -n 4(プランの組み立て・queue の差し込みの解決・開発ループの gate2_pipeline の単体テスト)
静的解析・型検査 リポジトリの CI と同じ検査(ruff。設定は pyproject.toml)
手動確認 リリース後テストで、merge の経路か手で届ける経路のスプリントを 1 本流し、まとめの close がリリース記録の PR を受けて課題を閉じること、記録を置かずに流したときにまとめが止まることを確かめる

前提とする取り決め

項目 参照先 / 決めたこと
プロジェクト構造 AGENTS.md の「マーケットプレイスの構造」。変更は plugins/ndf/scripts/ の中に置く。既定で動くコードから plugins/ndf/scripts/experimental/ を参照しない
コーディング規約 AGENTS.md の「最小限のコード実装」。結果の形は plugins/ndf/scripts/lib/README.md の「手順のスクリプトの結果」
テスト戦略 振る舞い(組んだプランの引数・差し込みの解決・止まる条件)を単体テストで押さえる。.md の文言を照合するテストは書かない

境界

区分 内容
常に行う 既存テストの実行、雛形・昇格の経路の出力が変わらないことの確認
確認してから行う queue の差し込みの規則(公開の契約)の拡張、sprint-close.py の引数の変更(承認ゲート 1 の設計で人が見る)
行わない リリース記録の形の変更、課題を閉じる条件の変更、#2121 の問題 2 の並びの直し

MVV との突き合わせ

反する疑いは無い。Value 6(根本原因の場所で直す: 子の 2 件を記録の PR を決める層で直す)、Value 5(どのプロジェクトでも使える形: 経路はリリースの宣言から決まり、ai-plugins の形を埋め込まない)、Value 2(利用者の手を止めない: 本番まで届けたスプリントの課題を手で閉じなくてよくなる)に沿う。記録が見つからないときに止まるのは、共通原則の優先順位(承認資料と記録の正しさを効率より上に置く)と親 A に沿う。

未決

  1. 記録の PR を実行の時点で解決する手段。 候補は (a) queue の {queue_pr:<名>} の探し場所を広げる(開発ループのまとめは本番のプランと別の queue・別のディレクトリにある)、(b) 手で行うリリースのステージ・merge の経路の導入の確認に、リリース記録を置いた PR の番号を記録するステップを足し、まとめがそれを読む(new close: merge・手動の経路のスプリントで、リリース記録の PR をまとめの close へ渡す手段が無く、課題が --record-pr unknown で開いたまま残る #2044 の直し方の案)、(c) まとめの close の前に、スプリントの PR を指すリリース記録を GitHub から探すステップを置く。設計(design)で決める
  2. sprint-close.py --record-pr unknown の受け付けを残すか。 まとめのプランは組まなくなる(受け入れ条件 5)。手で打つ入口として残すか、親 A に合わせて止まる(ok で返さない)値にするか、受け付けをやめるかを設計で決める

進行

モード: — / 進め方: auto

  • 要求と受け入れ条件 — 2026-11-01 01:19
  • 作業場所の用意 — 2026-11-01 02:53
  • 設計 — 2026-11-01 01:21
  • 素材の収集と出典の確定
  • ドキュメント再構成
  • ドキュメントレビュー
  • 計画 — 2026-11-01 02:53
  • 実装 — 2026-11-01 02:54
  • 構造改善 — 2026-11-01 05:23
  • 実装レビュー — 2026-11-01 06:24
  • 完了判定 — 2026-11-01 03:16
  • Pull Request — 2026-11-01 03:29
  • 確定仕様化 — 2026-11-01 07:43
  • 後片付け — 2026-11-01 07:42
  • 配布 — 2026-11-01 07:24
  • 体裁レビュー
  • リリース後テスト — 2026-11-01 07:59
  • 振り返り — 2026-11-01 08:07
  • 棚卸し — 2026-11-01 08:21

振り返り: #2352 (comment)

Activity

  1. added
    priority: high実害・安全機構の欠落など、優先して対応する
    priority: medium保守性・設計一貫性など、計画的に対応する
    and removed
    priority: high実害・安全機構の欠落など、優先して対応する
    on Oct 31, 2026
  2. takemi-ohama commented on Nov 7, 2026

    @takemi-ohama
    ContributorAuthor

    受け入れ条件 1〜8 は v10.27.6 のリリース後テスト #2351 で合格している。保留の 3 行のうち手動確認は、この NDF を使う private のリポジトリの開発ループの記録で確かめた。まとめのステップが流す時点で記録の PR を受け取った周回は課題を閉じ、リリース記録の無い PR を受け取った周回は止まった。残る 2 行(同じ課題が 2 つのスプリントで閉じられる頻度、手書きの記録)は受け入れ条件でない観測の項目で、止まり方はテストで押さえてある。実例が出て問題が起きたら別の課題にする。

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 本体priority: medium保守性・設計一貫性など、計画的に対応する

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions