Révision
Utilisez cette compétence pour évaluer le travail qui existe déjà. La révision ne crée jamais le changement qu'elle juge.
Il s'agit du flux de révision canonique. merge-readiness y dirige les tâches, et ai-slop-cleaner est un sous-flux optionnel inclus.
Objectif
Trouver ce qui est réellement incorrect, classé par sévérité, avec suffisamment de détails pour agir.
Flux de travail
- Établir ce qui a changé et ce que ce changement était censé faire.
- Lire le changement par rapport à cet objectif.
- Vérifier d'abord l'exactitude, puis le risque, puis la simplification.
- Vérifier chaque constatation potentielle avant de la signaler.
- Rapporter les constatations des plus sévères aux moins sévères.
Que vérifier
- Exactitude — défauts logiques, cas limites, chemins d'erreur, concurrence
- Risque — limites de sécurité, opérations destructrices, intégrité des données
- Réutilisation — utilitaires ou motifs existants que le changement aurait dû utiliser
- Simplification — code qui pourrait être supprimé ou condensé
- Couverture — comportement livré sans test
Règles
- Canaux séparés : le réviseur ne doit pas être dans le même contexte actif que l'auteur.
- Vérifier avant de signaler. Une constatation plausible mais non reproductible est du bruit.
- Indiquer honnêtement la sévérité ; ne pas gonfler la liste pour paraître minutieux.
- « Aucune constatation » est un résultat valide lorsque le travail est correct.
- Par défaut, le rôle est consultatif — la révision informe, elle ne bloque pas. Les blocages stricts (déploiement, sécurité, opérations destructrices) restent séparés et échouent de manière fermée.
Résultat
- Constatations, de la plus sévère à la moins sévère, chacune avec fichier, ligne et scénario d'échec concret
- Ce qui a été vérifié et trouvé propre
- Tout ce qui n'a pas pu être évalué