From 775690dcfc0fc7563c056b1e2421ffd8fae17623 Mon Sep 17 00:00:00 2001
From: null <60427892+null8626@users.noreply.github.com>
Date: Thu, 28 May 2026 14:35:10 +0700
Subject: [PATCH 1/2] fix: fix bugs from #4's conflict resolution

---
 src-tauri/src/package.rs    | 10 +++++-----
 src-tauri/src/script/ast.rs |  2 +-
 src-tauri/src/trigger.rs    | 36 +++++++++++++++++-------------------
 3 files changed, 23 insertions(+), 25 deletions(-)

diff --git a/src-tauri/src/package.rs b/src-tauri/src/package.rs
index 38aeca0..67be1d7 100644
--- a/src-tauri/src/package.rs
+++ b/src-tauri/src/package.rs
@@ -132,25 +132,25 @@ tauri_reexport! {
             self.state.read().unwrap().installed.clone()
         }
 
-        pub fn install_package(self: &Self, id: String) -> Result<(), String> {
+        pub fn install_package(self: &Self, id: String) -> Result<(), Cow<'static, str>> {
             {
                 let mut state = self.state.write().map_err(|e| e.to_string())?;
                 if !self.get_available_packages().iter().any(|p| p.id == id) {
-                    return Err("Package not found".to_string());
+                    return Err("Package not found".into());
                 }
                 if state.installed.contains(&id) {
-                    return Err("Package already installed".to_string());
+                    return Err("Package already installed".into());
                 }
                 state.installed.push(id);
             }
             self.save()
         }
 
-        pub fn uninstall_package(self: &Self, id: String) -> Result<(), String> {
+        pub fn uninstall_package(self: &Self, id: String) -> Result<(), Cow<'static, str>> {
             {
                 let mut state = self.state.write().map_err(|e| e.to_string())?;
                 if !state.installed.contains(&id) {
-                    return Err("Package not installed".to_string());
+                    return Err("Package not installed".into());
                 }
                 state.installed.retain(|p| p != &id);
             }
diff --git a/src-tauri/src/script/ast.rs b/src-tauri/src/script/ast.rs
index e3077b1..ba421af 100644
--- a/src-tauri/src/script/ast.rs
+++ b/src-tauri/src/script/ast.rs
@@ -109,7 +109,7 @@ impl Value<'_> {
             Self::Num(n) => *n != 0.0,
             Self::Str(s) => match s {
                 Cow::Borrowed(s) => !s.is_empty(),
-                Cow::Owned(s) => s.len() != 0,
+                Cow::Owned(s) => !s.is_empty(),
             },
             Self::Nil => false,
             Self::Fn { .. } => true,
diff --git a/src-tauri/src/trigger.rs b/src-tauri/src/trigger.rs
index 9b3fdce..bc753c8 100644
--- a/src-tauri/src/trigger.rs
+++ b/src-tauri/src/trigger.rs
@@ -244,9 +244,9 @@ tauri_reexport! {
             self.global_vars.read().unwrap().clone()
         }
 
-        pub fn add_global_var(self: &Self, name: String, script: String) -> Result<GlobalVar, String> {
+        pub fn add_global_var(self: &Self, name: String, script: String) -> Result<GlobalVar, Cow<'static, str>> {
             let global_var = GlobalVar {
-                id: uuid::Uuid::new_v4().to_string(),
+                id: uuid::Uuid::new_v4(),
                 name,
                 script,
                 enabled: true,
@@ -261,28 +261,26 @@ tauri_reexport! {
 
         pub fn update_global_var(
             self: &Self,
-            id: String,
+            id: Uuid,
             name: Option<String>,
             script: Option<String>,
             enabled: Option<bool>
-        ) -> Result<GlobalVar, String> {
-            {
-                let mut global_vars = self.global_vars.write().map_err(|e| e.to_string())?;
-                let gv = global_vars
-                    .iter_mut()
-                    .find(|g| g.id == id)
-                    .ok_or_else(|| "Global variable not found".to_string())?;
+        ) -> Result<GlobalVar, Cow<'static, str>> {
+            let mut global_vars = self.global_vars.write().map_err(|e| e.to_string())?;
+            let gv = global_vars
+                .iter_mut()
+                .find(|g| g.id == id)
+                .ok_or(Cow::Borrowed("Global variable not found"))?;
 
-                if let Some(v) = name { gv.name = v; }
-                if let Some(v) = script { gv.script = v; }
-                if let Some(v) = enabled { gv.enabled = v; }
+            if let Some(v) = name { gv.name = v; }
+            if let Some(v) = script { gv.script = v; }
+            if let Some(v) = enabled { gv.enabled = v; }
 
-                Ok(gv.clone())
-            }
-            .and_then(|gv| {
-                self.save_global_vars()?;
-                Ok(gv)
-            })
+            let result = Ok(gv.clone());
+
+            self.save_global_vars()?;
+
+            result
         }
     }
 }

From ac719b5fca804acb2e12d2a27c09ad1585021ae0 Mon Sep 17 00:00:00 2001
From: null <60427892+null8626@users.noreply.github.com>
Date: Thu, 28 May 2026 14:55:27 +0700
Subject: [PATCH 2/2] refactor: use Uuid instead of manually accessing the uuid
 module first

---
 src-tauri/src/trigger.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src-tauri/src/trigger.rs b/src-tauri/src/trigger.rs
index bc753c8..03f78e9 100644
--- a/src-tauri/src/trigger.rs
+++ b/src-tauri/src/trigger.rs
@@ -189,7 +189,7 @@ tauri_reexport! {
         ) -> Result<Trigger, String> {
             let now = chrono::Utc::now().to_rfc3339();
             let trigger = Trigger {
-                id: uuid::Uuid::new_v4().to_string(),
+                id: Uuid::new_v4().to_string(),
                 trigger_text,
                 replacement,
                 enabled: true,
@@ -246,7 +246,7 @@ tauri_reexport! {
 
         pub fn add_global_var(self: &Self, name: String, script: String) -> Result<GlobalVar, Cow<'static, str>> {
             let global_var = GlobalVar {
-                id: uuid::Uuid::new_v4(),
+                id: Uuid::new_v4(),
                 name,
                 script,
                 enabled: true,