Show a completion status when a Review Helper re-review has no findings

Aperta Adatta ai principianti
#6,832 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
78/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
python
Ambito
api, backend

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
  1. Request a Review Helper review on a revision. The first review posts its summary and any inline findings.
  2. Upload a new diff that addresses the findings.
  3. Request another Review Helper review.
  4. 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
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

  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 mozilla/bugbug

Tutte le issue di mozilla/bugbug

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.