refactor: intégration d'EngineRunner comme source de vérité unique pour la GUI

Author: raidos23

Core/Compiler/engine_runner.py

@@ -46,17 +46,17 @@ def resolve_engine_command(
     engine_id: str,
     context: BuildContext,
     engine_config: dict[str, Any] | None = None,
-) -> tuple[str, list[str]]:
+) -> tuple[str, list[str], dict[str, str]]:
     """
-    Load *engine_id* and derive the (program, args) pair for *context*.
+    Load *engine_id* and derive (program, args, env) for *context*.
 
     Args:
         engine_id:     Registered engine identifier (e.g. ``"pyinstaller"``).
         context:       BuildContext describing the project.
         engine_config: Optional per-engine config overrides.
 
     Returns:
-        A ``(program, args)`` tuple ready to be passed to subprocess.
+        A ``(program, args, env)`` tuple ready to be passed to subprocess.
 
     Raises:
         EngineRunnerError: When the engine cannot be loaded, does not support
@@ -76,11 +76,11 @@ def resolve_engine_command(
         pass
 
     try:
-        resolved = engine.program_and_args_from_context(context)
-    except NotImplementedError as exc:
+        resolved = engine.program_and_args(context)
+    except NotImplementedError:
         raise EngineRunnerError(
-            f"Engine '{engine_id}' does not support BuildContext builds"
-        ) from exc
+            f"Engine '{engine_id}' does not implement build_command"
+        )
     except Exception as exc:
         raise EngineRunnerError(
             f"Engine '{engine_id}' failed to build command: {exc}"
@@ -90,7 +90,16 @@ def resolve_engine_command(
         raise EngineRunnerError(f"Engine '{engine_id}' returned no command")
 
     program, args = resolved
-    return str(program), list(args)
+    
+    # Retrieve engine-specific environment
+    try:
+        env = engine.environment() if hasattr(engine, "environment") else {}
+        if env is None:
+            env = {}
+    except Exception:
+        env = {}
+
+    return str(program), list(args), dict(env)
 
 
 def run_engine_compile(
@@ -138,16 +147,20 @@ def run_engine_compile(
             f"Entrypoint missing or obsolete: {context.entry_point}"
         )
 
-    # ── 2. Resolve (program, args) from engine ───────────────────────────────
+    # ── 2. Resolve (program, args, env) from engine ──────────────────────────
     try:
-        program, args = resolve_engine_command(engine_id, context, engine_config)
+        program, args, engine_env = resolve_engine_command(engine_id, context, engine_config)
     except EngineRunnerError as exc:
         return _failure(str(exc))
 
     # ── 3. Security hardening ────────────────────────────────────────────────
     try:
+        # Merge engine env with mandatory ARK variables
+        full_env = dict(engine_env)
+        full_env["ARK_WORKSPACE"] = str(workspace)
+        
         safe_program, safe_args, safe_env = secure_command(
-            program, args, {"ARK_WORKSPACE": str(workspace)}
+            program, args, full_env
         )
     except Exception as exc:
         return _failure(f"Unsafe compile command blocked: {exc}")

Core/Compiler/mainprocess.py

@@ -406,7 +406,7 @@ def compile_from_context(
         """
         Start an async compilation from a :class:`BuildContext`.
 
-        Resolves ``(program, args)`` via the engine, then delegates to
+        Resolves ``(program, args, env)`` via the engine, then delegates to
         :class:`CompilerCore` (Qt thread) for non-blocking execution.
 
         Args:
@@ -422,18 +422,21 @@ def compile_from_context(
 
         # Resolve command via engine (pure-Python, no Qt)
         try:
-            program, args = resolve_engine_command(engine_id, context, engine_config)
+            program, args, engine_env = resolve_engine_command(engine_id, context, engine_config)
         except EngineRunnerError as exc:
             self.log_message.emit("error", str(exc))
             return False
 
-        env = {"ARK_WORKSPACE": str(workspace)}
+        # Merge engine env with mandatory ARK variables
+        full_env = dict(engine_env)
+        full_env["ARK_WORKSPACE"] = str(workspace)
+        
         file_path = str(workspace / context.entry_point)
 
         return self.compile(
             program=program,
             args=args,
-            env=env,
+            env=full_env,
             engine_id=engine_id,
             file_path=file_path,
             workspace_dir=str(workspace),