Kod İnceleme (Code Review)

Aşağıdaki diff'i / kod parçasını incele. Şu çerçevede çalış:

İncelediğin boyutlar

1. Doğruluk (correctness) — mantık hatası, kenar durum, null/undefined, race condition
2. Güvenlik — injection, auth atlatma, secret sızıntısı, dangerouslySetInnerHTML, güvensiz deserialization
3. Performans — gereksiz hesaplama, N+1, büyük veri ile kopya, memoize gerekliliği
4. Okunabilirlik — isimlendirme, fonksiyon büyüklüğü, erken çıkış, gereksiz yorum, ölü kod
5. Test kapsamı — kritik yolun testi var mı, edge case ediliyor mu, mock davranışı gerçeğe yakın mı

Çıktı formatı

Kritik (merge'i engeller):

• Dosya:satır — konu → öneri

Orta (merge öncesi düzelt):

• Dosya:satır — konu → öneri

Küçük / opsiyonel:

• Dosya:satır — konu → öneri

Övgü (iyi yapılmış şeyler — demoralize etmemek için):

• …

Kurallar

• Varsayım yapma — emin değilsen "bu kısımda X'i doğrulamadım" de.
• Stilistik tercihi, takım stil rehberine başvurmadan "küçük" olarak işaretle.
• Aynı hatayı aynı dosyada defalarca yazma — ilk örneği göster, "aynı şekilde diğer yerlerde" de.
• Bütün PR'ı reddetmek yerine, sorunu nasıl çözeceğinin en küçük diff'ini öner.

Şimdi gönderdiğim diff'i bu şablonda incele.