feat: more information to logging

also fix game library to don't list non-game data
better game-library error catching

Author: viniciuslrangel

bun.lock

@@ -31,6 +31,7 @@
         "clsx": "^2.1.1",
         "date-fns": "^4.1.0",
         "dequal": "^2.0.3",
+        "error-stack-parser": "^2.1.4",
         "jotai": "^2.11.1",
         "jotai-optics": "^0.4.0",
         "jotai-tanstack-query": "^0.9.0",
@@ -557,6 +558,8 @@
 
     "enhanced-resolve": ["[email protected]", "", { "dependencies": { "graceful-fs": "^4.2.4", "tapable": "^2.2.0" } }, "sha512-ZSW3ma5GkcQBIpwZTSRAI8N71Uuwgs93IezB7mf7R60tC8ZbJideoDNKjHn2O9KIlx6rkGTTEk1xUCK2E1Y2Yg=="],
 
+    "error-stack-parser": ["[email protected]", "", { "dependencies": { "stackframe": "^1.3.4" } }, "sha512-Sk5V6wVazPhq5MhpO+AUxJn5x7XSXGl1R93Vn7i+zS15KDVxQijejNCrz8340/2bgLBjR9GtEG8ZVKONDjcqGQ=="],
+
     "esbuild": ["[email protected]", "", { "optionalDependencies": { "@esbuild/aix-ppc64": "0.24.2", "@esbuild/android-arm": "0.24.2", "@esbuild/android-arm64": "0.24.2", "@esbuild/android-x64": "0.24.2", "@esbuild/darwin-arm64": "0.24.2", "@esbuild/darwin-x64": "0.24.2", "@esbuild/freebsd-arm64": "0.24.2", "@esbuild/freebsd-x64": "0.24.2", "@esbuild/linux-arm": "0.24.2", "@esbuild/linux-arm64": "0.24.2", "@esbuild/linux-ia32": "0.24.2", "@esbuild/linux-loong64": "0.24.2", "@esbuild/linux-mips64el": "0.24.2", "@esbuild/linux-ppc64": "0.24.2", "@esbuild/linux-riscv64": "0.24.2", "@esbuild/linux-s390x": "0.24.2", "@esbuild/linux-x64": "0.24.2", "@esbuild/netbsd-arm64": "0.24.2", "@esbuild/netbsd-x64": "0.24.2", "@esbuild/openbsd-arm64": "0.24.2", "@esbuild/openbsd-x64": "0.24.2", "@esbuild/sunos-x64": "0.24.2", "@esbuild/win32-arm64": "0.24.2", "@esbuild/win32-ia32": "0.24.2", "@esbuild/win32-x64": "0.24.2" }, "bin": { "esbuild": "bin/esbuild" } }, "sha512-+9egpBW8I3CD5XPe0n6BfT5fxLzxrlDzqydF3aviG+9ni1lDC/OvMHcxqEFV0+LANZG5R1bFMWfUrjVsdwxJvA=="],
 
     "escalade": ["[email protected]", "", {}, "sha512-WUj2qlxaQtO4g6Pq5c29GTcWGDyd8itL8zTlipgECz3JesAiiOKotd8JU6otB3PACgG6xkJUyVhboMS+bje/jA=="],
@@ -767,6 +770,8 @@
 
     "spawn-sync": ["[email protected]", "", { "dependencies": { "concat-stream": "^1.4.7", "os-shim": "^0.1.2" } }, "sha512-9DWBgrgYZzNghseho0JOuh+5fg9u6QWhAWa51QC7+U5rCheZ/j1DrEZnyE0RBBRqZ9uEXGPgSSM0nky6burpVw=="],
 
+    "stackframe": ["[email protected]", "", {}, "sha512-oeVtt7eWQS+Na6F//S4kJ2K2VbRlS9D43mAlMyVpVWovy9o+jfgH8O9agzANzaiLjclA0oYzUXEM4PurhSUChw=="],
+
     "string_decoder": ["[email protected]", "", { "dependencies": { "safe-buffer": "~5.1.0" } }, "sha512-n/ShnvDi6FHbbVfviro+WojiFzv+s8MPMHBczVePfUpDJLwoLT0ht1l4YwBCbi8pJAveEEdnkHyPyTP/mzRfwg=="],
 
     "strip-json-comments": ["[email protected]", "", {}, "sha512-6fPc+R4ihwqP6N/aIv2f1gMH8lOVtWQHoqC4yK6oSDVVocumAsfCqjkXnqiYMhmMwS/mEHLp7Vehlt3ql6lEig=="],

package.json

@@ -40,6 +40,7 @@
         "clsx": "^2.1.1",
         "date-fns": "^4.1.0",
         "dequal": "^2.0.3",
+        "error-stack-parser": "^2.1.4",
         "jotai": "^2.11.1",
         "jotai-optics": "^0.4.0",
         "jotai-tanstack-query": "^0.9.0",

src-tauri/Cargo.lock

@@ -38,6 +38,7 @@ tauri-plugin-store = "2"
 tauri-plugin-updater = "2"
 tauri-plugin-upload = { version = "2", features = ["rustls-tls"] }
 thiserror = "2.0.11"
+time = "0.3"
 tokio = { version = "1.44.1", features = ["full"] }
 zerocopy = { version = "0.8.17", features = ["std"] }
 zerocopy-derive = "0.8.17"

src-tauri/Cargo.toml

@@ -1,6 +1,7 @@
 use crate::game_process::state::GameBridge;
 use log::info;
 use tauri::Manager;
+use crate::logger::build_log_plugin;
 
 mod file_format;
 mod game_process;
@@ -9,9 +10,10 @@ mod logger;
 mod utility_commands;
 
 pub fn run() {
+    let logger_plugin = build_log_plugin();
     tauri::Builder::default()
         .invoke_handler(handlers::all_handlers())
-        .plugin(logger::build_log_plugin())
+        .plugin(logger_plugin)
         .plugin(tauri_plugin_dialog::init())
         .plugin(tauri_plugin_fs::init())
         .plugin(tauri_plugin_opener::init())

src-tauri/src/lib.rs

@@ -1,7 +1,8 @@
+use log::kv::{Key, ToKey};
 use log::LevelFilter;
 use std::env;
 use tauri::plugin::TauriPlugin;
-use tauri::Runtime;
+use tauri::Wry;
 use tauri_plugin_log::fern::colors::{Color, ColoredLevelConfig};
 use tauri_plugin_log::{RotationStrategy, Target, TargetKind};
 
@@ -61,9 +62,14 @@ fn parse_spec(spec: &str) -> ParseResult {
     result
 }
 
-pub fn build_log_plugin<R: Runtime>() -> TauriPlugin<R> {
+pub fn build_log_plugin() -> TauriPlugin<Wry> {
+    let tz = tauri_plugin_log::TimezoneStrategy::UseLocal;
+    let format =
+        time::format_description::parse("[[[year]-[month]-[day]][[[hour]:[minute]:[second]]")
+            .unwrap();
+    let column_key = Key::from_str("column");
+
     let mut log_plugin = tauri_plugin_log::Builder::new()
-        .timezone_strategy(tauri_plugin_log::TimezoneStrategy::UseLocal)
         .rotation_strategy(RotationStrategy::KeepAll)
         .max_file_size(1024 * 1024 * 32) // 32MB
         .targets([
@@ -103,5 +109,37 @@ pub fn build_log_plugin<R: Runtime>() -> TauriPlugin<R> {
         }
     }
 
+    let log_plugin = log_plugin.format(move |out, message, record| {
+        let target = record.target();
+        if target.starts_with("webview") {
+            match (
+                record.file(),
+                record.line(),
+                record.key_values().get(column_key.to_key()),
+            ) {
+                (Some(file), Some(line), Some(column)) => {
+                    out.finish(format_args!(
+                        "{}[{}][webview {}:{}:{}] {}",
+                        tz.get_now().format(&format).unwrap(),
+                        record.level(),
+                        file,
+                        line,
+                        column,
+                        message
+                    ));
+                    return;
+                }
+                _ => {}
+            }
+        }
+        out.finish(format_args!(
+            "{}[{}][{}] {}",
+            tz.get_now().format(&format).unwrap(),
+            record.level(),
+            record.target(),
+            message
+        ))
+    });
+
     log_plugin.build()
 }

src-tauri/src/logger.rs

@@ -76,7 +76,7 @@ export async function installUpdate() {
         console.info("Update Installed. Restarting app");
         await relaunch();
     } catch (e: unknown) {
-        console.error(e);
+        console.error("download update error", e);
         toast.error("Error downloading update: " + stringifyError(e));
     }
 }

src/handlers/auto-update.ts

@@ -3,21 +3,65 @@ import {
     debug,
     error,
     info,
+    type LogOptions,
     trace,
     warn,
 } from "@tauri-apps/plugin-log";
+import { format } from "date-fns";
+import ErrorStackParser from "error-stack-parser";
 
 function forwardConsole(
     fnName: "log" | "trace" | "debug" | "info" | "warn" | "error",
-    logger: (message: string) => Promise<void>,
+    logger: (message: string, options?: LogOptions) => Promise<void>,
+    level: string,
 ) {
     const original = console[fnName];
     // biome-ignore lint/suspicious/noExplicitAny: This is the original signature
-    console[fnName] = (...data: any[]) => {
-        original(...data);
+    console[fnName] = function (...data: any[]) {
+        let callFrame: ErrorStackParser.StackFrame | undefined = undefined;
+        try {
+            callFrame = ErrorStackParser.parse(new Error())?.[1];
+        } catch (_e) {
+            void 0;
+        }
+        const now = new Date();
+        let prefix = "";
+        prefix += `${format(now, "[y-MM-dd][HH:mm:ss]")}`;
+        prefix += `[${level}]`;
+        prefix += callFrame?.fileName
+            ? `[webview ${callFrame.fileName}:${callFrame.lineNumber}:${callFrame.columnNumber}]`
+            : "[webview]";
+        prefix += "\n";
+
+        if (typeof data?.[0] === "string") {
+            original(prefix + data.shift(), ...data);
+        } else {
+            original(prefix, ...data);
+        }
+
+        let opts: LogOptions | undefined;
+        if (callFrame) {
+            opts = {
+                file: callFrame.fileName?.replace(
+                    /^http:\/\/(localhost|127\.0\.0\.1)(:\d{1,5})?/,
+                    "",
+                ),
+                line: callFrame.lineNumber,
+                keyValues: {
+                    column: callFrame.columnNumber?.toString(),
+                },
+            };
+        }
+
         logger(
             data
-                .map((e) => {
+                .map((e, i) => {
+                    if (e instanceof Error) {
+                        return (
+                            (i > 0 ? "\n" : "") +
+                            (e.stack || `${e.name}: ${e.message}`)
+                        );
+                    }
                     switch (typeof e) {
                         case "string":
                         case "number":
@@ -33,6 +77,7 @@ function forwardConsole(
                     }
                 })
                 .join(" "),
+            opts,
         );
     };
 
@@ -41,12 +86,12 @@ function forwardConsole(
 
 export async function setupForwardingConsole() {
     const c = {
-        log: forwardConsole("log", info),
-        trace: forwardConsole("trace", trace),
-        debug: forwardConsole("debug", debug),
-        info: forwardConsole("info", info),
-        warn: forwardConsole("warn", warn),
-        error: forwardConsole("error", error),
+        log: forwardConsole("log", info, "INFO"),
+        trace: forwardConsole("trace", trace, "TRACE"),
+        debug: forwardConsole("debug", debug, "DEBUG"),
+        info: forwardConsole("info", info, "INFO"),
+        warn: forwardConsole("warn", warn, "WARN"),
+        error: forwardConsole("error", error, "ERROR"),
     } as const;
 
     window.addEventListener("error", (event) => {
@@ -68,7 +113,7 @@ export async function setupForwardingConsole() {
     });
 
     await attachLogger(({ level, message }) => {
-        if (message.includes("[webview:console.<computed>")) {
+        if (message.includes("][webview")) {
             return;
         }
         switch (level) {