gl_rubocop 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 83a8c15ce9c97fee3534fd6ec1b13efa8c150b4a6a06042f1408be213022c453
-  data.tar.gz: aef1892a704ffc69e5b4fc5ecc35d3c45461a8cac740c07a635e09a46811895b
+  metadata.gz: 55c5e4d549add201fba2d9cdcf5c63b3adf2fff339785be177d2b992c841750a
+  data.tar.gz: 50d1687524ca84b65fca2ab5e8d9502446991aa69b3d0b06f9711d614393785d
 SHA512:
-  metadata.gz: 796c52aff461d138901b41715b5d677518057ae3cb81715f39036f4e4c92c9c1294732d5e149160addc245eee87065b7fc49ffaad3c936ae0d586f5263dedf42
-  data.tar.gz: 35f95f56ace1f7c2e529df369e5d397b7a6cfd741be5a4a7d53fb7a9ffd849b200919343a0006586cb772403b81d5369c3bda5e406fa924de127ec1c656e5f76
+  metadata.gz: 3ddaebd810428c15841487137a7f692a17124bd95278244422107540690858398c69667111102f9f9d43e18a5e6326eef3e95fe6e77ff5341fbcdb6844b64aef
+  data.tar.gz: dd7ad7ea6d9f11648bf103da2e743540c6709444e28f926eb6d4692d768a4ae5a6c3da37f023b5ab9f08305fb7c6c52e91a43a57ec89e69555bf716564fa5432

data/default.yml

@@ -21,6 +21,9 @@ require:
   - ./lib/gl_rubocop/gl_cops/valid_data_test_id.rb
   - ./lib/gl_rubocop/gl_cops/vcr_cassette_names.rb
   - ./lib/gl_rubocop/gl_cops/view_component_initialize_keyword_args.rb
+  - ./lib/gl_rubocop/gl_cops/view_component_class_naming.rb
+  - ./lib/gl_rubocop/gl_cops/view_component_inheritance.rb
+  - ./lib/gl_rubocop/gl_cops/view_component_directory_structure.rb
 
 AllCops:
   SuggestExtensions: false
@@ -96,6 +99,21 @@ GLCops/VcrCassetteNames:
   Include:
     - "**/*spec.rb"
 
+GLCops/ViewComponentClassNaming:
+  Enabled: true
+  Include:
+    - "app/components/**/*.rb"
+
+GLCops/ViewComponentDirectoryStructure:
+  Enabled: true
+  Include:
+    - "app/components/**/component.rb"
+
+GLCops/ViewComponentInheritance:
+  Enabled: true
+  Include:
+    - "app/components/**/component.rb"
+
 GLCops/ViewComponentInitializeKeywordArgs:
   Enabled: true
   Include:

data/lib/gl_rubocop/gl_cops/view_component_class_naming.rb

@@ -0,0 +1,25 @@
+module GLRubocop
+  module GLCops
+    # This cop checks that the class name is "Component" or "ApplicationViewComponent".
+    #
+    # Good:
+    #   class Component < ViewComponent::Base
+    #   end
+    #
+    #   class ApplicationViewComponent < ViewComponent::Base
+    #   end
+    #
+    # Bad:
+    #   class UserCardComponent < ViewComponent::Base
+    #   end
+    class ViewComponentClassNaming < RuboCop::Cop::Base
+      def on_class(node)
+        class_name = node.identifier.const_name
+        return true if class_name == 'Component'
+        return true if class_name == 'ApplicationViewComponent'
+
+        add_offense(node, message: 'ViewComponent class names must be "Component".')
+      end
+    end
+  end
+end

data/lib/gl_rubocop/gl_cops/view_component_directory_structure.rb

@@ -0,0 +1,33 @@
+module GLRubocop
+  module GLCops
+    # This cop checks that all ViewComponent is part of an allowlisted base module.
+    #
+    # Good:
+    #   module Core
+    #     module Users
+    #       class Component < ApplicationViewComponent
+    #       end
+    #     end
+    #   end
+    #
+    # Bad:
+    #   module Billing
+    #     class Component < ApplicationViewComponent
+    #     end
+    #   end
+    class ViewComponentDirectoryStructure < RuboCop::Cop::Base
+      MSG = 'ViewComponent must belong to an allowed base module: %<allowed>s'.freeze
+      ALLOWED_MODULES = %w[Core Admin NonprofitAdmin Packs Users].freeze
+
+      def on_class(node)
+        return true if node.identifier.const_name == 'ApplicationViewComponent'
+
+        base_module = node.parent_module_name&.split('::')&.first
+        return true if base_module.nil?
+        return true if ALLOWED_MODULES.include?(base_module)
+
+        add_offense(node, message: format(MSG, allowed: ALLOWED_MODULES.join(', ')))
+      end
+    end
+  end
+end

data/lib/gl_rubocop/gl_cops/view_component_inheritance.rb

@@ -0,0 +1,38 @@
+module GLRubocop
+  module GLCops
+    # This cop checks that all ViewComponent classes inherit from an allowlisted base class.
+    #
+    # Good:
+    #   class Components::HeroComponent < ApplicationViewComponent
+    #   end
+    #
+    #   class Components::CardComponent < ViewComponent::Base
+    #   end
+    #
+    # Bad:
+    #   class Components::HeroComponent < ViewComponent
+    #   end
+    #
+    #   class Components::CardComponent
+    #   end
+    class ViewComponentInheritance < RuboCop::Cop::Base
+      INHERITANCE_MSG = 'ViewComponent must inherit from ApplicationViewComponent'.freeze
+
+      def on_class(node)
+        return true if inherits_from_application_view_component(node)
+
+        add_offense(node, message: INHERITANCE_MSG)
+      end
+
+      private
+
+      def inherits_from_application_view_component(node)
+        parent = node.parent_class
+        return false unless parent
+
+        parent_name = parent.const_name
+        %w[ApplicationViewComponent ViewComponent::Base].include?(parent_name)
+      end
+    end
+  end
+end

data/lib/gl_rubocop/version.rb

@@ -1,3 +1,3 @@
 module GLRubocop
-  VERSION = '0.5.0'.freeze
+  VERSION = '0.5.1'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gl_rubocop
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Give Lively
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-03-03 00:00:00.000000000 Z
+date: 2026-04-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -174,6 +174,9 @@ files:
 - lib/gl_rubocop/gl_cops/unique_identifier.rb
 - lib/gl_rubocop/gl_cops/valid_data_test_id.rb
 - lib/gl_rubocop/gl_cops/vcr_cassette_names.rb
+- lib/gl_rubocop/gl_cops/view_component_class_naming.rb
+- lib/gl_rubocop/gl_cops/view_component_directory_structure.rb
+- lib/gl_rubocop/gl_cops/view_component_inheritance.rb
 - lib/gl_rubocop/gl_cops/view_component_initialize_keyword_args.rb
 - lib/gl_rubocop/helpers/erb_content_helper.rb
 - lib/gl_rubocop/helpers/haml_content_helper.rb
@@ -198,7 +201,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.22
+rubygems_version: 3.3.26
 signing_key:
 specification_version: 4
 summary: A shareable configuration of Give Lively's rubocop rules.