Refonte de l'orchestration de compilation GUI et corrections de bugs critiques

Author: raidos23

Ui/Gui/Compilation/compiler.py

@@ -23,6 +23,7 @@
 from __future__ import annotations
 
 import os
+import re
 from datetime import datetime
 from enum import Enum
 from pathlib import Path
@@ -43,6 +44,14 @@ class CompilationStatus(Enum):
     FAILED = "failed"
 
 
+# Pre-compiled regex patterns for progress detection
+PROGRESS_PATTERNS = [
+    re.compile(r"\[(\d+)%\]"),
+    re.compile(r"Progress:\s*(\d+)"),
+    re.compile(r"(\d+)/(\d+)"),
+]
+
+
 class CompilationThread(QThread):
     """
     Thread used to run compilation without blocking the UI.
@@ -60,7 +69,7 @@ def __init__(
         engine_id: str,
         context: BuildContext,
         engine_config: Optional[Dict[str, Any]] = None,
-        is_rebuild: any = False,
+        is_rebuild: bool = False,
     ):
         """
         Initialize the compilation thread.
@@ -114,27 +123,25 @@ def _stop_signal():
 
     def _update_progress(self, line: str) -> None:
         """Update progress based on process output."""
-        # Détecter les patterns de progression courants
-        progress_patterns = [
-            r"\[(\d+)%\]",
-            r"Progress:\s*(\d+)",
-            r"(\d+)/(\d+)",
-        ]
-
-        import re
-
-        for pattern in progress_patterns:
-            match = re.search(pattern, line)
+        for pattern in PROGRESS_PATTERNS:
+            match = pattern.search(line)
             if match:
-                if len(match.groups()) == 1:
-                    progress = int(match.group(1))
-                    self.progress_update.emit(progress, line)
-                elif len(match.groups()) == 2:
-                    current = int(match.group(1))
-                    total = int(match.group(2))
-                    if total > 0:
-                        progress = int((current / total) * 100)
+                groups = match.groups()
+                if len(groups) == 1:
+                    try:
+                        progress = int(groups[0])
                         self.progress_update.emit(progress, line)
+                    except (ValueError, TypeError):
+                        pass
+                elif len(groups) == 2:
+                    try:
+                        current = int(groups[0])
+                        total = int(groups[1])
+                        if total > 0:
+                            progress = int((current / total) * 100)
+                            self.progress_update.emit(progress, line)
+                    except (ValueError, TypeError):
+                        pass
                 break
 
     def cancel(self) -> None:
@@ -180,7 +187,7 @@ def status(self) -> CompilationStatus:
         return self._status
 
     @property
-    def is_running(self) -> any:
+    def is_running(self) -> bool:
         """Return True when a compilation is currently running."""
         return self._status == CompilationStatus.RUNNING
 
@@ -200,7 +207,7 @@ def compile(
         engine_id: Optional[str] = None,
         file_path: Optional[str] = None,
         workspace_dir: Optional[str] = None,
-    ) -> any:
+    ) -> bool:
         """
         Legacy compile method. Use compile_from_context instead.
         """
@@ -226,8 +233,8 @@ def compile_from_context(
         engine_id: str,
         context: BuildContext,
         engine_config: Optional[Dict[str, Any]] = None,
-        is_rebuild: any = False,
-    ) -> any:
+        is_rebuild: bool = False,
+    ) -> bool:
         """
         Start an async compilation from a BuildContext.
         """
@@ -272,7 +279,7 @@ def compile_from_context(
 
         return True
 
-    def cancel(self) -> any:
+    def cancel(self) -> bool:
         """
         Cancel current compilation.
         """

Ui/Gui/Compilation/helpers.py

@@ -83,7 +83,7 @@ def run_bcasl_before_compile(
                 pass
 
 
-def bcasl_report_allows_compile(gui_instance, report) -> any:
+def bcasl_report_allows_compile(gui_instance, report) -> bool:
     """Return True when BCASL pre-compile report allows compilation to continue."""
     try:
         if report is None:

Ui/Gui/Compilation/mainprocess.py

@@ -140,21 +140,21 @@ def current_engine(self) -> Optional[str]:
         return self._current_engine
 
     @property
-    def is_ready(self) -> any:
+    def is_ready(self) -> bool:
         """Return True when the process is ready."""
         return self._state in (ProcessState.READY, ProcessState.IDLE)
 
     @property
-    def is_compiling(self) -> any:
+    def is_compiling(self) -> bool:
         """Return True when a compilation is currently running."""
         return self._state == ProcessState.COMPILING
 
     @property
-    def is_idle(self) -> any:
+    def is_idle(self) -> bool:
         """Return True when the process is idle."""
         return self._state == ProcessState.IDLE
 
-    def set_workspace(self, workspace_dir: str) -> any:
+    def set_workspace(self, workspace_dir: str) -> bool:
         """
         Set workspace directory.
 
@@ -180,7 +180,7 @@ def set_workspace(self, workspace_dir: str) -> any:
 
         return True
 
-    def set_file(self, file_path: str) -> any:
+    def set_file(self, file_path: str) -> bool:
         """
         Set file to compile.
 
@@ -218,7 +218,7 @@ def compile(
         engine_id: Optional[str] = None,
         file_path: Optional[str] = None,
         workspace_dir: Optional[str] = None,
-    ) -> any:
+    ) -> bool:
         """
         Start a compilation.
 
@@ -296,7 +296,7 @@ def compile(
 
         return success
 
-    def cancel(self) -> any:
+    def cancel(self) -> bool:
         """
         Cancel current compilation.
 
@@ -385,8 +385,8 @@ def compile_from_context(
         engine_id: str,
         context: BuildContext,
         engine_config: Optional[Dict[str, Any]] = None,
-        is_rebuild: any = False,
-    ) -> any:
+        is_rebuild: bool = False,
+    ) -> bool:
         """
         Start an async compilation from a :class:`BuildContext`.
 
@@ -455,7 +455,7 @@ def get_exclusion_patterns(self) -> List[str]:
                 pass
         return DEFAULT_EXCLUDE_PATTERNS
 
-    def should_exclude(self, file_path: str) -> any:
+    def should_exclude(self, file_path: str) -> bool:
         """
         Determine whether a file must be excluded from compilation.