Merge branch 'dev' into chore/coverage-tests-frontend-ci

Résolution du conflit sur .github/workflows/frontend.yml : - conserve les étapes upload/download de l'artifact lcov ajoutées sur dev, sans lesquelles le rapport ne quitte pas le runner du job test et n'atteint jamais le scanner Sonar ; - conserve le job security-audit et le retrait du bloc deploy commenté ; - retient la commande de test de la branche. Le séparateur `--` manquait côté dev : npm consommait les trois flags comme sa propre configuration et ng test tournait sans argument. De plus `--code-coverage` n'existe pas dans le builder @angular/build:unit-test, où l'option s'appelle `coverage`.
Merge pull request #100 from ineszang/chore/audit-dependances-ci
2026-09-18 12:30:10 +02:00 · 2026-09-18 12:26:46 +02:00 · 2026-09-18 12:23:50 +02:00 · 2026-09-18 12:23:25 +02:00 · 2026-09-17 13:55:17 +02:00 · 2026-09-17 12:25:15 +02:00
2 changed files with 45 additions and 10 deletions
@@ -56,3 +56,27 @@ jobs:
      # Le marqueur `integration` est exclu par défaut, donc aucune base n'est nécessaire ici.
      - name: Tests et couverture
        run: uv run pytest --cov-fail-under=85
+
+  security-audit:
+    name: Audit des dépendances
+    runs-on: ubuntu-latest
+    defaults:
+      run:
+        working-directory: apps/backend
+
+    steps:
+      - name: Récupère le dépôt
+        uses: actions/checkout@v4
+
+      - name: Installe uv
+        uses: astral-sh/setup-uv@v5
+        with:
+          enable-cache: true
+          cache-dependency-glob: apps/backend/uv.lock
+
+      # L'audit porte sur le verrou, pas sur l'environnement : sinon pip-audit auditerait
+      # aussi les paquets que son propre `--with` injecte, hors dépendances du projet.
+      - name: Audite les dépendances livrées
+        # Piège : sans `shell: bash`, un échec de `uv export` serait masqué par le pipe.
+        shell: bash
+        run: uv export --frozen --no-dev --no-emit-project --no-hashes | uvx pip-audit --requirement /dev/stdin --no-deps
@@ -1,5 +1,4 @@
 name: Frontend
-# Pipeline à choix multiple

 on:
  push:
@@ -10,9 +9,9 @@ on:
    paths:
      - "apps/frontend/**"
      - ".github/workflows/frontend.yml"
-      
-# Ordre de lancement des jobs
-# build -> test -> sonarqube -> deploy
+
+permissions:
+  contents: read

 jobs:
  build:
@@ -24,13 +23,25 @@ jobs:
          node-version: 24
          cache: npm
          cache-dependency-path: apps/frontend/package-lock.json
-      
-      - run: npm ci 
+
+      - run: npm ci
        working-directory: apps/frontend
      - run: npm run build
        working-directory: apps/frontend

-  test:  
+  security-audit:
+    name: Audit des dépendances
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v6
+      - uses: actions/setup-node@v6
+        with:
+          node-version: 24
+      # Seuil high : une vulnérabilité moderate de devDependency ne doit pas bloquer une livraison.
+      - run: npm audit --audit-level=high --package-lock-only
+        working-directory: apps/frontend
+
+  test:
    needs: build
    runs-on: ubuntu-latest
    steps:
@@ -40,16 +51,16 @@ jobs:
          node-version: 24
          cache: npm
          cache-dependency-path: apps/frontend/package-lock.json
-      - run: npm ci 
+      - run: npm ci
        working-directory: apps/frontend
-      - run: npm test --watch=false --code-coverage --coverageReporters=lcov
+      - run: npm test -- --watch=false --coverage
        working-directory: apps/frontend
      - name: Upload coverage
        uses: actions/upload-artifact@v4
        with:
          name: frontend-coverage
          path: apps/frontend/coverage/frontend/lcov.info
-        
+
  sonarqube:
    needs: [build, test]
    name: SonarQube
Author	SHA1	Message	Date
Johan LEROY	7be8a44e89	Merge branch 'dev' into chore/coverage-tests-frontend-ci Résolution du conflit sur .github/workflows/frontend.yml : - conserve les étapes upload/download de l'artifact lcov ajoutées sur dev, sans lesquelles le rapport ne quitte pas le runner du job test et n'atteint jamais le scanner Sonar ; - conserve le job security-audit et le retrait du bloc deploy commenté ; - retient la commande de test de la branche. Le séparateur `--` manquait côté dev : npm consommait les trois flags comme sa propre configuration et ng test tournait sans argument. De plus `--code-coverage` n'existe pas dans le builder @angular/build:unit-test, où l'option s'appelle `coverage`.	2026-09-18 12:30:10 +02:00
Johan LEROYandGitHub	2095ad6bd3	Merge pull request #100 from ineszang/chore/audit-dependances-ci chore(ci): ajoute un audit de sécurité des dépendances (npm audit, pi…	2026-09-18 12:26:46 +02:00
Johan LEROY	fb06bf0062	chore(ci): isole l'audit de sécurité et le fait porter sur le verrou Backend / Audit des dépendances (push) Successful in 1m30s Details Frontend / Audit des dépendances (push) Successful in 7s Details Backend / Lint, typage et tests (push) Successful in 4m15s Details Frontend / build (push) Successful in 11m3s Details Frontend / test (push) Failing after 5m7s Details Frontend / SonarQube (push) Skipped Details L'audit backend était la dernière étape du job de vérification : un lint ou un test en échec suffisait à le sauter, et `pip-audit` sans argument auditait l'environnement courant, donc aussi les 28 paquets injectés par son propre `--with`. Il audite maintenant l'export du verrou, dans un job dédié, en symétrie avec le frontend. Côté frontend, `npm audit` lit le verrou et n'a besoin ni de `npm ci` ni du job `build`. Le workflow déclare enfin ses permissions, comme backend.yml et ml.yml.	2026-09-18 12:23:50 +02:00
Johan LEROY	e27142c7db	Merge branch 'dev' into chore/audit-dependances-ci	2026-09-18 12:23:25 +02:00
Valentin	1cca4f130c	chore(ci): ajoute la couverture de tests au job test du frontend Frontend / build (push) Successful in 10m14s Details Frontend / test (push) Failing after 5m1s Details Frontend / SonarQube (push) Skipped Details	2026-09-17 13:55:17 +02:00
Valentin	f5cac1c2a8	chore(ci): ajoute un audit de sécurité des dépendances (npm audit, pip-audit)	2026-09-17 12:25:15 +02:00