rfmt 1.6.0-aarch64-linux → 1.6.1-aarch64-linux

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6aa4ee193a6d3771e9fdf0c73c7bd930640e8aacbf01656c21a40392c9ea6304
-  data.tar.gz: b3449cbc83b606d8d13bf8e9235f02a1f6b6e7441e55dcfd50de4c1e14b8139e
+  metadata.gz: e36c79a715864d1545b7646e65a9de6afe9d2502fdc60702fe595c9c94fccba0
+  data.tar.gz: 40c48ce63e3d9d7f64593635dd13217079eab96e209861fe5529f715160c8ea3
 SHA512:
-  metadata.gz: 926ea7ec34c5cab902782f7d94313b7c66515e5763d65a4d493be558416e137a09cb2c94c90b92a93fa74c21df38b59466fa7cf807f37a0693dea53946eb2263
-  data.tar.gz: 8c7d31435fa536fc516243869de9c2f37f910a5dbfd280a7c613e9bf3b7db8b1fe9429800bbaf8844a432457d26087b515a6429594b2dcce46e8b3600f49bbeb
+  metadata.gz: d0e411f31a9c4c49be7cff8ecbe7f15f2196b4a005888eecdf206a7fdf6ba098e85fae7ce5ec8675920a43eb7a0fe22687930660b66a88c411d5353dcf3fbd23
+  data.tar.gz: 969b21932415241c2868e1adcda98e9c7e98942522c7aacb025fbaaa53b6543665142cebcdaaead4ff86c4f360e9d17286aa50b06599dddc7b2bf491b2799ad6

data/CHANGELOG.md

@@ -1,5 +1,25 @@
 ## [Unreleased]
 
+## [1.6.1] - 2026-04-24
+
+Follow-up release consolidating the 1.6.0 architecture work.
+
+### Rule-based formatter
+
+The rule-based `format/` pipeline (`Formatter`, `Registry`, `Rule`) is the canonical formatting path, replacing the legacy monolithic emitter. Rules covering body indentation (`StatementsRule`), singleton classes (`SingletonClassRule`), and variable writes (`VariableWriteRule`) ship as part of the default registry. The Intermediate Representation (IR) module decouples parsing from emission for composability and testability.
+
+### Method chain reformatting
+
+Multi-line method chains can be reformatted from aligned style (indented under the first dot) to indented style (one level beyond the receiver), preserving the source's base indent. The pass is wired into the fallback path for resilience.
+
+### Printer optimizations
+
+The printer carries a pre-computed indent cache and inline hints for the hot path; `reformat_chain_lines` has been deduplicated across rules and uses `Cow<str>` to avoid allocations on pass-through.
+
+### Editor integration
+
+Setup guides for VSCode, Neovim, Helix, Emacs, and Zed land in the repository; every editor uses the Ruby LSP addon system, so there are no editor-specific plugins to maintain. The README's Editor Integration section replaces the previous "Coming Soon" placeholder with a VSCode quick start.
+
 ## [1.6.0] - 2026-04-23
 
 ### Added

data/lib/rfmt/prism_bridge.rb

@@ -62,17 +62,28 @@ module Rfmt
     # Serialize the Prism AST with comments to JSON
     def self.serialize_ast_with_comments(result)
       comments = result.comments.map do |comment|
+        loc = comment.location
+        end_line = loc.end_line
+        # Prism's `=begin ... =end` (EmbDocComment) location ends at
+        # `<=end_line>:0`, i.e. the column-0 start of the line AFTER the
+        # `=end` terminator. Reporting that larger end_line makes the
+        # comment appear to overlap with the next statement, so the
+        # comment gets misattributed to a deeper node (e.g. the first
+        # expression inside the next method body) and ends up emitted in
+        # the wrong place. Snap it back to the terminator line.
+        end_line = loc.end_line - 1 if loc.end_column.zero? && loc.end_line > loc.start_line
+
         {
           comment_type: comment.class.name.split('::').last.downcase.gsub('comment', ''),
           location: {
-            start_line: comment.location.start_line,
-            start_column: comment.location.start_column,
-            end_line: comment.location.end_line,
-            end_column: comment.location.end_column,
-            start_offset: comment.location.start_offset,
-            end_offset: comment.location.end_offset
+            start_line: loc.start_line,
+            start_column: loc.start_column,
+            end_line: end_line,
+            end_column: loc.end_column,
+            start_offset: loc.start_offset,
+            end_offset: loc.end_offset
           },
-          text: comment.location.slice,
+          text: loc.slice,
           position: 'leading' # Default position, will be refined by Rust
         }
       end
@@ -120,7 +131,7 @@ module Rfmt
         closing = node.closing_loc
         if closing.end_offset > end_offset
           end_offset = closing.end_offset
-          end_line = closing.end_line
+          end_line = heredoc_terminator_line(closing)
           end_column = closing.end_column
         end
       end
@@ -145,6 +156,19 @@ module Rfmt
       }
     end
 
+    # Prism reports a heredoc's `closing_loc` as `<terminator_line>:0..(terminator_line+1):0`,
+    # i.e. its `end_line` is the LINE AFTER the terminator. Using that as the
+    # node's `end_line` makes blank-line preservation fail: a real blank line
+    # right after the terminator looks identical (diff == 1) to two adjacent
+    # statements with no separator. Use the terminator's own line instead.
+    def self.heredoc_terminator_line(closing_loc)
+      if closing_loc.end_column.zero? && closing_loc.end_line > closing_loc.start_line
+        closing_loc.start_line
+      else
+        closing_loc.end_line
+      end
+    end
+
     # Recursively find the maximum closing_loc among all descendant nodes
     # Returns nil if no closing_loc found, otherwise { end_offset:, end_line:, end_column: }
     def self.find_max_closing_loc_recursive(node, depth: 0)
@@ -158,7 +182,7 @@ module Rfmt
           if max_closing.nil? || closing.end_offset > max_closing[:end_offset]
             max_closing = {
               end_offset: closing.end_offset,
-              end_line: closing.end_line,
+              end_line: heredoc_terminator_line(closing),
               end_column: closing.end_column
             }
           end
@@ -424,9 +448,16 @@ module Rfmt
           metadata['name'] = name
         end
       when Prism::DefNode
-        if (name = extract_node_name(node))
-          metadata['name'] = name
-        end
+        # Prefer `name_loc.slice` over `name.to_s` so unary operator suffixes
+        # (`def !@`, `def +@`, `def -@`) survive the round-trip. Prism
+        # normalizes `name` to the symbol with the `@` stripped for `!@`,
+        # which would otherwise rewrite `def !@` to `def !` silently.
+        name = if node.respond_to?(:name_loc) && node.name_loc
+                 node.name_loc.slice
+               else
+                 extract_node_name(node)
+               end
+        metadata['name'] = name if name
         metadata['parameters_count'] = extract_parameter_count(node).to_s
         # Extract parameters text directly from source
         if node.parameters

data/lib/rfmt/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Rfmt
-  VERSION = '1.6.0'
+  VERSION = '1.6.1'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rfmt
 version: !ruby/object:Gem::Version
-  version: 1.6.0
+  version: 1.6.1
 platform: aarch64-linux
 authors:
 - fujitani sora
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2026-04-23 00:00:00.000000000 Z
+date: 2026-04-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: diff-lcs