active_genie 0.25.2 → 0.26.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a47b22f8cc9f75c2538e99ea16e5116970651e93d7e085ac224e0c7cb5b94131
-  data.tar.gz: c516cb0b89906f21d29c6be507b1fde213bf3c0cd0d4b303af2d5297636f0f7f
+  metadata.gz: 848de45258263935accfd02ce16d14eb96c6b5a73f42979fe0a74e5b9b746fb4
+  data.tar.gz: cb1f9eb843559a7040dc20b1442e120325d734b7610a75b57bfde91cba63dc4e
 SHA512:
-  metadata.gz: ca01c7dc21605c869147aa07c4fc3104da04d0bf0d22ac37e0e0040047d74e3d1a1a843fa5b18592c5f641783cd4c1962cad5dc71c38d408f12d05368362d144
-  data.tar.gz: bd3377cfe1a2452ba4e838306c11405c73e57bf437281a5d0888fe0482aa0efbe2f8b381ec1b8e22d09221b6d67b582be9a2ccaf853ac7b20727783646e4e29f
+  metadata.gz: 12f38e95348fae6beab3cc7a74cfdefc132ef9e8308303f2df04a40703d6f86a9fe6f859c958dc8a1d9fca52d41ceb1f380324c5b34d5d13e0bde4a702362830
+  data.tar.gz: c512e6ae0fb0b47403f9c2fc3be5c55a458320877fe2aae93780af1fab4e95b2fa1553ff60a62ec30653933f77a5c8892e96227ee637e8ab03a30cf0c9ad177d

data/VERSION

@@ -1 +1 @@
-0.25.2
+0.26.0

data/lib/active_genie/clients/providers/anthropic_client.rb

@@ -30,9 +30,7 @@ module ActiveGenie
           temperature: @config.llm.temperature || 0
         }
 
-        retry_with_backoff do
-          request(payload).dig('content', 0, 'input')
-        end
+        request(payload).dig('content', 0, 'input')
       end
 
       ANTHROPIC_ENDPOINT = '/v1/messages'

data/lib/active_genie/clients/providers/base_client.rb

@@ -81,16 +81,13 @@ module ActiveGenie
 
         response = http_request(request, uri)
 
-        case response
-        when Net::HTTPSuccess
-          parsed_response = parse_response(response)
+        raise ClientError, "Unexpected response: #{response.code} - #{response.body}" unless response.is_a?(Net::HTTPSuccess)
 
-          log_request_details(uri:, request:, response:, start_time:, parsed_response:)
+        parsed_response = parse_response(response)
 
-          parsed_response
-        else
-          raise ClientError, "Unexpected response: #{response.code} - #{response.body}"
-        end
+        log_request_details(uri:, request:, response:, start_time:, parsed_response:)
+
+        parsed_response
       end
 
       # Create and configure an HTTP client
@@ -103,7 +100,10 @@ module ActiveGenie
         http.verify_mode = OpenSSL::SSL::VERIFY_PEER
         http.read_timeout = @config.llm.read_timeout || DEFAULT_TIMEOUT
         http.open_timeout = @config.llm.open_timeout || DEFAULT_OPEN_TIMEOUT
-        http.request(request)
+
+        retry_with_backoff do
+          http.request(request)
+        end
       end
 
       # Apply headers to the request
@@ -170,7 +170,7 @@ module ActiveGenie
 
         begin
           yield
-        rescue Net::HTTPError, ClientError => e
+        rescue Net::OpenTimeout, Net::ReadTimeout, ClientError => e
           raise if retries > max_retries
 
           sleep_time = retry_delay * (2**retries)

data/lib/active_genie/clients/providers/deepseek_client.rb

@@ -25,18 +25,16 @@ module ActiveGenie
           model:
         }
 
-        retry_with_backoff do
-          response = request(payload)
+        response = request(payload)
 
-          if response.nil? || response.keys.empty?
-            raise InvalidResponseError,
-                  "Invalid response: #{response}"
-          end
+        if response.nil? || response.keys.empty?
+          raise InvalidResponseError,
+                "Invalid response: #{response}"
+        end
 
-          ActiveGenie::Logger.call({ code: :function_calling, fine_tune: true, payload:, response: })
+        ActiveGenie::Logger.call({ code: :function_calling, fine_tune: true, payload:, response: })
 
-          response
-        end
+        response
       end
 
       private

data/lib/active_genie/clients/providers/google_client.rb

@@ -29,14 +29,12 @@ module ActiveGenie
         }
         params = { key: provider_config.api_key }
 
-        retry_with_backoff do
-          response = request(payload, params)
+        response = request(payload, params)
 
-          json_string = response&.dig('candidates', 0, 'content', 'parts', 0, 'text')
-          return nil if json_string.nil? || json_string.empty?
+        json_string = response&.dig('candidates', 0, 'content', 'parts', 0, 'text')
+        return nil if json_string.nil? || json_string.empty?
 
-          normalize_response(json_string)
-        end
+        normalize_response(json_string)
       end
 
       API_VERSION_PATH = 'v1beta/models'

data/lib/active_genie/clients/providers/openai_client.rb

@@ -25,15 +25,13 @@ module ActiveGenie
           model:
         }
 
-        retry_with_backoff do
-          response = request(payload)
+        response = request(payload)
 
-          raise InvalidResponseError, "Invalid response: #{response}" if response.nil? || response.keys.empty?
+        raise InvalidResponseError, "Invalid response: #{response}" if response.nil? || response.keys.empty?
 
-          ActiveGenie::Logger.call({ code: :function_calling, fine_tune: true, payload:, response: })
+        ActiveGenie::Logger.call({ code: :function_calling, fine_tune: true, payload:, response: })
 
-          response
-        end
+        response
       end
 
       private

data/lib/active_genie/config/log_config.rb

@@ -24,25 +24,27 @@ module ActiveGenie
       end
 
       def output_call(log)
-        output.call(log)
+        output&.call(log)
 
         Array(@observers).each do |obs|
           next unless obs[:scope].all? { |key, value| log[key.to_sym] == value }
 
-          obs[:observer].call(log)
+          obs[:observer]&.call(log)
         rescue StandardError => e
           ActiveGenie::Logger.call(code: :observer_error, **obs, error: e.message)
         end
       end
 
-      def add_observer(observers: [], scope: nil, &block)
+      def add_observer(observers: [], scope: {}, &block)
         @observers ||= []
 
-        raise ArgumentError, 'Scope must be a hash' unless scope.is_a?(Hash)
+        raise ArgumentError, 'Scope must be a hash' if scope && !scope.is_a?(Hash)
 
-        @observers << { observer: block, scope: scope || {} } if block_given?
+        @observers << { observer: block, scope: } if block_given?
         Array(observers).each do |observer|
-          @observers << { observer:, scope: scope || {} }
+          next unless observer.respond_to?(:call)
+
+          @observers << { observer:, scope: }
         end
       end
 

data/lib/active_genie/data_extractor/from_informal.rb

@@ -37,8 +37,8 @@ module ActiveGenie
       def call
         response = Generalist.call(@text, data_to_extract_with_litote, config: @config)
 
-        if response['message_litote']
-          response = Generalist.call(response['litote_rephrased'], @data_to_extract, config: @config)
+        if response[:message_litote]
+          response = Generalist.call(response[:litote_rephrased], @data_to_extract, config: @config)
         end
 
         response

data/lib/active_genie/logger.rb

@@ -21,13 +21,15 @@ module ActiveGenie
       log
     end
 
-    def with_context(context)
+    def with_context(context, observer: nil)
       @context ||= {}
       begin
         @context = @context.merge(context)
+        config.add_observer(observers: [observer])
         yield if block_given?
       ensure
         @context.delete_if { |key, _| context.key?(key) }
+        config.remove_observer([observer])
       end
     end
 

data/lib/active_genie/ranking/elo_round.rb

@@ -17,11 +17,13 @@ module ActiveGenie
         @config = config
         @tmp_defenders = []
         @total_tokens = 0
-        @previous_elo = players.to_h { |player| [player.id, player.elo] }
+        @previous_elo = {}
         @previous_highest_elo = @defender_tier.max_by(&:elo).elo
       end
 
       def call
+        @previous_elo = @players.to_h { |player| [player.id, player.elo] }
+
         ActiveGenie::Logger.with_context(log_context) do
           matches.each do |player_a, player_b|
             # TODO: battle can take a while, can be parallelized
@@ -38,10 +40,6 @@ module ActiveGenie
 
       private
 
-      def save_previous_elo
-        @previous_elo = @players.to_h { |player| [player.id, player.elo] }
-      end
-
       def matches
         @relegation_tier.each_with_object([]) do |attack_player, matches|
           BATTLE_PER_PLAYER.times do
@@ -130,7 +128,7 @@ module ActiveGenie
         elo_diffs = players_in_round.map do |player|
           [player.id, player.elo - @previous_elo[player.id]]
         end
-        elo_diffs.sort_by { |_, diff| -diff }.to_h
+        elo_diffs.sort_by { |_, diff| -(diff || 0) }.to_h
       end
 
       def log_observer(log)

data/lib/active_genie/ranking/free_for_all.rb

@@ -18,7 +18,7 @@ module ActiveGenie
       end
 
       def call
-        ActiveGenie::Logger.with_context(log_context, observer: method(:log_observer)) do
+        ActiveGenie::Logger.with_context(log_context, observer: ->(log) { log_observer(log) }) do
           matches.each do |player_a, player_b|
             winner, loser = battle(player_a, player_b)
 

data/lib/active_genie/ranking/player.rb

@@ -10,8 +10,6 @@ module ActiveGenie
         @params[:content] ||= @params
       end
 
-      attr_accessor :rank
-
       def content
         @content ||= @params[:content]
       end
@@ -29,7 +27,13 @@ module ActiveGenie
       end
 
       def elo
-        @elo ||= @params[:elo]
+        @elo = if @params[:elo]
+                 @params[:elo]
+               elsif @score
+                 generate_elo_by_score
+               else
+                 BASE_ELO
+               end
       end
 
       def ffa_win_count
@@ -56,7 +60,7 @@ module ActiveGenie
 
       def elo=(value)
         ActiveGenie::Logger.call({ code: :new_elo, player_id: id, elo: value }) if value != @elo
-        @elo = value
+        @elo = value || BASE_ELO
       end
 
       def eliminated=(value)
@@ -93,7 +97,9 @@ module ActiveGenie
 
       def to_h
         {
-          id:, name:, content:, score:, elo:,
+          id:, name:, content:,
+
+          score:, elo:,
           ffa_win_count:, ffa_lose_count:, ffa_draw_count:,
           eliminated:, ffa_score:, sort_value:
         }

data/lib/active_genie/ranking/players_collection.rb

@@ -52,19 +52,13 @@ module ActiveGenie
       end
 
       def sorted
-        sorted_players = @players.sort_by { |p| -p.sort_value }
-        sorted_players.each_with_index { |p, i| p.rank = i + 1 }
-        sorted_players
+        @players.sort_by { |p| -p.sort_value }
       end
 
       def to_json(*_args)
         to_h.to_json
       end
 
-      def to_h
-        sorted.map(&:to_h)
-      end
-
       def method_missing(...)
         @players.send(...)
       end

data/lib/active_genie/ranking/ranking_scoring.rb

@@ -20,8 +20,7 @@ module ActiveGenie
         ActiveGenie::Logger.with_context(log_context) do
           @reviewers = generate_reviewers
 
-          players_to_score = players_without_score
-          players_to_score.each do |player|
+          players_without_score.each do |player|
             player.score = generate_score(player)
           end
         end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_genie
 version: !ruby/object:Gem::Version
-  version: 0.25.2
+  version: 0.26.0
 platform: ruby
 authors:
 - Radamés Roriz
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-06-01 00:00:00.000000000 Z
+date: 2025-06-02 00:00:00.000000000 Z
 dependencies: []
 description: |
   The lodash for GenAI, stop reinventing the wheel