43 Commits

Include renames

Author	SHA1	Message	Date
williamandClaude Sonnet 5	4db1269348	Audit complet de mise en forme — Liste à puces/numérotée (4e élément) ... Implémente toutes les options manquantes identifiées pour les listes : typographie complète, style/position/couleur/taille de puce (validés selon le kind), puce personnalisée en SVG pour les listes à puces, padding uniforme par élément (nouveau, il n'y en avait aucun), espacement entre éléments réglable, et tous les attributs de boîte partagés sur la liste entière. Bordure/fond/padding par élément individuel et sous-listes imbriquées volontairement différés (portée actée avec l'utilisateur avant implémentation : transformeraient le stockage des éléments en objets structurés, chantier bien plus lourd). Trois ajouts transversaux bénéficiant à plusieurs éléments : sections "Contenu"/"Conteneur" dans tous les panneaux de propriétés, alignement vertical du contenu dans son bloc (Titre/Paragraphe/Liste/Image légendée), et une option pour retirer un thème appliqué ("Aucun modèle" dans la modale, avec une nouvelle fonction db.remove_document_theme). Quatre bugs réels trouvés et corrigés en chaîne pendant la validation avec le thème "Sécurité incendie" : un badge de thème s'affichait au-dessus du texte au lieu d'à côté ; le correctif a d'abord fait disparaître les puces/numéros natifs de TOUTES les listes (bug plus grave que celui corrigé) ; puis un marqueur natif redondant apparaissait à côté du badge du thème ; puis une règle CSS site-large de spécificité supérieure empêchait silencieusement ce dernier correctif. Chaque étape vérifiée par navigateur automatisé sur un support jetable. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-26 15:13:14 +02:00
williamandClaude Sonnet 5	9ad50c58b8	Audit complet de mise en forme — Image (2e élément du tableau) ... Implémente toutes les options manquantes identifiées pour l'élément Image : dimensionnement/ratio/object-fit, filtres CSS, upload de fichier (en plus de l'URL), lien/plein écran au clic, chargement différé, légende, et tous les attributs de boîte partagés déjà créés pour Titre/Paragraphe (padding/margin/fond/bordure/ombre/opacité/ position du bloc). Système de pages : un support peut désormais avoir 0 page (un nouveau support démarre vide), suppression de toutes les pages en un clic, et la pagination automatique insère intelligemment la nouvelle page juste après celle qui déborde plutôt qu'en toute fin de liste. Bugs réels trouvés et corrigés en cours de route : le style de bloc (dont align-self) ciblait l'élément interne au lieu de son enveloppe (légende/lien) ; une image à sa taille native pouvait déclencher une pagination infinie ; upload/mise à jour d'attribut ne déclenchaient jamais le contrôle de débordement. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-26 12:06:44 +02:00
williamandClaude Sonnet 5	746feb3796	Ajoute l'alignement vertical du contenu d'une page, réglable depuis l'onglet "Pages" ... Nouvelle colonne _document_pages.vertical_align (top/center/bottom, "top" par défaut, migration incluse pour les supports existants). Quand l'onglet "Pages" du panneau gauche est actif, le panneau Propriétés (droite) affiche maintenant l'alignement de la page active au lieu des propriétés d'un élément — un contrôle segmenté qui persiste via une nouvelle route dédiée et met à jour le canevas immédiatement. Le contenu-seed des thèmes porte désormais aussi ce réglage par page (seed_pages devient une liste de {vertical_align, blocks} plutôt qu'une liste de listes de blocs) : la page de titre du thème "Sécurité Incendie" est centrée verticalement, comme demandé, cohérente avec la maquette d'origine. L'aperçu de thème (iframe de la modale) reflète aussi ce réglage par page. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-24 10:24:48 +02:00
williamandClaude Sonnet 5	864ae697fd	Ajoute le système de modèles/thèmes de document ("Utiliser un modèle") ... Nouveau bouton dans le topbar de l'éditeur, à côté d'Aperçu, qui ouvre une modale listant les thèmes du catalogue (document_engine/themes/). Cliquer un thème charge un VRAI aperçu (rendu serveur réel dans un iframe, jamais une resucée CSS côté client) avec le choix de garder le contenu actuel ou de le remplacer par le contenu de démonstration du modèle. Architecture pensée pour une centaine de thèmes futurs : chaque thème est une feuille de style externe (static/document/themes/<id>.css) qui habille les classes fixes du moteur, jamais du code qui en changerait la structure. Premier thème implémenté pour valider le mécanisme : "Sécurité Incendie" (6 pages de contenu réel, quiz inclus). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-24 09:12:57 +02:00
williamandClaude Sonnet 5	a34bcf4159	Ajoute 4 mécanismes moteur manquants pour le thème sécurité incendie : étiquette, carte, image SVG inline, bouton avec pièce jointe ... Contenu et mécanisme uniquement, aucun style ajouté (voir consigne du 24/09/2026) : deux nouveaux kinds de contenu (badge/carte, rendu en div brutes sans CSS), un mode SVG inline pour l'image (svg_markup, nettoyé par un nouveau sanitizer allow-list avant chaque rendu) et un fichier téléchargeable joignable à un bouton (upload/download routes, stockage sous db.support_dir). Le futur système de templates portera l'habillage visuel de ces éléments. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-24 08:12:29 +02:00
williamandClaude Sonnet 5	ecd483f352	Implemente un systeme de pages pour le support de formation ... Retour utilisateur : "il faut implementer un systeme de page". Un support est desormais compose de PLUSIEURS pages (_document_pages), chacune un document independant affiche seul sur le canevas -- chaque element appartient a exactement une page via page_id (document_engine/ elements/*, routes/document/document_element_add.py/document_render.py revalident desormais un page_id explicite). Migration automatique et silencieuse pour les supports crees avant cette fonctionnalite (db/supports/ensure_document_pages_schema.py, meme convention que les ensure_X_schema.py existants) : leurs elements deviennent tous les enfants d'une "Page 1" creee a la volee, aucune perte de contenu. Nouveau paquet document_engine/pages/ (add/list/get/rename/move/delete) et 4 routes dediees (routes/document/document_page_*.py) -- supprimer la DERNIERE page restante est refuse (garde-fou pose a la route, meme decoupage que routes/game/screens/screen_delete.py cote jeu, jamais dans la fonction bas niveau). Cote editeur : une bande d'ONGLETS au-dessus du canevas (jamais un panneau lateral, choix explicite de l'utilisateur) -- clic pour changer de page, double-clic pour renommer (contenteditable), glisser pour reordonner, "+" pour ajouter, "x" pour supprimer. Changer de page vide la pile Annuler/Retablir (une commande empilee sur une autre page n'a plus de sens). Mode Apercu : navigation Page precedente/suivante avec indicateur "Page X / N" (choix explicite : page par page, pas de defilement continu), jamais affichee s'il n'y a qu'une seule page. Verifie : suite pytest complete (702 tests, dont 14 nouveaux pour les routes de pages), simulation DOM reelle (jsdom, 25 assertions couvrant tout le cycle de vie cote client -- creation/bascule/renommage/ reordonnancement/suppression de page, portee correcte des elements par page, pile Annuler/Retablir videe au changement de page, pilule de navigation en Apercu), et un test de fumee HTTP reel contre le serveur de dev en marche (creation/ajout d'element/rendu/renommage/suppression d'une page, refus de supprimer la derniere page, page inconnue -> 404). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-21 13:24:23 +02:00
williamandClaude Sonnet 5	7ca03380b7	Retire l'action Publier du support de formation ... Un apprenant n'a jamais acces a l'editeur : un etat "publie" persiste en base (published_at) ne servait donc a rien tant qu'aucune vue apprenant/export ne le consomme (retour utilisateur direct). Retrait complet : route /document/<slug>/publish, db.mark_support_published, la colonne logique published_at dans support_meta, le bouton et son CSS/JS. "Aperçu" devient l'action primaire du bandeau (repond au vrai besoin : voir le rendu avant un futur export). Un export reel (SCORM ou equivalent) reste a specifier separement le jour venu. SKIP=djlint : meme backlog H021 pre-existant que le commit precedent, aucun fichier touche ici n'y figure. ruff/mypy --strict/vulture/ bandit/import-linter/eslint/stylelint tous verts ; 615 tests Python (617 - 2 tests du Publier retire) + verification manuelle live du retrait (route 404, bouton absent du HTML, Apercu confirme fonctionnel par simulation DOM). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-20 09:58:53 +02:00
williamandClaude Sonnet 5	bab581737a	Ajoute le support de formation : entite racine separee du jeu 2D ... Nouveau moteur document_engine/ (elements CRUD + rendering + labels), db/supports/ (stockage independant de db/games), routes/document/ (CRUD AJAX + publication), et l'editeur frontend complet (templates/document/, static/document/) avec moteur de layout reel (glisser-deposer -> fusion en rangee ou insertion avant/apres), vrai Undo/Redo par pile de commandes, grille d'accroche pour les formes libres, apercu responsive a largeurs fixes, mode Apercu, et publication persistee. "Mes formations" (templates/index.html) liste desormais les environnements 2D et les supports de formation cote a cote ; l'onboarding et core/auth_guard.py sont generalises pour qu'un compte restreint puisse posseder un projet de chaque type independamment. SKIP=djlint : le hook ne signale que le backlog H021 (styles en ligne) deja documente dans CODE_QUALITY.md sur des fichiers pre-existants non touches ici (base.html, game/play.html, scene_edit.html, game_dashboard_simple.html, clause_row.html) plus une ligne de index.html deja presente avant cette session — aucun nouveau fichier (document_edit.html compris) n'y figure. Tous les autres outils (ruff, mypy --strict, vulture, bandit, import-linter, eslint, stylelint) passent sans erreur ; 617 tests Python + 276 tests JS verts, plus une verification manuelle complete du cycle de vie via le serveur de developpement. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-20 09:20:33 +02:00
williamandClaude Sonnet 5	b2e933f322	Reorganisation game/document : renommage screens->game_engine + sous-dossiers game/ dans routes, scripts, static, templates, tests ... Prepare la scission a venir entre l'editeur Jeu 2D et le futur editeur Support de formation (voir docs/plan/PLAN.md), sans toucher a l'architecture en couches existante : - screens/ renomme en game_engine/ (nom clair pour le moteur du jeu 2D, avant l'arrivee d'un second "moteur" cote document) : ~85 imports corriges, contrat import-linter mis a jour, meme forme de couches. - routes/, scripts/, static/, templates/, tests/ : tout ce qui est propre au jeu 2D deplace dans un sous-dossier game/ de chacun (routes/game/, static/game/, templates/game/, tests/game/, scripts/game/) ; ce qui est partage par le site (auth, onboarding, dashboard, uploads, db/) reste a la racine de chaque dossier. Un sous-dossier document/ (vide) cree dans chacun pour le futur chantier. - styles/ volontairement inchange : les 3 fichiers sources sont concatenes en un seul static/style.css charge par tout le site, scinder leur CONTENU (editeur vs partage) serait un refactor CSS distinct, pas un deplacement mecanique. - Chaine d'export SCORM (publish/build_scorm_package.py) mise a jour en profondeur : copie des assets, URLs d'icones relatives a static/style.css (qui ne bouge pas), manifeste, wrapper SCORM. - Deux regressions d'un sweep de renommage anterieur corrigees au passage (screens.js/screens/scene-objects incorrectement convertis en game_engine.js/game_engine/scene-objects dans des commentaires). - Effet de bord Windows decouvert et corrige : git mv + Path.write_text convertissent des fichiers en CRLF (core.autocrlf=true) - ~189 fichiers normalises en LF. - .eslintrc.json/package.json : uniquement les chemins de glob mis a jour (static/game/js/...) ; la preparation eslint-plugin-unicorn du lot 7 reste volontairement non committee (package-lock.json restaure a la version precedente). Verifications : ruff, mypy --strict (391 fichiers), vulture, bandit, lint-imports tous verts ; 591/591 tests Python, 276/276 tests JS ; demarrage serveur + requetes HTTP manuelles confirmant que les assets deplaces repondent en 200 au nouvel emplacement et 404 a l'ancien. SKIP=djlint : backlog H021 (styles inline) deja documente comme dette assumee dans CODE_QUALITY.md section 6, aucun template touche par ce commit au-dela d'un deplacement de fichier. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-19 12:27:53 +02:00
williamandClaude Sonnet 5	55e81c0fbb	Ajoute CODE_QUALITY.md et corrige S7632 a la racine (helper assert_not_none) ... - Nouveau db/assert_not_none.py : centralise l'unique suppression Bandit/Ruff (# nosec B101 / # noqa: S101) de narrowing de type dans tout le moteur. Remplace 11 sites disperses (ai/chat.py, ai/tools.py, routes/scenes/scene_object_{add,collision,geometry,personnage_data, quiz_config}.py, screens/payload/full_game_payload.py, scripts/build_demo_dialogues.py) qui repetaient chacun le meme commentaire empile - Sonar (python:S7632) ne parse pas deux commentaires # sur une ligne, meme si Ruff et Bandit les acceptent chacun tres bien (contrainte structurelle documentee dans CODE_QUALITY.md : chaque outil exige son propre mot-cle immediatement apres un #, aucun format a un seul # ne peut satisfaire les deux a la fois). Les 6 sites # nosec B608 (SQL dynamique) restent inchanges, nature differente, hors perimetre de ce refactor. Verifie : mypy --strict propre (389 fichiers), ruff/bandit/import-linter clean, suite complete verte (591 tests), scan SonarQube local relance confirmant S7632 a 7 (1 seul site restant dans le helper lui-meme + les 6 B608), 0 bug (une regression S8371 trouvee et corrigee en route). - 4 sites |safe repositionnes sur leur ligne exacte (scene_edit.html, register_2fa.html, onboarding_new.html, play.html) - le marqueur NOSONAR etait sur la ligne precedente par erreur (meme piege que celui documente pour S8371 ci-dessus). Confirme par scan que meme corrige, l'analyseur Web de Sonar ne supporte aucune syntaxe de suppression inline testee pour la regle Web:S5247 - documente comme limitation technique connue dans CODE_QUALITY.md plutot que force. - CODE_QUALITY.md (nouveau) : reference complete du dispositif qualite - vue d'ensemble par outil, configuration de chacun, commandes de lancement local, procedure de justification d'une exception (avec le format exact attendu par Bandit/Ruff/Sonar, verifie empiriquement), table des 10 exceptions documentees, backlog (djLint H021, code smells Sonar). djLint (H021, styles inline) volontairement saute pour ce commit - meme backlog assume que les commits precedents, aucun rapport avec ce changement. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-16 12:41:02 +02:00
williamandClaude Sonnet 5	c57420c8c9	Phase 3 : hardening qualite de code - typage strict, securite, dead code, a11y ... Config strictement stricte partout (ruff, mypy --strict, bandit, vulture, import-linter, eslint, stylelint), aucune regle desactivee "pour ne pas casser le build" - l'existant a ete corrige pour la satisfaire plutot que l'inverse. Hooks pre-commit locaux (language: system) bloquants. - Typage mypy --strict propage a tout le moteur (db, screens, auth, core, ai, routes, puis publish/scripts/tests/app.py/build_css.py). - Securite : fuite de handle fichier Windows corrigee dans l'export SCORM (routes/publish/export_scorm.py), CSRF/RNG non-crypto/xAPI documentes (# nosec, # NOSONAR justifies), nouveau db.json_for_script() (echappe "</script>" dans le JSON embarque en <script>, 25 sites). - Architecture : imports circulaires/F811 nettoyes, contrats import-linter respectes, code mort retire (vulture). - Accessibilite : 69 champs de formulaire sans label correctement associe corriges (for/id ou aria-label) sur 11 templates. - ESLint/Stylelint : lot mecanique JS/CSS, regles ajustees puis appliquees (aucune desactivee sans verification individuelle). - Tests : isolation du compte admin partage (nettoyage ponctuel + fixture de teardown automatique en filet de securite), suite complete verte (591 tests Python, 241 tests JS). - SonarQube Community Build self-heberge (Docker + PostgreSQL) : rapport complet analyse point par point, faux positifs documentes. - .gitattributes ajoute (LF force) : core.autocrlf=true sur cette machine faisait echouer ESLint (linebreak-style) via un bug connu de git (checkout "en place" qui ignore l'eol force sur un fichier deja present sur disque - contourne en supprimant puis recreant chaque fichier suivi). djLint (H021, styles inline) volontairement saute pour ce commit - backlog assume, deja documente, traite dans un lot separe. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-15 16:06:15 +02:00
williamandClaude Sonnet 5	559331f9cf	Enrichit les declencheurs/actions de scene (clic/survol/affichage, surbrillance/video/son/visibilite/indication/attendre) et fiabilise la pose d'un fond/decor importe ... - Ajoute clic/survol/affichage-ecran comme declencheurs, et surbrillance, video, son, visibilite, indication, attendre comme actions, utilisables aussi bien par l'editeur manuel (menu lateral Objets/Ecran) que par Ruby (IA), avec blocs deplacables/supprimables dans une chaine. - Corrige plusieurs variantes du bug "impossible de poser un objet hors du champ de la camera" (troncature du chainage d'actions a 4 maillons, fond importe pose a 128x128 au lieu de sa taille reelle, decalage du fond au vrai glisser-depose, redimensionnement manuel jamais propage au monde). - Ajoute un vrai glisser-depose depuis la galerie vers la scene, la gestion complete de "Mes assets" (sous-sections Fonds/Decors/Sons/ Videos, suppression, reclassement fond<->decor sans re-upload). - Ajoute l'upload de son (limite 3 min) et de video (MP4 uniquement, limite 5 min), avec validation de la duree reelle du fichier, et une replique audio optionnelle dans une bulle de dialogue. - Fixe la taille de pose d'un objet/decor importe a 200x200 avec une boite de collision de 150x150. - Filtre le selecteur de fichier des actions son/video par type reel. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-12 00:28:46 +02:00
williamandClaude Sonnet 5	b07b231a61	Ajoute l'assistant IA "Ruby" (Claude + Scenario) et corrige plusieurs bugs de la scène ... Intègre un chat IA capable de manipuler la scène via les mêmes fonctions que l'éditeur manuel (objets, variables, déclencheurs, images générées), avec conversations multiples par écran façon Claude. Corrige au passage le rafraîchissement pjax hors-ordre, l'onglet IA/déclencheurs vide après sélection d'un objet, la comparaison de booléens dans les conditions, et le blocage du glisser-déposer hors du cadre caméra après un redimensionnement de fond par l'IA. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-09 19:40:31 +02:00
williamandClaude Sonnet 5	d664ed5637	Remplace le système de quêtes par des déclencheurs, ajoute l'action variable et le chaînage ... Supprime le concept de "quête" au profit d'un onglet unique "Déclencheurs" portant toute la logique (dialogue, condition, marquage terminé) directement sur l'objet de scène. Ajoute une nouvelle action "Modifier une variable" (réutilisant le vocabulaire du graphe de flow) utilisable après une collision, une interaction ou une branche de condition, ainsi qu'un chaînage d'actions ("then") permettant d'enchaîner plusieurs actions à la suite et d'étendre un déclencheur déjà posé sans le recréer. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-07 22:05:06 +02:00
william	c504ace167	Enrichit le reporting SCORM/xAPI et met en conformité RGAA le player ... SCORM/xAPI : - Ajoute l'export SCORM 2004 (3rd/4th edition) au choix, en plus du 1.2 par défaut : sépare completion_status/success_status (un échec reste "completed" au lieu de retomber à tort en "incomplete" comme force la 1.2), remonte aussi cmi.interactions.n.* par question répondue. - Fournit cmi.core.score.min/max (1.2 et 2004), calculé depuis les récompenses de quiz du jeu, pour que le LMS affiche un vrai pourcentage au lieu du score brut à tort étiqueté "%". - Libellés de verbes xAPI en français en plus de l'anglais. Accessibilité (RGAA/WCAG 2.1 AA) sur le player : - Navigation clavier des objets de scène "au clic"/"au survol" (tabindex, role, Entrée/Espace, focus/blur). - alt sur les images (nom auteur ou décoratif), aria-hidden sur les icônes seules, role="dialog"/aria-live sur les boîtes de dialogue/quiz. - Landmark <main> + titre de page, respect de prefers-reduced-motion. - Le quiz n'avance plus automatiquement après un délai fixe : bouton "Continuer →" explicite (RGAA 2.2.1). - Avertissement de contraste dans l'éditeur de style de dialogue. - Déclaration d'accessibilité téléchargeable depuis la modale d'export.	2026-09-06 00:17:51 +02:00
william	f7a50e5afe	Ajoute un suivi xAPI optionnel (bolt-on) au paquet SCORM exporté ... Greffe l'envoi de statements xAPI vers un LRS configurable par le créateur du jeu, en parallèle du reporting SCORM existant : réglages stockés en base (_meta), formulaire dans la modale d'export, injection dans le paquet exporté. Relié aussi bien à l'action de flow "Modifier un score/statut" qu'au parcours quête/quiz (qui alimentait déjà le SCORM classique via un chemin séparé). L'export ne se lance plus automatiquement à l'ouverture de la modale, pour laisser le temps d'enregistrer les réglages xAPI avant de générer le paquet.	2026-09-05 10:00:57 +02:00
william	7ebc9b143f	Corrige une faille d'isolation entre comptes et retire l'email des chemins de projet ... Le rôle "admin" contournait entièrement l'isolation par projet (core/auth_guard.py) : il pouvait ouvrir/modifier/supprimer le jeu de n'importe quel autre compte en connaissant son slug, et la page d'accueil listait sans filtrage tous les projets de tous les comptes. - La propriété d'un projet se vérifie désormais sur le segment "propriétaire" du slug (id du compte), pour tous les rôles y compris admin — un slug "à plat" (sans compte associé) reste réservé à l'admin, comportement historique conservé pour ce cas précis. - routes/games/index.py ne liste plus que les projets du compte connecté. - Le dossier propriétaire d'un projet est maintenant l'id numérique du compte, plus jamais son email slugifié (visible en clair dans chaque URL auparavant) — script de migration fourni et déjà exécuté sur les données existantes. - Changer d'email ne renomme plus aucun dossier (n'en dépend plus). - Deux nouveaux tests de régression, fixtures corrigées en conséquence. - README réécrit pour refléter l'état actuel du produit (jeu 2D uniquement, plus de traces de l'ancien éditeur "document").	2026-09-04 22:52:05 +02:00
william	50835a18e2	Supprime le type d'écran "document" et recentre le produit sur le jeu 2D ... Le document de cadrage produit cible des formateurs non techniques créant des serious games/quiz gamifiés — l'éditeur générique "document" (blocs de logique en nœuds, timeline d'animation, définitions d'objets/relations, templates réutilisables) est une complexité hors cible que l'effort d'ingénierie récent avait déjà abandonnée au profit du jeu_2d. - Onboarding : ne garde que le parcours "RPG" (jeu_2d), retire Quiz/Embranchement/Créer mon jeu de A à Z (tous document-only) - Suppression en bloc des modules exclusifs au document : routes/elements, routes/element_types, routes/objects, routes/legacy_actions, screens/elements, screens/element_types, screens/widgets, screens/legacy_actions, le rendu render_element_html.py et son cluster, templates/screen_edit.html, templates/game_dashboard.html, flow-editor.js/tabs-and-blocks.js/animation-timeline.js - Dashboard toujours simplifié (un seul mode possible désormais) - Tests document-only supprimés, tests de logique partagée (flow, événements personnalisés, animations) retargetés sur des écrans jeu_2d - Aucune régression jeu_2d : 299 tests passent Carte d'onboarding retravaillée : argumentaire RH non technique (liste à coche, badge "Compatible LMS"), taille et interaction de retournement ajustées.	2026-09-04 20:56:32 +02:00
williamandClaude Sonnet 5	b5546df044	Quiz jouable : budget de récompense, boîte à quiz, widget score ... - Récompense d'une quête = budget MAXIMUM pour ses questions : la somme des points des "❓ Question" ne peut plus dépasser recompense_score (routes/quests/quest_dialogues.py à l'enregistrement des dialogues, quest_update.py si on abaisse la récompense sous ce qui est déjà réparti) — message d'erreur explicite (400) dans les deux sens, affiché via alert() côté éditeur (quest-editor.js), jamais enregistré silencieusement dans un état incohérent. - Nombre de choix par question plafonné à 4 (au lieu de 8). - Deux nouveaux widgets "🖥️ Interface" (mêmes fondations que "💬 Boîte de dialogue" — screens/rendering/dialogue_box_style.py, même panneau "🎨 Style") : - "❓ Boîte à quiz" : affiche la question et ses choix, sans pied (une question se résout au clic sur un choix, pas de "Suivant"). - "🏆 Score" : affiche en continu les points gagnés, TOUJOURS visible une fois posé (contrairement aux boîtes de dialogue/quiz, masquées par défaut). - Moteur d'exécution (static/js/play/dialogue-box-controller.js, refonte) : une conversation de quête alterne maintenant répliques (boîte de dialogue) et questions (boîte à quiz) selon le type de chaque ligne. Bonne réponse -> crédite le score (compteur runtime dédié, jamais lié au score du moteur document) et avance ; mauvaise réponse -> rien, la question reste affichée pour réessayer ; aucune boîte à quiz posée -> la question est ignorée plutôt que de bloquer la conversation. Le dialogue "en_cours" épuisé (donc, s'il contenait des questions, toutes répondues) fait passer la quête "terminee" (en mémoire seulement, comme le reste de ce système — voir accepter/ refuser une offre de quête) : le joueur peut alors quitter la scène normalement, ces widgets n'étant jamais des obstacles. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-03 15:42:42 +02:00
williamandClaude Sonnet 5	e972acf9e6	Quêtes : bulle "❓ Question" à choix multiples, à côté de "+ Réplique" ... Nouveau type de ligne dans un dialogue de quête (voir db/quests/sanitize_quest_dialogues.py) : une bulle JAUNE, dans la même chaîne verticale que les répliques (même alignement, reliée par un trait) — header : nombre de choix (2 à 8) + récompense (score, montant en points) ; body : la question, ses choix et un bouton radio pour désigner la bonne réponse. Aucune limite au nombre de questions par colonne, comme pour une réplique. Les lignes de dialogue portent maintenant explicitement {"type": "dialogue", ...} (au lieu d'un objet sans type) pour distinguer les deux formes — migration de forme, tests mis à jour. Portée de ce commit : l'ÉDITEUR (créer/modifier une question). Le jeu lui-même (afficher la question au joueur, vérifier la réponse, attribuer la récompense) reste à construire — le widget "💬 Boîte de dialogue" ne sait aujourd'hui afficher qu'une réplique. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-03 14:48:14 +02:00
williamandClaude Sonnet 5	521792fe00	Dialogues de quête : bulles illimitées, n'importe quel objet, timeline reliée ... - "qui parle" n'est plus réservé au personnage : N'IMPORTE QUEL objet de scène nommé (personnage, décor, fond — voir "ℹ️ Informations", screens/rendering/scene_object_names.py, ex-personnage_names.py) peut parler dans un dialogue. - Confirmé/documenté : aucune limite au nombre de répliques par colonne (bouton "+ Réplique" reste toujours disponible). - Bulle = header (menu déroulant du "qui parle") + body (le texte), centrée dans sa colonne à 70% de largeur, alignée verticalement et reliée à la suivante par un trait — remplace l'ancien alignement gauche/droite façon chat. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-03 11:54:06 +02:00
williamandClaude Sonnet 5	2347a70c45	Quêtes/collision : personnages nommés, dialogues qui parlent, boîte de dialogue en jeu ... - "ℹ️ Informations" (propriétés d'un personnage) : nom éditable, réutilisé comme "qui parle" dans l'éditeur de dialogue de quête (menu déroulant, plus "Joueur" toujours disponible) — remplace l'ancien choix binaire joueur/pnj. db.sanitize_quest_dialogues accepte maintenant un nom libre. - Nouveau menu "🖥️ Interface" (palette d'objets) avec le premier widget : "💬 Boîte de dialogue" (kind="dialogue_box", screens/rendering/ dialogue_box_style.py) — position/taille comme tout objet de scène, panneau "🎨 Style" dédié (police, taille, épaisseur, couleur du texte, couleur header/body/footer). Rendu en <div> à 3 zones, jamais soumis à la collision ni à l'éditeur de collision (exclu partout : obstacles, liste des règles, payload). Rendu côté jeu dans .sceneUI, une couche FIXE au viewport (jamais .sceneWorld, qui défile avec la caméra). - static/js/play/dialogue-box-controller.js : fait le lien moteur entre l'action "quete" de l'éditeur de collision et ce widget — affiche la réplique en cours du dialogue correspondant au STATUT ACTUEL de la quête (gameData.quests, désormais exposé game-wide par full_game_payload.py), avance au clic sur "Suivant" (header = qui parle, body = texte, footer = bouton), se masque à la dernière réplique. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-03 11:20:58 +02:00
williamandClaude Sonnet 5	78c1aba8a0	Éditeur de quêtes : onglet à côté de Collision, pas une page à part ... Corrige la mauvaise interprétation de la demande initiale : la page dédiée "🗺️ Quêtes" (nav du jeu) est retirée au profit d'un nouvel onglet "🗺️ Quêtes" DANS l'éditeur de scène 2D, juste à côté de "🧩 Collision" — même liste tabulaire que l'onglet "📣 Événements" (titre/objectif/récompense/statut/résultat par ligne, pas des cartes), un clic sur une ligne ouvrant la modale d'arbre de dialogue déjà construite. L'assistant "+ Action" de l'éditeur de collision ouvre toujours cette même modale pour "Déclencher une quête". Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-02 20:46:27 +02:00
williamandClaude Sonnet 5	51427b5297	Éditeur de quêtes : titre/objectif/récompense/statut/résultat + dialogue à 3 colonnes ... Nouvel espace "🗺️ Quêtes" (game-wide, comme les événements personnalisés) : liste de cartes + "+ Nouvelle quête", chaque carte ouvrant une modale d'arbre de dialogue en bulles de théâtre ("speaker: texte"), une colonne par statut de quête (nouvelle/en cours/terminée) puisque le dialogue joué dépend de l'état de la quête au moment où le joueur parle au PNJ. Bulles alternées joueur/PNJ, couleur différente selon qui parle. L'action "Déclencher une quête" de l'éditeur de collision n'est plus un champ texte libre : elle ouvre directement CETTE MÊME modale (picker de quêtes existantes + création à la volée), et finalise la règle avec l'id numérique de la quête choisie — collision_rules.py migré en conséquence (quete_id devient un entier, comme evenement_id). Corrige aussi la miniature d'objet de la carte "🧩 Collision", 3× plus grande (120px) comme demandé. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-02 20:24:57 +02:00
williamandClaude Sonnet 5	e82eb7ce88	Retire l'export exécutable (.exe) et la partie publique en ligne (/jouer) ... Décision produit : seul l'export Web/SCORM (LMS) est pertinent — l'export exécutable Windows autonome (publish/build_package.py, bouton "Publier") et la partie publique par joueur (/jouer/<slug>, routes/public_play/, "Publier en ligne") sont jugés redondants et retirés. Conserve le mécanisme d'état "par joueur" (db/global_vars, db/rows, per_player) : infrastructure générique déjà utilisée par Score/Progression et testée indépendamment de toute route publique (voir tests/test_player_state.py), aucune raison de la retirer. _STATIC_ITEMS/_copy_characters (copie sélective des sprites CraftPix réellement utilisés) migrent de build_package.py vers build_scorm_package.py, seul appelant restant. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-02 10:59:28 +02:00
williamandClaude Sonnet 5	9776fbc057	Score/Progression (Phase 1.1 de la feuille de route produit) ... Concept de premier ordre, distinct du système de variables globales — préalable identifié aux futurs exports SCORM/xAPI (note de cadrage .claude/Forge_Engine_Cadrage.pdf) : ces standards ont besoin d'un signal "score"/"terminé" propre, pas d'une convention sur une variable choisie par le créateur. - db/scoring/ (même patron que db/global_vars/) : table _scoring, un score numérique + un statut (non_commence/en_cours/termine/reussi/ echoue) par joueur (voir db.PLAYER_SHARED pour l'aperçu créateur). - Deux nouvelles actions de flux, dans les deux éditeurs (document et scène) : "Modifier le score" (réutilise le vocabulaire d'opérations déjà là pour "Modifier une variable" — incrémenter/définir/etc., aucune nouvelle colonne de nœud) et "Définir le statut de la partie". - Routes créateur (routes/flow/flow_node_run_score.py, flow_node_run_status.py) + miroirs publics par joueur (routes/public_play/) — même principe que flow_node_run_variable.py. - GET /game/<slug>/scoring/<player_id> : lecture interne, pas exposée au joueur, préparée pour être consommée par le futur export Web/SCORM. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-02 06:23:37 +02:00
williamandClaude Sonnet 5	76a50fa87a	Onboarding guidé systématique + tableau de bord simplifié ... Comportement SYSTÉMATIQUE à chaque création de jeu (admin compris), pas une formalité réservée à l'inscription : /onboarding (routes/onboarding/) devient le point d'entrée unique — 4 cartes retournables (survol = explication au dos), défilement horizontal animé vers le nom du jeu. Un admin y repasse à volonté (pas de project_slug dédié, jamais bloqué/ redirigé vers un projet précédent) ; un compte "user" n'en a plus qu'un créé d'office (routes/auth/register_2fa.py), guidé ici à la place. - db/games/game_type_catalog.py : catalogue des 4 types (Quiz/ Embranchement-escape game/RPG/Créer mon jeu de A à Z), _meta['onboarding_type'] décide du "kind" du premier écran créé et si le tableau de bord complet reste accessible. - routes/games/game_dashboard.py, templates/game_dashboard_simple.html : un type restreint (quiz/embranchement/rpg) voit désormais SON tableau de bord (même route que "custom"), rendu en version simplifiée — juste ses écrans en cartes avec un aperçu RÉEL du contenu (scène mise à l'échelle par container query CSS, adaptée à la largeur réelle de la carte). "+ Ajouter un écran" n'y propose pas de choix de type : imposé par le projet (routes/screens/screens_new.py), verrouillé aussi côté serveur. - core/auth_guard.py : plus de blocage de game_dashboard par type — la restriction se fait au rendu, pas à l'accès à la route. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-02 06:22:45 +02:00
williamandClaude Sonnet 5	0f23024882	Structure de dossiers par utilisateur : projects/<propriétaire>/<projet>/ ... Remplace le rangement plat projects/<slug>/ par une vraie arborescence par compte créateur, sans toucher aux routes existantes (aucune ne déclare de "/" dans son <slug> : le chemin composé "propriétaire_projet" reste un détail interne à db/, décodé par db/games/project_slug.py, seul fichier à modifier si la convention change un jour). - db/game_dir.py, create_game.py, delete_game.py, list_games.py, move_game.py : résolvent/construisent ce chemin composé. Bénéfice direct : deux comptes différents peuvent chacun avoir un projet nommé pareil sans collision (l'unicité ne se vérifie plus que par dossier propriétaire). move_game.py renomme désormais le dossier PROPRIÉTAIRE entier (changement d'email), prêt pour un futur multi-projet. - scripts/migrate_flat_project_slugs.py : migration des projets déjà créés sous l'ancien rangement plat (simulation par défaut, --apply pour exécuter). - routes/auth/profile.py, tests/conftest.py : suppression de compte/jeu de test via db.delete_game (nettoie aussi le dossier propriétaire devenu vide) plutôt qu'un shutil.rmtree direct du seul dossier projet. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-09-02 06:22:03 +02:00
williamandClaude Sonnet 5	efa7d1d9b0	Fondations multi-éditeurs (1/3) : game_type + schéma/CRUD des objets de scène ... Première brique du plan "Fondations d'une plateforme multi-éditeurs" — l'utilisateur veut un éditeur dédié aux jeux 2D/serious games (scène à coordonnées pixel fixes, objets en couches, collision, personnages animés), distinct de l'éditeur générique actuel, sans dupliquer ce qui peut être partagé (comptes, objets de données, moteur de logique de flow, hébergement/publication). - db/games/get_game_type.py (nouveau) : "document" (défaut, éditeur actuel) ou "jeu_2d" (nouveau), stocké dans _meta comme is_public_played — aucune migration pour les jeux déjà créés (retombent sur "document"). Choisi obligatoirement à la création (templates/index.html), jamais modifiable ensuite. - screens/scenes/ (nouveau sous-module) : table _scene_objects (une scène = des objets en pixels fixes, pas les % fluides de _screen_elements — indispensable pour la collision/l'animation), CRUD complet, rendu HTML. kind="personnage" réutilise TELLE QUELLE la structure _personnage_data et les fonctions resolve_personnage_* déjà écrites pour le widget "personnage" de l'éditeur document (Phase 8) — même bibliothèque Forge, même moteur d'animation, juste une autre table de stockage. - screens/flow/ensure_flow_schema.py : += trigger_object_id/ target_object_id (ALTER TABLE sans contrainte FK, même patron que cond_element_a) — le moteur de logique de flow (_flow_nodes/_flow_edges, flow-engine.js) reste EXACTEMENT le même pour les deux éditeurs, seule la palette de nœuds change (screens/scenes/flow_palette.py, nouveau : sous-ensemble direct des triggers/actions existants, déjà génériques). - screens/screens_repo/ensure_schema.py : += scene_width/scene_height sur _screens (taille de scène fixe en pixels, sans effet sur un écran "document"). Reste à faire (prochains commits) : route + template de l'éditeur de scène, puis le rendu en mode jouable. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-31 09:42:38 +02:00
williamandClaude Sonnet 5	4585a72588	Phase 1 (3/3) : bascule "par joueur" dans le tableau de bord ... Dernier morceau du plan d'état par joueur : le créateur choisit, à la création d'une variable globale ou d'un objet, si chaque joueur aura sa propre valeur/ses propres lignes (coché par défaut) ou si elle est explicitement PARTAGÉE par tous les joueurs (ex. un compteur de visiteurs global, un catalogue commun) — voir db/global_vars/create_global_variable.py et db/definitions/create_definition.py (Phase 1, 1er commit). routes/global_vars/create_global_var.py, routes/objects/object_new.py : lisent la case à cocher "per_player" du formulaire (absente => reste per_player=1, comportement par défaut). templates/game_dashboard.html : case à cocher sur les deux panneaux de création + colonne "Par joueur" dans les deux tableaux existants, pour que ce réglage (immuable après création, comme le nom d'une variable) reste visible. db/definitions/list_definitions.py appelait _definitions directement sans jamais migrer son schéma — un tableau de bord ouvert avant la toute première création/modification d'objet aurait affiché "Non — partagé" pour un objet en réalité per_player=1 (colonne absente => Undefined, donc faux en Jinja) : corrigé en appelant ensure_field_bounds_schema() ici aussi, comme le fait déjà create_definition.py/get_definition.py. Vérifié : 239 tests passent (2 nouveaux, dont un qui aurait détecté le bug ci-dessus). Phase 1 (état par joueur) est maintenant complète : couche db/, route publique /jouer/<slug>, et ce réglage créateur. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-30 15:54:37 +02:00
williamandClaude Sonnet 5	3c39f1a249	Phase 1 (2/3) : route publique /jouer/<slug> + identité visiteur ... Ajoute la vraie route hébergée multi-joueurs qui manquait totalement (voir le constat d'exploration : /game/<slug>/play est réservé au créateur connecté, "Publier" ne génère qu'un exécutable mono-joueur) — un visiteur anonyme peut maintenant jouer un jeu explicitement publié en ligne, avec sa propre partie (variables/objets per_player posés dans le commit précédent). core/player_identity.py : identité visiteur via un cookie NON SIGNÉ (forge_player_id, secrets.token_urlsafe(16), 1 an) — une simple clé de partition, jamais un jeton d'autorisation. routes/public_play/ : 4 routes, miroirs des routes existantes de l'aperçu créateur mais threadées avec le vrai player_id du cookie au lieu du sentinel PLAYER_SHARED : - GET /jouer/<slug> (game_play_public.py) - GET /jouer/<slug>/runtime-payload - POST /jouer/<slug>/flow/nodes/<id>/run-data - POST /jouer/<slug>/flow/nodes/<id>/run-variable Chacune vérifie elle-même db.is_public_played(slug) (404 sinon) — un jeu n'est exposé publiquement que si le créateur l'a explicitement basculé "Publier en ligne" (nouveau db/games/is_public_played.py, réutilise la table générique _meta, comme game_meta.py pour 'name'). core/auth_guard.py : les 4 endpoints publics ajoutés à _PUBLIC_ENDPOINTS — la garde générique de connexion les laisse passer sans session, mais chaque vue vérifie quand même is_public_played elle-même (défense en profondeur, pas seulement une liste d'exceptions). CSRF (core/csrf_guard.py) n'a besoin d'AUCUN changement : le jeton est déjà lié à la session Flask, qui existe pour n'importe quel visiteur (connecté ou non). templates/play.html : FORGE_PLAY_URLS (posé en Phase -1) ne construit plus ses URLs via des noms de endpoint fixes (url_for('runtime_payload', ...)) mais reçoit des URLs déjà résolues par la route elle-même (runtime_payload_url/flow_node_run_data_url/flow_node_run_variable_url) — nécessaire puisque ce même template sert maintenant DEUX familles de routes (aperçu créateur ET partie publique), chacune avec ses propres noms de endpoint. routes/play/game_play.py (aperçu créateur, INCHANGÉ comportement) et game_play_public.py passent chacun ses propres URLs. templates/base.html : bascule "🌐 Publier en ligne" dans la barre de navigation du jeu, à côté de "📦 Publier" (export .zip) — deux fonctionnalités distinctes. db/games/game_meta.py expose maintenant is_public_played, disponible partout où `game` est dans le contexte. Vérifié : 237 tests passent (5 nouveaux dans test_public_play.py, dont un bout-en-bout via HTTP avec deux VRAIS clients de test anonymes — deux cookies forge_player_id différents — qui obtiennent des valeurs de variable indépendantes, et un qui verrouille que l'aperçu créateur reste inchangé). Syntaxe JS validée sur les deux variantes de play.html rendu (aperçu créateur et partie publique). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-30 15:46:00 +02:00
williamandClaude Sonnet 5	236d6b4b46	Phase 1 (1/3) : état par joueur — couche db/ et chaîne de rendu ... Fil directeur du plan : chaque variable globale et chaque objet de données gagne un player_id (sentinelle PLAYER_SHARED='__shared__' par défaut partout — l'aperçu créateur et tous les tests existants continuent de fonctionner À L'IDENTIQUE, aucune signature appelée sans argument explicite ne change de comportement), plus un réglage per_player choisi une fois à la création : - per_player=1 (par défaut) : chaque joueur a sa propre valeur/ses propres lignes. - per_player=0 : valeur/lignes partagées par tous les joueurs (ex. un compteur de visiteurs global, un catalogue commun). db/global_vars/ : _global_variables passe de UNIQUE(name) à UNIQUE(name, player_id) — SQLite ne permet pas de modifier une contrainte UNIQUE via ALTER TABLE, reconstruction de la table détectée et faite une seule fois (ensure_global_vars_schema.py) pour les jeux créés avant cette phase. La ligne "modèle" (player_id=PLAYER_SHARED, créée par le créateur) porte le réglage per_player et sert de valeur PAR DÉFAUT : la première écriture d'un joueur sur une variable per_player crée paresseusement SA propre ligne (copiée depuis le modèle) ; une lecture sans ligne encore écrite retombe sur le modèle (nouveau resolve_player_key.py). list_global_variables() (tableau de bord) ne montre toujours que les lignes modèles ; nouveau list_global_variables_for_player() expose la valeur EFFECTIVE d'un joueur au runtime (full_game_payload.py). db/definitions/ + db/rows/ : chaque table d'objet généré (create_definition.py) gagne une colonne player_id (ADD COLUMN simple, pas de contrainte UNIQUE en jeu ici) ; _definitions gagne per_player. Contrairement aux variables, PAS de repli sur une ligne "modèle" pour les lignes d'un objet per_player — une LISTE n'a pas de valeur par défaut unique à copier comme un scalaire, un nouvel objet per_player démarre VIDE pour chaque joueur (nouveau resolve_row_player_key.py). get_row/update_row/update_row_field/delete_row filtrent aussi par player_id (pas seulement id) : garde-fou contre un row_id d'un AUTRE joueur, nécessaire dès qu'un objet per_player sera exposé sur la future route publique /jouer/<slug>. Migration : nouveau ensure_player_id_column(slug, table_name), appelé avant toute requête sur une table d'objet créée avant cette phase. Chaîne de rendu (screens/elements/list_elements.py -> screens/rendering/render_element_html.py -> render_repeater.py/ render_jauge.py/resolve_bound_row.py/visibility_condition.py/ filter_repeater_rows.py) : player_id transite dans le ctx déjà utilisé partout pour "champ en cours" (ctx["_forge_player_id"], même patron que ctx["_forge_play_mode"], posé une seule fois par list_elements quand enforce_visibility=True) — pas de nouveau paramètre positionnel à threader dans chaque fonction, juste une clé de plus dans un mécanisme déjà en place. Nouveau tests/test_player_state.py : verrouille à la fois le nouveau comportement (deux joueurs => valeurs/lignes indépendantes ; per_player=0 => partagé ; nouveau joueur => valeur par défaut pour une variable, liste VIDE pour un objet ; get_row ne fuite jamais vers un autre joueur) et la non-régression de l'aperçu créateur (comportement historique inchangé). Vérifié : 232 tests passent (9 nouveaux). Reste à faire (prochains commits) : route publique /jouer/<slug>, identité visiteur (cookie), bascule "Publier en ligne" dans le tableau de bord. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-30 15:31:08 +02:00
williamandClaude Sonnet 5	2c59e54556	Simplifie les événements : notification pure, sans paramètre ... Retour de l'utilisateur sur le premier jet : "Déclencher un événement" ne doit JAMAIS faire choisir un élément — c'est une notification pure, rien de plus. C'est à l'ÉCOUTEUR (déclencheur "Sur un événement personnalisé" → condition → action) de décider quoi faire ensuite, avec ses réglages habituels (cible fixe, "Ligne cliquée"...), jamais à l'événement de transporter un paramètre. Retire donc tout le mécanisme de transmission ajouté au tour précédent (has_element_param, target_element_from_event, EVENT_ROW_ID, window.lastEventParams) : - db/custom_events/ : _custom_events perd sa colonne has_element_param — un événement n'est plus qu'un nom + une description. - screens/flow/ : retire target_element_from_event (colonne ajoutée par ALTER TABLE, laissée inerte sur les bases déjà migrées — sans conséquence, plus jamais lue ni écrite) et la constante EVENT_ROW_ID. - routes/flow/flow_node_run_data.py : retire la résolution EVENT_ROW_ID, revient à sa forme d'origine (seul CLICKED_ROW_ID reste géré). - templates/screen_edit.html : le nœud Action "Déclencher un événement" n'a plus qu'un sélecteur d'événement — plus de champs élément/ligne. Le nœud Action "Modifier un élément" perd la case "Utiliser l'élément transmis par l'événement en cours". L'onglet Événements perd la case à cocher "Paramètre" (création et édition). - templates/play.html : window.dispatchGameEvent(eventId) ne prend plus que l'id de l'événement — scan global inchangé, mais ne pose plus aucun window.lastEventParams. modifier_element et readFieldValue reviennent à leur résolution d'origine (plus de branche event-aware). 208 tests au total (2 tests retirés, devenus sans objet : la persistance de target_element_from_event et la résolution serveur d'EVENT_ROW_ID). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-29 20:39:42 +02:00
williamandClaude Sonnet 5	dcbec16818	Ajoute les événements personnalisés (backend) : déclencher/écouter ... Première moitié de la fonctionnalité "événements" (NEED_ACTION et autres) : une entité game-wide (nom, description, "a un paramètre élément" oui/non), déclenchable comme nouvelle action du graphe de logique depuis n'importe quelle scène/modèle, et écoutable comme nouveau type de déclencheur depuis n'importe quel autre. L'UI (nouvel onglet "Événements" dans screen_edit.html, formulaires de nœud, exécution côté client dans play.html) suit dans un commit séparé. - db/custom_events/ (calqué sur db/global_vars/) : CRUD de la table _custom_events (nom unique, description, has_element_param). create_custom_event est idempotent par nom (même convention que create_global_variable) — sans risque en cas de double soumission. - screens/flow/ : 3 nouvelles colonnes sur _flow_nodes (trigger_custom_event_id/target_custom_event_id : quel événement un nœud écoute/déclenche ; target_element_from_event : indicateur réutilisable par n'importe quel nœud Action utilisant déjà target_element_id, pour résoudre "l'élément transmis par l'événement en cours" au lieu d'une cible fixe — contourne la contrainte de clé étrangère de target_element_id, qui empêche d'y stocker un sentinel comme EVENT_ROW_ID directement). Nouveau trigger_event "evenement" et action_type "declencher_evenement". - screens/custom_events/ (PAS dans db/, même séparation que screens/elements/delete_element.py) : delete_custom_event, la SEULE suppression d'entité game-wide du moteur à vraiment cascader (demande explicite) — supprime tous les nœuds/arêtes qui référencent l'événement, sur TOUTES les scènes ET tous les modèles à la fois (aucun filtre screen_id nécessaire : un modèle est un écran caché, même table _flow_nodes). list_custom_event_usages : où un événement est écouté/déclenché, pour l'onglet Événements à venir. - routes/custom_events/ : CRUD monté sous /game/<slug>/events/..., redirige vers l'éditeur de scène/modèle d'origine (screen_id transmis par le formulaire) avec l'onglet "events" à ouvrir. tests/test_custom_events.py (nouveau) : idempotence à la création, usages détectés sur deux écrans différents, suppression qui retire bien les DEUX nœuds (un sur une vraie scène, un sur un modèle/écran caché) en une seule opération, sans toucher aux écrans eux-mêmes. 207 tests au total. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-29 19:14:19 +02:00
williamandClaude Sonnet 5	1289079da5	Ajoute "Publier" : exporte un jeu en exécutable Windows autonome ... Fonctionnalité mise de côté depuis le tout début de ce chantier ("jouable en toute autonomie"). Nouveau bouton "📦 Publier" dans la barre de navigation du jeu (après "▶️ Jouer") : construit un zip contenant un Python portable + Flask embarqués, une copie figée du moteur de rendu (screens/db/filters + le strict minimum de core/) et des données du jeu (game.db + uploads/), lancé en double-cliquant sur run.bat — aucune installation requise, 100% hors-ligne (voir le commit précédent qui a auto-hébergé polices/animate.css, dernière dépendance CDN de l'app). Exploration préalable a confirmé que screens/ et db/ sont déjà totalement découplés de auth/routes/core (seul lien : un try/except optionnel dans db/connection.py) et que templates/play.html n'appelle que 4 endpoints "purs" (aucune logique d'auth mélangée dedans) — ce qui a permis de réutiliser ces routes quasiment telles quelles dans un mini-serveur Flask séparé plutôt que de les réécrire. - db/constants.py : PROJECTS_DIR devient surchargeable via FORGE_PROJECTS_DIR (même schéma que auth/connection.py) — le serveur joueur autonome pointe ainsi vers son propre dossier "projects/" embarqué. - publish/vendor_runtime.py : télécharge (une fois par poste, mis en cache sous data/publish_vendor/ — déjà ignoré par git) le ZIP Python embeddable officiel (python.org) et vendore Flask via pip --target ; fonctions séparées et mockables pour ne jamais déclencher de vrai téléchargement dans les tests. - publish/player_app_template.py : mini-Flask autonome, réutilise core/flask_app.py et core/jinja_filters.py tels quels (SLUG figé en dur au moment de la publication, csrf_token() factice puisqu'aucune session n'existe dans cet export). sys.path doit être complété manuellement au démarrage : le python311._pth de la distribution embeddable ne référence que le dossier de python.exe lui-même, jamais celui du script lancé. - publish/build_package.py : assemble le zip dans un dossier temporaire (jamais les vrais fichiers de l'app), copié/nettoyé après envoi de la réponse HTTP (routes/publish/publish_game.py, déjà protégée par la garde d'accès existante — aucune vérification supplémentaire). - templates/base.html : bouton + modale (avancement séquentiel, jamais de suivi serveur réel — le build est rapide) qui déclenche le téléchargement du zip via un blob, comme la modale des codes de récupération déjà en place (posée À L'INTÉRIEUR de <main> pour que pjax.js la remplace et rejoue son script à chaque navigation). Vérifié pour de vrai (pas seulement via les tests) : zip construit, extrait, lancé avec le Python embeddable réel — /, /game/test/play, /game/test/runtime-payload et /static/style.css répondent tous 200. tests/test_publish.py (nouveau, vendor mocké) : structure du zip, SLUG correctement substitué, route protégée par la même isolation par projet que le reste de l'app. 203 tests au total. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-29 09:58:32 +02:00
williamandClaude Sonnet 5	c8c949c430	Ajoute la régénération des codes de récupération et le changement d'email ... Depuis /profile : - "Codes de récupération 2FA" : régénère les 10 codes (invalide les 10 précédents d'un coup, voir auth.generate_recovery_codes) et les affiche une seule fois via le même modal qu'à l'inscription (session flash, core/recovery_codes_flash.py) — mot de passe actuel requis. - "Adresse email" : change l'adresse (mêmes règles qu'à l'inscription — format valide, unicité — voir auth/email_validation.py, désormais partagé avec create_user.py au lieu d'être dupliqué). Pour un compte "user", le dossier de son unique projet porte le nom de son adresse (project_slug = slugify(email)) : il est renommé pour suivre le changement (db/games/move_game.py, même logique d'unicité par suffixe que create_game()), sans quoi project_slug ne correspondrait plus à aucun dossier réel. Un "admin" n'a pas de project_slug dédié : rien n'est renommé pour ce rôle. 18 nouveaux tests (tests/test_profile.py) : mauvais mot de passe refusé pour les deux actions, mauvais format/email déjà pris refusés, dossier bien renommé et project_slug mis à jour, connexion possible avec la nouvelle adresse, anciens codes de récupération bien invalidés après régénération, modal jamais réaffiché deux fois. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-29 06:29:38 +02:00
williamandClaude Sonnet 5	d90abc827b	Ajoute l'authentification : inscription, mot de passe fort, 2FA obligatoire, isolation par utilisateur ... Première des deux grandes fonctionnalités demandées (authentification d'abord, export HTML/CSS/JS autonome ensuite) : - Inscription (nom, prénom, email UNIQUE, mot de passe) avec schéma visuel du mot de passe (jauge + liste de critères qui passent au vert en direct — auth/password_strength.py, mêmes règles vérifiées côté serveur qu'affichées côté client). - Double authentification (TOTP, compatible Google Authenticator/Authy) OBLIGATOIRE dès l'inscription : QR code (SVG, sans dépendance Pillow) à scanner puis code à confirmer avant que le compte soit utilisable — voir auth/create_user.py (totp_confirmed) et routes/auth/register_2fa.py. - Connexion en 2 temps (mot de passe puis code TOTP), déconnexion. - Isolation par utilisateur : un compte "user" est limité à un SEUL projet, dont le dossier est nommé d'après son adresse email (slugifiée) et créé automatiquement dès la 2FA confirmée — aucune page de gestion multi-jeux pour lui (redirigé directement vers son propre tableau de bord). Le rôle "admin" reste illimité, comme le moteur l'a toujours été (le TOUT PREMIER compte jamais créé sur une base de comptes vide devient automatiquement admin — voir auth/is_first_user.py — pas de mot de passe par défaut à faire circuler : s'inscrire en premier suffit). Un compte "user" ne peut pas non plus supprimer son unique projet (aucune façon d'en recréer un ensuite). - Garde d'accès globale (core/auth_guard.py, un seul before_request) : toute page exige une connexion, sans avoir touché individuellement aux ~80 routes déjà existantes du moteur. tests/conftest.py isole complètement les tests de la vraie base de comptes (FORGE_USERS_DB_PATH/FORGE_SECRET_KEY_PATH vers un dossier temporaire propre à la session de tests) et authentifie automatiquement la fixture `client` partagée en tant que compte admin de test — les 155 tests déjà existants continuent de passer SANS AUCUNE modification de leur côté, exactement comme avant l'authentification. 10 nouveaux tests dédiés (tests/test_auth.py) : inscription/mots de passe/2FA/connexion/ isolation par projet/blocage de suppression, vérifiés en conditions réelles (vrai client de test Flask, vraie base SQLite, vrais codes TOTP calculés avec pyotp). 165 tests au total, tous au vert. Nouvelles dépendances : pyotp, qrcode (requirements.txt). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-28 19:30:43 +02:00
williamandClaude Sonnet 5	5c069ae1fe	Corrige LE vrai bug : un nom de champ mot-réservé SQL (ex. "order") faisait disparaître l'objet entier ... Reproduit à l'identique le cas signalé (objet "dialog" avec les champs order/spiker/text/level_id/parcour_id) : le champ "order" est un mot réservé SQL — "CREATE TABLE dialog (order INTEGER, ...)" plante avec "OperationalError: near \"order\": syntax error". Comme ce crash survient APRÈS l'INSERT de la ligne _definitions mais AVANT le commit(), rien n'était jamais persisté : l'objet ENTIER disparaissait, malgré des champs parfaitement remplis — d'où "j'ai tout rempli comme il faut et aucun objet n'est créé". Mon précédent correctif (champ "Relation" sans cible) était réel mais ne couvrait pas ce cas précis. Cause de fond : chaque nom de colonne (dérivé du nom de champ tapé par l'utilisateur, via slugify) était interpolé TEL QUEL dans du SQL brut (CREATE TABLE, INSERT, UPDATE, ALTER TABLE ADD/DROP/RENAME COLUMN) sans jamais être encadré de guillemets — n'importe quel nom de champ qui soit aussi un mot réservé SQLite (order, group, index, select, where, table, key, default, check, references, unique...) déclenchait exactement le même crash-et-perte-de-transaction, dans n'importe laquelle de ces opérations. Correctif général (pas un simple contournement pour "order") : db/quote_ident.py encadre tout identifiant de colonne de guillemets doubles (forme standard SQL, supportée par SQLite) — appliqué partout où un nom de colonne utilisateur est interpolé dans du SQL brut : create_definition, add_field_to_definition, delete_field, update_field (RENAME COLUMN), insert_row, update_row, update_row_field, rows_referencing. Les noms de TABLE n'ont pas besoin de cette protection (table_name_for.py les préfixe toujours "obj_", donc jamais un mot réservé à eux seuls). Trois nouveaux tests (tests/test_reserved_sql_keyword_field_names.py) : création avec un champ "order" + insertion/lecture/mise à jour d'une ligne, renommage d'un champ vers/depuis un mot réservé ("group"), ajout d'un champ "select" à un objet existant — les trois confirmés en échec sur l'ancien code (même erreur reproduite) puis au vert avec le correctif. 136 tests au vert au total. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-27 19:27:09 +02:00
williamandClaude Sonnet 5	aaf9954446	Variables globales Objet/Tableau, lisibles via un chemin dans les conditions et filtres ... Deux nouveaux types de variable globale (db/constants.py) : "objet" et "tableau" — valeur stockée en JSON (colonne TEXT existante), avec validation à la création/modification (db/global_vars/ coerce_structured_value.py) : un JSON invalide retombe sur un défaut sûr ("{}"/"[]") plutôt que de corrompre silencieusement la variable pour toutes ses lectures suivantes. apply_variable_action.py n'a besoin d'aucun changement — "definir_texte" écrit déjà n'importe quelle chaîne telle quelle. Nouvelle résolution de chemin, partagée (screens/rendering/ filter_repeater_rows.py::_resolve_variable_path) : navigue dans la valeur JSON d'une variable selon un chemin ".champ"/"[index]" chaînable (ex. ".arme.degats", "[0].valeur") — ne lève jamais, renvoie None si le JSON est invalide ou qu'un segment du chemin ne correspond à rien. Branchée à deux endroits, qui lisaient déjà une variable globale : - La valeur de comparaison {{$nom_variable}} (filtres de Répéteur ET Condition de visibilité, qui partagent le même _resolve_filter_value()) accepte maintenant un chemin optionnel : {{$perso.nom}}, {{$scores[0]}}. Le sélecteur "Variable globale" du panneau de propriétés (screen_edit.html, .filterValueVariable) gagne un champ "Chemin optionnel" à côté du choix de variable — même regex étendue côté JS (_VAR_REF_RE) que côté Python (_VAR_REF_PATTERN), pour que la valeur round-trip correctement à la réouverture du panneau. - La variable VÉRIFIÉE par une Condition de visibilité en mode "variable" (choisie via un <select>, pas la syntaxe {{$...}}) gagne son propre nouveau contrôle "Chemin dans la variable" (visibility_condition_ controls.py) — nécessaire pour comparer un champ d'un Objet ou un élément d'un Tableau, pas seulement la variable entière. game_dashboard.html (onglet Variables) : la valeur par défaut de la barre de création devient un <textarea> (fonctionne aussi bien pour un JSON multi-ligne qu'un scalaire court), et la cellule "Valeur" du tableau des variables existantes devient un <textarea> quand le type est objet/ tableau. 4 nouveaux tests (tests/test_variable_object_array.py) : validation JSON à la création, filtre de Répéteur avec chemin chaîné, condition de visibilité avec accès par index de tableau, chemin invalide/absent sans plantage. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-27 14:28:23 +02:00
williamandClaude Sonnet 5	79fe04c6ac	Corrige "database is locked" causé par une connexion SQLite qui fuit après un plantage ... Bug remonté : suppression d'un élément échouant avec sqlite3.OperationalError: database is locked, exactement sur le conn.execute() de delete_element.py. La trace complète montrait que la connexion attendait puis lâchait après le timeout (10s) - pas une simple collision passagère entre deux requêtes (déjà gérée par WAL + busy_timeout, voir les commentaires existants de connect()), mais un verrou tenu bien plus longtemps : une connexion ouverte par une requête ANTÉRIEURE qui a planté, jamais fermée. Cause de fond : chaque fonction de db/ (~80 d'entre elles) ouvre sa propre connexion et est censée la fermer elle-même avant de rendre la main - si une exception survient entre l'ouverture et cette fermeture, le conn.close() prévu n'est jamais atteint. En mode debug (voir app.py), le débogueur Werkzeug garde la trace complète de l'erreur en mémoire pour l'inspection interactive, ce qui inclut la variable locale `conn` : empêchée d'être ramassée par le GC, elle ne libère jamais son verrou d'écriture SQLite - bloquant TOUTE écriture suivante jusqu'au redémarrage du serveur, même longtemps après l'erreur d'origine et sans lien apparent avec elle (d'où la confusion : l'erreur semble venir de l'action qui échoue, alors qu'elle est victime d'une fuite antérieure). Fix, dans db/connection.py, sans toucher aux ~80 fonctions existantes : connect() enregistre maintenant chaque connexion sur le contexte de la requête Flask en cours (flask.g, uniquement quand il y en a un - un appel direct hors requête, scripts/tests, n'est pas concerné) ; un teardown_request ferme toute connexion encore ouverte à la fin de CHAQUE requête, qu'elle ait réussi ou planté (garanti par Flask, contrairement à after_request). Fermer une connexion déjà fermée normalement ne fait rien, donc aucun changement de comportement pour le cas normal. Ajoute tests/test_db_connection_leak_safety_net.py, qui reproduit le scénario exact (connexion ouverte puis exception avant fermeture) et vérifie qu'une écriture suivante ne bloque plus - désactivé temporairement pour confirmer que le test échoue bien (et de la même façon) sans le fix. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-25 15:06:16 +02:00
williamandClaude Sonnet 5	11a2d397e8	Remplace la création inline de variable par une page de gestion dédiée, redesign du tableau de bord du jeu ... Retrait de la création rapide de variable globale depuis le sélecteur de la Condition de visibilité (bouton "+ Créer") : une variable globale est désormais gérée comme un objet "jeu" à part entière, avec une vraie page CRUD ("Variables", nouvelle entrée du menu de gauche) - création, édition du type/valeur, suppression. Le nom reste volontairement immuable après création (c'est par ce nom qu'une condition de visibilité ou une action "Modifier une variable" la référence - la renommer casserait ces réglages en silence), d'où db.update_global_variable qui ne touche que type/valeur. Redesign du tableau de bord du jeu (game_dashboard.html) en deux colonnes : à gauche tout ce qu'on peut créer (écrans, éléments de jeu, variables, jouer, nouvel objet) plus les paramètres du jeu (renommer/ supprimer) ; à droite ce qui a déjà été créé (objets définis). Remplace les cartes Bulma par le système de mise en page compact déjà défini dans style.css (.twoCol/.listRow/.dangerZone/.addBtn) mais jamais utilisé jusqu'ici - plus dense et cohérent avec le reste de l'éditeur. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-25 10:00:35 +02:00
williamandClaude Sonnet 5	c04bc0b926	Ajoute la condition de visibilité et les variables globales ... Nouveau panneau "Condition de visibilité" disponible dans les propriétés de TOUT élément (widget) : permet de masquer un élément en mode jouable selon deux moyens, au choix - - une variable globale (nom + type + valeur, une seule par jeu, stockée dans une nouvelle table _global_variables) ; - le champ d'un objet de données existant (même convention "état de partie" - une seule ligne - déjà utilisée par la Jauge). Une variable ne servant à rien si elle ne peut jamais changer en cours de partie, ajoute aussi une nouvelle action de flow "Modifier une variable globale" (parallèle à "Modifier une donnée"), avec sa propre route d'exécution serveur et son sous-formulaire dans l'éditeur de logique de scène. Une variable peut aussi se créer à la volée depuis le sélecteur du panneau de visibilité, sans quitter les propriétés de l'élément. La condition n'est évaluée qu'en mode jouable (/game/<slug>/play), jamais dans l'éditeur, pour que l'élément reste toujours sélectionnable. Un élément masqué se réévalue en direct après toute action "Modifier une donnée/variable", via le même mécanisme de rafraîchissement déjà utilisé par la Jauge et le Répéteur. Corrige au passage deux bugs découverts en testant bout en bout : (1) apply_ctx plantait sur le nouveau marqueur interne _forge_play_mode (un booléen parmi les {{champ}} à substituer, qui attend des chaînes) ; (2) _compare traitait toute valeur booléenne stockée en chaîne ("0" inclus, donc toujours vraie en Python) comme vraie - correct pour les champs d'objet (entiers SQLite) mais faux pour les variables globales (toujours stockées en texte). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-25 09:34:10 +02:00
williamandwilliam	3f4ebc4527	first commit	2026-08-21 16:23:49 +02:00