docs(01): add code review fix report

2026-04-08 15:18:04 +02:00
parent 427270f972
commit 1b54d4463f
1 changed files with 63 additions and 0 deletions
@@ -0,0 +1,63 @@
 ---
 phase: 01-foundation
 fixed_at: 2026-04-08T00:00:00Z
 review_path: .planning/phases/01-foundation/01-REVIEW.md
 iteration: 1
 findings_in_scope: 5
 fixed: 5
 skipped: 0
 status: 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_