[takemi-ohama]

ndf プラグインで、スプリント m26-2 の変更をスプリントのブランチへ集める流れを整えました（#2228）。sprint-close.py は最終工程の記録を確かめてから Issue を閉じます。既定ブランチ宛の PR の本文では、Issue を閉じる語を「閉じる予定の参照」へ置き換えます。

利用者向けの変化

• ndf: 既定ブランチ宛の PR の本文にある closing keywords は、「閉じる予定の参照」に置き換わる
• ndf: sprint-close.py は、最終工程の記録を確かめてから Issue を閉じる
• ndf: まとめのプランは「振り返り → Issue を閉じる → 棚卸し」の順に進む
• ndf: Issue を閉じる時点の決まりは、progress-tracking Skill の「スプリントを閉じる」にまとまっている

移行の手順

• sprint-close.py の閉じる条件が増える（契約の変更）。呼ぶのは同じ版の plugins/ndf の中のプラン・Skill の本文だけなので、同じ版で呼ぶ側も直す。利用者の手での移行は要らない。この版より前に工程を通り、判定の材料を持たないまま開いている課題の扱いは未決 2（#2228の要求の移行性）

変更の要点

• 置き換えの判定を lib/closing.py の defer_for_base に寄せた（課題を閉じる時点を 1 つにし、最終の工程を通ってから閉じる #2228・既定ブランチ宛の実装 PR の本文の閉じる語で、マージの時点に課題が閉じ、仕上げの工程と受け入れ条件のチェックが閉じた課題に残る #2082）
• sprint-close.py の閉じる条件に、最終工程の記録の確認を足した（課題を閉じる時点を 1 つにし、最終の工程を通ってから閉じる #2228・sprint-close: 最終工程を通っていない課題を、リリース後テストの前に閉じないようにする #2119）
  • これは契約の変更にあたる。呼ぶ側は同じ版の plugins/ndf の中のプランと Skill の本文だけなので、同じ版でまとめて直した
  • この版より前に工程を通り、判定の材料を持たないまま開いている Issue の扱いは、未決 2 として残っている
• まとめのプランの順を「振り返り → Issue を閉じる → 棚卸し」に並べ替えた（課題を閉じる時点を 1 つにし、最終の工程を通ってから閉じる #2228・sprint-close: 最終工程を通っていない課題を、リリース後テストの前に閉じないようにする #2119）
• 正本と参照先を次のように整理した
  • Issue を閉じる時点の正本を、progress-tracking の「スプリントを閉じる」に置いた
  • merged / pr / release / release-verification / retrospective の各 Skill と、development-workflow の references は、その節を指す
• スクリプトを次のように分けた
  • sprint-close.py の最終工程の判定と Issue の集め方を sprint_close_lib へ移した
  • pr-steps.py の PR の読み書きを pr_steps_lib へ移した
• 文書を更新した
  • docs/glossary.md を用語集から作り直した
  • 確定仕様にあるまとめのプランの並び・閉じる条件・閉じる予定の参照を、課題を閉じる時点を 1 つにし、最終の工程を通ってから閉じる #2228 の振る舞いに合わせた
• テストを足した
  • test_supervise_pr_closing.py を新設した
  • test_supervise_pace.py と test_supervise_pace_verify.py を更新した
• 変更の規模は 47 ファイル、追加 1086 行・削除 372 行

テスト

supervise.py の run のステップの結果です。

ステップ	exit	結果
base-sync	0	ベースブランチ sprint/m26-2 を取り込んだ
format	0	整形 1 本が通った（変わったファイルは無い）
sync	0	同期とチェック 5 本が通った
lint-limited	0	範囲テスト 2 本が通った（8.7 秒）
test-limited	0	範囲テスト 1 本が通った（657.8 秒）
design-tests	1	stopped: 設計文書 2 件（issues/issue-2228-design-decisions.md ほか）で止まった
design-specs	0	既存の確定仕様と変更履歴 84 件と照合した（変えたパス 41・シンボル 130）

design-tests は exit 1（stopped）で止まっています。マージの前に確認が要ります。

設計と違う点

設計の箇所	違い	理由
決定 6・構成要素の scripts/sprint-close.py の行・構造の図の sprint_close	final_stage_gap は sprint-close.py でなく plugins/ndf/scripts/sprint_close_lib/final_stage.py にある。引数も (stage, body) でなく (stage, body, since)（since は決定 10 の比べる基準）	リファクタリングのコミット（bf6f06b8）で、sprint-close.py の行数を抑えるため sprint-close 専用のパッケージへ分けた。共有の lib には置いておらず、決定 6 の「lib に置かない」は守っている。since は決定 10 と入出力の契約の表が求める引数で、構造の図が古い
入出力の契約の「記録の PR の読み」の read_record	read_record は sprint-close.py でなく plugins/ndf/scripts/sprint_close_lib/collect.py にある	上と同じリファクタリングで分けた。返すもの（部分ごとの先頭の行とコメントの識別子）は契約のとおり
構成要素の scripts/pr-steps.py・scripts/supervise_lib/pr.py・処理の流れの「Pull Request の本文を送る」	2 つの呼び手は merge_closes_issues と defer_closing を別々に呼ばず、lib/closing.py の defer_for_base(root, base, text) を通して呼ぶ。pr-steps.py の PR の読み書きは plugins/ndf/scripts/pr_steps_lib/gh_pr.py へ分けた	2 つの呼び手で同じ判定と置き換えを繰り返さないよう、共通の関数へ寄せた（522fc96a・bf6f06b8）。判定（lib/repo.py の merge_closes_issues）と置き換え（defer_closing）の振る舞いは設計のとおり

モード: standard / 通した工程: 実装 → 完了判定 → Pull Request

🤖 Generated with Claude Code