ast_transform 2.1.1 → 2.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bc4f949d11b3c6ae864a5c773e6df3f286f88da246499eca57d13069e87c7376
-  data.tar.gz: ba829cf5e50d1c23c4fad2aed2dcd0bd5cbb2ffa15bbc4a9aab0d2abbf09df1e
+  metadata.gz: 3322dbf3c31a96df5ddea9892b60c40b785842500c13f178d1a9b11b5842bf12
+  data.tar.gz: aef4ed8d812275368ec5f22c61d9b3d049ecc42490c406d08e8afd4e34e589e7
 SHA512:
-  metadata.gz: 037e381eade462b8c9259ef14620ca55d1da3576a1028321b698dfa34cafb063fbfb8c41920aa60d56ec793bbe05842309568b5c2282006b2ce7e952bce5f4e7
-  data.tar.gz: d4284341fa943f57666c3ca5ace2038708ed1e2e284b26c82b17c241db6b2487b638990f2a55703561f116693438a30ee4f1a556a83f31037ff44e7f5b554a50
+  metadata.gz: bfd3828a33d2668a854ed4d5b1c299615ca47e92232ccf6705b008437cdb43e6497b9ef517e2664f5cd68b9197f9cc78ef2bd50090152f7889490a18e28cf72a
+  data.tar.gz: 6f9647eaad1208d3cacb9ba038f8e63ff213d1ae7030bb96ec0a28386125096526f64bffa93100795427d67c2671b2638711c9ca891222beac01b0c046a07afc

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    ast_transform (2.1.0)
+    ast_transform (2.1.2)
       parser (>= 3.0)
       prism (>= 1.5)
       unparser (>= 0.6)

data/lib/ast_transform/kwargs_builder.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require 'prism/translation/parser'
+
+module ASTTransform
+  # Extends the default Prism parser builder to distinguish keyword arguments
+  # from hash literals in the AST.
+  #
+  # The upstream builder always emits :hash nodes for both `foo(bar: 1)` and
+  # `foo({ bar: 1 })`. Unparser uses the node type to decide whether to emit
+  # braces: :hash gets `{}`, :kwargs does not. Since Ruby 3.0+ treats these as
+  # semantically different (strict keyword/positional separation), we need the
+  # AST to preserve the distinction.
+  class KwargsBuilder < Prism::Translation::Parser::Builder
+    def associate(begin_t, pairs, end_t)
+      node = super
+      return node unless begin_t.nil? && end_t.nil?
+
+      node.updated(:kwargs)
+    end
+  end
+end

data/lib/ast_transform/transformer.rb

@@ -2,6 +2,7 @@
 require 'prism'
 require 'prism/translation/parser'
 require 'unparser'
+require 'ast_transform/kwargs_builder'
 require 'ast_transform/source_map'
 
 module ASTTransform
@@ -9,10 +10,8 @@ module ASTTransform
     # Constructs a new Transformer instance.
     #
     # @param transformations [Array<ASTTransform::AbstractTransformation>] The transformations to be run.
-    # @param builder [Prism::Translation::Parser::Builder] The AST Node builder.
-    def initialize(*transformations, builder: Prism::Translation::Parser::Builder.new)
+    def initialize(*transformations)
       @transformations = transformations
-      @builder = builder
     end
 
     # Builds the AST for the given +source+.
@@ -100,7 +99,7 @@ module ASTTransform
 
     def parser
       @parser&.reset
-      @parser ||= Prism::Translation::Parser.new(@builder)
+      @parser ||= Prism::Translation::Parser.new(ASTTransform::KwargsBuilder.new)
     end
 
     def register_source_map(source_file_path, transformed_file_path, transformed_ast, transformed_source)

data/lib/ast_transform/version.rb

@@ -1,3 +1,3 @@
 module ASTTransform
-  VERSION = "2.1.1"
+  VERSION = "2.1.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ast_transform
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 2.1.2
 platform: ruby
 authors:
 - Jean-Philippe Duchesne
@@ -164,6 +164,7 @@ files:
 - lib/ast_transform/instruction_sequence/bootsnap_mixin.rb
 - lib/ast_transform/instruction_sequence/mixin.rb
 - lib/ast_transform/instruction_sequence/mixin_utils.rb
+- lib/ast_transform/kwargs_builder.rb
 - lib/ast_transform/source_map.rb
 - lib/ast_transform/transformation.rb
 - lib/ast_transform/transformation_helper.rb