feat: misc

Author: Zaimwa9

lib/flagsmith/engine/segments/evaluator.rb

@@ -9,10 +9,10 @@ module Engine
     module Segments
       # Evaluator methods
       module Evaluator
-        extend self
         include Flagsmith::Engine::Segments::Constants
         include Flagsmith::Engine::Utils::HashFunc
 
+        module_function
         # Context-based segment evaluation (new approach)
         # Returns all segments that the identity belongs to based on segment rules evaluation
         #
@@ -111,7 +111,7 @@ def traits_match_segment_rule_from_context(rule, segment_key, context)
         # @param context [Hash] The evaluation context
         # @return [Boolean] True if conditions match according to rule type
         def evaluate_conditions_from_context(rule, segment_key, context)
-          return true if rule[:conditions].nil? || rule[:conditions].empty?
+          return true unless rule[:conditions]&.any?
 
           condition_results = rule[:conditions].map do |condition|
             traits_match_segment_condition_from_context(condition, segment_key, context)

lib/flagsmith/engine/segments/models.rb

@@ -58,7 +58,7 @@ def match_trait_value?(trait_value)
           if @value.is_a?(String) && @value.match?(/:semver$/)
             begin
               trait_value = Semantic::Version.new(trait_value.to_s.gsub(/:semver$/, ''))
-            rescue StandardError
+            rescue ArgumentError, Semantic::Version::ValidationFailed => _e
               return false
             end
           end