Dev:review

Objectif

Phase 6 du workflow de développement : review qualité complète du code implémenté.

Prérequis

⚠️ Plugin feature-dev requis pour l'agent code-reviewer.

Si non installé :

/plugin install feature-dev@claude-code-plugins

Instructions à Exécuter

IMPORTANT : Exécute ce workflow étape par étape :

1. Vérifier le contexte

• Lis .claude/data/.dev-workflow-state.json avec Read
• Extrais les fichiers modifiés et l'état de la phase 5 (code)
• Si phase 5 non complétée, affiche :

  ❌ La phase d'implémentation (code) n'est pas terminée
  
  Lance d'abord : /dev:code
  

  • Arrête le workflow

2. Créer les tâches de review

• Utilise TaskCreate pour chaque tâche de review :

TaskCreate #1: Code Review - Simplicité/bugs/conventions (feature-dev)
TaskCreate #2: PHPStan - Résoudre erreurs niveau 9
TaskCreate #3: Elegant Objects - Conformité principes
TaskCreate #4: Consolider - Agréger résultats et décider
TaskCreate #5: Examine - Adversarial review (challenge decisions, edge cases)

Important :

• Utiliser activeForm (ex: "Reviewing code quality", "Résolvant erreurs PHPStan")
• Les 3 premières tâches peuvent se lancer en parallèle
• La tâche #4 dépend des 3 premières (utiliser addBlockedBy)

3. Lancer les reviews en parallèle

⚠️ Avant de lancer les agents : Marquer les 3 tâches en in_progress :

• TaskUpdate → tâche #1 en in_progress
• TaskUpdate → tâche #2 en in_progress
• TaskUpdate → tâche #3 en in_progress

Review 1 : Code Review (feature-dev)

Lancer l'agent code-reviewer avec le focus sur :

• Simplicité / DRY / Élégance
• Bugs / Correction fonctionnelle
• Conventions du projet

Quand terminé : TaskUpdate → tâche #1 en completed

Review 2 : PHPStan

Lancer l'agent phpstan-error-resolver (local)

Quand terminé : TaskUpdate → tâche #2 en completed

Review 3 : Elegant Objects

Lancer l'agent elegant-objects-reviewer (local)

Quand terminé : TaskUpdate → tâche #3 en completed

4. Consolider les résultats

🔄 Progression : TaskUpdate → tâche #4 en in_progress

Agréger les résultats des 3 reviews.

Quand terminé : TaskUpdate → tâche #4 en completed

4.5. Examine (Adversarial Review) ⭐ NOUVEAU

🔄 Progression : TaskUpdate → tâche #5 en in_progress

Challenge l'implémentation avec perspective adversariale :

Questions à poser :

• Edge cases : Qu'arrive-t-il si input null/vide/invalide ?
• Limites : Quelle est la taille maximale acceptable ? Le timeout ?
• Sécurité : Injection possible ? Exposition de données sensibles ?
• Performance : N+1 queries ? Boucles infinies possibles ?
• Concurrence : Race conditions ? Deadlocks ?
• Rollback : Comment annuler si ça échoue en prod ?
• Décisions d'architecture : Pourquoi ce pattern ? Alternative meilleure ?

Focus prioritaire :

1. Chercher ce qui pourrait casser en production
2. Identifier hypothèses non validées dans le code
3. Tester mentalement scénarios extrêmes
4. Challenger décisions d'implémentation (pourquoi X au lieu de Y ?)

Output :

🔍 Examine Results:

Edge cases trouvés:
- [cas] : [risque] → [suggestion]

Limites détectées:
- [limite] : [impact] → [mitigation]

Décisions challengées:
- [décision] : [alternative] → [trade-off]

Quand terminé : TaskUpdate → tâche #5 en completed

5. Demander l'action utilisateur

Que souhaites-tu faire ?

1. 🔧 Fix now - Corriger toutes les issues maintenant
2. 📋 Fix later - Noter pour plus tard et continuer
3. ✅ Proceed - Continuer sans corrections (non recommandé)

⚠️ Attendre la décision avant de continuer.

6. Si "Fix now" choisi

• Appliquer les corrections PHPStan en priorité (bloquent la CI)
• Appliquer les corrections Elegant Objects
• Relancer une review pour vérifier

7. Finaliser

Mettre à jour le workflow state

Prochaine étape

✅ Review complétée

Prochaine étape : /dev:summary pour le résumé final

Règles

• Task Management :
  • Créer 5 tâches au démarrage (3 reviews + 1 consolidation + 1 examine)
  • Marquer les 3 reviews en in_progress avant lancement parallèle
  • La tâche de consolidation est bloquée par les 3 reviews (addBlockedBy)
  • La tâche examine est bloquée par la consolidation (addBlockedBy)
  • Utiliser TaskList pour afficher la progression
• PHPStan erreurs = BLOQUANT (font échouer la CI)
• Confiance minimum 80% pour les issues code review
• Respecter le choix utilisateur (ne pas forcer les corrections)