Mevcut PR'ı veya belirtilen PR numarasını kapsamlı şekilde inceler
Scanned 6/1/2026
Install via CLI
openskills install berkcangumusisik/claude-code-practices---
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.
No comments yet. Be the first to comment!