Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

fix(ci): check-changed-skills.sh fails a stale branch on skills only main changed

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#7,014 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@kyle-sexton đang làm issue này rồi.

Từ ngày 11/10/2026.

  • #7083 của @kyle-sexton — đang mở

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
72/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
bash, git
Lĩnh vực
build-system

Hướng nghiên cứu

Lỗi nằm trong scripts/check-changed-skills.sh: tập hợp tệp đã thay đổi được lấy từ changed_files::into changed_paths "$BASE" (khoảng dòng 123), còn kiểm tra theo từng skill chạy git diff --name-only "$BASE" -- "$skill_dir/SKILL.md" (khoảng dòng 175). Để tái hiện, hãy chạy script với origin/main trên một nhánh đang chậm hơn main, sau đó phân giải BASE bằng git merge-base "$BASE" HEAD trước cả hai lệnh gọi. Hoàn thành khi một nhánh chậm hơn base của nó không còn báo rằng chỉ main đã thay đổi skill, và một bài kiểm thử mới bao phủ trường hợp nhánh chậm hơn base của nó.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

good first issue needs-triage priority: medium

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.

Ngôn ngữ chính
Shell
Star
22
Fork
2
Merge trung bình
6 giờ 10 phút
Pull request đã merge (30 ngày)
937

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của melodic-software/claude-code-plugins

Tất cả issue của melodic-software/claude-code-plugins

Issue tương tự

Thêm issue về Shell/Bash

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.