feat(source-control): review instruction-surface diffs in pull-request ready
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 68/100
Direzione di ricerca
Leggi plugins/source-control/skills/pull-request/reference/ready-for-review.md, in particolare la sezione 2.5.3 e le condizioni per le modifiche che riguardano solo la documentazione. Aggiorna la regola affinché i diff alle superfici di istruzioni come AGENTS.md, CLAUDE.md, .claude/rules/ e ai contenuti di skill o agent ricevano la revisione di sicurezza prima del passaggio a ready. Il lavoro è completato quando tale revisione viene eseguita per questi diff e i diff ordinari che riguardano solo la documentazione mantengono il comportamento indicato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Refs: #7346
/source-control:pull-request ready skips both the code and security review for a docs-only diff (plugins/source-control/skills/pull-request/reference/ready-for-review.md 2.5.3, the "for any diff that is not docs-only" conditions). A diff that changes an instruction surface (AGENTS.md, CLAUDE.md, .claude/rules/, skill and agent bodies) is docs-only by file type but changes what agents do. Under the carve-out #7346 proposes, an agent could flip such a PR it opened without any local review.
Mitigating today: the flip starts the CI review lanes, and merging stays separately gated.
Proposed: treat an instruction-surface diff as not docs-only for 2.5.3, so at least the security review runs before the flip.
Found by the claude-review lane on #7346 and by a local security review during the ready flow.
🤖 Generated with Claude Code
- Lingua principale
- Shell
- Stelle
- 22
- Fork
- 2
- Merge medio
- 8h 27m
- PR unite (30g)
- 957
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di melodic-software/claude-code-plugins
-
agent-ready priority: high work-class: scoped
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
melodic-software/claude-code-plugins#7312 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
priority: medium
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
melodic-software/claude-code-plugins#7288 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
knowledge: docpage-digest dual verification defines no verdict vocabulary or pass/fail thresholdApertaneeds-human
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
melodic-software/claude-code-plugins#7286 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
priority: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
melodic-software/claude-code-plugins#7282 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
needs-triage priority: medium
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
melodic-software/claude-code-plugins#7273 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di melodic-software/claude-code-plugins
Issue simili
-
Sourced-library load-guards use a top-level `return` that errors on direct execution (repo-wide)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
rossoctl/automation#115 ·
I maintainer di solito rispondono entro 3 giorni
-
loongarch64: `unsupported linker arg: --no-rosegment` with rustc 1.100Forse già presa @monosans l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
rust-cross/cargo-zigbuild#503 ·
I maintainer di solito rispondono entro 1 giorno
-
netbox status: backlog topic: testing type: housekeeping
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
netbox-community/netbox#23395 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno