Skip to content

merged-steps.py promote の --help が、照合が食い違ったときに承認ゲート 2 へ戻ることを書いていない(#1529 の条件 9) #2142

Closed

Description

@takemi-ohama

何を見つけたか

ndf 10.22.0 のリリース後テストで、#1529 の受け入れ条件 9 が満たされていなかった。条件 9 は「merged-steps.py promote の --help と pace.md の昇格のプランの説明が、照合と、食い違ったときに承認ゲート 2 へ戻ることを書いている」である。

  • pace.md(skills/development-workflow/references/pace.md の 228〜236 行)は、照合と承認ゲート 2 へ戻ることの両方を書いている
  • python3 <ndf 10.22.0>/scripts/merged-steps.py promote --help の --expect-head-from の説明は「確かめたコミットが PR の先端のときだけマージする」だけである。「承認ゲート」の語は出てこない
  • モジュールの docstring は「違えば head_moved で 1、読めなければ 2」までを書く。承認ゲート 2 へ戻す(handoff が by: mvv の記録を外して 10 を返す)ことは書いていない
  • テスト test_promote_help_shows_the_verified_commit_argument は引数名が出ることしか見ていない

動きは条件 1〜8 のとおり正しい(リリース後テストで合格)。欠けているのは help の文言だけである。

あわせて、40 桁でない資料で止まるときの summary が「承認したコミットが 40 桁の 16 進でない」となる。読んだのは導入の確認の資料なので、語が合わない(parse_sha の文言をそのまま使っている)。

どこで見つけたか

PR #2138(ndf v10.22.0 のリリース)のリリース後テストで、#1529 の条件 9 を確かめたとき。plugins/ndf/scripts/merged-steps.py の promote のサブコマンドの引数の定義と、plugins/ndf/scripts/lib/approved_commit.py の止まるときの文言。

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

#1529 は 10.22.0 でリリース済みで、リリース後テストの不合格は新しい変更として扱う(release-verification の「不合格を即時修正しない」)。根拠: MVV 版 3(判断の範囲は目の前の変更に限る)。

直さないと何が起きるか

promote が 1 / 2 で止まったとき、help だけを読んだ人には、次に何が起きるかが分からない(承認ゲート 2 へ戻り、--from verify から流し直す)。照合を持たない promote-approved を選んで、確かめていない先端をマージする誤りを防ぐ手がかりが pace.md にしか無い。#1529 の受け入れ条件 9 はこの課題が引き継ぐ(#1529 の条件 1〜8 は 10.22.0 のリリース後テストで合格し、#1529 は閉じた)。

起票者の推定

  • help に足す文は「食い違えばプランの handoff が承認ゲート 2 へ戻す」程度でよい(推した出所: リリース後テストの worker の報告)
  • summary の語の不一致もこの課題で直す(推した出所: 同上)

由来

PR #2138

受け入れ条件

  1. python3 plugins/ndf/scripts/merged-steps.py promote --help の出力が、照合(確かめたコミットが PR の先端と一致するときだけマージすること)と、食い違ったとき・読めないときにプランの handoff が承認ゲート 2 へ戻すこと(--from verify から流し直す)の両方を書いている
  2. merged-steps.py の promote のモジュールの docstring が、終了コード 1 / 2 の後に承認ゲート 2 へ戻ることを書いている
  3. 導入の確認の資料の SHA が 40 桁の 16 進でないときに止まる summary が、読んだもの(導入の確認の資料)を指す語になる。parse_sha を使う他の呼び出し元(承認したコミットの照合)の summary は変わらない
  4. 照合の振る舞い(昇格のプランが、導入の確認をしたコミットと違う先頭を本番チャネルへマージしうる #1529 の条件 1〜8)は変わらない。既存のテストが通る

進行

モード: —

  • 要求と受け入れ条件
  • 作業場所の用意 — 2026-10-25 11:42
  • 設計
  • 素材の収集と出典の確定
  • ドキュメント再構成
  • ドキュメントレビュー
  • 計画 — 2026-10-25 11:42
  • 実装 — 2026-10-25 11:42
  • 構造改善 — 2026-10-25 13:38
  • 実装レビュー — 2026-10-29 13:37
  • 完了判定 — 2026-10-25 13:14
  • Pull Request — 2026-10-25 13:24
  • 確定仕様化
  • 後片付け — 2026-10-29 22:32
  • 配布 — 2026-10-29 13:56
  • 体裁レビュー
  • リリース後テスト — 2026-10-29 22:32
  • 振り返り — 2026-10-29 22:52
  • 棚卸し — 2026-10-29 22:58

振り返り: #2160 (comment)

Activity

  1. added a commit that references this issue on Oct 25, 2026
  2. takemi-ohama commented on Oct 29, 2026

    @takemi-ohama
    ContributorAuthor

    スプリントの終わりの工程を通りました

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 working

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions