docs(01): add code review fix report

2026-04-08 15:18:04 +02:00
parent 43356352b3
commit f64a6754c1
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_