feat: nested and more flexible fractional enhancements

- support nested properties in fractional targeting rules
- support non-string variant keys in fractional operator
- bump grpcio and grpcio-health-checking to >=1.80.0
- update test harness and add comprehensive fractional tests
- adjust test assertions for invalid-fractional-weights behavior change

Author: leakonvalinka

providers/openfeature-provider-flagd/openfeature/test-harness

@@ -18,7 +18,7 @@ classifiers = [
 keywords = []
 dependencies = [
   "openfeature-sdk>=0.8.2",
-  "grpcio>=1.78.0",
+  "grpcio>=1.80.0",
   "protobuf>=6.30.0,<7.0.0",
   "mmh3>=5.0.0,<6.0.0",
   "panzi-json-logic>=1.0.1",
@@ -35,7 +35,7 @@ Homepage = "https://github.com/open-feature/python-sdk-contrib"
 dev = [
   "asserts>=0.13.0,<0.14.0",
   "coverage[toml]>=7.10.0,<8.0.0",
-  "grpcio-health-checking>=1.74.0,<2.0.0",
+  "grpcio-health-checking>=1.80.0,<2.0.0",
   "mypy>=1.18.0,<2.0.0",
   "poethepoet>=0.37.0",
   "pytest>=9.0.0,<10.0.0",

providers/openfeature-provider-flagd/pyproject.toml

@@ -6,6 +6,8 @@
 import mmh3
 import semver
 
+MAX_WEIGHT_SUM = 2_147_483_647  # MaxInt32
+
 JsonPrimitive: typing.TypeAlias = str | bool | float | int
 JsonLogicArg: typing.TypeAlias = JsonPrimitive | Sequence[JsonPrimitive]
 
@@ -14,50 +16,56 @@
 
 @dataclass
 class Fraction:
-    variant: str
+    variant: str | float | int | bool | None
     weight: int = 1
 
 
-def fractional(data: dict, *args: JsonLogicArg) -> str | None:
+def _resolve_bucket_by(data: dict, args: tuple) -> tuple[str | None, tuple]:
+    if isinstance(args[0], str):
+        return args[0], args[1:]
+
+    seed = data.get("$flagd", {}).get("flagKey", "")
+    targeting_key = data.get("targetingKey")
+    if not targeting_key:
+        logger.error("No targetingKey provided for fractional shorthand syntax.")
+        return None, args
+    return seed + targeting_key, args
+
+
+def fractional(data: dict, *args: JsonLogicArg) -> str | float | int | bool | None:
     if not args:
         logger.error("No arguments provided to fractional operator.")
         return None
 
-    bucket_by = None
-    if isinstance(args[0], str):
-        bucket_by = args[0]
-        args = args[1:]
-    else:
-        seed = data.get("$flagd", {}).get("flagKey", "")
-        targeting_key = data.get("targetingKey")
-        if not targeting_key:
-            logger.error("No targetingKey provided for fractional shorthand syntax.")
-            return None
-        bucket_by = seed + targeting_key
+    bucket_by, args = _resolve_bucket_by(data, args)
 
     if not bucket_by:
         logger.error("No hashKey value resolved")
         return None
 
-    hash_ratio = abs(mmh3.hash(bucket_by)) / (2**31 - 1)
-    bucket = hash_ratio * 100
+    hash_value = mmh3.hash(bucket_by, signed=False)
 
     total_weight = 0
     fractions = []
     try:
         for arg in args:
             fraction = _parse_fraction(arg)
-            if fraction:
-                fractions.append(fraction)
-                total_weight += fraction.weight
+            fractions.append(fraction)
+            total_weight += fraction.weight
 
     except ValueError:
         logger.debug(f"Invalid {args} configuration")
         return None
 
-    range_end: float = 0
+    if total_weight > MAX_WEIGHT_SUM:
+        logger.error(f"Total fractional weight exceeds MaxInt32 ({MAX_WEIGHT_SUM:,}).")
+        return None
+
+    bucket = (hash_value * total_weight) >> 32
+
+    range_end = 0
     for fraction in fractions:
-        range_end += fraction.weight * 100 / total_weight
+        range_end += fraction.weight
         if bucket < range_end:
             return fraction.variant
     return None
@@ -69,19 +77,21 @@ def _parse_fraction(arg: JsonLogicArg) -> Fraction:
             "Fractional variant weights must be (str, int) tuple or [str] list"
         )
 
-    if not isinstance(arg[0], str):
-        raise ValueError(
-            "Fractional variant identifier (first element) isn't of type 'str'"
-        )
-
-    if len(arg) >= 2 and not isinstance(arg[1], int):
-        raise ValueError(
-            "Fractional variant weight value (second element) isn't of type 'int'"
-        )
-
-    fraction = Fraction(variant=arg[0])
-    if len(arg) >= 2:
-        fraction.weight = arg[1]
+    variant = arg[0]
+
+    weight = None
+    if len(arg) == 2:
+        w = arg[1]
+        if isinstance(w, bool):
+            raise ValueError("Fractional weight value isn't of type 'int'")
+        elif isinstance(w, int):
+            weight = w
+        else:
+            raise ValueError("Fractional weight value isn't of type 'int'")
+
+    fraction = Fraction(variant=variant)
+    if weight is not None:
+        fraction.weight = weight
 
     return fraction
 

providers/openfeature-provider-flagd/src/openfeature/contrib/provider/flagd/resolvers/process/custom_ops.py

@@ -14,6 +14,7 @@
     "~grace",
     "~contextEnrichment",
     "~deprecated",
+    "~fractional-v1",
 }
 
 

providers/openfeature-provider-flagd/tests/e2e/file/conftest.py

@@ -4,7 +4,7 @@
 from tests.e2e.testfilter import TestFilter
 
 resolver = ResolverType.IN_PROCESS
-feature_list = ["~targetURI", "~unixsocket", "~deprecated"]
+feature_list = ["~targetURI", "~unixsocket", "~deprecated", "~fractional-v1"]
 
 
 def pytest_collection_modifyitems(config, items):

providers/openfeature-provider-flagd/tests/e2e/inprocess/conftest.py

@@ -10,6 +10,9 @@
     "~sync",
     "~metadata",
     "~deprecated",
+    "~fractional-v1",
+    "~fractional-v2",
+    "~fractional-nested",
 ]
 
 

providers/openfeature-provider-flagd/tests/e2e/rpc/conftest.py

@@ -84,7 +84,6 @@ def test_broken_targeting():
         "invalid-stringcomp-args.json",
         "invalid-fractional-args.json",
         "invalid-fractional-args-wrong-content.json",
-        "invalid-fractional-weights.json",
         "invalid-fractional-weights-strings.json",
     ],
 )