ruby2ruby 2.5.1 → 2.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ce3916335beb18e635634b93a85be8c88aaaab6029aea5ba5426b429cebbb2a6
-  data.tar.gz: e443e89cd7216bee0ae5d2fc8b182cafb94827357cb82d2b248cb19e2a26b6bd
+  metadata.gz: c6f7e4e4fcbd415175663732f01b2dbb0291578dc12e0121e8e777d73b9dbb90
+  data.tar.gz: 6282de7f06e33662c5d9f3b8fc531685cd32e90d74682f119f13f6d1d5e39702
 SHA512:
-  metadata.gz: 62655af018fce2402bb410d7606afb34f7a6f332c88fc74aca6ebf9faf5f37abd23d9827cfd5d5742d3240d89ae838f4ef7d28f92da7bc520c335467b89a448a
-  data.tar.gz: 637c87a736b4da72487c95bc8fa981d49ba9b57e3e8f48c2013ef52bbbcb09176c0abfae18b272cba551adcb5961bcad6e94fa38dee8352f154986861cf7e616
+  metadata.gz: a8d4ba7db3c8b768c659c0873e85037a98ba39d1008f91c3f84bc5f3d3ffa5e7efad9fc05d3c610dbab5f71239b1f068a8a727292dcbdc21311ab11ba7141d83
+  data.tar.gz: cace0a14c47e09fc9ef555b62207ebe630e958d51bcc0cdb9493c6e279065a0b23229c983c3e865c0ecd70aef3b779ea10f4426aa57a50880fe0163b9bf32574

data/History.rdoc

@@ -1,3 +1,21 @@
+=== 2.6.0 / 2025-12-24
+
+* 3 minor enhancements:
+
+  * Added prism as a development dependency.
+  * Bumped ruby_version to 3.2+.
+  * Removed ruby_parser as a runtime dependency.
+
+* 1 bug fix:
+
+  * Minor fixes to impl for comment processing. Prism returns chomped string.
+
+=== 2.5.2 / 2025-03-25
+
+* 1 bug fix:
+
+  * Added support for 3 slot op_asgn (eg `a &= 1`)
+
 === 2.5.1 / 2024-07-08
 
 * 1 bug fix:

data/Rakefile

@@ -17,7 +17,11 @@ Hoe.spec 'ruby2ruby' do
   license "MIT"
 
   dependency "sexp_processor", "~> 4.6"
-  dependency "ruby_parser",    "~> 3.1"
+  dependency "prism",          "~> 1.7", :dev
+
+  require_ruby_version ">= 3.2"
+
+  self.isolate_multiruby = true
 end
 
 def process ruby, file="stdin"

data/lib/ruby2ruby.rb

@@ -31,7 +31,7 @@ end
 # Generate ruby code from a sexp.
 
 class Ruby2Ruby < SexpProcessor
-  VERSION = "2.5.1" # :nodoc:
+  VERSION = "2.6.0" # :nodoc:
 
   # cutoff for one-liners
   LINE_LENGTH = 78
@@ -350,8 +350,14 @@ class Ruby2Ruby < SexpProcessor
     end
   end
 
+  def comments exp # :nodoc:
+    comments = exp.comments
+    comments += "\n" unless comments.end_with?("\n") if comments
+    comments
+  end
+
   def process_class exp # :nodoc:
-    "#{exp.comments}class #{util_module_or_class(exp, true)}"
+    "#{comments exp}class #{util_module_or_class(exp, true)}"
   end
 
   def process_colon2 exp # :nodoc:
@@ -411,7 +417,7 @@ class Ruby2Ruby < SexpProcessor
   def process_defn(exp) # :nodoc:
     _, name, args, *body = exp
 
-    comm = exp.comments
+    comm = comments exp
     args = process args
     args = "" if args == "()"
 
@@ -887,7 +893,7 @@ class Ruby2Ruby < SexpProcessor
   end
 
   def process_module(exp) # :nodoc:
-    "#{exp.comments}module #{util_module_or_class(exp)}"
+    "#{comments exp}module #{util_module_or_class(exp)}"
   end
 
   def process_next(exp) # :nodoc:
@@ -916,11 +922,14 @@ class Ruby2Ruby < SexpProcessor
   end
 
   def process_op_asgn exp # :nodoc:
-    # [[:lvar, :x], [:call, nil, :z, [:lit, 1]], :y, :"||"]
-
     case exp.length
     when 4
-      raise "NOT YET: op_asgn 4"
+      _, lhs, op, rhs = exp
+
+      lhs = process lhs
+      rhs = process rhs
+
+      "#{lhs} #{op}= #{rhs}"
     when 5
       _, lhs, rhs, index, op = exp
 

data/test/test_ruby2ruby.rb

@@ -9,7 +9,21 @@ require "ruby2ruby"
 require "pt_testcase"
 require "fileutils"
 require "tmpdir"
-require "ruby_parser" if ENV["CHECK_SEXPS"]
+require "prism"
+
+class NotRubyParser < Prism::Translation::RubyParser
+  attr_accessor :scopes
+
+  def initialize scopes:nil
+    super()
+    self.scopes = [scopes] if scopes
+  end
+
+  # overridden from prism to add scopes arg
+  def parse(source, filepath = "(string)")
+    translate(Prism.parse(source, filepath:, partial_script: true, scopes:), filepath)
+  end
+end
 
 class R2RTestCase < ParseTreeTestCase
   def self.previous key
@@ -42,13 +56,8 @@ class TestRuby2Ruby < R2RTestCase
     @processor = Ruby2Ruby.new
   end
 
-  # some things don't work in earlier rubies... oh well.
-  def skip30
-    skip unless RUBY_VERSION > "3.0"
-  end
-
-  def skip31
-    skip unless RUBY_VERSION > "3.1"
+  def skip_prism
+    skip "not fully happy with prism yet"
   end
 
   def do_not_check_sexp!
@@ -135,8 +144,6 @@ class TestRuby2Ruby < R2RTestCase
   end
 
   def test_hash_shorthand_invalid_key_type
-    do_not_check_sexp!
-
     inn = s(:hash, s(:str, 'k'), nil)
     out = '{ k: }'
     assert_raises do
@@ -185,11 +192,11 @@ class TestRuby2Ruby < R2RTestCase
   end
 
   def assert_str exp, src
-    assert_equal s(:str, exp), RubyParser.new.process(src)
+    assert_equal s(:str, exp), NotRubyParser.new.process(src)
   end
 
   def assert_dstr exp, int, src
-    assert_equal s(:dstr, exp, s(:evstr, int).compact), RubyParser.new.process(src)
+    assert_equal s(:dstr, exp, s(:evstr, int).compact), NotRubyParser.new.process(src)
   end
 
   def assert_r2r exp, sexp
@@ -197,7 +204,7 @@ class TestRuby2Ruby < R2RTestCase
   end
 
   def assert_rt src, exp = src.dup
-    assert_equal exp, Ruby2Ruby.new.process(RubyParser.new.parse(src))
+    assert_equal exp, Ruby2Ruby.new.process(NotRubyParser.new.parse(src))
   end
 
   def test_bug_033
@@ -329,8 +336,6 @@ class TestRuby2Ruby < R2RTestCase
   end
 
   def test_dregx_slash
-    do_not_check_sexp!
-
     inn = util_thingy(:dregx)
     out = '/a"b#{(1 + 1)}c"d\/e/'
     assert_parse inn, out, /a"b2c"d\/e/
@@ -349,8 +354,6 @@ class TestRuby2Ruby < R2RTestCase
   end
 
   def test_lit_regexp_slash
-    do_not_check_sexp! # dunno why on this one
-
     assert_parse s(:lit, /blah\/blah/), '/blah\/blah/', /blah\/blah/
   end
 
@@ -447,7 +450,6 @@ class TestRuby2Ruby < R2RTestCase
   end
 
   def test_forward_args__call
-    skip31
 
     inn = s(:defn, :x, s(:args, s(:forward_args)), s(:call, nil, :y, s(:forward_args)))
     out = "def x(...)\n  y(...)\nend"
@@ -746,7 +748,12 @@ class TestRuby2Ruby < R2RTestCase
   end
 
   def test_case_in__array_pat_04
-    assert_case_in "[[:b, ^c], [:d, ^e]]", s(:array_pat, nil, s(:array_pat, nil, s(:lit, :b), s(:lvar, :c)), s(:array_pat, nil, s(:lit, :d), s(:lvar, :e)))
+    pt = s(:array_pat,
+           nil,
+           s(:array_pat, nil, s(:lit, :b), s(:lvar, :c)),
+           s(:array_pat, nil, s(:lit, :d), s(:lvar, :a)))
+
+    assert_case_in "[[:b, ^c], [:d, ^a]]", pt
   end
 
   def test_case_in__array_pat_06
@@ -774,22 +781,16 @@ class TestRuby2Ruby < R2RTestCase
   end
 
   def test_case_in__array_pat_19
-    skip31
-
     assert_case_in "[^@a, ^$b, ^@@c]", s(:array_pat, nil, s(:ivar, :@a), s(:gvar, :$b), s(:cvar, :@@c)) # HACK: really not sure about this one
   end
 
   def test_case_in__find_pat_1
-    skip30
-
     assert_case_in "[*a, :+, *b]", s(:find_pat, nil, :"*a",
                                      s(:lit, :+),
                                      :"*b")
   end
 
   def test_case_in__find_pat_2
-    skip30
-
     assert_case_in "[*, :b, ^c, *]", s(:find_pat, nil,
                                        :*,
                                        s(:lit, :b), s(:lvar, :c),
@@ -797,35 +798,30 @@ class TestRuby2Ruby < R2RTestCase
   end
 
   def test_case_in__find_pat_3
-    skip30
-
-    assert_case_in("Array(*b, n, { a: }, m, *a)",
+    assert_case_in("Array(*b, n, { a: }, m, *c)",
                    s(:find_pat,
                      s(:const, :Array),
                      :"*b",
                      s(:lasgn, :n),
                      s(:hash_pat, nil, s(:lit, :a), nil),
                      s(:lasgn, :m),
-                     :"*a"),
-                   "Array[*b, n, { a: }, m, *a]")
+                     :"*c"),
+                   "Array[*b, n, { a: }, m, *c]")
   end
 
   def test_case_in__find_pat_4
-    skip30
-
-    assert_case_in("*b, n, { a: }, m, *a", s(:find_pat,
-                                             nil,
-                                             :"*b",
-                                             s(:lasgn, :n),
-                                             s(:hash_pat, nil, s(:lit, :a), nil),
-                                             s(:lasgn, :m),
-                                             :"*a"),
-                   "[*b, n, { a: }, m, *a]")
+    assert_case_in("*b, n, { a: }, m, *c",
+                   s(:find_pat,
+                     nil,
+                     :"*b",
+                     s(:lasgn, :n),
+                     s(:hash_pat, nil, s(:lit, :a), nil),
+                     s(:lasgn, :m),
+                     :"*c"),
+                   "[*b, n, { a: }, m, *c]")
   end
 
   def test_case_in__find_pat_5
-    skip30
-
     assert_case_in("Array(*lhs, ^b, *rhs)", s(:find_pat,
                                               s(:const, :Array),
                                               :"*lhs",
@@ -949,10 +945,10 @@ class TestRuby2Ruby < R2RTestCase
            s(:array_pat,
              nil,
              s(:lit, :d),
-             s(:lvar, :e)),
+             s(:lvar, :a)),
           )
 
-    assert_case_in "[[:b, c], [:d, ^e]]", pt
+    assert_case_in "[[:b, c], [:d, ^a]]", pt
   end
 
   def test_case_in__hash_pat_00
@@ -1067,6 +1063,13 @@ class TestRuby2Ruby < R2RTestCase
     assert_parse inn, out
   end
 
+  def test_op_asgn_4
+    inn = s(:op_asgn, s(:colon3, :X), :"&", s(:lit, 1))
+
+    out = "::X &= 1"
+    assert_parse inn, out
+  end
+
   def test_rescue_block
     inn = s(:rescue,
             s(:call, nil, :alpha),
@@ -1096,7 +1099,7 @@ class TestRuby2Ruby < R2RTestCase
   end
 
   def test_unless_vs_if_not
-    do_not_check_sexp! # TODO: remove? dunno if that's possible w/ this one
+    do_not_check_sexp! # dunno if it's possible to remove this one
 
     rb1 = "a unless b"
     rb2 = "a if (not b)"
@@ -1110,13 +1113,7 @@ class TestRuby2Ruby < R2RTestCase
   end
 
   def ruby_parser
-    parser = RubyParser.latest
-
-    %i[a b c d].each do |v|
-      parser.env[v] = :lvar
-    end
-
-    parser
+    NotRubyParser.new scopes: %i[a b c d]
   end
 
   def assert_parse sexp, expected_ruby, expected_eval = nil
@@ -1178,8 +1175,7 @@ ir2r = File.read("lib/ruby2ruby.rb")
 require "ruby_parser"
 
 def morph_and_eval src, from, to, processor
-  parser = RubyParser.latest
-  new_src = processor.new.process(parser.process(src.sub(from, to)))
+  new_src = processor.new.process(NotRubyParser.new.process(src.sub(from, to)))
 
   eval new_src
 

metadata

@@ -1,18 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: ruby2ruby
 version: !ruby/object:Gem::Version
-  version: 2.5.1
+  version: 2.6.0
 platform: ruby
 authors:
 - Ryan Davis
-autorequire:
 bindir: bin
 cert_chain:
 - |
   -----BEGIN CERTIFICATE-----
-  MIIDPjCCAiagAwIBAgIBCDANBgkqhkiG9w0BAQsFADBFMRMwEQYDVQQDDApyeWFu
+  MIIDPjCCAiagAwIBAgIBCTANBgkqhkiG9w0BAQsFADBFMRMwEQYDVQQDDApyeWFu
   ZC1ydWJ5MRkwFwYKCZImiZPyLGQBGRYJemVuc3BpZGVyMRMwEQYKCZImiZPyLGQB
-  GRYDY29tMB4XDTI0MDEwMjIxMjEyM1oXDTI1MDEwMTIxMjEyM1owRTETMBEGA1UE
+  GRYDY29tMB4XDTI1MDEwNjIzMjcwMVoXDTI2MDEwNjIzMjcwMVowRTETMBEGA1UE
   AwwKcnlhbmQtcnVieTEZMBcGCgmSJomT8ixkARkWCXplbnNwaWRlcjETMBEGCgmS
   JomT8ixkARkWA2NvbTCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBALda
   b9DCgK+627gPJkB6XfjZ1itoOQvpqH1EXScSaba9/S2VF22VYQbXU1xQXL/WzCkx
@@ -22,14 +21,14 @@ cert_chain:
   qhtV7HJxNKuPj/JFH0D2cswvzznE/a5FOYO68g+YCuFi5L8wZuuM8zzdwjrWHqSV
   gBEfoTEGr7Zii72cx+sCAwEAAaM5MDcwCQYDVR0TBAIwADALBgNVHQ8EBAMCBLAw
   HQYDVR0OBBYEFEfFe9md/r/tj/Wmwpy+MI8d9k/hMA0GCSqGSIb3DQEBCwUAA4IB
-  AQCygvpmncmkiSs9r/Kceo4bBPDszhTv6iBi4LwMReqnFrpNLMOWJw7xi8x+3eL2
-  XS09ZPNOt2zm70KmFouBMgOysnDY4k2dE8uF6B8JbZOO8QfalW+CoNBliefOTcn2
-  bg5IOP7UoGM5lC174/cbDJrJnRG9bzig5FAP0mvsgA8zgTRXQzIUAZEo92D5K7p4
-  B4/O998ho6BSOgYBI9Yk1ttdCtti6Y+8N9+fZESsjtWMykA+WXWeGUScHqiU+gH8
-  S7043fq9EbQdBr2AXdj92+CDwuTfHI6/Hj5FVBDULufrJaan4xUgL70Hvc6pTTeW
-  deKfBjgVAq7EYHu1AczzlUly
+  AQAC0WQJcPOWPFwkojhzweilRVjTJ19UiLhiBTw3C1wJO3LVdBkWDmnnhAmKuX4D
+  r7vjQvESlABGIPdutI1Yl7mrHQzTkfLfXvNN6MT0nLChPyIYauT6SZZxubwJrUfA
+  7R0c2CJTIboZ0XaGpLsXqHEF1c29H7TV1QvVuqKAN2mCjh4N82QVn+ZKtys28AwT
+  6GfQX2fqLoi4KSc7xIzHKaNzqxeOICmJofk9w5VZ2rRN6yes8jvFYwz9HR41wdj8
+  bwfinv7Yp5fA6AysuZLhCykyfDuZVRrUp0Vb68YCKsLjJly/Theak+euNTxvHsB+
+  al9oSgPPHICMEX65qvLywitx
   -----END CERTIFICATE-----
-date: 2024-07-08 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sexp_processor
@@ -46,19 +45,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '4.6'
 - !ruby/object:Gem::Dependency
-  name: ruby_parser
+  name: prism
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.1'
-  type: :runtime
+        version: '1.7'
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '1.7'
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
@@ -85,14 +84,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '4.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '4.3'
 description: |-
   ruby2ruby provides a means of generating pure ruby code easily from
   RubyParser compatible Sexps. This makes making dynamic language
@@ -120,7 +119,6 @@ licenses:
 - MIT
 metadata:
   homepage_uri: https://github.com/seattlerb/ruby2ruby
-post_install_message:
 rdoc_options:
 - "--main"
 - README.rdoc
@@ -130,15 +128,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '3.2'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.14
-signing_key:
+rubygems_version: 3.7.2
 specification_version: 4
 summary: ruby2ruby provides a means of generating pure ruby code easily from RubyParser
   compatible Sexps