From c2d083a10d2dd83fbe0d810972ccecd78d592b5a Mon Sep 17 00:00:00 2001
From: zhom <2717306+zhom@users.noreply.github.com>
Date: Wed, 13 Aug 2025 10:03:53 +0400
Subject: [PATCH] fix: properly warmup nodecar

---
 src-tauri/src/lib.rs | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/src-tauri/src/lib.rs b/src-tauri/src/lib.rs
index d024e4a..c4f62fb 100644
--- a/src-tauri/src/lib.rs
+++ b/src-tauri/src/lib.rs
@@ -119,14 +119,21 @@ impl<R: Runtime> WindowExt for WebviewWindow<R> {
 }
 
 #[tauri::command]
-async fn warm_up_nodecar() -> Result<(), String> {
-  use tokio::process::Command;
+async fn warm_up_nodecar(app: tauri::AppHandle) -> Result<(), String> {
+  use tauri_plugin_shell::ShellExt;
   use tokio::time::{timeout, Duration};
 
   let start_time = std::time::Instant::now();
 
-  let warmup_future = Command::new("nodecar").output();
-  match timeout(Duration::from_secs(15), warmup_future).await {
+  // Use sidecar to execute a fast, harmless command that ensures the binary is loaded
+  let cmd = app
+    .shell()
+    .sidecar("nodecar")
+    .map_err(|e| format!("Failed to create nodecar sidecar: {e}"))?
+    .arg("help")
+
+  let exec_future = async { cmd.output().await };
+  match timeout(Duration::from_secs(30), exec_future).await {
     Ok(Ok(_output)) => {
       let duration = start_time.elapsed();
       println!(
@@ -135,8 +142,8 @@ async fn warm_up_nodecar() -> Result<(), String> {
       );
       Ok(())
     }
-    Ok(Err(e)) => Err(format!("Failed to start nodecar warm-up: {e}")),
-    Err(_) => Err("Nodecar warm-up timed out after 15s".to_string()),
+    Ok(Err(e)) => Err(format!("Failed to execute nodecar for warm-up: {e}")),
+    Err(_) => Err("Nodecar warm-up timed out after 30s".to_string()),
   }
 }