fix(ci): check-changed-skills.sh fails a stale branch on skills only main changed
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 72/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- bash, git
- 領域
- build-system
調査の方向性
バグは scripts/check-changed-skills.sh にあります。変更ファイルの集合は changed_files::into changed_paths "$BASE"（123行目付近）から得られ、スキルごとのチェックは git diff --name-only "$BASE" -- "$skill_dir/SKILL.md"（175行目付近）を実行します。再現手順は、main より遅れているブランチで origin/main を指定してスクリプトを実行し、その後、両方の呼び出しの前に git merge-base "$BASE" HEAD で BASE を解決することです。ベースより遅れているブランチで、main だけがスキルを変更したと報告されなくなり、ベースより遅れているブランチを対象とする新しいテストが追加されたら完了です。
索引モデルが issue の本文から書いたものです。
説明
Problem
Run locally as scripts/check-changed-skills.sh origin/main on a branch behind main, the script also checks skills that only main changed and fails on them.
Seen on 2026-10-10 on feat/agent-tool-spawn-effort (#6913), 44 commits behind main. It failed on plugins/harness-config/skills/audit/scripts/fix-plugin-drift.test.sh, which the branch never touched. Against the merge base (e971f640b3) it passed: "6 skill(s) checked, 0 failed."
Cause: the changed set and the per-skill diff are taken against the base ref as given. The changed set comes from changed_files::into changed_paths "$BASE" and the per-skill check from git diff --name-only "$BASE" -- "$skill_dir/SKILL.md" (scripts/check-changed-skills.sh:123, :175). That is a two-dot comparison against the working tree, so it includes main's own changes since the branch point.
Proposed next step
Resolve $BASE to git merge-base "$BASE" HEAD before computing the changed set, so the script checks only what the branch changed. Keep CHECK_SKILL_BASE_REF semantics unchanged unless they have the same problem. Add a test with a branch behind its base.
- 主要言語
- Shell
- スター
- 22
- フォーク
- 2
- 平均マージ
- 6時間 10分
- マージ済み PR（30日）
- 937
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
melodic-software/claude-code-plugins のほかの issue
-
docs(evals): note CLAUDE_CODE_PRINT_BG_WAIT_CEILING_MS for headless eval runs that use workflowsオープンgood first issue needs-triage priority: low
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
melodic-software/claude-code-plugins#7022 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
good first issue needs-triage priority: low
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
melodic-software/claude-code-plugins#7019 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
good first issue needs-triage priority: low
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
melodic-software/claude-code-plugins#6982 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
fix(knowledge): frame the video title and URL in the video-digest resume prompt as untrusted data対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンneeds-human needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
melodic-software/claude-code-plugins#6980 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
needs-human needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
melodic-software/claude-code-plugins#6897 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
melodic-software/claude-code-plugins の issue をすべて見る
似ている issue
-
type: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
blamechris/chroxy#8579 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
alunduil/alunduil-chezmoi#895 ·
メンテナーはふだん 1 日以内に返信
-
Sub-second duration displays incorrectly for commands taking < 100ms (e.g. 50ms renders as 0.50s)オープン
難易度 1/5 1時間未満 初心者へのやさしさ 91/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
package-update
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
oSoWoSo/vOid_Community_repOsitory#333 ·
メンテナーはふだん 1 日以内に返信