Classifier les résultats de la revue
Catégories de sévérité
| Emoji | Catégorie | Critères |
|---|---|---|
| ❌ | CRITICAL | Va casser, planter, exposer des données ou violer les exigences |
| ⚠️ | IMPORTANT | Gestion d'erreur manquante, cas limites non gérés, pourrait causer des bugs |
| ♻️ | DEBT | Duplique des patterns, viole les conventions, nécessite une rework sous 6 mois |
| 🎨 | SUGGESTED | Améliore mesurément la sécurité, réduit la complexité de 3+, élimine des bug classes |
| ❓ | QUESTION | Nécessite une connaissance humaine - exigences floues, intention ou conflits système |
TOUJOURS utiliser le format hybride emoji + texte pour chaque résultat (si plusieurs sévérités s'appliquent, utiliser la plus grave : ❌ > ⚠️ > ♻️ > 🎨 > ❓) :
Avant de classifier
Vérifier LES TROIS points suivants :
- Pouvez-vous tracer le chemin d'exécution montrant le comportement incorrect ?
- Ceci est-il géré ailleurs (error boundaries, middleware, validators) ?
- Êtes-vous certain du comportement du framework et de la sémantique du langage ?
Si l'une des réponses est « non » ou « pas sûr » → NE PAS classifier comme un résultat.
Résultats non valides (Rejeter)
- Éloges (« excellente implémentation »)
- Suggestions vagues (« pourrait être plus simple »)
- Préférences de style sans standard appliqué
- Critiques de noms sauf si activement trompeurs
- Problèmes de métadonnées PR (titre, description, plan de test) - gérés par le summary skill, non classifiés ici
- Mises à jour mineures/patch Renovate/Dependabot de dépendances existantes avec CI qui passe — ce sont des activités Stage 5 de monitoring routinières, non des résultats à reviewer
Critères des améliorations suggérées (🎨)
Suggérer uniquement les améliorations apportant une valeur mesurable :
- Gain de sécurité - Élimine une classe entière de vulnérabilités (SQL injection, XSS, etc.)
- Réduction de complexité - Réduit la complexité cyclomatique de 3+, élimine un niveau d'imbrication
- Prévention de bugs - Rend une catégorie entière de bugs impossible (type safety, null safety)
- Gain de performance - Réduit O(n²) à O(n), élimine les requêtes N+1 (fournir une preuve)
Fournir des métriques concrètes :
- ❌ « Ceci pourrait être plus simple »
- ✅ « Ceci a une complexité cyclomatique de 12 ; extraire la logique de validation la réduirait à 6 »
Si vous ne pouvez pas mesurer l'amélioration, ne la suggérez pas.