rubocop-socketry 0.10.0 → 0.11.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36f3f182b831bb4fbae7b9eddb5504cecb05d10616de4b25307d83baa9e578e1
-  data.tar.gz: a12e31b5ad9834c890ecc20afd9d443ba0b6a760078ee4a5550e63b731e45c4e
+  metadata.gz: a21ad68f85134a6078dd60a1c413e3bf0b7ef289ff5a0bfe9f248f441c9c4311
+  data.tar.gz: 146288237609f6577df58e013d0803737e25fa2815c8706740c004bc4c0099c6
 SHA512:
-  metadata.gz: ffe0bfaa3a740753d590f0852f87374976c0a76e931d0184f63fc553f37a4c5547838d6ab1103f658fecdc4cc3c2a7c30502e813f7987e0dad25d3f4e86ac74f
-  data.tar.gz: c1b77f49c5bdb0fee928666a1f9436b37ae2d38f35cc17fc5aa274034cd61a31002d7435caf9d4e308fe49ed871e28987d3a9355a255c00c9ec1c06047d4afbe
+  metadata.gz: cbfb896891caa89fc6cbdb8aa470520f7412608fad2f6b785d3d5adbb9907787e9bb6f9489dd34e256b067ec917ae054c4e3d6b56c47462a79a0baea49fb3fd3
+  data.tar.gz: 38a101cefb2e29216db89282579c227878b4630703275d8a038dadee2cb11bd95385b828b9f11a29012eb2c5ce396c17ad95a5b289d42b6bf4c7d8faa9ba56e2

data/lib/rubocop/socketry/layout/block_delimiter_spacing.rb

@@ -27,6 +27,8 @@ module RuboCop
 				MSG_REMOVE_SPACE_LAMBDA = "Remove space before the opening brace for lambdas/procs."
 				MSG_REMOVE_SPACE_EXPRESSION = "Remove space before the opening brace for expressions."
 				
+				# Check each brace-delimited block for the required spacing before `{`.
+				# @parameter node [RuboCop::AST::BlockNode] The block node to inspect.
 				def on_block(node)
 					return unless node.braces?
 					

data/lib/rubocop/socketry/layout/consistent_blank_line_indentation.rb

@@ -49,6 +49,7 @@ module RuboCop
 				def on_new_investigation
 					indentation_deltas = build_indentation_deltas
 					current_level = 0
+					semantic_level = 0
 					
 					processed_source.lines.each_with_index do |line, index|
 						line_number = index + 1
@@ -75,7 +76,16 @@ module RuboCop
 							end
 						end
 						
-						current_level += delta
+						if delta > 0
+							current_level += 1
+							semantic_level += delta
+						elsif delta < 0
+							semantic_level += delta
+							current_level = [current_level, semantic_level].min
+						end
+						
+						current_level = [current_level, 0].max
+						semantic_level = [semantic_level, 0].max
 					end
 				end
 				
@@ -87,10 +97,6 @@ module RuboCop
 				def build_indentation_deltas
 					Hash.new(0).tap do |deltas|
 						walk_ast_for_indentation(processed_source.ast, deltas)
-						
-						# Cap deltas to a maximum of +1 or -1 per line to handle cases where
-						# multiple structures open/close on the same line (e.g., `[..., {`)
-						deltas.transform_values!{|delta| delta.nil? ? nil : delta.clamp(-1, 1)}
 					end
 				end
 				
@@ -114,8 +120,8 @@ module RuboCop
 							end
 						end
 					when :if
-						# We don't want to add deltas for elsif, because it's handled by the if node:
-						if node.keyword == "if" || node.keyword == "unless"
+						# Modifier conditionals don't introduce indentation, and elsif is handled by the if node:
+						if !node.modifier_form? && (node.keyword == "if" || node.keyword == "unless")
 							if location = node.location
 								deltas[location.line] += 1
 								deltas[location.last_line] -= 1

data/lib/rubocop/socketry/style/global_exception_variables.rb

@@ -66,6 +66,8 @@ module RuboCop
 				
 				EXCEPTION_VARIABLES = %i[$! $@ $ERROR_INFO $ERROR_POSITION].freeze
 				
+				# Check each global variable reference for unsafe exception state access.
+				# @parameter node [RuboCop::AST::GlobalVariableNode] The global variable node to inspect.
 				def on_gvar(node)
 					variable_name = node.children.first
 					
@@ -122,4 +124,3 @@ module RuboCop
 		end
 	end
 end
-

data/lib/rubocop/socketry/version.rb

@@ -5,6 +5,6 @@
 
 module RuboCop
 	module Socketry
-		VERSION = "0.10.0"
+		VERSION = "0.11.1"
 	end
 end 

data/readme.md

@@ -27,6 +27,14 @@ Layout/ConsistentBlankLineIndentation:
 
 Please see the [project releases](https://socketry.github.io/rubocop-socketry/releases/index) for all releases.
 
+### v0.11.1
+
+  - Fixed `Layout/ConsistentBlankLineIndentation` to ignore modifier conditionals when calculating indentation depth.
+
+### v0.11.0
+
+  - Fixed `Layout/ConsistentBlankLineIndentation` to preserve semantic indentation depth for same-line nested structures, avoiding negative indentation levels when later closing lines are processed.
+
 ### v0.10.0
 
   - Fixed `Layout/BlockDelimiterSpacing` to correctly treat a block as a statement (requiring space before `{`) when it is the sole body of a multi-line outer block or method/class/module definition. Previously, the absence of a `:begin` wrapper in the AST caused such blocks (e.g. `Async {foo}` or `let(:bar) {baz}` inside a `describe`/`context` block) to be misclassified as expression-context and have their space incorrectly removed.
@@ -65,10 +73,6 @@ Please see the [project releases](https://socketry.github.io/rubocop-socketry/re
 
   - Added `Layout/BlockDelimiterSpacing` cop to enforce consistent spacing before block delimiters.
 
-### v0.1.0
-
-  - Initial implementation of `Layout/ConsistentBlankLineIndentation`.
-
 ## Contributing
 
 We welcome contributions to this project.

data/releases.md

@@ -1,5 +1,13 @@
 # Releases
 
+## v0.11.1
+
+  - Fixed `Layout/ConsistentBlankLineIndentation` to ignore modifier conditionals when calculating indentation depth.
+
+## v0.11.0
+
+  - Fixed `Layout/ConsistentBlankLineIndentation` to preserve semantic indentation depth for same-line nested structures, avoiding negative indentation levels when later closing lines are processed.
+
 ## v0.10.0
 
   - Fixed `Layout/BlockDelimiterSpacing` to correctly treat a block as a statement (requiring space before `{`) when it is the sole body of a multi-line outer block or method/class/module definition. Previously, the absence of a `:begin` wrapper in the AST caused such blocks (e.g. `Async {foo}` or `let(:bar) {baz}` inside a `describe`/`context` block) to be misclassified as expression-context and have their space incorrectly removed.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-socketry
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.11.1
 platform: ruby
 authors:
 - Samuel Williams
@@ -103,7 +103,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.6
+rubygems_version: 4.0.10
 specification_version: 4
 summary: RuboCop rules for Socketry projects
 test_files: []