Luijaa/web-template

Pré-PR check

Effectuer toutes les étapes, puis rapporter sous forme de tableau Réussi/Échoué avec preuves (commandes et résultats succincts).

1. make check doit réussir entièrement (docs-check, lint + SAST, test, build, license) — sauf 2 cas :

   • PR de documentation uniquement (docs(design): …, ADR, docs) : make docs-check suffit
   • Dépôt sans stack choisie (la cible affiche not configured — ADR-0001 non acceptée) : cette cible est rapportée comme N/A (non configuré), pas comme réussite, et n'est pas un motif d'arrêt — ne pas modifier le recipe pour qu'il exit 0 afin de passer

   et make e2e si ce qui est visible par l'utilisateur change (ajouter/modifier e2e en conséquence) et make dast si l'on touche middleware, en‑tête, cookie, page d'erreur ou nouvelle route

2. git diff origin/main...HEAD --stat — vérifier que seuls les fichiers liés à la tâche sont modifiés, pas de fichiers superflus (sorties de build, .env, fichiers personnels)

3. Logs temporaires oubliés : git diff origin/main...HEAD | grep -n '\[DBG-' ne doit rien trouver (fix-bug étape 3)

4. Recherche de secrets dans le diff : git diff origin/main...HEAD | grep -nEi '(api[_-]?key|secret|token|password|BEGIN .*PRIVATE KEY)' — si trouvé, arrêter et signaler

5. Les nouvelles variables d'environnement sont toutes présentes dans .env.example

6. Dépendances ajoutées/mises à jour (vérifier le diff du lockfile, go.mod, FROM dans Dockerfile) : version et licence conformes à docs/dependencies.md et indiquées dans la description de la PR

7. Nouvelle migration (si elle existe) : exécutable et réversible

8. Documentation : README, docs, ADR (docs/adr/) mis à jour selon les changements ; les nouveaux termes du domaine sont dans CONTEXT.md et les noms dans le code correspondent à CONTEXT.md

9. Revue selon le skill review (Claude Code : sous‑agent reviewer ; autre agent : lire .agents/skills/review/SKILL.md puis vérifier le diff soi‑même) et corriger les points "à corriger" avant

10. Le message de commit est un Conventional Commit et la branche n'est pas main

11. La description de PR contient Closes #<issue> (sans issue, en créer une d'abord) et une section ## CHANGELOG (catégorie + une ligne en thaï du point de vue utilisateur, ou -) — ne pas modifier CHANGELOG.md directement (docs/release.md)

12. Si l'on touche à l'UI : pas de texte en dur, toutes les clés de traduction sont présentes pour chaque langue, pas de couleur en dur, vérifier les modes th/en × light/dark (docs/ui.md)

Si une étape échoue, ne pas ouvrir de PR ; corriger d'abord ou rapporter ce qui bloque.