fix(cli): garantir une sortie JSON sans bruit pour les commandes engine

Author: raidos23

OnlyMod/EngineOnlyMod/app.py

@@ -72,11 +72,11 @@ class MockGUI:
     en mode standalone sans l'application complète.
     """
 
-    def __init__(self, workspace_dir: Optional[str] = None):
+    def __init__(self, workspace_dir: Optional[str] = None, log_echo: bool = True):
         self.workspace_dir = (
             os.path.abspath(str(workspace_dir)) if workspace_dir else workspace_dir
         )
-        self.log = MockLog()
+        self.log = MockLog(echo=log_echo)
         self._tr = {}
         self.venv_path_manuel = None
         self.venv_path = None
@@ -120,13 +120,15 @@ def _init_venv_context(self) -> None:
 class MockLog:
     """Mock log object qui collecte les messages pour affichage."""
 
-    def __init__(self):
+    def __init__(self, echo: bool = True):
         self.messages: List[str] = []
+        self._echo = bool(echo)
 
     def append(self, message: str) -> None:
         """Ajoute un message au log."""
         self.messages.append(message)
-        print(message)
+        if self._echo:
+            print(message)
 
     def clear(self) -> None:
         """Efface le log."""
@@ -385,6 +387,7 @@ def __init__(
         theme: str = "dark",
         dry_run: bool = False,
         headless: bool = False,
+        quiet_logs: bool = False,
     ):
         """
         Initialise l'application standalone engines.
@@ -397,8 +400,9 @@ def __init__(
             theme: Nom du thème ('light' ou 'dark')
             dry_run: Si True, affiche uniquement la commande sans exécuter
             headless: Si True, fonctionne sans interface GUI (mode CLI)
+            quiet_logs: Disable stdout log echo (used for strict JSON payload mode)
         """
-        self.gui = MockGUI(workspace_dir)
+        self.gui = MockGUI(workspace_dir, log_echo=not bool(quiet_logs))
         self.language_manager = LanguageManager()
         self.theme_manager = ThemeManager()
 

cli/click_app.py

@@ -590,6 +590,7 @@ def engine_dry_run(engine_id, file_path, workspace, as_json):
             theme="dark",
             headless=True,
             dry_run=True,
+            quiet_logs=bool(as_json),
         )
         result = app.run_compilation(engine_id, str(Path(file_path)), dry_run=True)
         if as_json:
@@ -617,6 +618,7 @@ def engine_compile(engine_id, file_path, workspace, as_json):
             language="en",
             theme="dark",
             headless=True,
+            quiet_logs=bool(as_json),
         )
         result = app.run_compilation(engine_id, str(Path(file_path)), dry_run=False)
         if as_json:

tests/test_engine_standalone_required_tools.py

@@ -5,7 +5,7 @@
 
 from pathlib import Path
 
-from OnlyMod.EngineOnlyMod.app import EnginesStandaloneApp
+from OnlyMod.EngineOnlyMod.app import EnginesStandaloneApp, MockLog
 
 
 class _DummyEngine:
@@ -78,3 +78,20 @@ def test_run_compilation_headless_returns_preflight_error(monkeypatch) -> None:
     assert result["success"] is False
     assert result["return_code"] == -1
     assert "preflight failed" in result["error"]
+
+
+def test_mock_log_can_disable_stdout_echo(monkeypatch) -> None:
+    called = {"n": 0}
+
+    def _fake_print(*_args, **_kwargs):
+        called["n"] += 1
+
+    monkeypatch.setattr("builtins.print", _fake_print)
+
+    silent_log = MockLog(echo=False)
+    silent_log.append("hello")
+
+    loud_log = MockLog(echo=True)
+    loud_log.append("world")
+
+    assert called["n"] == 1