From abb37ecd704ec3cd77d4d86f90987a26aad8a7d1 Mon Sep 17 00:00:00 2001
From: Spencer Phillip Young <spencer.young@spyoung.com>
Date: Tue, 21 Jan 2025 20:43:32 -0800
Subject: [PATCH] return error instead of panic

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

diff --git a/src/lib.rs b/src/lib.rs
index 08e9974..f547f0c 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -382,8 +382,10 @@ fn set_primary_monitor(display_name: String) -> PyResult<bool> {
         }
     }
 
-    // todo: raise a proper exception instead of a panic exception
-    assert!(maybe_this_monitor.is_some(), "Monitor with name {:?} not found", display_name);
+    if maybe_this_monitor.is_none() {
+        let err = PyErr::new::<PyValueError, _>((format!("Monitor with name {:?} not found", display_name),));
+        return Err(err)
+    }
 
     let this_monitor = maybe_this_monitor.unwrap();