Kod İncelemesi (Code Review)

Kod incelemesi, önerilen bir değişikliğin — tipik olarak bir çekme isteği (PR) veya birleştirme isteği — paylaşılan kod tabanına birleştirilmeden önce yazarından başka birinin incelemesi uygulamasıdır. İnceleyen, doğruluğu (mantık gerçekten iddia ettiğini yapıyor mu), sürdürülebilirliği (okunabilir mi, ekip kurallarına uyuyor mu), güvenlik sorunlarını ve gözden kaçan uç durumları kontrol eder ve onaylamadan önce yorum bırakır veya değişiklik talep eder. AI/SaaS geliştiricileri için önemi: kod incelemesi, modern bir geliştirme iş akışındaki birincil insan kalite kapısıdır ve AI-kodlama çağında önemi azalmadı, arttı — AI tarafından üretilen kod genellikle makul görünür ancak kendinden emin görünen bir diff'in gizleyebileceği ince mantık hataları, güvenlik açıkları (eksik yetkilendirme kontrolleri gibi) veya gereksiz karmaşıklık içerebilir. İnceleme aynı zamanda örtük bilginin bir ekip genelinde nasıl yayıldığı ve deneyimsiz geliştiricilerin kod tabanı kurallarını nasıl öğrendiğidir. Giderek daha fazla, AI araçları da doğrudan bu adıma katılıyor: otomatik inceleme botları (Claude destekli bir PR inceleyicisi veya GitHub'ın Copilot kod incelemesi gibi), bir insan son, muhakeme ağırlıklı geçişi yapmadan önce ilk yorum geçişini bırakır — bariz sorunları işaretleyerek. Nasıl çalışır: bir geliştirici, bir değişikliği hedef dala karşı bir diff olarak tanımlayan bir PR açar. İnceleyenler diff'i okur (genellikle belirli satırlarda satır içi yorumlarla), değişiklik talep edebilir (ele alınana kadar birleştirmeyi engeller), onaylayabilir veya isteğe bağlı öneriler bırakabilir. Birçok ekip, bir PR'nin birleşebilmesi için minimum sayıda onay ve geçen CI kontrolleri gerektirir; bu, barındırma platformundaki dal koruma kurallarıyla uygulanır. Uygulamalı örnek: bir geliştirici "Raporlar için toplu CSV dışa aktarma ekle" başlıklı bir PR açar. Bir AI inceleme botu otomatik olarak çalışır ve yorum yapar: "Satır 34: bu, CSV'ye akıtmadan önce tüm veri kümesini belleğe yüklüyor — büyük hesaplar için bu OOM'a neden olabilir; bunun yerine bir imleç tabanlı akış düşünün." Ardından bir insan inceleyici ekler: "Ayrıca, bu uç nokta, isteği yapan kullanıcının bu belirli rapor üzerinde dışa aktarma izni olup olmadığını kontrol etmiyor — `reports_controller.rb` dosyasının 12. satırındaki desene bakın." Yazar her iki sorunu da düzeltir, güncellenmiş bir commit gönderir (PR tartışma dizisi orijinal yorumlara bağlı kalır, böylece inceleyenler düzeltmeyi doğrulayabilir) ve insan inceleyici "Onayla"ya tıklar; bu, yeşil bir CI çalışmasıyla birleştiğinde birleştirme düğmesinin kilidini açar. Tüm alışveriş PR'de kalıcı olarak korunur, böylece altı ay sonra "bu uç nokta neden her şeyi bir kerede yüklemek yerine akıtıyor" diye merak eden yeni bir ekip üyesi orijinal inceleme yorumunu okuyup mantığı hemen anlayabilir.