Show a completion status when a Review Helper re-review has no findings
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
Direzione di ricerca
Inizia in services/reviewhelper-api/app/review_processor.py, nella gestione del messaggio della revisione successiva, e confrontala con il percorso del riepilogo della prima revisione. Poi esamina services/reviewhelper-api/app/routers/internal.py per capire quando la pubblicazione viene contrassegnata come completata. Il lavoro è completato quando una revisione successiva senza rilievi lascia una transazione di completamento visibile senza ripetere il riepilogo della patch.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Review Helper re-reviews can complete successfully without leaving any visible indication on the Phabricator revision.
Steps to reproduce
- Request a Review Helper review on a revision. The first review posts its summary and any inline findings.
- Upload a new diff that addresses the findings.
- Request another Review Helper review.
- If the re-review generates no findings, inspect the revision timeline and try requesting another review for the same diff.
D325624 demonstrates this with Diff 1374611. Review Helper reports:
Review Helper already posted its review for Diff 1374611.
However, the revision timeline contains Review Helper output only for the previous Diff 1374609. There is no visible transaction for Diff 1374611.
Actual behavior
Subsequent reviews intentionally omit the patch summary, following https://github.com/mozilla/bugbug/issues/5869. If the review also has no inline findings, differential.createcomment receives no message and no inline comments. Phabricator accepts the no-effect transaction, and Review Helper records the request as published, but nothing appears in the revision timeline.
From the revision alone, reviewers cannot distinguish a successful review with no findings from a request that was never made, is still processing, or failed silently. The only confirmation is attempting another request and receiving the “already posted” dialog.
Expected behavior
Post a lightweight completion transaction for a subsequent review with no findings, without repeating the patch summary. For example:
Review Helper completed its review of Diff 1374611 with no new comments.
This would preserve the reduced-noise behavior from https://github.com/mozilla/bugbug/issues/5869 while providing an auditable completion signal.
Relevant implementation
- Subsequent reviews omit the general message: https://github.com/mozilla/bugbug/blob/f786d08f857a3a2620ce9c3c601c3c8f5636cc15/services/reviewhelper-api/app/review_processor.py#L156-L167
- The “completed with no comments” text currently belongs to the first-review summary path: https://github.com/mozilla/bugbug/blob/f786d08f857a3a2620ce9c3c601c3c8f5636cc15/services/reviewhelper-api/app/review_processor.py#L170-L188
- The request is marked published after the no-effect publication succeeds: https://github.com/mozilla/bugbug/blob/f786d08f857a3a2620ce9c3c601c3c8f5636cc15/services/reviewhelper-api/app/routers/internal.py#L155-L165
- Lingua principale
- Python
- Stelle
- 570
- Fork
- 351
- Merge medio
- 2g 11h
- PR unite (30g)
- 61
Guida per i contributori
Apri 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 mozilla/bugbug
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
good-first-bug hackbot
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
-
hackbot
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
-
hackbot
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
hackbot
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
Tutte le issue di mozilla/bugbug
Issue simili
-
triage/confirmed
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
agentscope-ai/agentscope#2775 ·
-
comp/desktop P3 type/bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
NousResearch/hermes-agent#118866 ·
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
apache/cloudstack#14222 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100