Code Review

Ne zaman tetiklenir

• Yeni kod yazıldıktan hemen sonra
• PR/commit öncesi
• Kullanıcı "review" istediğinde

İnceleme alanları

Kalite

• Clarity: İsimlendirme net mi?
• Cognitive load: Fonksiyon anlamak zor mu?
• Structure: Tek sorumluluk (SRP) uygulanmış mı?
• Consistency: Projenin stil kılavuzuna uyuyor mu?

Doğruluk

• Edge cases: null, undefined, boş dizi düşünülmüş mü?
• Error handling: Try/catch doğru yerlerde mi?
• Async: Promise chain'leri ve await'ler doğru mu?

Performans

• N+1 queries: Veritabanı çağrılarında
• O(n²) algoritmalar: Büyük dizilerde
• Re-renders: React komponentlerinde

Tests

• Coverage: Yeni kod için test var mı?
• Missing cases: Edge case'ler test ediliyor mu?
• Brittle mocks: Aşırı mocklanmış testler?

Önem seviyeleri

• CRITICAL — commit edilmemeli
• HIGH — sonraki release öncesi düzelt
• MEDIUM — zaman olduğunda
• LOW — nice-to-have

Çıktı formatı

## Kritik (must-fix)
- [dosya:satır] [sorun] [öneri]

## Yüksek
- ...

## Orta / Düşük
- ...

Kurallar

• Her bulguya: önem, dosya + satır, sorun, neden önemli, nasıl düzeltilir yaz
• CRITICAL bulduysan merge'i durdur
• Sadece sorunu değil, çözümü de söyle