oai-statsig-python-core 0.23.1__tar.gz → 0.24.0__tar.gz

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/Cargo.lock

@@ -1722,7 +1722,7 @@ dependencies = [
 
 [[package]]
 name = "oai-statsig-grpc"
-version = "0.23.1"
+version = "0.24.0"
 dependencies = [
  "async-trait",
  "chrono",
@@ -1737,7 +1737,7 @@ dependencies = [
 
 [[package]]
 name = "oai-statsig-rust"
-version = "0.23.1"
+version = "0.24.0"
 dependencies = [
  "ahash",
  "arc-swap",
@@ -3011,7 +3011,7 @@ checksum = "a2eb9349b6444b326872e140eb1cf5e7c522154d69e7a0ffb0fb81c06b37543f"
 
 [[package]]
 name = "statsig-node"
-version = "0.23.1"
+version = "0.24.0"
 dependencies = [
  "async-trait",
  "napi",
@@ -3026,7 +3026,7 @@ dependencies = [
 
 [[package]]
 name = "statsig-pyo3"
-version = "0.23.1"
+version = "0.24.0"
 dependencies = [
  "async-trait",
  "lazy_static",
@@ -3041,7 +3041,7 @@ dependencies = [
 
 [[package]]
 name = "statsig_elixir"
-version = "0.23.1"
+version = "0.24.0"
 dependencies = [
  "async-trait",
  "oai-statsig-rust",
@@ -3054,7 +3054,7 @@ dependencies = [
 
 [[package]]
 name = "statsig_ffi"
-version = "0.23.1"
+version = "0.24.0"
 dependencies = [
  "async-trait",
  "cbindgen",

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/Cargo.toml

@@ -8,6 +8,7 @@ strip = true
 members = ["statsig-rust", "statsig-grpc", "statsig-pyo3"]
 resolver = "2"
 exclude = [
+    "photon_rust",
     "examples/rust/sample-usage",
     "examples/rust/perf-monitor",
     "examples/rust/mem-bench",
@@ -19,7 +20,7 @@ exclude = [
 [workspace.package]
 edition = "2021"
 license = "ISC"
-version = "0.23.1"
+version = "0.24.0"
 homepage = "https://statsig.com/"
 authors = ["Statsig", "Daniel Loomb <daniel@statsig.com>"]
 readme = "README.md"

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: oai_statsig_python_core
-Version: 0.23.1
+Version: 0.24.0
 Classifier: Programming Language :: Rust
 Classifier: Programming Language :: Python :: Implementation :: CPython
 Classifier: Programming Language :: Python :: Implementation :: PyPy

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/py_src/statsig_python_core/statsig_python_core.pyi

@@ -213,6 +213,7 @@ class StatsigBasePy:
     def initialize_with_details(self) -> typing.Any: ...
     def get_initialize_details(self) -> InitializeDetails: ...
     def is_initialized(self) -> builtins.bool: ...
+    def is_config_spec_ready(self) -> builtins.bool: ...
     def flush_events(self) -> typing.Any: ...
     def shutdown(self) -> typing.Any: ...
     def _INTERNAL_subscribe(self, event_name: builtins.str, callback: typing.Any) -> builtins.str: ...
@@ -345,6 +346,10 @@ class StatsigOptions:
     @service_name.setter
     def service_name(self, value: typing.Optional[builtins.str]) -> None: ...
     @property
+    def sdk_instance_id(self) -> typing.Optional[builtins.str]: ...
+    @sdk_instance_id.setter
+    def sdk_instance_id(self, value: typing.Optional[builtins.str]) -> None: ...
+    @property
     def output_log_level(self) -> typing.Optional[builtins.str]: ...
     @output_log_level.setter
     def output_log_level(self, value: typing.Optional[builtins.str]) -> None: ...
@@ -400,7 +405,7 @@ class StatsigOptions:
     def experimental_flags(self) -> typing.Optional[builtins.set[builtins.str]]: ...
     @experimental_flags.setter
     def experimental_flags(self, value: typing.Optional[builtins.set[builtins.str]]) -> None: ...
-    def __new__(cls, specs_url: typing.Optional[builtins.str] = None, specs_sync_interval_ms: typing.Optional[builtins.int] = None, init_timeout_ms: typing.Optional[builtins.int] = None, log_event_url: typing.Optional[builtins.str] = None, disable_all_logging: typing.Optional[builtins.bool] = None, disable_network: typing.Optional[builtins.bool] = None, log_event_connection_reuse: typing.Optional[builtins.bool] = None, event_logging_flush_interval_ms: typing.Optional[builtins.int] = None, event_logging_max_queue_size: typing.Optional[builtins.int] = None, event_logging_max_pending_batch_queue_size: typing.Optional[builtins.int] = None, enable_id_lists: typing.Optional[builtins.bool] = None, enable_dcs_deltas: typing.Optional[builtins.bool] = None, wait_for_user_agent_init: typing.Optional[builtins.bool] = None, wait_for_country_lookup_init: typing.Optional[builtins.bool] = None, disable_country_lookup: typing.Optional[builtins.bool] = None, id_lists_url: typing.Optional[builtins.str] = None, download_id_list_file_api: typing.Optional[builtins.str] = None, id_lists_sync_interval_ms: typing.Optional[builtins.int] = None, fallback_to_statsig_api: typing.Optional[builtins.bool] = None, environment: typing.Optional[builtins.str] = None, service_name: typing.Optional[builtins.str] = None, output_log_level: typing.Optional[builtins.str] = None, global_custom_fields: typing.Optional[typing.Mapping[builtins.str, typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]] = None, observability_client: typing.Optional[ObservabilityClientBase] = None, data_store: typing.Optional[DataStoreBase] = None, data_store_key_schema_version: typing.Optional[builtins.str] = None, persistent_storage: typing.Optional[PersistentStorageBaseClass] = None, config_compression_mode: typing.Optional[builtins.str] = None, proxy_config: typing.Optional[ProxyConfig] = None, output_logger_provider: typing.Optional[OutputLoggerProviderBase] = None, spec_adapter_configs: typing.Optional[list] = None, use_third_party_ua_parser: typing.Optional[builtins.bool] = None, disable_disk_access: typing.Optional[builtins.bool] = None, experimental_flags: typing.Optional[builtins.set[builtins.str]] = None, event_logging_adapter: typing.Optional[EventLoggingAdapterBase] = None) -> StatsigOptions: ...
+    def __new__(cls, specs_url: typing.Optional[builtins.str] = None, specs_sync_interval_ms: typing.Optional[builtins.int] = None, init_timeout_ms: typing.Optional[builtins.int] = None, log_event_url: typing.Optional[builtins.str] = None, disable_all_logging: typing.Optional[builtins.bool] = None, disable_network: typing.Optional[builtins.bool] = None, log_event_connection_reuse: typing.Optional[builtins.bool] = None, event_logging_flush_interval_ms: typing.Optional[builtins.int] = None, event_logging_max_queue_size: typing.Optional[builtins.int] = None, event_logging_max_pending_batch_queue_size: typing.Optional[builtins.int] = None, enable_id_lists: typing.Optional[builtins.bool] = None, enable_dcs_deltas: typing.Optional[builtins.bool] = None, wait_for_user_agent_init: typing.Optional[builtins.bool] = None, wait_for_country_lookup_init: typing.Optional[builtins.bool] = None, disable_country_lookup: typing.Optional[builtins.bool] = None, id_lists_url: typing.Optional[builtins.str] = None, download_id_list_file_api: typing.Optional[builtins.str] = None, id_lists_sync_interval_ms: typing.Optional[builtins.int] = None, fallback_to_statsig_api: typing.Optional[builtins.bool] = None, environment: typing.Optional[builtins.str] = None, service_name: typing.Optional[builtins.str] = None, sdk_instance_id: typing.Optional[builtins.str] = None, output_log_level: typing.Optional[builtins.str] = None, global_custom_fields: typing.Optional[typing.Mapping[builtins.str, typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool, typing.List[typing.Optional[typing.Union[builtins.str, builtins.int, builtins.float, builtins.bool]]]]]] = None, observability_client: typing.Optional[ObservabilityClientBase] = None, data_store: typing.Optional[DataStoreBase] = None, data_store_key_schema_version: typing.Optional[builtins.str] = None, persistent_storage: typing.Optional[PersistentStorageBaseClass] = None, config_compression_mode: typing.Optional[builtins.str] = None, proxy_config: typing.Optional[ProxyConfig] = None, output_logger_provider: typing.Optional[OutputLoggerProviderBase] = None, spec_adapter_configs: typing.Optional[list] = None, use_third_party_ua_parser: typing.Optional[builtins.bool] = None, disable_disk_access: typing.Optional[builtins.bool] = None, experimental_flags: typing.Optional[builtins.set[builtins.str]] = None, event_logging_adapter: typing.Optional[EventLoggingAdapterBase] = None) -> StatsigOptions: ...
 
 @typing.final
 class StatsigUser:

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-pyo3/src/statsig_base_py.rs

@@ -26,6 +26,7 @@ use pyo3::{
 use pyo3_stub_gen::derive::*;
 use serde_json::Value;
 use statsig_rust::interned_string::InternedString;
+use statsig_rust::user::fast_statsig_user::FastStatsigUser;
 use statsig_rust::user::StatsigUserInternal;
 use statsig_rust::{
     log_e, sdk_event_emitter::SubscriptionID, BulkEvaluationOptions, ClientInitResponseOptions,
@@ -139,6 +140,10 @@ impl StatsigBasePy {
         self.inner.is_initialized()
     }
 
+    pub fn is_config_spec_ready(&self) -> bool {
+        self.inner.is_config_spec_ready()
+    }
+
     pub fn flush_events(&self, py: Python) -> PyResult<Py<PyAny>> {
         let (completion_event, event_clone) = get_completion_event(py)?;
 
@@ -315,16 +320,17 @@ impl StatsigBasePy {
     #[pyo3(signature = (user, name, options=None))]
     pub fn check_gate(
         &self,
+        py: Python<'_>,
         user: &StatsigUserPy,
         name: &str,
         options: Option<FeatureGateEvaluationOptionsPy>,
     ) -> bool {
-        let user_internal = StatsigUserInternal::from_fast_user(&user.inner, Some(&self.inner));
-        self.inner.check_gate_with_options_for_internal_user(
-            &user_internal,
-            name,
-            options.map_or(FeatureGateEvaluationOptions::default(), |o| o.into()),
-        )
+        let statsig = self.inner.clone();
+        let user = user.inner.clone();
+        let name = name.to_string();
+        let options = options.map_or(FeatureGateEvaluationOptions::default(), |o| o.into());
+
+        py.detach(move || check_gate_for_python(statsig.as_ref(), &user, &name, options))
     }
 
     #[pyo3(name="_INTERNAL_get_feature_gate", signature = (user, name, options=None))]
@@ -784,6 +790,16 @@ fn extract_event_metadata(metadata: Option<Bound<PyDict>>) -> Option<HashMap<Str
     metadata.map(|m| py_dict_to_json_value_map(&m))
 }
 
+fn check_gate_for_python(
+    statsig: &Statsig,
+    user: &FastStatsigUser,
+    name: &str,
+    options: FeatureGateEvaluationOptions,
+) -> bool {
+    let user_internal = StatsigUserInternal::from_fast_user(user, Some(statsig));
+    statsig.check_gate_with_options_for_internal_user(&user_internal, name, options)
+}
+
 fn bulk_evaluate_to_py_dict(
     py: Python,
     statsig: &Statsig,

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-pyo3/src/statsig_options_py.rs

@@ -156,6 +156,8 @@ pub struct StatsigOptionsPy {
     #[pyo3(get, set)]
     pub service_name: Option<String>,
     #[pyo3(get, set)]
+    pub sdk_instance_id: Option<String>,
+    #[pyo3(get, set)]
     pub output_log_level: Option<String>,
     #[pyo3(get, set)]
     pub global_custom_fields: Option<HashMap<String, ValidPrimitivesPy>>,
@@ -211,6 +213,7 @@ impl StatsigOptionsPy {
         fallback_to_statsig_api=None,
         environment=None,
         service_name=None,
+        sdk_instance_id=None,
         output_log_level=None,
         global_custom_fields=None,
         observability_client=None,
@@ -249,6 +252,7 @@ impl StatsigOptionsPy {
         fallback_to_statsig_api: Option<bool>,
         environment: Option<String>,
         service_name: Option<String>,
+        sdk_instance_id: Option<String>,
         output_log_level: Option<String>,
         global_custom_fields: Option<HashMap<String, ValidPrimitivesPy>>,
         observability_client: Option<Py<ObservabilityClientBasePy>>,
@@ -285,6 +289,7 @@ impl StatsigOptionsPy {
             fallback_to_statsig_api,
             environment,
             service_name,
+            sdk_instance_id,
             output_log_level,
             global_custom_fields,
             observability_client,
@@ -381,6 +386,7 @@ fn create_inner_statsig_options(
         fallback_to_statsig_api: opts.fallback_to_statsig_api,
         environment: opts.environment.clone(),
         service_name: opts.service_name.clone(),
+        sdk_instance_id: opts.sdk_instance_id.clone(),
         id_lists_adapter: None,
         override_adapter: None,
         output_log_level: opts

oai_statsig_python_core-0.24.0/statsig-pyo3/tests/event_listener_reentrancy_subprocess.py

@@ -0,0 +1,13 @@
+from statsig_python_core import Statsig, StatsigUser
+
+
+statsig = Statsig("secret-key")
+
+
+def unsubscribe_from_callback(_event):
+    statsig.unsubscribe("gate_evaluated")
+
+
+statsig.subscribe("gate_evaluated", unsubscribe_from_callback)
+statsig.check_gate(StatsigUser("a-user"), "test_gate")
+statsig.shutdown().wait(timeout=1)

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-pyo3/tests/test_initialize_with_details.py

@@ -40,6 +40,24 @@ def test_initialize_with_details_success(statsig_setup):
     assert init_details.failure_details is None
 
 
+def test_is_config_spec_ready_reports_current_specs_state(statsig_setup):
+    mock_server = statsig_setup
+    options = StatsigOptions(
+        specs_url=mock_server.url_for("/v2/download_config_specs"),
+        log_event_url=mock_server.url_for("/v1/log_event"),
+    )
+    statsig = Statsig("secret-key", options)
+
+    try:
+        assert not statsig.is_config_spec_ready()
+
+        statsig.initialize().wait()
+
+        assert statsig.is_config_spec_ready()
+    finally:
+        statsig.shutdown().wait()
+
+
 def test_initialize_with_details_failure(statsig_setup):
     mock_server = statsig_setup
     options = StatsigOptions(

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-pyo3/tests/test_interned_store_preload.py

@@ -48,3 +48,30 @@ def test_interned_store_preload(server_setup):
 
     assert gate.details.reason == "Network:Recognized"
     assert log_provider.error_count == 0
+
+
+def test_interned_store_mmap_preload(server_setup):
+    specs_url, log_event_url = server_setup
+    sdk_key = "secret-key"
+
+    fetch_complete = InternedStore.fetch_and_write_mmap(sdk_key, specs_url)
+    assert fetch_complete.wait(5)
+    InternedStore.preload_mmap(sdk_key)
+
+    log_provider = MockOutputLoggerProvider()
+    log_provider.logs = []
+
+    statsig = Statsig(
+        sdk_key,
+        StatsigOptions(
+            specs_url=specs_url,
+            log_event_url=log_event_url,
+            output_logger_provider=log_provider,
+        ),
+    )
+    statsig.initialize().wait()
+    gate = statsig.get_feature_gate(StatsigUser("a-user"), "test_public")
+    statsig.shutdown().wait()
+
+    assert gate.details.reason == "Network:Recognized"
+    assert log_provider.error_count == 0

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-pyo3/tests/test_sdk_event_emitter.py

@@ -1,6 +1,9 @@
 import gc
 import json
+import subprocess
+import sys
 from threading import Event
+from pathlib import Path
 from time import monotonic, sleep
 from weakref import ref
 
@@ -11,6 +14,9 @@ from werkzeug import Response
 from utils import get_test_data_resource
 
 
+TESTS_DIR = Path(__file__).parent
+
+
 def test_subscribe_receives_gate_event():
     statsig = Statsig("secret-key")
     received_events = []
@@ -230,6 +236,14 @@ def test_unsubscribe_by_id_stops_callback_delivery():
         statsig.shutdown().wait()
 
 
+def test_listener_can_unsubscribe_from_callback_without_deadlocking():
+    subprocess.run(
+        [sys.executable, TESTS_DIR / "event_listener_reentrancy_subprocess.py"],
+        check=True,
+        timeout=2,
+    )
+
+
 def test_unsubscribe_by_event_stops_callback_delivery():
     statsig = Statsig("secret-key")
     received_events = []

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-rust/Cargo.toml

@@ -43,7 +43,7 @@ serde_json = { version = "1.0.143", features = [
 ] }
 serde_with = "3.4.0"
 sha2 = "0.10.8"
-oai-statsig-grpc = { path = "../statsig-grpc", version = "0.23.1", optional = true }
+oai-statsig-grpc = { path = "../statsig-grpc", version = "0.24.0", optional = true }
 simple_logger = { version = "5.0.0" }
 tempfile = "3.8.1"
 tokio = { version = "1.39.1", features = ["full"] }
@@ -55,7 +55,7 @@ brotli = "8.0.2"
 bytes = "1.7.0"
 prost = { version = "0.14", features = ["derive"] }
 memmap2 = "0.9"
-rkyv = "0.8"
+rkyv = { version = "0.8", features = ["aligned", "little_endian", "pointer_width_32"] }
 ouroboros = "0.18.0"
 
 [target.'cfg(target_env = "gnu")'.dependencies]

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-rust/src/__tests__/mod.rs

@@ -1,3 +1,4 @@
 mod interned_string_tests;
 mod spec_store_tests;
+mod statsig_options_tests;
 mod user_value_tests;

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-rust/src/__tests__/spec_store_tests.rs

@@ -1,5 +1,6 @@
 use std::collections::HashMap;
-use std::sync::Arc;
+use std::sync::{mpsc, Arc};
+use std::time::Duration;
 
 use async_trait::async_trait;
 use parking_lot::Mutex;
@@ -11,7 +12,7 @@ use crate::{
     output_logger::{
         initialize_output_logger, shutdown_output_logger, LogLevel, OutputLogProvider,
     },
-    sdk_event_emitter::SdkEventEmitter,
+    sdk_event_emitter::{SdkEvent, SdkEventEmitter},
     SpecStore, SpecsSource, SpecsUpdate, StatsigErr, StatsigOptions, StatsigRuntime,
 };
 
@@ -381,3 +382,44 @@ fn test_no_updates_update_field_short_circuit_parse() {
 
     assert!(update_result.is_ok())
 }
+
+#[test]
+fn test_specs_updated_callback_runs_without_holding_store_lock() {
+    let event_emitter = Arc::new(SdkEventEmitter::default());
+    let spec_store = Arc::new(SpecStore::new(
+        "test",
+        "test".to_string(),
+        StatsigRuntime::get_runtime(),
+        event_emitter.clone(),
+        None,
+    ));
+    let callback_store = spec_store.clone();
+
+    event_emitter.subscribe(SdkEvent::SPECS_UPDATED, move |event| {
+        let SdkEvent::SpecsUpdated { source, values, .. } = event else {
+            panic!("expected a specs updated event");
+        };
+        assert_eq!(source, &SpecsSource::Network);
+        assert!(values.time > 0);
+
+        callback_store.set_source(SpecsSource::Bootstrap);
+    });
+
+    let (completed_tx, completed_rx) = mpsc::channel();
+    std::thread::spawn(move || {
+        let contents = include_bytes!("../../tests/data/eval_proj_dcs.json");
+        let result = spec_store.set_values(SpecsUpdate {
+            data: ResponseData::from_bytes(contents.to_vec()),
+            source: SpecsSource::Network,
+            received_at: 2000,
+            source_api: None,
+            has_updates: None,
+        });
+        completed_tx.send(result).unwrap();
+    });
+
+    let result = completed_rx
+        .recv_timeout(Duration::from_millis(500))
+        .expect("specs updated callback deadlocked while re-entering the spec store");
+    assert!(result.is_ok());
+}

oai_statsig_python_core-0.24.0/statsig-rust/src/__tests__/statsig_options_tests.rs

@@ -0,0 +1,20 @@
+use crate::StatsigOptions;
+
+#[test]
+fn test_sdk_instance_id_defaults_to_sdk_key() {
+    let options = StatsigOptions::new();
+
+    assert_eq!(options.get_sdk_instance_id("secret-key"), "secret-key");
+}
+
+#[test]
+fn test_sdk_instance_id_can_be_overridden() {
+    let options = StatsigOptions::builder()
+        .sdk_instance_id(Some("validator:route:target".to_string()))
+        .build();
+
+    assert_eq!(
+        options.get_sdk_instance_id("secret-key"),
+        "validator:route:target"
+    );
+}

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-rust/src/console_capture/console_capture_instances.rs

@@ -42,7 +42,8 @@ impl ConsoleCaptureInstance {
             .clone()
             .unwrap_or_default();
         let enabled = console_capture_options.enabled;
-        let ops_stats_instance = OPS_STATS.get_for_instance(sdk_key);
+        let sdk_instance_id = statsig_options.get_sdk_instance_id(sdk_key);
+        let ops_stats_instance = OPS_STATS.get_for_instance(sdk_instance_id);
         let allowed_log_levels = console_capture_options
             .log_levels
             .filter(|levels| !levels.is_empty())
@@ -95,7 +96,8 @@ impl ConsoleCaptureRegistry {
             .try_read_for(std::time::Duration::from_secs(5))
         {
             Some(read_guard) => {
-                if let Some(instance) = read_guard.get(sdk_key) {
+                let sdk_instance_id = options.get_sdk_instance_id(sdk_key);
+                if let Some(instance) = read_guard.get(sdk_instance_id) {
                     if let Some(instance) = instance.upgrade() {
                         return instance.clone();
                     }
@@ -115,7 +117,8 @@ impl ConsoleCaptureRegistry {
             .try_write_for(std::time::Duration::from_secs(5))
         {
             Some(mut write_guard) => {
-                write_guard.insert(sdk_key.into(), Arc::downgrade(&instance));
+                let sdk_instance_id = options.get_sdk_instance_id(sdk_key);
+                write_guard.insert(sdk_instance_id.into(), Arc::downgrade(&instance));
             }
             None => {
                 log_e!(

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-rust/src/evaluation/evaluation_types.rs

@@ -67,6 +67,7 @@ pub struct BaseEvaluation {
     pub name: InternedString,
     pub rule_id: InternedString,
     pub secondary_exposures: Vec<SecondaryExposure>,
+    pub version: Option<u32>,
 
     #[serde(skip_serializing)]
     pub(crate) exposure_info: Option<ExtraExposureInfo>,
@@ -133,12 +134,13 @@ pub struct GateEvaluation {
 
 impl GCIRHashable for GateEvaluation {
     fn create_hash(&self, name: &InternedString) -> u64 {
-        let hash_array = vec![
+        let mut hash_array = vec![
             name.hash,
             self.value as u64,
             self.base.rule_id.hash,
             hash_secondary_exposures(&self.base.secondary_exposures),
         ];
+        push_optional_version_hash_values(&mut hash_array, &self.base.version);
         hashing::hash_one(hash_array)
     }
 }
@@ -159,13 +161,14 @@ pub struct DynamicConfigEvaluation {
 
 impl GCIRHashable for DynamicConfigEvaluation {
     fn create_hash(&self, name: &InternedString) -> u64 {
-        let hash_array = vec![
+        let mut hash_array = vec![
             name.hash,
             self.value.get_hash(),
             self.base.rule_id.hash,
             hash_secondary_exposures(&self.base.secondary_exposures),
             self.passed as u64,
         ];
+        push_optional_version_hash_values(&mut hash_array, &self.base.version);
         hashing::hash_one(hash_array)
     }
 }
@@ -208,6 +211,7 @@ impl GCIRHashable for ExperimentEvaluation {
             hash_secondary_exposures(&self.base.secondary_exposures),
             self.is_in_layer as u64,
         ];
+        push_optional_version_hash_values(&mut hash_array, &self.base.version);
         let mut explicit_params_hashes = Vec::new();
         if let Some(explicit_parameters) = &self.explicit_parameters {
             for value in explicit_parameters.to_vec_interned() {
@@ -284,6 +288,7 @@ impl GCIRHashable for LayerEvaluation {
                 .as_ref()
                 .map_or(0, |n| n.hash),
         ];
+        push_optional_version_hash_values(&mut hash_array, &self.base.version);
         let mut explicit_params_hashes = Vec::new();
         for value in self.explicit_parameters.to_vec_interned() {
             explicit_params_hashes.push(value.hash);
@@ -316,3 +321,29 @@ fn hash_secondary_exposures(exposures: &Vec<SecondaryExposure>) -> u64 {
     }
     hashing::hash_one(secondary_exposure_hashes)
 }
+
+pub(crate) fn push_optional_version_hash_values(hash_array: &mut Vec<u64>, version: &Option<u32>) {
+    hash_array.push(version.is_some() as u64);
+    hash_array.push(version.map_or(0, u64::from));
+}
+
+#[cfg(test)]
+mod tests {
+    use super::push_optional_version_hash_values;
+
+    #[test]
+    fn optional_version_hash_values_distinguish_missing_from_zero() {
+        let mut missing = Vec::new();
+        push_optional_version_hash_values(&mut missing, &None);
+
+        let mut zero = Vec::new();
+        push_optional_version_hash_values(&mut zero, &Some(0));
+
+        let mut one = Vec::new();
+        push_optional_version_hash_values(&mut one, &Some(1));
+
+        assert_eq!(missing, vec![0, 0]);
+        assert_eq!(zero, vec![1, 0]);
+        assert_eq!(one, vec![1, 1]);
+    }
+}

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-rust/src/evaluation/evaluation_types_v2.rs

@@ -3,7 +3,8 @@ use crate::gcir::gcir_formatter::GCIRHashable;
 use crate::hashing::opt_bool_to_hashable;
 use crate::interned_string::InternedString;
 use crate::{
-    evaluation::evaluation_types::is_false, specs_response::explicit_params::ExplicitParameters,
+    evaluation::evaluation_types::{is_false, push_optional_version_hash_values},
+    specs_response::explicit_params::ExplicitParameters,
 };
 use crate::{hashing, SecondaryExposure};
 use serde::{Deserialize, Serialize};
@@ -13,6 +14,7 @@ pub struct BaseEvaluationV2 {
     pub name: String,
     pub rule_id: InternedString,
     pub secondary_exposures: Vec<String>,
+    pub version: Option<u32>,
 }
 
 #[derive(Serialize, Deserialize, Clone)]
@@ -27,12 +29,13 @@ pub struct GateEvaluationV2 {
 
 impl GCIRHashable for GateEvaluationV2 {
     fn create_hash(&self, name: &InternedString) -> u64 {
-        let hash_array = vec![
+        let mut hash_array = vec![
             name.hash,
             self.value as u64,
             self.base.rule_id.hash,
             hash_secondary_exposures(&self.base.secondary_exposures),
         ];
+        push_optional_version_hash_values(&mut hash_array, &self.base.version);
         hashing::hash_one(hash_array)
     }
 }
@@ -53,13 +56,14 @@ pub struct DynamicConfigEvaluationV2 {
 
 impl GCIRHashable for DynamicConfigEvaluationV2 {
     fn create_hash(&self, name: &InternedString) -> u64 {
-        let hash_array = vec![
+        let mut hash_array = vec![
             name.hash,
             self.value.get_hash(),
             self.base.rule_id.hash,
             hash_secondary_exposures(&self.base.secondary_exposures),
             self.passed as u64,
         ];
+        push_optional_version_hash_values(&mut hash_array, &self.base.version);
         hashing::hash_one(hash_array)
     }
 }
@@ -102,6 +106,7 @@ impl GCIRHashable for ExperimentEvaluationV2 {
             hash_secondary_exposures(&self.base.secondary_exposures),
             self.is_in_layer as u64,
         ];
+        push_optional_version_hash_values(&mut hash_array, &self.base.version);
         let mut explicit_params_hashes = Vec::new();
         if let Some(explicit_parameters) = &self.explicit_parameters {
             for value in explicit_parameters.to_vec_interned() {
@@ -175,6 +180,7 @@ impl GCIRHashable for LayerEvaluationV2 {
                 .as_ref()
                 .map_or(0, |n| n.hash),
         ];
+        push_optional_version_hash_values(&mut hash_array, &self.base.version);
         let mut explicit_params_hashes = Vec::new();
         for value in self.explicit_parameters.to_vec_interned() {
             explicit_params_hashes.push(value.hash);

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-rust/src/evaluation/evaluator_result.rs

@@ -560,6 +560,7 @@ fn result_to_base_eval(spec_name: &str, result: &mut EvaluatorResult) -> BaseEva
         name,
         rule_id,
         secondary_exposures: std::mem::take(&mut result.secondary_exposures),
+        version: result.version,
         exposure_info: Some(exposure_info),
     }
 }
@@ -598,6 +599,7 @@ fn result_to_base_eval_v2(
         name: spec_name.to_string(),
         rule_id,
         secondary_exposures: exposures,
+        version: result.version,
     }
 }
 

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-rust/src/event_logging/event_logger.rs

@@ -74,6 +74,7 @@ impl EventLogger {
         event_logging_adapter: &Arc<dyn EventLoggingAdapter>,
         statsig_rt: &Arc<StatsigRuntime>,
     ) -> Arc<Self> {
+        let sdk_instance_id = options.get_sdk_instance_id(sdk_key);
         let me = Arc::new(Self {
             queue: EventQueue::new(
                 options
@@ -83,7 +84,7 @@ impl EventLogger {
                     .event_logging_max_pending_batch_queue_size
                     .unwrap_or(DEFAULT_PENDING_BATCH_COUNT_MAX),
             ),
-            event_sampler: ExposureSampling::new(sdk_key),
+            event_sampler: ExposureSampling::new(sdk_instance_id),
             flush_interval: FlushInterval::new(),
             options: options.clone(),
             logging_adapter: event_logging_adapter.clone(),
@@ -91,8 +92,8 @@ impl EventLogger {
             shutdown_notify: Notify::new(),
             limit_flush_notify: Notify::new(),
             limit_flush_semaphore: Arc::new(Semaphore::new(MAX_LIMIT_FLUSH_TASKS)),
-            ops_stats: OPS_STATS.get_for_instance(sdk_key),
-            sec_expo_experiment: SecExpoAsPrimaryExperiment::new(sdk_key, options),
+            ops_stats: OPS_STATS.get_for_instance(sdk_instance_id),
+            sec_expo_experiment: SecExpoAsPrimaryExperiment::new(sdk_instance_id, options),
             enqueue_dropped_events_count: AtomicU64::new(0),
         });
 

{oai_statsig_python_core-0.23.1 → oai_statsig_python_core-0.24.0}/statsig-rust/src/event_logging/exposure_sampling.rs

@@ -53,7 +53,7 @@ pub struct ExposureSampling {
 }
 
 impl ExposureSampling {
-    pub fn new(sdk_key: &str) -> Self {
+    pub fn new(sdk_instance_id: &str) -> Self {
         let now = Utc::now().timestamp_millis() as u64;
 
         Self {
@@ -63,7 +63,7 @@ impl ExposureSampling {
             exposure_dedupe_set: RwLock::from(AHashSet::default()),
             last_exposure_dedupe_reset: AtomicU64::from(now),
 
-            global_configs: GlobalConfigs::get_instance(sdk_key),
+            global_configs: GlobalConfigs::get_instance(sdk_instance_id),
         }
     }