Refactor/readability

[Lskrzypc]

🎯 Objectif

Améliorer la qualité et la maintenabilité du projet API grâce à l’ajout d’ESLint et à une meilleure organisation du build.

🔍 Contexte

Pour garantir un code plus propre et éviter l’introduction de vulnérabilités, j’ai intégré des vérifications automatiques via GitHub Actions. De plus, j’ai modifié l’organisation du projet afin de séparer le code source et le code buildé pour plus de lisibilité.

🛠️ Changements apportés

• Mise en place d’ESLint sur le projet API
• Ajout d’une étape check-service.yml dans GitHub Actions qui :
  • Vérifie que le projet build correctement
  • Vérifie l’absence de vulnérabilités
  • Vérifie que chaque dépendence installée est réellement utilisée
  • Vérifie le respect des règles de lint
• Déplacement des fichiers générés dans un dossier dist/ au lieu de source/

📦 Dépendances

• rimraf : utilisé pour nettoyer le dossier dist/ avant chaque build
• depcheck : permet de détecter les dépendances inutilisées ou manquantes

✅ Comment tester

1. Lancer un build local (pnpm run build) et vérifier que les fichiers se génèrent dans dist/
2. Lancer un lint (pnpm run lint) et s’assurer que les règles sont bien respectées (normalement c'est censé lint au save du fichier. Si non, ctrl + maj + p et restart eslint server sur vscode)
3. Vérifier l’exécution du workflow check-service.yml sur GitHub lors d’une PR sur preprod

📌 Notes complémentaires

• L’étape CI bloque le merge si le build échoue, si des vulnérabilités sont détectées ou si le lint n’est pas respecté

NB sur les nouvelles dépendances:

• Rimraf permet de supprimer le dossier build a chaque fois que l'on lance la commande
• Depcheck vérifie l'utilisation des dépendances installées dans le projet

[Lskrzypc]

Hi Milo, you should ensure approving the workflow if you're okay with that. Github prevent me to run new actions without your permission :)

[MylowMntr]

Je suis vraiment pas fan du tab size 2:
je préfère 4 pour la simple et bonne raison que ca force à faire des lignes courtes (et en plus on gagne en lisibilité)

Le reste, yes pourquoi pas !