chore: validate request option coercion correctness

Author: stainless-app[bot]

lib/finch_api/internal/type/base_model.rb

@@ -397,15 +397,7 @@ def to_yaml(*a) = FinchAPI::Internal::Type::Converter.dump(self.class, self).to_
         # Create a new instance of a model.
         #
         # @param data [Hash{Symbol=>Object}, self]
-        def initialize(data = {})
-          case FinchAPI::Internal::Util.coerce_hash(data)
-          in Hash => coerced
-            @data = coerced
-          else
-            message = "Expected a #{Hash} or #{FinchAPI::Internal::Type::BaseModel}, got #{data.inspect}"
-            raise ArgumentError.new(message)
-          end
-        end
+        def initialize(data = {}) = (@data = FinchAPI::Internal::Util.coerce_hash!(data).to_h)
 
         class << self
           # @api private

lib/finch_api/internal/type/request_parameters.rb

@@ -28,14 +28,8 @@ def dump_request(params)
             state = {can_retry: true}
             case (dumped = dump(params, state: state))
             in Hash
-              options = FinchAPI::Internal::Util.coerce_hash(dumped[:request_options])
-              request_options =
-                case [options, state.fetch(:can_retry)]
-                in [Hash | nil, false]
-                  {**options.to_h, max_retries: 0}
-                else
-                  options
-                end
+              options = FinchAPI::Internal::Util.coerce_hash!(dumped[:request_options]).to_h
+              request_options = state.fetch(:can_retry) ? options : {**options, max_retries: 0}
               [dumped.except(:request_options), request_options]
             else
               [dumped, nil]

lib/finch_api/internal/util.rb

@@ -128,6 +128,22 @@ def coerce_hash(input)
             input.respond_to?(:to_h) ? input.to_h : input
           end
         end
+
+        # @api private
+        #
+        # @param input [Object]
+        #
+        # @raise [ArgumentError]
+        # @return [Hash{Object=>Object}, nil]
+        def coerce_hash!(input)
+          case coerce_hash(input)
+          in Hash | nil => coerced
+            coerced
+          else
+            message = "Expected a #{Hash} or #{FinchAPI::Internal::Type::BaseModel}, got #{data.inspect}"
+            raise ArgumentError.new(message)
+          end
+        end
       end
 
       class << self

rbi/finch_api/internal/util.rbi

@@ -42,6 +42,10 @@ module FinchAPI
         # @api private
         sig { params(input: T.anything).returns(T.any(T::Hash[T.anything, T.anything], T.anything)) }
         def coerce_hash(input); end
+
+        # @api private
+        sig { params(input: T.anything).returns(T.nilable(T::Hash[T.anything, T.anything])) }
+        def coerce_hash!(input); end
       end
 
       class << self

sig/finch_api/internal/util.rbs

@@ -19,6 +19,8 @@ module FinchAPI
 
       def self?.coerce_hash: (top input) -> (::Hash[top, top] | top)
 
+      def self?.coerce_hash!: (top input) -> ::Hash[top, top]?
+
       def self?.deep_merge_lr: (top lhs, top rhs, ?concat: bool) -> top
 
       def self?.deep_merge: (