Repository navigation
まとめの close: どのリリースの経路でも、まとめを流す時点でリリース記録の PR を見つけて課題を閉じる（#2230） - #2332
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
takemi-ohama
left a comment
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | codex | REQUEST_CHANGES
記録の選択条件に、まとめの対象スプリントとの対応を追加してください。
takemi-ohama
left a comment
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | kiro | APPROVE
ドメインモデルの節と用語集の差分を確認。コンテキストマップ（開発の工程がリリースの順応者）・集約の持ち主・不変条件 I1〜I8・ドメインイベント E1〜E7（受け手あり・要求の番号を引き継ぐ）は内部で整合し、用語集の 2 行（追加「記録の PR」・意味変更「記録の置き場が分からない」）は glossary.md と glossary.json で一致し、設計の用語の表とも対応する。実装する人が違うものを作る食い違い・利用者の経路が壊れる点は、この範囲に見当たらない。修正アクションの要る指摘なし。
持ち越した課題を 2 つのスプリントで閉じる PR を出すと、最新の記録を採る規則では別のスプリントの記録を選び、 sprint-close.py が対象外の課題まで閉じる。まとめの課題すべてを閉じる PR を含む記録に限り、 スプリントの並びが 2 通り以上残れば見つからない扱いで止める。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🔧 /ndf:fix サマリ | round 1 | commit 245caf2対応件数: critical=0 / major=1 / minor=0（合計 1 件） |
takemi-ohama
left a comment
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | claude | REQUEST_CHANGES
I5 を「まとめの課題の 1 件ごとに閉じる PR を含む」へ強めたことで、閉じる PR を持たない課題が 1 件でもあるまとめは、記録が 1 つしか無くても毎回止まる。条件を掛ける課題の範囲を決め直してください。
takemi-ohama
left a comment
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | kiro | REQUEST_CHANGES
round 1 の指摘（対象スプリントの対応を記録の選択条件に足す）に答える差分を確認。I5・find_record の手順（新しい 6）・決定 2・テスト設計の追加は内部で整合し、GitHub の読み回数も増やさない（手順 6 は 2 で集めた PR で判定する）。ただし新しく足した「2 つ以上のスプリントにある」の失敗理由が、RecordHit の構造が持たないデータを求めるため、実装する人がこの理由文を組めない。構造の節へ候補の記録と スプリント: の集合を持つ項目を足す必要がある。
- 閉じる PR を持たない課題が混ざったまとめで毎回止まらないよう、I5・手順 2/5・決定 2 を直す - 手順 2 で課題ごとの閉じる PR の対応を保つことを明記する - RecordHit に ambiguous（記録の PR と スプリント: の集合の組）を足す - テスト設計に 2 行を足す Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🔧 /ndf:fix サマリ | commit 72e5452対応件数: critical=0 / major=0 / minor=0（合計 0 件） |
#2230 の設計文書と、要求・受け入れ条件の写しを足します（スプリント m26-3）。設計の中身は、まとめを流す時点でリリース記録の PR を GitHub の参照から見つけるやり方です。あわせて用語集を更新しています。
利用者向けの変化
移行の手順
変更の要点
issues/issue-2230-requirements.mdを足しました。リリース記録の PR を、まとめのプランを組む時点でなく実行の時点で解決する #2230 の要求と受け入れ条件の写しです（162 行）。issues/issue-2230-design.mdを足しました。リリース記録の PR を、まとめのプランを組む時点でなく実行の時点で解決する #2230 の設計で、まとめを流す時点にリリース記録の PR を GitHub の参照から見つける方式を書いています（384 行）。docs/glossary.mdとdocs/glossary/glossary.jsonを更新しました。設計で使う語を用語集に揃えています。テスト
supervise.py の run のステップの結果です。
issues/issue-2230-requirements.mdを読めず、[Errno 2] No such fileで止まりましたrequirements-check は通っていません。対象の
issues/issue-2230-requirements.mdはこの PR の変更に含まれています。そのため、ファイルがある状態で再実行して結果を確かめる必要があります。モード: standard / 通した工程: 設計 → 要求と受け入れ条件 → ドキュメントレビュー
🤖 Generated with Claude Code
決めたこと
issues/issue-2230-design.mdスプリント:の PR がまとめの課題を閉じる記録だけを採る{queue_record}として、まとめのステージを流し始める時点に 1 回解決する{queue_pr:…}を残す{queue_record}を組むsprint-close.py --record-pr unknownの受け付けと振る舞いを変えない