pr-review-security lane reports success and marks the head reviewed after permission denials stop it early
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
- 52/100
Direzione di ricerca
Reproduce from the described pr-review-security lane on PR #6382 (run 37247247574, head f1d3233): permission denials stopped the checklist before the hunter step, yet the run was success and a reviewed-through marker was written. Find where that marker is written and where the lane exit status is set after denials. Done when an incomplete run does not mark the head reviewed and does not report success, so the next push gets a full review.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
On #6382 head f1d3233, the pr-review-security lane run ended success after 29 turns with 7 permission denials. Its checklist stopped before the hunter step and it posted no findings, yet it wrote a "reviewed through f1d3233" marker. Later pushes therefore got only an incremental review of new commits, so the full diff was never security-reviewed by the lane (a local review stood in).
Expected
A run that did not complete its review steps must not write the reviewed-through marker and should not conclude success (fail or neutral with a reason), so the next push gets a full review.
Evidence
- PR #6382, security-review lane run 37247247574 (head f1d3233).
🤖 Generated with Claude Code
- Lingua principale
- Shell
- Stelle
- 22
- Fork
- 2
- Merge medio
- 5h 15m
- PR unite (30g)
- 833
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
-
good first issue needs-triage priority: medium
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
melodic-software/claude-code-plugins#6631 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
melodic-software/claude-code-plugins#6547 ·
I maintainer di solito rispondono entro 1 giorno
-
needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
melodic-software/claude-code-plugins#6535 ·
I maintainer di solito rispondono entro 1 giorno
-
test_comment_census.py: SccArgv flag-shaped-filename test errors on Windows (#!/bin/sh scc shim)Apertagood first issue needs-triage priority: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
melodic-software/claude-code-plugins#6532 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue needs-triage priority: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
melodic-software/claude-code-plugins#6390 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di melodic-software/claude-code-plugins
Issue simili
-
status:needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
enhancement good first issue help wanted
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Sorogate/example-consumer#14 ·
-
github_actions good first issue help wanted
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 65/100
-
documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
alunduil/alunduil-chezmoi#874 ·
I maintainer di solito rispondono entro 1 giorno