fix: taskkill /T vor child.kill() – Python-Child überlebt nicht mehr

child.kill() tötete nur den PyInstaller-Bootloader (Parent); der
Python/uvicorn-Child-Prozess wurde zur Waise und lief weiter.
Lösung: Handle droppen, dann taskkill /F /T /PID für den ganzen Baum.
onCloseRequested ohne preventDefault – App schließt sofort.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: 2 people

src-tauri/src/lib.rs

@@ -15,29 +15,36 @@ fn get_backend_port(state: tauri::State<BackendPort>) -> u16 {
     *state.0.lock().unwrap()
 }
 
-/// IPC-Command: Frontend beendet den Backend-Sidecar explizit (beim Schließen und vor Update-Exit).
+/// Backend-Prozessbaum beenden.
 /// PyInstaller --onefile startet auf Windows zwei Prozesse (Bootloader + Python-Child).
-/// child.kill() beendet nur den Bootloader – /T killt den gesamten Prozessbaum.
-#[tauri::command]
-fn kill_backend(state: tauri::State<BackendChild>) {
-    if let Some(child) = state.0.lock().unwrap().take() {
-        #[cfg(target_os = "windows")]
+/// child.kill() tötet nur den Parent → Child läuft weiter.
+/// Lösung: zuerst taskkill /F /T /PID (killt ganzen Baum), dann Handle droppen.
+fn kill_backend_inner(child: CommandChild) {
+    #[cfg(target_os = "windows")]
+    {
         let pid = child.pid();
-
+        // Handle freigeben OHNE kill() – taskkill übernimmt den ganzen Baum atomisch
+        drop(child);
+        let _ = std::process::Command::new("taskkill")
+            .args(["/F", "/T", "/PID", &pid.to_string()])
+            .output();
+        log::info!("Backend-Prozessbaum (PID {}) per taskkill /T beendet", pid);
+    }
+    #[cfg(not(target_os = "windows"))]
+    {
         let _ = child.kill();
-
-        #[cfg(target_os = "windows")]
-        {
-            let _ = std::process::Command::new("taskkill")
-                .args(["/F", "/T", "/PID", &pid.to_string()])
-                .output();
-            log::info!("Backend-Sidecar Prozessbaum (PID {}) per taskkill /T beendet", pid);
-        }
-        #[cfg(not(target_os = "windows"))]
         log::info!("Backend-Sidecar beendet");
     }
 }
 
+/// IPC-Command: wird vom Frontend beim Schließen und vor dem Update-Exit aufgerufen
+#[tauri::command]
+fn kill_backend(state: tauri::State<BackendChild>) {
+    if let Some(child) = state.0.lock().unwrap().take() {
+        kill_backend_inner(child);
+    }
+}
+
 #[cfg_attr(mobile, tauri::mobile_entry_point)]
 pub fn run() {
     tauri::Builder::default()
@@ -56,15 +63,10 @@ pub fn run() {
                 .build(),
         )
         .setup(|app| {
-            // Freien Port wählen
             let port = portpicker::pick_unused_port().expect("Kein freier Port gefunden");
-
             log::info!("Backend-Port: {}", port);
-
-            // State registrieren, damit get_backend_port es lesen kann
             app.manage(BackendPort(Mutex::new(port)));
 
-            // Sidecar starten
             let sidecar_cmd = app
                 .shell()
                 .sidecar("backend")
@@ -75,9 +77,7 @@ pub fn run() {
                 .spawn()
                 .expect("Backend-Sidecar konnte nicht gestartet werden");
 
-            // Child-Handle speichern, damit er beim App-Exit sauber beendet wird
             app.manage(BackendChild(Mutex::new(Some(child))));
-
             log::info!("Backend-Sidecar gestartet auf Port {}", port);
             Ok(())
         })
@@ -87,20 +87,7 @@ pub fn run() {
         .run(|app_handle, event| {
             if let tauri::RunEvent::Exit = event {
                 if let Some(child) = app_handle.state::<BackendChild>().0.lock().unwrap().take() {
-                    #[cfg(target_os = "windows")]
-                    let pid = child.pid();
-
-                    let _ = child.kill();
-
-                    #[cfg(target_os = "windows")]
-                    {
-                        let _ = std::process::Command::new("taskkill")
-                            .args(["/F", "/T", "/PID", &pid.to_string()])
-                            .output();
-                        log::info!("Backend-Sidecar Prozessbaum (PID {}) per taskkill /T beendet", pid);
-                    }
-                    #[cfg(not(target_os = "windows"))]
-                    log::info!("Backend-Sidecar beendet");
+                    kill_backend_inner(child);
                 }
             }
         });

src/frontend/src/App.tsx

@@ -95,12 +95,11 @@ export default function App() {
     async function register() {
       const { getCurrentWindow } = await import('@tauri-apps/api/window')
       const { invoke } = await import('@tauri-apps/api/core')
-      const { exit } = await import('@tauri-apps/plugin-process')
 
-      unlisten = await getCurrentWindow().onCloseRequested(async event => {
-        event.preventDefault()
+      // Kein preventDefault – Fenster schließt normal weiter.
+      // kill_backend wird proaktiv aufgerufen; RunEvent::Exit tut es nochmals (idempotent).
+      unlisten = await getCurrentWindow().onCloseRequested(async () => {
         await invoke('kill_backend').catch(() => {})
-        await exit(0)
       })
     }
 

src/frontend/src/data/changelog.ts

@@ -22,6 +22,14 @@ export type ChangelogVersion = {
 }
 
 export const CHANGELOG: ChangelogVersion[] = [
+  {
+    version: 'v0.1.21',
+    datum: 'März 2026',
+    eintraege: [
+      { typ: 'fix', text: 'App schließt wieder sofort – kein preventDefault mehr im Close-Handler' },
+      { typ: 'fix', text: 'Backend-Prozess wird jetzt zuverlässig beendet – taskkill /T läuft jetzt VOR dem child.kill(), damit der Python-Child-Prozess (PyInstaller) nicht als Waise weiterläuft' },
+    ],
+  },
   {
     version: 'v0.1.19',
     datum: 'März 2026',