rubocop-socketry 0.9.0 → 0.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bcb70c0ac2421d12d0a38c1e28338e67b3d57e1bef6d9d0121a00c6d042953d5
-  data.tar.gz: f6754cbd98784332753e86c9fc5261ba8214708f94c4d7ca2e7783f21520c956
+  metadata.gz: 6afce664365a0b7fa9a50a48b6aebc9f05b1de2755f8bc133a85b051b0819d79
+  data.tar.gz: fb88cb2a6b157b82269bc2afa51cbe08d9327b8d885b4b7baab08180bf2dcd62
 SHA512:
-  metadata.gz: 696a82e7609dc56b42bb91bbd4a13d998c1bc64f65e227fc63dcba97e0f15740e883007e1766cc9b30e36ad30fca6e74b03d3aca10c0f23fce47dd87c9b1062a
-  data.tar.gz: 60e83ca55460e2af2dac385563f139edb7fbe4e0fc2b0b7de82c0110ca06c4360d6b4c639278a09f64be313ccfe4c442a3ecc4a7e024923e86487967ce5fc15a
+  metadata.gz: a433da1fbc1394419fc20e73c4c3a883b5a18ca58b7e0ff46e070a9f7aee7139e6dee1edda29053c1d2015650714db1aaa3152ae479716ef6269d880191e4dc0
+  data.tar.gz: 3aa6a42823410aae0c2b7889915c0857c54efbcb3f83e8a7f07a83cf98014a862ec479f1ba0db7aeb1017bb3b4aabd3197be7a194d6f7a4473723eee5f265300

checksums.yaml.gz.sig

@@ -1,3 +1,2 @@
-��=�-��rY�!5A��Y�.��7LY붺�
-��� L�5�V����b�
-��is�0'(Ȟ���ASKUk���D���*zKm4�T�#4��3����:|җK뷪e��!u@�*L1��45p*M�����+�M>��v��eK���,���_�$�B�L�1���},P���C�v��/�`L9�/�8���Fޱ2by˵QN�������I5����J�F\��8o-?2�d����X4	�<:���9��y�&��:싺Y@�M��4���'O��;��θ��eC���T63
=�9�M���W��ߡ�\eEX���ͱ�D�ޗ��9��s�u��b��8r�F�8���+u�A�
+OHBp��&�s&o_�i���R�2�N��K-��@����7�ʓµ���27�,����"K/
+B;�Z��g@�����f�e�4�2���*��"���=�6٠��IqC��'�(�;��[�c{_g>^�<s�z#�/Ş��GF�9�
�j��}�@�������?�����

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?
 					
@@ -86,9 +88,25 @@ module RuboCop
 					parent = node.parent
 					return false unless parent
 					
-					# If parent is a :begin node (sequence of statements), this is top-level
-					# Otherwise, it's part of an expression or nested context
-					parent.type != :begin
+					case parent.type
+					when :begin
+						# Sequence of statements — this is a top-level statement.
+						false
+					when :block, :numblock
+						# When a block is the sole body statement of an outer block there is no
+						# :begin wrapper; the outer block node is the direct parent.
+						# Only the body slot counts as a statement context, and only when the
+						# outer block itself spans multiple lines (e.g. a do…end describe/context
+						# block).  Single-line inline blocks (`foo {bar{baz}}`) keep their
+						# compact style.
+						parent.body == node ? !parent.multiline? : true
+					when :def, :defs, :class, :module, :sclass
+						# Same situation for method / class / module bodies with a single
+						# statement — they are always multi-line in practice.
+						parent.body != node
+					else
+						true
+					end
 				end
 				
 				# Check that there's no space before the opening brace for lambdas

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
 				

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.9.0"
+		VERSION = "0.11.0"
 	end
 end 

data/readme.md

@@ -27,6 +27,20 @@ Layout/ConsistentBlankLineIndentation:
 
 Please see the [project releases](https://socketry.github.io/rubocop-socketry/releases/index) for all releases.
 
+### 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.
+  - Single-line inline outer blocks (e.g. `foo {bar{baz}}`) continue to use compact style with no space before the inner brace.
+
+### v0.9.0
+
+  - Fixed `Layout/ConsistentBlankLineIndentation` to correctly handle files containing multiple blocks.
+  - Expanded `Layout/BlockDelimiterSpacing` test coverage to include method chaining scenarios.
+
 ### v0.8.0
 
   - Fixed `Layout/BlockDelimiterSpacing` to correctly distinguish between statement and expression contexts for blocks inside `do...end` blocks.

data/releases.md

@@ -1,5 +1,19 @@
 # Releases
 
+## 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.
+  - Single-line inline outer blocks (e.g. `foo {bar{baz}}`) continue to use compact style with no space before the inner brace.
+
+## v0.9.0
+
+  - Fixed `Layout/ConsistentBlankLineIndentation` to correctly handle files containing multiple blocks.
+  - Expanded `Layout/BlockDelimiterSpacing` test coverage to include method chaining scenarios.
+
 ## v0.8.0
 
   - Fixed `Layout/BlockDelimiterSpacing` to correctly distinguish between statement and expression contexts for blocks inside `do...end` blocks.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-socketry
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.11.0
 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: []