refactor: hidden window is noe enabled rn due to slow boot noted

Author: abose

src-tauri/src/boot_config.rs

@@ -1,29 +1,18 @@
-use std::sync::Arc;
-use once_cell::sync::OnceCell;
 use serde_json::Value;
 use serde::Serialize;
 use crate::utilities::read_json_file;
-use std::path::PathBuf;
-use std::fs::File;
-use std::io::Write;
-
-pub struct AppConstants {
-    pub tauri_config : Arc<tauri::Config>,
-    pub app_local_data_dir: std::path::PathBuf
-}
-pub static APP_CONSTANTS: OnceCell<AppConstants> = OnceCell::new();
+use std::path::{Path, PathBuf};
 
 #[derive(Serialize)]
 pub struct BootConfig {
     pub version: u32,
     pub start_as_hidden_window: bool,
 }
-static BOOT_CONFIG_FILE_NAME: &'static str = "boot_config.json";
 
-fn get_boot_config_file_path(app_local_data_dir: &PathBuf) -> PathBuf {
-    let mut config_file_path = app_local_data_dir.clone();
-    config_file_path.push(BOOT_CONFIG_FILE_NAME);
-    return config_file_path;
+fn get_boot_config_file_path(base_path: &Path) -> PathBuf {
+    let mut config_file_path = base_path.to_path_buf();
+    config_file_path.push("boot_config.json");
+    config_file_path
 }
 
 fn _set_boot_config(boot_config: &mut BootConfig, value: &Value) {
@@ -34,23 +23,31 @@ fn _set_boot_config(boot_config: &mut BootConfig, value: &Value) {
         .unwrap_or(false); // Default to `false` if missing or invalid
 }
 
-pub fn read_boot_config() -> BootConfig {
+/// Reads boot_config.json from the given `Option<PathBuf>`.
+/// If `None` is provided, it returns a default `BootConfig`.
+pub fn read_boot_config(base_path: &Option<PathBuf>) -> BootConfig {
     let mut boot_config = BootConfig {
         version: 1,
-        start_as_hidden_window: false
+        start_as_hidden_window: false,
     };
-    if let Some(app_constants) = APP_CONSTANTS.get() {
-        let boot_config_file_path = get_boot_config_file_path(&app_constants.app_local_data_dir);
+
+    if let Some(ref path) = base_path {
+        let boot_config_file_path = get_boot_config_file_path(path);
+
         match read_json_file(&boot_config_file_path) {
-            Some(value) =>{
+            Some(value) => {
                 _set_boot_config(&mut boot_config, &value);
             }
             None => {
-                eprintln!("No boot restore config file found {}", boot_config_file_path.display());
+                eprintln!(
+                    "No boot restore config file found at {}",
+                    boot_config_file_path.display()
+                );
             }
         }
+    } else {
+        eprintln!("Base path is None, using default boot config.");
     }
-    return boot_config;
-}
 
-// writing the boot config is always done from js to be simpler, as js is eazier to handle json.
+    boot_config
+}

src-tauri/src/init.rs

@@ -36,8 +36,10 @@ use native_dialog::{MessageDialog, MessageType};
 use regex::Regex;
 extern crate percent_encoding;
 use tauri::http::ResponseBuilder;
-use tauri::GlobalWindowEvent;
-mod init;
+use tauri::api::path::app_local_data_dir;
+use tauri::generate_context;
+use crate::utilities::ensure_dir_exists;
+use crate::boot_config::read_boot_config;
 mod bugsnag;
 mod utilities;
 mod boot_config;
@@ -258,6 +260,7 @@ fn zoom_window(window: tauri::Window, scale_factor: f64) {
 }
 
 // here in case you need to process windows events
+// use tauri::GlobalWindowEvent;
 // fn process_window_event(event: &GlobalWindowEvent) {
 //     if let tauri::WindowEvent::CloseRequested { .. } = event.event() {
 //         // this does nothing and is here if in future you need to persist something on window close.
@@ -366,6 +369,28 @@ fn main() {
         tauri_plugin_deep_link::prepare("io.phcode");
     }
 
+    let context = generate_context!();
+    let mut tauri_config = context.config().clone(); // Clone the config to modify it
+
+    // Get the app data directory before Tauri starts
+    let app_data_dir: Option<PathBuf> = app_local_data_dir(&tauri_config);
+
+    if let Some(dir) = &app_data_dir {
+        println!("App Data Directory: {}", dir.display());
+        ensure_dir_exists(dir);
+    } else {
+        eprintln!("Failed to retrieve app data directory.");
+    }
+
+    let boot_config = read_boot_config(&app_data_dir);
+
+    // Modify the first window's visibility based on boot_config.start_as_hidden_window
+    if let Some(first_window) = tauri_config.tauri.windows.get_mut(0) {
+        if !boot_config.start_as_hidden_window {
+            first_window.visible = true;
+        }
+    }
+
     // warning: any string that resembles the following strings will be rewritten in source in prod by build scripts.
     // This is so that app bundle IDs are correct. IF they are app bundle IDs use the strings. else dont.
     // do not use strings: "io.phcode.dev" "io.phcode.staging" "io.phcode" for anything other than bundle identifiers
@@ -439,15 +464,7 @@ fn main() {
             put_item, get_item, get_all_items, delete_item,
             _get_windows_drives, _rename_path, show_in_folder, move_to_trash, zoom_window,
             _get_clipboard_files, _open_url_in_browser_win])
-        .setup(|app| {
-            let boot_config = init::init_app(app);
-
-            if boot_config.start_as_hidden_window {
-                if let Some(main_window) = app.get_window("main") {
-                    main_window.hide().expect("Failed to hide main window");
-                }
-            }
-
+        .setup(move |_app| {
             #[cfg(target_os = "linux")]
             {
                 // In linux, f10 key press events are reserved for gtk-menu-bar-accel and not passed.
@@ -478,6 +495,6 @@ fn main() {
             }
             Ok(())
         })
-        .run(tauri::generate_context!())
+        .run(context)
         .expect("error while running tauri application");
 }

src-tauri/src/main.rs

@@ -484,4 +484,4 @@
             }
         ]
     }
-}
+}