reviewing-code

Systematically evaluate code changes for security, correctness, performance, and spec alignment. Use when reviewing PRs, assessing code quality, or verifying implementation against requirements.

1. Przygotuj dokumentację projektu: upewnij się, że masz dostęp do specyfikacji funkcji (docs/feature-spec/), historii użytkownika (docs/user-stories/), kontraktów API (docs/api-contracts.yaml) i planu danych (docs/data-plan.md). Te materiały stanowią punkt odniesienia dla przeglądu.

2. Określ zakres przeglądu: zidentyfikuj liczbę zmienionych plików i powiązane funkcje (numery F-##), implementowane historie (numery US-###) oraz wszelkie zmiany w API, bazie danych lub schemacie. Dla małych PR-ów (poniżej 5 plików) wybierz przegląd sekwencyjny, dla dużych (powyżej 5 plików) rozważ podejście równoległe.

3. W przypadku przeglądu sekwencyjnego przejdź przez kod kolejno, skupiając się na pięciu wymiarach: bezpieczeństwo (walidacja danych, uwierzytelnianie, obsługa wrażliwych informacji), poprawność (logika zgodna z kryteriami akceptacji, obsługa przypadków granicznych), zgodność ze specyfikacją (API, zdarzenia danych, interfejs użytkownika), wydajność i utrzymywalność.

4. W przypadku dużych PR-ów zidentyfikuj niezależne aspekty przeglądu (bezpieczeństwo, API, interfejs użytkownika, dane) i przydziel każdy wymiar specjalistycznym agentom do równoległej analizy, a następnie skonsoliduj wyniki.

5. Zbierz ustalenia z przeglądu i posortuj je według priorytetu. Dla każdego problemu wskaż, który wymiar go dotyka i jak wpływa na jakość kodu.

6. Wydaj rekomendację: zatwierdź PR, poproś o rewizję (drobne poprawki) lub zalecaj pełną przebudowę (krytyczne problemy). Uzasadnij decyzję na podstawie zebranych ustaleń.