refactor: renommage Core/sys_deps.py vers

Core/SysDependencyManager.py

Author: raidos23

Core/sys_deps.py Core/SysDependencyManager.pyCore/sys_deps.py renamed to Core/SysDependencyManager.py

@@ -197,7 +197,7 @@ def ensure_tools_installed(self, gui, stop_signal: Optional[Callable[[], bool]]
             if system_tools:
                 try:
                     # Import and use SysDependencyManager directly for full GUI support
-                    from Core.sys_deps import (
+                    from Core.SysDependencyManager import (
                         SysDependencyManager,
                         check_system_packages,
                     )

Core/engine/base.py

@@ -335,7 +335,7 @@ def apply_file_filter(self, text: Optional[str] = None) -> None:
     # PRÉFÉRENCES (délégation à Core/PreferencesManager)
     # =========================================================================
 
-    from Core.PreferencesManager import load_preferences, save_preferences
+    from Ui.PreferencesManager import load_preferences, save_preferences
 
     # =========================================================================
     # DÉPENDANCES (délégation à Core/deps_analyser)

Ui/Gui/Gui.py

@@ -84,35 +84,13 @@ def load_preferences(self):
                 # Dernier recours: fichier à la racine du cwd
                 with open(PREFS_BASENAME, encoding="utf-8") as f:
                     prefs = json.load(f)
-        self.icon_path = prefs.get("icon_path", None)
-        self.opt_onefile_state = prefs.get("opt_onefile", False)
-        self.opt_windowed_state = prefs.get("opt_windowed", False)
-        self.opt_noconfirm_state = prefs.get("opt_noconfirm", False)
-        self.opt_clean_state = prefs.get("opt_clean", False)
-        self.opt_noupx_state = prefs.get("opt_noupx", False)
-        self.opt_main_only_state = prefs.get("opt_main_only", False)
-        self.opt_debug_state = prefs.get("opt_debug", False)
-        self.opt_auto_install_state = prefs.get("auto_install", True)
-        # self.custom_args_text supprimé (widget supprimé)
-        self.output_dir = prefs.get("output_dir", "")
-        # Préférence de langue (conserve "System" si choisi)
+        
         self.language_pref = prefs.get("language_pref", prefs.get("language", "System"))
         # Compat: conserver self.language utilisé ailleurs
         self.language = self.language_pref
         # Thème UI
         self.theme = prefs.get("theme", "System")
     except Exception:
-        self.icon_path = None
-        self.opt_onefile_state = False
-        self.opt_windowed_state = False
-        self.opt_noconfirm_state = False
-        self.opt_clean_state = False
-        self.opt_noupx_state = False
-        self.opt_main_only_state = False
-        self.opt_debug_state = False
-        self.opt_auto_install_state = True
-        # self.custom_args_text supprimé (widget supprimé)
-        self.output_dir = ""
         # Préférence de langue par défaut
         self.language_pref = "System"
         self.language = "System"

Core/PreferencesManager.py Ui/PreferencesManager.pyCore/PreferencesManager.py renamed to Ui/PreferencesManager.py

@@ -459,7 +459,7 @@ def get_current_language_sync() -> str:
     """Return current language from user preferences (sync)."""
     try:
         # Absolute import to avoid relative-import issues outside Core package
-        from Core.PreferencesManager import PREFS_FILE
+        from Ui.PreferencesManager import PREFS_FILE
 
         if os.path.isfile(PREFS_FILE):
             with open(PREFS_FILE, encoding="utf-8") as f:

Ui/i18n.py

@@ -16,6 +16,6 @@
 from __future__ import annotations
 
 # Pass-through to host system dependency manager
-from Core.sys_deps import SysDependencyManager  # type: ignore[F401]
+from Core.SysDependencyManager import SysDependencyManager  # type: ignore[F401]
 
 __all__ = ["SysDependencyManager"]

engine_sdk/Sys_Deps.py

@@ -75,11 +75,11 @@ def test_ensure_tools_installed_runs_system_before_python(monkeypatch) -> None:
     events: list[str] = []
     process = DummyProcess(wait_result=True, exit_code=0)
 
-    import Core.sys_deps as sys_deps
+    import Core.SysDependencyManager as SysDependencyManager
 
-    monkeypatch.setattr(sys_deps, "check_system_packages", lambda _packages: False)
+    monkeypatch.setattr(SysDependencyManager, "check_system_packages", lambda _packages: False)
     monkeypatch.setattr(
-        sys_deps,
+        SysDependencyManager,
         "SysDependencyManager",
         lambda gui: DummySysDepsManager(events, process),
     )
@@ -101,11 +101,11 @@ def test_ensure_tools_installed_continues_python_after_system_timeout(
     events: list[str] = []
     process = DummyProcess(wait_result=False, exit_code=1)
 
-    import Core.sys_deps as sys_deps
+    import Core.SysDependencyManager as SysDependencyManager
 
-    monkeypatch.setattr(sys_deps, "check_system_packages", lambda _packages: False)
+    monkeypatch.setattr(SysDependencyManager, "check_system_packages", lambda _packages: False)
     monkeypatch.setattr(
-        sys_deps,
+        SysDependencyManager,
         "SysDependencyManager",
         lambda gui: DummySysDepsManager(events, process),
     )

tests/test_engine_tool_install_order.py

@@ -22,7 +22,7 @@ def test_install_system_packages_headless_linux_uses_noninteractive_sudo(
     monkeypatch,
 ) -> None:
     """Headless mode should run package install commands without Qt widgets."""
-    import Core.sys_deps as sys_deps
+    import Core.SysDependencyManager as SysDependencyManager
 
     executed: list[list[str]] = []
 
@@ -34,17 +34,17 @@ def __init__(self, returncode: int = 0) -> None:
 
     monkeypatch.setattr("platform.system", lambda: "Linux")
     monkeypatch.setattr(
-        sys_deps.shutil, "which", lambda name: "/usr/bin/apt" if name == "apt" else None
+        SysDependencyManager.shutil, "which", lambda name: "/usr/bin/apt" if name == "apt" else None
     )
-    monkeypatch.setattr(sys_deps.os, "geteuid", lambda: 1000, raising=False)
+    monkeypatch.setattr(SysDependencyManager.os, "geteuid", lambda: 1000, raising=False)
 
     def _fake_run(cmd, **_kwargs):
         executed.append(list(cmd))
         return _Result(0)
 
-    monkeypatch.setattr(sys_deps.subprocess, "run", _fake_run)
+    monkeypatch.setattr(SysDependencyManager.subprocess, "run", _fake_run)
 
-    assert sys_deps.install_system_packages(["cloc"], gui=None) is True
+    assert SysDependencyManager.install_system_packages(["cloc"], gui=None) is True
     assert executed == [
         ["sudo", "-n", "apt-get", "-o", "Acquire::Retries=3", "update"],
         [
@@ -67,15 +67,15 @@ def _fake_run(cmd, **_kwargs):
 
 def test_install_system_packages_headless_does_not_use_qt_manager(monkeypatch) -> None:
     """Headless mode must bypass the Qt-based SysDependencyManager path."""
-    import Core.sys_deps as sys_deps
+    import Core.SysDependencyManager as SysDependencyManager
 
     monkeypatch.setattr("platform.system", lambda: "Linux")
     monkeypatch.setattr(
-        sys_deps.shutil, "which", lambda name: "/usr/bin/dnf" if name == "dnf" else None
+        SysDependencyManager.shutil, "which", lambda name: "/usr/bin/dnf" if name == "dnf" else None
     )
-    monkeypatch.setattr(sys_deps.os, "geteuid", lambda: 0, raising=False)
+    monkeypatch.setattr(SysDependencyManager.os, "geteuid", lambda: 0, raising=False)
     monkeypatch.setattr(
-        sys_deps,
+        SysDependencyManager,
         "SysDependencyManager",
         lambda _gui: (_ for _ in ()).throw(
             AssertionError("Qt manager should not be used in headless mode")
@@ -88,16 +88,16 @@ def __init__(self, returncode: int = 0) -> None:
             self.stdout = ""
             self.stderr = ""
 
-    monkeypatch.setattr(sys_deps.subprocess, "run", lambda *_a, **_k: _Result(0))
+    monkeypatch.setattr(SysDependencyManager.subprocess, "run", lambda *_a, **_k: _Result(0))
 
-    assert sys_deps.install_system_packages(["cloc"], gui=None) is True
+    assert SysDependencyManager.install_system_packages(["cloc"], gui=None) is True
 
 
 def test_install_system_packages_headless_retries_transient_failures(
     monkeypatch,
 ) -> None:
     """Headless install should retry command steps before failing."""
-    import Core.sys_deps as sys_deps
+    import Core.SysDependencyManager as SysDependencyManager
 
     attempts = {"count": 0}
 
@@ -109,20 +109,20 @@ def __init__(self, returncode: int) -> None:
 
     monkeypatch.setattr("platform.system", lambda: "Linux")
     monkeypatch.setattr(
-        sys_deps.shutil,
+        SysDependencyManager.shutil,
         "which",
         lambda name: "/usr/bin/dnf" if name == "dnf" else None,
     )
-    monkeypatch.setattr(sys_deps.os, "geteuid", lambda: 0, raising=False)
-    monkeypatch.setattr(sys_deps.time, "sleep", lambda *_a, **_k: None)
+    monkeypatch.setattr(SysDependencyManager.os, "geteuid", lambda: 0, raising=False)
+    monkeypatch.setattr(SysDependencyManager.time, "sleep", lambda *_a, **_k: None)
 
     def _fake_run(_cmd, **_kwargs):
         attempts["count"] += 1
         if attempts["count"] == 1:
             return _Result(1)
         return _Result(0)
 
-    monkeypatch.setattr(sys_deps.subprocess, "run", _fake_run)
+    monkeypatch.setattr(SysDependencyManager.subprocess, "run", _fake_run)
 
-    assert sys_deps.install_system_packages(["cloc"], gui=None) is True
+    assert SysDependencyManager.install_system_packages(["cloc"], gui=None) is True
     assert attempts["count"] == 2

tests/test_sys_deps_headless.py

@@ -48,8 +48,13 @@ inclusion_patterns:
 
   et aussi  les pattern dinclusion ne sont plus necesaire car l'exclusion est suffisante.
 
-  dans Core/Configs il faut eviter la retro compatibilité et revoir
+  dans Core/Configs il faut eviter la retro compatibilité et revoir.
 
 ## VenvManager [x]
 
-dans venvmanager les logique comme celle de la generation de la reuirments etc ne doit normalement pas être dedans mais plutot gerer par le depsanalyser qui dispose de tout la technologie de pointe pour une fiablité .  pour reduire le niveau de complexité inutile n'est rtil pas mieux que venvmanager ne gere que pip comme gestionnaiaire ?? si oui fait un plan
+dans venvmanager les logique comme celle de la generation de la reuirments etc ne doit normalement pas être dedans mais plutot gerer par le depsanalyser qui dispose de tout la technologie de pointe pour une fiablité .  pour reduire le niveau de complexité inutile n'est rtil pas mieux que venvmanager ne gere que pip comme gestionnaiaire ?? si oui fait un plan.
+
+## refactor: Séparation Core(logique métier) et Ui(interface utilisateur ou de pilotage) []
+
+Core/SysDependencyManager.py doit purement contenir de la logique metier et son usage gui sera dans Ui/Gui/Dialogs/.
+Services/AdvancedAuth.py doit purement contenir la logique service et son usage gui sera dans Ui/Gui/Dialogs/.