fix: résolution de l'erreur invokeMethod via QTimer.singleShot et correction définitive du threading QProcess

Author: raidos23

pycompiler_ark/Core/SysDependencyManager.py

@@ -36,7 +36,7 @@ class SysDependencyManager:
     """
 
     def __init__(self, parent_widget: Optional[QObject] = None):
-        # parent_widget is used as the parent for QProcess
+        # parent_widget is used for UI callbacks and global task registration
         self.parent_widget = parent_widget
         self._ui_callbacks: dict = {}
 
@@ -77,9 +77,10 @@ def shell_run(
     ) -> Optional[QProcess]:
         """Runs a command without elevation."""
         try:
-            # IMPORTANT: QProcess must be created on the GUI thread if parent is a widget.
-            # However, if parent_widget is a QObject on the correct thread, it's fine.
-            proc = QProcess(self.parent_widget)
+            # IMPORTANT: We create QProcess WITHOUT a parent here. 
+            # This allows it to be created and waited for (waitForFinished) 
+            # in the background thread without violating Qt thread ownership rules.
+            proc = QProcess()
             if cwd:
                 proc.setWorkingDirectory(cwd)
 
@@ -101,6 +102,7 @@ def _finished_wrapper(ec, es):
 
             proc.finished.connect(_finished_wrapper)
 
+            # Note: The process is registered for global cleanup tracking
             self._register_task(proc, None, "commande système", "system command")
             proc.start()
             return proc

pycompiler_ark/Ui/Gui/Dialogs/SysDependencyUI.py

@@ -22,7 +22,7 @@
 import shutil
 from typing import Any, Optional
 
-from PySide6.QtCore import QProcess, QObject, Signal, Slot, QMetaObject, Qt
+from PySide6.QtCore import QProcess, QObject, QTimer, Qt
 from PySide6.QtWidgets import QMessageBox
 
 from pycompiler_ark.Core.SysDependencyManager import SysDependencyManager
@@ -32,7 +32,7 @@
 class SysDependencyUI(SysDependencyManager):
     """
     GUI extension of SysDependencyManager.
-    Proxies GUI calls to the main thread to avoid segfaults.
+    Proxies GUI calls to the main thread to avoid segfaults and PySide6 signature errors.
     """
 
     def __init__(self, parent_widget=None):
@@ -54,7 +54,7 @@ def _ui_tr(self, fr: str, en: str) -> str:
 
     def _ui_register_task(self, proc: QProcess, dlg: Optional[Any], label_fr: str, label_en: str) -> None:
         if self.parent_widget is None: return
-        # Simple task registration is generally thread-safe if it's just a list
+        # Task registration is generally thread-safe for list operations
         try:
             tasks = getattr(self.parent_widget, "_sysdep_tasks", [])
             tasks.append({"process": proc, "dialog": dlg, "label_fr": label_fr, "label_en": label_en})
@@ -71,9 +71,13 @@ def _ui_unregister_task(self, proc: QProcess) -> None:
             pass
 
     def _invoke_gui(self, method: callable, *args):
-        """Invoke a method on the GUI thread."""
+        """
+        Invoke a method on the GUI thread using QTimer.singleShot.
+        This is the most compatible way to pass a lambda/callable across threads in PySide6.
+        """
         if self.parent_widget:
-            QMetaObject.invokeMethod(self.parent_widget, lambda: method(*args), Qt.QueuedConnection)
+            # QTimer.singleShot(0, context, callable) ensures execution in context's thread
+            QTimer.singleShot(0, self.parent_widget, lambda: method(*args))
         else:
             method(*args)
 
@@ -84,14 +88,20 @@ def _show():
 
     def _show_progress(self, title_fr: str, title_en: str, msg_fr: str, msg_en: str):
         def _create():
+            # Close existing one if any
+            if self._progress_dlg:
+                try: self._progress_dlg.close()
+                except Exception: pass
+            
             self._progress_dlg = ProgressDialog(self.tr(title_fr, title_en), self.parent_widget)
             self._progress_dlg.set_message(self.tr(msg_fr, msg_en))
             self._progress_dlg.show()
         self._invoke_gui(_create)
 
     def _update_progress(self, msg: str):
         def _upd():
-            if self._progress_dlg: self._progress_dlg.set_message(msg)
+            if self._progress_dlg:
+                self._progress_dlg.set_message(msg)
         self._invoke_gui(_upd)
 
     def _close_progress(self):