Checklist pour une PR touchant aux paiements
Checklists de revue adaptées au domaine — paiements, authentification, infra — pour que les relecteurs ne ratent pas l'évident
Voir l’aperçu de l’entrée et de la sortie
Entrée
- Language
- TypeScript
- Domain
- payments
- Risk
- high
Sortie (extrait)
## Checklist de revue de code — PR paiements (risque élevé) - [ ] Tout l'argent manipulé en centimes entiers (aucune arithmétique flottante nulle part dans le diff). - [ ] Le code de devise accompagne chaque montant (aucun USD implicite). - [ ] Clé d'idempotence exigée à chaque écriture vers la passerelle ; réutilisation vérifiée contre un cache de 24h. - [ ] Les PII (4 derniers chiffres de la carte, adresse de facturation) ne sont jamais journalisées, y compris dans les traces d'erreur. - [ ] Les écritures en base encapsulées dans une transaction, l'appel à la passerelle effectué après le commit. - [ ] Les tests couvrent : succès, challenge 3DS, refus, timeout de la passerelle, remboursement partiel, rejeu en double soumission. - [ ] Feature flag en place ; plan de déploiement documenté dans la description de la PR.