eppo-server-sdk 3.1.0 → 3.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 13377928dbb61c86f0f55c300cce4fd1bbd9298014c34d6e766e6807eec306bb
-  data.tar.gz: cbaefafb70a5e1267092d7d01a816ffc2520e8cc5b9e21fb22ccd30fc26f6ea0
+  metadata.gz: d6fb69972c523dd9d2c7c683c751c012608c63bb36fd3a6fe9463fb27d77d3d6
+  data.tar.gz: dfb6b27af63f652094f2141791aaf31be01ce8277801ce6537b30f0cbfa64820
 SHA512:
-  metadata.gz: d9edcdd5df7dd9abc465abc275ed46b3e16edc541a5f7eaf5108cc2d21e42a293fbee4d93ad7b0e5db9625397ceb1df0cbdda4558686ee5313cb7c2812b58ca8
-  data.tar.gz: add2316372855812a3b54c024b171c85a0a87124ff8645ee4bf352897c46b26b2b47627d96726c9b93d4fd8407b3488cf6845de7fe9edae55e0efaf423a44d6f
+  metadata.gz: 1d191dfefc76fb61a1b873c61da265066e1d674e628864d326d3d51bb3a5c14933fb69aa48e72a1325bd2753a4acc2083bde48d1daf1a408c29b90cea4f858b5
+  data.tar.gz: b79006cc38615b15e44d12362fcc61520ae34f852272fe5a42d4761d7e36e5c407aa1a63df0226109c2198a4798fbb0ded3ce3c0b22fa27e004a69165a2a1696

data/Cargo.lock

@@ -304,7 +304,7 @@ dependencies = [
 
 [[package]]
 name = "eppo_client"
-version = "3.1.0"
+version = "3.1.1"
 dependencies = [
  "env_logger",
  "eppo_core",
@@ -317,9 +317,9 @@ dependencies = [
 
 [[package]]
 name = "eppo_core"
-version = "2.0.0"
+version = "3.0.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "aff693ab3cd82f61d90249aa02c6f7e173202d92727d1036b8fc1468201f9848"
+checksum = "34f3fc5a7f54cc47a5ebf063025176726db7eb5e51661185b5f4d20aaacea611"
 dependencies = [
  "chrono",
  "derive_more",

data/ext/eppo_client/Cargo.toml

@@ -1,6 +1,7 @@
 [package]
 name = "eppo_client"
-version = "3.1.0"
+# TODO: this version and lib/eppo_client/version.rb should be in sync
+version = "3.1.1"
 edition = "2021"
 license = "MIT"
 publish = false
@@ -11,7 +12,7 @@ crate-type = ["cdylib"]
 
 [dependencies]
 env_logger = { version = "0.11.3", features = ["unstable-kv"] }
-eppo_core = { version = "2.0.0" }
+eppo_core = { version = "3.0.0" }
 log = { version = "0.4.21", features = ["kv_serde"] }
 magnus = { version = "0.6.2" }
 serde = { version = "1.0.203", features = ["derive"] }

data/ext/eppo_client/src/client.rs

@@ -1,12 +1,12 @@
 use std::{cell::RefCell, sync::Arc};
 
 use eppo_core::{
-    configuration_fetcher::ConfigurationFetcher,
+    configuration_fetcher::{ConfigurationFetcher, ConfigurationFetcherConfig},
     configuration_store::ConfigurationStore,
-    eval::{get_assignment, get_assignment_details, get_bandit_action, get_bandit_action_details},
+    eval::{Evaluator, EvaluatorConfig},
     poller_thread::PollerThread,
     ufc::VariationType,
-    Attributes, ContextAttributes,
+    Attributes, ContextAttributes, SdkMetadata,
 };
 use magnus::{error::Result, exception, prelude::*, Error, TryConvert, Value};
 
@@ -28,7 +28,7 @@ impl TryConvert for Config {
 
 #[magnus::wrap(class = "EppoClient::Core::Client")]
 pub struct Client {
-    configuration_store: Arc<ConfigurationStore>,
+    evaluator: Evaluator,
     // Magnus only allows sharing aliased references (&T) through the API, so we need to use RefCell
     // to get interior mutability.
     //
@@ -41,21 +41,28 @@ impl Client {
     pub fn new(config: Config) -> Client {
         let configuration_store = Arc::new(ConfigurationStore::new());
 
+        let sdk_metadata = SdkMetadata {
+            name: "ruby",
+            version: env!("CARGO_PKG_VERSION"),
+        };
+
         let poller_thread = PollerThread::start(
-            ConfigurationFetcher::new(
-                eppo_core::configuration_fetcher::ConfigurationFetcherConfig {
-                    base_url: config.base_url,
-                    api_key: config.api_key,
-                    sdk_name: "ruby".to_owned(),
-                    sdk_version: env!("CARGO_PKG_VERSION").to_owned(),
-                },
-            ),
+            ConfigurationFetcher::new(ConfigurationFetcherConfig {
+                base_url: config.base_url,
+                api_key: config.api_key,
+                sdk_metadata: sdk_metadata.clone(),
+            }),
             configuration_store.clone(),
         )
         .expect("should be able to start poller thread");
 
-        Client {
+        let evaluator = Evaluator::new(EvaluatorConfig {
             configuration_store,
+            sdk_metadata,
+        });
+
+        Client {
+            evaluator,
             poller_thread: RefCell::new(Some(poller_thread)),
         }
     }
@@ -70,16 +77,16 @@ impl Client {
         let expected_type: VariationType = serde_magnus::deserialize(expected_type)?;
         let subject_attributes: Attributes = serde_magnus::deserialize(subject_attributes)?;
 
-        let config = self.configuration_store.get_configuration();
-        let result = get_assignment(
-            config.as_ref().map(AsRef::as_ref),
-            &flag_key,
-            &subject_key,
-            &subject_attributes,
-            Some(expected_type),
-        )
-        // TODO: maybe expose possible errors individually.
-        .map_err(|err| Error::new(exception::runtime_error(), err.to_string()))?;
+        let result = self
+            .evaluator
+            .get_assignment(
+                &flag_key,
+                &subject_key,
+                &subject_attributes,
+                Some(expected_type),
+            )
+            // TODO: maybe expose possible errors individually.
+            .map_err(|err| Error::new(exception::runtime_error(), err.to_string()))?;
 
         Ok(serde_magnus::serialize(&result).expect("assignment value should be serializable"))
     }
@@ -94,9 +101,7 @@ impl Client {
         let expected_type: VariationType = serde_magnus::deserialize(expected_type)?;
         let subject_attributes: Attributes = serde_magnus::deserialize(subject_attributes)?;
 
-        let config = self.configuration_store.get_configuration();
-        let result = get_assignment_details(
-            config.as_ref().map(AsRef::as_ref),
+        let result = self.evaluator.get_assignment_details(
             &flag_key,
             &subject_key,
             &subject_attributes,
@@ -125,9 +130,7 @@ impl Client {
         })?;
         let actions = serde_magnus::deserialize(actions)?;
 
-        let config = self.configuration_store.get_configuration();
-        let result = get_bandit_action(
-            config.as_ref().map(AsRef::as_ref),
+        let result = self.evaluator.get_bandit_action(
             &flag_key,
             &subject_key,
             &subject_attributes,
@@ -157,9 +160,7 @@ impl Client {
         })?;
         let actions = serde_magnus::deserialize(actions)?;
 
-        let config = self.configuration_store.get_configuration();
-        let result = get_bandit_action_details(
-            config.as_ref().map(AsRef::as_ref),
+        let result = self.evaluator.get_bandit_action_details(
             &flag_key,
             &subject_key,
             &subject_attributes,

data/lib/eppo_client/client.rb

@@ -144,7 +144,6 @@ module EppoClient
       # events for both flag assignment and bandit actions.
       event = event.to_h { |key, value| [key.to_sym, value]}
 
-      enrich_event_metadata(event)
       begin
         @assignment_logger.log_assignment(event)
       rescue EppoClient::AssignmentLoggerError
@@ -158,7 +157,6 @@ module EppoClient
     def log_bandit_action(event)
       if not event then return end
 
-      enrich_event_metadata(event)
       begin
         @assignment_logger.log_bandit_action(event)
       rescue EppoClient::AssignmentLoggerError
@@ -169,11 +167,6 @@ module EppoClient
       end
     end
 
-    def enrich_event_metadata(event)
-      event[:metaData]["sdkName"] = "ruby"
-      event[:metaData]["sdkVersion"] = EppoClient::VERSION
-    end
-
     def coerce_context_attributes(attributes)
       numeric_attributes = attributes[:numeric_attributes] || attributes["numericAttributes"]
       categorical_attributes = attributes[:categorical_attributes] || attributes["categoricalAttributes"]

data/lib/eppo_client/version.rb

@@ -2,5 +2,5 @@
 
 # TODO: this version and ext/eppo_rb/Cargo.toml should be in sync
 module EppoClient
-  VERSION = "3.1.0"
+  VERSION = "3.1.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: eppo-server-sdk
 version: !ruby/object:Gem::Version
-  version: 3.1.0
+  version: 3.1.1
 platform: ruby
 authors:
 - Eppo
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-08-16 00:00:00.000000000 Z
+date: 2024-09-19 00:00:00.000000000 Z
 dependencies: []
 description: 
 email:
@@ -64,7 +64,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 3.3.11
 requirements: []
-rubygems_version: 3.5.11
+rubygems_version: 3.5.16
 signing_key: 
 specification_version: 4
 summary: Eppo SDK for Ruby