Skip to content

feat(ricochets): validate pairs automatically once every brick is linked - #81

Merged
Lunik merged 2 commits into
developfrom
claude/ricochets-auto-validation-nyazfc
Oct 16, 2026
Merged

Lunik merged 2 commits into
developfrom
claude/ricochets-auto-validation-nyazfc

Conversation

@Lunik

@Lunik Lunik commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Comme Découper et Assembler (ADR 0068), le ricochet Paires n’a plus d’étape « Valider les paires » : dès que la dernière brique est reliée à un sens, les réponses partent et le verdict s’affiche.

  • Paires (web/src/composants/Ricochets.tsx) : relier envoie les paires quand toutes les briques sont reliées ; bouton et .rico-valider retirés.
  • Une paire refaite (sens repris par une autre brique) laisse une brique seule : rien n’est envoyé avant que toutes soient reliées.
  • Colonnes stables : sur mobile, la colonne des briques est en max-content et le numéro de paire n’apparaissait qu’une fois la paire reliée. Chaque paire élargissait donc la colonne et faisait passer les sens à la ligne. La place du numéro est désormais toujours réservée (.rico-numero.vide, visibility: hidden). Mesuré dans Chromium à 390 px : sur develop, la colonne passe de 94 à 116 puis 122 px ; ici, 122 px du début à la fin.
  • Tests : web/src/composants/Paires.test.tsx (envoi à la dernière paire, dans les deux ordres ; pas d’envoi tant qu’une brique est seule ; place du numéro toujours présente).
  • Roadmap mise à jour.

Lint, typecheck et tests de la PWA passent (197 tests).

🤖 Generated with Claude Code

https://claude.ai/code/session_016qZaPC2oCkVdwhDngVK5hx

Lunik commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

CI rouge : « API et tâches (Rust) » n’est pas causée par cette PR.

  • Les tests Rust passent tous ; l’étape qui échoue est cargo audit, sur quatre avis RustSec parus le 2026-10-14 contre rustls 0.23.45 (RUSTSEC-2026-0346, 0347, 0348, 0349 ; correctif : >= 0.23.46).
  • Cette PR ne touche que web/, docs/roadmap.md et du CSS : ni code Rust, ni Cargo.lock.
  • develop est rouge pour la même raison (CI en échec sur ses deux derniers pushs).
  • Aucune PR de correctif n’existe encore. Correctif proposé, à porter dans une PR à part : cargo update -p rustls --precise 0.23.46 (3 lignes dans Cargo.lock, vérifié en local que la résolution passe). Je ne l’ajoute pas ici pour ne pas élargir cette PR avec un changement de dépendance TLS.

Les contrôles de la PWA (lint, typecheck, tests) de cette PR sont verts en local.


Generated by Claude Code

@Lunik
Lunik marked this pull request as ready for review October 16, 2026 11:08
@Lunik
Lunik merged commit 071b824 into develop Oct 16, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants