@code-reviewer-proLe mentor senior — review approfondie de PR
Maifady est un plugin Claude Code 100% gratuit avec 30 agents IA — @code-reviewer-pro est le mentor senior virtuel que tu invoques avant d'envoyer une PR importante : il lit les fichiers en entier, leurs appelants, et juge le design, la perf, la concurrence, l'observabilité et la dette accumulée.
Le problème qu'il résout
Tu finis une PR qui ajoute la facturation. Tu sais qu'elle est sensible. Tu pourrais l'envoyer à ton collègue, qui mettra 2 jours à répondre et te demandera "as-tu pensé à X ?" — alors qu'à ce moment tu ne sais plus exactement où X se trouve dans ton code. @code-reviewer-pro te donne cette review senior maintenant, pendant que le contexte est encore frais dans ta tête.
Comment il bosse
- Capture le scope de la PR :
git diff <base>...HEAD, plus la description de la PR et les messages de commit. - Lit chaque fichier touché en entier, plus ses appelants directs (Grep) et ses appelés directs.
- Échantillonne 2-3 fichiers voisins du module pour établir les conventions du projet.
- Applique la checklist deep : design (layering, couplage, SRP), perf (N+1, O(n²), allocations), concurrence (race, lost update, deadlock), observabilité (logs, métriques), tests (profondeur, brittle), migrations.
- Classe en P0 / P1 / P2 / Discussion avec math perf quand pertinent et références à des patterns existants dans le repo (
file:line).
Exemple concret
Tu tapes
›@code-reviewer-pro review ma branche feature/billing vs main. Focus sur design et perf, c'est une PR de paiement.
Tu obtiens (extrait)
# Deep review — 12 files, 8 findings ## Blocking (P0) — 2 1. **BillingService.php:88** — Multi-write sans transaction Charge la carte, puis INSERT en orders. Si l'INSERT plante après le charge → carte débitée, commande absente, support à appeler. Fix: wrap dans transaction OU saga avec compensating action. ## Strong (P1) — 3 2. **InvoicePdf.php:42** — Allocation dans hot path string concat dans une boucle de 500 lignes → ~12MB allocations/req. Fix: array push + implode à la fin. ## Discussion - L'event 'invoice.created' est émis APRÈS la transaction. Si un consumer dépend du chargement, il peut le rater. Outbox pattern ?
Pourquoi tu vas l'aimer
Pense en senior
Design, perf, concurrence, observabilité, tests. Pas du linting — du jugement architectural.
Lit les callers
Il ne regarde pas le hunk isolé : il charge les appelants pour comprendre les contrats, et détecte les régressions cachées.
Math perf concrète
"50 users × 1 query = 50 round-trips, ~250ms." Pas du "ça pourrait être plus rapide" vague.
Section Discussion
Pose des questions ouvertes que tu te poserais en pair-programming. Pas juste des verdicts.
Quand l'utiliser
✅ Idéal pour
- PR qui touche au paiement, à l'auth, à la DB schema
- Refactor important d'un module critique
- Avant un release majeur
- Quand @code-reviewer-lite flag des trucs "hors scope"
⚠️ Pas adapté quand
- Diff trivial (changement de copy, fix typo) → utilise @code-reviewer-lite
- Audit sécurité dédié → @security-auditor
- Optim DB pure → @sql-optimizer
Combine avec ces agents
Questions fréquentes
Quelle différence avec @code-reviewer-lite ?
-lite fait ~30 secondes par diff, focalisé sur correctness/sécurité/data. -pro lit les fichiers en entier, leurs callers et callees, et regarde design, perf, concurrence, observabilité, tests, dette. Plus lent, beaucoup plus profond.
Combien de temps prend une review pro ?
De 2 à 10 minutes selon la taille de la PR et le nombre de fichiers à charger en contexte. C'est l'équivalent d'une review humaine senior — vise un usage 1-3 fois par jour, pas par commit.
Identifie-t-il les régressions de performance ?
Oui : N+1 queries, O(n²), allocations hot path, transactions longues, cache stampede. Avec math back-of-envelope quand pertinent.
Lis-t-il les tests ?
Oui. Il évalue la profondeur des tests : assertions tautologiques, mocks couplés à l'implémentation, branches non testées. Il propose des tests manquants ciblés.
Prêt à essayer ?
44 KB · 30 agents · 0 € · MIT.
Télécharger Maifady