Code Review deux axes (Matt Pocock skill)
Tu invoques /code-review sur le diff depuis un point fixe (un SHA, une branche, un tag, main) et la skill lance deux sous-agents en parallèle : Standards (le code respecte-t-il les conventions documentées du repo ?) et Spec (le code fait-il ce que l'issue d'origine demandait ?). Elle vérifie d'abord que la référence résout et que le diff n'est pas vide, puis rapporte les deux verdicts côte à côte, sans les fusionner ni les reclasser. L'axe Standards embarque en dur une base de douze code smells de Fowler (Refactoring, ch.3), appliquée même quand le repo ne documente rien.
Forces
- Deux sous-agents parallèles avec contextes séparés, donc l'axe Spec ne se laisse pas contaminer par les remarques de style
- Base de douze code smells de Fowler embarquée : la revue tient même sur un repo qui ne documente aucune convention
- Le standard documenté du repo écrase toujours la base, et chaque smell est étiqueté comme jugement, jamais comme violation dure
- Fail-fast sur la référence : mauvaise ref ou diff vide s'arrêtent avant de brûler deux sous-agents
Limites
- L'axe Spec ne sert à rien sans spec : sans issue ni fichier de spec, le sous-agent est sauté et il ne reste qu'une demi-revue
- Suppose le tracker configuré (docs/agents/issue-tracker.md), donc /setup-matt-pocock-skills est un prérequis de fait
- Ne cherche pas les bugs : c'est une revue conventions plus conformité, elle ne remplace pas une passe correctness ni les tests
Pour qui
- Devs qui veulent une revue avant de pousser une branche, sans attendre un humain
- Équipes avec un CONTRIBUTING.md ou un fichier de conventions que personne ne relit jamais au moment du diff
- Repos où l'écart entre ce que le ticket demandait et ce qui a été codé passe régulièrement en prod