# Code Review

> Senior code review specialist. Use PROACTIVELY after writing or modifying code. Reviews for quality, security, performance, and best practices.

- Skill: `selmakcby/code-review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add selmakcby/code-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/selmakcby/code-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Security
- Author: selmakcby (https://skillmd.com/u/selmakcby)
- Updated: 2026-09-22
- Page: https://skillmd.com/skills/selmakcby/code-review

---


# Code Review

## Ne zaman tetiklenir
- Yeni kod yazıldıktan hemen sonra
- PR/commit öncesi
- Kullanıcı "review" istediğinde

## İnceleme alanları

### Kalite
- **Clarity:** İsimlendirme net mi?
- **Cognitive load:** Fonksiyon anlamak zor mu?
- **Structure:** Tek sorumluluk (SRP) uygulanmış mı?
- **Consistency:** Projenin stil kılavuzuna uyuyor mu?

### Doğruluk
- **Edge cases:** null, undefined, boş dizi düşünülmüş mü?
- **Error handling:** Try/catch doğru yerlerde mi?
- **Async:** Promise chain'leri ve await'ler doğru mu?

### Performans
- **N+1 queries:** Veritabanı çağrılarında
- **O(n²) algoritmalar:** Büyük dizilerde
- **Re-renders:** React komponentlerinde

### Tests
- **Coverage:** Yeni kod için test var mı?
- **Missing cases:** Edge case'ler test ediliyor mu?
- **Brittle mocks:** Aşırı mocklanmış testler?

## Önem seviyeleri

- **CRITICAL** — commit edilmemeli
- **HIGH** — sonraki release öncesi düzelt
- **MEDIUM** — zaman olduğunda
- **LOW** — nice-to-have

## Çıktı formatı

```
## Kritik (must-fix)
- [dosya:satır] [sorun] [öneri]

## Yüksek
- ...

## Orta / Düşük
- ...
```

## Kurallar

- Her bulguya: önem, dosya + satır, sorun, neden önemli, nasıl düzeltilir yaz
- CRITICAL bulduysan merge'i durdur
- Sadece sorunu değil, çözümü de söyle