posthog-ruby 3.29.0 → 3.29.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9a3e8eecf2cef5e82e5cc8ba7f7ddefc8c9f372b86039c9e834ad6308b083a3e
-  data.tar.gz: 7ea54f05eaf248de3f5106729d8f5d02c919b49c1e32db7030704288c8e6c8b2
+  metadata.gz: dc3cd53140f36b4f77fb11d60a903858d5053362e7c8fd3d8d4bd55472ed1c4a
+  data.tar.gz: f1213ce104a582ae6b9e82315538e31b77fc85dbe7336cdf0e41cbb169b0b47e
 SHA512:
-  metadata.gz: 6a72a75666f8c77ab2aa9abfb863404ca9076de5f205da215f7fc24271999c75e144d8ded893d2e389218c0068383dfdcdbcbead6ce35e643947adc476a76602
-  data.tar.gz: 6be683bbd6b6831913a146ac1cea6ac348c5dc929af6443131459febedf1e1890c32fb7b7482ba8f42402d7cdd1c9ebab7934b720be915bc432d376caa3389b7
+  metadata.gz: 489f3bd76d70c2034c62c02e217d6e09bd16e6600d9de7b757ae5bda1a0a24d7dafad88a735ddf13ab39fa18cffdab59fcb20e2d080689ff373238fce11f9925
+  data.tar.gz: ab7bb81da61fc82e4058355c95bc1d9b97018f0cbcfd5b476f380a058a39e9a0ce6ae3b8ce1fd11aa18acb4d4a1c4fb0cec3546f4729ee5a56dcf4e959e5ee3a

data/lib/posthog/client.rb

@@ -671,7 +671,7 @@ module PostHog
         )
       end
 
-      person_properties, group_properties = add_local_person_and_group_properties(
+      groups, person_properties, group_properties = add_local_person_and_group_properties(
         groups, person_properties, group_properties
       )
 
@@ -806,8 +806,9 @@ module PostHog
     )
       return {} if @disabled
 
-      person_properties, group_properties = add_local_person_and_group_properties(groups, person_properties,
-                                                                                  group_properties)
+      groups, person_properties, group_properties = add_local_person_and_group_properties(
+        groups, person_properties, group_properties
+      )
       @feature_flags_poller.get_all_flags(distinct_id, groups, person_properties, group_properties,
                                           only_evaluate_locally)
     end
@@ -846,8 +847,9 @@ module PostHog
       return nil if @disabled
 
       key = key.to_s
-      person_properties, group_properties = add_local_person_and_group_properties(groups, person_properties,
-                                                                                  group_properties)
+      groups, person_properties, group_properties = add_local_person_and_group_properties(
+        groups, person_properties, group_properties
+      )
       @feature_flags_poller.get_feature_flag_payload(key, distinct_id, match_value, groups, person_properties,
                                                      group_properties, only_evaluate_locally)
     end
@@ -873,7 +875,7 @@ module PostHog
     )
       return { featureFlags: {}, featureFlagPayloads: {} } if @disabled
 
-      person_properties, group_properties = add_local_person_and_group_properties(
+      groups, person_properties, group_properties = add_local_person_and_group_properties(
         groups, person_properties, group_properties
       )
       response = @feature_flags_poller.get_all_flags_and_payloads(
@@ -1124,7 +1126,7 @@ module PostHog
       return nil if @disabled
 
       key = key.to_s
-      person_properties, group_properties = add_local_person_and_group_properties(
+      groups, person_properties, group_properties = add_local_person_and_group_properties(
         groups, person_properties, group_properties
       )
       feature_flag_response, flag_was_locally_evaluated, request_id, evaluated_at, feature_flag_error, payload,
@@ -1290,17 +1292,9 @@ module PostHog
     end
 
     def add_local_person_and_group_properties(groups, person_properties, group_properties)
-      groups ||= {}
-      person_properties ||= {}
-      group_properties ||= {}
-
-      symbolize_keys! groups
-      symbolize_keys! person_properties
-      symbolize_keys! group_properties
-
-      group_properties.each_value do |value|
-        symbolize_keys! value
-      end
+      groups, person_properties, group_properties = normalize_flag_evaluation_inputs(
+        groups, person_properties, group_properties
+      )
 
       all_group_properties = {}
       groups&.each do |group_name, group_key|
@@ -1309,7 +1303,7 @@ module PostHog
         }.merge((group_properties && group_properties[group_name]) || {})
       end
 
-      [person_properties, all_group_properties]
+      [groups, person_properties, all_group_properties]
     end
   end
 end

data/lib/posthog/feature_flags.rb

@@ -150,6 +150,9 @@ module PostHog
       only_evaluate_locally = false,
       raise_on_error = false
     )
+      groups, person_properties, group_properties = normalize_flag_evaluation_inputs(
+        groups, person_properties, group_properties
+      )
       # TODO: Convert to options hash for easier argument passing
       flags_data = get_all_flags_and_payloads(
         distinct_id,
@@ -246,14 +249,6 @@ module PostHog
       load_feature_flags
       snapshot ||= _evaluation_snapshot
 
-      symbolize_keys! groups
-      symbolize_keys! person_properties
-      symbolize_keys! group_properties
-
-      group_properties.each_value do |value|
-        symbolize_keys!(value)
-      end
-
       response = nil
       payload = nil
       feature_flag = snapshot[:flags_by_key]&.[](key)
@@ -734,9 +729,6 @@ module PostHog
     def self.match_property(property, property_values, cohort_properties = {}, property_matching_version: 1)
       # only looks for matches where key exists in property_values
 
-      PostHog::Utils.symbolize_keys! property
-      PostHog::Utils.symbolize_keys! property_values
-
       # Handle cohort properties
       if extract_value(property, :type) == 'cohort'
         return match_cohort(property, property_values, cohort_properties,
@@ -753,25 +745,23 @@ module PostHog
     end
 
     def self.match_property_value(property, property_values, property_matching_version)
-      key = property[:key].to_sym
-      value = property[:value]
-      operator = property[:operator] || 'exact'
-
-      if !property_values.key?(key)
-        raise InconclusiveMatchError.new("Property #{key} not found in property_values",
-                                         reason: UnresolvedFlagReason::MISSING_CONTEXT)
-      elsif operator == 'is_not_set'
-        return false
+      key = (property[:key] || property['key']).to_sym
+      value = property.key?(:value) ? property[:value] : property['value']
+      operator = property[:operator] || property['operator'] || 'exact'
+      override_value = property_values.fetch(key) do
+        property_values.fetch(key.to_s) do
+          raise InconclusiveMatchError.new("Property #{key} not found in property_values",
+                                           reason: UnresolvedFlagReason::MISSING_CONTEXT)
+        end
       end
-
-      override_value = property_values[key]
+      return false if operator == 'is_not_set'
 
       case operator
       when 'exact', 'is_not'
         matches = exact_property_match?(value, override_value, property_matching_version)
         operator == 'exact' ? matches : !matches
       when 'is_set'
-        property_values.key?(key)
+        true
       when 'icontains'
         string_operand(override_value).downcase(:ascii).include?(value.to_s.downcase(:ascii))
       when 'not_icontains'
@@ -847,8 +837,9 @@ module PostHog
     # Raises an unsupported_definition error when local evaluation cannot use the filter, whatever
     # the call passes: an unknown operator, or a filter value that the operator cannot parse.
     def self.assert_filter_supported(property)
-      value = property[:value]
-      case property[:operator] || 'exact'
+      value = property.key?(:value) ? property[:value] : property['value']
+      operator = property[:operator] || property['operator'] || 'exact'
+      case operator
       when 'exact', 'is_not', 'is_set', 'is_not_set', 'icontains', 'not_icontains', 'starts_with',
            'not_starts_with', 'ends_with', 'not_ends_with'
         nil
@@ -871,7 +862,7 @@ module PostHog
       when 'semver_wildcard'
         semver_wildcard_bounds(value)
       else
-        raise InconclusiveMatchError.new("Unknown operator: #{property[:operator]}",
+        raise InconclusiveMatchError.new("Unknown operator: #{operator}",
                                          reason: UnresolvedFlagReason::UNSUPPORTED_DEFINITION)
       end
     end
@@ -1086,10 +1077,9 @@ module PostHog
         end
       else
         property_group_items_match?(properties, group_type) do |prop|
-          PostHog::Utils.symbolize_keys!(prop)
           matches = match_property(prop, property_values, cohort_properties,
                                    property_matching_version: property_matching_version)
-          prop[:negation] ? !matches : matches
+          extract_value(prop, :negation) ? !matches : matches
         end
       end
     end

data/lib/posthog/utils.rb

@@ -21,6 +21,14 @@ module PostHog
       hash.replace symbolize_keys hash
     end
 
+    def normalize_flag_evaluation_inputs(groups, person_properties, group_properties)
+      [
+        symbolize_keys(groups || {}),
+        symbolize_keys(person_properties || {}),
+        symbolize_keys(group_properties || {}).transform_values { |properties| symbolize_keys(properties) }
+      ]
+    end
+
     # public: Return a new hash with keys as strings
     #
     def stringify_keys(hash)

data/lib/posthog/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PostHog
-  VERSION = '3.29.0'
+  VERSION = '3.29.1'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: posthog-ruby
 version: !ruby/object:Gem::Version
-  version: 3.29.0
+  version: 3.29.1
 platform: ruby
 authors:
 - ''