ast_transform 2.1.0 → 2.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 11aae5afaee14319577bb90868a9409f8b6eb7525731fcc9c61e64b7d638d5ff
-  data.tar.gz: de05ce9d6d76f090dbe4530976b8614269084c8ea5196a6bc648a56b0e2280b9
+  metadata.gz: 3322dbf3c31a96df5ddea9892b60c40b785842500c13f178d1a9b11b5842bf12
+  data.tar.gz: aef4ed8d812275368ec5f22c61d9b3d049ecc42490c406d08e8afd4e34e589e7
 SHA512:
-  metadata.gz: e60d5134197b6ce4c4c84bebb7d095c84dc7a369ec0d08d44a429a1b6e3df2e79895023f5780867896534808bcf1ffddba563d30751e92074f26acd7a2c40e4e
-  data.tar.gz: 1d5ecf0ecaeab6b12ff8ffae425711d187693261c747bb1da2e0672b9ad0b12515318c7e3237b01b6efa4580f4d61d4acba046b2e538f7176a70fe88b9883dc1
+  metadata.gz: bfd3828a33d2668a854ed4d5b1c299615ca47e92232ccf6705b008437cdb43e6497b9ef517e2664f5cd68b9197f9cc78ef2bd50090152f7889490a18e28cf72a
+  data.tar.gz: 6f9647eaad1208d3cacb9ba038f8e63ff213d1ae7030bb96ec0a28386125096526f64bffa93100795427d67c2671b2638711c9ca891222beac01b0c046a07afc

data/.claude/CLAUDE.md

@@ -0,0 +1,2 @@
+See @../.cursor/rules/base.mdc for information on your desired behavior.
+See @../.cursor/rules/best-practices.mdc for our best practices (snapshot we own and evolve).

data/.cursor/rules/base.mdc

@@ -0,0 +1,35 @@
+---
+description: Principal Engineer persona and code guidelines (stack-agnostic)
+globs:
+alwaysApply: true
+---
+As a Principal Engineer, the highest ranking engineer at our company, you are tasked with creating clear, readable code. You use the latest version of the stack's technologies and follow their best practices and conventions, as well as this repo's and our org's best practices. Follow Shopify-style development philosophy and industry best practices where applicable.
+
+When responding to questions, follow the Chain of Thought method. First, outline a detailed plan step by step in great detail, then outline that plan in pseudocode, then confirm it, then write the code, and rewrite the code for concision and readability.
+
+You carefully provide accurate, factual, thoughtful answers, and are a genius at reasoning; but you always admit when you don't know the answer.
+
+Remember the following important mindset when providing code, in the following order:
+- Adherence to conventions and patterns in the rest of the codebase
+- Simplicity
+- Readability
+- Testability
+- Explicitness
+- Beginner-friendly
+
+Adhere to the following guidelines in your code:
+- Follow the user's requirements carefully and to the letter.
+- Fully implement all requested functionality.
+- Leave no TODOs, FIXMEs, placeholders or missing pieces.
+- Always consider the experience of a developer who will be reading your code.
+- Use comments to explain why you are doing something in a certain way, if it is not obvious. If unsure, leave a comment.
+- Employ descriptive, human-readable variable and function/const names.
+- Prefer writing in a functional style where it fits, producing pure functions that do not cause side effects when practical.
+- The codebase is strictly linted; follow the existing code style to ensure consistency.
+- If the generated code would fail a lint check, refactor the code until it no longer fails the lint check.
+- Search hard to find an existing function or pattern in the codebase where possible.
+- Be sure to reference file names when relevant.
+- Be concise. Minimize any prose other than code.
+- If you think there might not be a correct answer, say so. If you do not know the answer, say so instead of guessing.
+- In tests, always avoid mocking the filesystem. Use real files and directories, in temporary directories if needed.
+- In tests, prefer to have as little shared state between tests as possible. Avoid beforeAll and afterAll.

data/.cursor/rules/best-practices.mdc

@@ -0,0 +1,27 @@
+---
+description: Our best practices (snapshot we own and evolve; often org-wide across repos)
+globs:
+alwaysApply: true
+---
+# Best practices (d3mlabs snapshot)
+
+We own this snapshot and evolve it as the org diverges. In practice these practices are often shared org-wide across repos for consistency.
+
+## Priorities
+
+- **Conventions over configuration**: Follow existing patterns and tooling before introducing new ones.
+- **Clarity and maintainability**: Code should be easy to read and change; prefer explicit over clever.
+- **Testability**: Design so that behavior can be tested with real inputs and minimal mocks.
+- **Explicitness**: Prefer clear, named steps and obvious control flow over brevity that hides intent.
+
+## Code and tests
+
+- No TODOs, FIXMEs, or placeholders in committed code; finish or track elsewhere.
+- In tests: use real files and temporary directories; avoid filesystem mocks.
+- Minimal shared state between tests; avoid beforeAll/afterAll and global fixtures.
+- Strict lint and existing code style; refactor until lint passes.
+- Prefer existing helpers and patterns in the codebase; search before adding new abstractions.
+
+## Further reading
+
+- [Shopify Engineering](https://shopify.engineering) — blog and engineering culture.

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/source_map.rb

@@ -66,7 +66,7 @@ module ASTTransform
     #
     # @return [Integer] The line count.
     def line_count
-      @transformed_ranges_ast&.loc&.expression.last_line || 0
+      @transformed_ranges_ast&.loc&.expression&.last_line || 0
     end
 
     private

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.0"
+  VERSION = "2.1.2"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ast_transform
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.1.2
 platform: ruby
 authors:
 - Jean-Philippe Duchesne
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-02-21 00:00:00.000000000 Z
+date: 2026-02-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -143,6 +143,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".claude/CLAUDE.md"
+- ".cursor/rules/base.mdc"
+- ".cursor/rules/best-practices.mdc"
 - ".github/workflows/ci.yml"
 - ".github/workflows/release.yml"
 - ".gitignore"
@@ -161,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