rubocop-ast 1.18.0 → 1.24.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ecc44200cb01d2a920c3f0dc730904dce038f0ce7ad48b18c1188f5f39395bf6
-  data.tar.gz: 65cf679ce211a1772606eef3ecf27c29732f8bd757b332d0324323f1b1898fb6
+  metadata.gz: f460fee9d6682f0942b107a76c8a24e59b3992e27cdb791a8d9d3e8902596bf0
+  data.tar.gz: 27b55278782ffad52905335f6ae6b1b1709e6156802d669158f903c4b04fecbd
 SHA512:
-  metadata.gz: b9da19828233f088bb4de259c1448d33a4c021daceca170c809eb7c413496bf0791476e032b0632eedff463adc6a16916c6c37c5c7deb3957e0bb0ebbcaf6bbe
-  data.tar.gz: 1cbf941b340dcc64fa3d727aaff4c5dbba9a47561b02b01ed12ac96f1b8ef53fa9f6b7f3b361e622368d4215c373b3af2bee0b012b9078b2460726a2306b3bd3
+  metadata.gz: f6302e8a65e1504fbe3a929f0e50d063ea07873c9650edc1f68f65525e57a2717f06fc87b64970cc98c06e63efa1058b16d6563aa2b2863a0a80a67b04c31e98
+  data.tar.gz: f8c7043155b9fb25ae1509c5bc8d3efe986ae9a74e302e886c3ad4947e5c06b79fd2ede7ba54aebdd3dc8bfe8b9f813ee5b6f2209140c7fee475dc31830f3d97

data/lib/rubocop/ast/builder.rb

@@ -75,7 +75,7 @@ module RuboCop
         rescue:       RescueNode,
         resbody:      ResbodyNode,
         return:       ReturnNode,
-        csend:        SendNode,
+        csend:        CsendNode,
         send:         SendNode,
         str:          StrNode,
         xstr:         StrNode,

data/lib/rubocop/ast/ext/range.rb

@@ -25,4 +25,4 @@ module RuboCop
   end
 end
 
-::Parser::Source::Range.include ::RuboCop::AST::Ext::Range
+Parser::Source::Range.include RuboCop::AST::Ext::Range

data/lib/rubocop/ast/node/csend_node.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module AST
+    # A node extension for `send` nodes. This will be used in place of a plain
+    # node when the builder constructs the AST, making its methods available
+    # to all `send` nodes within RuboCop.
+    class CsendNode < SendNode
+      def send_type?
+        false
+      end
+    end
+  end
+end

data/lib/rubocop/ast/node/mixin/descendence.rb

@@ -36,7 +36,10 @@ module RuboCop
       #
       # @return [Array<Node>] an array of child nodes
       def child_nodes
-        each_child_node.to_a
+        # Iterate child nodes directly to avoid allocating an Enumerator.
+        nodes = []
+        each_child_node { |node| nodes << node }
+        nodes
       end
 
       # Calls the given block for each descendant node with depth first order.
@@ -102,7 +105,9 @@ module RuboCop
       protected
 
       def visit_descendants(types, &block)
-        each_child_node do |child|
+        children.each do |child|
+          next unless child.is_a?(::AST::Node)
+
           yield child if types.empty? || types.include?(child.type)
           child.visit_descendants(types, &block)
         end

data/lib/rubocop/ast/node/mixin/method_dispatch_node.rb

@@ -211,7 +211,7 @@ module RuboCop
       #
       # @return [Boolean] whether this method is a lambda literal
       def lambda_literal?
-        block_literal? && loc.expression && loc.expression.source == '->'
+        loc.expression.source == '->' && block_literal?
       end
 
       # Checks whether this is a unary operation.
@@ -233,7 +233,7 @@ module RuboCop
       #
       #   foo + bar
       #
-      # @return [Bookean] whether this method is a binary operation
+      # @return [Boolean] whether this method is a binary operation
       def binary_operation?
         return false unless loc.selector
 
@@ -249,7 +249,7 @@ module RuboCop
           ^{                                       # or the parent is...
             sclass class module class_constructor? # a class-like node
             [ {                                    # or some "wrapper"
-                kwbegin begin block
+                kwbegin begin block numblock
                 (if _condition <%0 _>)  # note: we're excluding the condition of `if` nodes
               }
               #in_macro_scope?                     # that is itself in a macro scope

data/lib/rubocop/ast/node/mixin/parameterized_node.rb

@@ -82,9 +82,17 @@ module RuboCop
       # and optimizes other calls
       module RestArguments
         include ParameterizedNode
+
+        EMPTY_ARGUMENTS = [].freeze
+
         # @return [Array<Node>] arguments, if any
         def arguments
-          children[first_argument_index..].freeze
+          if arguments?
+            children.drop(first_argument_index).freeze
+          else
+            # Skip unneeded Array allocation.
+            EMPTY_ARGUMENTS
+          end
         end
 
         # A shorthand for getting the first argument of the node.

data/lib/rubocop/ast/node/regexp_node.rb

@@ -11,6 +11,7 @@ module RuboCop
         i: Regexp::IGNORECASE,
         m: Regexp::MULTILINE,
         n: Regexp::NOENCODING,
+        u: Regexp::FIXEDENCODING,
         o: 0
       }.freeze
       private_constant :OPTIONS
@@ -87,6 +88,11 @@ module RuboCop
         regopt_include?(:n)
       end
 
+      # @return [Bool] if regexp uses the fixed-encoding regopt
+      def fixed_encoding?
+        regopt_include?(:u)
+      end
+
       private
 
       def regopt_include?(option)

data/lib/rubocop/ast/node/send_node.rb

@@ -15,6 +15,10 @@ module RuboCop
          (_    _    _                                                 _ ...)]
       PATTERN
 
+      def send_type?
+        true
+      end
+
       private
 
       def first_argument_index

data/lib/rubocop/ast/node/str_node.rb

@@ -8,6 +8,10 @@ module RuboCop
     class StrNode < Node
       include BasicLiteralNode
 
+      def character_literal?
+        loc.respond_to?(:begin) && loc.begin && loc.begin.is?('?')
+      end
+
       def heredoc?
         loc.is_a?(Parser::Source::Map::Heredoc)
       end

data/lib/rubocop/ast/node.rb

@@ -84,8 +84,12 @@ module RuboCop
       LITERAL_RECURSIVE_TYPES = (OPERATOR_KEYWORDS + COMPOSITE_LITERALS + %i[begin pair]).freeze
       private_constant :LITERAL_RECURSIVE_METHODS, :LITERAL_RECURSIVE_TYPES
 
+      EMPTY_CHILDREN = [].freeze
+      EMPTY_PROPERTIES = {}.freeze
+      private_constant :EMPTY_CHILDREN, :EMPTY_PROPERTIES
+
       # @see https://www.rubydoc.info/gems/ast/AST/Node:initialize
-      def initialize(type, children = [], properties = {})
+      def initialize(type, children = EMPTY_CHILDREN, properties = EMPTY_PROPERTIES)
         @mutable_attributes = {}
 
         # ::AST::Node#initialize freezes itself.
@@ -101,11 +105,19 @@ module RuboCop
         end
       end
 
-      Parser::Meta::NODE_TYPES.each do |node_type|
+      (Parser::Meta::NODE_TYPES - [:send]).each do |node_type|
         method_name = "#{node_type.to_s.gsub(/\W/, '')}_type?"
-        define_method(method_name) do
-          type == node_type
-        end
+        class_eval <<~RUBY, __FILE__, __LINE__ + 1
+          def #{method_name}          # def block_type?
+            @type == :#{node_type}    #   @type == :block
+          end                         # end
+        RUBY
+      end
+
+      # Most nodes are of 'send' type, so this method is defined
+      # separately to make this check as fast as possible.
+      def send_type?
+        false
       end
 
       # Returns the parent node, or `nil` if the receiver is a root node.
@@ -203,9 +215,7 @@ module RuboCop
       # destructuring method.
       #
       # @return [Array<Node>] the different parts of the ndde
-      def node_parts
-        to_a
-      end
+      alias node_parts to_a
 
       # Calls the given block for each ancestor node from parent to root.
       # If no block is given, an `Enumerator` is returned.

data/lib/rubocop/ast/node_pattern/compiler/binding.rb

@@ -20,8 +20,8 @@ module RuboCop
             end
 
             if var == :forbidden_unification
-              raise Invalid, "Wildcard #{name} was first seen in a subset of a" \
-                             " union and can't be used outside that union"
+              raise Invalid, "Wildcard #{name} was first seen in a subset of a " \
+                             "union and can't be used outside that union"
             end
             var
           end

data/lib/rubocop/ast/node_pattern.rb

@@ -54,6 +54,22 @@ module RuboCop
 
       VAR = 'node'
 
+      # Yields its argument and any descendants, depth-first.
+      #
+      def self.descend(element, &block)
+        return to_enum(__method__, element) unless block
+
+        yield element
+
+        if element.is_a?(::RuboCop::AST::Node)
+          element.children.each do |child|
+            descend(child, &block)
+          end
+        end
+
+        nil
+      end
+
       attr_reader :pattern, :ast, :match_code
 
       def_delegators :@compiler, :captures, :named_parameters, :positional_parameters
@@ -100,22 +116,6 @@ module RuboCop
         initialize(coder['pattern'])
       end
 
-      # Yields its argument and any descendants, depth-first.
-      #
-      def self.descend(element, &block)
-        return to_enum(__method__, element) unless block
-
-        yield element
-
-        if element.is_a?(::RuboCop::AST::Node)
-          element.children.each do |child|
-            descend(child, &block)
-          end
-        end
-
-        nil
-      end
-
       def freeze
         @match_code.freeze
         @compiler.freeze

data/lib/rubocop/ast/processed_source.rb

@@ -12,6 +12,9 @@ module RuboCop
       # @api private
       STRING_SOURCE_NAME = '(string)'
 
+      INVALID_LEVELS = %i[error fatal].freeze
+      private_constant :INVALID_LEVELS
+
       attr_reader :path, :buffer, :ast, :comments, :tokens, :diagnostics,
                   :parser_error, :raw_source, :ruby_version
 
@@ -20,9 +23,6 @@ module RuboCop
         new(file, ruby_version, path)
       end
 
-      INVALID_LEVELS = %i[error fatal].freeze
-      private_constant :INVALID_LEVELS
-
       def initialize(source, ruby_version, path = nil)
         # Defaults source encoding to UTF-8, regardless of the encoding it has
         # been read with, which could be non-utf8 depending on the default

data/lib/rubocop/ast/sexp.rb

@@ -9,7 +9,8 @@ module RuboCop
     module Sexp
       # Creates a {Node} with type `type` and children `children`.
       def s(type, *children)
-        Node.new(type, children)
+        klass = Builder::NODE_MAP[type] || Node
+        klass.new(type, children)
       end
     end
   end

data/lib/rubocop/ast/token.rb

@@ -4,6 +4,8 @@ module RuboCop
   module AST
     # A basic wrapper around Parser's tokens.
     class Token
+      LEFT_PAREN_TYPES = %i[tLPAREN tLPAREN2].freeze
+
       attr_reader :pos, :type, :text
 
       def self.from_parser_token(parser_token)
@@ -89,7 +91,7 @@ module RuboCop
       end
 
       def left_parens?
-        %i[tLPAREN tLPAREN2].include?(type)
+        LEFT_PAREN_TYPES.include?(type)
       end
 
       def right_parens?
@@ -100,6 +102,14 @@ module RuboCop
         type == :tCOMMA
       end
 
+      def dot?
+        type == :tDOT
+      end
+
+      def regexp_dots?
+        %i[tDOT2 tDOT3].include?(type)
+      end
+
       def rescue_modifier?
         type == :kRESCUE_MOD
       end
@@ -111,6 +121,10 @@ module RuboCop
       def equal_sign?
         %i[tEQL tOP_ASGN].include?(type)
       end
+
+      def new_line?
+        type == :tNL
+      end
     end
   end
 end

data/lib/rubocop/ast/traversal.rb

@@ -75,7 +75,8 @@ module RuboCop
       ### arity == 0
       no_children = %i[true false nil self cbase zsuper redo retry
                        forward_args forwarded_args match_nil_pattern
-                       forward_arg lambda empty_else kwnilarg
+                       forward_arg forwarded_restarg forwarded_kwrestarg
+                       lambda empty_else kwnilarg
                        __FILE__ __LINE__ __ENCODING__]
 
       ### arity == 0..1

data/lib/rubocop/ast/version.rb

@@ -3,7 +3,7 @@
 module RuboCop
   module AST
     module Version
-      STRING = '1.18.0'
+      STRING = '1.24.1'
     end
   end
 end

data/lib/rubocop/ast.rb

@@ -76,6 +76,7 @@ require_relative 'ast/node/resbody_node'
 require_relative 'ast/node/return_node'
 require_relative 'ast/node/self_class_node'
 require_relative 'ast/node/send_node'
+require_relative 'ast/node/csend_node'
 require_relative 'ast/node/str_node'
 require_relative 'ast/node/dstr_node'
 require_relative 'ast/node/super_node'
@@ -91,5 +92,5 @@ require_relative 'ast/token'
 require_relative 'ast/traversal'
 require_relative 'ast/version'
 
-::RuboCop::AST::NodePattern::Parser.autoload :WithMeta, "#{__dir__}/ast/node_pattern/with_meta"
-::RuboCop::AST::NodePattern::Compiler.autoload :Debug, "#{__dir__}/ast/node_pattern/compiler/debug"
+RuboCop::AST::NodePattern::Parser.autoload :WithMeta, "#{__dir__}/ast/node_pattern/with_meta"
+RuboCop::AST::NodePattern::Compiler.autoload :Debug, "#{__dir__}/ast/node_pattern/compiler/debug"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-ast
 version: !ruby/object:Gem::Version
-  version: 1.18.0
+  version: 1.24.1
 platform: ruby
 authors:
 - Bozhidar Batsov
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-05-13 00:00:00.000000000 Z
+date: 2022-12-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parser
@@ -76,6 +76,7 @@ files:
 - lib/rubocop/ast/node/casgn_node.rb
 - lib/rubocop/ast/node/class_node.rb
 - lib/rubocop/ast/node/const_node.rb
+- lib/rubocop/ast/node/csend_node.rb
 - lib/rubocop/ast/node/def_node.rb
 - lib/rubocop/ast/node/defined_node.rb
 - lib/rubocop/ast/node/dstr_node.rb