featurevisor 0.3.0 → 1.1.0

data/lib/featurevisor/child_instance.rb

@@ -3,8 +3,6 @@
 module Featurevisor
   # Child instance class for managing child contexts and sticky features
   class ChildInstance
-    attr_reader :parent, :context, :sticky, :emitter
-
     # Initialize a new child instance
     # @param options [Hash] Child instance options
     # @option options [Instance] :parent Parent instance
@@ -94,8 +92,8 @@ module Featurevisor
           **context
         },
         {
-          sticky: @sticky,
-          **options
+          **options,
+          __featurevisor_child_sticky: @sticky
         }
       )
     end
@@ -113,8 +111,8 @@ module Featurevisor
           **context
         },
         {
-          sticky: @sticky,
-          **options
+          **options,
+          __featurevisor_child_sticky: @sticky
         }
       )
     end
@@ -134,8 +132,8 @@ module Featurevisor
           **context
         },
         {
-          sticky: @sticky,
-          **options
+          **options,
+          __featurevisor_child_sticky: @sticky
         }
       )
     end
@@ -155,8 +153,8 @@ module Featurevisor
           **context
         },
         {
-          sticky: @sticky,
-          **options
+          **options,
+          __featurevisor_child_sticky: @sticky
         }
       )
     end
@@ -176,8 +174,8 @@ module Featurevisor
           **context
         },
         {
-          sticky: @sticky,
-          **options
+          **options,
+          __featurevisor_child_sticky: @sticky
         }
       )
     end
@@ -197,8 +195,8 @@ module Featurevisor
           **context
         },
         {
-          sticky: @sticky,
-          **options
+          **options,
+          __featurevisor_child_sticky: @sticky
         }
       )
     end
@@ -218,8 +216,8 @@ module Featurevisor
           **context
         },
         {
-          sticky: @sticky,
-          **options
+          **options,
+          __featurevisor_child_sticky: @sticky
         }
       )
     end
@@ -239,8 +237,8 @@ module Featurevisor
           **context
         },
         {
-          sticky: @sticky,
-          **options
+          **options,
+          __featurevisor_child_sticky: @sticky
         }
       )
     end
@@ -260,8 +258,8 @@ module Featurevisor
           **context
         },
         {
-          sticky: @sticky,
-          **options
+          **options,
+          __featurevisor_child_sticky: @sticky
         }
       )
     end
@@ -281,8 +279,8 @@ module Featurevisor
           **context
         },
         {
-          sticky: @sticky,
-          **options
+          **options,
+          __featurevisor_child_sticky: @sticky
         }
       )
     end
@@ -300,8 +298,8 @@ module Featurevisor
         },
         feature_keys,
         {
-          sticky: @sticky,
-          **options
+          **options,
+          __featurevisor_child_sticky: @sticky
         }
       )
     end

data/lib/featurevisor/datafile_reader.rb

@@ -5,7 +5,7 @@ require "json"
 module Featurevisor
   # DatafileReader class for reading and processing Featurevisor datafiles
   class DatafileReader
-    attr_reader :schema_version, :revision, :segments, :features, :logger, :regex_cache
+    attr_reader :schema_version, :revision, :featurevisor_version, :segments, :features, :logger, :regex_cache
 
     # Initialize a new DatafileReader
     # @param options [Hash] Options hash containing datafile and logger
@@ -17,6 +17,7 @@ module Featurevisor
 
       @schema_version = datafile[:schemaVersion]
       @revision = datafile[:revision]
+      @featurevisor_version = datafile[:featurevisorVersion]
       @segments = (datafile[:segments] || {}).transform_keys(&:to_sym)
       @features = (datafile[:features] || {}).transform_keys(&:to_sym)
 
@@ -65,6 +66,16 @@ module Featurevisor
       @schema_version
     end
 
+    def get_datafile
+      {
+        schemaVersion: @schema_version,
+        revision: @revision,
+        featurevisorVersion: @featurevisor_version,
+        segments: @segments,
+        features: @features
+      }.compact
+    end
+
     # Get a segment by key
     # @param segment_key [String] Segment key to retrieve
     # @return [Hash, nil] Segment data or nil if not found
@@ -172,15 +183,11 @@ module Featurevisor
       end
 
       if conditions.is_a?(Hash) && conditions[:not] && conditions[:not].is_a?(Array)
-        return conditions[:not].all? do
-          all_conditions_are_matched({ and: conditions[:not] }, context) == false
-        end
+        return !all_conditions_are_matched({ and: conditions[:not] }, context)
       end
 
       if conditions.is_a?(Hash) && conditions["not"] && conditions["not"].is_a?(Array)
-        return conditions["not"].all? do
-          all_conditions_are_matched({ "and" => conditions["not"] }, context) == false
-        end
+        return !all_conditions_are_matched({ "and" => conditions["not"] }, context)
       end
 
 
@@ -237,15 +244,11 @@ module Featurevisor
         end
 
         if group_segments[:not] && group_segments[:not].is_a?(Array)
-          return group_segments[:not].all? do
-            all_segments_are_matched({ and: group_segments[:not] }, context) == false
-          end
+          return !all_segments_are_matched({ and: group_segments[:not] }, context)
         end
 
         if group_segments["not"] && group_segments["not"].is_a?(Array)
-          return group_segments["not"].all? do
-            all_segments_are_matched({ "and" => group_segments["not"] }, context) == false
-          end
+          return !all_segments_are_matched({ "and" => group_segments["not"] }, context)
         end
 
 
@@ -351,4 +354,6 @@ module Featurevisor
       segments
     end
   end
+
+  private_constant :DatafileReader
 end

data/lib/featurevisor/emitter.rb

@@ -2,7 +2,7 @@
 
 module Featurevisor
   # Event names for the emitter
-  EVENT_NAMES = %w[datafile_set context_set sticky_set].freeze
+  EVENT_NAMES = %w[datafile_set context_set sticky_set error].freeze
 
   # Event emitter class for handling event subscriptions and triggers
   class Emitter
@@ -42,7 +42,7 @@ module Featurevisor
 
       return unless listeners
 
-      listeners.each do |listener|
+      listeners.dup.each do |listener|
         begin
           listener.call(details)
         rescue => err

data/lib/featurevisor/evaluate.rb

@@ -36,20 +36,17 @@ module Featurevisor
   # Evaluation module for feature flag evaluation
   module Evaluate
 
-    # Evaluate with hooks
+    # Evaluate with modules
     # @param options [Hash] Evaluation options
     # @return [Hash] Evaluation result
-    def self.evaluate_with_hooks(options)
+    def self.evaluate_with_modules(options)
       begin
-        hooks_manager = options[:hooks_manager]
-        hooks = hooks_manager.get_all
+        modules_manager = options[:modules_manager]
 
-        # Run before hooks
+        # Run before modules
         result_options = options
-        hooks.each do |hook|
-          if hook.respond_to?(:call_before)
-            result_options = hook.call_before(result_options)
-          end
+        if modules_manager
+          result_options = modules_manager.run_before_modules(result_options)
         end
 
         # Evaluate
@@ -69,11 +66,9 @@ module Featurevisor
           evaluation[:variable_value] = options[:default_variable_value]
         end
 
-        # Run after hooks
-        hooks.each do |hook|
-          if hook.respond_to?(:call_after)
-            evaluation = hook.call_after(evaluation, result_options)
-          end
+        # Run after modules
+        if modules_manager
+          evaluation = modules_manager.run_after_modules(evaluation, result_options)
         end
 
         evaluation
@@ -108,9 +103,7 @@ module Featurevisor
       logger = options[:logger]
       datafile_reader = options[:datafile_reader]
       sticky = options[:sticky]
-      hooks_manager = options[:hooks_manager]
-
-      hooks = hooks_manager.get_all
+      modules_manager = options[:modules_manager]
       evaluation = nil
 
       begin
@@ -422,24 +415,24 @@ module Featurevisor
           logger: logger
         })
 
-        # Run bucket key hooks
-        bucket_key = hooks_manager.run_bucket_key_hooks({
+        # Run bucket key modules
+        bucket_key = modules_manager.run_bucket_key_modules({
           feature_key: feature_key,
           context: context,
           bucket_by: feature[:bucketBy],
           bucket_key: bucket_key
-        })
+        }) if modules_manager
 
         # bucketValue
         bucket_value = Featurevisor::Bucketer.get_bucketed_number(bucket_key)
 
-        # Run bucket value hooks
-        bucket_value = hooks_manager.run_bucket_value_hooks({
+        # Run bucket value modules
+        bucket_value = modules_manager.run_bucket_value_modules({
           feature_key: feature_key,
           bucket_key: bucket_key,
           context: context,
           bucket_value: bucket_value
-        })
+        }) if modules_manager
 
         matched_traffic = nil
         matched_allocation = nil

data/lib/featurevisor/events.rb

@@ -24,7 +24,7 @@ module Featurevisor
     # @param previous_reader [DatafileReader] Previous datafile reader
     # @param new_reader [DatafileReader] New datafile reader
     # @return [Hash] Event parameters
-    def self.get_params_for_datafile_set_event(previous_reader, new_reader)
+    def self.get_params_for_datafile_set_event(previous_reader, new_reader, replace = false)
       previous_revision = previous_reader.get_revision
       previous_feature_keys = previous_reader.get_feature_keys
 
@@ -67,9 +67,10 @@ module Featurevisor
 
       {
         revision: new_revision,
-        previous_revision: previous_revision,
-        revision_changed: previous_revision != new_revision,
-        features: all_affected_features
+        previousRevision: previous_revision,
+        revisionChanged: previous_revision != new_revision,
+        features: all_affected_features,
+        replaced: replace
       }
     end
   end