chore: update bucketing lib and use correct eval reasons

Author: jsalaber

lib/devcycle-ruby-server-sdk/api/client.rb

@@ -195,16 +195,17 @@ def variable(user, key, default, opts = {})
           value = default
           type = determine_variable_type(default)
           defaulted = true
-          eval = { reason: DevCycle::EVAL_REASONS::DEFAULT, details: DevCycle::DEFAULT_REASON_DETAILS::USER_NOT_TARGETED }
+          eval = { reason: DevCycle::DEFAULT_REASONS::USER_NOT_TARGETED, details: DevCycle::DEFAULT_REASON_DETAILS::USER_NOT_TARGETED }
           if local_bucketing_initialized? && @local_bucketing.has_config
             type_code = variable_type_code_from_type(type)
             variable_pb = variable_for_user_pb(user, key, type_code)
             unless variable_pb.nil?
               value = get_variable_value(variable_pb)
               defaulted = false
-              eval = get_eval_reason(variable_pb)
             end
+            eval = get_eval_reason(variable_pb)
           else
+            eval = { reason: DevCycle::DEFAULT_REASONS::DEFAULT, details: DevCycle::DEFAULT_REASON_DETAILS::MISSING_CONFIG }
             @logger.warn("Local bucketing not initialized, returning default value for variable #{key}")
             variable_event = Event.new({ type: DevCycle::EventTypes[:agg_variable_defaulted], target: key })
             bucketed_config = BucketedUserConfig.new({}, {}, {}, {}, {}, {}, [])
@@ -578,11 +579,14 @@ def get_variable_value(variable_pb)
     end
 
     def get_eval_reason(variable_pb)
-      if variable_pb.eval.nil?
-        { reason: DevCycle::DEFAULT_REASONS::USER_NOT_TARGETED, details: DevCycle::DEFAULT_REASON_DETAILS::USER_NOT_TARGETED }
+      if variable_pb.nil?
+        { reason: DevCycle::DEFAULT_REASONS::DEFAULT, details: DevCycle::DEFAULT_REASON_DETAILS::USER_NOT_TARGETED}
       else
-        puts ("variable_pb.eval bruh: #{variable_pb.eval}")
-        { reason: variable_pb.eval.reason, details: variable_pb.eval.details }
+        if variable_pb.eval.nil?
+          { reason: DevCycle::DEFAULT_REASONS::USER_NOT_TARGETED, details: DevCycle::DEFAULT_REASON_DETAILS::USER_NOT_TARGETED }
+        else
+          { reason: variable_pb.eval.reason, details: variable_pb.eval.details, target_id: variable_pb.eval.target_id }
+        end
       end
     end
 

lib/devcycle-ruby-server-sdk/eval_reasons.rb

@@ -10,6 +10,7 @@ module DEFAULT_REASONS
     USER_NOT_TARGETED = 'USER_NOT_TARGETED'
     INVALID_VARIABLE_TYPE = 'INVALID_VARIABLE_TYPE'
     UNKNOWN = 'UNKNOWN'
+    DEFAULT = 'DEFAULT'
   end
 
   # Evaluation reasons for successful evaluations
@@ -53,7 +54,7 @@ module EVAL_REASON_DETAILS
     UNKNOWN = 'Unknown'
   end
 
-  # Default reason details2
+  # Default reason details
   module DEFAULT_REASON_DETAILS
     MISSING_CONFIG = 'Missing Config'
     MISSING_VARIABLE = 'Missing Variable'

lib/devcycle-ruby-server-sdk/localbucketing/bucketing-lib.release.wasm

@@ -2,4 +2,4 @@
 BUCKETING_LIB_VERSION="1.40.0"
 WAT_DOWNLOAD=0
 rm bucketing-lib.release.wasm
-wget "https://unpkg.com/@devcycle/bucketing-assembly-script@$BUCKETING_LIB_VERSION/build/bucketing-lib.release.wasm"
+wget "https://unpkg.com/@devcycle/bucketing-assembly-script@$BUCKETING_LIB_VERSION/build/bucketing-lib.release.wasm"

lib/devcycle-ruby-server-sdk/localbucketing/update_wasm.sh

@@ -34,7 +34,7 @@ class Feature
     attr_accessor :variationName
 
     # Evaluation reasoning
-    attr_accessor :eval
+    attr_accessor :eval_reason
 
     class EnumAttributeValidator
       attr_reader :datatype
@@ -67,7 +67,7 @@ def self.attribute_map
         :'_variation' => :'_variation',
         :'variationKey' => :'variationKey',
         :'variationName' => :'variationName',
-        :'eval' => :'eval'
+        :'eval_reason' => :'evalReason'
       }
     end
 
@@ -133,6 +133,10 @@ def initialize(attributes = {})
       if attributes.key?(:'variationName')
         self.variationName = attributes[:'variationName']
       end
+
+      if attributes.key?(:'eval_reason')
+        self.eval_reason = attributes[:'eval_reason']
+      end
     end
 
     # Show invalid properties with the reasons. Usually used together with valid?