prefab-cloud-ruby 0.24.6 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36d1b27d1c03aef56882735b73835b65b508b5eef821f65e89ab5ba77f54a8ab
-  data.tar.gz: 00a336ac664ab495ce1c5fff4b5ab6972169a095087887958c3a0881016ba273
+  metadata.gz: eb91b8be5a01c3bab3312d277a27117c0449713623e1605863486c3809cff0cd
+  data.tar.gz: 3deaeb9fcb458ef461771865463c5163f8b5c6833a45d1e7af4ec3474f5d04bd
 SHA512:
-  metadata.gz: b0dc48a30fd50f53e9709e138afd408bfc2e91e6badb473c1d141db56c7cdfa507aef81db0f06674b8d640dec2a7ac147654def0cdd7e7bcebdd3b0eb2a79854
-  data.tar.gz: a3bd2f91776a25d646f1ab28d54d395bd24283e16dbca2f0bc7a1d1f77c9bb063488e9f62e00030a8acc84b2ecb8b07791bf84c1893e091f40edbb75e55870ca
+  metadata.gz: 2555fc0364fef33c11e3e7866a00ab5df766e859c73baedbb39712ddceb07ec96d0216c5e79dd7336e40951de85b773100864356fd24423f3257a8e3ab40e073
+  data.tar.gz: 44c9813ede3c95db9820e0247c71c7e4a1e0e21b5981d074025baf546e5f3e7de6362744acedd831437e3b10ca747cdb73ec596ec1394b82ff7e70481ab57a52

data/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 1.0.1 - 2023-08-17
+
+- Bug fix for StringList w/ ExampleContextsAggregator (#141)
+
+## 1.0.0 - 2023-08-10
+
+- Removed EvaluatedKeysAggregator (#137)
+- Change `collect_evaluation_summaries` default to true (#136)
+- Removed some backwards compatibility shims (#133)
+- Standardizing options (#132)
+  - Note that the default value for `context_upload_mode` is `:periodic_example` which means example contexts will be collected.
+    This enables easy variant override assignment in our UI. More at https://prefab.cloud/blog/feature-flag-variant-assignment/
+
 ## 0.24.6 - 2023-07-31
 
 - Logger Client compatibility (#129)

data/VERSION

@@ -1 +1 @@
-0.24.6
+1.0.1

data/lib/prefab/client.rb

@@ -7,12 +7,10 @@ module Prefab
     MAX_SLEEP_SEC = 10
     BASE_SLEEP_SEC = 0.5
 
-    attr_reader :shared_cache, :stats, :namespace, :interceptor, :api_key, :prefab_api_url, :options, :instance_hash
+    attr_reader :namespace, :interceptor, :api_key, :prefab_api_url, :options, :instance_hash
 
     def initialize(options = Prefab::Options.new)
       @options = options.is_a?(Prefab::Options) ? options : Prefab::Options.new(options)
-      @shared_cache = @options.shared_cache
-      @stats = @options.stats
       @namespace = @options.namespace
       @stubs = {}
       @instance_hash = UUID.new.generate
@@ -32,11 +30,6 @@ module Prefab
       config_client
     end
 
-    def with_log_context(_lookup_key, properties, &block)
-      warn '[DEPRECATION] `$prefab.with_log_context` is deprecated.  Please use `with_context` instead.'
-      with_context(properties, &block)
-    end
-
     def with_context(properties, &block)
       Prefab::Context.with_context(properties, &block)
     end
@@ -73,13 +66,6 @@ module Prefab
                                                                sync_interval: @options.collect_sync_interval)
     end
 
-    def evaluated_keys_aggregator
-      return nil if @options.collect_max_keys <= 0
-
-      @evaluated_keys_aggregator ||= EvaluatedKeysAggregator.new(client: self, max_keys: @options.collect_max_keys,
-                                                                 sync_interval: @options.collect_sync_interval)
-    end
-
     def example_contexts_aggregator
       return nil if @options.collect_max_example_contexts <= 0
 
@@ -117,19 +103,15 @@ module Prefab
       log.log_internal msg, path, nil, level
     end
 
-    def enabled?(feature_name, lookup_key = NO_DEFAULT_PROVIDED, properties = NO_DEFAULT_PROVIDED)
-      _, properties = handle_positional_arguments(lookup_key, properties, :enabled?)
-
-      feature_flag_client.feature_is_on_for?(feature_name, properties)
+    def enabled?(feature_name, jit_context = NO_DEFAULT_PROVIDED)
+      feature_flag_client.feature_is_on_for?(feature_name, jit_context)
     end
 
-    def get(key, default_or_lookup_key = NO_DEFAULT_PROVIDED, properties = NO_DEFAULT_PROVIDED, ff_default = nil)
+    def get(key, default = NO_DEFAULT_PROVIDED, jit_context = NO_DEFAULT_PROVIDED)
       if is_ff?(key)
-        default, properties = handle_positional_arguments(default_or_lookup_key, properties, :get)
-
-        feature_flag_client.get(key, properties, default: ff_default)
+        feature_flag_client.get(key, jit_context, default: default)
       else
-        config_client.get(key, default_or_lookup_key, properties)
+        config_client.get(key, default, jit_context)
       end
     end
 
@@ -161,26 +143,5 @@ module Prefab
 
       raw && raw.allowable_values.any?
     end
-
-    # The goal here is to ease transition from the old API to the new one. The
-    # old API had a lookup_key parameter that is deprecated. This method
-    # handles the transition by checking if the first parameter is a string and
-    # if so, it is assumed to be the lookup_key and a deprecation warning is
-    # issued and we know the second argument is the properties. If the first
-    # parameter is a hash, you're on the new API and no further action is
-    # required.
-    def handle_positional_arguments(lookup_key, properties, method)
-      # handle JIT context
-      if lookup_key.is_a?(Hash) && properties == NO_DEFAULT_PROVIDED
-        properties = lookup_key
-        lookup_key = nil
-      end
-
-      if lookup_key.is_a?(String)
-        warn "[DEPRECATION] `$prefab.#{method}`'s lookup_key argument is deprecated. Please remove it or use context instead."
-      end
-
-      [lookup_key, properties]
-    end
   end
 end

data/lib/prefab/config_client.rb

@@ -58,14 +58,13 @@ module Prefab
     def get(key, default = NO_DEFAULT_PROVIDED, properties = NO_DEFAULT_PROVIDED)
       context = @config_resolver.make_context(properties)
 
-      if properties != NO_DEFAULT_PROVIDED && @base_client.example_contexts_aggregator
+      if !context.blank? && @base_client.example_contexts_aggregator
         @base_client.example_contexts_aggregator.record(context)
       end
 
       evaluation = _get(key, context)
 
       @base_client.context_shape_aggregator&.push(context)
-      @base_client.evaluated_keys_aggregator&.push(key)
 
       if evaluation
         evaluation.report_and_return(@base_client.evaluation_summary_aggregator)
@@ -89,7 +88,7 @@ module Prefab
     end
 
     def _get(key, properties)
-      # wait timeout sec for the initalization to be complete
+      # wait timeout sec for the initialization to be complete
       @initialized_future.value(@options.initialization_timeout_sec)
       if @initialized_future.incomplete?
         unless @options.on_init_failure == Prefab::Options::ON_INITIALIZATION_FAILURE::RETURN
@@ -157,7 +156,6 @@ module Prefab
         @base_client.log_internal ::Logger::DEBUG,
                                   "Checkpoint with highwater id #{@config_loader.highwater_mark} from #{source}. No changes.", 'load_configs'
       end
-      @base_client.stats.increment('prefab.config.checkpoint.load')
       @config_resolver.update
       finish_init!(source)
     end

data/lib/prefab/config_resolver.rb

@@ -10,6 +10,7 @@ module Prefab
       @config_loader = config_loader
       @project_env_id = 0 # we don't know this yet, it is set from the API results
       @base_client = base_client
+      @on_update = nil
       make_local
     end
 
@@ -54,7 +55,7 @@ module Prefab
     end
 
     def make_context(properties)
-      if properties == NO_DEFAULT_PROVIDED
+      if properties == NO_DEFAULT_PROVIDED || properties.nil?
         Context.current
       elsif properties.is_a?(Context)
         properties

data/lib/prefab/config_value_wrapper.rb

@@ -9,7 +9,7 @@ module Prefab
       when TrueClass, FalseClass
         PrefabProto::ConfigValue.new(bool: value)
       when Array
-        PrefabProto::ConfigValue.new(string_list: Prefab::StringList.new(values: value.map(&:to_s)))
+        PrefabProto::ConfigValue.new(string_list: PrefabProto::StringList.new(values: value.map(&:to_s)))
       else
         PrefabProto::ConfigValue.new(string: value.to_s)
       end

data/lib/prefab/context.rb

@@ -91,6 +91,10 @@ module Prefab
       end
     end
 
+    def blank?
+      contexts.empty?
+    end
+
     def set(name, hash)
       @contexts[name.to_s] = NamedContext.new(name, hash)
     end

data/lib/prefab/feature_flag_client.rb

@@ -11,8 +11,6 @@ module Prefab
     end
 
     def feature_is_on_for?(feature_name, properties)
-      @base_client.stats.increment('prefab.featureflag.on', tags: ["feature:#{feature_name}"])
-
       variant = @base_client.config_client.get(feature_name, false, properties)
 
       is_on?(variant)

data/lib/prefab/options.rb

@@ -7,8 +7,6 @@ module Prefab
     attr_reader :logdev
     attr_reader :log_prefix
     attr_reader :log_formatter
-    attr_reader :stats
-    attr_reader :shared_cache
     attr_reader :namespace
     attr_reader :prefab_api_url
     attr_reader :on_no_default
@@ -33,22 +31,21 @@ module Prefab
     }
 
     module ON_INITIALIZATION_FAILURE
-      RAISE = 1
-      RETURN = 2
+      RAISE = :raise
+      RETURN = :return
     end
 
     module ON_NO_DEFAULT
-      RAISE = 1
-      RETURN_NIL = 2
+      RAISE = :raise
+      RETURN_NIL = :return_nil
     end
 
     module DATASOURCES
-      ALL = 1
-      LOCAL_ONLY = 2
+      ALL = :all
+      LOCAL_ONLY = :local_only
     end
 
     DEFAULT_MAX_PATHS = 1_000
-    DEFAULT_MAX_CONTEXT_KEYS = 100_000
     DEFAULT_MAX_KEYS = 100_000
     DEFAULT_MAX_EXAMPLE_CONTEXTS = 100_000
     DEFAULT_MAX_EVAL_SUMMARIES = 100_000
@@ -56,37 +53,28 @@ module Prefab
     private def init(
       api_key: ENV['PREFAB_API_KEY'],
       logdev: $stdout,
-      stats: NoopStats.new, # receives increment("prefab.limitcheck", {:tags=>["policy_group:page_view", "pass:true"]})
-      shared_cache: NoopCache.new, # Something that quacks like Rails.cache ideally memcached
       namespace: '',
       log_formatter: DEFAULT_LOG_FORMATTER,
       log_prefix: nil,
       prefab_api_url: ENV['PREFAB_API_URL'] || 'https://api.prefab.cloud',
       on_no_default: ON_NO_DEFAULT::RAISE, # options :raise, :warn_and_return_nil,
       initialization_timeout_sec: 10, # how long to wait before on_init_failure
-      on_init_failure: ON_INITIALIZATION_FAILURE::RAISE, # options :unlock_and_continue, :lock_and_keep_trying, :raise
-      # new_config_callback: nil, #callback method
-      # live_override_url: nil,
+      on_init_failure: ON_INITIALIZATION_FAILURE::RAISE,
       prefab_datasources: ENV['PREFAB_DATASOURCES'] == 'LOCAL_ONLY' ? DATASOURCES::LOCAL_ONLY : DATASOURCES::ALL,
       prefab_config_override_dir: Dir.home,
-      prefab_config_classpath_dir: '.',
+      prefab_config_classpath_dir: '.', # where to load local overrides
       prefab_envs: ENV['PREFAB_ENVS'].nil? ? [] : ENV['PREFAB_ENVS'].split(','),
-      collect_logs: true,
+      collect_logger_counts: true,
       collect_max_paths: DEFAULT_MAX_PATHS,
       collect_sync_interval: nil,
-      collect_shapes: true,
-      collect_max_shapes: DEFAULT_MAX_CONTEXT_KEYS,
-      collect_keys: false,
-      collect_max_keys: DEFAULT_MAX_KEYS,
-      collect_example_contexts: false,
-      collect_max_example_contexts: DEFAULT_MAX_EXAMPLE_CONTEXTS,
-      collect_evaluation_summaries: false,
-      collect_max_evaluation_summaries: DEFAULT_MAX_EVAL_SUMMARIES
+      context_upload_mode: :periodic_example, # :periodic_example, :shape_only, :none
+      context_max_size: DEFAULT_MAX_EVAL_SUMMARIES,
+      collect_evaluation_summaries: true,
+      collect_max_evaluation_summaries: DEFAULT_MAX_EVAL_SUMMARIES,
+      allow_telemetry_in_local_mode: false
     )
       @api_key = api_key
       @logdev = logdev
-      @stats = stats
-      @shared_cache = shared_cache
       @namespace = namespace
       @log_formatter = log_formatter
       @log_prefix = log_prefix
@@ -98,17 +86,31 @@ module Prefab
       @prefab_config_classpath_dir = prefab_config_classpath_dir
       @prefab_config_override_dir = prefab_config_override_dir
       @prefab_envs = Array(prefab_envs)
-      @collect_logs = collect_logs
+      @collect_logger_counts = collect_logger_counts
       @collect_max_paths = collect_max_paths
       @collect_sync_interval = collect_sync_interval
-      @collect_shapes = collect_shapes
-      @collect_max_shapes = collect_max_shapes
-      @collect_keys = collect_keys
-      @collect_max_keys = collect_max_keys
-      @collect_example_contexts = collect_example_contexts
-      @collect_max_example_contexts = collect_max_example_contexts
       @collect_evaluation_summaries = collect_evaluation_summaries
       @collect_max_evaluation_summaries = collect_max_evaluation_summaries
+      @allow_telemetry_in_local_mode = allow_telemetry_in_local_mode
+
+      # defaults that may be overridden by context_upload_mode
+      @collect_shapes = false
+      @collect_max_shapes = 0
+      @collect_example_contexts = false
+      @collect_max_example_contexts = 0
+
+      case context_upload_mode
+      when :none
+        # do nothing
+      when :periodic_example
+        @collect_example_contexts = true
+        @collect_max_example_contexts = context_max_size
+      when :shape_only
+        @collect_shapes = true
+        @collect_max_shapes = context_max_size
+      else
+        raise "Unknown context_upload_mode #{context_upload_mode}. Please provide :periodic_example, :shape_only, or :none."
+      end
     end
 
     def initialize(options = {})
@@ -120,7 +122,7 @@ module Prefab
     end
 
     def collect_max_paths
-      return 0 unless telemetry_allowed?(@collect_logs)
+      return 0 unless telemetry_allowed?(@collect_logger_counts)
 
       @collect_max_paths
     end
@@ -131,12 +133,6 @@ module Prefab
       @collect_max_shapes
     end
 
-    def collect_max_keys
-      return 0 unless telemetry_allowed?(@collect_keys)
-
-      @collect_max_keys
-    end
-
     def collect_max_example_contexts
       return 0 unless telemetry_allowed?(@collect_example_contexts)
 
@@ -157,7 +153,7 @@ module Prefab
     private
 
     def telemetry_allowed?(option)
-      option && !local_only? || option == :force
+      option && (!local_only? || @allow_telemetry_in_local_mode)
     end
 
     def remove_trailing_slash(url)

data/lib/prefab-cloud-ruby.rb

@@ -22,7 +22,6 @@ require 'prefab/options'
 require 'prefab/internal_logger'
 require 'prefab/rate_limit_cache'
 require 'prefab/context_shape_aggregator'
-require 'prefab/evaluated_keys_aggregator'
 require 'prefab/example_contexts_aggregator'
 require 'prefab/evaluation_summary_aggregator'
 require 'prefab/log_path_aggregator'
@@ -43,6 +42,4 @@ require 'prefab/logger_client'
 require 'prefab/client'
 require 'prefab/config_client'
 require 'prefab/feature_flag_client'
-require 'prefab/noop_cache'
-require 'prefab/noop_stats'
 require 'prefab/murmer3'

data/prefab-cloud-ruby.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Juwelier::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: prefab-cloud-ruby 0.24.6 ruby lib
+# stub: prefab-cloud-ruby 1.0.1 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "prefab-cloud-ruby".freeze
-  s.version = "0.24.6"
+  s.version = "1.0.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
   s.authors = ["Jeff Dwyer".freeze]
-  s.date = "2023-07-31"
+  s.date = "2023-08-17"
   s.description = "Feature Flags, Live Config, and Dynamic Log Levels as a service".freeze
   s.email = "jdwyer@prefab.cloud".freeze
   s.executables = ["console".freeze]
@@ -52,7 +52,6 @@ Gem::Specification.new do |s|
     "lib/prefab/errors/initialization_timeout_error.rb",
     "lib/prefab/errors/invalid_api_key_error.rb",
     "lib/prefab/errors/missing_default_error.rb",
-    "lib/prefab/evaluated_keys_aggregator.rb",
     "lib/prefab/evaluation.rb",
     "lib/prefab/evaluation_summary_aggregator.rb",
     "lib/prefab/example_contexts_aggregator.rb",
@@ -64,8 +63,6 @@ Gem::Specification.new do |s|
     "lib/prefab/log_path_aggregator.rb",
     "lib/prefab/logger_client.rb",
     "lib/prefab/murmer3.rb",
-    "lib/prefab/noop_cache.rb",
-    "lib/prefab/noop_stats.rb",
     "lib/prefab/options.rb",
     "lib/prefab/periodic_sync.rb",
     "lib/prefab/rate_limit_cache.rb",
@@ -89,11 +86,11 @@ Gem::Specification.new do |s|
     "test/test_config_loader.rb",
     "test/test_config_resolver.rb",
     "test/test_config_value_unwrapper.rb",
+    "test/test_config_value_wrapper.rb",
     "test/test_context.rb",
     "test/test_context_shape.rb",
     "test/test_context_shape_aggregator.rb",
     "test/test_criteria_evaluator.rb",
-    "test/test_evaluated_keys_aggregator.rb",
     "test/test_evaluation_summary_aggregator.rb",
     "test/test_example_contexts_aggregator.rb",
     "test/test_exponential_backoff.rb",

data/test/integration_test.rb

@@ -18,8 +18,6 @@ class IntegrationTest
       :raise
     elsif @expected[:value].nil?
       :nil
-    elsif @func == :feature_is_on_for?
-      :feature_flag
     else
       :simple_equality
     end
@@ -60,7 +58,7 @@ class IntegrationTest
   end
 
   def parse_ff_input(input)
-    [input['flag'], input['context']]
+    [input['flag'], input['default'], input['context']]
   end
 
   def parse_expected(expected)

data/test/integration_test_helpers.rb

@@ -14,7 +14,6 @@ module IntegrationTestHelpers
       raise message if RAISE_IF_NO_TESTS_FOUND
 
       puts message
-
     end
 
     files

data/test/support/common_helpers.rb

@@ -1,6 +1,32 @@
 # frozen_string_literal: true
 
 module CommonHelpers
+  require 'timecop'
+
+  def setup
+    $oldstderr, $stderr = $stderr, StringIO.new
+
+    $logs = nil
+    Timecop.freeze('2023-08-09 15:18:12 -0400')
+  end
+
+  def teardown
+    if $logs && !$logs.string.empty?
+      raise "Unexpected logs. Handle logs with assert_only_expected_logs or assert_logged\n\n#{$logs.string}"
+    end
+
+    if $stderr != $oldstderr && !$stderr.string.empty?
+      # we ignore 2.X because of the number of `instance variable @xyz not initialized` warnings
+      if !RUBY_VERSION.start_with?('2.')
+        raise "Unexpected stderr. Handle stderr with assert_stderr\n\n#{$stderr.string}"
+      end
+    end
+
+    $stderr = $oldstderr
+
+    Timecop.return
+  end
+
   def with_env(key, value, &block)
     old_value = ENV.fetch(key, nil)
 
@@ -18,10 +44,16 @@ module CommonHelpers
   }.freeze
 
   def new_client(overrides = {})
+    $logs ||= StringIO.new
+
     config = overrides.delete(:config)
     project_env_id = overrides.delete(:project_env_id)
 
-    options = Prefab::Options.new(**DEFAULT_NEW_CLIENT_OPTIONS.merge(overrides))
+    options = Prefab::Options.new(
+      **DEFAULT_NEW_CLIENT_OPTIONS.merge(
+        overrides.merge(logdev: $logs)
+      )
+    )
 
     Prefab::Client.new(options).tap do |client|
       inject_config(client, config) if config
@@ -51,26 +83,31 @@ module CommonHelpers
   FakeResponse = Struct.new(:status, :body)
 
   def wait_for_post_requests(client, max_wait: 2, sleep_time: 0.01)
-    requests = []
+    # we use ivars to avoid re-mocking the post method on subsequent calls
+    client.instance_variable_set("@_requests", [])
 
-    client.define_singleton_method(:post) do |*params|
-      requests.push(params)
+    if !client.instance_variable_get("@_already_faked_post")
+      client.define_singleton_method(:post) do |*params|
+        @_requests.push(params)
 
-      FakeResponse.new(200, '')
+        FakeResponse.new(200, '')
+      end
     end
 
+    client.instance_variable_set("@_already_faked_post", true)
+
     yield
 
     # let the flush thread run
     wait_time = 0
-    while requests.empty?
+    while client.instance_variable_get("@_requests").empty?
       wait_time += sleep_time
       sleep sleep_time
 
       raise "Waited #{max_wait} seconds for the flush thread to run, but it never did" if wait_time > max_wait
     end
 
-    requests
+    client.instance_variable_get("@_requests")
   end
 
   def assert_summary(client, data)
@@ -102,4 +139,36 @@ module CommonHelpers
   def context(properties)
     Prefab::Context.new(properties)
   end
+
+  def assert_only_expected_logs
+    assert_equal "WARN  2023-08-09 15:18:12 -0400: cloud.prefab.client No success loading checkpoints\n", $logs.string
+    # mark nil to indicate we handled it
+    $logs = nil
+  end
+
+  def assert_logged(expected)
+    # we do a uniq here because logging can happen in a separate thread so the
+    # number of times a log might happen could be slightly variable.
+    assert_equal expected, $logs.string.split("\n").uniq
+    # mark nil to indicate we handled it
+    $logs = nil
+  end
+
+  def assert_stderr(expected)
+    assert ($stderr.string.split("\n").uniq & expected).size > 0
+
+    # Ruby 2.X has a lot of warnings about instance variables not being
+    # initialized so we don't try to assert on stderr for those versions.
+    # Instead we just stop after asserting that our expected errors are
+    # included in the output.
+    if RUBY_VERSION.start_with?('2.')
+      puts $stderr.string
+      return
+    end
+
+    assert_equal expected, $stderr.string.split("\n")
+
+    # restore since we've handled it
+    $stderr = $oldstderr
+  end
 end

data/test/support/mock_base_client.rb

@@ -34,10 +34,10 @@ class MockBaseClient
 
   def context_shape_aggregator; end
 
-  def evaluated_keys_aggregator; end
-
   def evaluation_summary_aggregator; end
 
+  def example_contexts_aggregator; end
+
   def config_value(key)
     @config_values[key]
   end

data/test/test_client.rb

@@ -24,34 +24,33 @@ class TestClient < Minitest::Test
     ]
   )
 
-  def setup
-    @client = new_client
-  end
-
   def test_get
     _, err = capture_io do
-      assert_equal 'default', @client.get('does.not.exist', 'default')
-      assert_equal 'test sample value', @client.get('sample')
-      assert_equal 123, @client.get('sample_int')
+      client = new_client
+      assert_equal 'default', client.get('does.not.exist', 'default')
+      assert_equal 'test sample value', client.get('sample')
+      assert_equal 123, client.get('sample_int')
     end
     assert_equal '', err
   end
 
   def test_get_with_default
+    client = new_client
     # A `false` value is not replaced with the default
-    assert_equal false, @client.get('false_value', 'red')
+    assert_equal false, client.get('false_value', 'red')
 
     # A falsy value is not replaced with the default
-    assert_equal 0, @client.get('zero_value', 'red')
+    assert_equal 0, client.get('zero_value', 'red')
 
     # A missing value returns the default
-    assert_equal 'buckets', @client.get('missing_value', 'buckets')
+    assert_equal 'buckets', client.get('missing_value', 'buckets')
   end
 
   def test_get_with_missing_default
+    client = new_client
     # it raises by default
     err = assert_raises(Prefab::Errors::MissingDefaultError) do
-      assert_nil @client.get('missing_value')
+      assert_nil client.get('missing_value')
     end
 
     assert_match(/No value found for key/, err.message)
@@ -63,47 +62,81 @@ class TestClient < Minitest::Test
   end
 
   def test_enabled
-    assert_equal false, @client.enabled?('does_not_exist')
-    assert_equal true, @client.enabled?('enabled_flag')
-    assert_equal false, @client.enabled?('disabled_flag')
-    assert_equal false, @client.enabled?('flag_with_a_value')
+    client = new_client
+    assert_equal false, client.enabled?('does_not_exist')
+    assert_equal true, client.enabled?('enabled_flag')
+    assert_equal false, client.enabled?('disabled_flag')
+    assert_equal false, client.enabled?('flag_with_a_value')
   end
 
   def test_ff_enabled_with_user_key_match
-    assert_equal_context_and_jit(false, :enabled?, 'user_key_match', { user: { key: 'jimmy' } })
-    assert_equal_context_and_jit(true, :enabled?, 'user_key_match', { user: { key: 'abc123' } })
-    assert_equal_context_and_jit(true, :enabled?, 'user_key_match', { user: { key: 'xyz987' } })
+    client = new_client
+
+    ctx = { user: { key: 'jimmy' } }
+    assert_equal false, client.enabled?('user_key_match', ctx)
+    assert_equal false, Prefab::Context.with_context(ctx) { client.enabled?('user_key_match') }
+
+    ctx = { user: { key: 'abc123' } }
+    assert_equal true, client.enabled?('user_key_match', ctx)
+    assert_equal true, Prefab::Context.with_context(ctx) { client.enabled?('user_key_match') }
+
+    ctx = { user: { key: 'xyz987' } }
+    assert_equal true, client.enabled?('user_key_match', ctx)
+    assert_equal true, Prefab::Context.with_context(ctx) { client.enabled?('user_key_match') }
   end
 
+  # NOTE: these are all `false` because we're doing a enabled? on a FF with string variants
+  # see test_ff_get_with_context for the raw value tests
   def test_ff_enabled_with_context
-    assert_equal_context_and_jit(false, :enabled?, 'just_my_domain', user: { domain: 'gmail.com' })
-    assert_equal_context_and_jit(false, :enabled?, 'just_my_domain', user: { domain: 'prefab.cloud' })
-    assert_equal_context_and_jit(false, :enabled?, 'just_my_domain', user: { domain: 'example.com' })
+    client = new_client
+
+    ctx = { user: { domain: 'gmail.com' } }
+    assert_equal false, client.enabled?('just_my_domain', ctx)
+    assert_equal false, Prefab::Context.with_context(ctx) { client.enabled?('just_my_domain') }
+
+    ctx = { user: { domain: 'prefab.cloud' } }
+    assert_equal false, client.enabled?('just_my_domain', ctx)
+    assert_equal false, Prefab::Context.with_context(ctx) { client.enabled?('just_my_domain') }
+
+    ctx = { user: { domain: 'example.com' } }
+    assert_equal false, client.enabled?('just_my_domain', ctx)
+    assert_equal false, Prefab::Context.with_context(ctx) { client.enabled?('just_my_domain') }
   end
 
   def test_ff_get_with_context
-    assert_nil @client.get('just_my_domain', 'abc123', user: { domain: 'gmail.com' })
-    assert_equal 'DEFAULT', @client.get('just_my_domain', 'abc123', { user: { domain: 'gmail.com' } }, 'DEFAULT')
+    client = new_client
+
+    ctx = { user: { domain: 'gmail.com' } }
+    assert_equal 'DEFAULT', client.get('just_my_domain', 'DEFAULT', ctx)
+    assert_equal 'DEFAULT', Prefab::Context.with_context(ctx) { client.get('just_my_domain', 'DEFAULT') }
 
-    assert_equal_context_and_jit('new-version', :get, 'just_my_domain', { user: { domain: 'prefab.cloud' } })
-    assert_equal_context_and_jit('new-version', :get, 'just_my_domain', { user: { domain: 'example.com' } })
+    ctx = { user: { domain: 'prefab.cloud' } }
+    assert_equal 'new-version', client.get('just_my_domain', 'DEFAULT', ctx)
+    assert_equal 'new-version', Prefab::Context.with_context(ctx) { client.get('just_my_domain', 'DEFAULT') }
+
+    ctx = { user: { domain: 'example.com' } }
+    assert_equal 'new-version', client.get('just_my_domain', 'DEFAULT', ctx)
+    assert_equal 'new-version', Prefab::Context.with_context(ctx) { client.get('just_my_domain', 'DEFAULT') }
   end
 
   def test_deprecated_no_dot_notation_ff_enabled_with_jit_context
+    client = new_client
     # with no lookup key
-    assert_equal false, @client.enabled?('deprecated_no_dot_notation', { domain: 'gmail.com' })
-    assert_equal true, @client.enabled?('deprecated_no_dot_notation', { domain: 'prefab.cloud' })
-    assert_equal true, @client.enabled?('deprecated_no_dot_notation', { domain: 'example.com' })
-
-    # with a lookup key
-    assert_equal false, @client.enabled?('deprecated_no_dot_notation', 'some-lookup-key', { domain: 'gmail.com' })
-    assert_equal true, @client.enabled?('deprecated_no_dot_notation', 'some-lookup-key', { domain: 'prefab.cloud' })
-    assert_equal true, @client.enabled?('deprecated_no_dot_notation', 'some-lookup-key', { domain: 'example.com' })
+    assert_equal false, client.enabled?('deprecated_no_dot_notation', { domain: 'gmail.com' })
+    assert_equal true, client.enabled?('deprecated_no_dot_notation', { domain: 'prefab.cloud' })
+    assert_equal true, client.enabled?('deprecated_no_dot_notation', { domain: 'example.com' })
+
+    assert_stderr [
+      "[DEPRECATION] Prefab contexts should be a hash with a key of the context name and a value of a hash.",
+      "[DEPRECATION] Prefab contexts should be a hash with a key of the context name and a value of a hash.",
+      "[DEPRECATION] Prefab contexts should be a hash with a key of the context name and a value of a hash."
+    ]
   end
 
   def test_getting_feature_flag_value
-    assert_equal false, @client.enabled?('flag_with_a_value')
-    assert_equal 'all-features', @client.get('flag_with_a_value')
+    client = new_client
+    assert_equal false, client.enabled?('flag_with_a_value')
+    assert_equal 'all-features', client.get('flag_with_a_value')
   end
 
   def test_initialization_with_an_options_object
@@ -134,43 +167,32 @@ class TestClient < Minitest::Test
     fake_api_key = '123-development-yourapikey-SDK'
 
     # it is nil by default
-    assert_nil Prefab::Client.new(api_key: fake_api_key).evaluation_summary_aggregator
+    assert_nil new_client(api_key: fake_api_key).evaluation_summary_aggregator
 
     # it is nil when local_only even if collect_max_evaluation_summaries is true
-    assert_nil Prefab::Client.new(prefab_datasources: LOCAL_ONLY,
-                                  collect_evaluation_summaries: true).evaluation_summary_aggregator
+    assert_nil new_client(prefab_datasources: LOCAL_ONLY,
+                                  collect_evaluation_summaries: true, ).evaluation_summary_aggregator
 
     # it is nil when collect_max_evaluation_summaries is false
-    assert_nil Prefab::Client.new(api_key: fake_api_key,
+    assert_nil new_client(api_key: fake_api_key,
+                                  prefab_datasources: :all,
                                   collect_evaluation_summaries: false).evaluation_summary_aggregator
 
     # it is not nil when collect_max_evaluation_summaries is true and the datasource is not local_only
     assert_equal Prefab::EvaluationSummaryAggregator,
-                 Prefab::Client.new(api_key: fake_api_key,
-                                    collect_evaluation_summaries: true).evaluation_summary_aggregator.class
+                 new_client(api_key: fake_api_key,
+                            prefab_datasources: :all,
+                            collect_evaluation_summaries: true).evaluation_summary_aggregator.class
+
+    assert_logged [
+      "WARN  2023-08-09 15:18:12 -0400: cloud.prefab.client No success loading checkpoints"
+    ]
   end
 
   def test_get_with_basic_value
-    config = PrefabProto::Config.new(
-      id: 123,
-      key: KEY,
-      config_type: PrefabProto::ConfigType::CONFIG,
-      rows: [
-        DEFAULT_ROW,
-        PrefabProto::ConfigRow.new(
-          project_env_id: PROJECT_ENV_ID,
-          values: [
-            PrefabProto::ConditionalValue.new(
-              criteria: [PrefabProto::Criterion.new(operator: PrefabProto::Criterion::CriterionOperator::ALWAYS_TRUE)],
-              value: DESIRED_VALUE_CONFIG
-            )
-          ]
-        )
-      ]
-    )
-
-    client = new_client(config: config, project_env_id: PROJECT_ENV_ID, collect_evaluation_summaries: :force,
-                        collect_example_contexts: :force)
+    config = basic_value_config
+    client = new_client(config: config, project_env_id: PROJECT_ENV_ID, collect_evaluation_summaries: true,
+                        context_upload_mode: :periodic_example, allow_telemetry_in_local_mode: true)
 
     assert_equal DESIRED_VALUE, client.get(config.key, IRRELEVANT, 'user' => { 'key' => 99 })
 
@@ -190,6 +212,31 @@ class TestClient < Minitest::Test
     assert_example_contexts client, [Prefab::Context.new({ user: { 'key' => 99 } })]
   end
 
+  def test_get_with_basic_value_with_context
+    config = basic_value_config
+    client = new_client(config: config, project_env_id: PROJECT_ENV_ID, collect_evaluation_summaries: true,
+                        context_upload_mode: :periodic_example, allow_telemetry_in_local_mode: true)
+
+    client.with_context('user' => { 'key' => 99 }) do
+      assert_equal DESIRED_VALUE, client.get(config.key)
+    end
+
+    assert_summary client, {
+      [KEY, :CONFIG] => {
+        {
+          config_id: config.id,
+          config_row_index: 1,
+          selected_value: DESIRED_VALUE_CONFIG,
+          conditional_value_index: 0,
+          weighted_value_index: nil,
+          selected_index: nil
+        } => 1
+      }
+    }
+
+    assert_example_contexts client, [Prefab::Context.new({ user: { 'key' => 99 } })]
+  end
+
   def test_get_with_weighted_values
     config = PrefabProto::Config.new(
       id: 123,
@@ -210,8 +257,8 @@ class TestClient < Minitest::Test
       ]
     )
 
-    client = new_client(config: config, project_env_id: PROJECT_ENV_ID, collect_evaluation_summaries: :force,
-                        collect_example_contexts: :force)
+    client = new_client(config: config, project_env_id: PROJECT_ENV_ID, collect_evaluation_summaries: true,
+                       context_upload_mode: :periodic_example, allow_telemetry_in_local_mode: true)
 
     2.times do
       assert_equal 'abc', client.get(config.key, IRRELEVANT, 'user' => { 'key' => '1' })
@@ -310,7 +357,7 @@ class TestClient < Minitest::Test
     )
 
     client = new_client(config: [config, segment_config], project_env_id: PROJECT_ENV_ID,
-                        collect_evaluation_summaries: :force, collect_example_contexts: :force)
+                        collect_evaluation_summaries: true, context_upload_mode: :periodic_example, allow_telemetry_in_local_mode: true)
 
     assert_equal DEFAULT_VALUE, client.get(config.key)
     assert_equal DEFAULT_VALUE,
@@ -356,7 +403,7 @@ class TestClient < Minitest::Test
     )
 
     client = new_client(config: config, project_env_id: PROJECT_ENV_ID,
-                        collect_evaluation_summaries: :force)
+                        collect_evaluation_summaries: true, allow_telemetry_in_local_mode: true)
 
     assert_equal :DEBUG, client.get(config.key, IRRELEVANT)
 
@@ -366,11 +413,23 @@ class TestClient < Minitest::Test
 
   private
 
-  def assert_equal_context_and_jit(expected, method, key, context)
-    assert_equal expected, @client.send(method, key, context)
-
-    Prefab::Context.with_context(context) do
-      assert_equal expected, @client.send(method, key)
-    end
+  def basic_value_config
+    PrefabProto::Config.new(
+      id: 123,
+      key: KEY,
+      config_type: PrefabProto::ConfigType::CONFIG,
+      rows: [
+        DEFAULT_ROW,
+        PrefabProto::ConfigRow.new(
+          project_env_id: PROJECT_ENV_ID,
+          values: [
+            PrefabProto::ConditionalValue.new(
+              criteria: [PrefabProto::Criterion.new(operator: PrefabProto::Criterion::CriterionOperator::ALWAYS_TRUE)],
+              value: DESIRED_VALUE_CONFIG
+            )
+          ]
+        )
+      ]
+    )
   end
 end

data/test/test_config_client.rb

@@ -4,6 +4,7 @@ require 'test_helper'
 
 class TestConfigClient < Minitest::Test
   def setup
+    super
     options = Prefab::Options.new(
       prefab_config_override_dir: 'none',
       prefab_config_classpath_dir: 'test',

data/test/test_config_loader.rb

@@ -4,6 +4,7 @@ require 'test_helper'
 
 class TestConfigLoader < Minitest::Test
   def setup
+    super
     options = Prefab::Options.new(
       prefab_config_override_dir: 'none',
       prefab_config_classpath_dir: 'test',

data/test/test_config_value_wrapper.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class TestConfigValueWrapper < Minitest::Test
+  def test_wrap_integer
+    result = Prefab::ConfigValueWrapper.wrap(42)
+    assert_instance_of PrefabProto::ConfigValue, result
+    assert_equal 42, result.int
+  end
+
+  def test_wrap_float
+    result = Prefab::ConfigValueWrapper.wrap(3.14)
+    assert_instance_of PrefabProto::ConfigValue, result
+    assert_equal 3.14, result.double
+  end
+
+  def test_wrap_boolean_true
+    result = Prefab::ConfigValueWrapper.wrap(true)
+    assert_instance_of PrefabProto::ConfigValue, result
+    assert_equal true, result.bool
+  end
+
+  def test_wrap_boolean_false
+    result = Prefab::ConfigValueWrapper.wrap(false)
+    assert_instance_of PrefabProto::ConfigValue, result
+    assert_equal false, result.bool
+  end
+
+  def test_wrap_array
+    result = Prefab::ConfigValueWrapper.wrap(['one', 'two', 'three'])
+    assert_instance_of PrefabProto::ConfigValue, result
+    assert_instance_of PrefabProto::StringList, result.string_list
+    assert_equal ['one', 'two', 'three'], result.string_list.values
+  end
+
+  def test_wrap_string
+    result = Prefab::ConfigValueWrapper.wrap('hello')
+    assert_instance_of PrefabProto::ConfigValue, result
+    assert_equal 'hello', result.string
+  end
+end

data/test/test_context.rb

@@ -6,6 +6,7 @@ class TestContext < Minitest::Test
   EXAMPLE_PROPERTIES = { user: { key: 'some-user-key', name: 'Ted' }, team: { key: 'abc', plan: 'pro' } }.freeze
 
   def setup
+    super
     Prefab::Context.current = nil
   end
 

data/test/test_context_shape_aggregator.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require 'test_helper'
+require 'timecop'
 
 class TestContextShapeAggregator < Minitest::Test
   DOB = Date.new
@@ -49,6 +50,8 @@ class TestContextShapeAggregator < Minitest::Test
 
     assert_equal [['user', 'name', 2], ['user', 'email', 2], ['user', 'age', 4], ['subscription', 'plan', 2], ['subscription', 'free', 5], ['user', 'dob', 2], ['device', 'name', 2], ['device', 'os', 2], ['device', 'version', 1]],
                  aggregator.data.to_a
+
+    assert_only_expected_logs
   end
 
   def test_prepare_data
@@ -82,6 +85,7 @@ class TestContextShapeAggregator < Minitest::Test
     }
 
     assert_equal [], aggregator.data.to_a
+    assert_only_expected_logs
   end
 
   def test_sync
@@ -117,6 +121,12 @@ class TestContextShapeAggregator < Minitest::Test
                                        ])
       ]
     ], requests
+
+
+    assert_logged [
+      "WARN  2023-08-09 15:18:12 -0400: cloud.prefab.client No success loading checkpoints",
+      "WARN  2023-08-09 15:18:12 -0400: cloud.prefab.client Couldn't Initialize In 0. Key some.key. Returning what we have"
+    ]
   end
 
   private
@@ -127,6 +137,7 @@ class TestContextShapeAggregator < Minitest::Test
       initialization_timeout_sec: 0,
       on_init_failure: Prefab::Options::ON_INITIALIZATION_FAILURE::RETURN,
       api_key: '123-development-yourapikey-SDK',
+      context_upload_mode: :shape_only
     }.merge(overrides))
   end
 

data/test/test_criteria_evaluator.rb

@@ -24,6 +24,7 @@ class TestCriteriaEvaluator < Minitest::Test
   )
 
   def setup
+    super
     @base_client = FakeBaseClient.new
   end
 

data/test/test_integration.rb

@@ -25,11 +25,13 @@ class TestIntegration < Minitest::Test
               assert_match(/#{it.expected[:message]}/, err.message)
             when :nil
               assert_nil it.test_client.send(it.func, *it.input)
-            when :feature_flag
-              flag, context = *it.input
-              assert_equal it.expected[:value], it.test_client.send(it.func, flag, context)
             when :simple_equality
-              assert_equal it.expected[:value], it.test_client.send(it.func, *it.input)
+              if it.func == :enabled?
+                flag, _default, context = *it.input
+                assert_equal it.expected[:value], it.test_client.send(it.func, flag, context)
+              else
+                assert_equal it.expected[:value], it.test_client.send(it.func, *it.input)
+              end
             when :log_level
               assert_equal it.expected[:value].to_sym, it.test_client.send(it.func, *it.input)
             else

data/test/test_log_path_aggregator.rb

@@ -8,7 +8,8 @@ class TestLogPathAggregator < Minitest::Test
   SLEEP_TIME = 0.01
 
   def test_push
-    aggregator = Prefab::LogPathAggregator.new(client: new_client, max_paths: 2, sync_interval: 1000)
+    client = new_client
+    aggregator = Prefab::LogPathAggregator.new(client: client, max_paths: 2, sync_interval: 1000)
 
     aggregator.push('test.test_log_path_aggregator.test_push.1', ::Logger::INFO)
     aggregator.push('test.test_log_path_aggregator.test_push.2', ::Logger::DEBUG)
@@ -18,6 +19,8 @@ class TestLogPathAggregator < Minitest::Test
     # we've reached the limit, so no more
     aggregator.push('test.test_log_path_aggregator.test_push.3', ::Logger::INFO)
     assert_equal 2, aggregator.data.size
+
+    assert_only_expected_logs
   end
 
   def test_sync
@@ -42,6 +45,11 @@ class TestLogPathAggregator < Minitest::Test
           namespace: 'this.is.a.namespace'
         )
       ]], requests
+
+      assert_logged [
+        'WARN  2023-08-09 15:18:12 -0400: cloud.prefab.client No success loading checkpoints',
+        'ERROR 2023-08-09 15:18:12 -0400: test.test_log_path_aggregator.test_sync here is a message'
+      ]
     end
   end
 

data/test/test_logger.rb

@@ -19,6 +19,7 @@ class TestLogger < Minitest::Test
   )
 
   def setup
+    super
     Prefab::LoggerClient.send(:public, :get_path)
     Prefab::LoggerClient.send(:public, :get_loc_path)
     Prefab::LoggerClient.send(:public, :level_of)

data/test/test_options.rb

@@ -36,17 +36,40 @@ class TestOptions < Minitest::Test
     assert_equal 0, options.collect_max_paths
   end
 
-  def test_collect_max_paths_with_collect_logs_false
+  def test_collect_max_paths_with_collect_logger_counts_false
     options = Prefab::Options.new(collect_max_paths: 100,
-                                  collect_logs: false)
+                                  collect_logger_counts: false)
     assert_equal 0, options.collect_max_paths
   end
 
   def test_collect_max_evaluation_summaries
-    assert_equal 0, Prefab::Options.new.collect_max_evaluation_summaries
-    assert_equal 100_000, Prefab::Options.new(collect_evaluation_summaries: true).collect_max_evaluation_summaries
+    assert_equal 100_000, Prefab::Options.new.collect_max_evaluation_summaries
+    assert_equal 0, Prefab::Options.new(collect_evaluation_summaries: false).collect_max_evaluation_summaries
     assert_equal 3,
-                 Prefab::Options.new(collect_evaluation_summaries: true,
-                                     collect_max_evaluation_summaries: 3).collect_max_evaluation_summaries
+                 Prefab::Options.new(collect_max_evaluation_summaries: 3).collect_max_evaluation_summaries
+  end
+
+  def test_context_upload_mode_periodic
+    options = Prefab::Options.new(context_upload_mode: :periodic_example, context_max_size: 100)
+    assert_equal 100, options.collect_max_example_contexts
+
+    options = Prefab::Options.new(context_upload_mode: :none)
+    assert_equal 0, options.collect_max_example_contexts
+  end
+
+  def test_context_upload_mode_shape_only
+    options = Prefab::Options.new(context_upload_mode: :shape_only, context_max_size: 100)
+    assert_equal 100, options.collect_max_shapes
+
+    options = Prefab::Options.new(context_upload_mode: :none)
+    assert_equal 0, options.collect_max_shapes
+  end
+
+  def test_context_upload_mode_none
+    options = Prefab::Options.new(context_upload_mode: :none)
+    assert_equal 0, options.collect_max_example_contexts
+
+    options = Prefab::Options.new(context_upload_mode: :none)
+    assert_equal 0, options.collect_max_shapes
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: prefab-cloud-ruby
 version: !ruby/object:Gem::Version
-  version: 0.24.6
+  version: 1.0.1
 platform: ruby
 authors:
 - Jeff Dwyer
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-07-31 00:00:00.000000000 Z
+date: 2023-08-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -211,7 +211,6 @@ files:
 - lib/prefab/errors/initialization_timeout_error.rb
 - lib/prefab/errors/invalid_api_key_error.rb
 - lib/prefab/errors/missing_default_error.rb
-- lib/prefab/evaluated_keys_aggregator.rb
 - lib/prefab/evaluation.rb
 - lib/prefab/evaluation_summary_aggregator.rb
 - lib/prefab/example_contexts_aggregator.rb
@@ -223,8 +222,6 @@ files:
 - lib/prefab/log_path_aggregator.rb
 - lib/prefab/logger_client.rb
 - lib/prefab/murmer3.rb
-- lib/prefab/noop_cache.rb
-- lib/prefab/noop_stats.rb
 - lib/prefab/options.rb
 - lib/prefab/periodic_sync.rb
 - lib/prefab/rate_limit_cache.rb
@@ -248,11 +245,11 @@ files:
 - test/test_config_loader.rb
 - test/test_config_resolver.rb
 - test/test_config_value_unwrapper.rb
+- test/test_config_value_wrapper.rb
 - test/test_context.rb
 - test/test_context_shape.rb
 - test/test_context_shape_aggregator.rb
 - test/test_criteria_evaluator.rb
-- test/test_evaluated_keys_aggregator.rb
 - test/test_evaluation_summary_aggregator.rb
 - test/test_example_contexts_aggregator.rb
 - test/test_exponential_backoff.rb

data/lib/prefab/evaluated_keys_aggregator.rb

@@ -1,41 +0,0 @@
-# frozen_string_literal: true
-
-require_relative 'periodic_sync'
-
-module Prefab
-  class EvaluatedKeysAggregator
-    include Prefab::PeriodicSync
-
-    attr_reader :data
-
-    def initialize(client:, max_keys:, sync_interval:)
-      @max_keys = max_keys
-      @client = client
-      @name = 'evaluated_keys_aggregator'
-
-      @data = Concurrent::Set.new
-
-      start_periodic_sync(sync_interval)
-    end
-
-    def push(key)
-      return if @data.size >= @max_keys
-
-      @data.add(key)
-    end
-
-    private
-
-    def flush(to_ship, _)
-      pool.post do
-        log_internal "Uploading evaluated keys for #{to_ship.size}"
-
-        keys = PrefabProto::EvaluatedKeys.new(keys: to_ship.to_a, namespace: @client.namespace)
-
-        result = @client.post('/api/v1/evaluated-keys', keys)
-
-        log_internal "Uploaded #{to_ship.size} keys: #{result.status}"
-      end
-    end
-  end
-end

data/lib/prefab/noop_cache.rb

@@ -1,15 +0,0 @@
-# frozen_string_literal: true
-
-module Prefab
-  class NoopCache
-    def fetch(_name, _opts)
-      yield
-    end
-
-    def write(name, value, opts = nil); end
-
-    def read(name); end
-
-    def delete(name); end
-  end
-end

data/lib/prefab/noop_stats.rb

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-module Prefab
-  class NoopStats
-    # receives increment("prefab.ratelimit.limitcheck", {:tags=>["policy_group:page_view", "pass:true"]})
-    def increment(name, opts = {}); end
-  end
-end

data/test/test_evaluated_keys_aggregator.rb

@@ -1,54 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_helper'
-
-class TestEvaluatedKeysAggregator < Minitest::Test
-  MAX_WAIT = 2
-  SLEEP_TIME = 0.01
-
-  def test_push
-    aggregator = Prefab::EvaluatedKeysAggregator.new(client: new_client, max_keys: 2, sync_interval: 1000)
-
-    aggregator.push('key.1')
-    aggregator.push('key.2')
-
-    assert_equal 2, aggregator.data.size
-
-    # we've reached the limit, so no more
-    aggregator.push('key.3')
-    assert_equal 2, aggregator.data.size
-  end
-
-  def test_sync
-    client = new_client(namespace: 'this.is.a.namespace')
-
-    client.get 'key.1', 'default', {}
-    client.get 'key.1', 'default', {}
-    client.get 'key.2', 'default', {}
-
-    requests = wait_for_post_requests(client) do
-      client.evaluated_keys_aggregator.send(:sync)
-    end
-
-    assert_equal [[
-      '/api/v1/evaluated-keys',
-      PrefabProto::EvaluatedKeys.new(
-        keys: ['key.1', 'key.2'],
-        namespace: 'this.is.a.namespace'
-      )
-    ]], requests
-  end
-
-  private
-
-  def new_client(overrides = {})
-    super(**{
-      prefab_datasources: Prefab::Options::DATASOURCES::ALL,
-      initialization_timeout_sec: 0,
-      on_init_failure: Prefab::Options::ON_INITIALIZATION_FAILURE::RETURN,
-      api_key: '123-development-yourapikey-SDK',
-      collect_sync_interval: 1000, # we'll trigger sync manually in our test
-      collect_keys: true
-    }.merge(overrides))
-  end
-end