diff --git a/.planning/phases/01-foundation/01-REVIEW-FIX.md b/.planning/phases/01-foundation/01-REVIEW-FIX.md new file mode 100644 index 0000000..3a73192 --- /dev/null +++ b/.planning/phases/01-foundation/01-REVIEW-FIX.md @@ -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 `