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: vendored skills' drift-check scripts disagree on what their exit codes mean

Đang mở
  • Có gì không ổn?
#6,493 0 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

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
70/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
github-actions, shell
Lĩnh vực
ci-cd, tooling

Hướng nghiên cứu

Start with the six vendored skills' scripts/update.sh (playwright, firecrawl, playbooks/boris, playbooks/skill-authoring, dometrain sync, context7 lookup) and map each script's current exit paths onto the proposed contract: 0 no drift, 1 upstream drift, 2 unverifiable. Then look at firecrawl's --check baseline-reading step to find why a missing CLI counts as drift and why the report shows last sync: recorded). Done means every script returns the shared codes with a test pinning each code, and the maintenance-check-upstream-drift workflow matrix (issue #6477) uses one drift rule for all rows.

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

Mô tả

needs-triage

Problem

The six vendored skills' scripts/update.sh drift checks each define their own exit codes, so nothing can read them the same way:

Script Exit 1 Exit 2 Drift
playwright, firecrawl, playbooks boris, playbooks skill-authoring drift prerequisite or network failure exit 1
dometrain sync prerequisite missing or unverifiable drift exit 2
context7 lookup prerequisite missing unverifiable exit 0, reported only on stdout

Two more quirks found on 2026-10-07:

  • firecrawl's --check counts a CLI that is not installed as drift, so it exits 1 on any machine without the CLI, whether upstream moved or not.
  • firecrawl's report printed last sync: recorded) and recorded SHA256: recorded), which looks like its baseline-reading step picks up the wrong field.

The weekly maintenance-check-upstream-drift workflow (melodic-software/claude-code-plugins#6477) works around all of this with a per-row drift rule, which is fragile.

Proposed

One exit-code contract for every vendored drift check: 0 no drift, 1 upstream drift, 2 unverifiable (prerequisite or network). A missing local CLI is reported but is not upstream drift. Fix the firecrawl baseline read. Then simplify the workflow's matrix to the shared contract.

Acceptance

  • All six scripts follow the shared contract, each with a test that pins it.
  • The workflow's matrix uses one drift rule for every row.
Ngôn ngữ chính
Shell
Star
22
Fork
2
Merge trung bình
5 giờ 4 phút
Pull request đã merge (30 ngày)
825

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.