docs(source-control): define when a push needs a fresh AI review round
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 50/100
- issue の種類
- ドキュメント
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- shell
調査の方向性
Start with plugins/source-control/skills/pull-request and read the B2 rule, then inspect the PR monitor and the draft-PR guidance in AGENTS.md. Define the merge-commit diff test and distinguish clean merges or allowed-only file changes from code conflict resolutions; done when the skill states the rule and the monitor applies it.
索引モデルが issue の本文から書いたものです。
説明
The B2 rule in plugins/source-control/skills/pull-request treats an AI review lane as absent when it is green but posted nothing for the current head. On 2026-10-07 every merge-main push and every push that only added changelog fragments waited about 5 minutes for the lanes to re-run on code they had already reviewed. ci-status is the only required check (AGENTS.md, "Open a pull request as a draft").
An interim rule, "skip the lane wait on merge-only pushes", let #6516 merge after a merge push that had resolved a real conflict in lib/docs-cache.sh. A separate review found the resolution clean, but the auto-mode classifier flagged it as "Merge Without Review".
Proposal: state the rule in the skill.
- A push needs a fresh review round when it changes reviewed content. Any conflict resolution in code counts as changing reviewed content.
- A merge of main needs no fresh round when it merged cleanly, or when the only files it changed were CHANGELOG files, plugin.json versions or
.changesfragments. - Give the test: the diff of the merge commit against its main parent, limited to non-changelog paths, is empty.
Acceptance: the skill gives the test, and the PR monitor applies it.
- 主要言語
- Shell
- スター
- 22
- フォーク
- 2
- 平均マージ
- 8時間 41分
- マージ済み PR（30日）
- 892
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
melodic-software/claude-code-plugins のほかの issue
-
good first issue needs-human priority: low work-class: mechanical
難易度 2/5 1〜3時間 初心者へのやさしさ 92/100
melodic-software/claude-code-plugins#8023 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
needs-human work-class: mechanical
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
melodic-software/claude-code-plugins#8021 ·
メンテナーはふだん 1 日以内に返信
-
needs-human work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
melodic-software/claude-code-plugins#8018 ·
メンテナーはふだん 1 日以内に返信
-
docs(planning): point the interview surface context at the surface README instead of restating itオープンneeds-human work-class: mechanical
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
melodic-software/claude-code-plugins#8015 ·
メンテナーはふだん 1 日以内に返信
-
needs-human work-class: mechanical
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
melodic-software/claude-code-plugins#8004 ·
メンテナーはふだん 1 日以内に返信
melodic-software/claude-code-plugins の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
-
bug good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
scarif-labs/horus#61 ·
メンテナーはふだん 1 日以内に返信
-
documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
IntersectMBO/developer-experience#362 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
registrystack/registry-stack#2168 ·
メンテナーはふだん 1 日以内に返信