fix: correction duplication navigation - détection et suppression des doublons structurels
- 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.
This commit is contained in:
syoul
@@ -136,6 +136,64 @@ try:
|
|||||||
with open(f, 'r', encoding='utf-8') as file:
|
with open(f, 'r', encoding='utf-8') as file:
|
||||||
content = file.read()
|
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)
|
# É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...")
|
print("🧹 Nettoyage de tous les liens Équipe existants...")
|
||||||
lines = content.splitlines(keepends=True)
|
lines = content.splitlines(keepends=True)
|
||||||
@@ -255,6 +313,55 @@ if python3 /tmp/add_team_link.py; then
|
|||||||
head -50 "$NAV_FILE" || true
|
head -50 "$NAV_FILE" || true
|
||||||
exit 1
|
exit 1
|
||||||
fi
|
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
|
else
|
||||||
echo "❌ Erreur lors de l'exécution du script Python"
|
echo "❌ Erreur lors de l'exécution du script Python"
|
||||||
exit 1
|
exit 1
|
||||||
@@ -302,7 +409,30 @@ RUN if [ -d "out" ]; then \
|
|||||||
fi && \
|
fi && \
|
||||||
echo "" && \
|
echo "" && \
|
||||||
echo "📋 Vérification finale de Navigation.tsx après build:" && \
|
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
|
WORKDIR /app
|
||||||
|
|
||||||
# Exposition du port interne
|
# Exposition du port interne
|
||||||
|
|||||||
Reference in New Issue
Block a user