PD-80 — Revue d'acceptabilité (post-correction)¶

1. Références¶

• Spécification : PD-80-specification.md (v3)
• Tests contractuels : PD-80-tests.md (v3)
• Acceptabilité existante : PD-80-acceptability.md
• Date de revue : 2026-03-12
• Reviewer : Claude (auditeur indépendant — ni auteur spec, ni implémenteur)

2. Suivi des écarts (append-only)¶

[2026-03-12] — Suivi E-01¶

• Statut précédent : MAJEUR
• Statut actuel : RÉSOLU
• Justification factuelle :
• Le catchup delay est désormais mesuré AVANT la mise à jour de lastActivityAt (ligne 131 de seal-reconciliation.service.ts), puis lastActivityAt est mis à jour à la ligne 135.
• L'ordre est : const catchupDelay = (Date.now() - seal.lastActivityAt.getTime()) / 60000; → seal.lastActivityAt = new Date(); → save().
• Le calcul produit un délai réel (et non ~0) permettant la détection de dépassement reconciliation_max_catchup_delay.
• Preuve de vérification :
• Commit 3d2199c : fix(PD-80): reconciliation catchup delay measured before lastActivityAt update
• Fichier : src/modules/urgent-seal/services/seal-reconciliation.service.ts:130-136
• Tests : 135/135 PASS (11 suites, exécution 2026-03-12), incluant seal-reconciliation.service.spec.ts (TC-NR-12, TC-NOM-11, TC-NOM-18)

[2026-03-12] — Suivi E-02¶

• Statut précédent : MINEUR
• Statut actuel : OUVERT
• Justification factuelle :
• Le stub requestTsa() est toujours présent à la ligne 231 de priority-tsa.processor.ts. Il throw immédiatement avec le message 'TSA service not yet integrated (PD-39 stub)'.
• Story destination PD-39 (TSA RFC3161) correctement tracée dans le commentaire.
• Aucune correction attendue dans le périmètre PD-80.
• Preuve de vérification :
• Fichier : src/modules/urgent-seal/processors/priority-tsa.processor.ts:230-236

[2026-03-12] — Suivi E-03¶

• Statut précédent : MINEUR
• Statut actuel : OUVERT
• Justification factuelle :
• Le stub buildMerkleTree() est toujours présent à la ligne 206 de priority-anchor.processor.ts. Story destination PD-54 tracée.
• Aucune correction attendue dans le périmètre PD-80.
• Preuve de vérification :
• Fichier : src/modules/urgent-seal/processors/priority-anchor.processor.ts:205-209

[2026-03-12] — Suivi E-04¶

• Statut précédent : MINEUR
• Statut actuel : OUVERT
• Justification factuelle :
• Le stub anchorBatch() est toujours présent à la ligne 216 de priority-anchor.processor.ts. Story destination PD-55 tracée.
• Aucune correction attendue dans le périmètre PD-80.
• Preuve de vérification :
• Fichier : src/modules/urgent-seal/processors/priority-anchor.processor.ts:215-219

[2026-03-12] — Suivi E-05¶

• Statut précédent : MINEUR
• Statut actuel : OUVERT
• Justification factuelle :
• 5 stubs notifications toujours présents dans seal-notification.service.ts : checkPushDevice() (L129), sendPush() (L139), sendSealedEmail() (L152), sendFailedEmail() (L164), scheduleWebhookDelivery() (L205).
• Stories destinations tracées : PD-105 (push), modules Email et Webhook (non PD).
• Aucune correction attendue dans le périmètre PD-80.
• Preuve de vérification :
• Fichier : src/modules/urgent-seal/services/seal-notification.service.ts:128-214

[2026-03-12] — Suivi E-06¶

• Statut précédent : INFO
• Statut actuel : OUVERT
• Justification factuelle :
• Le pattern .catch() sur notifySealed() est toujours présent à la ligne 191 de priority-anchor.processor.ts.
• Pattern acceptable : la notification est fire-and-forget (non-bloquante pour le flux principal). Le check INV-80-08 (>= 1 canal réussi) est géré en interne par SealNotificationService.notifySealed() qui audite atLeastOneSuccess.
• Aucune modification nécessaire.
• Preuve de vérification :
• Fichier : src/modules/urgent-seal/processors/priority-anchor.processor.ts:191-193

3. Verdict d'acceptabilité (courant)¶

Verdict actuel : ⚠️ ACCEPTÉ AVEC RÉSERVES Date : 2026-03-12 Motif synthétique : Bug E-01 (catchup delay) résolu et prouvé par commit + tests PASS. Les 5 écarts restants (E-02 à E-06) sont des stubs inter-PD ou patterns acceptables, tous MINEUR/INFO avec stories destinations tracées. Réserves structurelles inchangées (R1: coverage 78.88% < 85% dû aux stubs, R2: 8 stubs inter-PD tracés, R3: controller sans tests). Module prêt pour Gate 8.

4. Historique des verdicts¶