Updated EngineLoader registry to include create function and fixed engine creation errors in Compiler

Author: raidos23

CORRECTION_PLAN.md

@@ -0,0 +1,47 @@
+# Plan de Correction - Erreur TypeError: CompilerEngine.ensure_tools_installed()
+
+## Problème Identifié
+
+Dans `Core/Compiler/__init__.py`, la fonction `get_engine(engine_id)` retourne la **classe** de l'engine, mais le code appelle des méthodes d'instance (`ensure_tools_installed`, `build_command`, `on_success`) sur cette classe au lieu d'une instance.
+
+### Lieux du problème (ligne 221 et suivantes):
+1. `compile_all()` - ligne ~221: `engine = get_engine(engine_id)` puis `engine.ensure_tools_installed(self)`
+2. `_start_compilation_queue()` - ligne ~249: `if not engine.ensure_tools_installed(self)`
+3. `start_compilation_process()` - ligne ~308: `engine = get_engine(engine_id)` puis `engine.ensure_tools_installed(self)`
+4. `handle_finished()` - ligne ~400: `engine.on_success(self, file_path)`
+
+## Solution
+
+Remplacer `get_engine(engine_id)` par `create(engine_id)` dans `Core/Compiler/__init__.py` là où des méthodes d'instance sont appelées sur l'engine.
+
+La fonction `create()` dans `EngineLoader/registry.py` instancie correctement l'engine:
+```python
+def create(eid: str) -> CompilerEngine:
+    cls = get_engine(eid)
+    if not cls:
+        raise KeyError(f"Engine '{eid}' is not registered")
+    try:
+        return cls()  # Crée une instance!
+    except Exception as e:
+        raise RuntimeError(f"Failed to instantiate engine '{eid}': {e}")
+```
+
+## Fichiers à Modifier
+
+| Fichier | Modification |
+|---------|--------------|
+| `Core/Compiler/__init__.py` | Remplacer `get_engine()` par `create()` pour les appels aux méthodes d'instance |
+
+## Étapes de Correction
+
+1. Importer `create` depuis EngineLoader.registry
+2. Remplacer `get_engine(engine_id)` par `create(engine_id)` dans:
+   - `compile_all()` - pour l'appel à `ensure_tools_installed()`
+   - `_start_compilation_queue()` - pour les appels à `ensure_tools_installed()`, `build_command()`, et `environment()`
+   - `start_compilation_process()` - pour les appels à `ensure_tools_installed()`, `build_command()`, et `on_success()`
+   - `handle_finished()` - pour l'appel à `on_success()`
+
+## Note Importante
+
+Ne pas modifier les imports de `get_engine` car il est toujours utilisé ailleurs dans le code (par exemple pour obtenir la classe sans l'instancier).
+

Core/Compiler/__init__.py

@@ -89,7 +89,7 @@
 )
 
 # Importations de EngineLoader
-from EngineLoader.registry import get_engine
+from EngineLoader.registry import get_engine, create
 
 __all__ = [
     # compiler.py
@@ -187,12 +187,13 @@ def compile_all(self) -> None:
     if not engine_id:
         engine_id = "pyinstaller"  # Moteur par défaut
 
-    # Obtenir le moteur
-    engine = get_engine(engine_id)
-    if not engine:
+    # Obtenir le moteur (instance)
+    try:
+        engine = create(engine_id)
+    except Exception as e:
         self.log_i18n(
-            f"❌ Moteur de compilation '{engine_id}' non trouvé.",
-            f"❌ Compilation engine '{engine_id}' not found.",
+            f"❌ Erreur création moteur '{engine_id}': {e}",
+            f"❌ Engine creation error '{engine_id}': {e}",
         )
         return
 
@@ -376,19 +377,19 @@ def start_compilation_process(self, engine_id: str, file_path: str) -> bool:
     Returns:
         True if compilation started, False otherwise
     """
-    # Obtenir le moteur
-    engine = get_engine(engine_id)
-    if not engine:
+    # Obtenir le moteur (instance)
+    try:
+        engine = create(engine_id)
+    except Exception as e:
         self.log_i18n(
-            f"❌ Moteur '{engine_id}' non trouvé.",
-            f"❌ Engine '{engine_id}' not found.",
+            f"❌ Erreur création moteur '{engine_id}': {e}",
+            f"❌ Engine creation error '{engine_id}': {e}",
         )
         return False
 
     # Vérifier les prérequis
-    if hasattr(engine, "ensure_tools_installed"):
-        if not engine.ensure_tools_installed(self):
-            return False
+    if not engine.ensure_tools_installed(self):
+        return False
 
     # Construire la commande
     cmd = engine.build_command(self, file_path)
@@ -530,7 +531,10 @@ def handle_finished(self, return_code: int, info: dict) -> None:
         # Appeler on_success du moteur si disponible
         engine_id = info.get("engine")
         if engine_id:
-            engine = get_engine(engine_id)
+            try:
+                engine = create(engine_id)
+            except Exception:
+                engine = None
             if engine and hasattr(engine, "on_success"):
                 file_path = info.get("file")
                 if file_path: