mcpbeat

Code Review

selmakcby/claude-agents-code-review

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

381 tokens
context cost
the whole folder, loaded on every use
1
files
instructions only
0
copies elsewhere
how many repositories repackaged it
112
stars on the repo
on the repository, not the skill itself

Install

one command, takes just this skill from the repository
npx skills add https://github.com/selmakcby/claude-agents-skills --skill code-review

The instruction itself

10 sections, as written by the author

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

How to use it

Copy the folder

Take selmakcby/claude-agents-code-review from the repository into ~/.claude/skills for personal use, or into .claude/skills inside a project.

Check the name does not clash

The agent identifies a skill by the name field in its header. Two skills with the same name cannot sit side by side — one of them will be ignored.