rubocop-discourse 3.13.3 → 3.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9dd760fae002d89a13b998aaf4077f886c31d8ea3afd66000b4921760e8e6768
-  data.tar.gz: 31c6b0d5198304e37ee23126d89ed1f0c739557f9e00dd64f7701e7b6797b267
+  metadata.gz: e722863ce072ee0a4d928e09b36a1e12e3b653618fdc0fe70a2f417d466e0ffa
+  data.tar.gz: bd4a868a26bd7df10e3196c9d9c419f1b4268051d6302b335994eb539ae6e61e
 SHA512:
-  metadata.gz: 9bbb4b4dc1209b4408043fad8e6e2ba010c845da3f2b4c7029faa4b0fb0b4c4a9093624945ab08723646c49757d92e80a538c63ed688800c24ec49b1f10786f5
-  data.tar.gz: e03b63f14b554756ed2e39f5c1a8464f4caa6d358b0b009bb547133a56cc622a8b8785d5a1273305b212b16e0b79bc88374110724d803a1ca6fad802c60ae0d1
+  metadata.gz: 0e266174ecceb19102b630bdb01419b0b22580fcadaf5cc5f69a3ba7c7c91a6274ff391a7cf43769c71cb035699e1867d96b0c72ba28cd0706335a5821ce3fe4
+  data.tar.gz: 3e5d42d6aa34bf2eb23daaf91eef254623cd83fe812525c2bda429cd96e9f3ec0b0d0b19129d3faa5e64e749decf0164f012d99755787559d2e715ec3e16441c

data/README.md

@@ -1,11 +1,43 @@
 # rubocop-discourse
 
-## Usage
+Custom Discourse RuboCop cops plus our shared project configuration (RSpec, Rails, Capybara, FactoryBot). Most Discourse projects use Syntax Tree for formatting, so we recommend the Syntax Tree-compatible config by default.
 
-Add `gem "rubocop-discourse"` to your Gemfile.
-Then add the following to `.rubocop.yml`:
+## Installation
+
+Add the gem to your development group:
+
+```ruby
+group :development, :test do
+  gem "rubocop-discourse"
+end
+```
+
+## Configuration
+
+Recommended (Syntax Tree-friendly, omits Layout cops so it can be used with the formatter):
+
+```yml
+inherit_gem:
+  rubocop-discourse: stree-compat.yml
+```
+
+`stree-compat.yml` includes Discourse cops plus core/RSpec/Rails/Capybara/FactoryBot, but leaves layout to Syntax Tree.
+
+Base config with layout cops (for projects not using Syntax Tree):
 
 ```yml
 inherit_gem:
   rubocop-discourse: default.yml
 ```
+
+`default.yml` is kept for backwards compatibility and pulls in `stree-compat.yml` plus `rubocop-layout.yml`.
+
+Then run `bundle exec rubocop` as usual.
+
+Switching an existing project to Syntax Tree:
+
+```diff
+ inherit_gem:
+-  rubocop-discourse: default.yml # includes layout cops
++  rubocop-discourse: stree-compat.yml # defers layout to Syntax Tree
+```

data/lib/rubocop/cop/discourse/plugins/call_requires_plugin.rb

@@ -44,8 +44,7 @@ module RuboCop
 
           def requires_plugin_present_in_parent_classes(node)
             return unless processed_source.path
-            controller_path =
-              base_controller_path(node.parent_class&.const_name.to_s)
+            controller_path = base_controller_path(node.parent_class&.const_name.to_s)
             return unless controller_path
             Commissioner
               .new([self.class.new(config, @options)])

data/lib/rubocop/cop/discourse/plugins/no_monkey_patching.rb

@@ -46,14 +46,11 @@ module RuboCop
         #   add_to_serializer(:user, :new_method) { do_processing }
         #
         class NoMonkeyPatching < Base
-          MSG =
-            "Don’t reopen existing classes. Instead, create a mixin and use `prepend`."
-          MSG_CLASS_EVAL =
-            "Don’t call `class_eval`. Instead, create a mixin and use `prepend`."
+          MSG = "Don’t reopen existing classes. Instead, create a mixin and use `prepend`."
+          MSG_CLASS_EVAL = "Don’t call `class_eval`. Instead, create a mixin and use `prepend`."
           MSG_CLASS_EVAL_SERIALIZERS =
             "Don’t call `class_eval` on a serializer. If you’re adding new methods, use `add_to_serializer`. Otherwise, create a mixin and use `prepend`."
-          MSG_SERIALIZERS =
-            "Don’t reopen serializers. Instead, use `add_to_serializer`."
+          MSG_SERIALIZERS = "Don’t reopen serializers. Instead, use `add_to_serializer`."
           RESTRICT_ON_SEND = [:class_eval].freeze
 
           def_node_matcher :existing_class?, <<~MATCHER
@@ -65,18 +62,14 @@ module RuboCop
           MATCHER
 
           def on_send(node)
-            if serializer?(node)
-              return add_offense(node, message: MSG_CLASS_EVAL_SERIALIZERS)
-            end
+            return add_offense(node, message: MSG_CLASS_EVAL_SERIALIZERS) if serializer?(node)
             add_offense(node, message: MSG_CLASS_EVAL)
           end
 
           def on_class(node)
             return unless in_plugin_rb_file?
             return unless existing_class?(node)
-            if serializer?(node)
-              return add_offense(node, message: MSG_SERIALIZERS)
-            end
+            return add_offense(node, message: MSG_SERIALIZERS) if serializer?(node)
             add_offense(node, message: MSG)
           end
 

data/lib/rubocop/cop/discourse/plugins/use_plugin_instance_on.rb

@@ -16,8 +16,7 @@ module RuboCop
         class UsePluginInstanceOn < Base
           MSG =
             "Use `on` instead of `DiscourseEvent.on` as the latter will listen to events even if the plugin is disabled."
-          NOT_OUTSIDE_PLUGIN_RB =
-            "Don’t call `DiscourseEvent.on` outside `plugin.rb`."
+          NOT_OUTSIDE_PLUGIN_RB = "Don’t call `DiscourseEvent.on` outside `plugin.rb`."
           RESTRICT_ON_SEND = [:on].freeze
 
           def_node_matcher :discourse_event_on?, <<~MATCHER

data/lib/rubocop/cop/discourse/services/empty_lines_around_blocks.rb

@@ -57,18 +57,14 @@ module RuboCop
 
             if missing_empty_lines?(node)
               add_offense(node, message: MSG) do |corrector|
-                if missing_empty_line_before?(node) &&
-                     corrected_after.exclude?(node.left_sibling)
+                if missing_empty_line_before?(node) && corrected_after.exclude?(node.left_sibling)
                   corrected_before << node
                   corrector.insert_before(
-                    node.loc.expression.adjust(
-                      begin_pos: -node.loc.expression.column
-                    ),
-                    "\n"
+                    node.loc.expression.adjust(begin_pos: -node.loc.expression.column),
+                    "\n",
                   )
                 end
-                if missing_empty_line_after?(node) &&
-                     corrected_before.exclude?(node.right_sibling)
+                if missing_empty_line_after?(node) && corrected_before.exclude?(node.right_sibling)
                   corrected_after << node
                   corrector.insert_after(node.loc.end, "\n")
                 end
@@ -91,13 +87,11 @@ module RuboCop
           end
 
           def missing_empty_line_before?(node)
-            processed_source[node.loc.expression.line - 2].present? &&
-              node.left_siblings.present?
+            processed_source[node.loc.expression.line - 2].present? && node.left_siblings.present?
           end
 
           def missing_empty_line_after?(node)
-            processed_source[node.loc.end.line].present? &&
-              node.right_siblings.present?
+            processed_source[node.loc.end.line].present? && node.right_siblings.present?
           end
 
           def corrected_before

data/lib/rubocop/cop/discourse/services/group_keywords.rb

@@ -46,9 +46,7 @@ module RuboCop
 
             add_offense(node, message: MSG) do |corrector|
               range =
-                node.loc.expression.end.with(
-                  end_pos: node.right_sibling.loc.expression.begin_pos
-                )
+                node.loc.expression.end.with(end_pos: node.right_sibling.loc.expression.begin_pos)
               content = range.source.gsub(/^(\n)+/, "\n")
               corrector.replace(range, content)
             end
@@ -62,10 +60,7 @@ module RuboCop
 
           def extra_empty_line_after?(node)
             processed_source[node.loc.expression.line].blank? &&
-              (
-                service_keyword?(node.right_sibling) ||
-                  single_line_block?(node.right_sibling)
-              )
+              (service_keyword?(node.right_sibling) || single_line_block?(node.right_sibling))
           end
 
           def service_keyword?(node)

data/lib/rubocop/discourse/version.rb

@@ -2,6 +2,6 @@
 
 module RuboCop
   module Discourse
-    VERSION = "3.13.3"
+    VERSION = "3.14.0"
   end
 end

data/rubocop-core.yml

@@ -1,143 +1,2 @@
-Security:
-  Enabled: true
-
-Security/IoMethods:
-  Enabled: true
-
-Security/CompoundHash:
-  Enabled: true
-
-# Prefer &&/|| over and/or.
-Style/AndOr:
-  Enabled: true
-
-Style/FrozenStringLiteralComment:
-  Enabled: true
-
-# Use Ruby >= 1.9 syntax for hashes. Prefer { a: :b } over { :a => :b }.
-Style/HashSyntax:
-  Enabled: true
-  EnforcedShorthandSyntax: either
-
-# Defining a method with parameters needs parentheses.
-Style/MethodDefParentheses:
-  Enabled: true
-
-Style/SingleLineMethods:
-  Enabled: true
-
-Style/Semicolon:
-  Enabled: true
-  AllowAsExpressionSeparator: true
-
-Style/RedundantReturn:
-  Enabled: true
-
-Style/GlobalVars:
-  Enabled: true
-  Severity: warning
-  Exclude:
-    - "lib/tasks/**/*"
-    - "script/**/*"
-    - "**/spec/**/*"
-
-Style/InvertibleUnlessCondition:
-  Enabled: true
-
-Style/NegatedUnless:
-  Enabled: true
-
-Style/UnlessElse:
-  Enabled: true
-
-Style/UnlessLogicalOperators:
-  Enabled: true
-
-Lint/Debugger:
-  Enabled: true
-
-# Use my_method(my_arg) not my_method( my_arg ) or my_method my_arg.
-Lint/RequireParentheses:
-  Enabled: true
-
-Lint/ShadowingOuterLocalVariable:
-  Enabled: true
-
-Bundler/OrderedGems:
-  Enabled: false
-
-Layout/LineLength:
-  Enabled: false
-
-Lint/ParenthesesAsGroupedExpression:
-  Enabled: true
-
-Lint/DuplicateMethods:
-  Enabled: true
-
-Lint/RedundantWithIndex:
-  Enabled: true
-
-Lint/RedundantRequireStatement:
-  Enabled: true
-
-Lint/LiteralInInterpolation:
-  Enabled: true
-
-Lint/RedundantStringCoercion:
-  Enabled: true
-
-Lint/BooleanSymbol:
-  Enabled: true
-  AutoCorrect: false # it breaks the code
-
-Lint/ShadowedArgument:
-  Enabled: true
-
-Lint/RedundantCopDisableDirective:
-  Enabled: true
-
-Lint/EmptyEnsure:
-  Enabled: true
-
-Lint/RedundantWithObject:
-  Enabled: true
-
-Lint/SelfAssignment:
-  Enabled: true
-
-Lint/Void:
-  Enabled: true
-
-Lint/UselessMethodDefinition:
-  Enabled: true
-
-Lint/UnreachableCode:
-  Enabled: true
-
-Lint/DeprecatedOpenSSLConstant:
-  Enabled: true
-
-Lint/DisjunctiveAssignmentInConstructor:
-  Enabled: true
-
-Lint/NonLocalExitFromIterator:
-  Enabled: true
-
-Lint/DeprecatedClassMethods:
-  Enabled: true
-
-Lint/SafeNavigationChain:
-  Enabled: true
-
-Lint/RedundantSafeNavigation:
-  Enabled: true
-
-Lint/EmptyConditionalBody:
-  Enabled: true
-
-Lint/SafeNavigationConsistency:
-  Enabled: true
-
-Lint/OrAssignmentToConstant:
-  Enabled: true
+inherit_gem:
+  rubocop-discourse-base: rubocop-core.yml

data/rubocop-discourse.gemspec

@@ -12,12 +12,25 @@ Gem::Specification.new do |s|
   s.license = "MIT"
   s.homepage = "https://github.com/discourse/rubocop-discourse"
 
-  s.files = `git ls-files`.split($/)
+  config_files = %w[
+    default.yml
+    rubocop-core.yml
+    rubocop-layout.yml
+    rubocop-capybara.yml
+    rubocop-factory_bot.yml
+    rubocop-rspec.yml
+    rubocop-rails.yml
+    stree-compat.yml
+    config/default.yml
+  ]
+
+  s.files = %w[LICENSE README.md rubocop-discourse.gemspec] + config_files + Dir["lib/**/*.rb"]
   s.require_paths = ["lib"]
   s.metadata["default_lint_roller_plugin"] = "RuboCop::Discourse::Plugin"
 
+  s.required_ruby_version = ">= 3.0"
   s.add_runtime_dependency "activesupport", ">= 6.1"
-  s.add_runtime_dependency "rubocop", ">= 1.73.2"
+  s.add_runtime_dependency "rubocop-discourse-base", ">= 1.0.0"
   s.add_runtime_dependency "rubocop-rspec", ">= 3.0.1"
   s.add_runtime_dependency "rubocop-factory_bot", ">= 2.27.0"
   s.add_runtime_dependency "rubocop-capybara", ">= 2.22.0"

data/rubocop-layout.yml

@@ -1,80 +1,2 @@
-# Align `when` with `case`.
-Layout/CaseIndentation:
-  Enabled: true
-
-# Align comments with method definitions.
-Layout/CommentIndentation:
-  Enabled: true
-
-# No extra empty lines.
-Layout/EmptyLines:
-  Enabled: true
-
-# Use empty lines between definitions.
-Layout/EmptyLineBetweenDefs:
-  Enabled: true
-
-# Two spaces, no tabs (for indentation).
-Layout/IndentationWidth:
-  Enabled: true
-
-Layout/SpaceAfterColon:
-  Enabled: true
-
-Layout/SpaceAfterComma:
-  Enabled: true
-
-Layout/SpaceAroundEqualsInParameterDefault:
-  Enabled: true
-
-Layout/SpaceAroundKeyword:
-  Enabled: true
-
-Layout/SpaceAroundOperators:
-  Enabled: true
-
-Layout/SpaceBeforeFirstArg:
-  Enabled: true
-
-# Use `foo {}` not `foo{}`.
-Layout/SpaceBeforeBlockBraces:
-  Enabled: true
-
-# Use `foo { bar }` not `foo {bar}`.
-Layout/SpaceInsideBlockBraces:
-  Enabled: true
-
-# Use `{ a: 1 }` not `{a:1}`.
-Layout/SpaceInsideHashLiteralBraces:
-  Enabled: true
-
-Layout/SpaceInsideParens:
-  Enabled: true
-
-# Detect hard tabs, no hard tabs.
-Layout/IndentationStyle:
-  Enabled: true
-
-# Blank lines should not have any spaces.
-Layout/TrailingEmptyLines:
-  Enabled: true
-
-# No trailing whitespace.
-Layout/TrailingWhitespace:
-  Enabled: true
-
-Layout/BlockAlignment:
-  Enabled: true
-
-# Align `end` with the matching keyword or starting expression except for
-# assignments, where it should be aligned with the LHS.
-Layout/EndAlignment:
-  Enabled: true
-  EnforcedStyleAlignWith: variable
-
-Layout/MultilineMethodCallIndentation:
-  Enabled: true
-  EnforcedStyle: indented
-
-Layout/HashAlignment:
-  Enabled: true
+inherit_gem:
+  rubocop-discourse-base: rubocop-layout.yml

data/rubocop-rspec.yml

@@ -1,37 +1,16 @@
-plugins:
-  - rubocop-rspec
-  - rubocop-rspec_rails
+inherit_mode:
+  merge:
+    - plugins
+    - Exclude
+    - Include
 
-RSpec/AnyInstance:
-  Enabled: false # To be decided
-
-RSpec/AroundBlock:
-  Enabled: true
+inherit_gem:
+  rubocop-discourse-base: rubocop-rspec.yml
 
-RSpec/BeforeAfterAll:
-  Enabled: true
-
-RSpec/ChangeByZero:
-  Enabled: true
-
-RSpec/ContextMethod:
-  Enabled: true
-
-RSpec/ContextWording:
-  Enabled: true
-  Prefixes:
-    - when
-    - with
-    - without
-    - for
-    - while
-    - if
-    - as
-    - after
-    - in
+plugins:
+  - rubocop-rspec_rails
 
 RSpec/DescribeClass:
-  Enabled: true
   Exclude:
     - "**/spec/features/**/*"
     - "**/spec/requests/**/*"
@@ -45,194 +24,9 @@ RSpec/DescribeClass:
     - "**/spec/lib/freedom_patches/**/*"
     - "**/spec/multisite/**/*"
 
-RSpec/DescribeMethod:
-  Enabled: true
-
-RSpec/DescribeSymbol:
-  Enabled: false # To be decided
-
-RSpec/DescribedClass:
-  Enabled: false # To be decided
-
-RSpec/DescribedClassModuleWrapping:
-  Enabled: true
-
 RSpec/EmptyExampleGroup:
-  Enabled: true
   Exclude:
     - "spec/requests/api/*"
 
-RSpec/EmptyLineAfterExample:
-  Enabled: false # TODO
-
-RSpec/EmptyLineAfterExampleGroup:
-  Enabled: false # TODO
-
-RSpec/EmptyLineAfterFinalLet:
-  Enabled: false # TODO
-
-RSpec/EmptyLineAfterHook:
-  Enabled: false # TODO
-
-RSpec/EmptyLineAfterSubject:
-  Enabled: true
-
-RSpec/ExampleLength:
-  Enabled: false # To be decided
-
-RSpec/ExampleWithoutDescription:
-  Enabled: true
-
-RSpec/ExampleWording:
-  Enabled: false # TODO
-
-RSpec/ExpectActual:
-  Enabled: true
-
-RSpec/ExpectChange:
-  Enabled: false # To be decided
-
-RSpec/ExpectInHook:
-  Enabled: false
-
-RSpec/ExpectOutput:
-  Enabled: true
-
-RSpec/SpecFilePathFormat:
-  Enabled: false # To be decided
-
-RSpec/SpecFilePathSuffix:
-  Enabled: true
-
-RSpec/Focus:
-  Enabled: true
-
-RSpec/HookArgument:
-  Enabled: false # TODO
-
-RSpec/HooksBeforeExamples:
-  Enabled: false # TODO
-
-RSpec/ImplicitBlockExpectation:
-  Enabled: true
-
-RSpec/ImplicitExpect:
-  Enabled: false # To be decided
-
-RSpec/ImplicitSubject:
-  Enabled: false # To be decided
-
-RSpec/InstanceSpy:
-  Enabled: true
-
-RSpec/InstanceVariable:
-  Enabled: false # TODO
-
-RSpec/ItBehavesLike:
-  Enabled: true
-
-RSpec/IteratedExpectation:
-  Enabled: false # To be decided
-
-RSpec/LeadingSubject:
-  Enabled: true
-
-RSpec/LeakyConstantDeclaration:
-  Enabled: false # To be decided
-
-RSpec/LetBeforeExamples:
-  Enabled: false # TODO
-
-RSpec/LetSetup:
-  Enabled: false # TODO
-
-RSpec/MessageChain:
-  Enabled: true
-
-RSpec/MessageSpies:
-  Enabled: true
-
-RSpec/MissingExampleGroupArgument:
-  Enabled: true
-
-RSpec/MultipleExpectations:
-  Enabled: false
-
-RSpec/MultipleDescribes:
-  Enabled: false # TODO
-
-RSpec/MultipleMemoizedHelpers:
-  Enabled: false
-
-RSpec/MultipleSubjects:
-  Enabled: true
-
-RSpec/NamedSubject:
-  Enabled: true
-
-RSpec/NestedGroups:
-  Enabled: false # To be decided
-
-RSpec/OverwritingSetup:
-  Enabled: true
-
-RSpec/ReceiveCounts:
-  Enabled: true
-
-RSpec/ReceiveNever:
-  Enabled: true
-
-RSpec/RepeatedDescription:
-  Enabled: false
-
-RSpec/RepeatedExample:
-  Enabled: false # TODO
-
-RSpec/RepeatedExampleGroupBody:
-  Enabled: false # TODO
-
-RSpec/RepeatedExampleGroupDescription:
-  Enabled: false
-
-RSpec/ReturnFromStub:
-  Enabled: true
-
-RSpec/ScatteredSetup:
-  Enabled: false # TODO
-
-RSpec/SharedContext:
-  Enabled: true
-
-RSpec/SharedExamples:
-  Enabled: true
-
-RSpec/SingleArgumentMessageChain:
-  Enabled: true
-
-RSpec/SubjectStub:
-  Enabled: true
-
-RSpec/UnspecifiedException:
-  Enabled: true
-
-RSpec/VerifiedDoubles:
-  Enabled: true
-
-RSpec/VoidExpect:
-  Enabled: true
-
-RSpec/Yield:
-  Enabled: true
-
-RSpec/Dialect:
-  Enabled: true
-  PreferredMethods:
-    background: :before
-    scenario: :it
-    xscenario: :xit
-    given: :let
-    given!: :let!
-    feature: :describe
-
 RSpecRails/HttpStatus:
   Enabled: true