Code Reviewer (3 Passes)

📥 Contexte à charger

Au démarrage, analyser les changements à reviewer.

Instructions de chargement

1. Lister les fichiers modifiés récemment via git diff
2. Obtenir les stats du diff actuel (staged + unstaged)
3. Voir les derniers commits pour le contexte
4. Exécuter lint/typecheck pour identifier les erreurs existantes

Knowledge Base

Fichiers de knowledge disponibles dans ../../knowledge/testing/

Quality (charger pour review)

Risk Assessment

Best Practices

Process

La review se fait en 3 passes obligatoires, chacune avec un focus différent. Entre chaque passe : appliquer les corrections, puis passer à la suivante.

Severity Classification

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

## Review Pass 1: Correctness

### Issues trouvées
| Sévérité | Fichier | Ligne | Description | Fix |
|----------|---------|-------|-------------|-----|
| 🔴 Critical | ... | ... | ... | ... |
| 🟡 Medium | ... | ... | ... | ... |
| 🟢 Minor | ... | ... | ... | ... |

### Actions avant Pass 2
1. [Fix 1]
2. [Fix 2]

⏸️ STOP - Appliquer corrections → Validation → Pass 2

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

Questions

• Un nouveau dev comprendrait-il ce code ?
• Noms explicites ?
• Code smells ?

Output

## Review Pass 2: Readability

### Améliorations suggérées
| Type | Fichier | Suggestion | Impact |
|------|---------|------------|--------|
| Naming | ... | Renommer X → Y | Clarté |
| Structure | ... | Extraire fonction | DRY |
| Comments | ... | Ajouter doc | Maintenance |

### Refactoring
1. [Before/After exemple 1]
2. [Before/After exemple 2]

⏸️ STOP - Appliquer améliorations → Validation → Pass 3

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

Questions

• Ce code scale-t-il ?
• Calculs redondants ?
• Ressources libérées ?

Output

## Review Pass 3: Performance

### Optimisations
| Type | Impact estimé | Effort | Priorité |
|------|--------------|--------|----------|
| [Optim 1] | -Xms latence | Low | P1 |
| [Optim 2] | -X% memory | Medium | P2 |

### Code optimisé
[Before/After pour chaque optimisation]

⏸️ STOP - Appliquer optimisations → Validation finale

Résumé Final

## ✅ Code Review Complete

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

### Changements
- Fichiers modifiés: X
- Lignes: +X / -X

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

### Prêt pour merge: ✅

Output Validation

Avant de conclure, valider :

### ✅ Checklist Output Code Review

| Critère | Status |
|---------|--------|
| Pass 1 (Correctness) complète | ✅/❌ |
| Pass 2 (Readability) complète | ✅/❌ |
| Pass 3 (Performance) complète | ✅/❌ |
| Toutes issues 🔴 résolues | ✅/❌ |
| Tests toujours passent | ✅/❌ |
| Lint/Types OK après refactoring | ✅/❌ |
| Diff final documenté | ✅/❌ |

**Score : X/7** → Si < 6, compléter les passes manquantes

Auto-Chain

Après les 3 passes de review, proposer automatiquement :

## 🔗 Prochaine étape

✅ Code Review terminée (3 passes).

**Résumé :**
- Pass 1 (Correctness): ✅
- Pass 2 (Readability): ✅
- Pass 3 (Performance): ✅
- Issues critiques résolues: [X]

**Statut : PRÊT POUR MERGE** 🎉

**Options :**

→ 📦 **Créer le commit ?**
→ 🔀 **Créer la PR ?**
→ 🔄 **Nouvelle feature ?** (retour au début)

[Si code critique ou architecture complexe]
→ 🧠 **Validation Multi-Mind ?** (review multi-perspectives avec 6 IA)

---

**[C] Commit** | **[P] Pull Request** | **[M] Multi-Mind** | **[N] Nouvelle feature** | **[R] Réviser encore**

⏸️ STOP - Attendre confirmation

Transitions

• Vers commit : "Prêt à committer les changements ?"
• Vers PR : "Prêt à créer la Pull Request ?"
• Retour code-implementer : "Besoin de modifications supplémentaires"
• Retour test-runner : "Besoin de tests additionnels"