rubocop-packs 0.0.27 → 0.0.28

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e25f6b7faaf75d2877d830e784211d96ba5d6cb29b442c959552e1de28c5adbd
-  data.tar.gz: eff6ee7fe761edb2bb142f4cf543557b4d17a160990dccc364972bd87208ccd8
+  metadata.gz: e4888c0b36e59d1da1aaa62d1390da897fce811909d99c5e1189bca0efd36050
+  data.tar.gz: 7d1957e8c598a6acbcfa3b0b8f49578dcb3424d01c20178c2045459654a2478f
 SHA512:
-  metadata.gz: cd2ffeb81ef84b49be4ae73216d3318f45d1bf9ad70bcc77061509bbae0f848a1ff0a87ba12082c6091f32b00fef0a1a303d360dba7d07a8c2de60e255186888
-  data.tar.gz: 52163c30ba8037b43c0aaa1659bf42723e4dcd0b570b5b6e7d0f2e72e526a2e3fab6456ed3c9e934bad894b3aed9b9e3103b908dbe9d012be2081c5d42312022
+  metadata.gz: b55558140df2b0d0e68f64668a94e814c30a6ac42b144d0dc7fbc7118bdf982fb87b45085592ca219946ae6a2df73bb22562a83ae888fb021343ec0cea7c685c
+  data.tar.gz: fa54dc58358e0f32104c2acfa0536ff6239430c287a75e5ad0a346ae9979360f59a67037a34599e98a1df525cbe6c31c7350af95b6c33dba1dcdacb02cbdfe88

data/config/default.yml

@@ -5,6 +5,7 @@ Packs/ClassMethodsAsPublicApis:
     - T::Struct
     - Struct
     - OpenStruct
+  AcceptableMixins: []
   FailureMode: default
 
 Packs/RootNamespaceIsPackName:

data/lib/rubocop/cop/packs/class_methods_as_public_apis.rb

@@ -10,6 +10,7 @@ module RuboCop
       # Options:
       #
       # * `AcceptableParentClasses`: A list of classes that, if inherited from, non-class methods are permitted (useful when value objects are a part of your public API)
+      # * `AcceptableMixins`: A list of modules that, if included, non-class methods are permitted
       #
       # @example
       #
@@ -47,11 +48,14 @@ module RuboCop
 
           acceptable_parent_classes = cop_config['AcceptableParentClasses'] || []
 
-          # Used this PR as inspiration to check if we're within a `class << self` block
           uses_implicit_static_methods = node.each_ancestor(:sclass).first&.identifier&.source == 'self'
           class_is_allowed_to_have_instance_methods = acceptable_parent_classes.include?(parent_class&.const_name)
           return if uses_implicit_static_methods || class_is_allowed_to_have_instance_methods
 
+          is_sorbet_interface_or_abstract_class = !module_node.nil? && module_node.descendants.any? { |d| d.is_a?(RuboCop::AST::SendNode) && (d.method_name == :interface! || d.method_name == :abstract!) }
+          return if is_sorbet_interface_or_abstract_class
+          return if node_includes_acceptable_mixin?(class_node || module_node)
+
           add_offense(
             node.source_range,
             message: format(
@@ -59,6 +63,22 @@ module RuboCop
             )
           )
         end
+
+        private
+
+        sig { params(node: T.untyped).returns(T::Boolean) }
+        def node_includes_acceptable_mixin?(node)
+          acceptable_mixins = cop_config['AcceptableMixins'] || []
+          return false if node.nil?
+
+          node.descendants.any? do |d|
+            d.is_a?(RuboCop::AST::SendNode) &&
+              d.method_name == :include &&
+              d.arguments.count == 1 &&
+              d.arguments.first.is_a?(RuboCop::AST::ConstNode) &&
+              acceptable_mixins.include?(d.arguments.first.const_name)
+          end
+        end
       end
     end
   end

data/lib/rubocop/packs.rb

@@ -51,9 +51,11 @@ module RuboCop
           rubocop_todo = {}
         end
 
-        offenses_for_pack.each do |offense|
-          rubocop_todo[offense.cop_name] ||= { 'Exclude' => [] }
-          rubocop_todo[offense.cop_name]['Exclude'] << offense.filepath
+        offenses_for_pack.group_by(&:filepath).each do |filepath, offenses_by_filepath|
+          offenses_by_filepath.map(&:cop_name).uniq.each do |cop_name|
+            rubocop_todo[cop_name] ||= { 'Exclude' => [] }
+            rubocop_todo[cop_name]['Exclude'] << filepath
+          end
         end
 
         next if rubocop_todo.empty?

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-packs
 version: !ruby/object:Gem::Version
-  version: 0.0.27
+  version: 0.0.28
 platform: ruby
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-11-19 00:00:00.000000000 Z
+date: 2022-11-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -178,7 +178,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: Fill this out!
+description: A collection of Rubocop rules for gradually modularizing a ruby codebase
 email:
 - dev@gusto.com
 executables: []
@@ -230,5 +230,5 @@ requirements: []
 rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
-summary: Fill this out!
+summary: A collection of Rubocop rules for gradually modularizing a ruby codebase
 test_files: []