runger_style 4.2.0 → 4.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: da6291e7956c095b8e8f891f07c63879ca8691bfedebdc27ec44bf83b1429691
-  data.tar.gz: 5e772363e0dd0d10e33fd1b0fd1c249b9e0a37212dba4cedd45aeb06be7cf05e
+  metadata.gz: e07fef8f115633d528bdd3ba1dd11e486ddb30ce5d7de78546d09821340f200d
+  data.tar.gz: 608b9111baa0c4a043e18cba9de3bfb9171016fb3d2154b8ef5e2f175b5542dc
 SHA512:
-  metadata.gz: 16c79aa8c9a343c321d0bd280d5d757755bff6103bdcbd6267157f1617c53d5ceba66bcc7a910b913c5f76872ba083dabbabfba1f6b9c1d671d79ca39c705a43
-  data.tar.gz: b0809a24102e25b4557c8a7ff9c293b16dbdf46cff85b67e7da66c7e0f654ac15a7884105ace85be9d3d8df62958fddf7ead57d5b2a5338fbecd1cf9b800d45f
+  metadata.gz: 7639738ec440dfa2a8e792f85df5d6bf4b195ae1d633ad11fdc2ebc8add0c2bc341902eda81af1aed06afda378809fbf3c9e153de00d3b28e3275c3ce8712eb6
+  data.tar.gz: 77956254b806844912a1c3047a9b6b339a742d806799820f8997a3300a43764556cbc136bcbb8f53c5a09ccff94dfde79acae30982f3f03a6e642f6c770d970c

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+## v4.2.2 (2025-01-27)
+- Fix `RungerStyle/MultilineMethodArgumentsLineBreaks` false positives.
+
+## v4.2.1 (2025-01-27)
+- Explicitly declare `module RungerStyle` in `RungerStyle/MultilineMethodArgumentsLineBreaks` definition.
+
 ## v4.2.0 (2025-01-27)
 - Add `RungerStyle/MultilineMethodArgumentsLineBreaks` cop.
 

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    runger_style (4.2.0)
+    runger_style (4.2.2)
       prism (>= 0.24.0)
       rubocop (>= 1.68.0)
 
@@ -189,7 +189,7 @@ CHECKSUMS
   rubocop-rspec_rails (2.30.0) sha256=888112e83f9d7ef7ad2397e9d69a0b9614a4bae24f072c399804a180f80c4c46
   ruby-progressbar (1.13.0) sha256=80fc9c47a9b640d6834e0dc7b3c94c9df37f08cb072b7761e4a71e22cff29b33
   runger_release_assistant (0.14.0) sha256=7abc5eb1e0f63d8d611691bc21698d2b3b8f148a5f47cacd8dd32c21d16ee9be
-  runger_style (4.2.0)
+  runger_style (4.2.2)
   securerandom (0.4.1) sha256=cc5193d414a4341b6e225f0cb4446aceca8e50d5e1888743fac16987638ea0b1
   slop (4.10.1) sha256=844322b5ffcf17ed4815fdb173b04a20dd82b4fd93e3744c88c8fafea696d9c7
   stringio (3.1.2) sha256=204f1828f85cdb39d57cac4abc6dc44b04505a223f131587f2e20ae3729ba131

data/lib/runger_style/cops/multiline_method_arguments_line_breaks.rb

@@ -1,41 +1,48 @@
 # frozen_string_literal: true
 
-class RungerStyle::MultilineMethodArgumentsLineBreaks < RuboCop::Cop::Base
-  extend RuboCop::Cop::AutoCorrector
-  include RuboCop::Cop::RangeHelp
+module RungerStyle # rubocop:disable Style/ClassAndModuleChildren
+  class MultilineMethodArgumentsLineBreaks < ::RuboCop::Cop::Base
+    extend RuboCop::Cop::AutoCorrector
+    include RuboCop::Cop::RangeHelp
 
-  MSG = 'Each argument in a multi-line method call must start on a separate line.'
+    MSG = 'Each argument in a multi-line method call must start on a separate line.'
 
-  def on_send(node)
-    return unless node.multiline?
-    return if node.arguments.size <= 1
+    def on_send(node)
+      return if node.arguments.size <= 1
+      return unless node.parenthesized?
+      return unless multiline_parentheses?(node)
 
-    node.arguments.each_cons(2) do |arg1, arg2|
-      next unless same_line?(arg1, arg2)
+      node.arguments.each_cons(2) do |arg1, arg2|
+        next unless same_line?(arg1, arg2)
 
-      separator = separator_range(arg1, arg2)
+        separator = separator_range(arg1, arg2)
 
-      add_offense(separator, message: MSG) do |corrector|
-        base_indent = base_indentation(arg1)
-        replacement = ",\n#{base_indent}"
-        corrector.replace(separator, replacement)
+        add_offense(separator, message: MSG) do |corrector|
+          base_indent = base_indentation(arg1)
+          replacement = ",\n#{base_indent}"
+          corrector.replace(separator, replacement)
+        end
       end
     end
-  end
 
-  private
+    private
 
-  def same_line?(arg1, arg2)
-    arg1.source_range.last_line == arg2.source_range.first_line
-  end
+    def multiline_parentheses?(node)
+      node.loc.begin.line != node.loc.end.line
+    end
 
-  def base_indentation(arg)
-    arg.source_range.source_line[/^\s*/]
-  end
+    def same_line?(arg1, arg2)
+      arg1.source_range.last_line == arg2.source_range.first_line
+    end
+
+    def base_indentation(arg)
+      arg.source_range.source_line[/^\s*/]
+    end
 
-  def separator_range(arg1, arg2)
-    end_pos = arg1.source_range.end.end_pos
-    begin_pos = arg2.source_range.begin.begin_pos
-    range_between(end_pos, begin_pos)
+    def separator_range(arg1, arg2)
+      end_pos = arg1.source_range.end.end_pos
+      begin_pos = arg2.source_range.begin.begin_pos
+      range_between(end_pos, begin_pos)
+    end
   end
 end

data/lib/runger_style/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RungerStyle
-  VERSION = '4.2.0'
+  VERSION = '4.2.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: runger_style
 version: !ruby/object:Gem::Version
-  version: 4.2.0
+  version: 4.2.2
 platform: ruby
 authors:
 - David Runger