Aller au contenu

PD-32 — Plan d'implementation : Revue v2

1. References

  • Specification : PD-32-specification.md (v2)
  • Tests contractuels : PD-32-tests.md (v2)
  • Plan d'implementation : PD-32-plan.md (v2)
  • Code contracts : PD-32-code-contracts.yaml (v2)
  • Date de revue : 2026-02-05
  • Reviewer : ChatGPT

2. Constatations (ecarts)

Type Reference (Spec/Test/Plan/Contract) Description Impact Gravite
Suivi correction v1 DIV-01 / Plan §2.1 (6.1, response), Plan §2.2 (response) Champ plan retire du SELECT et du payload GET/PUT. Ecart v1 corrige. RESOLU
Suivi correction v1 DIV-02 / Plan §5 (note TC-ERR-03/04) Mecanisme de test acces croise ajoute (U1/U2 + controle avant/apres). Ecart v1 corrige sur la documentation de testabilite. RESOLU
Suivi correction v1 DIV-03 / Plan §6.3 Mapping ExceptionFilter rendu contractuel (DOIT) + lien TC-INV-02. Ecart v1 corrige. RESOLU
Suivi correction v1 DIV-04 / Plan §1.3 (RateLimitGuard) Reference config rate limit + precondition test seuil reduit ajoutees. Ecart v1 corrige. RESOLU
Suivi correction v1 DIV-08 / Contracts module user-module Invariant non issu de la spec retire (invariants: []). Ecart v1 corrige. RESOLU
Suivi correction v1 DIV-09 / Contracts module user-profile-controller INV-32-08 corrige avec preferences dans le contrat de sortie. Ecart v1 corrige. RESOLU
Suivi correction v1 (optionnel) DIV-06 / Plan §9.3 Piege ClassSerializerInterceptor ajoute. Correctif optionnel applique. RESOLU
Suivi correction v1 (optionnel) DIV-07 / Contracts en-tete Exclusion AppModule des code contracts documentee en note technique. Correctif optionnel applique. RESOLU
Non-conformite Spec Spec INV-32-06, CA-32-10/11 ; Plan §2.2 etape 3.1 Le flux §2.2 indique encore "whitelist: true rejecte" pour champ protege. La spec et le plan §3/§4 contractualisent forbidNonWhitelisted pour ce rejet explicite. Ambiguite contractuelle sur le mecanisme de rejet ERR-32-PROTECTED-FIELD. MAJEUR

3. Synthese

  • Ecarts v1 corriges : 8/8 (dont 2 optionnels appliques)
  • Nouvel ecart introduit : 0
  • Ecart residuel non resolu : 1 (coherence interne du plan sur le rejet de champs proteges)

4. Verdict de la revue

  • Statut : ⚠️ Accepte avec reserves
  • Motif synthetique : les corrections v1 sont appliquees et coherentes avec spec/tests/contrats ; une incoherence interne persiste dans le flux PUT (§2.2, etape 3.1) sur le mecanisme de rejet des champs proteges.