pr-review

name	pr-review
description	Mevcut PR'ı veya belirtilen PR numarasını kapsamlı şekilde inceler
argument-hint	["pr-number"]
user-invocable	true
allowed-tools	Bash(gh *) Read Grep
effort	medium

PR Bağlamı

Diff:

!`gh pr diff $1 2>/dev/null || gh pr diff`

Açıklama ve yorumlar:

!`gh pr view $1 --comments 2>/dev/null || gh pr view --comments`

CI durumu:

!`gh pr checks $1 2>/dev/null || gh pr checks`

---

İnceleme Kriterleri

Aşağıdaki alanlarda kapsamlı inceleme yap:

Doğruluk

• Değişiklikler PR açıklamasındaki amacı karşılıyor mu?
• Edge case'ler ele alınmış mı?
• Hata yönetimi yeterli mi?

Güvenlik

• SQL injection, XSS, CSRF gibi açıklar var mı?
• Hassas veriler loglanıyor veya expose ediliyor mu?
• Yetkilendirme kontrolleri doğru mu?

Kod Kalitesi

• Kod DRY prensiplerine uyuyor mu?
• Fonksiyonlar tek sorumluluk ilkesini takip ediyor mu?
• Karmaşık mantık için yeterli yorum var mı?

Test

• Yeni kod için test yazılmış mı?
• Mevcut testler hâlâ geçiyor mu?
• Test senaryoları yeterli kapsamı sağlıyor mu?

Performans

• Gereksiz veritabanı sorgusu veya API çağrısı var mı?
• Büyük veri setleri için düşünülmüş mü?

---

Çıktı Formatı

Şu başlıklar altında bulgularını sun:

Kritik Sorunlar - merge öncesi mutlaka düzeltilmeli
Öneriler - kaliteyi artıracak değişiklikler
Küçük Notlar - stil veya tercih meselesi
Olumlu Noktalar - iyi yapılan şeyler

Her bulgu için ilgili dosya ve satır numarasını belirt.