boxt_rubocop 2.7.1 → 2.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: da4431fc2e5dd61c47c41cc3e600a32dfdedbd9910fce42f83826ad1ae7a4901
-  data.tar.gz: 5a3e6f1865866d306b64c3e7273919f7b86ae75fbbcf3c76a2c2d7102c7fe8ff
+  metadata.gz: 5088d5c0f8a597d082572e25832ca5f401a3e6487b8d944b606fd7162a612f90
+  data.tar.gz: 4791d465fddfc8c8a363a4fa9078cac6bf20655ec356c5f3ff8c075e96a9b51c
 SHA512:
-  metadata.gz: 6982d5c1a949cda60701aecc3dedb3db9d1d7ddad288d5a99f573a803cf3a5a79af0a9779f0b84cb8ae55b07d0fc39a661441692f5d5bca909f55f8f49d52f8a
-  data.tar.gz: '0129159bc386f8426e2135844a53e21c2c320b0145d2d10444abccc89c9ba271949f57563c2705ef03a48f3133c361fb8c5a060519b6a666b7d50f9e36c3157e'
+  metadata.gz: 434b1a3bd1911a39bfc0a5eb54ecb553fcd437ddb50d5892745f272e9fe241402a01d9e9a6de927b720bfa280364d30dc3392200bdcaea3be77c66c2c3e1b662
+  data.tar.gz: df1a0a656f781d196bd0cae1900bc5224cf8dea75af518cde5b3fcb4721f1fdbd16b0bda2898125ae1bb4e0682c052d7a2c9cb9a6349608970b3b58f2a8c8659

data/VERSION

@@ -1 +1 @@
-2.7.1
+2.8.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.8.0
 platform: ruby
 authors:
 - Boxt
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-06-13 00:00:00.000000000 Z
+date: 2024-06-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -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