Refactor should_exclude_file function in ArkConfigManager and WorkspaceAdvancedManipulation

Author: raidos23

Core/ArkConfigManager.py

@@ -31,8 +31,9 @@
 """
 
 import os
-from pathlib import Path
-from typing import Any
+import fnmatch
+from pathlib import Path, PurePosixPath
+from typing import Any, Optional
 import yaml
 
 
@@ -262,15 +263,35 @@ def get_environment_manager_options(config: dict[str, Any]) -> dict[str, Any]:
     return config.get("environment_manager", {})
 
 
+def _normalize_exclusion_pattern(pattern: str) -> str:
+    """
+    Normalise un pattern d'exclusion pour un matching fiable cross-plateforme.
+
+    - Convertit les séparateurs Windows en "/"
+    - Retire le préfixe "./"
+    - Si le pattern se termine par "/", on l'étend à "/**"
+    """
+    p = str(pattern).strip()
+    if not p:
+        return ""
+    p = p.replace("\\", "/")
+    if p.startswith("./"):
+        p = p[2:]
+    if p.endswith("/"):
+        p = p.rstrip("/") + "/**"
+    return p
+
+
 def should_exclude_file(
-    file_path: str, workspace_dir: str, exclusion_patterns: list[str]
+    file_path: str, workspace_dir: str, exclusion_patterns: Optional[list[str]]
 ) -> bool:
     """
     Détermine si un fichier doit être exclu de la compilation.
 
     Cette fonction compare le chemin du fichier avec les patterns
-    d'exclusion définis dans la configuration. Elle utilise la méthode
-    Path.match() qui supporte les patterns glob standards.
+    d'exclusion définis dans la configuration. Elle normalise les
+    séparateurs puis combine Path.match() et fnmatch pour supporter
+    les patterns glob avec "**" de façon cross-plateforme.
 
     Args:
         file_path: Chemin absolu du fichier à vérifier
@@ -281,23 +302,59 @@ def should_exclude_file(
         True si le fichier doit être exclu, False sinon
     """
     try:
+        if not file_path or not workspace_dir:
+            return False
+
         file_path_obj = Path(file_path)
         workspace_path_obj = Path(workspace_dir)
 
+        # Résolution pour éviter les surprises (symlinks, chemins relatifs)
+        try:
+            file_abs = file_path_obj.resolve()
+            workspace_abs = workspace_path_obj.resolve()
+        except Exception:
+            file_abs = file_path_obj
+            workspace_abs = workspace_path_obj
+
         # Calcul du chemin relatif par rapport au workspace
         try:
-            relative_path = file_path_obj.relative_to(workspace_path_obj)
+            relative_path = file_abs.relative_to(workspace_abs)
         except ValueError:
             # Le fichier est hors du workspace
             return True
 
+        # Normalisation vers POSIX pour matcher les patterns avec "/"
+        rel_posix = PurePosixPath(relative_path.as_posix())
+        abs_posix = PurePosixPath(file_abs.as_posix())
+        rel_str = rel_posix.as_posix()
+        abs_str = abs_posix.as_posix()
+        file_name = file_abs.name
+
+        patterns = exclusion_patterns or []
+
         # Vérification des patterns d'exclusion
-        for pattern in exclusion_patterns:
-            # Comparaison avec le chemin relatif complet
-            if relative_path.match(pattern):
-                return True
+        for pattern in patterns:
+            pat = _normalize_exclusion_pattern(pattern)
+            if not pat:
+                continue
+
+            # Patterns avec "**" : matcher via fnmatch pour supporter les répertoires
+            if "**" in pat:
+                if fnmatch.fnmatch(rel_str, pat):
+                    return True
+                if fnmatch.fnmatch(abs_str, pat):
+                    return True
+            else:
+                # Comparaison avec le chemin relatif complet
+                if rel_posix.match(pat):
+                    return True
+
+                # Autoriser les patterns absolus si fournis par l'utilisateur
+                if abs_posix.match(pat):
+                    return True
+
             # Comparaison avec juste le nom du fichier (pour patterns comme "*.pyc")
-            if file_path_obj.match(pattern):
+            if "/" not in pat and PurePosixPath(file_name).match(pat):
                 return True
 
         return False

Core/WorkSpaceManager/WorkspaceAdvancedManipulation.py

@@ -19,6 +19,7 @@
 from PySide6.QtWidgets import QFileDialog, QMessageBox
 
 from Core.Globals import _workspace_dir_cache, _workspace_dir_lock
+from Core.ArkConfigManager import load_ark_config, should_exclude_file
 
 
 class WorkspaceAdvancedManipulation:
@@ -138,14 +139,17 @@ def handle_drop_event(gui_instance, event: QDropEvent):
 
         urls = event.mimeData().urls()
         added = 0
+        excluded = 0
+        workspace_dir = getattr(gui_instance, "workspace_dir", None)
+        ark_config = load_ark_config(workspace_dir) if workspace_dir else {}
+        exclusion_patterns = ark_config.get("exclusion_patterns", [])
 
         for url in urls:
             path = url.toLocalFile()
             if os.path.isdir(path):
                 added += SetupWorkspace.add_py_files_from_folder(gui_instance, path)
             elif path.endswith(".py"):
                 # Vérifie que le fichier est dans workspace (si défini)
-                workspace_dir = getattr(gui_instance, "workspace_dir", None)
                 if (
                     workspace_dir
                     and not os.path.commonpath([path, workspace_dir]) == workspace_dir
@@ -155,6 +159,12 @@ def handle_drop_event(gui_instance, event: QDropEvent):
                         f"⚠️ Ignored (outside workspace): {path}",
                     )
                     continue
+                # Vérifie les patterns d'exclusion ARK
+                if workspace_dir and should_exclude_file(
+                    path, workspace_dir, exclusion_patterns
+                ):
+                    excluded += 1
+                    continue
                 if path not in gui_instance.python_files:
                     gui_instance.python_files.append(path)
                     relative_path = (
@@ -168,6 +178,11 @@ def handle_drop_event(gui_instance, event: QDropEvent):
             f"✅ {added} fichier(s) ajouté(s) via drag & drop.",
             f"✅ {added} file(s) added via drag & drop.",
         )
+        if excluded > 0:
+            gui_instance.log_i18n(
+                f"⏩ Exclusion appliquée : {excluded} fichier(s) ignoré(s) (ARK_Main_Config.yml).",
+                f"⏩ Exclusion applied: {excluded} file(s) ignored (ARK_Main_Config.yml).",
+            )
 
         if hasattr(gui_instance, "update_command_preview"):
             gui_instance.update_command_preview()