improve error handling

Signed-off-by: christian.lutnik <[email protected]>

Author: chrfwow

providers/openfeature-provider-env-var/.python-version

@@ -1 +1 @@
-3.12
+3.9

providers/openfeature-provider-env-var/pyproject.toml

@@ -3,7 +3,7 @@ name = "openfeature-provider-env-var"
 version = "0.1.0"
 description = "OpenFeature Python Environment Variable Provider"
 readme = "README.md"
-requires-python = ">=3.12"
+requires-python = ">=3.9"
 dependencies = [
     "openfeature-sdk>=0.6.0",
 ]
@@ -13,9 +13,6 @@ authors = [{ name = "OpenFeature", email = "[email protected]" }]
 requires = ["hatchling"]
 build-backend = "hatchling.build"
 
-[tool.ruff.lint]
-extend-select = ["B"]
-
 [dependency-groups]
 dev = [
     "pytest>=8.3.5",

providers/openfeature-provider-env-var/src/openfeature/contrib/provider/envvar/provider.py

@@ -1,9 +1,9 @@
 import json
-import typing
 import os
+import typing
 
 from openfeature.evaluation_context import EvaluationContext
-from openfeature.exception import FlagNotFoundError
+from openfeature.exception import FlagNotFoundError, ParseError
 from openfeature.flag_evaluation import FlagResolutionDetails, Reason
 from openfeature.provider import AbstractProvider, Metadata
 
@@ -14,7 +14,15 @@ def _load_and_parse_env_var(
     value = os.environ.get(flag_key)
     if value is None:
         raise FlagNotFoundError()
-    return FlagResolutionDetails(parse(value), None, None, Reason.DEFAULT, None, {})
+
+    try:
+        parsed = parse(value)
+    except ValueError as error:
+        raise ParseError(
+            f'Error decoding object for flag with key "${flag_key}"'
+        ) from error
+
+    return FlagResolutionDetails(parsed, None, None, Reason.DEFAULT, None, {})
 
 
 class EnvVarProvider(AbstractProvider):
@@ -61,10 +69,11 @@ def resolve_object_details(
     ) -> FlagResolutionDetails[typing.Union[dict, list]]:
         def parse(value: str) -> typing.Union[dict, list]:
             result = json.loads(value)
-            if isinstance(result, dict) or isinstance(result, list):
+
+            if isinstance(result, (dict, list)):
                 return result
             else:
-                raise TypeError(
+                raise ParseError(
                     f'Value for feature flag with key "${flag_key}" does not resolve to a JSON object or list'
                 )