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>
35 lines
1.7 KiB
Python
35 lines
1.7 KiB
Python
import io
|
|
|
|
from mutagen import File as MutagenFile
|
|
|
|
# Demande explicite : une vidéo importée dans "Mes assets" ne doit jamais
|
|
# dépasser 5 minutes (voir l'action "vidéo", game_engine/rendering/
|
|
# collision_rules.py — pensée pour une courte cinématique/cutscene, pas un
|
|
# film entier), même esprit que MAX_AUDIO_SECONDS pour un son.
|
|
MAX_VIDEO_SECONDS = 300
|
|
|
|
|
|
def validate_video_duration(
|
|
content: bytes, filename: str = "video.mp4", max_seconds: int = MAX_VIDEO_SECONDS
|
|
) -> str | None:
|
|
"""Lit la durée d'un fichier vidéo MP4 (seul format accepté à l'upload,
|
|
voir routes/assets/upload_user_asset.py — mutagen ne sait pas lire
|
|
fiablement la durée de webm/mov/avi, contrairement au conteneur MP4/
|
|
MOV ISO base media, qu'il décode via mutagen.mp4) et renvoie un
|
|
message d'erreur (français, prêt à afficher) si elle dépasse
|
|
`max_seconds`, ou si le format n'a pas pu être reconnu du tout (repli
|
|
prudent : un fichier illisible n'est jamais accepté silencieusement).
|
|
Renvoie None si la durée est correcte.
|
|
|
|
filename (juste l'extension importe) : contrairement à un son (voir
|
|
validate_audio_duration.py), la détection MP4 de mutagen SANS indice
|
|
de nom de fichier échoue silencieusement sur certains fichiers
|
|
(moov/ftyp minimaux) — passer ".mp4" explicitement la rend fiable."""
|
|
video = MutagenFile(io.BytesIO(content), filename=filename)
|
|
if video is None or video.info is None or not getattr(video.info, "length", None):
|
|
return "Format vidéo non reconnu."
|
|
duration = video.info.length
|
|
if duration > max_seconds:
|
|
return f"Cette vidéo dépasse {max_seconds // 60} minutes (durée : {int(duration)} secondes)."
|
|
return None
|