boxt_rubocop 2.7.1 → 2.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: da4431fc2e5dd61c47c41cc3e600a32dfdedbd9910fce42f83826ad1ae7a4901
-  data.tar.gz: 5a3e6f1865866d306b64c3e7273919f7b86ae75fbbcf3c76a2c2d7102c7fe8ff
+  metadata.gz: efbb89d5e5ebcae88178adc5256bd18f0c5faa4b8857e5e670484c9f4b19ec0b
+  data.tar.gz: 55d39a7885b15dd4af2da827c487f47d719133c853a40ff6676ddb6cda160595
 SHA512:
-  metadata.gz: 6982d5c1a949cda60701aecc3dedb3db9d1d7ddad288d5a99f573a803cf3a5a79af0a9779f0b84cb8ae55b07d0fc39a661441692f5d5bca909f55f8f49d52f8a
-  data.tar.gz: '0129159bc386f8426e2135844a53e21c2c320b0145d2d10444abccc89c9ba271949f57563c2705ef03a48f3133c361fb8c5a060519b6a666b7d50f9e36c3157e'
+  metadata.gz: f339f801d1c2bccb817933d83febb732547a84cd8e0c2e123dc3d53924312810d85f52e06d5f98452212b6127ee9a84efaff048af80e3f1db3a6082b68b35f73
+  data.tar.gz: 5b58de10c91b936bbe5ab9f58fcf3992a994d95f0fc80551e3ded6b65c842c36f1e9b99044d134ad4e297790ea485837c7caf9c30d4e6b5d5338b4fbca772d11

data/VERSION

@@ -1 +1 @@
-2.7.1
+2.9.0

data/config/default.yml

@@ -11,6 +11,10 @@ Boxt/ApiPathFormat:
   Description: 'Ensure that the API path uses kebab case'
   Enabled: false
 
+Boxt/ApiTypeParameters:
+  Description: 'Ensure that API parameters are typed'
+  Enabled: false
+
 Layout/ClassStructure:
   Enabled: true
 

data/lib/rubocop/cop/boxt/api_type_parameters.rb

@@ -0,0 +1,65 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Boxt
+      # This cop ensures that each parameter in a Grape API has a type specified.
+      #
+      # @example
+      #   # bad
+      #   requires :name
+      #   optional :age
+      #
+      #   # good
+      #   requires :name, type: String
+      #   optional :age, type: Integer
+      #
+      class ApiTypeParameters < Cop
+        API_MESSAGE = "Ensure each parameter has a type specified, e.g., `type: String`."
+        ENTITY_MESSAGE = "Ensure each parameter has a type specified, e.g., `documentation: { type: String }`."
+
+        def_node_matcher :param_declaration, <<-PATTERN
+          (send nil? {:optional :requires :expose} _ $...)
+        PATTERN
+
+        def_node_search :param_with_type, <<-PATTERN
+          (send nil? {:optional :requires} _ (hash <(pair (sym :type) $_)>))
+        PATTERN
+
+        def_node_search :entity_with_type_documentation, <<-PATTERN
+          (send nil? :expose _ (hash <(pair (sym :documentation) (hash <(pair (sym :type) $_)>)) ...>))
+        PATTERN
+
+        def on_send(node)
+          param_declaration(node) do
+            next unless grape_api_class?(node) || grape_entity_class?(node)
+
+            if grape_api_class?(node) && param_with_type(node).none?
+              add_offense(node, message: API_MESSAGE)
+            elsif grape_entity_class?(node) && entity_with_type_documentation(node).none?
+              add_offense(node, message: ENTITY_MESSAGE)
+            end
+          end
+        end
+
+        private
+
+        def grape_api_class?(node)
+          grape_parent_class?(node, "Grape::API")
+        end
+
+        def grape_entity_class?(node)
+          grape_parent_class?(node, "Grape::Entity")
+        end
+
+        def grape_parent_class?(node, class_name)
+          node.each_ancestor(:class).any? do |ancestor|
+            ancestor.children.any? do |child|
+              child&.source == class_name
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/boxt_cops.rb

@@ -1,3 +1,4 @@
 # frozen_string_literal: true
 
 require_relative "boxt/api_path_format"
+require_relative "boxt/api_type_parameters"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: boxt_rubocop
 version: !ruby/object:Gem::Version
-  version: 2.7.1
+  version: 2.9.0
 platform: ruby
 authors:
 - Boxt
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-06-13 00:00:00.000000000 Z
+date: 2024-06-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.21.0
+        version: 1.21.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.21.0
+        version: 1.21.1
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
@@ -86,14 +86,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.31.0
+        version: 3.0.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.31.0
+        version: 3.0.1
 description: Base Rubocop settings for all Boxt Ruby projects
 email:
 - developers@boxt.co.uk
@@ -111,6 +111,7 @@ files:
 - lib/rubocop/boxt/inject.rb
 - lib/rubocop/boxt/version.rb
 - lib/rubocop/cop/boxt/api_path_format.rb
+- lib/rubocop/cop/boxt/api_type_parameters.rb
 - lib/rubocop/cop/boxt_cops.rb
 - rails.yml
 - rspec.yml
@@ -134,7 +135,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.9
+rubygems_version: 3.5.11
 signing_key:
 specification_version: 4
 summary: Base Rubocop settings for all Boxt Ruby projects