Que fait expert-en-revue-de-code ?
Revue de code experte des modifications Git actuelles avec le regard d'un ingénieur senior. Détecte les violations SOLID, les risques de sécurité et propose des améliorations actionnables.
Vue d'ensemble
Cette compétence fournit une revue de code automatisée mais approfondie des changements Git non commités ou du dernier commit. Elle agit comme un ingénieur senior en révision de code, identifiant les problèmes de conception, les vulnérabilités de sécurité et les goulots d'étranglement de performance, tout en suggérant des améliorations concrètes.
Utilisation
- Assurez-vous que vous êtes dans un dépôt Git avec des modifications non suivies ou non commitées.
- Invoquez la compétence via l'outil CLI de votre agent :
# Réviser les modifications non commitées
code-review-expert
Ce qu'elle examine
- Principes SOLID : Violations de la responsabilité unique, de l'interface ségrégée, de l'inversion de dépendance, etc.
- Risques de sécurité : Injections SQL, XSS, fuites de données sensibles, validation incorrecte.
- Performance : Requêtes N+1, calculs redondants, utilisation inefficace des structures de données.
- Style et maintenabilité : Incohérences de nommage, complexité cyclomatique élevée, code mort.
- Tests : Manque de tests pour les nouveaux chemins de code, couverture insuffisante.
Format du rapport
Le rapport de revue est structuré comme suit :
- Résumé exécutif : Vue d'ensemble de l'état des modifications (qualité générale, principaux problèmes).
- Problèmes détectés : Liste de chaque problème avec sa sévérité (
critique,élevé,moyen,faible), le fichier et la ligne concernés, et une description. - Suggestions d'amélioration : Pour chaque problème, un extrait de code montrant comment le corriger.
- Forces : Commentaires positifs sur les bonnes pratiques observées dans les modifications.
Exemple
## Résumé exécutif
Les modifications sont globalement solides, mais introduisent une vulnérabilité de sécurité dans le module d'authentification et violent le principe de responsabilité unique dans le contrôleur.
## Problèmes détectés
### Critique
- **Fichier** : `src/auth/login.ts`, ligne 42
- **Description** : Le mot de passe est enregistré en texte clair dans les logs. Utilisez une fonction de hachage avant toute journalisation.
### Élevé
- **Fichier** : `src/controllers/user.ts`, ligne 15
- **Description** : La classe `UserController` gère à la fois la validation et la persistance des données. Séparez ces responsabilités.
...
Dépannage
- Aucune modification détectée : Vérifiez que vous avez des modifications non commitées ou exécutez sur le dernier commit.
- Erreur "Not a git repository" : Assurez-vous d'exécuter la compétence dans un dépôt Git valide.
Compatibilité
- Agents : Claude Code, Cursor, Codex, Windsurf
- Environnement : N'importe quel terminal avec Git installé
Généré avec ❤️ par l'équipe de développement