Preskoči na sadržaj

PR proces i follow-up discipline

TL;DR

Minimum Round 1 review (code-reviewer + silent-failure-hunter) za svaki PR — bez izuzetka, bez obzira na trivialnost. Follow-up posao ide na Jira CI board (project CI, label followup), NE u docs/FOLLOW_UPS.md. Za ne-trivijalne promjene pokreni i Round 2 (code-simplifier) i Round 3 (final verification).

Minimum Round 1

Kreiraš PR (bilo kroz /pr skill, bilo ručno). Uvijek pokreni:

  1. code-reviewer — provjerava adherence na CLAUDE.md pravila, style, anti-patterns.
  2. silent-failure-hunter — traži neadekvatno error handling, try/except blokove koji swallowaju, fallback logiku.

Bez ovog review-a, PR se NE SMIJE merge-ati. Čak i za typo fix.

Opcioni Round 2 + Round 3

Za veće promjene (refactoring, nova shema, novi vendor, breaking change):

  1. code-simplifier — reuse, simplification, efficiency.
  2. Final verification — read konačni diff, sanity check, korisnikov "sve vidim, merge".

Bot review dinamika

  • Bot (Glitchy) NE formalno approve-uje (approved=false je normalno). Verdict = tekst REVIEW_RESULT comment-a.
  • "Verdict: APPROVED" u komentaru = merge-ready.
  • "Verdict: REQUEST_CHANGES" = fix pa re-push.
  • "Looking for bugs..." = aktivan pass, NE merge-aj dok ne završi.

Kad je PR "merge-ready" — NE na temelju boja pipeline-a

PR green pipeline ≠ merge-ready. Gate merge-a na reviewer's EKSPLICITNI verdict na tačno trenutni HEAD, ne na timing/quiet-poll heuristiku. Bot review laguje (tišina ≠ gotovo).

Čekaj dok:

  1. Svi prr/* pass-ovi su DONE (ne INPROGRESS)
  2. Zadnji komentar ima verdict (REQUEST_CHANGES ili APPROVED)
  3. Ako je APPROVED — provjeri da nema naknadnih REQUEST_CHANGES

Follow-up discipline

"Deferiraću ovo za follow-up" u PR prose → NE POSTOJI.
Samo Jira ticket na CI board-u je realan follow-up.

Kad odlažeš:

  1. File Jira issue u projektu CI (cloudId 0676a856-6048-409e-bd2d-740346693261), label followup.
  2. U ticketu zapiši:
  3. Deferred from: branch + PR (broj, stanje) + commit short-SHA
  4. Date: YYYY-MM-DD
  5. What / Why / Next step: sa file:line anchorima
  6. Link key u PR description — CI-42 na vidljivom mjestu.
  7. Ako se follow-up našao u toku rada na drugom ticketu, link Relates (Rovo--createIssueLink) da veza bude vidljiva na board-u.

Tri tipične greške

  • ❌ "Malo je, ne treba review" → uvijek treba.
  • ❌ "Pipeline je zelen, merge-aj" → ne bez verdict-a.
  • ❌ "Ostaviću TODO u komentaru" → ne, file Jira.

Anti-patterns (zaobilaze discipline)

  • ❌ Merge bez review round-a (čak i 1-liner).
  • ❌ Auto-merge na "quiet polls" heuristici.
  • ❌ Follow-up bez Jira ticket-a.
  • ❌ "Dismiss as scope creep" review flag bez čitanja fajla.

Vidi i