Update decision service

Author: esrakartalOpt

lib/optimizely/decision_service.rb

@@ -312,14 +312,45 @@ def get_variations_for_feature_list(project_config, feature_flags, user_context,
 
       decisions = []
       feature_flags.each do |feature_flag|
-        # check if the feature is being experiment on and whether the user is bucketed into the experiment
-        decision_result = get_variation_for_feature_experiment(project_config, feature_flag, user_context, user_profile_tracker, decide_options)
-        # Only process rollout if no experiment decision was found and no error
-        if decision_result.decision.nil? && !decision_result.error
-          decision_result_rollout = get_variation_for_feature_rollout(project_config, feature_flag, user_context) unless decision_result.decision
-          decision_result.decision = decision_result_rollout.decision
-          decision_result.reasons.push(*decision_result_rollout.reasons)
+        # Check holdouts first
+        holdouts = project_config.get_holdouts_for_flag(feature_flag['id'])
+        decision_result = nil
+
+        if holdouts && !holdouts.empty?
+          # Check each holdout
+          holdouts.each do |holdout|
+            holdout_decision = get_variation_for_holdout(holdout, user_context, project_config)
+
+            if holdout_decision.decision
+              # User is bucketed into a holdout
+              decision_result = holdout_decision
+              break
+            else
+              # User didn't bucket into this holdout, try next or continue to experiments
+              decision_result = holdout_decision unless decision_result
+            end
+          end
         end
+
+        # If no holdout decision, check if the feature is being experiment on and whether the user is bucketed into the experiment
+        if decision_result.nil? || decision_result.decision.nil?
+          decision_result_exp = get_variation_for_feature_experiment(project_config, feature_flag, user_context, user_profile_tracker, decide_options)
+          if decision_result
+            decision_result.reasons.push(*decision_result_exp.reasons)
+            decision_result.decision = decision_result_exp.decision if decision_result_exp.decision
+            decision_result.error = decision_result_exp.error if decision_result_exp.error
+          else
+            decision_result = decision_result_exp
+          end
+
+          # Only process rollout if no experiment decision was found and no error
+          if decision_result.decision.nil? && !decision_result.error
+            decision_result_rollout = get_variation_for_feature_rollout(project_config, feature_flag, user_context) unless decision_result.decision
+            decision_result.decision = decision_result_rollout.decision
+            decision_result.reasons.push(*decision_result_rollout.reasons)
+          end
+        end
+
         decisions << decision_result
       end
       user_profile_tracker&.save_user_profile