fix(Alert): honour explicit role={undefined} to suppress role attribute

[jrlprost]

Fixes #478.

Problème

Passer role={undefined} sur <Alert /> rendait quand même role="alert" dans le DOM, ce qui est une non-conformité RGAA (critère 8.7 — changement de contexte non demandé par l'utilisateur·rice).

La cause : le default dans le destructuring (role = "alert") remplace silencieusement undefined par "alert", même lorsque undefined est explicitement fourni par l'appelant·e.

Correction

Au lieu d'un default dans le destructuring, on utilise "role" in props pour distinguer les deux cas :

• role non fourni → comportement par défaut conservé : "alert" (compatibilité ascendante intacte)
• role={undefined} explicitement passé → aucun attribut role dans le DOM rendu

// Avant
role = "alert",

// Après
role: roleFromProps,
// ...
const role = "role" in props ? roleFromProps : ("alert" as const);

Le rendu JSX passe de role ? à role !== undefined pour gérer correctement la valeur undefined :

// Avant
{...(role ? { "role": role } : {})}

// Après
{...(role !== undefined ? { "role": role } : {})}

La même correction est appliquée au spread conditionnel refShouldSetRole (ré-annonce par les lecteurs d'écran quand l'alerte se rouvre).

Tests

• La pre-commit hook (ESLint + Prettier) passe sur le fichier modifié.
• Aucun nouveau warning lint introduit (Notice.tsx avait une warning pre-existing).
• Comportement attendu : <Alert role={undefined} ... /> → pas d'attribut role dans le DOM rendu.

[revolunet]

à quoi sert cette ligne, c'est toujours écrasé en-dessous, non ?

[jrlprost]

Vous avuez raison. Le premier est redondant : quand role !== undefined les deux gèrent la même valeur, et quand role === undefined aucun des deux ne s'applique de toute façon. J'ai supprimé ce spread inutile ainsi que la ref refShouldSetRole et son assignation. Le diff est meilleur.