6 Commits

Include renames

Author	SHA1	Message	Date
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	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	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	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	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