From 376e82570f8d2440359877fa5cba1debde0ae7b3 Mon Sep 17 00:00:00 2001
From: syoul <syoul@librezo.fr>
Date: Tue, 9 Dec 2025 11:17:04 +0100
Subject: [PATCH] =?UTF-8?q?fix:=20correction=20duplication=20navigation=20?=
 =?UTF-8?q?-=20d=C3=A9tection=20et=20suppression=20des=20doublons=20struct?=
 =?UTF-8?q?urels?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Script Python amélioré pour détecter et supprimer les doublons de composant Navigation
- Vérifications post-modification dans le script shell (fonctions, nav, ul, logo, export)
- Vérification post-build pour détecter les doublons dans le HTML généré
- Détection automatique des composants Navigation dupliqués
- Arrêt du build si duplication détectée

Le script détecte maintenant :
- Duplication du composant Navigation (plusieurs fonctions)
- Duplication structurelle (nav, ul, logo)
- Duplication dans le HTML généré (headers/nav)

Toutes les vérifications s'exécutent automatiquement lors du build Docker.
---
 Dockerfile.business | 132 +++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 131 insertions(+), 1 deletion(-)

diff --git a/Dockerfile.business b/Dockerfile.business
index 9048c4b..77d8381 100644
--- a/Dockerfile.business
+++ b/Dockerfile.business
@@ -136,6 +136,64 @@ try:
     with open(f, 'r', encoding='utf-8') as file:
         content = file.read()
 
+    # VÉRIFICATION PRÉLIMINAIRE: Détecter les doublons structurels
+    print("🔍 Vérification de la structure du composant...")
+    
+    # Compter les éléments structurels
+    nav_patterns = [
+        r'<nav\s',
+        r'className.*[Nn]avigation',
+        r'export\s+(default\s+)?function\s+Navigation',
+        r'const\s+Navigation\s*='
+    ]
+    nav_count = sum(len(re.findall(pattern, content)) for pattern in nav_patterns)
+    
+    # Compter les <ul> (peut y en avoir 2 pour mobile/desktop)
+    ul_count = len(re.findall(r'<ul[^>]*>', content))
+    
+    # Compter les logos (Image avec logo ou logoFile)
+    logo_patterns = [
+        r'logoFile',
+        r'logo\.svg',
+        r'[Ll]ogo',
+        r'Image.*logo'
+    ]
+    logo_count = sum(len(re.findall(pattern, content)) for pattern in logo_patterns)
+    
+    # Compter les fonctions Navigation
+    function_count = len(re.findall(r'(export\s+(default\s+)?function\s+Navigation|const\s+Navigation\s*=\s*\(|function\s+Navigation\s*\()', content))
+    
+    print(f"📊 Structure détectée: {nav_count} nav, {ul_count} ul, {logo_count} logo, {function_count} fonction(s)")
+    
+    # Détecter les duplications structurelles
+    if function_count > 1:
+        print(f"⚠️ ATTENTION: {function_count} fonction(s) Navigation détectée(s) - possible duplication du composant", file=sys.stderr)
+        # Extraire uniquement la première fonction Navigation
+        matches = list(re.finditer(r'(export\s+(default\s+)?function\s+Navigation|const\s+Navigation\s*=\s*\(|function\s+Navigation\s*\()', content))
+        if len(matches) > 1:
+            # Garder seulement jusqu'à la fin de la première fonction
+            first_end = matches[1].start() if len(matches) > 1 else len(content)
+            content = content[:first_end]
+            # Trouver la fin de la fonction (dernière accolade fermante avant la prochaine fonction)
+            brace_count = 0
+            in_function = False
+            for i, char in enumerate(content[matches[0].start():], matches[0].start()):
+                if char == '{':
+                    brace_count += 1
+                    in_function = True
+                elif char == '}':
+                    brace_count -= 1
+                    if in_function and brace_count == 0:
+                        content = content[:i+1]
+                        break
+            print(f"✅ Duplication du composant supprimée")
+    
+    if ul_count > 3:  # Plus de 3 ul suggère une duplication
+        print(f"⚠️ ATTENTION: {ul_count} éléments <ul> détectés - possible duplication", file=sys.stderr)
+    
+    if logo_count > 5:  # Plus de 5 références au logo suggère une duplication
+        print(f"⚠️ ATTENTION: {logo_count} références au logo détectées - possible duplication", file=sys.stderr)
+
     # ÉTAPE 1: Supprimer TOUS les liens Équipe existants (même s'il n'y en a qu'un)
     print("🧹 Nettoyage de tous les liens Équipe existants...")
     lines = content.splitlines(keepends=True)
@@ -255,6 +313,55 @@ if python3 /tmp/add_team_link.py; then
     head -50 "$NAV_FILE" || true
     exit 1
   fi
+  
+  # VÉRIFICATIONS POST-MODIFICATION: Détecter les doublons structurels
+  echo "🔍 Vérification des doublons structurels..."
+  
+  # Compter les fonctions Navigation
+  function_count=$(grep -cE '(export\s+(default\s+)?function\s+Navigation|const\s+Navigation\s*=\s*\(|function\s+Navigation\s*\()' "$NAV_FILE" 2>/dev/null || echo "0")
+  if [ "$function_count" -gt "1" ]; then
+    echo "❌ ERREUR: $function_count fonction(s) Navigation détectée(s) - duplication du composant!"
+    echo "📄 Recherche des fonctions Navigation:"
+    grep -nE '(export\s+(default\s+)?function\s+Navigation|const\s+Navigation\s*=\s*\(|function\s+Navigation\s*\()' "$NAV_FILE" || true
+    exit 1
+  else
+    echo "✅ Composant Navigation unique ($function_count fonction)"
+  fi
+  
+  # Compter les éléments <nav> ou className Navigation
+  nav_count=$(grep -cE '<nav\s|className.*[Nn]avigation' "$NAV_FILE" 2>/dev/null || echo "0")
+  if [ "$nav_count" -gt "2" ]; then
+    echo "⚠️ ATTENTION: $nav_count éléments nav détectés (attendu: 1-2)"
+  else
+    echo "✅ Structure nav correcte ($nav_count élément(s))"
+  fi
+  
+  # Compter les éléments <ul>
+  ul_count=$(grep -c '<ul' "$NAV_FILE" 2>/dev/null || echo "0")
+  if [ "$ul_count" -gt "3" ]; then
+    echo "⚠️ ATTENTION: $ul_count éléments <ul> détectés (attendu: 1-3 pour mobile/desktop)"
+  else
+    echo "✅ Structure ul correcte ($ul_count élément(s))"
+  fi
+  
+  # Compter les références au logo
+  logo_count=$(grep -cE 'logoFile|logo\.svg|[Ll]ogo' "$NAV_FILE" 2>/dev/null || echo "0")
+  if [ "$logo_count" -gt "5" ]; then
+    echo "⚠️ ATTENTION: $logo_count références au logo détectées (possible duplication)"
+  else
+    echo "✅ Références logo correctes ($logo_count référence(s))"
+  fi
+  
+  # Vérifier qu'il n'y a qu'un seul export default
+  export_count=$(grep -c 'export default' "$NAV_FILE" 2>/dev/null || echo "0")
+  if [ "$export_count" -gt "1" ]; then
+    echo "❌ ERREUR: $export_count export default détectés - duplication du composant!"
+    exit 1
+  else
+    echo "✅ Export unique ($export_count export default)"
+  fi
+  
+  echo "✅ Toutes les vérifications structurelles passées"
 else
   echo "❌ Erreur lors de l'exécution du script Python"
   exit 1
@@ -302,7 +409,30 @@ RUN if [ -d "out" ]; then \
     fi && \
     echo "" && \
     echo "📋 Vérification finale de Navigation.tsx après build:" && \
-    grep -q 'href="/team"' src/components/Navigation/Navigation.tsx && echo "✅ Lien Équipe toujours présent dans Navigation.tsx après build" || echo "❌ Lien Équipe absent de Navigation.tsx après build"
+    grep -q 'href="/team"' src/components/Navigation/Navigation.tsx && echo "✅ Lien Équipe toujours présent dans Navigation.tsx après build" || echo "❌ Lien Équipe absent de Navigation.tsx après build" && \
+    echo "" && \
+    echo "🔍 Vérification des doublons dans le HTML généré..." && \
+    if [ -f "out/index.html" ]; then \
+      header_count=$(grep -oE '<header|<nav[^>]*>' out/index.html | wc -l | tr -d ' '); \
+      nav_count=$(grep -oE '<nav[^>]*>' out/index.html | wc -l | tr -d ' '); \
+      logo_count=$(grep -oE 'logo\.svg|logoFile|CoeurBox' out/index.html | wc -l | tr -d ' '); \
+      echo "📊 HTML généré: $header_count header/nav, $nav_count nav, $logo_count logo"; \
+      if [ "$header_count" -gt "2" ] || [ "$nav_count" -gt "2" ]; then \
+        echo "❌ ERREUR: Duplication détectée dans le HTML généré!"; \
+        echo "📄 Recherche des headers/nav dans index.html:"; \
+        grep -nE '<header|<nav' out/index.html | head -10 || true; \
+        exit 1; \
+      else \
+        echo "✅ HTML généré correct (pas de duplication structurelle)"; \
+      fi; \
+      if [ "$logo_count" -gt "5" ]; then \
+        echo "⚠️ ATTENTION: $logo_count références au logo dans le HTML (possible duplication)"; \
+      else \
+        echo "✅ Références logo correctes dans le HTML"; \
+      fi; \
+    else \
+      echo "⚠️ out/index.html non trouvé, vérification HTML ignorée"; \
+    fi
 WORKDIR /app
 
 # Exposition du port interne