Remove rubocop generation

gems/smithy/lib/smithy/generators/client.rb

@@ -35,7 +35,6 @@ def gem_files
           e.yield 'VERSION', Views::Client::Version.new(@plan).render
           e.yield 'CHANGELOG.md', Views::Client::ChangelogMd.new.render
           e.yield "#{@gem_name}.gemspec", Views::Client::Gemspec.new(@plan).render
-          e.yield '.rubocop.yml', Views::Client::RubocopYml.new(@plan).render
 
           source_files.each { |file, content| e.yield file, content }
           e.yield "lib/#{@gem_name}/customizations.rb", Views::Client::Customizations.new.render

gems/smithy/lib/smithy/generators/schema.rb

@@ -35,7 +35,6 @@ def gem_files
           e.yield 'VERSION', Views::Client::Version.new(@plan).render
           e.yield 'CHANGELOG.md', Views::Client::ChangelogMd.new.render
           e.yield "#{@gem_name}.gemspec", Views::Client::Gemspec.new(@plan).render
-          e.yield '.rubocop.yml', Views::Client::RubocopYml.new(@plan).render
 
           source_files.each { |file, content| e.yield file, content }
           e.yield "lib/#{@gem_name}/customizations.rb", Views::Client::Customizations.new.render

gems/smithy/lib/smithy/templates/client/auth_parameters.erb

@@ -19,10 +19,8 @@ module <%= module_name %>
 <% end -%>
     keyword_init: true
   ) do
-
     # @api private
     def self.create(context)
-      # TODO: support more properties
       new(operation_name: context.operation_name)
     end
   end

gems/smithy/lib/smithy/templates/client/endpoint_parameters.erb

@@ -19,7 +19,6 @@ module <%= module_name %>
 <% end -%>
     keyword_init: true
   ) do
-
     def initialize(options = {})
 <% parameters.each do |param| -%>
       self.<%= param.name %> = options.fetch(:<%= param.name %>, <%= param.default_value %>)

gems/smithy/lib/smithy/templates/client/endpoint_plugin.erb

@@ -9,7 +9,7 @@ module <%= module_name %>
       option(
         :endpoint_provider,
         doc_type: '<%= module_name %>::EndpointProvider',
-        docstring: <<~DOCS) do |config|
+        docstring: <<~DOCS) do |_config|
           The endpoint provider used to resolve endpoints. Any object that responds to `#resolve(parameters)`.
         DOCS
         EndpointProvider.new

gems/smithy/lib/smithy/templates/client/errors.erb

@@ -39,7 +39,7 @@ module <%= module_name %>
       # @param [Smithy::Client::HandlerContext] context
       # @param [<%= module_name %>::Types::<%= error.name %>] data
       def initialize(context, data = Smithy::Schema::EmptyStructure.new)
-        super(context, data)
+        super
       end
 <% error.members.each do |member| -%>
 

gems/smithy/lib/smithy/templates/client/gemspec.erb

@@ -4,7 +4,7 @@
 
 Gem::Specification.new do |spec|
   spec.name        = '<%= gem_name %>'
-  spec.version     = File.read(File.expand_path('../VERSION', __FILE__))
+  spec.version     = File.read(File.expand_path('VERSION', __dir__))
   spec.summary     = 'Generated gem using Smithy'
   spec.authors     = ['Smithy Ruby']
   spec.files       = Dir['VERSION', 'CHANGELOG.md', 'lib/**/*.rb', base: __dir__]

gems/smithy/lib/smithy/templates/client/module.erb

@@ -4,7 +4,7 @@
 
 <% requires.each do |require| -%>
 require '<%= require %>'
-<% end %>
+<% end -%>
 
 <% ["Object"].concat(module_names).each_cons(2) do |parent, child| -%>
 <%= parent %>.const_set('<%= child %>', Module.new) unless <%= parent %>.const_defined?('<%= child %>')

gems/smithy/lib/smithy/templates/client/schema.erb

@@ -61,7 +61,6 @@ module <%= module_name %>
 <% end -%>
       end)
 <% end -%>
-
     end
 
     class << self

gems/smithy/lib/smithy/templates/client/types.erb

@@ -19,7 +19,8 @@ module <%= module_name %>
 <% type.members.each do |member| -%>
       :<%= member.name.underscore %>,
 <% end -%>
-      keyword_init: true)
+      keyword_init: true
+    )
       include ::Smithy::Schema::Structure
 <% if !type.input? && type.defaults.any? -%>
 
@@ -40,7 +41,8 @@ module <%= module_name %>
       :<%= member.name.underscore %>,
 <% end -%>
       :unknown,
-      keyword_init: true)
+      keyword_init: true
+    )
       include ::Smithy::Schema::Union
 
 <% type.members.each do |member| -%>

gems/smithy/lib/smithy/templates/client/waiters.erb

@@ -39,4 +39,4 @@ module <%= module_name %>
 
 <% end -%>
   end
-end
+end

gems/smithy/lib/smithy/views/client.rb

@@ -38,7 +38,6 @@ module Client; end
 require_relative 'client/module_rbs'
 require_relative 'client/paginators'
 require_relative 'client/protocol_spec'
-require_relative 'client/rubocop_yml'
 require_relative 'client/schema'
 require_relative 'client/schema_rbs'
 require_relative 'client/spec_helper'

gems/smithy/lib/smithy/views/client/endpoint_provider.rb

@@ -12,10 +12,8 @@ def initialize(plan)
           @model = plan.model
           service = @plan.service.values.first
           @endpoint_rules = service['traits']['smithy.rules#endpointRuleSet']
-          @parameters = @endpoint_rules['parameters']
-                        .map { |id, data| EndpointParameter.new(id, data, @plan) }
-          @endpoint_function_bindings =
-            plan.welds.map(&:endpoint_function_bindings).reduce({}, :merge)
+          @parameters = @endpoint_rules['parameters'].map { |id, data| EndpointParameter.new(id, data, @plan) }
+          @endpoint_function_bindings = plan.welds.map(&:endpoint_function_bindings).reduce({}, :merge)
           @assigned_variables = []
           super()
         end
@@ -116,7 +114,9 @@ def error_rule_with_condition(levels, rule)
         end
 
         def error(error, levels)
-          indent("raise ArgumentError, #{str(error)}\n", levels)
+          error_str = "raise ArgumentError, #{str(error)}"
+          error_str += "\n" unless levels == 3
+          indent(error_str, levels)
         end
 
         def tree_rule(rule, levels = 3)

gems/smithy/lib/smithy/views/client/gemspec.rb

@@ -19,7 +19,7 @@ def dependencies
           dependencies = dependencies.except(@plan.welds.map(&:remove_dependencies).reduce([], :+))
           dependencies.merge!(
             if @plan.type == :schema
-              { 'smithy-schema' => '~> 1' }
+              { 'smithy-schema' => '1.0.0.pre1' }
             else
               { 'smithy-client' => '1.0.0.pre1' }
             end

gems/smithy/lib/smithy/views/client/paginators.rb

@@ -51,7 +51,7 @@ def initialize(name, trait)
           def next_tokens_code
             next_token_getter = output_getter(@output_token)
             code = ["next_token = #{next_token_getter}"]
-            code << 'return {} if next_token.nil? || next_token.empty?'
+            code << "return {} if next_token.nil? || next_token.empty?\n"
             code << 'tokens = Hash.new { |h, k| h[k] = {} }'
             next_token_setter = input_getter(@input_token, 'tokens')
             code << "#{next_token_setter} = next_token"
@@ -62,7 +62,7 @@ def next_tokens_code
           def prev_tokens_code
             prev_token_getter = input_getter(@input_token)
             code = ["prev_token = #{prev_token_getter}"]
-            code << 'return {} if prev_token.nil? || prev_token.empty?'
+            code << "return {} if prev_token.nil? || prev_token.empty?\n"
             code << 'tokens = Hash.new { |h, k| h[k] = {} }'
             prev_token_setter = input_getter(@input_token, 'tokens')
             code << "#{prev_token_setter} = prev_token"

gems/smithy/lib/smithy/welds.rb

@@ -11,9 +11,6 @@
 require_relative 'welds/default_plugins'
 require_relative 'welds/protocols'
 
-# Disabled due to performance
-# require_relative 'welds/rubocop'
-
 module Smithy
   # @api private
   module Welds

gems/smithy/spec/support/examples/gemspec_examples.rb

@@ -28,7 +28,7 @@
       expect(gem.files).to include("lib/#{gem_name}/types.rb", "lib/#{gem_name}/schema.rb")
       expect(gem.license).to eq('Apache-2.0')
       dependency = context.include?('schema') ? 'smithy-schema' : 'smithy-client'
-      expect(gem.dependencies).to include(Gem::Dependency.new(dependency, '~> 1'))
+      expect(gem.dependencies).to include(Gem::Dependency.new(dependency, '1.0.0.pre1'))
       expect(gem.required_ruby_version).to eq(Gem::Requirement.new('>= 3.3'))
     end
   end
@@ -60,7 +60,7 @@
       expect(gem.files).to include("lib/#{gem_name}/types.rb", "lib/#{gem_name}/schema.rb")
       expect(gem.license).to eq('Apache-2.0')
       dependency = context.include?('schema') ? 'smithy-schema' : 'smithy-client'
-      expect(gem.dependencies).to include(Gem::Dependency.new(dependency, '~> 1'))
+      expect(gem.dependencies).to include(Gem::Dependency.new(dependency, '1.0.0.pre1'))
       expect(gem.required_ruby_version).to eq(Gem::Requirement.new('>= 3.3'))
     end
   end

projections/shapes/lib/shapes.rb

@@ -4,7 +4,6 @@
 
 require 'smithy-client'
 
-
 Object.const_set('ShapeService', Module.new) unless Object.const_defined?('ShapeService')
 
 module ShapeService

projections/shapes/lib/shapes/auth_parameters.rb

@@ -13,10 +13,8 @@ module ShapeService
     :operation_name,
     keyword_init: true
   ) do
-
     # @api private
     def self.create(context)
-      # TODO: support more properties
       new(operation_name: context.operation_name)
     end
   end

projections/shapes/lib/shapes/endpoint_parameters.rb

@@ -13,7 +13,6 @@ module ShapeService
     :endpoint,
     keyword_init: true
   ) do
-
     def initialize(options = {})
       self.endpoint = options.fetch(:endpoint, nil)
     end

projections/shapes/lib/shapes/endpoint_provider.rb

@@ -13,7 +13,6 @@ def resolve(parameters)
         return Smithy::Client::EndpointRules::Endpoint.new(uri: parameters.endpoint)
       end
       raise ArgumentError, "Endpoint is not set - you must configure an endpoint."
-
     end
   end
 end

projections/shapes/lib/shapes/plugins/endpoint.rb

@@ -9,7 +9,7 @@ class Endpoint < Smithy::Client::Plugin
       option(
         :endpoint_provider,
         doc_type: 'ShapeService::EndpointProvider',
-        docstring: <<~DOCS) do |config|
+        docstring: <<~DOCS) do |_config|
           The endpoint provider used to resolve endpoints. Any object that responds to `#resolve(parameters)`.
         DOCS
         EndpointProvider.new

projections/shapes/lib/shapes/schema.rb

@@ -93,7 +93,6 @@ module Schema
         operation.output = ::Smithy::Schema::Shapes::ShapeRef.new(shape: OperationOutput)
         operation.traits = {"smithy.ruby.tests#shape" => {}}
       end)
-
     end
 
     class << self