Fix module and class scope closing by passing false for close_siblings

Co-authored-by: kwerle <23320+kwerle@users.noreply.github.com>

Author: Copilot

lib/ruby_language_server/scope_parser.rb

@@ -41,7 +41,7 @@ def visit_class_node(node)
       end_line = node.location.end_line
       column = node.location.start_column
 
-      scope = push_scope(ScopeData::Scope::TYPE_CLASS, name, line, column, end_line)
+      scope = push_scope(ScopeData::Scope::TYPE_CLASS, name, line, column, end_line, false)
 
       # Handle superclass
       if node.superclass
@@ -60,7 +60,7 @@ def visit_module_node(node)
       end_line = node.location.end_line
       column = node.location.start_column
 
-      push_scope(ScopeData::Scope::TYPE_MODULE, name, line, column, end_line)
+      push_scope(ScopeData::Scope::TYPE_MODULE, name, line, column, end_line, false)
       super
       pop_scope
     end

spec/lib/ruby_language_server/scope_parser_spec.rb

@@ -167,5 +167,69 @@ def some_method
         assert_equal('item', scope_parser.root_scope.self_and_descendants.last.variables.first.name)
       end
     end
+
+    describe 'sibling modules' do
+      let(:sibling_modules_source) do
+        <<-RUBY
+        module Foo
+          module Bar
+          end 
+          module Baz
+          end
+        end
+        RUBY
+      end
+      let(:scope_parser) { RubyLanguageServer::ScopeParser.new(sibling_modules_source) }
+
+      it 'should place sibling modules at the same level' do
+        foo = scope_parser.root_scope.children.first
+        assert_equal('Foo', foo.name)
+        
+        # Both Bar and Baz should be children of Foo, not Bar containing Baz
+        children = foo.children
+        assert_equal(2, children.size, "Foo should have 2 children, but has #{children.size}")
+        
+        bar = children.detect { |c| c.name == 'Bar' }
+        baz = children.detect { |c| c.name == 'Baz' }
+        
+        assert_not_nil(bar, "Bar should be a child of Foo")
+        assert_not_nil(baz, "Baz should be a child of Foo")
+        
+        # Verify Baz is not a child of Bar
+        assert_equal(0, bar.children.size, "Bar should have no children, but has #{bar.children.size}")
+      end
+    end
+
+    describe 'sibling classes' do
+      let(:sibling_classes_source) do
+        <<-RUBY
+        module Foo
+          class Bar
+          end 
+          class Baz
+          end
+        end
+        RUBY
+      end
+      let(:scope_parser) { RubyLanguageServer::ScopeParser.new(sibling_classes_source) }
+
+      it 'should place sibling classes at the same level' do
+        foo = scope_parser.root_scope.children.first
+        assert_equal('Foo', foo.name)
+        
+        # Both Bar and Baz should be children of Foo, not Bar containing Baz
+        children = foo.children
+        assert_equal(2, children.size, "Foo should have 2 children, but has #{children.size}")
+        
+        bar = children.detect { |c| c.name == 'Bar' }
+        baz = children.detect { |c| c.name == 'Baz' }
+        
+        assert_not_nil(bar, "Bar should be a child of Foo")
+        assert_not_nil(baz, "Baz should be a child of Foo")
+        
+        # Verify Baz is not a child of Bar
+        assert_equal(0, bar.children.size, "Bar should have no children, but has #{bar.children.size}")
+      end
+    end
   end
 end