Refactor: centralisation de l'ouverture du dossier de sortie dans le core runner (ARK gère le succès par défaut)

Author: raidos23

Core/Compiler/engine_runner.py

@@ -430,8 +430,16 @@ def _read_stream(stream, callback):
         stdout_thread.join()
         stderr_thread.join()
 
-        # Call on_success hook if compilation succeeded
+        # Compilation success hooks
         if process.returncode == 0:
+            # 1. ARK opens the output directory by default
+            try:
+                if hasattr(engine_instance, "open_output_dir"):
+                    engine_instance.open_output_dir(context.output_dir)
+            except Exception:
+                pass
+
+            # 2. Call engine-specific on_success hook
             try:
                 if hasattr(engine_instance, "on_success"):
                     engine_instance.on_success(bridge, str(entry_path))

engines/cx_freeze/__init__.py

@@ -66,7 +66,6 @@ def preflight(self, gui, file: str) -> bool:
 
     def build_command(self, context: BuildContext) -> list[str]:
         """Build a cx_Freeze command line from a normalized build context."""
-        self._last_context = context
         cfg = getattr(self, "_config_overrides", {})
         if not isinstance(cfg, dict):
             cfg = {}
@@ -150,10 +149,6 @@ def on_success(self, gui, file: str) -> None:
                     "success",
                     "Compilation CX_Freeze terminée avec succès.",
                 )
-            
-            # Open output directory automatically
-            if hasattr(self, "_last_context") and self._last_context:
-                self.open_output_dir(self._last_context.output_dir)
         except Exception:
             pass
 

engines/nuitka/__init__.py

@@ -71,7 +71,6 @@ def preflight(self, gui, file: str) -> bool:
 
     def build_command(self, context: BuildContext) -> list[str]:
         """Build a Nuitka command line from a normalized build context."""
-        self._last_context = context
         cfg = getattr(self, "_config_overrides", {})
         if not isinstance(cfg, dict):
             cfg = {}
@@ -173,10 +172,6 @@ def on_success(self, gui, file: str) -> None:
                     "success",
                     "Compilation Nuitka terminée avec succès.",
                 )
-            
-            # Open output directory automatically
-            if hasattr(self, "_last_context") and self._last_context:
-                self.open_output_dir(self._last_context.output_dir)
         except Exception:
             pass
 

engines/pyinstaller/__init__.py

@@ -66,7 +66,6 @@ def preflight(self, gui, file: str) -> bool:
 
     def build_command(self, context: BuildContext) -> list[str]:
         """Build a PyInstaller command line from a normalized build context."""
-        self._last_context = context
         cfg = getattr(self, "_config_overrides", {})
         if not isinstance(cfg, dict):
             cfg = {}
@@ -155,10 +154,6 @@ def on_success(self, gui, file: str) -> None:
                     "success",
                     "Compilation PyInstaller terminée avec succès.",
                 )
-            
-            # Open output directory automatically
-            if hasattr(self, "_last_context") and self._last_context:
-                self.open_output_dir(self._last_context.output_dir)
         except Exception:
             pass