Fixed issues

Author: Jordonbc

Backend/src/lib.rs

@@ -38,9 +38,23 @@ mod validate;
 mod workarounds;
 
 /// Loads `Client/.env` for local development without overwriting existing env vars.
+///
+/// Missing .env file is silently ignored. Malformed or unreadable .env files
+/// are reported with context for debugging before structured logging is ready.
 fn load_local_dotenv() {
     let dotenv_path = std::path::Path::new(env!("CARGO_MANIFEST_DIR")).join("../.env");
-    let _ = dotenvy::from_path(&dotenv_path);
+
+    match dotenvy::from_path(&dotenv_path) {
+        Ok(_) => {}
+        Err(dotenvy::Error::Io(error)) if error.kind() == std::io::ErrorKind::NotFound => {}
+        Err(err) => {
+            eprintln!(
+                "warning: failed to load local .env from {}: {}",
+                dotenv_path.display(),
+                err
+            );
+        }
+    }
 }
 
 /// Selects preferred backend from settings or first available plugin backend.

Backend/src/tauri_commands/commit.rs

@@ -145,12 +145,10 @@ pub async fn commit_selected<R: Runtime>(
         on(VcsEvent::Info {
             msg: "Staging selected files…".into(),
         });
-        repo.inner()
-            .stage_paths(&paths)
-            .map_err(|e| {
-                error!("stage_paths failed: {e}");
-                e.to_string()
-            })?;
+        repo.inner().stage_paths(&paths).map_err(|e| {
+            error!("stage_paths failed: {e}");
+            e.to_string()
+        })?;
 
         on(VcsEvent::Info {
             msg: "Writing commit…".into(),
@@ -322,12 +320,10 @@ pub async fn commit_patch_and_files<R: Runtime>(
             on(VcsEvent::Info {
                 msg: "Staging selected files…".into(),
             });
-            repo.inner()
-                .stage_paths(&paths)
-                .map_err(|e| {
-                    error!("stage_paths failed: {e}");
-                    e.to_string()
-                })?;
+            repo.inner().stage_paths(&paths).map_err(|e| {
+                error!("stage_paths failed: {e}");
+                e.to_string()
+            })?;
             repo.inner()
                 .commit(&message, &name, &email, &paths)
                 .map_err(|e| e.to_string())?

Backend/src/validate.rs

@@ -1,6 +1,15 @@
 // Copyright © 2025-2026 OpenVCS Contributors
 // SPDX-License-Identifier: GPL-3.0-or-later
 use std::path::Path;
+use std::sync::LazyLock;
+
+/// Regex pattern for scp-like Git URLs.
+static SCP_LIKE_RE: LazyLock<regex::Regex> =
+    LazyLock::new(|| regex::Regex::new(r"^[\w.-]+@[\w.-]+:[\w./-]+\.git$").unwrap());
+
+/// Regex pattern for Windows absolute paths.
+static WIN_ABS_RE: LazyLock<regex::Regex> =
+    LazyLock::new(|| regex::Regex::new(r"^[A-Za-z]:[\\/]").unwrap());
 
 #[derive(serde::Serialize)]
 pub struct Validation {
@@ -51,8 +60,7 @@ fn is_probably_git_url(u: &str) -> bool {
         return true;
     }
     // scp-like: git@host:org/repo.git
-    let scp_like = regex::Regex::new(r"^[\w.-]+@[\w.-]+:[\w./-]+\.git$").unwrap();
-    if scp_like.is_match(u) {
+    if SCP_LIKE_RE.is_match(u) {
         return true;
     }
     false
@@ -76,8 +84,7 @@ fn looks_like_path(s: &str) -> bool {
         return true;
     }
     // Windows drive letter absolute, e.g. C:\...
-    let win_abs = regex::Regex::new(r"^[A-Za-z]:[\\/]").unwrap();
-    win_abs.is_match(s)
+    WIN_ABS_RE.is_match(s)
 }
 
 /// Validates whether a string looks like a supported Git URL.

Frontend/src/scripts/main.ts

@@ -390,14 +390,22 @@ async function boot() {
                 break;
             case '__plugin_menu_action__': {
                 if (!TAURI.has) { notify('Plugin actions are available in the desktop app'); break; }
-                const pluginId = String(payload?.pluginId || '').trim();
-                const actionId = String(payload?.actionId || '').trim();
-                if (!pluginId || !actionId) break;
+                const pluginId = typeof payload?.pluginId === 'string' ? payload.pluginId.trim() : '';
+                const actionId = typeof payload?.actionId === 'string' ? payload.actionId.trim() : '';
+                if (!pluginId || !actionId) {
+                    console.warn(`Plugin menu action skipped: missing pluginId (${!!pluginId}) or actionId (${!!actionId})`);
+                    notify(!pluginId && !actionId
+                        ? 'Plugin action is missing plugin and action IDs'
+                        : !pluginId
+                            ? 'Plugin action missing plugin ID'
+                            : `Plugin action missing action for "${pluginId}"`);
+                    break;
+                }
                 try {
                     await invokePluginAction(pluginId, actionId);
                 } catch (e) {
                     console.error(`Plugin menu action failed: ${pluginId}/${actionId}`, e);
-                    notify('Plugin action failed');
+                    notify(`Plugin action for "${pluginId}" failed`);
                 }
                 break;
             }