Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

fix(ci): fail post.sh when the model stages a path outside the conflict set

Aperta
  • Qualcosa non va?
#7,297 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
74/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
git, github-actions, shell
Ambito
ci-cd, tooling

Direzione di ricerca

Read .github/scripts/merge-base/post.sh and its existing staged-path checks, then inspect .github/scripts/merge-base/post.test.sh to follow the test patterns. Compare the final index against the stage-0 snapshot and the snapshot’s unmerged paths; add a test for an unexpected staged path. Done when that case fails with a distinct reason and the existing checks still pass.

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

Descrizione

needs-human needs-triage

Problem

.github/scripts/merge-base/post.sh escalates on an unmerged entry, a
conflict marker in any entry staged since the snapshot, and any change
under .github/workflows/. It does not limit which paths the model may
stage. If resolve-conflicts in CI mode ever runs git add -A, git add . or a directory (a model mistake, or steered by text in the conflicted
files), any other modified or untracked file in the checkout is staged,
passes the marker scan, and reaches git write-tree and the commit.

The skill's allowed-tools cannot close this: Bash(git add:*) is a
prefix rule, and the rule syntax has no way to exclude -A, --all,
. or :/. The skill forbids bulk staging in prose, and the
no-wildcard-staging eval grader measures compliance, but nothing on
the runner enforces it.

Proposed fix

In post.sh, after the existing checks, fail closed (new reason, for
example unexpected-path-staged) when the final index holds an entry
that is neither identical to a stage-0 snapshot entry nor a path the
snapshot held unmerged. Mirror the workflow-staged check and add the
case to post.test.sh.

Related

Raised in review of #7178:
https://github.com/melodic-software/claude-code-plugins/pull/7178#discussion_r4244581612

Lingua principale
Shell
Stelle
22
Fork
2
Merge medio
8h 34m
PR unite (30g)
946

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.