Peer Review

name: peer_review router_kit: FullStackKit description: Akademik/teknik doküman review, methodology değerlendirme. ⚠️ Doküman/araştırma için kullan. Kod review için → code-review. metadata: skillport: category: research tags: [architecture, automation, best practices, clean code, coding, collaboration, compliance, debugging, design patterns, development, documentation, efficiency, git, optimization, peer review, productivity, programming, project management, quality assurance, refactoring, software engineering, standards, testing, utilities, version control, workflow] - quality

📝 Peer Review

Akademik ve teknik peer review metodolojisi rehberi.

📋 Review Framework

Değerlendirme Alanları

🔍 Code Review Checklist

## Functionality
- [ ] Kod beklendiği gibi çalışıyor mu?
- [ ] Edge case'ler handle ediliyor mu?
- [ ] Error handling yeterli mi?

## Code Quality
- [ ] DRY prensibi uygulanmış mı?
- [ ] Naming convention tutarlı mı?
- [ ] Comments yeterli mi?

## Security
- [ ] Input validation var mı?
- [ ] SQL injection riski var mı?
- [ ] Sensitive data korumalı mı?

## Performance
- [ ] Gereksiz işlem var mı?
- [ ] Memory leak riski var mı?

📄 Document Review Template

## Review Summary
**Document:** [Doküman adı]
**Reviewer:** [İsim]
**Date:** [Tarih]

## Overall Assessment
[Genel değerlendirme - 1-2 paragraf]

## Strengths
1. ...
2. ...

## Areas for Improvement
1. ...
2. ...

## Specific Comments
| Section | Comment | Severity |
|---------|---------|----------|
| ... | ... | Major/Minor |

## Recommendation
[ ] Accept
[ ] Minor Revisions
[ ] Major Revisions
[ ] Reject

💬 Constructive Feedback

İyi Feedback

✅ "Bu fonksiyon X durumunda hata verebilir.
    Try-catch eklemeyi düşünebilir misin?"

✅ "Güzel implementasyon! Bir öneri:
    Bu method extract edilse daha okunabilir olur."

Kaçınılması Gereken

❌ "Bu yanlış"
❌ "Neden böyle yaptın?"
❌ "Ben olsam böyle yapmazdım"

Peer Review v1.1 - Enhanced

🔄 Workflow

Kaynak: Conventional Comments & Google Engineering Practices

Aşama 1: Preparation (Reviewee)

• Self-Review: Kodu pushlamadan önce kendin oku, gereksiz logları temizle.
• Context: PR açıklamasında "Ne?", "Neden?" ve "Nasıl Test Edilir?" sorularını yanıtla.
• Scope: Değişikliği yönetilebilir boyutta tut (<400 satır tercihen).

Aşama 2: Review Process (Reviewer)

• Clarity: Kod ne yaptığını anlatıyor mu? Değişken isimleri açıklayıcı mı?
• Security: Kullanıcı girdisi sanitize ediliyor mu? Auth kontrolü var mı?
• Performance: Gereksiz döngüler veya N+1 sorguları var mı?
• Tone: Yorumlarda nit:, suggestion:, blocking: gibi etiketler kullan (Conventional Comments). "Sen" dili yerine "Kod" dili kullan.

Aşama 3: Approval & Merge

• CI Checks: Tüm testler ve lint kontrolleri yeşil mi?
• Resolution: Tüm kritik yorumlar (blocking) çözüldü mü?
• Squash & Merge: History temizliği için commitleri birleştir.

Kontrol Noktaları