Checklist code review — Kevin Delfour

Lisibilité

0/5

Le code est lisible sans commentaire explicatif

Les noms de variables et fonctions sont explicites

Les fonctions sont courtes et font une seule chose

Le formatting est cohérent avec le reste du codebase

Il n'y a pas de code mort ou commenté

0/5

La responsabilité du code est clairement délimitée

Les dépendances sont justifiées et minimales

Le code respecte les patterns existants du projet

Le couplage est faible, la cohésion est forte

Il n'y a pas de duplication logique évitable

0/5

Les cas nominaux sont testés

Les cas limites sont couverts

Les tests sont lisibles et maintenables

Les tests passent en local

Il n'y a pas de tests flaky ou fragiles

0/5

Les entrées utilisateur sont validées et sanitizées

Il n'y a pas de secret en dur dans le code

Les permissions et accès sont vérifiés

Les requêtes SQL sont paramétrées

Les logs ne contiennent pas de données sensibles

0/4

Il n'y a pas de requête N+1 évidente

Les opérations coûteuses sont optimisées ou cachées

La pagination est en place pour les listes

Les index de base de données sont adaptés

0/4

Le message de commit est clair et descriptif

La description de PR explique le contexte

Les changements de contrat API sont documentés

Le README est mis à jour si nécessaire