Repository navigation
昇格の承認の後のマージ: 承認ゲート 2 で利用者が見た先端だけを本番チャネルへ入れる(#2104) - #2145
Conversation
pace: normal の昇格で、承認ゲート 2 の承認資料に載せた先端の SHA と promote-approved がマージする先端を照合する要求を、課題の本文から写した。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
pace: normal の昇格で、承認ゲート 2 の停止が昇格の承認資料へ承認したコミットの行を書き、 promote-approved が --approved-from でその SHA と先端を照合する設計を書いた。 照合の基準は承認資料の行だけにし(未決 1)、資料なしの --gate-approved user は 2 で止める(未決 2)。 用語集へ「昇格の承認資料」を足した。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | claude | REQUEST_CHANGES
ドメインモデルの集約「昇格の承認資料」の持ち主に、この変更で新たにこの資料へ書くようになる mvv-gate.py(承認の記録の行)が無く、44 行の「pace ごとに書き手は 1 つ」とも食い違う。持ち主の欄と 44 行を直す。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | claude-2 | APPROVE
ドメインモデルの節と用語集の差分に、指摘の基準に当たる修正点は無い(コンテキストマップの関係・集約の書き手・不変条件 I1〜I8・要求の E1〜E8 の引き継ぎ・用語集の「昇格の承認資料」の追加を確かめた)。
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🔧 /ndf:fix サマリ | round 1 | commit d571962対応件数: critical=0 / major=1 / minor=0(合計 1 件) |
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | claude | APPROVE
round 1 の指摘(昇格の承認資料の持ち主に mvv-gate.py の承認の記録の行が無い)は、40 行の持ち主の欄・44 行の行ごとの持ち主・I2 の追記で解消している。記述は plugins/ndf/scripts/mvv-gate.py:352 と lib/approved_commit.py の record_approval(承認したコミットの行の直後へ挿入)に一致する。新しい指摘は無い。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | claude-2 | APPROVE
前のラウンドの指摘(承認資料の持ち主に mvv-gate.py の承認の記録の行を足す)への対応を確かめた。集約の持ち主・44 行の行ごとの持ち主・I2 の追記は lib/approved_commit.py の record_approval(承認したコミットの行の直後へ挿入し、既存の記録は置き換え、承認したコミットの行は書き換えない)と一致し、指摘の基準に当たる修正点は無い。
🔧 /ndf:fix サマリ | commit (なし)対応件数: critical=0 / major=0 / minor=0(合計 0 件) |
#2104 の要求と受け入れ条件の写しと、設計文書を追加しました。設計の主旨は「昇格の承認の後は、承認資料の先端のときだけマージする」です(スプリント m2087)。あわせて用語集に 1 項目を足しています。
利用者向けの変化
移行の手順
変更の要点
issues/issue-2104-requirements.mdを追加(160 行): pace: normal の昇格で、承認ゲート 2 で見た先端と promote-approved がマージする先端を照合しない #2104 の要求と受け入れ条件の写しissues/issue-2104-design.mdを追加(281 行): pace: normal の昇格で、承認ゲート 2 で見た先端と promote-approved がマージする先端を照合しない #2104 の設計。昇格の承認の後は、承認資料の先端のときだけマージするdocs/glossary.md・docs/glossary/glossary.jsonに用語を 1 項目追加テスト
supervise.py の run のステップの結果です。
issues/issue-2104-requirements.mdを読めない([Errno 2] No such file o…)requirements-check は対象のファイルを読めずに止まっています。このファイルは本 PR に含まれているため、マージ前に確認が必要です。
モード: standard / 通した工程: 設計 → 要求と受け入れ条件 → ドキュメントレビュー
🤖 Generated with Claude Code
決めたこと
issues/issue-2104-design.mdmaterial_row/material_shaで共有する--gate-approved userのpromoteは--approved-fromを必須にし、無ければ 2 で止める(未決 2)normalとfast/autoで同じPROMOTE_MATERIALにし、promote-approvedのコマンドは両方で等しいままにするhead_movedのnextはpaceで分けず--from promote(fast/autoは--from verify)を並べて書く