rubocop-vibe 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f214588afca7a72328401c384b819415ddae3ac7e4fa2c214bfb7225f4253b56
-  data.tar.gz: d4a9267839bcd11320ca97f7ef64f17134618f7efee156b4f35b8f1a2a07ad1e
+  metadata.gz: 32efe500f7fb50e4005a4ef34081405eee947bc058b96c684e60d3edb1095976
+  data.tar.gz: 79b4d44786594d509536e369756ead61b0cd34cd54cf288f857b4fa04e90f0a5
 SHA512:
-  metadata.gz: bb99fe49c2b921d70e07a6d2c6c4ea035fa76867a3391803f586575a7631d19d176af76791778aedf33178bfa6cb8f9fa3aebe1a66950b14b88c7dfd3823b101
-  data.tar.gz: 4527a16179dbb9b2fa026639f47409a706ba393564b50cc0f6f5544df4b23c9faf6abfc720e319c1a88e92b7f40e2eed553eb720853345134d6d8f754791a9c9
+  metadata.gz: 62fb11ebb915d21241f1c0da9b6684564e457aeaa0469ce2fc5ec708bfaf1754f8b4f85dfc7167b7ea68b9e498424d6feb3e6afb57673f28a2921e88fba56f3d
+  data.tar.gz: 10bdf89f1dcfd15dbdfe06d293d3a6633193ecf953d50133be333d50b70b3ed7f84f081f6d40c4cbafd3fa68ae50839958ad1fe92eafddd44151b3f1c068e090

data/config/default.yml

@@ -85,6 +85,7 @@ Vibe/ClassOrganization:
   Enabled: true
   SafeAutoCorrect: false
   VersionAdded: "0.1.0"
+  VersionChanged: "0.6.0"
 
 Vibe/ConsecutiveAssignmentAlignment:
   Description: "Enforces alignment of consecutive variable assignments at the = operator."
@@ -133,6 +134,7 @@ Vibe/DescribeBlockOrder:
   Enabled: true
   SafeAutoCorrect: false
   VersionAdded: "0.1.0"
+  VersionChanged: "0.6.0"
 
 Vibe/ExplicitReturnConditional:
   Description: "Enforces explicit if/else/end blocks instead of ternary or trailing conditionals for return values."

data/lib/rubocop/cop/vibe/attr_order.rb

@@ -42,14 +42,6 @@ module RuboCop
 
         private
 
-        # Check if the node is an attr_* method call.
-        #
-        # @param [RuboCop::AST::Node] node The send node.
-        # @return [Boolean]
-        def attr_method?(node)
-          node.receiver.nil? && ATTR_METHODS.include?(node.method_name)
-        end
-
         # Check if all arguments are symbols.
         #
         # @param [Array<RuboCop::AST::Node>] arguments The arguments.
@@ -68,6 +60,14 @@ module RuboCop
           names == names.sort
         end
 
+        # Check if the node is an attr_* method call.
+        #
+        # @param [RuboCop::AST::Node] node The send node.
+        # @return [Boolean]
+        def attr_method?(node)
+          node.receiver.nil? && ATTR_METHODS.include?(node.method_name)
+        end
+
         # Auto-correct by reordering arguments alphabetically.
         #
         # @param [RuboCop::AST::Corrector] corrector The corrector.

data/lib/rubocop/cop/vibe/blank_line_after_assignment.rb

@@ -68,6 +68,45 @@ module RuboCop
 
         private
 
+        # Get the variable name from an assignment node.
+        #
+        # @param [RuboCop::AST::Node] node The assignment node.
+        # @return [Symbol, nil]
+        def assigned_variable_name(node)
+          return node.children.first if node.lvasgn_type?
+
+          target = node.children.first
+
+          if target.lvasgn_type?
+            target.children.first
+          end
+        end
+
+        # Check if a node is a variable assignment.
+        #
+        # @param [RuboCop::AST::Node] node The node to check.
+        # @return [Boolean]
+        def assignment?(node)
+          node.type?(:lvasgn, :op_asgn, :or_asgn, :and_asgn)
+        end
+
+        # Get the value being assigned.
+        #
+        # @param [RuboCop::AST::Node] node The assignment node.
+        # @return [RuboCop::AST::Node, nil]
+        def assignment_value(node)
+          node.children.last
+        end
+
+        # Check if there's a blank line between two statements.
+        #
+        # @param [RuboCop::AST::Node] previous_node The previous statement.
+        # @param [RuboCop::AST::Node] current_node The current statement.
+        # @return [Boolean]
+        def blank_line_between?(previous_node, current_node)
+          current_node.loc.line - previous_node.loc.last_line > 1
+        end
+
         # Check the body for missing blank lines after assignments.
         #
         # @param [RuboCop::AST::Node] body The body node.
@@ -82,18 +121,6 @@ module RuboCop
           end
         end
 
-        # Extract statements from a body node.
-        #
-        # @param [RuboCop::AST::Node] body The body node.
-        # @return [Array<RuboCop::AST::Node>]
-        def extract_statements(body)
-          if body.begin_type?
-            body.children
-          else
-            [body]
-          end
-        end
-
         # Check a pair of statements for missing blank line after assignment.
         #
         # @param [RuboCop::AST::Node] current The current statement.
@@ -114,17 +141,6 @@ module RuboCop
           end
         end
 
-        # Get the inner statement from a modifier node.
-        #
-        # @param [RuboCop::AST::Node] node The node to unwrap.
-        # @return [RuboCop::AST::Node]
-        def inner_statement(node)
-          return node.if_branch if node.type?(:if) && node.modifier_form?
-          return node.body if node.type?(:while, :until) && node.modifier_form?
-
-          node
-        end
-
         # Check if both statements are FactoryBot calls.
         #
         # @param [RuboCop::AST::Node] assignment The assignment node.
@@ -134,12 +150,16 @@ module RuboCop
           factory_bot_call?(assignment_value(assignment)) && factory_bot_call?(following)
         end
 
-        # Get the value being assigned.
+        # Extract statements from a body node.
         #
-        # @param [RuboCop::AST::Node] node The assignment node.
-        # @return [RuboCop::AST::Node, nil]
-        def assignment_value(node)
-          node.children.last
+        # @param [RuboCop::AST::Node] body The body node.
+        # @return [Array<RuboCop::AST::Node>]
+        def extract_statements(body)
+          if body.begin_type?
+            body.children
+          else
+            [body]
+          end
         end
 
         # Check if a node is a FactoryBot method call.
@@ -167,18 +187,15 @@ module RuboCop
           receiver.lvar_type? && receiver.children.first == var_name
         end
 
-        # Get the variable name from an assignment node.
+        # Get the inner statement from a modifier node.
         #
-        # @param [RuboCop::AST::Node] node The assignment node.
-        # @return [Symbol, nil]
-        def assigned_variable_name(node)
-          return node.children.first if node.lvasgn_type?
-
-          target = node.children.first
+        # @param [RuboCop::AST::Node] node The node to unwrap.
+        # @return [RuboCop::AST::Node]
+        def inner_statement(node)
+          return node.if_branch if node.type?(:if) && node.modifier_form?
+          return node.body if node.type?(:while, :until) && node.modifier_form?
 
-          if target.lvasgn_type?
-            target.children.first
-          end
+          node
         end
 
         # Get the leftmost receiver in a method chain.
@@ -195,23 +212,6 @@ module RuboCop
 
           current
         end
-
-        # Check if a node is a variable assignment.
-        #
-        # @param [RuboCop::AST::Node] node The node to check.
-        # @return [Boolean]
-        def assignment?(node)
-          node.type?(:lvasgn, :op_asgn, :or_asgn, :and_asgn)
-        end
-
-        # Check if there's a blank line between two statements.
-        #
-        # @param [RuboCop::AST::Node] previous_node The previous statement.
-        # @param [RuboCop::AST::Node] current_node The current statement.
-        # @return [Boolean]
-        def blank_line_between?(previous_node, current_node)
-          current_node.loc.line - previous_node.loc.last_line > 1
-        end
       end
     end
   end

data/lib/rubocop/cop/vibe/blank_line_before_expectation.rb

@@ -55,36 +55,13 @@ module RuboCop
 
         private
 
-        # Check if the block should be processed.
+        # Check if there's a blank line between two statements.
         #
-        # @param [RuboCop::AST::Node] node The block node.
+        # @param [RuboCop::AST::Node] previous_node The previous statement.
+        # @param [RuboCop::AST::Node] current_node The current statement.
         # @return [Boolean]
-        def processable_block?(node)
-          spec_file? && example_block?(node) && node.body
-        end
-
-        # Extract statements from the block body.
-        #
-        # @param [RuboCop::AST::Node] body The block body.
-        # @return [Array<RuboCop::AST::Node>]
-        def extract_statements(body)
-          if body.begin_type?
-            body.children
-          else
-            [body]
-          end
-        end
-
-        # Check statements for missing blank lines before expectations.
-        #
-        # @param [Array<RuboCop::AST::Node>] statements The statements to check.
-        # @return [void]
-        def check_statements(statements)
-          statements.each_with_index do |statement, index|
-            next if index.zero?
-
-            check_statement_pair(statements[index - 1], statement)
-          end
+        def blank_line_between?(previous_node, current_node)
+          current_node.loc.line - previous_node.loc.last_line > 1
         end
 
         # Check a pair of statements for missing blank line.
@@ -102,14 +79,27 @@ module RuboCop
           register_offense(expect_node, previous_statement)
         end
 
-        # Register an offense for missing blank line.
+        # Check statements for missing blank lines before expectations.
         #
-        # @param [RuboCop::AST::Node] expect_node The expect node.
-        # @param [RuboCop::AST::Node] previous_statement The previous statement.
+        # @param [Array<RuboCop::AST::Node>] statements The statements to check.
         # @return [void]
-        def register_offense(expect_node, previous_statement)
-          add_offense(expect_node.loc.selector) do |corrector|
-            corrector.insert_after(previous_statement, "\n")
+        def check_statements(statements)
+          statements.each_with_index do |statement, index|
+            next if index.zero?
+
+            check_statement_pair(statements[index - 1], statement)
+          end
+        end
+
+        # Extract statements from the block body.
+        #
+        # @param [RuboCop::AST::Node] body The block body.
+        # @return [Array<RuboCop::AST::Node>]
+        def extract_statements(body)
+          if body.begin_type?
+            body.children
+          else
+            [body]
           end
         end
 
@@ -133,13 +123,23 @@ module RuboCop
           node.each_descendant(:send).find { |send_node| expect_call?(send_node) }
         end
 
-        # Check if there's a blank line between two statements.
+        # Check if the block should be processed.
         #
-        # @param [RuboCop::AST::Node] previous_node The previous statement.
-        # @param [RuboCop::AST::Node] current_node The current statement.
+        # @param [RuboCop::AST::Node] node The block node.
         # @return [Boolean]
-        def blank_line_between?(previous_node, current_node)
-          current_node.loc.line - previous_node.loc.last_line > 1
+        def processable_block?(node)
+          spec_file? && example_block?(node) && node.body
+        end
+
+        # Register an offense for missing blank line.
+        #
+        # @param [RuboCop::AST::Node] expect_node The expect node.
+        # @param [RuboCop::AST::Node] previous_statement The previous statement.
+        # @return [void]
+        def register_offense(expect_node, previous_statement)
+          add_offense(expect_node.loc.selector) do |corrector|
+            corrector.insert_after(previous_statement, "\n")
+          end
         end
       end
     end