open_router_enhanced 1.2.0 → 1.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1e39d5924a6388355a4ecae7a76933bf429546a3a5da1975d5dd020804735d24
-  data.tar.gz: 220e9504b58ad511e971a8c4a13939edef2c390f0ffeb031e21dfdb30ce78941
+  metadata.gz: d14a6a35444f5f2321779b457bcd3e72bbb6f549faed1271f845a094195a7cc7
+  data.tar.gz: 5633d120125f115e56cf00a4da5e6be0d2a16934180fd2c6f9b4088342ea2615
 SHA512:
-  metadata.gz: 61f753aedc0057758a7cb8310ba06b5751da07ca17ed6bd433473e8ce36184286a37060cd32b2595ac9a8b5e9c798cd2db078ae5c7ddf2232c4b6c271d44623a
-  data.tar.gz: bb9f8c9122ddc7fc1c850b34a76ee9e209ca14ff95ff4fe2dbd68b8fccdf10021f78111cd994c0e0f3a163ee61512434b39410227ec818cbf52a1cf22aac5ff6
+  metadata.gz: 5abeac77e6a1e81c3151df75b3450841912b74e3326f745eea5000be8f784822df73daaec90e92e928a1745df79a4eb335b1be922cd01c587a8b73c0b3cb391b
+  data.tar.gz: b6e344ffd954a89ebe79f7cf42a308f95e6e93569863e0a06ecb4b28d97d16ff6e656284312bb0dd80ad8943c3018f529794e03aa541624aa8f5b040dd1ca4cf

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 ## [Unreleased]
 
+## [1.2.1] - 2025-12-24
+
+### Fixed
+- Memoized `output_id` in `ResponsesToolResult` to ensure consistent IDs across multiple calls
+- Memoized `message_output` and `reasoning_output` finders in `ResponsesResponse` for performance
+
 ## [1.2.0] - 2025-12-24
 
 ### Added

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    open_router_enhanced (1.2.0)
+    open_router_enhanced (1.2.1)
       activesupport (>= 6.0, < 9.0)
       dotenv (>= 2.0, < 4.0)
       faraday (>= 1.0, < 3.0)

data/lib/open_router/client.rb

@@ -9,6 +9,7 @@ require "pry"
 module OpenRouter
   class ServerError < StandardError; end
 
+  # rubocop:disable Metrics/ClassLength
   class Client
     include OpenRouter::HTTP
 
@@ -103,8 +104,10 @@ module OpenRouter
     # @param extras [Hash] Optional hash of model-specific parameters to send to the OpenRouter API
     # @param stream [Proc, nil] Optional callable object for streaming
     # @return [Response] The completion response wrapped in a Response object.
+    # rubocop:disable Metrics/ParameterLists
     def complete(messages, model: "openrouter/auto", providers: [], transforms: [], plugins: [], tools: [], tool_choice: nil,
                  response_format: nil, force_structured_output: nil, prediction: nil, extras: {}, stream: nil)
+      # rubocop:enable Metrics/ParameterLists
       parameters = prepare_base_parameters(messages, model, providers, transforms, plugins, prediction, stream, extras)
       forced_extraction = configure_tools_and_structured_outputs!(parameters, model, tools, tool_choice,
                                                                   response_format, force_structured_output)
@@ -666,4 +669,5 @@ module OpenRouter
       INSTRUCTION
     end
   end
+  # rubocop:enable Metrics/ClassLength
 end

data/lib/open_router/model_registry.rb

@@ -1,11 +1,9 @@
 # frozen_string_literal: true
 
 require "json"
-require "net/http"
-require "uri"
+require "faraday"
 require "tmpdir"
 require "fileutils"
-require "openssl"
 
 module OpenRouter
   class ModelRegistryError < Error; end
@@ -18,30 +16,20 @@ module OpenRouter
     MAX_CACHE_SIZE_MB = 50 # Maximum cache size in megabytes
 
     class << self
-      # Fetch models from OpenRouter API
+      # Fetch models from OpenRouter API using Faraday for consistent SSL handling
       def fetch_models_from_api
-        uri = URI("#{API_BASE}/models")
-
-        # Use configurable timeout and SSL settings
-        http = Net::HTTP.new(uri.host, uri.port)
-        http.use_ssl = true
-        http.verify_mode = OpenSSL::SSL::VERIFY_PEER
-        http.read_timeout = OpenRouter.configuration.model_registry_timeout
-        http.open_timeout = OpenRouter.configuration.model_registry_timeout
-
-        request = Net::HTTP::Get.new(uri)
-        response = http.request(request)
-
-        unless response.code == "200"
-          raise ModelRegistryError,
-                "Failed to fetch models from OpenRouter API: #{response.message}"
+        conn = Faraday.new(url: API_BASE) do |f|
+          f.options[:timeout] = OpenRouter.configuration.model_registry_timeout
+          f.options[:open_timeout] = OpenRouter.configuration.model_registry_timeout
+          f.response :raise_error
         end
 
+        response = conn.get("models")
         JSON.parse(response.body)
+      rescue Faraday::Error => e
+        raise ModelRegistryError, "Failed to fetch models from OpenRouter API: #{e.message}"
       rescue JSON::ParserError => e
         raise ModelRegistryError, "Failed to parse OpenRouter API response: #{e.message}"
-      rescue StandardError => e
-        raise ModelRegistryError, "Network error fetching models: #{e.message}"
       end
 
       # Ensure cache directory exists and set up cleanup
@@ -367,9 +355,9 @@ module OpenRouter
         total_size = Dir.glob(File.join(CACHE_DIR, "**/*"))
                         .select { |f| File.file?(f) }
                         .sum do |f|
-          File.size(f)
-        rescue StandardError
-          0
+                          File.size(f)
+                        rescue StandardError
+                          0
         end
         total_size / (1024.0 * 1024.0)
       end

data/lib/open_router/responses_response.rb

@@ -48,9 +48,10 @@ module OpenRouter
     end
 
     # Check if reasoning was included in the response
-    def has_reasoning?
+    def reasoning?
       !reasoning_output.nil?
     end
+    alias has_reasoning? reasoning?
 
     # Get tool/function calls from the response as ResponsesToolCall objects
     #
@@ -68,9 +69,10 @@ module OpenRouter
       output.select { |o| o["type"] == "function_call" }
     end
 
-    def has_tool_calls?
+    def tool_calls?
       tool_calls.any?
     end
+    alias has_tool_calls? tool_calls?
 
     # Execute all tool calls and return results
     #
@@ -137,9 +139,7 @@ module OpenRouter
       end
 
       # Add assistant message if present
-      if message_output
-        input_items << message_output
-      end
+      input_items << message_output if message_output
 
       # Add follow-up user message if provided
       if follow_up_message
@@ -155,15 +155,15 @@ module OpenRouter
 
     # Token counts
     def input_tokens
-      usage.dig("input_tokens") || 0
+      usage["input_tokens"] || 0
     end
 
     def output_tokens
-      usage.dig("output_tokens") || 0
+      usage["output_tokens"] || 0
     end
 
     def total_tokens
-      usage.dig("total_tokens") || 0
+      usage["total_tokens"] || 0
     end
 
     def reasoning_tokens
@@ -182,11 +182,11 @@ module OpenRouter
     private
 
     def message_output
-      output.find { |o| o["type"] == "message" }
+      @message_output ||= output.find { |o| o["type"] == "message" }
     end
 
     def reasoning_output
-      output.find { |o| o["type"] == "reasoning" }
+      @reasoning_output ||= output.find { |o| o["type"] == "reasoning" }
     end
   end
 end

data/lib/open_router/responses_tool_call.rb

@@ -18,9 +18,7 @@ module OpenRouter
     end
 
     # Get the function name
-    def name
-      @name
-    end
+    attr_reader :name
 
     # Alias for consistency with ToolCall
     def function_name
@@ -56,12 +54,13 @@ module OpenRouter
   class ResponsesToolResult
     include ToolResultBase
 
-    attr_reader :tool_call, :result, :error
+    attr_reader :tool_call, :result, :error, :output_id
 
     def initialize(tool_call, result = nil, error = nil)
       @tool_call = tool_call
       @result = result
       @error = error
+      @output_id = "fc_output_#{SecureRandom.hex(8)}"
     end
 
     # Convert to function_call_output format for conversation continuation
@@ -78,7 +77,7 @@ module OpenRouter
 
       {
         "type" => "function_call_output",
-        "id" => "fc_output_#{SecureRandom.hex(8)}",
+        "id" => @output_id,
         "call_id" => @tool_call.call_id,
         "output" => output_content
       }

data/lib/open_router/tool_call_base.rb

@@ -50,6 +50,7 @@ module OpenRouter
       !success?
     end
 
+    # Factory methods for creating tool results
     module ClassMethods
       # Create a failed result
       def failure(tool_call, error)

data/lib/open_router/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module OpenRouter
-  VERSION = "1.2.0"
+  VERSION = "1.2.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: open_router_enhanced
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.2.2
 platform: ruby
 authors:
 - Eric Stiens