Simplify the setup-apply.mjs verification plugin code

オープン 初心者向け
  • 何か問題がありますか？
#15,008 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

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

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

評価

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

調査の方向性

plugins/verification/skills/setup/scripts/setup-apply.mjsから始め、列挙されたインターフェースと動作を維持しながら、ヘルパー、抽象化、効率性を見直してください。このスクリプトに関連するチェックを実行し、出力、終了コード、書き込まれたファイル、表示されたメッセージを比較してください。これらの結果を変えずにファイルをよりシンプルで明確にできれば完了です。

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

説明

status: ready

Agent Brief

Type: Task
Summary: Simplify these 1 files without changing their behavior (part of the /simplify epic).

Current behavior:
The files below work as they do on main. Their code changes; what they do does not.

Desired behavior:
Review the files through /simplify's four checks (reuse of existing helpers, simplification, efficiency, right level of abstraction), using subagents if useful, and apply the fixes. The files end up shorter and clearer, and whatever runs them gets the same results.

Files:

  • plugins/verification/skills/setup/scripts/setup-apply.mjs

Key interfaces:

  • Outputs, exit codes, written files and printed messages of these files: unchanged.
  • Command-line flags, exported names and anything another file calls: unchanged.

Acceptance criteria:

  • Behavior is unchanged: same outputs, exit codes, written files, printed text, command-line flags and exported names; no listed file renamed, moved or deleted.
  • The diff edits only the listed files, the lines of a test that assert on them when the change breaks that assertion, and changelog fragments.
  • The pull request stays within 400 changed lines and 15 hand-edited files (generated shared copies and changelog fragments not counted); anything beyond goes under ## Follow-ups in the pull request body.
  • ci-status is green.
  • Each touched plugin (verification) has a fragment from scripts/new-changelog-fragment.sh <plugin> patch whose body is ### Changed and the line "Internal simplification, no behavior change."

Out of scope:

  • Moving code between files or deduping across them: the sub-epic's restructuring pass does that after these chunks merge.
  • Tests beyond the asserted lines a change breaks, eval cases, fixtures, and skill and agent text.
  • Files listed in other chunks.
  • These eval and fixture files, which read the listed files' names, flags or printed text; the behavior rule keeps them accurate:
    • plugins/verification/evals/setup-apply-placement-not-configurable/case.yaml
    • plugins/verification/evals/setup-apply-refuses-thorough/case.yaml
    • plugins/verification/evals/setup-apply-refuses-zero-workers/case.yaml
    • plugins/verification/evals/setup-apply-strict-shows-diff/case.yaml
    • plugins/verification/evals/setup-apply-writes-live-workers/case.yaml
    • plugins/verification/evals/setup-check-memory-root/case.yaml
主要言語
Shell
スター
22
フォーク
2
平均マージ
1時間 5分
マージ済み PR（30日）
969

環境構築

はじめの一歩

  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 を短くまとめたダイジェスト。