Simplify the validate-plugin-contracts.mjs and validate-plugins.sh repository scripts

Chiusa Adatta ai principianti
  • Qualcosa non va?
#14,369 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Una pull request collegata è già stata integrata.

  • #14531 di @melodic-automation-lanes — integrata

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
72/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
javascript, shell
Ambito
tooling

Direzione di ricerca

Leggi scripts/validate-plugin-contracts.mjs e scripts/validate-plugins.sh, quindi confronta il loro comportamento attuale con i criteri di accettazione dell’issue. Esamina ogni file per riutilizzare le funzioni di supporto, semplificare e migliorare efficienza e astrazione; preserva output, codici di uscita, file, messaggi, flag e nomi esportati. Il lavoro è terminato quando i file elencati sono più brevi e chiari, il comportamento è invariato e il diff resta entro l’ambito indicato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Agent Brief

Type: Task
Summary: Simplify these 2 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:

  • scripts/validate-plugin-contracts.mjs
  • scripts/validate-plugins.sh

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.

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.
Lingua principale
Shell
Stelle
22
Fork
2
Merge medio
1h 5m
PR unite (30g)
969

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di melodic-software/claude-code-plugins

Tutte le issue di melodic-software/claude-code-plugins

Issue simili

Altre issue su Shell/Bash

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.