wip: osu! updating

src-tauri/src/commands.rs

@@ -224,11 +224,14 @@ pub fn set_osu_config_value(
 }
 
 #[tauri::command]
-pub fn run_osu_updater(osu_path: String) -> Result<(), String> {
-    let mut updater_process = Command::new(osu_path.clone())
-        .arg("-repair")
-        .spawn()
-        .map_err(|e| e.to_string())?;
+pub fn run_osu_updater(folder: String) -> Result<(), String> {
+    let osu_exe_path = PathBuf::from(folder).join("osu!.exe");
+    let mut updater_process = Command::new(osu_exe_path)
+      .arg("-repair")
+      .spawn()
+      .map_err(|e| e.to_string())?;
+
+    
 
     thread::sleep(Duration::from_millis(500));
 
@@ -243,8 +246,15 @@ pub fn run_osu_updater(osu_path: String) -> Result<(), String> {
                 let process_id = process.pid();
                 let window_title = get_window_title_by_pid(process_id);
 
+                println!("updater_process id: {}", updater_process.id());
+                println!("process_id: {}", process_id.as_u32());
+
+                println!("found osu!.exe process with window title: {}", window_title);
+
                 if !window_title.is_empty() {
+                    println!("Killing osu!.exe process");
                     if let Ok(_) = process.kill_and_wait() {
+                        println!("osu!.exe process killed");
                         termination_thread_running = false;
                         break;
                     }

src-tauri/src/utils.rs

@@ -4,10 +4,7 @@ use std::os::windows::ffi::OsStringExt;
 use std::path::Path;
 use std::ptr;
 use sysinfo::Pid;
-use winapi::{
-    shared::minwindef::LPARAM,
-    um::winuser::{FindWindowW, GetWindowTextW, GetWindowThreadProcessId},
-};
+use winapi::um::winuser::{FindWindowW, GetWindowTextW, GetWindowThreadProcessId};
 
 pub fn check_folder_completeness<P: AsRef<Path>>(folder_path: P, required_files: &[&str]) -> f32 {
     let mut found = 0;
@@ -27,12 +24,10 @@ pub fn check_folder_completeness<P: AsRef<Path>>(folder_path: P, required_files:
 pub fn get_osu_user_config<P: AsRef<Path>>(
     osu_folder_path: P,
 ) -> Option<std::collections::HashMap<String, String>> {
-    // Ensure the osu! folder path is valid
     if !osu_folder_path.as_ref().exists() {
         return None;
     }
 
-    // get the osu!{username}.cfg file from the osu! folder
     let current_user = std::env::var("USERNAME").unwrap_or_else(|_| "Admin".to_string());
     let osu_config_path = osu_folder_path
         .as_ref()
@@ -41,7 +36,6 @@ pub fn get_osu_user_config<P: AsRef<Path>>(
         return None;
     }
 
-    // read the osu config and return it as a map, key and value are separated by ' = '
     let mut config_map = std::collections::HashMap::new();
     if let Ok(contents) = std::fs::read_to_string(osu_config_path) {
         for line in contents.lines() {
@@ -97,18 +91,15 @@ pub fn set_osu_user_config_value(
 pub fn get_osu_config<P: AsRef<Path>>(
     osu_folder_path: P,
 ) -> Option<std::collections::HashMap<String, String>> {
-    // Ensure the osu! folder path is valid
     if !osu_folder_path.as_ref().exists() {
         return None;
     }
 
-    // get the osu!.cfg file from the osu! folder
     let osu_config_path = osu_folder_path.as_ref().join("osu!.cfg");
     if !osu_config_path.exists() {
         return None;
     }
 
-    // read the osu config and return it as a map, key and value are separated by ' = '
     let mut config_map = std::collections::HashMap::new();
     if let Ok(contents) = std::fs::read_to_string(osu_config_path) {
         for line in contents.lines() {