fix: align runtime and test compatibility

gems/smithy-client/lib/smithy-client/util.rb

@@ -1,15 +1,25 @@
 # frozen_string_literal: true
 
+require 'cgi/escape'
+require 'cgi/util' if RUBY_VERSION < '3.5'
+
 module Smithy
   module Client
     # @api private
     module Util
       def self.str_to_bool(str)
-        case str
+        case str.to_s
         when 'true' then true
         when 'false' then false
         end
       end
+
+      def self.escape(value)
+        encoded = CGI.escape(value.encode('UTF-8'))
+        encoded = encoded.gsub('+', '%20') if encoded.include?('+')
+        encoded = encoded.gsub('%7E', '~') if encoded.include?('%7E')
+        encoded
+      end
     end
   end
 end

gems/smithy-client/spec/smithy-client/util_spec.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+require_relative '../spec_helper'
+
+module Smithy
+  module Client
+    describe Util do
+      describe '.escape' do
+        it 'uses RFC 3986 query escaping' do
+          expect(described_class.escape('a value~')).to eq('a%20value~')
+        end
+      end
+
+      describe '.str_to_bool' do
+        it 'accepts values that stringify to a supported boolean' do
+          expect(described_class.str_to_bool(true)).to be(true)
+        end
+      end
+    end
+  end
+end

gems/smithy-client/spec/spec_helper.rb

@@ -4,12 +4,12 @@
 
 require 'simplecov'
 SimpleCov.start do
-  add_filter '/spec/'
-  add_filter 'gems/smithy/'
-  add_filter 'gems/smithy-cbor/'
-  add_filter 'gems/smithy-json/'
-  add_filter 'gems/smithy-schema/'
-  add_filter 'gems/smithy-xml/'
+  skip '/spec/'
+  skip 'gems/smithy/'
+  skip 'gems/smithy-cbor/'
+  skip 'gems/smithy-json/'
+  skip 'gems/smithy-schema/'
+  skip 'gems/smithy-xml/'
 end
 
 require 'smithy'

gems/smithy-schema/sig/smithy-schema/shapes.rbs

@@ -90,7 +90,7 @@ module Smithy
 
       class StructureShape < Shape
         attr_accessor members: Hash[Symbol, MemberShape]
-        attr_accessor type: Class | Struct[untyped]
+        attr_accessor type: (Class | Struct[untyped])?
         def add_member: (Symbol, MemberShape) -> MemberShape
         def member?: (Symbol?) -> bool
         def member: (Symbol) -> MemberShape?
@@ -103,7 +103,7 @@ module Smithy
         attr_accessor members: Hash[Symbol, MemberShape]
         attr_accessor member_types: Hash[Symbol, Class]
         attr_accessor members_by_type: Hash[Class, [Symbol, MemberShape]]
-        attr_accessor type: Class | Struct[untyped]
+        attr_accessor type: (Class | Struct[untyped])?
         def add_member: (Symbol, Class, MemberShape) -> MemberShape
         def member?: (Symbol?) -> bool
         def member: (Symbol) -> MemberShape?

gems/smithy-xml/spec/spec_helper.rb

@@ -2,12 +2,12 @@
 
 require 'simplecov'
 SimpleCov.start do
-  add_filter '/spec/'
-  add_filter 'gems/smithy/'
-  add_filter 'gems/smithy-cbor/'
-  add_filter 'gems/smithy-client/'
-  add_filter 'gems/smithy-json/'
-  add_filter 'gems/smithy-schema/'
+  skip '/spec/'
+  skip 'gems/smithy/'
+  skip 'gems/smithy-cbor/'
+  skip 'gems/smithy-client/'
+  skip 'gems/smithy-json/'
+  skip 'gems/smithy-schema/'
 end
 
 require 'smithy-xml'