fix: worker conflict in update install

Author: Leoglme

web/app/components/DesktopUpdaterPanel.vue

@@ -2,6 +2,7 @@
 import type { DownloadEvent } from '@tauri-apps/plugin-updater'
 import { check } from '@tauri-apps/plugin-updater'
 import { relaunch } from '@tauri-apps/plugin-process'
+import { invoke } from '@tauri-apps/api/core'
 
 type UpdaterStatus = 'idle' | 'available' | 'downloading' | 'installed' | 'error'
 
@@ -102,6 +103,20 @@ async function installUpdate() {
     status.value = 'downloading'
     errorMessage.value = null
     resetDownloadProgress()
+
+    // Sur Windows, l'installeur NSIS réécrit `goupix-vinted-worker.exe` :
+    // il faut impérativement tuer le sidecar (et ses enfants Chromium lancés
+    // par nodriver) AVANT, sinon l'install échoue avec « Error opening file
+    // for writing ». Le kill côté Rust propage un taskkill /F /T sur l'arbre.
+    try {
+      await invoke('stop_local_worker')
+    } catch (killError) {
+      console.warn('[Updater] stop_local_worker a échoué, on tente quand même l\'install :', killError)
+    }
+    // Laisse Windows libérer les handles sur le .exe et ses DLLs avant que
+    // NSIS ne commence l'extraction.
+    await new Promise((resolve) => setTimeout(resolve, 600))
+
     await pendingUpdate.value.downloadAndInstall(onDownloadEvent)
     status.value = 'installed'
   } catch (error) {

web/src-tauri/nsis-hooks.nsh

@@ -0,0 +1,20 @@
+; Hooks NSIS personnalisés pour le bundle Tauri Windows.
+;
+; Contexte : GoupixDex embarque un sidecar PyInstaller `goupix-vinted-worker.exe`
+; qui lance lui-même Chromium via nodriver. Pendant une mise à jour, NSIS doit
+; réécrire ce .exe ; s'il est encore en mémoire (ou si un des Chromium enfants
+; garde un handle), l'install échoue avec :
+;   « Error opening file for writing: ...\goupix-vinted-worker.exe »
+;
+; Côté app Tauri on appelle déjà la commande `stop_local_worker` avant
+; `downloadAndInstall`, mais ce hook joue le rôle de filet de sécurité
+; (install manuel depuis GitHub Releases, app crashée, anciens orphelins...).
+
+!macro NSIS_HOOK_PREINSTALL
+  DetailPrint "Arrêt du worker GoupixDex s'il est en cours d'exécution..."
+  ; /F = force, /T = tue aussi l'arborescence (Chromium lancés par nodriver).
+  ; On redirige la sortie vers nul pour rester silencieux si aucun process n'est trouvé.
+  nsExec::Exec 'cmd /C taskkill /F /T /IM goupix-vinted-worker.exe >nul 2>&1'
+  Pop $0
+  Sleep 500
+!macroend

web/src-tauri/src/lib.rs

@@ -1,5 +1,7 @@
 use std::path::PathBuf;
 use std::sync::Mutex;
+#[cfg(target_os = "windows")]
+use std::os::windows::process::CommandExt;
 use tauri::Manager;
 use tauri_plugin_shell::process::{CommandChild, CommandEvent};
 use tauri_plugin_shell::ShellExt;
@@ -122,6 +124,39 @@ fn check_browser_availability() -> BrowserInfo {
     detect_browsers()
 }
 
+#[cfg(target_os = "windows")]
+const CREATE_NO_WINDOW: u32 = 0x0800_0000;
+
+/// Tue le worker local et toute son arborescence de sous-processus (Chromium
+/// lancés par nodriver notamment). Indispensable avant un remplacement sur
+/// disque de `goupix-vinted-worker.exe` (mise à jour NSIS) pour ne pas se
+/// retrouver avec un fichier verrouillé.
+fn kill_worker_tree(child: CommandChild) {
+    #[cfg(target_os = "windows")]
+    {
+        let pid = child.pid();
+        let _ = std::process::Command::new("taskkill")
+            .args(["/F", "/T", "/PID", &pid.to_string()])
+            .creation_flags(CREATE_NO_WINDOW)
+            .status();
+    }
+    let _ = child.kill();
+}
+
+/// Commande exposée au front-end pour fermer proprement le worker avant une
+/// mise à jour Tauri (voir `DesktopUpdaterPanel.vue`).
+#[tauri::command]
+fn stop_local_worker(state: tauri::State<'_, VintedLocalChild>) -> Result<(), String> {
+    let child_opt = match state.0.lock() {
+        Ok(mut guard) => guard.take(),
+        Err(_) => None,
+    };
+    if let Some(child) = child_opt {
+        kill_worker_tree(child);
+    }
+    Ok(())
+}
+
 #[cfg(debug_assertions)]
 fn dev_repo_api_dir() -> Option<PathBuf> {
     let manifest = PathBuf::from(env!("CARGO_MANIFEST_DIR"));
@@ -276,7 +311,10 @@ pub fn run() {
         .plugin(tauri_plugin_process::init())
         .plugin(tauri_plugin_shell::init())
         .manage(VintedLocalChild(Mutex::new(None)))
-        .invoke_handler(tauri::generate_handler![check_browser_availability])
+        .invoke_handler(tauri::generate_handler![
+            check_browser_availability,
+            stop_local_worker
+        ])
         .setup(|app| {
             let handle = app.handle().clone();
             match spawn_worker(&handle) {
@@ -304,7 +342,7 @@ pub fn run() {
             if let tauri::RunEvent::Exit = event {
                 if let Ok(mut guard) = app.state::<VintedLocalChild>().0.lock() {
                     if let Some(child) = guard.take() {
-                        let _ = child.kill();
+                        kill_worker_tree(child);
                     }
                 }
             }

web/src-tauri/tauri.conf.json

@@ -34,7 +34,12 @@
       "../public/favicon-32x32.png",
       "../public/android-chrome-192x192.png",
       "../public/android-chrome-512x512.png"
-    ]
+    ],
+    "windows": {
+      "nsis": {
+        "installerHooks": "nsis-hooks.nsh"
+      }
+    }
   },
   "plugins": {
     "updater": {