2.4 KiB
phase, fixed_at, review_path, iteration, findings_in_scope, fixed, skipped, status
| phase | fixed_at | review_path | iteration | findings_in_scope | fixed | skipped | status |
|---|---|---|---|---|---|---|---|
| 01-foundation | 2026-04-08T00:00:00Z | .planning/phases/01-foundation/01-REVIEW.md | 1 | 5 | 5 | 0 | all_fixed |
Phase 01 : Rapport de correction de revue de code
Corrige le : 2026-04-08 Revue source : .planning/phases/01-foundation/01-REVIEW.md Iteration : 1
Resume :
- Findings en scope : 5
- Corriges : 5
- Ignores : 0
Corrections appliquees
CR-01 : Identifiant Google Analytics hardcoded dans le depot
Fichiers modifies : nuxt.config.ts, .env.example
Commit : 184e125
Correction appliquee : Remplace l'ID gtag hardcode par une variable d'environnement via runtimeConfig.public.gtag.id. Le champ gtag.id est vide par defaut et peuple via NUXT_PUBLIC_GTAG_ID. Active uniquement en production. Cree .env.example avec la variable documentee.
WR-01 : Configuration i18n incomplete
Fichiers modifies : nuxt.config.ts, app/locales/fr.json, app/locales/en.json
Commit : c6744ab
Correction appliquee : Ajout de strategy: 'prefix_except_default', langDir: 'locales/', objets locales complets avec language et file, et detectBrowserLanguage avec persistance cookie uniquement. Cree des fichiers placeholder fr.json et en.json vides pour eviter les erreurs du module.
WR-02 : Fuite silencieuse de cle i18n dans useProjects
Fichiers modifies : app/composables/useProjects.ts
Commit : 7d81d47
Correction appliquee : Remplace t(...) || undefined par te(...) (translation exists) suivi de t(...) pour detecter correctement les cles manquantes au lieu de retourner la cle brute comme valeur.
WR-03 : Bootstrap et Tailwind CSS mal classes dans database
Fichiers modifies : app/data/techstack.ts
Commit : 89ce718
Correction appliquee : Deplace Bootstrap et Tailwind CSS du tableau database vers le tableau front ou ils appartiennent en tant que frameworks CSS/UI.
WR-04 : Attribut lang absent sur l'element racine HTML
Fichiers modifies : app/app.vue
Commit : 4335635
Correction appliquee : Ajout d'un bloc <script setup> avec useI18n() et useHead({ htmlAttrs: { lang: locale } }) pour injecter dynamiquement l'attribut lang sur <html> en SSR.
Corrections ignorees
Aucune -- toutes les corrections ont ete appliquees avec succes.
Corrige le : 2026-04-08 Fixer : Claude (gsd-code-fixer) Iteration : 1