growthbook 1.2.1 → 1.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cba2938ec675b1626d643fb7493223a54d0faf1ffc37c2bacb5a06637c0a75cd
-  data.tar.gz: bce59918f346b53182450ec2fdd92776767954712334bc5b7c0bd48d952c05ce
+  metadata.gz: 71807a4bbc1fb91e816a74e32d7d6c9e7d1dbed782b19efd3592fd5f5a944801
+  data.tar.gz: 03fa6cf71822d7f239afda323e9a28476721d64c6234b199103869207a2cd060
 SHA512:
-  metadata.gz: 0c4ac9f40b23c8719b9cb585d92f089144df5441c787321fd0083ae72ff954f5f8027615cc3d226dcdb7c2bfdc3dc5911ef1abd472904c026a3e24c1dd81230c
-  data.tar.gz: 735bfc1c4b82432a4a9b0664282062f43ab644309933afcddb43c9dc53cd32a278d1cf14221ad9b298a0bbdb0e2a12a8e77c4aa814d8ba3719e7840c62a4d09a
+  metadata.gz: af78de253f4b3c2ecfbcdde348237923b3ecdd0dbe547be7d0f177bc7dff2a6299d6f6ac7786c4e4e96c2c031ff46598897ef79214efeaba837cce4db2e3739b
+  data.tar.gz: fd4b68b94b0448dd1cd88b8aec1b96c1d76755133917c7a915a03b18e6a02cd2b2bc5d9f6735c51742d4af5a8707cec13f61ed571fd013059d45b207688a83f8

data/lib/growthbook/conditions.rb

@@ -104,20 +104,68 @@ module Growthbook
       false
     end
 
+    def self.compare(val1, val2)
+      if val1.is_a?(Numeric) || val2.is_a?(Numeric)
+        val1 = val1.is_a?(Numeric) ? val1 : val1.to_f
+        val2 = val2.is_a?(Numeric) ? val2 : val2.to_f
+      end
+
+      return 1 if val1 > val2
+      return -1 if val1 < val2
+
+      0
+    end
+
     def self.eval_operator_condition(operator, attribute_value, condition_value)
       case operator
+      when '$veq'
+        padded_version_string(attribute_value) == padded_version_string(condition_value)
+      when '$vne'
+        padded_version_string(attribute_value) != padded_version_string(condition_value)
+      when '$vgt'
+        padded_version_string(attribute_value) > padded_version_string(condition_value)
+      when '$vgte'
+        padded_version_string(attribute_value) >= padded_version_string(condition_value)
+      when '$vlt'
+        padded_version_string(attribute_value) < padded_version_string(condition_value)
+      when '$vlte'
+        padded_version_string(attribute_value) <= padded_version_string(condition_value)
       when '$eq'
-        attribute_value == condition_value
+        begin
+          compare(attribute_value, condition_value).zero?
+        rescue StandardError
+          false
+        end
       when '$ne'
-        attribute_value != condition_value
+        begin
+          compare(attribute_value, condition_value) != 0
+        rescue StandardError
+          false
+        end
       when '$lt'
-        attribute_value < condition_value
+        begin
+          compare(attribute_value, condition_value).negative?
+        rescue StandardError
+          false
+        end
       when '$lte'
-        attribute_value <= condition_value
+        begin
+          compare(attribute_value, condition_value) <= 0
+        rescue StandardError
+          false
+        end
       when '$gt'
-        attribute_value > condition_value
+        begin
+          compare(attribute_value, condition_value).positive?
+        rescue StandardError
+          false
+        end
       when '$gte'
-        attribute_value >= condition_value
+        begin
+          compare(attribute_value, condition_value) >= 0
+        rescue StandardError
+          false
+        end
       when '$regex'
         silence_warnings do
           re = Regexp.new(condition_value)
@@ -126,9 +174,13 @@ module Growthbook
           false
         end
       when '$in'
-        condition_value.include? attribute_value
+        return false unless condition_value.is_a?(Array)
+
+        in?(attribute_value, condition_value)
       when '$nin'
-        !(condition_value.include? attribute_value)
+        return false unless condition_value.is_a?(Array)
+
+        !in?(attribute_value, condition_value)
       when '$elemMatch'
         elem_match(condition_value, attribute_value)
       when '$size'
@@ -162,6 +214,29 @@ module Growthbook
       end
     end
 
+    def self.padded_version_string(input)
+      # Remove build info and leading `v` if any
+      # Split version into parts (both core version numbers and pre-release tags)
+      # "v1.2.3-rc.1+build123" -> ["1","2","3","rc","1"]
+      parts = input.gsub(/(^v|\+.*$)/, '').split(/[-.]/)
+
+      # If it's SemVer without a pre-release, add `~` to the end
+      # ["1","0","0"] -> ["1","0","0","~"]
+      # "~" is the largest ASCII character, so this will make "1.0.0" greater than "1.0.0-beta" for example
+      parts << '~' if parts.length == 3
+
+      # Left pad each numeric part with spaces so string comparisons will work ("9">"10", but " 9"<"10")
+      parts.map do |part|
+        /^[0-9]+$/.match?(part) ? part.rjust(5, ' ') : part
+      end.join('-')
+    end
+
+    def self.in?(actual, expected)
+      return expected.include?(actual) unless actual.is_a?(Array)
+
+      (actual & expected).any?
+    end
+
     # Sets $VERBOSE for the duration of the block and back to its original
     # value afterwards. Used for testing invalid regexes.
     def self.silence_warnings

data/lib/growthbook/context.rb

@@ -329,7 +329,7 @@ module Growthbook
     def included_in_rollout?(seed:, hash_attribute:, hash_version:, range:, coverage:)
       return true if range.nil? && coverage.nil?
 
-      hash_value = get_attribute(hash_attribute)
+      hash_value = get_attribute(hash_attribute).to_s
 
       return false if hash_value.empty?
 
@@ -344,7 +344,7 @@ module Growthbook
 
     def filtered_out?(filters)
       filters.any? do |filter|
-        hash_value = get_attribute(filter['attribute'] || 'id')
+        hash_value = get_attribute(filter['attribute'] || 'id').to_s
 
         if hash_value.empty?
           false

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: growthbook
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.2.2
 platform: ruby
 authors:
 - GrowthBook
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-07-13 00:00:00.000000000 Z
+date: 2023-10-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec