envflags 0.1.0__tar.gz → 0.2.0__tar.gz

{envflags-0.1.0 → envflags-0.2.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: envflags
-Version: 0.1.0
+Version: 0.2.0
 Summary: Python SDK for envflags
 Author: envflags
 License: MIT
@@ -21,13 +21,16 @@ flags = create_client("envflags_server_...").initialize()
 try:
     enabled = flags.bool_variation(
         "new-checkout",
-        {"key": "user-123", "country": "BE"},
         False,
+        {"key": "user-123", "country": "BE"},
     )
 finally:
     flags.close()
 ```
 
+Context is optional. Omit it for flags that do not depend on who is asking; pass
+it last when targeting needs a user.
+
 The client polls every 30 seconds and fetches a payload only when its version
 changes. Set `poll_interval=None` to refresh manually. Supply a previously
 persisted payload through `bootstrap` to remain available when initialization

{envflags-0.1.0 → envflags-0.2.0}/README.md

@@ -10,13 +10,16 @@ flags = create_client("envflags_server_...").initialize()
 try:
     enabled = flags.bool_variation(
         "new-checkout",
-        {"key": "user-123", "country": "BE"},
         False,
+        {"key": "user-123", "country": "BE"},
     )
 finally:
     flags.close()
 ```
 
+Context is optional. Omit it for flags that do not depend on who is asking; pass
+it last when targeting needs a user.
+
 The client polls every 30 seconds and fetches a payload only when its version
 changes. Set `poll_interval=None` to refresh manually. Supply a previously
 persisted payload through `bootstrap` to remain available when initialization

{envflags-0.1.0 → envflags-0.2.0}/envflags/client.py

@@ -121,7 +121,10 @@ class Client:
             thread.join()
 
     def variation(
-        self, flag_key: str, context: Context, default_value: T
+        self,
+        flag_key: str,
+        default_value: T,
+        context: Optional[Context] = None,
     ) -> Any:
         with self._condition:
             payload = self._payload
@@ -130,21 +133,30 @@ class Client:
         return evaluate(payload, flag_key, context, default_value)
 
     def bool_variation(
-        self, flag_key: str, context: Context, default_value: bool
+        self,
+        flag_key: str,
+        default_value: bool,
+        context: Optional[Context] = None,
     ) -> bool:
-        value = self.variation(flag_key, context, default_value)
+        value = self.variation(flag_key, default_value, context)
         return value if isinstance(value, bool) else default_value
 
     def string_variation(
-        self, flag_key: str, context: Context, default_value: str
+        self,
+        flag_key: str,
+        default_value: str,
+        context: Optional[Context] = None,
     ) -> str:
-        value = self.variation(flag_key, context, default_value)
+        value = self.variation(flag_key, default_value, context)
         return value if isinstance(value, str) else default_value
 
     def number_variation(
-        self, flag_key: str, context: Context, default_value: float
+        self,
+        flag_key: str,
+        default_value: float,
+        context: Optional[Context] = None,
     ) -> float:
-        value = self.variation(flag_key, context, default_value)
+        value = self.variation(flag_key, default_value, context)
         if (
             isinstance(value, bool)
             or not isinstance(value, (int, float))
@@ -154,9 +166,12 @@ class Client:
         return float(value)
 
     def json_variation(
-        self, flag_key: str, context: Context, default_value: T
+        self,
+        flag_key: str,
+        default_value: T,
+        context: Optional[Context] = None,
     ) -> Any:
-        return self.variation(flag_key, context, default_value)
+        return self.variation(flag_key, default_value, context)
 
     def _perform_refresh(self) -> bool:
         with self._condition:

{envflags-0.1.0 → envflags-0.2.0}/envflags/evaluate.py

@@ -25,22 +25,23 @@ def bucket(flag_key: str, salt: str, bucket_by: str) -> float:
 def evaluate(
     payload: Payload,
     flag_key: str,
-    context: Context,
+    context: Optional[Context],
     default_value: Any,
 ) -> Any:
     """Evaluate one flag locally, falling back when no variation is valid."""
     flags = payload.get("flags")
     flag = flags.get(flag_key) if isinstance(flags, Mapping) else None
-    key = context.get("key")
-    if not isinstance(flag, Mapping) or not isinstance(key, str) or not key:
+    if not isinstance(flag, Mapping):
         return default_value
+    resolved: Mapping[str, Any] = {} if context is None else context
+    key = resolved.get("key")
 
     variation: Optional[int] = None
     if flag.get("on") is not True:
         variation = _index(flag.get("offVariation"))
     else:
         targets = flag.get("targets")
-        if isinstance(targets, list):
+        if isinstance(key, str) and key and isinstance(targets, list):
             for target in targets:
                 if not isinstance(target, Mapping):
                     continue
@@ -57,10 +58,10 @@ def evaluate(
                         continue
                     clauses = rule.get("clauses")
                     if isinstance(clauses, list) and _clauses_match(
-                        clauses, context, payload, set()
+                        clauses, resolved, payload, set()
                     ):
                         variation = _select_variation(
-                            rule, flag_key, context, payload
+                            rule, flag_key, resolved, payload
                         )
                         break
 
@@ -68,7 +69,7 @@ def evaluate(
             fallthrough = flag.get("fallthrough")
             if isinstance(fallthrough, Mapping):
                 variation = _select_variation(
-                    fallthrough, flag_key, context, payload
+                    fallthrough, flag_key, resolved, payload
                 )
 
     variations = flag.get("variations")
@@ -245,11 +246,12 @@ def _segment_matches(
 ) -> bool:
     key = context.get("key")
     excluded = segment.get("excluded")
-    if isinstance(excluded, list) and key in excluded:
-        return False
     included = segment.get("included")
-    if isinstance(included, list) and key in included:
-        return True
+    if isinstance(key, str) and key:
+        if isinstance(excluded, list) and key in excluded:
+            return False
+        if isinstance(included, list) and key in included:
+            return True
     rules = segment.get("rules")
     if not isinstance(rules, list):
         return False

{envflags-0.1.0 → envflags-0.2.0}/envflags.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: envflags
-Version: 0.1.0
+Version: 0.2.0
 Summary: Python SDK for envflags
 Author: envflags
 License: MIT
@@ -21,13 +21,16 @@ flags = create_client("envflags_server_...").initialize()
 try:
     enabled = flags.bool_variation(
         "new-checkout",
-        {"key": "user-123", "country": "BE"},
         False,
+        {"key": "user-123", "country": "BE"},
     )
 finally:
     flags.close()
 ```
 
+Context is optional. Omit it for flags that do not depend on who is asking; pass
+it last when targeting needs a user.
+
 The client polls every 30 seconds and fetches a payload only when its version
 changes. Set `poll_interval=None` to refresh manually. Supply a previously
 persisted payload through `bootstrap` to remain available when initialization

{envflags-0.1.0 → envflags-0.2.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "envflags"
-version = "0.1.0"
+version = "0.2.0"
 description = "Python SDK for envflags"
 readme = "README.md"
 requires-python = ">=3.9"

{envflags-0.1.0 → envflags-0.2.0}/tests/test_client.py

@@ -90,11 +90,12 @@ class ClientTest(unittest.TestCase):
         self.assertIs(client.initialize(), client)
         self.assertTrue(client.initialized)
         self.assertEqual(client.version, 3)
-        self.assertTrue(client.bool_variation("enabled", {"key": "user"}, False))
+        self.assertTrue(client.bool_variation("enabled", False, {"key": "user"}))
         self.assertEqual(
-            client.string_variation("greeting", {"key": "user"}, "fallback"),
+            client.string_variation("greeting", "fallback", {"key": "user"}),
             "hello",
         )
+        self.assertTrue(client.bool_variation("enabled", False))
         self.assertEqual(opener.requests[1].full_url, "https://edge.example.test/sdk/payloads/3")
 
         self.assertFalse(client.refresh())
@@ -121,7 +122,7 @@ class ClientTest(unittest.TestCase):
         )
         self.assertIs(client.initialize(), client)
         self.assertEqual(len(errors), 1)
-        self.assertTrue(client.bool_variation("enabled", {"key": "user"}, False))
+        self.assertTrue(client.bool_variation("enabled", False, {"key": "user"}))
 
     def test_rejects_offline_initialization_without_payload(self):
         client = Client(
@@ -132,17 +133,17 @@ class ClientTest(unittest.TestCase):
         with self.assertRaisesRegex(EnvflagsError, "offline"):
             client.initialize()
         self.assertFalse(
-            client.bool_variation("enabled", {"key": "user"}, False)
+            client.bool_variation("enabled", False, {"key": "user"})
         )
 
     def test_returns_typed_defaults(self):
         client = Client("client_test", bootstrap=payload(), poll_interval=None)
         context = {"key": "user"}
-        self.assertFalse(client.bool_variation("greeting", context, False))
+        self.assertFalse(client.bool_variation("greeting", False, context))
         self.assertEqual(
-            client.string_variation("missing", context, "fallback"), "fallback"
+            client.string_variation("missing", "fallback", context), "fallback"
         )
-        self.assertEqual(client.number_variation("enabled", context, 42), 42)
+        self.assertEqual(client.number_variation("enabled", 42, context), 42)
 
     def test_coalesces_concurrent_refreshes(self):
         started = threading.Event()