FF-3106 fix: ContextAttributes.from_dict() should not treat bool as numeric (#67)

Author: rasendubi

eppo_client/bandit.py

@@ -52,7 +52,7 @@ def from_dict(cls, attributes: Attributes):
         numeric_attributes = {
             key: float(value)
             for key, value in attributes.items()
-            if isinstance(value, (int, float))
+            if isinstance(value, (int, float)) and not isinstance(value, bool)
         }
         categorical_attributes = {
             key: to_string(value)

eppo_client/version.py

@@ -1,4 +1,4 @@
 # Note to developers: When ready to bump to 4.0, please change
 # the `POLL_INTERVAL_SECONDS` constant in `eppo_client/constants.py`
 # to 30 seconds to match the behavior of the other server SDKs.
-__version__ = "3.5.3"
+__version__ = "3.5.4"

test/context_attributes_test.py

@@ -0,0 +1,14 @@
+from eppo_client.bandit import ContextAttributes
+
+
+def test_from_dict_treats_bools_as_categorical():
+    attrs = ContextAttributes.from_dict(
+        {
+            "categorical": True,
+        }
+    )
+
+    assert attrs.categorical_attributes == {
+        "categorical": "true",
+    }
+    assert attrs.numeric_attributes == {}