unparser 0.6.15 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9d1d0033d21b1c64de6affcc9c555797edf5206dc2947484ff8391e3c8e739ce
-  data.tar.gz: 60732ffbcc8a43cd2f04a6cba76e955b7e764ada9d201cd51a0a624d093b3659
+  metadata.gz: d687ea37d01a5a689f6c2c909072090603ec406d0a0e0bc933d315362b10d5a3
+  data.tar.gz: a49fe55eb25755a6d4158084ca9ccd0df1e62741584af21d9313b8eb63d1d5af
 SHA512:
-  metadata.gz: 77db441f3e4da2b584065c597796d4adbb08b07e481b43ee85c80218375cbf8932bae847727afa9abee0be02ee5dea1c2effd5caa70216a5bafa34007d0c3820
-  data.tar.gz: c180d28c4ac14fe5d6b5756350592b472cd084ce932309fd6cc87f042dffb1f1870dd194260e71b53dbac71fa648bd359c7b4d3c8860c7508f7cc854c9d2ef25
+  metadata.gz: 5bc28be9868ed008eb3c7e072a01a9403921f2ee9a28566f2d4a616042cb2f3da659a817a79046462fd4d5a782219d2209072ed21942c54133bac82754f1103d
+  data.tar.gz: eaa194a4a148f61ffdefad476cadf99c600fa88f94618dbf408a8fed10027cde2f4151bb6d2ec4b35195ce54c10d07b90d2b58be83f7ef8abf24056aee3d828a

data/lib/unparser/anima.rb

@@ -134,14 +134,25 @@ module Unparser
     # @param [Class, Module] scope
     #
     # @return [undefined]
+    #
+    # mutant:disable
     def included(descendant)
       descendant.instance_exec(self, attribute_names) do |anima, names|
         # Define anima method
+
+        class << self
+          undef_method(:anima) if method_defined?(:anima)
+        end
+
         define_singleton_method(:anima) { anima }
 
         # Define instance methods
         include InstanceMethods
 
+        names.each do |name|
+          undef_method(name) if method_defined?(name)
+        end
+
         # Define attribute readers
         attr_reader(*names)
 

data/lib/unparser/ast/local_variable_scope.rb

@@ -1,11 +1,10 @@
 # frozen_string_literal: true
 
 module Unparser
-  module AST
-
+  class AST
     # Calculated local variable scope for a given node
     class LocalVariableScope
-      include Enumerable, Adamantium
+      include Adamantium, Anima.new(:static_local_variables, :node)
 
       # Initialize object
       #
@@ -13,13 +12,17 @@ module Unparser
       #
       # @return [undefined]
       #
-      # @api private
-      #
-      def initialize(node)
+      # mutant:disable
+      def initialize(*arguments)
+        super
+
         items = []
-        LocalVariableScopeEnumerator.each(node) do |*scope|
-          items << scope
-        end
+
+        LocalVariableScopeEnumerator.each(
+          node:  node,
+          stack: static_local_variables.dup
+        ) { |*scope| items << scope }
+
         @items = items
       end
 
@@ -53,6 +56,15 @@ module Unparser
         end
       end
 
+      # mutant:disable
+      def local_variables_for_node(needle)
+        @items.each do |node, current|
+          return current if node.equal?(needle)
+        end
+
+        Set.new
+      end
+
       # Test if local variables where first assigned in body and read by conditional
       #
       # @param [Parser::AST::Node] body
@@ -90,21 +102,13 @@ module Unparser
       #
       # @api private
       #
-      def initialize
-        @stack = [Set.new]
+      def initialize(stack:)
+        @stack = [stack]
       end
 
       # Enumerate each node with its local variable scope
-      #
-      # @param [Parser::AST::Node] node
-      #
-      # @return [self]
-      #
-      # @api private
-      #
-      def self.each(node, &block)
-        new.each(node, &block)
-        self
+      def self.each(node:, stack:, &block)
+        new(stack: stack).each(node: node, &block)
       end
 
       # Enumerate local variable scope scope
@@ -117,7 +121,7 @@ module Unparser
       #
       # @api private
       #
-      def each(node, &block)
+      def each(node:, &block)
         visit(node, &block)
       end
 
@@ -132,7 +136,7 @@ module Unparser
         enter(node)
         yield node, current.dup, before
         node.children.each do |child|
-          visit(child, &block) if child.is_a?(Parser::AST::Node)
+          visit(child, &block) if child.instance_of?(Parser::AST::Node)
         end
         leave(node)
       end
@@ -142,7 +146,7 @@ module Unparser
         when *RESET_NODES
           push_reset
         when ASSIGN_NODES
-          define(node.children.first)
+          value = node.children.first and define(value)
         when *INHERIT_NODES
           push_inherit
         end

data/lib/unparser/ast.rb

@@ -2,9 +2,10 @@
 
 module Unparser
   # Namespace for AST processing tools
-  module AST
+  class AST
+    include Anima.new(:comments, :explicit_encoding, :node, :static_local_variables)
+
     FIRST_CHILD = ->(node) { node.children.first }.freeze
-    TAUTOLOGY   = ->(_node) { true }.freeze
 
     RESET_NODES   = %i[module class sclass def defs].freeze
     INHERIT_NODES = [:block].freeze
@@ -16,12 +17,22 @@ module Unparser
         arg
         kwarg
         kwoptarg
+        kwrestarg
         lvasgn
         optarg
-        procarg0
         restarg
       ].to_set.freeze
 
+    # mutant:disable
+    def self.from_node(node:)
+      new(
+        comments:               EMPTY_ARRAY,
+        explicit_encoding:      nil,
+        node:,
+        static_local_variables: Set.new
+      )
+    end
+
     # Test for local variable inherited scope reset
     #
     # @param [Parser::AST::Node] node
@@ -69,6 +80,7 @@ module Unparser
     #
     # @api private
     #
+    # mutant:disable
     def self.local_variable_reads(node)
       Enumerator.new(
         node,
@@ -80,19 +92,6 @@ module Unparser
     class Enumerator
       include Adamantium, Concord.new(:node, :controller), Enumerable
 
-      # Return new instance
-      #
-      # @param [Parser::AST::Node] node
-      # @param [#call(node)] controller
-      #
-      # @return [Enumerator]
-      #
-      # @api private
-      #
-      def self.new(node, controller = TAUTOLOGY)
-        super
-      end
-
       # Return each node
       #
       # @return [Enumerator<Parser::AST::Node>]
@@ -103,8 +102,8 @@ module Unparser
       #
       # @api private
       #
-      def each(&block)
-        Walker.call(node, controller, &block)
+      def each(&)
+        Walker.call(node, controller, &)
       end
 
       # Return nodes selected by types
@@ -168,13 +167,10 @@ module Unparser
       #
       # @param [Parser::AST::Node] node
       #
-      # @return [self]
-      #
       # @api private
       #
-      def self.call(node, controller = TAUTOLOGY, &block)
+      def self.call(node, controller, &block)
         new(block, controller).call(node)
-        self
       end
 
       # Call walker with node

data/lib/unparser/buffer.rb

@@ -14,8 +14,10 @@ module Unparser
     # @api private
     #
     def initialize
-      @content = +''
-      @indent = 0
+      @content  = +''
+      @heredocs = []
+      @indent   = 0
+      @no_nl    = true
     end
 
     # Append string
@@ -34,6 +36,13 @@ module Unparser
       self
     end
 
+    # Push to heredoc stack
+    #
+    # @param [String] heredoc
+    def push_heredoc(heredoc)
+      @heredocs << heredoc
+    end
+
     # Append a string without an indentation prefix
     #
     # @param [String] string
@@ -68,6 +77,10 @@ module Unparser
       self
     end
 
+    def ensure_nl
+      nl unless fresh_line?
+    end
+
     # Write newline
     #
     # @return [self]
@@ -75,7 +88,27 @@ module Unparser
     # @api private
     #
     def nl
+      @no_nl = false
       write(NL)
+      flush_heredocs
+      self
+    end
+
+    # Write final newline
+    def final_newline
+      return if fresh_line? || @no_nl
+
+      write(NL)
+    end
+
+    def nl_flush_heredocs
+      return if @heredocs.empty?
+
+      if fresh_line?
+        flush_heredocs
+      else
+        nl
+      end
     end
 
     def root_indent
@@ -117,6 +150,10 @@ module Unparser
       self
     end
 
+    def write_encoding(encoding)
+      write("# -*- encoding: #{encoding} -*-\n")
+    end
+
   private
 
     INDENT_SPACE = '  '.freeze
@@ -125,5 +162,10 @@ module Unparser
       write(INDENT_SPACE * @indent)
     end
 
+    def flush_heredocs
+      @heredocs.each(&public_method(:write))
+      @heredocs = []
+    end
+
   end # Buffer
 end # Unparser

data/lib/unparser/cli.rb

@@ -60,6 +60,7 @@ module Unparser
     #
     # @api private
     #
+    # mutant:disable
     def self.run(*arguments)
       new(*arguments).exit_status
     end
@@ -71,15 +72,17 @@ module Unparser
     # @return [undefined]
     #
     # @api private
+    # mutant:disable
     def initialize(arguments)
       @ignore  = Set.new
       @targets = []
 
-      @fail_fast  = false
-      @start_with = nil
-      @success    = true
-      @validation = :validation
-      @verbose    = false
+      @fail_fast                    = false
+      @start_with                   = nil
+      @success                      = true
+      @validation                   = :validation
+      @verbose                      = false
+      @ignore_original_syntax_error = false
 
       opts = OptionParser.new do |builder|
         add_options(builder)
@@ -98,6 +101,7 @@ module Unparser
     #
     # @api private
     #
+    # mutant:disable
     # rubocop:disable Metrics/MethodLength
     def add_options(builder)
       builder.banner = 'usage: unparse [options] FILE [FILE]'
@@ -114,6 +118,9 @@ module Unparser
       builder.on('-l', '--literal') do
         @validation = :literal_validation
       end
+      builder.on('--ignore-original-syntax-error') do
+        @ignore_original_syntax_error = true
+      end
       builder.on('--ignore FILE') do |file|
         @ignore.merge(targets(file))
       end
@@ -129,6 +136,7 @@ module Unparser
     #
     # @api private
     #
+    # mutant:disable
     def exit_status
       effective_targets.each do |target|
         process_target(target)
@@ -140,11 +148,15 @@ module Unparser
 
   private
 
+    # mutant:disable
     def process_target(target)
       validation = target.public_send(@validation)
       if validation.success?
         puts validation.report if @verbose
         puts "Success: #{validation.identification}"
+      elsif ignore_original_syntax_error?(validation)
+        exception = validation.original_node.from_left
+        puts "#{exception.class}: #{validation.identification} #{exception}"
       else
         puts validation.report
         puts "Error: #{validation.identification}"
@@ -152,6 +164,14 @@ module Unparser
       end
     end
 
+    # mutant:disable
+    def ignore_original_syntax_error?(validation)
+      @ignore_original_syntax_error && validation.original_node.from_left do
+        nil
+      end.instance_of?(Parser::SyntaxError)
+    end
+
+    # mutant:disable
     def effective_targets
       if @start_with
         reject = true
@@ -167,6 +187,7 @@ module Unparser
       end.reject(&@ignore.method(:include?))
     end
 
+    # mutant:disable
     def targets(file_name)
       if File.directory?(file_name)
         Dir.glob(File.join(file_name, '**/*.rb'))

data/lib/unparser/either.rb

@@ -54,15 +54,15 @@ module Unparser
       # Evaluate functor block
       #
       # @return [Either::Left<Object>]
-      def fmap(&block)
-        require_block(&block)
+      def fmap(&)
+        require_block(&)
       end
 
       # Evaluate applicative block
       #
       # @return [Either::Left<Object>]
-      def bind(&block)
-        require_block(&block)
+      def bind(&)
+        require_block(&)
       end
 
       # Unwrap value from left
@@ -137,8 +137,8 @@ module Unparser
       # Map over left value
       #
       # @return [Either::Right<Object>]
-      def lmap(&block)
-        require_block(&block)
+      def lmap(&)
+        require_block(&)
       end
 
       # Evaluate right side of branch

data/lib/unparser/emitter/array.rb

@@ -6,10 +6,6 @@ module Unparser
     class Array < self
       handle :array
 
-      def emit_heredoc_reminders
-        emitters.each(&:emit_heredoc_reminders)
-      end
-
     private
 
       def dispatch

data/lib/unparser/emitter/array_pattern.rb

@@ -12,18 +12,10 @@ module Unparser
 
       def dispatch
         write('[')
-        delimited(children, &method(:emit_member))
+        delimited(children)
         write(', ') if node_type.equal?(:array_pattern_with_tail)
         write(']')
       end
-
-      def emit_member(node)
-        if n_match_rest?(node)
-          writer_with(MatchRest, node).emit_array_pattern
-        else
-          visit(node)
-        end
-      end
     end # Pin
   end # Emitter
 end # Unparser

data/lib/unparser/emitter/assignment.rb

@@ -11,12 +11,6 @@ module Unparser
         true
       end
 
-      def emit_heredoc_reminders
-        return unless right
-
-        emitter(right).emit_heredoc_reminders
-      end
-
     private
 
       def dispatch
@@ -30,12 +24,17 @@ module Unparser
         write(' = ')
 
         if BINARY_OPERATOR.include?(right.type)
-          writer_with(Writer::Binary, right).emit_operator
+          writer_with(Writer::Binary, node: right).emit_operator
         else
-          visit(right)
+          right_emitter.write_to_buffer
         end
       end
 
+      def right_emitter
+        emitter(right)
+      end
+      memoize :right_emitter
+
       abstract_method :emit_left
 
       # Variable assignment emitter

data/lib/unparser/emitter/begin.rb

@@ -8,12 +8,6 @@ module Unparser
       handle :begin
       children :body
 
-      def emit_heredoc_reminders
-        children.each do |child|
-          emitter(child).emit_heredoc_reminders
-        end
-      end
-
     private
 
       def dispatch

data/lib/unparser/emitter/binary.rb

@@ -13,7 +13,7 @@ module Unparser
       end
 
       def writer
-        writer_with(Writer::Binary, node)
+        writer_with(Writer::Binary, node:)
       end
       memoize :writer
     end # Binary

data/lib/unparser/emitter/block.rb

@@ -16,7 +16,6 @@ module Unparser
         ws
         write_open
         emit_block_arguments unless n_lambda?(target)
-        target_writer.emit_heredoc_reminders if n_send?(target)
         emit_optional_body_ensure_rescue(body)
         write_close
       end
@@ -42,7 +41,7 @@ module Unparser
       end
 
       def target_writer
-        writer_with(Writer::Send::Regular, target)
+        writer_with(Writer::Send::Regular, node: target)
       end
       memoize :target_writer
 
@@ -65,7 +64,7 @@ module Unparser
       end
 
       def emit_lambda_arguments
-        parentheses { writer_with(Args, arguments).emit_lambda_arguments }
+        parentheses { writer_with(Args, node: arguments).emit_lambda_arguments }
       end
 
       def numblock?
@@ -78,7 +77,7 @@ module Unparser
         ws
 
         parentheses('|', '|') do
-          writer_with(Args, arguments).emit_block_arguments
+          writer_with(Args, node: arguments).emit_block_arguments
         end
       end
 

data/lib/unparser/emitter/def.rb

@@ -26,7 +26,7 @@ module Unparser
         return if arguments.children.empty?
 
         parentheses do
-          writer_with(Args, arguments).emit_def_arguments
+          writer_with(Args, node: arguments).emit_def_arguments
         end
       end
 

data/lib/unparser/emitter/dstr.rb

@@ -7,15 +7,16 @@ module Unparser
 
       handle :dstr
 
-      def emit_heredoc_reminders
-        writer_with(Writer::DynamicString, node).emit_heredoc_reminder
-      end
-
     private
 
       def dispatch
-        writer_with(Writer::DynamicString, node).dispatch
+        dstr_writer.dispatch
+      end
+
+      def dstr_writer
+        writer_with(Writer::DynamicString, node:)
       end
+      memoize :dstr_writer
 
     end # DStr
   end # Emitter

data/lib/unparser/emitter/flow_modifier.rb

@@ -14,12 +14,6 @@ module Unparser
 
       handle(*MAP.keys)
 
-      def emit_heredoc_reminders
-        children.each do |node|
-          emitter(node).emit_heredoc_reminders
-        end
-      end
-
     private
 
       def dispatch

data/lib/unparser/emitter/for.rb

@@ -18,7 +18,7 @@ module Unparser
       end
 
       def emit_condition
-        visit(condition)
+        emitter(condition).emit_mlhs
         write(' in ')
         visit(assignment)
         write(' do')

data/lib/unparser/emitter/hash.rb

@@ -6,10 +6,6 @@ module Unparser
     class Hash < self
       handle :hash
 
-      def emit_heredoc_reminders
-        children.each(&method(:emit_heredoc_reminder_member))
-      end
-
     private
 
       def dispatch
@@ -24,10 +20,6 @@ module Unparser
         end
       end
 
-      def emit_heredoc_reminder_member(node)
-        emitter(node.children.last).emit_heredoc_reminders if n_pair?(node)
-      end
-
       def emit_hash_body
         delimited(children)
       end

data/lib/unparser/emitter/hash_pattern.rb

@@ -32,7 +32,7 @@ module Unparser
         when :match_var
           emit_match_var(node)
         when :match_rest
-          writer_with(MatchRest, node).emit_hash_pattern
+          writer_with(MatchRest, node:).emit_hash_pattern
         else
           visit(node)
         end

data/lib/unparser/emitter/index.rb

@@ -40,10 +40,6 @@ module Unparser
 
         private_constant(*constants(false))
 
-        def emit_heredoc_reminders
-          emitter(children.last).emit_heredoc_reminders
-        end
-
         def dispatch
           emit_receiver
           emit_operation(children[VALUE_RANGE])

data/lib/unparser/emitter/op_assign.rb

@@ -14,11 +14,6 @@ module Unparser
 
       handle(*MAP.keys)
 
-      def emit_heredoc_reminders
-        emitter(target).emit_heredoc_reminders
-        emitter(expression).emit_heredoc_reminders
-      end
-
     private
 
       def dispatch
@@ -35,11 +30,6 @@ module Unparser
 
       children :target, :operator, :value
 
-      def emit_heredoc_reminders
-        emitter(target).emit_heredoc_reminders
-        emitter(value).emit_heredoc_reminders
-      end
-
     private
 
       def dispatch