chore: linter fixes

Author: jsalaber

devcycle_python_sdk/api/local_bucketing.py

@@ -359,7 +359,7 @@ def store_config(self, config_json: str) -> None:
             config_addr = self._new_assembly_script_byte_array(data)
             self.setConfigDataUTF8(self.wasm_store, self.sdk_key_addr, config_addr)
 
-    def get_config_metadata(self) -> dict:
+    def get_config_metadata(self) -> Optional[ConfigMetadata]:
         with self.wasm_lock:
             config_addr = self.getConfigMetadata(self.wasm_store, self.sdk_key_addr)
             config_bytes = self._read_assembly_script_string(config_addr)

devcycle_python_sdk/managers/config_manager.py

@@ -16,6 +16,7 @@
 from wsgiref.handlers import format_date_time
 from devcycle_python_sdk.options import DevCycleLocalOptions
 from devcycle_python_sdk.managers.sse_manager import SSEManager
+from devcycle_python_sdk.models.config_metadata import ConfigMetadata
 
 logger = logging.getLogger(__name__)
 
@@ -108,7 +109,7 @@ def _get_config(self, last_modified: Optional[float] = None):
             )
             self._polling_enabled = False
 
-    def get_config_metadata(self) -> dict:
+    def get_config_metadata(self) -> Optional[ConfigMetadata]:
         return self._local_bucketing.get_config_metadata()
 
     def run(self):

devcycle_python_sdk/models/config_metadata.py

@@ -1,5 +1,6 @@
 from devcycle_python_sdk.models.environment_metadata import EnvironmentMetadata
 from devcycle_python_sdk.models.project_metadata import ProjectMetadata
+from typing import Dict, Any, Optional
 import json
 
 
@@ -16,10 +17,16 @@ def to_json(self) -> str:
         return json.dumps(self, default=lambda o: o.__dict__)
 
     @staticmethod
-    def from_json(json_str: str) -> "ConfigMetadata":
-        if json_str is None:
+    def from_json(json_obj: Optional[Dict[str, Any]]) -> Optional["ConfigMetadata"]:
+        if json_obj is None:
             return None
+        project = ProjectMetadata.from_json(json_obj.get("project"))
+        environment = EnvironmentMetadata.from_json(json_obj.get("environment"))
+
+        if project is None or environment is None:
+            return None
+
         return ConfigMetadata(
-            project=ProjectMetadata.from_json(json_str["project"]),
-            environment=EnvironmentMetadata.from_json(json_str["environment"]),
-        )
+            project=project,
+            environment=environment,
+        )

devcycle_python_sdk/models/environment_metadata.py

@@ -1,3 +1,6 @@
+from typing import Dict, Any, Optional
+
+
 class EnvironmentMetadata:
     def __init__(
         self,
@@ -8,10 +11,12 @@ def __init__(
         self.key = key
 
     @staticmethod
-    def from_json(json_str: str) -> "EnvironmentMetadata":
-        if json_str is None:
+    def from_json(
+        json_obj: Optional[Dict[str, Any]],
+    ) -> Optional["EnvironmentMetadata"]:
+        if json_obj is None:
             return None
         return EnvironmentMetadata(
-            id=json_str["id"],
-            key=json_str["key"],
-        )
+            id=json_obj["id"],
+            key=json_obj["key"],
+        )

devcycle_python_sdk/models/eval_hook_context.py

@@ -1,10 +1,17 @@
-from typing import Any
+from typing import Any, Optional
 
 from devcycle_python_sdk.models.user import DevCycleUser
 from devcycle_python_sdk.models.config_metadata import ConfigMetadata
 
+
 class HookContext:
-    def __init__(self, key: str, user: DevCycleUser, default_value: Any, config_metadata: ConfigMetadata = None):
+    def __init__(
+        self,
+        key: str,
+        user: DevCycleUser,
+        default_value: Any,
+        config_metadata: Optional[ConfigMetadata] = None,
+    ):
         self.key = key
         self.default_value = default_value
         self.user = user

devcycle_python_sdk/models/project_metadata.py

@@ -1,3 +1,6 @@
+from typing import Dict, Any, Optional
+
+
 class ProjectMetadata:
     def __init__(
         self,
@@ -8,10 +11,10 @@ def __init__(
         self.key = key
 
     @staticmethod
-    def from_json(json_str: str) -> "ProjectMetadata":
-        if json_str is None:
+    def from_json(json_obj: Optional[Dict[str, Any]]) -> Optional["ProjectMetadata"]:
+        if json_obj is None:
             return None
         return ProjectMetadata(
-            id=json_str["id"],
-            key=json_str["key"],
-        )
+            id=json_obj["id"],
+            key=json_obj["key"],
+        )

devcycle_python_sdk/options.py

@@ -90,7 +90,7 @@ def __init__(
 
         if self.event_request_chunk_size > self.max_event_queue_size:
             logger.warning(
-                f"DevCycle: event_request_chunk_size: {self.event_request_chunk_size} must be smaller than max_event_queue_size: { self.max_event_queue_size}"
+                f"DevCycle: event_request_chunk_size: {self.event_request_chunk_size} must be smaller than max_event_queue_size: {self.max_event_queue_size}"
             )
             self.event_request_chunk_size = 100
 

devcycle_python_sdk/protobuf/utils.py

@@ -34,11 +34,17 @@ def create_nullable_custom_data(val: Optional[dict]) -> pb2.NullableCustomData:
             if value is None:
                 values[key] = pb2.CustomDataValue(type=pb2.CustomDataType.Null)  # type: ignore
             elif isinstance(value, bool):
-                values[key] = pb2.CustomDataValue(type=pb2.CustomDataType.Bool, boolValue=value)  # type: ignore
+                values[key] = pb2.CustomDataValue(
+                    type=pb2.CustomDataType.Bool, boolValue=value
+                )  # type: ignore
             elif isinstance(value, str):
-                values[key] = pb2.CustomDataValue(type=pb2.CustomDataType.Str, stringValue=value)  # type: ignore
+                values[key] = pb2.CustomDataValue(
+                    type=pb2.CustomDataType.Str, stringValue=value
+                )  # type: ignore
             elif isinstance(value, (int, float)):
-                values[key] = pb2.CustomDataValue(type=pb2.CustomDataType.Num, doubleValue=value)  # type: ignore
+                values[key] = pb2.CustomDataValue(
+                    type=pb2.CustomDataType.Num, doubleValue=value
+                )  # type: ignore
             else:
                 logger.warning(
                     f"Custom Data contains data type that can't be written, will be ignored. Key: {key}, Type: {str(type(value))}"

devcycle_python_sdk/protobuf/variableForUserParams_pb2.py

@@ -1,5 +1,6 @@
 #!/usr/bin/env python
 """Django's command-line utility for administrative tasks."""
+
 import os
 import sys