Delivery · jakość kodu
Przegląda każdy pull request: bezpieczeństwo, wydajność, styl. Zostawia konkretne komentarze w liniach kodu i pilnuje standardów zespołu.
- Wejście
- pull request
- Wyjście
- review z komentarzami w liniach
- Zakres
- bezpieczeństwo · wydajność · styl
Co robi
Zakres pracy
Review każdego PR-a
Bezpieczeństwo, wydajność, czytelność i zgodność ze standardami zespołu - w każdym pull requeście, nie tylko wtedy, gdy ktoś ma czas.
Jakość · każdy PR
Komentarze w liniach
Konkretny problem, konkretna linia, propozycja poprawki. Zero ogólników w stylu „wygląda dobrze”.
Precyzja · line-level
Pilnuje standardów
Konwencje repo, wzorce architektury i checklisty zespołu są częścią jego kontekstu - egzekwuje je spójnie u ludzi i u agentów.
Standardy · spójność zespołu
W produkcie
Tak to wygląda w Braio
- Weryfikacja to osobny krok pętli - kod sprawdza inny agent niż ten, który go napisał
- Komentarze w liniach kodu w GitHub / GitLab
- Wynik review w raporcie z dowodami
Jak pracuje
Jedna pętla, sześć kroków
- 01Zadaniez czatu, backlogu lub harmonogramu
- 02Plan i kryteriajawny zakres, repo, gałąź, poziom ryzyka
- 03Twoja zgodabramka Tak/Nie przed uruchomieniem
- 04Wykonaniekod, treść lub analiza w Twoich narzędziach
- 05Weryfikacjacode review, testy, QA w przeglądarce
- 06Raport z dowodamizrzuty, oś czasu, metryki
Bramka zgody: review dzieje się automatycznie; decyzja o merge zawsze należy do człowieka albo do reguł, które sam ustawisz.
Kontrola
Autonomia na Twoich warunkach
Samo review to poziom 1 - komentarze i wpis w logu. Approve z automatycznym merge można włączyć wyłącznie jako poziom 2, z timerem SLA i planem wycofania.
Scenariusz · Software house
- Dziś
- code review blokuje sprinty i zależy od dostępności seniora
- Z Braio
- każdy PR ma review w minutach, a senior patrzy tylko tam, gdzie agent zgłasza ryzyko