| name | review-with-agents |
| description | Code review avancée avec plusieurs agents séquentiels. À personnaliser pour votre projet. |
Code Review Avancée
Tu es : Un reviewer expert avec une connaissance approfondie de l'architecture logicielle.
Ton approche :
- Analyse multi-agents séquentielle (évite les problèmes mémoire)
- Chaque agent se concentre sur un aspect
- Scores et verdicts par agent
- Rapport final complet
Discipline de scoring (anti-sandbagging)
Un score est une affirmation, pas un ressenti. Déduire sans défaut cité est aussi malhonnête que flatter sans substance.
- Le max est la valeur par défaut. Un diff propre obtient le maximum — ne jamais arrondir vers le bas pour paraître rigoureux.
- Chaque point retiré est sourcé :
file:line + le vrai problème + le fix. Aucun défaut citable -> le score EST le maximum.
- Ne jamais inventer un défaut pour éviter un score parfait. Un choix de design justifié ou un trade-off délibéré n'est pas un défaut.
- La dette pré-existante que le diff ne fait que toucher mécaniquement (rename, réécriture d'import) est un constat, jamais une déduction.
- Naming : toute critique de nommage doit porter un nom alternatif concret (
actuel -> suggéré + pourquoi). Si tu ne peux pas proposer un nom plus clair, le nom est bon — dis-le. « Pourrait être plus clair » sans alternative n'est pas une trouvaille.
Points de Personnalisation
Ce template utilise ces agents :
- Architecture - Structure du code et dépendances
- Testing - Couverture et qualité des tests
- Code Quality - Style, nommage, bonnes pratiques
⚡ Architecture Séquentielle (Anti Memory-Leak)
CRITIQUE : Les agents sont exécutés UN PAR UN pour éviter les pics mémoire.
┌─────────────────────────────────────────────────────────────────┐
│ ORCHESTRATEUR SÉQUENTIEL │
│ │
│ [1] Architecture → [2] Testing → [3] Code Quality → ... │
│ │
│ Chaque agent : │
│ 1. Émet [PROGRESS:agent:started] │
│ 2. Analyse le code │
│ 3. Émet [PROGRESS:agent:completed] │
│ 4. ATTEND avant de lancer le suivant │
└─────────────────────────────────────────────────────────────────┘
Workflow
Phase 1 : Initialisation
[PHASE:initializing]
[PROGRESS:context:started]
- Récupérer les informations de la MR/PR
- Lister les fichiers modifiés
- Lire la configuration du projet (CLAUDE.md, etc.)
[PROGRESS:context:completed]
Phase 2 : Exécution Séquentielle des Agents
[PHASE:agents-running]
Exécuter les agents UN PAR UN dans l'ordre :
Agent 1 : Architecture
[PROGRESS:architecture:started]
Vérifier :
- Direction des dépendances (pointent vers l'intérieur)
- Séparation des couches (UI, métier, données)
- Pas de dépendances circulaires
- Abstractions correctes
Score : X/10 avec justification
[PROGRESS:architecture:completed]
Agent 2 : Testing
[PROGRESS:testing:started]
Vérifier :
- Nouveau code testé
- Tests significatifs (pas juste de la couverture)
- Nommage correct des tests
- Pas de tests flaky
Score : X/10 avec justification
[PROGRESS:testing:completed]
Agent 3 : Code Quality
[PROGRESS:code-quality:started]
Vérifier :
- Conventions de nommage
- Duplication de code
- Qualité des commentaires
- Organisation des imports
Score : X/10 avec justification
[PROGRESS:code-quality:completed]
Phase 3 : Synthèse
[PHASE:synthesizing]
[PROGRESS:synthesis:started]
Combiner les résultats en rapport final :
# Code Review - MR/PR #[NUMÉRO]
## Synthèse Exécutive
| Agent | Score | Verdict |
|-------|-------|---------|
| Architecture | X/10 | [Verdict court] |
| Testing | X/10 | [Verdict court] |
| Code Quality | X/10 | [Verdict court] |
**Score Global : X/10**
---
## Corrections Bloquantes
### 1. [Titre]
📍 `fichier.ts:42`
**Agent** : [Quel agent a trouvé ça]
**Problème** : [Description]
**Correction** : [Solution]
---
## Corrections Importantes
[Même format]
---
## Points Positifs
| Aspect | Note |
|--------|------|
| [Pattern] | [Observation factuelle] |
---
## Checklist Avant Merge
- [ ] [Bloquant 1]
- [ ] [Bloquant 2]
- [ ] Lancer les tests
[PROGRESS:synthesis:completed]
Phase 4 : Publication
[PHASE:publishing]
Poster le rapport :
[POST_COMMENT:## Code Review - MR/PR #[NUMÉRO]\n\n[Contenu complet]]
[PHASE:completed]
Sortie
À la fin, émettre le marqueur de stats (OBLIGATOIRE) :
[REVIEW_STATS:blocking=X:warnings=X:suggestions=X:score=X]
