From 747362b88be9ecbf1e1b3440932bf277be315ac3 Mon Sep 17 00:00:00 2001
From: NotAShelf <raf@notashelf.dev>
Date: Sun, 18 May 2025 13:38:33 +0300
Subject: [PATCH] daemon: replace `debug_assert` with an actual error

---
 src/daemon.rs | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/daemon.rs b/src/daemon.rs
index 27748fd..9ce40a0 100644
--- a/src/daemon.rs
+++ b/src/daemon.rs
@@ -62,13 +62,13 @@ fn idle_multiplier(idle_secs: u64) -> f32 {
 ///
 /// Returns Ok with the calculated interval, or Err if the configuration is invalid
 fn compute_new(params: &IntervalParams, system_history: &SystemHistory) -> Result<u64, String> {
-    // Catch invalid configurations during development
-    debug_assert!(
-        params.max_interval >= params.min_interval,
-        "Configuration error: max_interval ({}) < min_interval ({})",
-        params.max_interval,
-        params.min_interval
-    );
+    // Validate interval configuration
+    if params.max_interval < params.min_interval {
+        return Err(format!(
+            "Invalid interval configuration: max_interval ({}) is less than min_interval ({})",
+            params.max_interval, params.min_interval
+        ));
+    }
 
     // Start with base interval
     let mut adjusted_interval = params.base_interval;