add file logging

yujonglee · yujonglee · commit 0f08cf16f858 · 2025-09-01T13:17:38.000-07:00
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/plugins/tracing/Cargo.toml b/plugins/tracing/Cargo.toml
@@ -17,12 +17,14 @@ specta-typescript = { workspace = true }
 tauri = { workspace = true, features = ["tray-icon", "image-png"] }
 tauri-specta = { workspace = true, features = ["derive", "typescript"] }
 
-tauri-plugin-sentry = { workspace = true }
+tauri-plugin-sentry = { workspace = true, features = ["tracing"] }
 
+dirs = { workspace = true }
 serde = { workspace = true, features = ["derive"] }
 serde_json = { workspace = true }
 specta = { workspace = true, features = ["derive"] }
 thiserror = { workspace = true }
 
 tracing = { workspace = true }
+tracing-appender = { version = "0.2" }
 tracing-subscriber = { workspace = true, features = ["env-filter", "chrono"] }
diff --git a/plugins/tracing/src/lib.rs b/plugins/tracing/src/lib.rs
@@ -5,17 +5,16 @@ mod ext;
 pub use errors::*;
 pub use ext::*;
 
+use std::{fs, path::PathBuf};
+use tauri::Manager;
+
+use tracing_appender::{non_blocking::WorkerGuard, rolling};
 use tracing_subscriber::{
     fmt, prelude::__tracing_subscriber_SubscriberExt, util::SubscriberInitExt, EnvFilter,
 };
 
 const PLUGIN_NAME: &str = "tracing";
 
-pub type ManagedState = std::sync::Mutex<State>;
-
-#[derive(Default)]
-pub struct State {}
-
 fn make_specta_builder() -> tauri_specta::Builder<tauri::Wry> {
     tauri_specta::Builder::<tauri::Wry>::new()
         .plugin_name(PLUGIN_NAME)
@@ -32,15 +31,25 @@ pub fn init() -> tauri::plugin::TauriPlugin<tauri::Wry> {
         .setup(move |app, _api| {
             specta_builder.mount_events(app);
 
-            {
-                let env_filter = EnvFilter::try_from_default_env()
-                    .unwrap_or_else(|_| EnvFilter::new("info"))
-                    .add_directive("ort=warn".parse().unwrap());
+            let env_filter = EnvFilter::try_from_default_env()
+                .unwrap_or_else(|_| EnvFilter::new("info"))
+                .add_directive("ort=warn".parse().unwrap());
+
+            let bundle_id = app.config().identifier.clone();
 
+            if let Some((file_writer, guard)) = make_file_writer_if_enabled(true, &bundle_id) {
                 tracing_subscriber::Registry::default()
+                    .with(env_filter)
+                    .with(tauri_plugin_sentry::sentry::integrations::tracing::layer())
                     .with(fmt::layer())
+                    .with(fmt::layer().with_ansi(false).with_writer(file_writer))
+                    .init();
+                assert!(app.manage(guard));
+            } else {
+                tracing_subscriber::Registry::default()
                     .with(env_filter)
                     .with(tauri_plugin_sentry::sentry::integrations::tracing::layer())
+                    .with(fmt::layer())
                     .init();
             }
 
@@ -49,6 +58,29 @@ pub fn init() -> tauri::plugin::TauriPlugin<tauri::Wry> {
         .build()
 }
 
+fn make_file_writer_if_enabled(
+    enabled: bool,
+    bundle_identifier: &str,
+) -> Option<(tracing_appender::non_blocking::NonBlocking, WorkerGuard)> {
+    if !enabled {
+        return None;
+    }
+
+    let base_dir: PathBuf = match dirs::data_dir() {
+        Some(p) => p,
+        None => return None,
+    };
+
+    let logs_dir = base_dir.join(bundle_identifier).join("logs");
+    if let Err(_e) = fs::create_dir_all(&logs_dir) {
+        return None;
+    }
+
+    let file_appender = rolling::daily(&logs_dir, "log.txt");
+    let (non_blocking, guard) = tracing_appender::non_blocking(file_appender);
+    Some((non_blocking, guard))
+}
+
 #[cfg(test)]
 mod test {
     use super::*;
diff --git a/plugins/tray/src/ext.rs b/plugins/tray/src/ext.rs
@@ -136,24 +136,26 @@ impl<T: tauri::Manager<tauri::Wry>> TrayPluginExt<tauri::Wry> for T {
                         let app_commit = app.get_git_hash();
                         let app_backends = app.list_ggml_backends();
 
+                        let backends_formatted = app_backends
+                            .iter()
+                            .map(|b| format!("{:?}", b))
+                            .collect::<Vec<_>>()
+                            .join("\n");
+
                         let message = format!(
-                            "{} v{}\n\nSHA: {}\n\nBackends: {}",
-                            app_name,
-                            app_version,
-                            app_commit,
-                            app_backends
-                                .iter()
-                                .map(|b| format!("{:?}", b))
-                                .collect::<Vec<_>>()
-                                .join("\n")
+                            "• App Name: {}\n• App Version: {}\n• SHA:\n  {}\n• Backends:\n{}",
+                            app_name, app_version, app_commit, backends_formatted
                         );
 
                         let app_clone = app.clone();
 
                         app.dialog()
                             .message(&message)
                             .title("About Hyprnote")
-                            .buttons(MessageDialogButtons::OkCustom("Copy".to_string()))
+                            .buttons(MessageDialogButtons::OkCancelCustom(
+                                "Copy".to_string(),
+                                "Cancel".to_string(),
+                            ))
                             .show(move |result| {
                                 if result {
                                     let _ = app_clone.clipboard().write_text(&message);
