Première brique du pipeline : lister les vidéos/streams d'une chaîne YouTube (yt-dlp --flat-playlist), filtrables par mots-clés sur le titre. - ytDlpRunner.ts : exécution de yt-dlp via child_process.spawn (jamais de shell, arguments en tableau - exclut l'injection de commande) - ChannelDiscoveryService.ts : validation stricte du handle/URL de chaîne + séparateur `--` avant l'URL (défense en profondeur contre l'injection d'options yt-dlp), parsing tolérant des lignes JSON, filtrage mots-clés - routes/channel.ts : GET /api/channel (validation de schéma Fastify, 400 si entrée invalide, 502 si yt-dlp échoue) - 14 tests (unitaires services + intégration route via app.inject) - un test de route a révélé une regex de validation trop permissive qu'un test de service avait manquée, corrigée avant ce commit La file d'attente et le téléchargement suivront dans une PR séparée. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
5.1 KiB
5.1 KiB
Architecture
Contexte
Remplace une série de scripts .cmd (./downloader, non versionné) par une application web : parcourir une chaîne YouTube, mettre des vidéos/audios en file d'attente, les télécharger en parallèle avec suivi en temps réel, et les livrer au navigateur de l'utilisateur. Pas d'intégration Emby, pas d'authentification (usage LAN).
Structure du dépôt
server/ Backend Fastify (TypeScript, ESM)
client/ Frontend TypeScript vanilla (esbuild + sass)
.forgejo/workflows/ CI : lint + typecheck + tests + build, puis publication Docker
Pas de docker-compose.yml pour l'instant (une seule image, pas encore de dépendances externes type base de données).
Backend (server/)
- Fastify 5, TypeScript strict, ESM (
"type": "module", imports relatifs en.js, résolutionNodeNext). src/index.ts: bootstrap (plugins, écoute, arrêt propre SIGINT/SIGTERM).src/config.ts: configuration runtime depuis l'environnement (PORT,HOST,CLIENT_DIST_PATH).src/routes/: routes Fastify enregistrées comme plugins (health.ts,channel.ts). Chaque domaine métier (découverte de chaîne, file d'attente, téléchargement, notifications temps réel à venir) suit le même principe : un plugin de route fin soussrc/routes/, qui délègue à un service soussrc/services/.src/services/:ytDlpRunner.ts: exécution bas niveau de yt-dlp (node:child_process.spawn, jamais de shell - arguments toujours passés en tableau, jamais concaténés dans une chaîne, pour exclure toute injection de commande).ChannelDiscoveryService.ts: liste les vidéos/streams d'une chaîne (yt-dlp --flat-playlist -j), filtre par mots-clés sur le titre. Valide strictement le handle/URL de chaîne en entrée (premier caractère forcément alphanumérique) et ajoute un séparateur--avant l'URL passée à yt-dlp, en défense en profondeur contre l'injection d'options.
- Le serveur sert aussi les fichiers statiques de
client/distvia@fastify/static. - Tests : Vitest (
server/test/). Deux niveaux : services testés directement (avecytDlpRunnermocké pourChannelDiscoveryService- seulytDlpRunnerlui-même spawn un vrai process,nodeen test), et routes testées viaapp.inject()(schéma de validation Fastify, mapping des erreurs en codes HTTP). Les deux niveaux comptent : un test de service avec un exemple d'entrée mal choisi avait laissé passer une regex de validation trop permissive (--execaccepté), repérée seulement par le test de route avec un autre exemple.
Frontend (client/)
- TypeScript vanilla, pas de framework. Chaque vue = un dossier avec
Vue.ts/Vue.html/Vue.scss(gabarit importé comme chaîne brute, style Sass importé et compilé). Voirsrc/shell/pour l'exemple actuel (placeholder). - Build via
build.mjs(esbuild), pas d'outil de build plus lourd (Vite, webpack) : le script compilesrc/app.tsen bundle ESM, copiesrc/index.html, et compile/regroupe les.scssimportés en unapp.cssvoisin (plugin esbuild maison qui délègue au paquetsass). - Pas encore de router : une seule vue (shell) pour l'instant. Un routeur sera introduit avec la première vraie vue (parcours de chaîne).
API
GET /health: statut du service (utilisé par leHEALTHCHECKDocker).GET /api/channel?channel=<@handle|URL>§ion=<videos|streams>&keywords=<a,b>: liste les vidéos/streams d'une chaîne, filtrés par mots-clés (ET, insensible à la casse) sur le titre.sectionpar défautvideos. Réponses :200 { videos: [...] },400sichannelest absent/invalide,502si yt-dlp échoue.
Lint
Biome.js à la racine (biome.json), applique aux deux paquets (server/src, client/src) - pas d'ESLint/Prettier. downloader/ (scripts legacy, non versionné) est explicitement exclu.
Docker / CI-CD
server/Dockerfile: image multi-étapes construite avec pour contexte la racine du dépôt (docker build -f server/Dockerfile .), car elle a besoin à la fois deserver/etclient/. Étapes : build client (esbuild) → build serveur (tsc) → image runtime (Node 22 Alpine +ffmpeg+ binaire autonomeyt-dlp, utilisateur non-rootnode).yt-dlpetffmpegsont installés dans l'image (pas vendorisés dans le dépôt, contrairement aux scripts.cmdd'origine qui embarquaientyt-dlp.exeet un build ffmpeg complet)..forgejo/workflows/publish-docker.yml: jobtest(lint racine, typecheck/tests/build serveur, typecheck/build client) puis jobpublish(build + push de l'image versgit.dev.kyuno.fr/kyuno/youtubedownloader, seulement sitestpasse et hors PR). Tags :latest+ sha court sur pushmain,X.Y.Z/X.Y/latestsur tagvX.Y.Z. Nécessite le secret de dépôtREGISTRY_TOKEN(à créer manuellement, voir l'en-tête du workflow - un jeton d'API n'a pas le droit d'écrire dans le registre de paquets).
Statut
- Fait : infrastructure (serveur qui répond, client placeholder, Docker, CI), découverte de chaîne (
GET /api/channel). - À venir (PR successives) : file d'attente, téléchargement yt-dlp (vidéo/audio/miniature), notifications temps réel, vues frontend (parcours de chaîne, file d'attente).