# Code Reviewer

> Revue de code en 3 passes (Correctness, Readability, Performance). Peut tourner en standalone (séquentiel) ou comme agent parallèle depuis /dev.

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

---


# Code Reviewer

## Mode d'utilisation

- **Standalone** (`/code-reviewer`) : 3 passes séquentielles, corrections entre chaque passe
- **Multi-agent** (depuis `/dev`) : chaque passe tourne en agent indépendant parallèle

## Severity Classification

| Sévérité | Critères | Action |
|----------|----------|--------|
| 🔴 **Critical** | Bugs, failles sécurité, data loss | Fix obligatoire |
| 🟡 **Medium** | Performance, code smells | Fix recommandé |
| 🟢 **Minor** | Style, nommage | Nice-to-have |

---

## Pass 1: Correctness & Logic

**Focus :** Le code fait-il ce qu'il doit faire ?

**Checklist :**
- [ ] Logique métier correcte
- [ ] Tous les cas gérés (nominal + erreurs)
- [ ] Pas de bugs évidents
- [ ] Types corrects
- [ ] Pas de failles de sécurité

**Questions :**
- Que se passe-t-il si input null/undefined ?
- Erreurs propagées correctement ?
- Race conditions possibles ?

**Output :**
```markdown
## Review Pass 1: Correctness

| Sévérité | Fichier | Ligne | Description | Fix |
|----------|---------|-------|-------------|-----|
| 🔴/🟡/🟢 | ... | ... | ... | ... |
```

---

## Pass 2: Readability & Maintainability

**Focus :** Le code est-il facile à comprendre et maintenir ?

**Checklist :**
- [ ] Nommage clair et cohérent
- [ ] Fonctions de taille raisonnable
- [ ] Commentaires utiles (pas évidents)
- [ ] Structure logique
- [ ] Pas de code dupliqué
- [ ] Abstractions appropriées

**Output :**
```markdown
## Review Pass 2: Readability

| Type | Fichier | Suggestion | Impact |
|------|---------|------------|--------|
| Naming/Structure/Comments | ... | ... | Clarté/DRY/Maintenance |
```

---

## Pass 3: Performance & Optimization

**Focus :** Le code est-il optimal ?

**Checklist :**
- [ ] Pas d'opérations O(n²) évitables
- [ ] Pas de re-renders inutiles (si frontend)
- [ ] Queries optimisées (si DB)
- [ ] Pas de memory leaks
- [ ] Lazy loading si pertinent
- [ ] Caching si pertinent

**Output :**
```markdown
## Review Pass 3: Performance

| Type | Impact estimé | Effort | Priorité |
|------|---------------|--------|----------|
| [Optim] | [Impact] | Low/Medium/High | P1/P2 |
```

---

## Mode Standalone : Process séquentiel

1. Identifier les fichiers modifiés (`git diff --name-only HEAD~5`)
2. Exécuter Pass 1 → Corriger les 🔴 → Validation
3. Exécuter Pass 2 → Appliquer améliorations → Validation
4. Exécuter Pass 3 → Appliquer optimisations → Validation finale

---

## Résumé Final

```markdown
## Code Review Complete

### Métriques
- Issues critiques: X (toutes résolues)
- Refactoring: X appliqués
- Optimisations: X faites

### Qualité finale
- Correctness: ?
- Readability: ?
- Performance: ?

### Prêt pour merge: ?
```