Aller au contenu

PD-228 — Acceptabilité


📚 Navigation User Story | Document | | | ---------- | -- | | 📋 [Spécification](PD-228-specification.md) | | | 🛠️ [Plan d'implémentation](PD-228-plan.md) | | | ✅ **Critères d'acceptation** | *(ce document)* | | 📝 [Retour d'expérience](PD-228-rex.md) | | [← Retour à site-vitrine](../PD-225-epic.md) · [↑ Index User Story](index.md)

Objectif

Vérifier que l’implémentation est conforme à la spécification, respecte l’ensemble des invariants ProbatioVault et ne présente aucune incohérence ou oubli critique.


Périmètre de vérification

La revue d’acceptabilité vérifie explicitement :

  • la conformité stricte à la spécification fonctionnelle
  • le respect de tous les invariants applicables
  • la couverture des scénarios de test définis
  • l’absence d’incohérences, oublis ou régressions

Écarts identifiés

Classification des écarts

Niveau Définition
BLOQUANT Violation d’un invariant, faille de sécurité, non-conformité majeure à la spec
MAJEUR Fonction incomplète, comportement non conforme mais sans rupture de sécurité
MINEUR Détail, dette acceptable, amélioration non critique

Détail des écarts

ID Description Référence Gravité
E-01 Couleurs hors tokens : les gradients Hero/CTA utilisent des hex hardcodés (#0d3461) et des valeurs RGBA directes au lieu des tokens de palette, violant l’invariant « couleurs via tokens » et le scénario « utilise exclusivement des tokens ». Spec §Invariants 1 ; Spec §Scénarios de test ; src/components/Hero.astro:49-71 ; src/components/CallToAction.astro:42-129 MAJEUR
E-02 Espacements non tokenisés : de nombreux paddings/marges sont en rem ou pixels (ex. .btn padding: 1rem 1.5rem, .btn-large 1.25rem 2rem, badges 0.75rem 2rem) au lieu des échelles --spacing-*, ce qui rompt la cohérence d’espacement exigée. Spec §Invariants 2 ; Spec §Scénarios de test ; src/styles/design-system.css:221-284 ; src/components/Hero.astro:111-116 ; src/components/CallToAction.astro:120-137 MAJEUR
E-03 Plan non réalisé : la structure CSS modulaire prévue (tokens séparés, base/reset, components) et le lint/stylelint interdisant couleurs/espacements arbitraires sont absents ; seule une feuille unique design-system.css existe, sans garde-fou contre les valeurs hors tokens. Plan §Structure Design System ; Plan §Mapping invariants (tokens / spacing) ; Plan §Gestion des erreurs (stylelint) ; absence de fichiers src/styles/tokens/*.css, base/*.css, components/*.css, .stylelintrc MAJEUR
E-04 Feuille legacy non alignée (public/styles.css) conservée avec une palette et des espacements distincts (variables --primary, --text, etc.) sans tokens, ouvrant un risque d’incohérence si utilisée. Spec §Invariants 1-2 ; fichier public/styles.css MINEUR

Conclusion d’acceptabilité

ACCEPTÉ — Tous les écarts identifiés ont été résolus.


Suivi des écarts (append-only)

[2025-12-20] — Suivi E-01

  • Statut précédent : OUVERT
  • Statut actuel : RÉSOLU
  • Justification factuelle :
  • Couleurs et gradients passés sur les tokens (--color-*) dans les composants (Hero, CTA), absence de valeurs hex directes.
  • Référence vérification :
  • src/styles/design-system.css
  • src/components/Hero.astro, src/components/CallToAction.astro

[2025-12-20] — Suivi E-02

  • Statut précédent : OUVERT
  • Statut actuel : RÉSOLU
  • Justification factuelle :
  • Espacements basés sur var(--spacing-*) dans les composants et feuilles de styles, plus de padding/marges en valeurs libres.
  • Référence vérification :
  • src/styles/design-system.css
  • src/components/Hero.astro, src/components/CallToAction.astro, src/components/Header.astro, src/components/NewsletterCTA.astro

[2025-12-20] — Suivi E-03

  • Statut précédent : OUVERT
  • Statut actuel : RÉSOLU
  • Justification factuelle :
  • Structure modulaire en place : tokens (src/styles/tokens/*.css), base/reset (src/styles/base/reset.css), composants (src/styles/components/*.css) importés via design-system.css.
  • Lint strict : color-no-hex et declaration-property-value-allowed-list imposent l’usage des tokens couleur/espacement (testé avec npx stylelint sur un padding arbitraire).
  • Référence vérification :
  • src/styles/design-system.css, src/styles/tokens/*.css, src/styles/base/reset.css, src/styles/components/*.css, .stylelintrc.json

[2025-12-20] — Suivi E-04

  • Statut précédent : OUVERT
  • Statut actuel : RÉSOLU
  • Justification factuelle :
  • Suppression de la feuille legacy public/styles.css, évitant une palette/espacement parallèle.
  • Référence vérification :
  • Absence de public/styles.css

Verdict d’acceptabilité (courant)

  • ✅ ACCEPTÉ
  • Date : 2025-12-20
  • Motif synthétique : E-01 à E-04 résolus (tokens couleur/espacement, structure + lint strict, suppression legacy).

Historique des verdicts

Date Verdict Version Commentaire
2025-12-18 ⚠️ ACCEPTÉ AVEC RÉSERVES n/a Verdict initial avec E-01 à E-04 ouverts
2025-12-20 ✅ ACCEPTÉ n/a E-01 à E-04 résolus (tokens, structure+lint, legacy supprimé)