Update requirements.txt

pyproject.toml

@@ -11,7 +11,7 @@ dependencies = [
     "pydantic==2.10.6",
     "python-dotenv>=1.0.1",
     "python-levenshtein>=0.26.1",
-    "specklepy>=2.21.3",
+    "specklepy>=2.21.2",
 ]
 
 

src/rules.py

@@ -253,11 +253,9 @@ class PropertyRules:
         parameter_value = PropertyRules.get_parameter_value(speckle_object, parameter_name)
         if parameter_value is None:
             return False
-        try:
-            parameter_value = float(parameter_value)
-        except (ValueError, TypeError):
-            return False  # Return False if the value cannot be converted to a number
-        return parameter_value > PropertyRules.parse_number_from_string(threshold)
+        if not isinstance(parameter_value, int | float):
+            raise ValueError(f"Parameter value must be a number, got {type(parameter_value)}")
+        return parameter_value <= PropertyRules.parse_number_from_string(threshold)
 
     @staticmethod
     def is_parameter_value_less_than(speckle_object: Base, parameter_name: str, threshold: str) -> bool:
@@ -269,12 +267,9 @@ class PropertyRules:
         parameter_value = PropertyRules.get_parameter_value(speckle_object, parameter_name)
         if parameter_value is None:
             return False
-        try:
-            parameter_value = float(parameter_value)
-        except (ValueError, TypeError):
-            return False  # Return False if the value cannot be converted to a number
-
-        return parameter_value < PropertyRules.parse_number_from_string(threshold)
+        if not isinstance(parameter_value, int | float):
+            raise ValueError(f"Parameter value must be a number, got {type(parameter_value)}")
+        return parameter_value >= PropertyRules.parse_number_from_string(threshold)
 
     @staticmethod
     def is_parameter_value_in_range(speckle_object: Base, parameter_name: str, value_range: str) -> bool:
@@ -290,10 +285,8 @@ class PropertyRules:
         parameter_value = PropertyRules.get_parameter_value(speckle_object, parameter_name)
         if parameter_value is None:
             return False
-        try:
-            parameter_value = float(parameter_value)
-        except (ValueError, TypeError):
-            return False  # Return False if the value cannot be converted to a number
+        if not isinstance(parameter_value, int | float):
+            raise ValueError(f"Parameter value must be a number, got {type(parameter_value)}")
 
         return min_value <= parameter_value <= max_value
 

tests/test_user_entry.py

@@ -16,8 +16,8 @@ from src.rule_processor import SeverityLevel, get_severity
         ("ERROR", SeverityLevel.ERROR),
         ("error", SeverityLevel.ERROR),
         ("Error", SeverityLevel.ERROR),
-        ("WARN", SeverityLevel.WARNING),  
-        ("warn", SeverityLevel.WARNING),  
+        ("WARN", SeverityLevel.WARNING),  # Invalid → Defaults to ERROR
+        ("warn", SeverityLevel.WARNING),  # Invalid → Defaults to ERROR
         ("Critical", SeverityLevel.ERROR),  # Invalid → Defaults to ERROR
         ("Severe", SeverityLevel.ERROR),  # Invalid → Defaults to ERROR
         ("", SeverityLevel.ERROR),  # Empty string → Defaults to ERROR