rufo 0.17.0 → 0.17.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2415782555ad95778ec90751f3c438320fb1797c44213c458c0d0a7f956cf2fc
-  data.tar.gz: 6e2bed742d8efedef98b59b74acaef679764eeb5911e6fc0b596da1a9cec7807
+  metadata.gz: f5df151931108df0740d8602c1bf8b931503e3bede6a337f25a9bf75cb549c70
+  data.tar.gz: 73a321c4a0960f8fc021b062149ccbe3f96ccfc231c780e2612de00e011f590b
 SHA512:
-  metadata.gz: f103da0397ad0415a031601e14016d0080c9c22cac9d4b3af45a01f7f32a6fd851118d871d09272a8c41c7373ab8e77caac81b815899634eac66941e1f67121c
-  data.tar.gz: dcc1b6e6430818500a3238c1d403dd23abdfc840757318e1746bd388e0d1f4ce9a0df12a232bad5f6691dc69cd9cf6421ea1c74df8ed58c29453ac5260c91c4b
+  metadata.gz: 8a2176ae1780bce5ec076052b63576fc54370a4a5b2a7837166e8d42ec4f6f4f47cc1b4590f5742a6b7a263d7cb2b59fc7985a2dd9f1a37fa41939c502515253
+  data.tar.gz: aad26fad18830aad211ba7d16f65643b78648b4f755d0a43ac1b65edbba1d7f97e05535f7c2f499dc3a5a7fa2cfff295e2a5da1aa63af859090873598406ab60

data/CHANGELOG.md

@@ -12,6 +12,17 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ### Added
 
+## [0.17.1] - 2024-02-03
+
+### Fixed
+- Fix incorrect indentation methods call with splat ([#320](https://github.com/ruby-formatter/rufo/pull/320))
+
+### Changed
+- Remove unused internal methods from `Rufo::Formatter` ([#321](https://github.com/ruby-formatter/rufo/pull/321))
+  - This change may affect behavior with versions of Ruby not currently supported by Rufo (< 2.7)
+
+### Added
+
 ## [0.17.0] - 2024-01-06
 
 ### Fixed

data/bin/verify-sample-code

@@ -1,9 +1,8 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 repos = {
-  "lostisland/faraday" => {
-    "--format" => "progress",
-  },
+  "rspec/rspec-expectations" => {},
+  "rspec/rspec-mocks" => {},
 }
 
 def run_command(cmd, allowed_statuses: [0])

data/lib/rufo/formatter.rb

@@ -396,8 +396,6 @@ class Rufo::Formatter
       visit_module(node)
     when :mrhs_new_from_args
       visit_mrhs_new_from_args(node)
-    when :mlhs_paren
-      visit_mlhs_paren(node)
     when :mlhs
       visit_mlhs(node)
     when :mrhs_add_star
@@ -457,8 +455,6 @@ class Rufo::Formatter
       visit_alias(node)
     when :undef
       visit_undef(node)
-    when :mlhs_add_star
-      visit_mlhs_add_star(node)
     when :rest_param
       visit_rest_param(node)
     when :kwrest_param
@@ -906,26 +902,6 @@ class Rufo::Formatter
     end
   end
 
-  def indentable_value?(value)
-    return unless current_token_kind == :on_kw
-
-    case current_token_value
-    when "if", "unless", "case"
-      true
-    when "begin"
-      # Only indent if it's begin/rescue
-      return false unless value[0] == :begin
-
-      body = value[1]
-      return false unless body[0] == :bodystmt
-
-      _, _, rescue_body, else_body, ensure_body = body
-      rescue_body || else_body || ensure_body
-    else
-      false
-    end
-  end
-
   def current_comment_aligned_to_previous_one?
     @last_comment &&
       @last_comment[0][0] + 1 == current_token_line &&
@@ -1541,7 +1517,8 @@ class Rufo::Formatter
 
     skip_space
 
-    write_params_comma if comma?
+    # Disable indentation in write_params_comma to avoid double indentation with write_indent
+    write_params_comma(with_indent: !needs_indent) if comma?
     write_indent(base_column) if needs_indent
     consume_op "*"
     skip_space_or_newline
@@ -1551,7 +1528,8 @@ class Rufo::Formatter
     end
 
     if post_args && !post_args.empty?
-      write_params_comma
+      # Disable indentation in write_params_comma to avoid double indentation with visit_comma_separated_list
+      write_params_comma(with_indent: !needs_indent)
       visit_comma_separated_list post_args, needs_indent: needs_indent, base_column: base_column
     end
   end
@@ -1659,23 +1637,10 @@ class Rufo::Formatter
     end
   end
 
-  def visit_mlhs_paren(node)
-    # [:mlhs_paren,
-    #   [[:mlhs_paren, [:@ident, "x", [1, 12]]]]
-    # ]
-    _, args = node
-
-    visit_mlhs_or_mlhs_paren(args)
-  end
-
   def visit_mlhs(node)
     # [:mlsh, *args]
     _, *args = node
 
-    visit_mlhs_or_mlhs_paren(args)
-  end
-
-  def visit_mlhs_or_mlhs_paren(args)
     # Sometimes a paren comes, some times not, so act accordingly.
     has_paren = current_token_kind == :on_lparen
     if has_paren
@@ -1819,35 +1784,6 @@ class Rufo::Formatter
     end
   end
 
-  def visit_mlhs_add_star(node)
-    # [:mlhs_add_star, before, star, after]
-    _, before, star, after = node
-
-    if before && !before.empty?
-      # Maybe a Ripper bug, but if there's something before a star
-      # then a star shouldn't be here... but if it is... handle it
-      # somehow...
-      if op?("*")
-        star = before
-      else
-        visit_comma_separated_list to_ary(before)
-        write_params_comma
-      end
-    end
-
-    consume_op "*"
-
-    if star
-      skip_space_or_newline
-      visit star
-    end
-
-    if after && !after.empty?
-      write_params_comma
-      visit_comma_separated_list after
-    end
-  end
-
   def visit_rest_param(node)
     # [:rest_param, name]
 
@@ -2250,12 +2186,14 @@ class Rufo::Formatter
     end
   end
 
-  def write_params_comma
+  def write_params_comma(with_indent: true)
     skip_space
     check :on_comma
     write ","
     next_token
-    skip_space_or_newline_using_setting(:one)
+
+    indent_size = with_indent ? @indent : 0
+    skip_space_or_newline_using_setting(:one, indent_size)
   end
 
   def visit_array(node)

data/lib/rufo/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Rufo
-  VERSION = "0.17.0"
+  VERSION = "0.17.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rufo
 version: !ruby/object:Gem::Version
-  version: 0.17.0
+  version: 0.17.1
 platform: ruby
 authors:
 - Ary Borenszweig
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-01-06 00:00:00.000000000 Z
+date: 2024-02-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -210,7 +210,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.10
+rubygems_version: 3.5.3
 signing_key:
 specification_version: 4
 summary: Ruby code formatter