williamandClaude Sonnet 5
b2e933f322
Build and deploy / test-python (push) Successful in 11m12s
Build and deploy / test-js (push) Successful in 53s
Build and deploy / lint-python (push) Successful in 3m56s
Build and deploy / lint-js (push) Successful in 3m1s
Build and deploy / build-and-push (push) Skipped
Build and deploy / deploy (push) Skipped
Build and deploy / sonarqube (push) Failing after 3m58s
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>
28 lines
1.3 KiB
Python
28 lines
1.3 KiB
Python
"""Garde d'accès aux sprites animaux CraftPix (game_engine.ADMIN_SPRITE_LIBRARY)
|
|
— licence CraftPix, réservés au rôle admin (voir le plan "Bibliothèque de
|
|
sprites animaux CraftPix"). La galerie de l'éditeur (routes/game/screens/
|
|
screen_edit.py, routes/game/scenes/scene_edit_view.py) ne les propose déjà pas
|
|
à un compte "user", mais un POST direct vers une route d'écriture
|
|
(element_add, scene_object_add, element_set_personnage_data,
|
|
scene_object_personnage_data) pourrait contourner ce filtrage d'UI — ce
|
|
garde-fou ferme cette brèche au moment de l'écriture."""
|
|
|
|
from flask import abort, g
|
|
|
|
import game_engine
|
|
|
|
|
|
def forbid_admin_only_character(forge_character: str | None) -> None:
|
|
if forge_character in game_engine.ADMIN_ONLY_CHARACTER_SLUGS and g.current_user["role"] != "admin":
|
|
abort(403)
|
|
|
|
|
|
# Même règle que forbid_admin_only_character ci-dessus, pour la
|
|
# bibliothèque d'images de fond CraftPix (game_engine.BACKGROUND_LIBRARY,
|
|
# voir game_engine/labels/background_library.py) — TOUJOURS admin-only
|
|
# aujourd'hui (un seul pack, tous sous licence CraftPix), contrairement
|
|
# aux personnages Forge/Kenney (domaine public, jamais gatés).
|
|
def forbid_admin_only_background(background_slug: str | None) -> None:
|
|
if background_slug in game_engine.ADMIN_ONLY_BACKGROUND_SLUGS and g.current_user["role"] != "admin":
|
|
abort(403)
|