fix(animation): make the animation knobs actually reach the solver

[LeadcodeDev]

Round 4, vague A — lot animation, 8 constats vérifiés.

Six des huit sont des boutons morts : un champ que l'auteur pose, que le schéma accepte, que validate déclare bon, et que le moteur ignore. Personne ne reçoit d'erreur ; la vidéo est simplement dépourvue de l'animation demandée. Pour un outil piloté par génération, c'est le pire mode de défaillance qui soit — la boucle de correction ne se referme jamais.

Constat	Sévérité	Mesure réelle en phase rouge
amplitude de float_3d jetée	High	amplitude: 60 demandé, -12 délivré (le défaut). La recette de parallaxe documentée ne produisait aucune parallaxe.
pulse/float/shake/spin ignorent delay/duration	High	Avec delay: 1.0, à t=0,5 s — donc censés être au repos : shake -10, float -10, pulse 1.05, spin 180.
"loop": true inerte	High	À t=2,5 s au-delà du dernier keyframe : opacité 1 (figée) au lieu de ≈0,5 (wrap).
--strict-anim ignore time_scale/time_offset	High	Violation signalée at t=0.13s sur un nœud réglé 5–7 s avant tout échantillon : un instant jamais rendu.
Budget de complétion	Medium	start_at: 1.5, delay: 0, duration: 1.0 dans une scène de 2 s → "animation finishes at 2.50s". Elle finit à 1,0 s.
Composition sum vs last-wins	Medium	Deux keyframes sur la même propriété : combined=120 vs B seul=20 selon qui porte le délai.
Spring : NaN et divergence	Medium	mass=0 → NaN. stiffness=0 → NaN. damping=-20 → NaN à t=5 s. Zéro validation.
Typos dans style.animation[*]	Medium	{"name":"fade_in_up","duratoin":0.6} → errors: [].

Le constat 6 est un choix de sémantique, pas une correction mécanique

Deux animations keyframes sur la même propriété donnaient soit une somme, soit un écrasement, selon qu'un delay était posé — le delay aiguillait l'effet vers l'un de deux seaux résolus séparément puis fusionnés additivement.

Sémantique retenue : le dernier effet déclaré dans style.animation gagne, la règle de cascade CSS, indépendamment du delay. C'est déjà le comportement intra-seau existant, donc le cas majoritaire (delay = 0) ne bouge pas ; seul le cas inter-seaux change, et il était de toute façon accidentel.

Le spring est traité aux deux bouts

Un NaN qui atteint le layout contamine l'arbre entier. Le solveur plafonne donc ses entrées, et validate rejette les configurations invalides en amont. Corriger seulement le solveur aurait laissé passer une configuration qui n'a aucun sens ; corriger seulement la validation aurait laissé render produire des NaN.

Conséquence assumée de deny_unknown_fields

À signaler explicitement, parce que ce n'est pas neutre. deserialize_children (render/scene.rs:534) saute un enfant qu'il ne sait pas parser. Avec deny_unknown_fields sur les neuf structs de config d'effet, une clé mal orthographiée cesse de produire une animation aux valeurs par défaut et se met à retirer le composant, avec un avertissement sur stderr.

C'est le contrat que CssStyle porte depuis toujours — c'est exactement pourquoi margin-top supprime un composant, cf. PR #156. Cette PR étend une politique existante, elle n'en invente pas une. Et validate, obligatoire selon CLAUDE.md, l'attrape en amont.

Si tu préfères l'inverse — détection côté validateur sans durcir serde — c'est faisable, mais ça demande une liste de champs connus maintenue à la main en plus des structs. Je n'ai pas pris cette décision seul : dis-le et je bascule.

Reste ouvert

schema/animation.rs était hors périmètre de l'agent. Les clés à l'intérieur d'un keyframes[*] (types Animation / Keyframe) restent non couvertes. Le patch est trivial (deny_unknown_fields sur les deux structs, vérifié innocent sur les 8 exemples) et revient au lot SERDE de la vague B, à qui ce fichier appartient.

Vérification

cargo test --workspace sur cette branche seule : intégralement vert, 20 cibles.