fix(pr-pipeline): mark check-signed-commits' needs-human comment so lane fixers hold on it
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 72/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- github, javascript, shell
調査の方向性
まず gate-source.mjs とその prHold ロジックを読み、次に check-signed-commits と clear-escalations.mjs を調べて、コメントとラベルがどのように扱われるかを追跡します。check のコメントにマーカーを追加し、両方の経路でマーク付きコメントを hold として数え、マーク付きコメントが hold を維持し、マークなしの bot コメントは維持しないことを確認するテストを追加します。
索引モデルが issue の本文から書いたものです。
説明
Problem
The lane fixers (fix-ci, fix-docs, resolve-conflicts, address-feedback) and clear-escalations.mjs treat a bot comment posted after the needs-human label as no hold, because only a person's comment or review holds (#9561, gate-source.mjs prHold). check-signed-commits adds an unmarked needs-human on top of a lane escalation, so the fixers keep running on a pull request that check flagged, and clear-escalations can lift the label.
Exploiting it needs write access to push unsigned commits, so this is low severity (P3 in #9561's security review).
Fix
Give check-signed-commits's comment a marker, and count that marker as a hold in prHold and in clear-escalations. Add tests for the marked comment holding and for an unmarked bot comment not holding.
Refs: #9561
- 主要言語
- Shell
- スター
- 22
- フォーク
- 2
- 平均マージ
- 3時間 42分
- マージ済み PR（30日）
- 938
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
melodic-software/claude-code-plugins のほかの issue
-
admission-deferred priority: medium status: ready work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
melodic-software/claude-code-plugins#9914 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: low status: ready work-class: mechanical
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
melodic-software/claude-code-plugins#9911 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: medium status: ready work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
melodic-software/claude-code-plugins#9901 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
admission-deferred needs-human needs-triage status: needs-decision
難易度 2/5 1〜2日 初心者へのやさしさ 78/100
melodic-software/claude-code-plugins#9865 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: medium status: needs-decision work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
melodic-software/claude-code-plugins#9855 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
melodic-software/claude-code-plugins の issue をすべて見る
似ている issue
-
resource-submission validation-passed
難易度 1/5 1時間未満 初心者へのやさしさ 67/100
hesreallyhim/awesome-claude-code#3264 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
area: install bug good first issue hacktoberfest
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 3 日以内に返信
-
Good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
armbian/build#11107 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
wimpysworld/deb-get#2069 ·
メンテナーはふだん 5 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
coco-research/coco#405 ·
メンテナーはふだん 1 日以内に返信