Merge queue squash commit fails required_signatures when a wrapped PR body line starts with 'committer '
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 55/100
調査の方向性
Start by inspecting the /source-control:pull-request body tooling and the pr-contract CI check mentioned in the issue. Reproduce the wrapping at 72 columns and check whether any wrapped body line starts with a Git commit header keyword. Done means the queue no longer creates an invalid signed squash commit for this case, with a check that catches the offending line before merge.
索引モデルが issue の本文から書いたものです。
説明
The merge queue removed #7225 twice with branch_protection_failure, even though its CI passed in the queue.
What happened
- The queue's squash commit c2b54b0, committed by GitHub, shows
verification.reason: invalid. Therequired_signaturesrule (org rulesetsigning, 17989003) then refuses it. - GitHub builds the squash message from the PR body and wraps it at about 72 columns. One wrapped line began
committer identity set in the step env. - In the signed payload, the
gpgsigblock sits right after that body line, not after the realcommitterheader. So the signing placed the signature after the first line startingcommitter. - The previous squash commit on main, 4a453b0, is
valid. - Rewording the body so no wrapped line starts with
committerfixed it.
Who acts and what they do
PR authors and the PR body tooling (/source-control:pull-request) should keep any line of the wrapped body from starting with a git header keyword, committer above all. One option is a ci-status pr-contract check that wraps the body at 72 columns and fails on ^(tree|parent|author|committer|gpgsig|encoding|mergetag) . That check would catch it before the queue does.
Evidence: gh api repos/melodic-software/claude-code-plugins/git/commits/c2b54b03e9c0bcacbf7367a2fa923d3ee308bcf9 --jq .verification
🤖 Generated with Claude Code
- 主要言語
- Shell
- スター
- 22
- フォーク
- 2
- 平均マージ
- 8時間 34分
- マージ済み PR（30日）
- 946
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
melodic-software/claude-code-plugins のほかの issue
-
needs-human needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
melodic-software/claude-code-plugins#7636 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
agent-ready needs-human
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
melodic-software/claude-code-plugins#7598 · コメント 4 件 ·
メンテナーはふだん 1 日以内に返信
-
good first issue priority: low
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
melodic-software/claude-code-plugins#7530 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
needs-human priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
melodic-software/claude-code-plugins#7455 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
melodic-software/claude-code-plugins#7351 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
melodic-software/claude-code-plugins の issue をすべて見る
似ている issue
-
Add Photocraftオープンenhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
-
needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
gke-labs/kube-agents#3394 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信