fix: remove CREATE_NEW_PROCESS_GROUP flag for proper Ctrl-C handling

qnixsynapse · qnixsynapse · commit 82ace3e72e4b · 2025-07-29T19:56:04.000+05:30
CREATE_NEW_PROCESS_GROUP prevented GenerateConsoleCtrlEvent from working,
causing graceful shutdown failures. Removed to enable proper signal handling.
diff --git a/src-tauri/src/core/utils/extensions/inference_llamacpp_extension/server.rs b/src-tauri/src/core/utils/extensions/inference_llamacpp_extension/server.rs
@@ -167,10 +167,10 @@ pub async fn load_llama_model(
     command.stderr(Stdio::piped());
     #[cfg(all(windows, target_arch = "x86_64"))]
     {
-        use std::os::windows::process::CommandExt;
+        // use std::os::windows::process::CommandExt;
         const CREATE_NO_WINDOW: u32 = 0x0800_0000;
-        const CREATE_NEW_PROCESS_GROUP: u32 = 0x0000_0200;
-        command.creation_flags(CREATE_NO_WINDOW | CREATE_NEW_PROCESS_GROUP);
+        // const CREATE_NEW_PROCESS_GROUP: u32 = 0x0000_0200;
+        command.creation_flags(CREATE_NO_WINDOW);
     }
 
     // Spawn the child process

Original file line number	Diff line number	Diff line change
`@@ -167,10 +167,10 @@ pub async fn load_llama_model(`
`167`	`167`	`command.stderr(Stdio::piped());`
`168`	`168`	`#[cfg(all(windows, target_arch = "x86_64"))]`
`169`	`169`	`{`
`170`		`- use std::os::windows::process::CommandExt;`
	`170`	`+ // use std::os::windows::process::CommandExt;`
`171`	`171`	`const CREATE_NO_WINDOW: u32 = 0x0800_0000;`
`172`		`- const CREATE_NEW_PROCESS_GROUP: u32 = 0x0000_0200;`
`173`		`- command.creation_flags(CREATE_NO_WINDOW \| CREATE_NEW_PROCESS_GROUP);`
	`172`	`+ // const CREATE_NEW_PROCESS_GROUP: u32 = 0x0000_0200;`
	`173`	`+ command.creation_flags(CREATE_NO_WINDOW);`
`174`	`174`	`}`
`175`	`175`
`176`	`176`	`// Spawn the child process`