Aller au contenu

PD-84 — Plan d'implémentation : Revue v2

1. Références

  • Spécification : PD-84-specification.md
  • Tests contractuels : PD-84-tests.md
  • Plan d'implémentation : PD-84-plan.md (v1.1.0)
  • Date de revue : 2026-02-24
  • Reviewer : ChatGPT (OpenCode, mode factuel)

2. Constatations (écarts)

Type Référence (Spec/Test/Plan) Description Impact Gravité
Test irréalisable Tests TC-10, TC-LIM-04 / Spec CA-84-10, INV-84-05, INV-84-06 / Plan §2.5, §9.2 Le plan fixe le comportement PREMIUM des endpoints export à 501 Not Implemented (PD-85). Les tests contractuels exigent qu'après upgrade FREE→PREMIUM, l'export soit autorisé sur dossier existant. Incompatibilité directe Plan↔Tests sur un scénario d'acceptation central (upgrade). BLOQUANT
Non-conformité Spec Spec §3.2.2, §9, tests TC-SLA-01 / Plan §2.4 "SLA — Garantie par construction" La spec contractualise la mesure SLA entre réception API PUT /account/plan et enregistrement append-only PD-31. Le plan décrit une mesure interne "début transaction→COMMIT" et conclut qu'aucune observabilité externe n'est requise. Le mécanisme décrit ne couvre pas strictement le contrat de mesure SLA. MAJEUR
Test irréalisable Tests TC-SLA-01 / Plan §2.4 Le protocole de test exige un point d'arrivée append-only PD-31 ; le plan ne décrit pas de point d'observabilité explicite aligné sur ce point (il documente duration_ms transactionnel). Validation contractuelle du SLA non objectivable telle que définie dans le cahier de tests. BLOQUANT
Hypothèse implicite Plan §8 (H-01) + MIN-05 Le plan suppose l'absence de valeur "business" en base pour la migration User.plan vers enum FREE/PREMIUM, sans preuve documentaire. Risque migration si donnée legacy non conforme. MINEUR
Contrainte technique non documentée Plan "Contraintes techniques" Variables CI requises (DATABASE_URL, CI=true) non documentées pour tests d'intégration PostgreSQL. Ambiguïté exécution CI/CD. MINEUR

3. Synthèse

  • Nombre d'écarts par gravité : BLOQUANT=2, MAJEUR=1, MINEUR=2
  • Corrections v1 levées : MAJ-02, MAJ-03, MAJ-04, MAJ-05, MAJ-06 (⅚ levées)
  • Réserve persistante : MAJ-01 (SLA non strictement aligné au contrat Spec+Tests)
  • Points critiques : export PREMIUM 501 vs tests TC-10/TC-LIM-04, mesure SLA append-only PD-31

4. Verdict de la revue

  • Statut : ⛔ Rejeté
  • Motif synthétique : Écarts BLOQUANT sur réalisabilité des tests contractuels (TC-10/TC-LIM-04 et TC-SLA-01), non-conformité majeure sur définition de mesure SLA.