Skip to content

feat(table): enquêtes en cours, les indices achetés restent sous les yeux - #92

Merged
Lunik merged 2 commits into
developfrom
claude/game-engagement-retention-lyk0n0-enquetes
Oct 24, 2026
Merged

Lunik merged 2 commits into
developfrom
claude/game-engagement-retention-lyk0n0-enquetes

Conversation

@Lunik

@Lunik Lunik commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

Closes #88. Piste 4 de la page de travail pistes-retention.html (EtymoLogique/Design#51).

Ce qui change

  • enquetesEnCours (web/src/services/indices.ts) :
    • Elle liste les cartes à trouver dont au moins un indice est acheté, par n° croissant. Elle reprend le filtre de ciblesIndice : jamais une carte trouvée ni une légendaire, et un fascicule verrouillé n’a déjà plus de silhouette dans le codex.
    • Elle ne renvoie que les indices obtenus, avec leur contenu lu dans le catalogue.
    • Elle ne dépend pas de la réserve : une enquête reste visible même quand le mot ne se forme plus.
  • Composant Enquetes, sous la table de fusion :
    • Une ligne repliée par enquête (« N° 003 · Silhouette, définition »), avec en aperçu le dernier texte acheté.
    • Dépliée, l’enquête montre la silhouette, la définition et le sens littéral achetés, rien d’autre.
    • Au-delà de 3 enquêtes, un bouton « Voir les N enquêtes ».
    • Aucun bouton d’achat : la feuille des indices reste le seul endroit où acheter.
  • CarteSilhouette passe dans son propre module, partagé avec la feuille des indices.
  • Styles : uniquement les jetons de la palette. check_identity : 0 erreur, 0 avertissement.
  • Roadmap : une ligne ajoutée au jalon J4.

Aucune règle, aucun contrat et aucune table ne changent : la PWA relit Etat.indices.

Écart avec l’issue

L’issue prévoyait aussi les enquêtes en tête de la feuille de réserve. Je ne l’ai pas fait : la modale de réserve est déjà dense (petite table, consigne, filtres). Les enquêtes restent visibles sous la table, juste derrière le voile. C’est à ajouter si vous le souhaitez.

Vérifications

  • npm run lint : seul reste l’avertissement de Plis.tsx, qui existait avant cette PR.
  • npm run typecheck : OK.
  • npm test : 203 tests passent, dont 2 nouveaux tests du service et 3 du composant.
  • Recette locale avec l’API, PostgreSQL et le catalogue d’essai :
    • J’ai acheté les trois indices de la carte n° 003 ; l’enquête apparaît sous la table.
    • À 390 px et à 1440 px : pas de débordement horizontal, aucune erreur dans la console.
Téléphone Dépliée
Table, puis « Enquête en cours » repliée avec l’aperçu du sens littéral Silhouette, définition, sens littéral

À faire dans Design : mettre à jour les maquettes I-09 et I-12, et corriger l’ADR 0051 pour y écrire « un indice acquis reste lisible ».


Generated by Claude Code

Under the fusion table, list every card to find with at least one hint
bought, folded by default; unfolded, it shows only what was paid for
(silhouette, definition, literal meaning) until the word is found.
No word, brick, rarity or buy button is ever shown (ADR 0051).

CarteSilhouette moves to its own module, shared with the hints sheet.

Closes #88

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0151stDRaZV3ScbhwvwjrfQ7
…udits

cargo audit flagged RUSTSEC-2026-0381/0382/0383/0385 and npm audit a high
advisory on vite 8.3.1 (and moderate ones on vitest): lockfile-only
updates to the patched versions.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0151stDRaZV3ScbhwvwjrfQ7
(cherry picked from commit a140f55df0d3ce823acc6ff6d9534bce30e72ab8)

Lunik commented Oct 23, 2026

Copy link
Copy Markdown
Contributor Author

Les jobs « API et tâches (Rust) » et « PWA (TypeScript) » échouent, mais pas à cause de cette PR. Seuls les audits de dépendances échouent, et develop est touché de la même façon :

  • cargo audit remonte RUSTSEC-2026-0381, 0382, 0383 et 0385, sur rustls-webpki, webpki-roots, webpki-root-certs et ctutils ;
  • npm audit remonte une alerte élevée sur vite 8.3.1 et des alertes moyennes sur vitest.

J’ai ajouté à cette branche le commit fix(deps), qui ne change que les fichiers de verrouillage (Cargo.lock, web/package-lock.json). En local, le build, les tests et npm audit passent avec ces versions. Ce commit est le même sur #93, #94 et #95 ; il ne fera plus rien dès que develop contiendra ces mises à jour.


Generated by Claude Code

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