chore: loosen const and integer coercion rules (#121)

Author: stainless-app[bot]

lib/openai/internal/type/converter.rb

@@ -149,9 +149,9 @@ def coerce(
                 if value.is_a?(Integer)
                   exactness[:yes] += 1
                   return value
-                elsif strictness == :strong
+                elsif strictness == :strong && Integer(value, exception: false) != value
                   message = "no implicit conversion of #{value.class} into #{target.inspect}"
-                  raise TypeError.new(message)
+                  raise value.is_a?(Numeric) ? ArgumentError.new(message) : TypeError.new(message)
                 else
                   Kernel.then do
                     return Integer(value).tap { exactness[:maybe] += 1 }
@@ -197,12 +197,20 @@ def coerce(
               else
               end
             in Symbol
-              if (value.is_a?(Symbol) || value.is_a?(String)) && value.to_sym == target
-                exactness[:yes] += 1
-                return target
-              elsif strictness == :strong
-                message = "cannot convert non-matching #{value.class} into #{target.inspect}"
-                raise ArgumentError.new(message)
+              case value
+              in Symbol | String
+                if value.to_sym == target
+                  exactness[:yes] += 1
+                  return target
+                else
+                  exactness[:maybe] += 1
+                  return value
+                end
+              else
+                if strictness == :strong
+                  message = "cannot convert non-matching #{value.class} into #{target.inspect}"
+                  raise ArgumentError.new(message)
+                end
               end
             else
             end

test/openai/internal/type/base_model_test.rb

@@ -111,7 +111,6 @@ def test_coerce_errors
       [Integer, "one"] => TypeError,
       [Float, "one"] => TypeError,
       [String, Time] => TypeError,
-      [:a, "one"] => ArgumentError,
       [Date, "one"] => ArgumentError,
       [Time, "one"] => ArgumentError
     }
@@ -346,7 +345,7 @@ def test_coerce
       [M2, {a: "1990-09-19", c: nil}] => [{yes: 2, maybe: 2}, {a: "1990-09-19", c: nil}],
 
       [M3, {c: "c", d: "d"}] => [{yes: 3}, {c: :c, d: :d}],
-      [M3, {c: "d", d: "c"}] => [{yes: 1, no: 2}, {c: "d", d: "c"}],
+      [M3, {c: "d", d: "c"}] => [{yes: 1, maybe: 2}, {c: "d", d: "c"}],
 
       [M4, {c: 2}] => [{yes: 5}, {c: 2}],
       [M4, {a: "1", c: 2}] => [{yes: 4, maybe: 1}, {a: "1", c: 2}],
@@ -404,7 +403,8 @@ def test_accessors
     cases = {
       M2.new({a: "1990-09-19", b: "1"}) => {a: Time.new(1990, 9, 19), b: TypeError},
       M2.new(a: "one", b: "one") => {a: ArgumentError, b: TypeError},
-      M2.new(a: nil, b: 2.0) => {a: TypeError, b: TypeError},
+      M2.new(a: nil, b: 2.0) => {a: TypeError},
+      M2.new(a: nil, b: 2.2) => {a: TypeError, b: ArgumentError},
 
       M3.new => {d: :d},
       M3.new(d: 1) => {d: ArgumentError},
@@ -520,8 +520,8 @@ def test_coerce
       [U0, :""] => [{no: 1}, 0, :""],
 
       [U1, "a"] => [{yes: 1}, 1, :a],
-      [U1, "2"] => [{maybe: 1}, 2, 2],
-      [U1, :b] => [{no: 1}, 2, :b],
+      [U1, "2"] => [{maybe: 1}, 2, "2"],
+      [U1, :b] => [{maybe: 1}, 2, :b],
 
       [U2, {type: :a}] => [{yes: 3}, 0, {t: :a}],
       [U2, {type: "b"}] => [{yes: 3}, 0, {type: :b}],