Constats non bloquants de la revue de la CI (PR #31) #33

New issue

Closed

opened 2026-09-28 00:51:03 +00:00 by maximus · 0 comments

maximus commented

2026-09-28 00:51:03 +00:00

Owner

Constats non bloquants de la revue de la PR #31 (commentaire 16873), à ne pas perdre avec la PR fermée.

Épingler l'absence de continue-on-error dans .forgejo/workflows/ci.yml. La mutation continue-on-error: true sur une étape ne fait tomber aucun test : une étape en échec passerait au vert en silence. Ajout proposé dans __tests__/ci.test.js : expect(ciCode).not.toMatch(/continue-on-error:/).
Port 3097 fixe dans scripts/boot-from-copy.sh : si un autre serveur l'occupe en local, les vérifications peuvent interroger ce voisin. Sans risque en CI (conteneur neuf) ; BOOT_PORT permet de contourner. Option : échouer si le port répond déjà avant le démarrage.
Le boot check ne sollicite que /health : un module chargé paresseusement par une autre route lui échapperait. Pas le cas aujourd'hui (tous les require en tête de index.js et metrics.js) ; à garder en tête, ou épingler l'absence de require hors du niveau module.

Constats non bloquants de la revue de la PR #31 (commentaire 16873), à ne pas perdre avec la PR fermée. 1. **Épingler l'absence de `continue-on-error`** dans `.forgejo/workflows/ci.yml`. La mutation `continue-on-error: true` sur une étape ne fait tomber aucun test : une étape en échec passerait au vert en silence. Ajout proposé dans `__tests__/ci.test.js` : `expect(ciCode).not.toMatch(/continue-on-error:/)`. 2. **Port 3097 fixe** dans `scripts/boot-from-copy.sh` : si un autre serveur l'occupe en local, les vérifications peuvent interroger ce voisin. Sans risque en CI (conteneur neuf) ; `BOOT_PORT` permet de contourner. Option : échouer si le port répond déjà avant le démarrage. 3. **Le boot check ne sollicite que `/health`** : un module chargé paresseusement par une autre route lui échapperait. Pas le cas aujourd'hui (tous les `require` en tête de `index.js` et `metrics.js`) ; à garder en tête, ou épingler l'absence de `require` hors du niveau module.