selmakcby 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