Finalise le lot P0 de parité IDE/classic et durcit deps_analyser

raidos23 · raidos23 · commit 03b37b845efe · 2026-03-30T19:04:28.000Z
Extrait un parseur d'imports testable dans Core/deps_analyser/analyser.py via des helpers dédiés pour l'analyse de source et de fichiers, puis réutilise ce chemin unique dans suggest_missing_dependencies(). Cette refactorisation évite la duplication de logique, simplifie les futurs durcissements de l'analyseur et rend les règles d'extraction d'imports vérifiables isolément.

Étend la couverture des tests heuristiques de l'analyseur pour inclure explicitement les imports relatifs et les imports dynamiques (__import__ et importlib.import_module), y compris lorsque la résolution dépend du contexte du workspace et de layouts de projet tels que src/ ou lib/python.

Finalise la parité fonctionnelle entre l'interface IDE-like et l'interface classique sur les points encore ouverts côté affordances IDE. Le menu d'actions supplémentaires (...) est désormais retraduit proprement, ses actions sont conservées comme objets réutilisables et ses libellés ainsi que certains tooltips spécifiques à l'IDE sont rafraîchis lors des changements de langue afin de limiter les écarts visibles avec le GUI classique.

Ajoute une documentation dédiée de matrice de parité IDE/classique afin de tracer explicitement ce qui est aligné, ce qui reste volontairement spécifique au mode IDE et la stratégie de maintenance recommandée pour les évolutions futures.

Met à jour TODO.md en cochant les tâches P0 réalisées, notamment la finalisation de la parité IDE/classic, la documentation de cette parité et le durcissement de l'analyseur de dépendances autour d'un parseur d'imports testable avec meilleure couverture des imports relatifs et dynamiques.
diff --git a/Core/IdeLikeGui/connections.py b/Core/IdeLikeGui/connections.py
@@ -307,80 +307,88 @@ def _tr(fr: str, en: str) -> str:
 
     try:
         menu = QMenu(more_btn)
+        self._ide_more_tools_menu = menu
 
-        act_workspace = QAction(_tr("Selectionner workspace", "Select workspace"), menu)
+        act_workspace = QAction(menu)
         act_workspace.triggered.connect(
             lambda: getattr(self, "select_workspace", lambda: None)()
         )
         menu.addAction(act_workspace)
 
-        act_venv = QAction(_tr("Selectionner venv", "Select venv"), menu)
+        act_venv = QAction(menu)
         act_venv.triggered.connect(
             lambda: getattr(self, "select_venv_manually", lambda: None)()
         )
         menu.addAction(act_venv)
 
-        act_add_files = QAction(_tr("Ajouter fichiers", "Add files"), menu)
+        act_add_files = QAction(menu)
         act_add_files.triggered.connect(
             lambda: getattr(self, "select_files_manually", lambda: None)()
         )
         menu.addAction(act_add_files)
 
-        act_clear_workspace = QAction(_tr("Vider workspace", "Clear workspace"), menu)
+        act_clear_workspace = QAction(menu)
         act_clear_workspace.triggered.connect(
             lambda: getattr(self, "clear_workspace", lambda: None)()
         )
         menu.addAction(act_clear_workspace)
 
-        act_stats = QAction(_tr("Statistiques", "Statistics"), menu)
+        act_stats = QAction(menu)
         act_stats.triggered.connect(
             lambda: getattr(self, "show_statistics", lambda: None)()
         )
         menu.addAction(act_stats)
 
         menu.addSeparator()
 
-        act_language = QAction(_tr("Langue", "Language"), menu)
-        act_language.triggered.connect(
-            lambda: getattr(self, "show_language_dialog", lambda: None)()
-        )
+        act_language = QAction(menu)
+        act_language.triggered.connect(lambda: getattr(self, "show_language_dialog", lambda: None)())
         menu.addAction(act_language)
 
-        act_theme = QAction(_tr("Theme", "Theme"), menu)
+        act_theme = QAction(menu)
         act_theme.triggered.connect(lambda: _open_theme_dialog(self))
         menu.addAction(act_theme)
 
         menu.addSeparator()
 
-        act_advanced = QAction(_tr("Config avancee", "Advanced config"), menu)
+        act_advanced = QAction(menu)
         act_advanced.triggered.connect(
             lambda: getattr(self, "open_advanced_config_editor", lambda: None)()
         )
         menu.addAction(act_advanced)
 
-        act_export = QAction(_tr("Exporter config", "Export config"), menu)
-        act_export.triggered.connect(
-            lambda: getattr(self, "export_config", lambda: None)()
-        )
+        act_export = QAction(menu)
+        act_export.triggered.connect(lambda: getattr(self, "export_config", lambda: None)())
         menu.addAction(act_export)
 
-        act_import = QAction(_tr("Importer config", "Import config"), menu)
-        act_import.triggered.connect(
-            lambda: getattr(self, "import_config", lambda: None)()
-        )
+        act_import = QAction(menu)
+        act_import.triggered.connect(lambda: getattr(self, "import_config", lambda: None)())
         menu.addAction(act_import)
 
-        act_help = QAction(_tr("Aide", "Help"), menu)
-        act_help.triggered.connect(
-            lambda: getattr(self, "show_help_dialog", lambda: None)()
-        )
+        act_help = QAction(menu)
+        act_help.triggered.connect(lambda: getattr(self, "show_help_dialog", lambda: None)())
         menu.addAction(act_help)
 
+        self._ide_more_menu_actions = {
+            "workspace": act_workspace,
+            "venv": act_venv,
+            "add_files": act_add_files,
+            "clear_workspace": act_clear_workspace,
+            "stats": act_stats,
+            "language": act_language,
+            "theme": act_theme,
+            "advanced": act_advanced,
+            "export": act_export,
+            "import": act_import,
+            "help": act_help,
+        }
+
         more_btn.setMenu(menu)
         more_btn.setPopupMode(QToolButton.InstantPopup)
         more_btn.setToolTip(_tr("Plus d'actions", "More actions"))
     except Exception:
         pass
+    _retranslate_ide_like_actions(self)
     _apply_activity_buttons_theme(self)
 
     # Avoid duplicated controls in the side panel.
@@ -488,17 +496,63 @@ def _apply_activity_buttons_theme(self) -> None:
     _apply_status_bar_theme(self, dark, fg, border)
 
 
-def _connect_ide_like_specific_signals(self) -> None:
-    """Connect only IDE-specific signals on top of the classic shared wiring."""
+def _retranslate_ide_like_actions(self) -> None:
+    """Apply translated labels/tooltips to IDE-specific actions and affordances."""
+    try:
+        trf = getattr(self, "tr", None)
+        _tr = trf if callable(trf) else (lambda fr, en: en)
+    except Exception:
+        _tr = lambda fr, en: en
+
+    actions = getattr(self, "_ide_more_menu_actions", {}) or {}
+    labels = {
+        "workspace": _tr("Choisir le Workspace", "Select workspace"),
+        "venv": _tr("Choisir le Venv", "Select venv"),
+        "add_files": _tr("Ajouter des fichiers", "Add files"),
+        "clear_workspace": _tr("Vider le Workspace", "Clear workspace"),
+        "stats": _tr("Statistiques", "Statistics"),
+        "language": _tr("Langue", "Language"),
+        "theme": _tr("Thème", "Theme"),
+        "advanced": _tr("Configuration avancée", "Advanced config"),
+        "export": _tr("Exporter la configuration", "Export config"),
+        "import": _tr("Importer la configuration", "Import config"),
+        "help": _tr("Aide", "Help"),
+    }
+    for key, action in actions.items():
+        if action is None:
+            continue
+        try:
+            action.setText(labels.get(key, action.text()))
+        except Exception:
+            pass
 
-        _connect_classic_signals(self)
+    try:
+        more_btn = getattr(self, "toolButton_more", None)
+        if more_btn is not None:
+            more_btn.setToolTip(_tr("Plus d'actions", "More actions"))
     except Exception:
         pass
-
-    # IDE activity-bar shortcut for dependency analysis.
     try:
-        if getattr(self, "activity_btn_deps", None):
-            self.activity_btn_deps.clicked.connect(self.suggest_missing_dependencies)
+        deps_btn = getattr(self, "activity_btn_deps", None)
+        if deps_btn is not None:
+            deps_btn.setToolTip(
+                _tr("Analyser les dependances", "Analyze dependencies")
+            )
+    except Exception:
+        pass
+    try:
+        if hasattr(self, "register_language_refresh"):
+            if not getattr(self, "_ide_menu_i18n_registered", False):
+                self.register_language_refresh(lambda: _retranslate_ide_like_actions(self))
+                self._ide_menu_i18n_registered = True
+    except Exception:
+        pass
+
+
+def _connect_ide_like_specific_signals(self) -> None:
+    """Connect only IDE-specific signals on top of the classic shared wiring."""
+
+        _connect_classic_signals(self)
     except Exception:
         pass
 
diff --git a/Core/deps_analyser/analyser.py b/Core/deps_analyser/analyser.py
@@ -15,6 +15,7 @@
 
 import functools
 import configparser
+import ast
 import json
 import os
 import platform
@@ -448,6 +449,62 @@ def _resolve_relative_import_root(
     return anchor_parts[0] if anchor_parts else ""
 
 
+def _extract_imported_modules_from_source(
+    source: str, file_path: str = "", workspace_dir: str | None = None
+) -> set[str]:
+    """Parse Python source and return normalized top-level imported modules."""
+    modules: set[str] = set()
+    try:
+        tree = ast.parse(source, filename=file_path or "<memory>")
+    except Exception:
+        return modules
+
+    for node in ast.walk(tree):
+        if isinstance(node, ast.Import):
+            for alias in node.names:
+                top = _top_level_module_name(alias.name)
+                if top:
+                    modules.add(top)
+        elif isinstance(node, ast.ImportFrom):
+            if node.level and node.level > 0:
+                rel_root = _resolve_relative_import_root(file_path, node.level, workspace_dir)
+                if rel_root:
+                    modules.add(rel_root)
+            elif node.module:
+                top = _top_level_module_name(node.module)
+                if top:
+                    modules.add(top)
+
+    dynamic_imports = re.findall(r"__import__\(['\"]([\w\.]+)['\"]\)", source)
+    modules.update(
+        [top for top in (_top_level_module_name(mod) for mod in dynamic_imports) if top]
+    )
+
+    importlib_imports = re.findall(
+        r"importlib\.import_module\(['\"]([\w\.]+)['\"]\)", source
+    )
+    modules.update(
+        [top for top in (_top_level_module_name(mod) for mod in importlib_imports) if top]
+    )
+    return modules
+
+
+def _extract_imported_modules_from_file(
+    file_path: str, workspace_dir: str | None = None
+) -> set[str]:
+    """Read a Python file and return normalized imported modules."""
+    try:
+        with open(file_path, encoding="utf-8") as f:
+            source = f.read()
+    except Exception:
+        return set()
+    return _extract_imported_modules_from_source(
+        source,
+        file_path=file_path,
+        workspace_dir=workspace_dir,
+    )
+
+
 def _collect_workspace_module_roots(
     filtered_files: list[str], workspace_dir: str | None
 ) -> set[str]:
@@ -783,38 +840,11 @@ def _t(_key: str, fr: str, en: str) -> str:
             if idx - last_pump >= 50:
                 QApplication.processEvents()
                 last_pump = idx
-
-            with open(file, encoding="utf-8") as f:
-                source = f.read()
-                tree = ast.parse(source, filename=file)
-            # Imports classiques (import ... / from ... import ...)
-            for node in ast.walk(tree):
-                if isinstance(node, ast.Import):
-                    for alias in node.names:
-                        top = _top_level_module_name(alias.name)
-                        if top:
-                            modules.add(top)
-                elif isinstance(node, ast.ImportFrom):
-                    if node.level and node.level > 0:
-                        rel_root = _resolve_relative_import_root(
-                            file, node.level, getattr(self, "workspace_dir", None)
-                        )
-                        if rel_root:
-                            modules.add(rel_root)
-                    elif node.module:
-                        top = _top_level_module_name(node.module)
-                        if top:
-                            modules.add(top)
-            # Imports dynamiques via __import__ ou importlib.import_module
-            dynamic_imports = re.findall(r"__import__\(['\"]([\w\.]+)['\"]\)", source)
             modules.update(
-                [top for top in (_top_level_module_name(mod) for mod in dynamic_imports) if top]
-            )
-            importlib_imports = re.findall(
-                r"importlib\.import_module\(['\"]([\w\.]+)['\"]\)", source
-            )
-            modules.update(
-                [top for top in (_top_level_module_name(mod) for mod in importlib_imports) if top]
+                _extract_imported_modules_from_file(
+                    file,
+                    workspace_dir=getattr(self, "workspace_dir", None),
+                )
             )
         except Exception as e:
             _log_append(self, f"⚠️ Erreur analyse dépendances dans {file} : {e}")
diff --git a/TODO.md b/TODO.md
@@ -43,6 +43,28 @@
 - [x] Ajouter une doc architecture Core/Engine/UI.
 - [x] Ajouter un guide contributeur (branches, commits, release flow).
 
-### Backlog historique
-- [x] Doc engines: preciser que la scroll area n'est pas necessaire en UI monolithique (gestion auto par l'UI).
-- [x] Venv manager: confirmer et tester le comportement `--break-system-packages` sous Linux system Python.
+- [x] Bcasl doit se lancer avan la compilation.
+
+## i18n 
+
+- [x] Des error d'ecirture sont present dans languages/** pour les fichier de i18n json. le probleme vient du fait que certaine chose sont mal orthographier.
+
+## Engines
+
+- [x] les engines sans exception presente tous un probleme de design au niveau du bouton de choix d'icon. En effet le button est un simple button donc on ne sais pas si licon est bien selectionner alors il serait pertinent dajoutetr un label a coter qui montre le chemin de licon selectionner ce qui peut aussi permetrttre un  parametrage a la foix ux/ui et manuel.
+ ## Docs
+
+ - [ ] il faut preciser dans la docs for engine a la section de monolitics ui que pas la peine de faire le scroll area cr l'ui s'en charge automatiquement dès qu-il voit que le tab est trop large.
+
+## venvmanager
+
+- [ ] il faut que sur linux si on utilise le python of system que les installation utilise le flag break system package pour s'assurer que l-instalation aura lieu coute que coute. sur windows cela nest pas necessaire  car windwos n-a pas souvent de problme niveau resolution de package.
+
+## ide gui 
+- [x] donner toute les capacité gui de classic gui à ide gui.
+- [x] pour les installations d'outils des engines ou meme des plugins en generale les action d'installation systeme et de python rentre en conflit le plus souvent alors lors de linstallation on dit dabord prioriser linstaltion systme avant meme de lancer les commande pour l'installation de type python. pour les boite de dialog qui saffiche sepcialemnt pour cela il faut un time out de taping de secret (mot de passe)  de l'user genre 120s si le delai est passer on annule linstallation system proprement et selon notrer regle si un processus du genre là est tuer tout les autre non plus le droit de sexecuter il sont comme des process enfant une fois que le processus premier est tuer on anneanti le reste pour securiser le system et surtout les projet des utilisateurs. au dela de cela il faut que l'on ameliore le venvnager car il doit suporter tous les types de venv a savvir .env env venv .venv etc et tout les non bizarre posible dans l'industrie. 
+
+## P0
+
+- [x] finaliser la parite IDE/classic et documenter la matrice de parite.
+- [x] durcir l'analyseur deps avec un parseur d'imports testable et une meilleure couverture des imports relatifs et dynamiques.
diff --git a/docs/ide_classic_parity.md b/docs/ide_classic_parity.md
@@ -0,0 +1,24 @@
+# IDE-Like vs Classic GUI Parity
+
+This matrix tracks the functional parity between the classic GUI and the IDE-like GUI.
+It focuses on shared user-facing behaviors and the remaining IDE-specific deltas.
+
+| Capability | Classic GUI | IDE-like GUI | Status | Notes |
+|---|---|---|---|---|
+| Shared core signal wiring | Yes | Yes | Done | IDE-like now reuses `UiConnection._connect_signals()`. |
+| Workspace / venv / files actions | Yes | Yes | Done | Routed through the same shared handlers. |
+| Compilation / cancel actions | Yes | Yes | Done | Uses the common signal connector. |
+| Dependencies analysis action | Yes | Yes | Done | Shared button plus IDE activity button. |
+| App icon button wiring | Yes | Yes | Done | Explicit IDE binding for `btn_select_icon`. |
+| Nuitka icon button wiring | Yes | Yes | Done | Explicit IDE binding for `btn_nuitka_icon`. |
+| Entrypoint selector | Yes | Yes | Done | `setup_entrypoint_selector()` is called during IDE init. |
+| Theme and language dialogs | Yes | Yes | Done | IDE keeps dedicated affordances on top of shared wiring. |
+| More-actions (`...`) menu | No | Yes | Done | IDE-only affordance, now translated and refreshed with language changes. |
+| IDE-specific tooltip parity | N/A | Yes | Done | `activity_btn_deps` and `toolButton_more` tooltips are refreshed. |
+| Status line | No | Yes | Intentional | IDE-specific enhancement, not a parity gap. |
+
+## Notes
+
+- The goal is behavioral parity, not a pixel-identical UI.
+- IDE-only affordances remain acceptable as long as the classic feature set stays available.
+- Future parity work should prefer shared wiring/helpers over duplicating signal logic in each UI variant.
diff --git a/tests/test_deps_analyser_heuristics.py b/tests/test_deps_analyser_heuristics.py
