posthog-ruby 2.9.0 → 2.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a9be467263d4138e0b223e18281a9b0af56351fbf9d807b01bf4ff8f87d004a7
-  data.tar.gz: 1c4a9990d2958023b7db53511e64aa90ade5364d06e42e23b2b981d6acb59ba2
+  metadata.gz: d987590608ff1c705f8f419d43cb6ef439233f8b3dbc5e3ac84bd64b3d552d9a
+  data.tar.gz: d4ae4f458a8a9faa654519b958cc20f828638ead4696dbc8a8c03ed198e44606
 SHA512:
-  metadata.gz: abde921140981a63fb82677e5024c4fcce79f9e560a870b0fd91114eaff0df1af0a5908742fbab3dee612d1d1a866998e9f1d10a587484e4baaf06b2f7cafd45
-  data.tar.gz: 7d88779cd59527ba500d8a2dcee0692a7135da88910d8045b276f32b635caee887b3fe2382a85fd6408b0e8cd57c58e0ca84495c99f912763a13f2b62e01dc78
+  metadata.gz: a9671f5f0fb37ae15a605b1118f09a526fa383aa129335a78a85f8b54c772e8c28ade9e3fd06bf36e142f7a994eca33400965add6c1e2ceebf3b8fcc32d8df23
+  data.tar.gz: 5fd8756e6c1f20bb0df3c40e8e72aa9c9d79dedd2bedb0c53930254d8a58417b75c38867f9e2e673bd0a53968cb7beded76cc6be18760d3fe8755d33cc75cd1b

data/bin/posthog

@@ -11,7 +11,7 @@ program :version, '1.0.0'
 program :description, 'PostHog API'
 
 def json_hash(str)
-  return JSON.parse(str) if str
+  JSON.parse(str) if str
 end
 
 command :capture do |c|
@@ -27,13 +27,13 @@ command :capture do |c|
   c.option '--event=<event>', String, 'The event name to send with the event'
   c.option '--properties=<properties>', 'The properties to send (JSON-encoded)'
 
-  c.action do |args, options|
+  c.action do |_args, options|
     posthog =
       PostHog::Client.new(
         {
           api_key: options.api_key,
           api_host: options.api_host,
-          on_error: Proc.new { |status, msg| print msg }
+          on_error: proc { |_status, msg| print msg }
         }
       )
 
@@ -61,13 +61,13 @@ command :identify do |c|
            'The distinct id to send the event as'
   c.option '--properties=<properties>', 'The properties to send (JSON-encoded)'
 
-  c.action do |args, options|
+  c.action do |_args, options|
     posthog =
       PostHog::Client.new(
         {
           api_key: options.api_key,
           api_host: options.api_host,
-          on_error: Proc.new { |status, msg| print msg }
+          on_error: proc { |_status, msg| print msg }
         }
       )
 
@@ -92,13 +92,13 @@ command :alias do |c|
   c.option '--distinct-id=<distinct_id>', String, 'The distinct id'
   c.option '--alias=<alias>', 'The alias to give to the distinct id'
 
-  c.action do |args, options|
+  c.action do |_args, options|
     posthog =
       PostHog::Client.new(
         {
           api_key: options.api_key,
           api_host: options.api_host,
-          on_error: Proc.new { |status, msg| print msg }
+          on_error: proc { |_status, msg| print msg }
         }
       )
 

data/lib/posthog/client.rb

@@ -22,8 +22,10 @@ class PostHog
     #   queued for testing. Defaults to +false+.
     # @option opts [Proc] :on_error Handles error calls from the API.
     # @option opts [String] :host Fully qualified hostname of the PostHog server. Defaults to `https://app.posthog.com`
-    # @option opts [Integer] :feature_flags_polling_interval How often to poll for feature flag definition changes. Measured in seconds, defaults to 30.
-    # @option opts [Integer] :feature_flag_request_timeout_seconds How long to wait for feature flag evaluation. Measured in seconds, defaults to 3.
+    # @option opts [Integer] :feature_flags_polling_interval How often to poll for feature flag definition changes.
+    #   Measured in seconds, defaults to 30.
+    # @option opts [Integer] :feature_flag_request_timeout_seconds How long to wait for feature flag evaluation.
+    #   Measured in seconds, defaults to 3.
     def initialize(opts = {})
       symbolize_keys!(opts)
 
@@ -34,10 +36,10 @@ class PostHog
       @max_queue_size = opts[:max_queue_size] || Defaults::Queue::MAX_SIZE
       @worker_mutex = Mutex.new
       @worker = if opts[:test_mode]
-        NoopWorker.new(@queue)
-      else
-        SendWorker.new(@queue, @api_key, opts)
-      end
+                  NoopWorker.new(@queue)
+                else
+                  SendWorker.new(@queue, @api_key, opts)
+                end
       @worker_thread = nil
       @feature_flags_poller = nil
       @personal_api_key = opts[:personal_api_key]
@@ -53,8 +55,10 @@ class PostHog
           opts[:feature_flag_request_timeout_seconds] || Defaults::FeatureFlags::FLAG_REQUEST_TIMEOUT_SECONDS,
           opts[:on_error]
         )
-      
-      @distinct_id_has_sent_flag_calls = SizeLimitedHash.new(Defaults::MAX_HASH_SIZE) { |hash, key| hash[key] = Array.new }
+
+      @distinct_id_has_sent_flag_calls = SizeLimitedHash.new(Defaults::MAX_HASH_SIZE) do |hash, key|
+        hash[key] = []
+      end
     end
 
     # Synchronously waits until the worker has cleared the queue.
@@ -88,6 +92,10 @@ class PostHog
     # @option attrs [String] :event Event name
     # @option attrs [Hash] :properties Event properties (optional)
     # @option attrs [Bool] :send_feature_flags Whether to send feature flags with this event (optional)
+    # @option attrs [String] :uuid ID that uniquely identifies an event;
+    #                             events in PostHog are deduplicated by the
+    #                             combination of teamId, timestamp date,
+    #                             event name, distinct id, and UUID
     # @macro common_attrs
     def capture(attrs)
       symbolize_keys! attrs
@@ -147,18 +155,34 @@ class PostHog
       @queue.length
     end
 
-    def is_feature_enabled(flag_key, distinct_id, groups: {}, person_properties: {}, group_properties: {}, only_evaluate_locally: false, send_feature_flag_events: true)
-      response = get_feature_flag(flag_key, distinct_id, groups: groups, person_properties: person_properties, group_properties: group_properties, only_evaluate_locally: only_evaluate_locally, send_feature_flag_events: send_feature_flag_events)
-      if response.nil?
-        return nil
-      end
+    # TODO: In future version, rename to `feature_flag_enabled?`
+    def is_feature_enabled( # rubocop:disable Naming/PredicateName
+      flag_key,
+      distinct_id,
+      groups: {},
+      person_properties: {},
+      group_properties: {},
+      only_evaluate_locally: false,
+      send_feature_flag_events: true
+    )
+      response = get_feature_flag(
+        flag_key,
+        distinct_id,
+        groups: groups,
+        person_properties: person_properties,
+        group_properties: group_properties,
+        only_evaluate_locally: only_evaluate_locally,
+        send_feature_flag_events: send_feature_flag_events
+      )
+      return nil if response.nil?
+
       !!response
     end
 
     # @param [String] flag_key The unique flag key of the feature flag
     # @return [String] The decrypted value of the feature flag payload
     def get_remote_config_payload(flag_key)
-      return @feature_flags_poller.get_remote_config_payload(flag_key)
+      @feature_flags_poller.get_remote_config_payload(flag_key)
     end
 
     # Returns whether the given feature flag is enabled for the given user or not
@@ -168,35 +192,56 @@ class PostHog
     # @param [Hash] groups
     # @param [Hash] person_properties key-value pairs of properties to associate with the user.
     # @param [Hash] group_properties
-    # 
+    #
     # @return [String, nil] The value of the feature flag
     #
     # The provided properties are used to calculate feature flags locally, if possible.
     #
-    # `groups` are a mapping from group type to group key. So, if you have a group type of "organization" and a group key of "5",
+    # `groups` are a mapping from group type to group key. So, if you have a group type of "organization"
+    # and a group key of "5",
     # you would pass groups={"organization": "5"}.
     # `group_properties` take the format: { group_type_name: { group_properties } }
-    # So, for example, if you have the group type "organization" and the group key "5", with the properties name, and employee count,
-    # you'll send these as:
+    # So, for example, if you have the group type "organization" and the group key "5", with the properties name,
+    # and employee count, you'll send these as:
     # ```ruby
     #     group_properties: {"organization": {"name": "PostHog", "employees": 11}}
     # ```
-    def get_feature_flag(key, distinct_id, groups: {}, person_properties: {}, group_properties: {}, only_evaluate_locally: false, send_feature_flag_events: true)
-      person_properties, group_properties = add_local_person_and_group_properties(distinct_id, groups, person_properties, group_properties)
-      feature_flag_response, flag_was_locally_evaluated, request_id = @feature_flags_poller.get_feature_flag(key, distinct_id, groups, person_properties, group_properties, only_evaluate_locally)
+    def get_feature_flag(
+      key,
+      distinct_id,
+      groups: {},
+      person_properties: {},
+      group_properties: {},
+      only_evaluate_locally: false,
+      send_feature_flag_events: true
+    )
+      person_properties, group_properties = add_local_person_and_group_properties(
+        distinct_id,
+        groups,
+        person_properties,
+        group_properties
+      )
+      feature_flag_response, flag_was_locally_evaluated, request_id = @feature_flags_poller.get_feature_flag(
+        key,
+        distinct_id,
+        groups,
+        person_properties,
+        group_properties,
+        only_evaluate_locally
+      )
 
       feature_flag_reported_key = "#{key}_#{feature_flag_response}"
       if !@distinct_id_has_sent_flag_calls[distinct_id].include?(feature_flag_reported_key) && send_feature_flag_events
         capture(
           {
-            'distinct_id': distinct_id,
-            'event': '$feature_flag_called',
-            'properties': {
+            distinct_id: distinct_id,
+            event: '$feature_flag_called',
+            properties: {
               '$feature_flag' => key,
               '$feature_flag_response' => feature_flag_response,
               'locally_evaluated' => flag_was_locally_evaluated
-            }.merge(request_id ? {'$feature_flag_request_id' => request_id} : {}),
-            'groups': groups,
+            }.merge(request_id ? { '$feature_flag_request_id' => request_id } : {}),
+            groups: groups
           }
         )
         @distinct_id_has_sent_flag_calls[distinct_id] << feature_flag_reported_key
@@ -210,11 +255,19 @@ class PostHog
     # @param [Hash] groups
     # @param [Hash] person_properties key-value pairs of properties to associate with the user.
     # @param [Hash] group_properties
-    # 
+    #
     # @return [Hash] String (not symbol) key value pairs of flag and their values
-    def get_all_flags(distinct_id, groups: {}, person_properties: {}, group_properties: {}, only_evaluate_locally: false)
-      person_properties, group_properties = add_local_person_and_group_properties(distinct_id, groups, person_properties, group_properties)
-      return @feature_flags_poller.get_all_flags(distinct_id, groups, person_properties, group_properties, only_evaluate_locally)
+    def get_all_flags(
+      distinct_id,
+      groups: {},
+      person_properties: {},
+      group_properties: {},
+      only_evaluate_locally: false
+    )
+      person_properties, group_properties = add_local_person_and_group_properties(distinct_id, groups,
+                                                                                  person_properties, group_properties)
+      @feature_flags_poller.get_all_flags(distinct_id, groups, person_properties, group_properties,
+                                          only_evaluate_locally)
     end
 
     # Returns payload for a given feature flag
@@ -227,13 +280,23 @@ class PostHog
     # @option [Hash] group_properties
     # @option [Boolean] only_evaluate_locally
     #
-    def get_feature_flag_payload(key, distinct_id, match_value: nil, groups: {}, person_properties: {}, group_properties: {}, only_evaluate_locally: false)
-      person_properties, group_properties = add_local_person_and_group_properties(distinct_id, 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)
+    def get_feature_flag_payload(
+      key,
+      distinct_id,
+      match_value: nil,
+      groups: {},
+      person_properties: {},
+      group_properties: {},
+      only_evaluate_locally: false
+    )
+      person_properties, group_properties = add_local_person_and_group_properties(distinct_id, 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
 
     # Returns all flags and payloads for a given user
-    # 
+    #
     # @return [Hash] A hash with the following keys:
     #   featureFlags: A hash of feature flags
     #   featureFlagPayloads: A hash of feature flag payloads
@@ -244,9 +307,20 @@ class PostHog
     # @option [Hash] group_properties
     # @option [Boolean] only_evaluate_locally
     #
-    def get_all_flags_and_payloads(distinct_id, groups: {}, person_properties: {}, group_properties: {}, only_evaluate_locally: false)
-      person_properties, group_properties = add_local_person_and_group_properties(distinct_id, groups, person_properties, group_properties)
-      response = @feature_flags_poller.get_all_flags_and_payloads(distinct_id, groups, person_properties, group_properties, only_evaluate_locally)
+    def get_all_flags_and_payloads(
+      distinct_id,
+      groups: {},
+      person_properties: {},
+      group_properties: {},
+      only_evaluate_locally: false
+    )
+      person_properties, group_properties = add_local_person_and_group_properties(
+        distinct_id, groups, person_properties, group_properties
+      )
+      response = @feature_flags_poller.get_all_flags_and_payloads(
+        distinct_id, groups, person_properties, group_properties, only_evaluate_locally
+      )
+
       response.delete(:requestId) # remove internal information.
       response
     end
@@ -283,8 +357,8 @@ class PostHog
       else
         logger.warn(
           'Queue is full, dropping events. The :max_queue_size ' \
-            'configuration parameter can be increased to prevent this from ' \
-            'happening.'
+          'configuration parameter can be increased to prevent this from ' \
+          'happening.'
         )
         false
       end
@@ -297,8 +371,10 @@ class PostHog
 
     def ensure_worker_running
       return if worker_running?
+
       @worker_mutex.synchronize do
         return if worker_running?
+
         @worker_thread = Thread.new { @worker.run }
       end
     end
@@ -308,7 +384,6 @@ class PostHog
     end
 
     def add_local_person_and_group_properties(distinct_id, groups, person_properties, group_properties)
-
       groups ||= {}
       person_properties ||= {}
       group_properties ||= {}
@@ -317,21 +392,22 @@ class PostHog
       symbolize_keys! person_properties
       symbolize_keys! group_properties
 
-      group_properties.each do |key, value|
+      group_properties.each_value do |value|
         symbolize_keys! value
       end
 
-      all_person_properties = { "distinct_id" => distinct_id }.merge(person_properties)
+      all_person_properties = { distinct_id: distinct_id }.merge(person_properties)
 
       all_group_properties = {}
       if groups
         groups.each do |group_name, group_key|
           all_group_properties[group_name] = {
-            "$group_key" => group_key}.merge(group_properties&.dig(group_name) || {})
+            :'$group_key' => group_key
+          }.merge((group_properties && group_properties[group_name]) || {})
         end
       end
 
-      return all_person_properties, all_group_properties
+      [all_person_properties, all_group_properties]
     end
   end
 end

data/lib/posthog/defaults.rb

@@ -1,25 +1,24 @@
 class PostHog
   module Defaults
-
     MAX_HASH_SIZE = 50_000
 
     module Request
-      HOST = 'app.posthog.com'
+      HOST = 'app.posthog.com'.freeze
       PORT = 443
-      PATH = '/batch/'
+      PATH = '/batch/'.freeze
       SSL = true
       HEADERS = {
         'Accept' => 'application/json',
         'Content-Type' => 'application/json',
         'User-Agent' => "posthog-ruby/#{PostHog::VERSION}"
-      }
+      }.freeze
       RETRIES = 10
     end
 
     module FeatureFlags
       FLAG_REQUEST_TIMEOUT_SECONDS = 3
     end
-    
+
     module Queue
       MAX_SIZE = 10_000
     end

data/lib/posthog/feature_flag.rb

@@ -4,34 +4,35 @@ class FeatureFlag
 
   def initialize(json)
     json.transform_keys!(&:to_s)
-    @key = json["key"]
-    @enabled = json["enabled"]
-    @variant = json["variant"]
-    @reason = json["reason"] ? EvaluationReason.new(json["reason"]) : nil
-    @metadata = json["metadata"] ? FeatureFlagMetadata.new(json["metadata"].transform_keys(&:to_s)) : nil
+    @key = json['key']
+    @enabled = json['enabled']
+    @variant = json['variant']
+    @reason = json['reason'] ? EvaluationReason.new(json['reason']) : nil
+    @metadata = json['metadata'] ? FeatureFlagMetadata.new(json['metadata'].transform_keys(&:to_s)) : nil
   end
 
-  def get_value
+  # TODO: Rename to `value` in future version
+  def get_value # rubocop:disable Naming/AccessorMethodName
     @variant || @enabled
   end
 
   def payload
-    @metadata&.payload
+    @metadata.payload if @metadata
   end
 
   def self.from_value_and_payload(key, value, payload)
     new({
-      "key" => key,
-      "enabled" => value.is_a?(String) ? true : value,
-      "variant" => value.is_a?(String) ? value : nil,
-      "reason" => nil,
-      "metadata" => {
-        "id" => nil,
-        "version" => nil,
-        "payload" => payload,
-        "description" => nil
-      }
-    })
+          'key' => key,
+          'enabled' => value.is_a?(String) || value,
+          'variant' => value.is_a?(String) ? value : nil,
+          'reason' => nil,
+          'metadata' => {
+            'id' => nil,
+            'version' => nil,
+            'payload' => payload,
+            'description' => nil
+          }
+        })
   end
 end
 
@@ -41,9 +42,9 @@ class EvaluationReason
 
   def initialize(json)
     json.transform_keys!(&:to_s)
-    @code = json["code"]
-    @description = json["description"]
-    @condition_index = json["condition_index"]&.to_i if json["condition_index"]
+    @code = json['code']
+    @description = json['description']
+    @condition_index = json['condition_index'].to_i if json['condition_index']
   end
 end
 
@@ -53,9 +54,9 @@ class FeatureFlagMetadata
 
   def initialize(json)
     json.transform_keys!(&:to_s)
-    @id = json["id"]
-    @version = json["version"]
-    @payload = json["payload"]
-    @description = json["description"]
+    @id = json['id']
+    @version = json['version']
+    @payload = json['payload']
+    @description = json['description']
   end
-end
+end