Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

fix: vendored skills' drift-check scripts disagree on what their exit codes mean

オープン
#6,493 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
70/100
issue の種類
リファクタリング
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
github-actions, shell
領域
ci-cd, tooling

調査の方向性

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.

索引モデルが issue の本文から書いたものです。

説明

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.
主要言語
Shell
スター
22
フォーク
2
平均マージ
5時間 11分
マージ済み PR(30日)
838

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

melodic-software/claude-code-plugins のほかの issue

melodic-software/claude-code-plugins の issue をすべて見る

似ている issue

Shell/Bash の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。