feat: add context_publisher setter and deprecate context_event_handler

Add context_publisher as the primary field in ABsmartlyConfig, replacing
context_event_handler. The old name is kept as a deprecated property that
delegates to context_publisher for backward compatibility. Export
ContextPublisher from the public SDK API.

Author: joalves

sdk/__init__.py

@@ -5,6 +5,7 @@
 from sdk.context import Context
 from sdk.context_config import ContextConfig
 from sdk.context_event_logger import ContextEventLogger
+from sdk.context_publisher import ContextPublisher
 from sdk.default_http_client import DefaultHTTPClient
 from sdk.default_http_client_config import DefaultHTTPClientConfig
 
@@ -18,6 +19,7 @@
     "Context",
     "ContextConfig",
     "ContextEventLogger",
+    "ContextPublisher",
     "DefaultHTTPClient",
     "DefaultHTTPClientConfig",
 ]

sdk/absmartly.py

@@ -1,3 +1,4 @@
+import warnings
 from concurrent.futures import Future
 from typing import Optional
 
@@ -66,21 +67,21 @@ def create(
 
     def __init__(self, config: ABsmartlyConfig):
         self.context_data_provider = config.context_data_provider
-        self.context_event_handler = config.context_event_handler
+        self.context_publisher = config.context_publisher
         self.context_event_logger = config.context_event_logger
         self.variable_parser = config.variable_parser
         self.audience_deserializer = config.audience_deserializer
 
         if self.context_data_provider is None or \
-                self.context_event_handler is None:
+                self.context_publisher is None:
             self.client = config.client
 
             if self.context_data_provider is None:
                 self.context_data_provider = \
                     DefaultContextDataProvider(self.client)
 
-            if self.context_event_handler is None:
-                self.context_event_handler = \
+            if self.context_publisher is None:
+                self.context_publisher = \
                     DefaultContextPublisher(self.client)
 
         if self.variable_parser is None:
@@ -89,6 +90,27 @@ def __init__(self, config: ABsmartlyConfig):
         if self.audience_deserializer is None:
             self.audience_deserializer = DefaultAudienceDeserializer()
 
+    def __getattr__(self, name):
+        if name == "context_event_handler":
+            warnings.warn(
+                "context_event_handler is deprecated, use context_publisher instead",
+                DeprecationWarning,
+                stacklevel=2,
+            )
+            return self.context_publisher
+        raise AttributeError(f"'{type(self).__name__}' object has no attribute '{name}'")
+
+    def __setattr__(self, name, value):
+        if name == "context_event_handler":
+            warnings.warn(
+                "context_event_handler is deprecated, use context_publisher instead",
+                DeprecationWarning,
+                stacklevel=2,
+            )
+            super().__setattr__("context_publisher", value)
+        else:
+            super().__setattr__(name, value)
+
     def get_context_data(self) -> Future[Optional[ContextData]]:
         return self.context_data_provider.get_context_data()
 
@@ -97,7 +119,7 @@ def create_context(self, config: ContextConfig) -> Context:
                        config,
                        self.context_data_provider.get_context_data(),
                        self.context_data_provider,
-                       self.context_event_handler,
+                       self.context_publisher,
                        self.context_event_logger,
                        self.variable_parser,
                        AudienceMatcher(self.audience_deserializer))
@@ -110,7 +132,7 @@ def create_context_with(self,
         return Context(SystemClockUTC(), config,
                        future_data,
                        self.context_data_provider,
-                       self.context_event_handler,
+                       self.context_publisher,
                        self.context_event_logger,
                        self.variable_parser,
                        AudienceMatcher(self.audience_deserializer))

sdk/absmartly_config.py

@@ -1,3 +1,4 @@
+import warnings
 from typing import Optional
 
 from sdk.audience_deserializer import AudienceDeserializer
@@ -10,16 +11,33 @@
 
 class ABsmartlyConfig:
     context_data_provider: Optional[ContextDataProvider] = None
-    context_event_handler: Optional[ContextPublisher] = None
+    context_publisher: Optional[ContextPublisher] = None
     context_event_logger: Optional[ContextEventLogger] = None
     audience_deserializer: Optional[AudienceDeserializer] = None
     client: Optional[Client] = None
     variable_parser: Optional[VariableParser] = None
 
+    @property
+    def context_event_handler(self) -> Optional[ContextPublisher]:
+        warnings.warn(
+            "context_event_handler is deprecated, use context_publisher instead",
+            DeprecationWarning,
+            stacklevel=2,
+        )
+        return self.context_publisher
+
+    @context_event_handler.setter
+    def context_event_handler(self, value: Optional[ContextPublisher]):
+        warnings.warn(
+            "context_event_handler is deprecated, use context_publisher instead",
+            DeprecationWarning,
+            stacklevel=2,
+        )
+        self.context_publisher = value
+
 
 class ABSmartlyConfig(ABsmartlyConfig):
     def __init__(self, *args, **kwargs):
-        import warnings
         warnings.warn(
             "ABSmartlyConfig is deprecated, use ABsmartlyConfig instead",
             DeprecationWarning,

test/test_named_param_init.py

@@ -19,7 +19,7 @@ def test_create_with_all_required_params(self):
 
         self.assertIsNotNone(sdk)
         self.assertIsNotNone(sdk.context_data_provider)
-        self.assertIsNotNone(sdk.context_event_handler)
+        self.assertIsNotNone(sdk.context_publisher)
         self.assertIsNotNone(sdk.variable_parser)
         self.assertIsNotNone(sdk.audience_deserializer)