ruby-lsp-mongoid 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 459274f7c2be280ff2e284869fcd8dc7e424dcce61799d881a27f7bc250edf75
-  data.tar.gz: 07ee1f8583968c5b01bc395df23e3b598967e9ae389b10100d4681eb5c592f9d
+  metadata.gz: 8fbccb41fea414098880ead42297015e17b7f8e375efd0a389fcf6ffc9f50ac8
+  data.tar.gz: f3e1eb7a9271a97b00f314f88e755a5aabfde5f9fb700d031e0709eb20bae9c2
 SHA512:
-  metadata.gz: c946f9ee440c08d5620f074272c1f34376624b4cfb7251bf26d4e7342a76792bd625d86b78eda2015da57eccdaf4b38be838e97e82a39ead6e8d73c8fe890750
-  data.tar.gz: 2343c3b4a06ecd64abf51015cac50ce6f45380148a796777860474e025e446526892a2f2a486235a0f5e38906258b9c2b6b07044d12500b38468ad2cfc8adcde
+  metadata.gz: f28b770bf135a4e1c1eddeeb3368ad05793054f535dd2ae8f08c07a6d2bbab75b2732227950cdb784e89d98ccf8747581eceff8fd347956eeda247aac21213a8
+  data.tar.gz: 17c8daf234ffb9ee7d3587437ae6439dd098018b4521dfc11ccb8315a20c2662e204f6385b66a065f2cf6e12f4184cdf2f433eb72fc23bc69f68a8722bc7905e

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 ## [Unreleased]
 
+### Fixed
+
+- Fix "can't add a new key into hash during iteration" error caused by concurrent hash modification when background signature update thread iterates while ruby-lsp modifies the index
+- Fix compatibility with ruby-lsp 0.26.5+ by using `@listener.add_method` instead of directly calling `Entry::Method.new`
+
 ## [0.1.2] - 2025-12-15
 
 ### Added

data/lib/ruby_lsp/ruby_lsp_mongoid/addon.rb

@@ -89,7 +89,10 @@ module RubyLsp
 
         # Find all classes that have this method registered by our addon
         # We look for methods that were registered with empty signatures
-        index.instance_variable_get(:@entries).each do |_name, entries|
+        # Use dup to avoid "can't add a new key into hash during iteration" error
+        # when ruby-lsp modifies the index concurrently
+        entries_snapshot = index.instance_variable_get(:@entries).dup
+        entries_snapshot.each do |_name, entries|
           entries.each do |entry|
             next unless entry.is_a?(RubyIndexer::Entry::Method)
             next unless entry.name == method_name
@@ -107,7 +110,9 @@ module RubyLsp
       def update_singleton_method_signatures_for_mongoid_models(index, method_name, new_signatures)
         updated = 0
 
-        index.instance_variable_get(:@entries).each do |_name, entries|
+        # Use dup to avoid "can't add a new key into hash during iteration" error
+        entries_snapshot = index.instance_variable_get(:@entries).dup
+        entries_snapshot.each do |_name, entries|
           entries.each do |entry|
             next unless entry.is_a?(RubyIndexer::Entry::Method)
             next unless entry.name == method_name

data/lib/ruby_lsp/ruby_lsp_mongoid/indexing_enhancement.rb

@@ -254,22 +254,14 @@ module RubyLsp
 
       def add_singleton_method_with_signatures(name, node_location, owner, signatures)
         index = @listener.instance_variable_get(:@index)
-        code_units_cache = @listener.instance_variable_get(:@code_units_cache)
-        uri = @listener.instance_variable_get(:@uri)
+        owner_stack = @listener.instance_variable_get(:@owner_stack)
 
-        location = RubyIndexer::Location.from_prism_location(node_location, code_units_cache)
         singleton = index.existing_or_new_singleton_class(owner.name)
+        owner_stack << singleton
 
-        index.add(RubyIndexer::Entry::Method.new(
-          name,
-          uri,
-          location,
-          location,
-          "",
-          signatures,
-          :public,
-          singleton,
-        ))
+        @listener.add_method(name, node_location, signatures)
+
+        owner_stack.pop
       end
 
       def extract_lambda_node(call_node)

data/lib/ruby_lsp_mongoid/version.rb

@@ -2,6 +2,6 @@
 
 module RubyLsp
   module Mongoid
-    VERSION = "0.1.2"
+    VERSION = "0.1.3"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ruby-lsp-mongoid
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Shia