name: receiving-code-review description: À utiliser lors de la réception de retours de revue de code, avant de mettre en œuvre les suggestions, particulièrement si le retour semble peu clair ou techniquement discutable - exige de la rigueur technique et de la vérification, pas d'accord performatif ou d'implémentation aveugle
tags: [development, superpowers, receiving-code-review, code-review] ------|-----| | Accord performatif | Stipuler l'exigence ou simplement agir | | Implémentation aveugle | Vérifier d'abord contre la codebase | | Traitement par lot sans tests | Un par un, tester chacun | | Assumer que le relecteur a raison | Vérifier si ça casse quelque chose | | Éviter la contre-argumentation | Correction technique > confort | | Implémentation partielle | Clarifier tous les éléments d'abord | | Impossible à vérifier, procéder quand même | Déclarer la limitation, demander des directives |
Exemples Concrets
Accord Performatif (Mauvais) :
Reviewer: "Remove legacy code"
❌ "You're absolutely right! Let me remove that..."Vérification Technique (Bon) :
Reviewer: "Remove legacy code"
✅ "Checking... build target is 10.15+, this API needs 13+. Need legacy for backward compat. Current impl has wrong bundle ID - fix it or drop pre-13 support?"YAGNI (Bon) :
Reviewer: "Implement proper metrics tracking with database, date filters, CSV export"
✅ "Grepped codebase - nothing calls this endpoint. Remove it (YAGNI)? Or is there usage I'm missing?"Élément Peu Clair (Bon) :
your human partner: "Fix items 1-6"
You understand 1,2,3,6. Unclear on 4,5.
✅ "Understand 1,2,3,6. Need clarification on 4 and 5 before implementing."Réponses dans les Threads GitHub
Lors de la réponse aux commentaires de revue en ligne sur GitHub, répondez dans le fil de commentaires (gh api repos/{owner}/{repo}/pulls/{pr}/comments/{id}/replies), pas comme commentaire de premier niveau sur la pull request.
L'Essentiel
Les retours externes = suggestions à évaluer, pas des ordres à suivre.
Vérifier. Questionner. Puis implémenter.
Pas d'accord performatif. Rigueur technique toujours.