Skip to content

Migrate find references to Rubydex#4066

Merged
vinistock merged 4 commits intorubydex_adoption_feature_branchfrom
vs_migrate_find_references_to_rubydex
Apr 30, 2026
Merged

Migrate find references to Rubydex#4066
vinistock merged 4 commits intorubydex_adoption_feature_branchfrom
vs_migrate_find_references_to_rubydex

Conversation

@vinistock
Copy link
Copy Markdown
Member

Motivation

Migrate find references to use Rubydex. There are two things missing, which I encoded in the tests:

  1. Indexing instance variable references (I have a branch going)
  2. Exposing method reference receivers, so that we can be more precise about references

Implementation

I migrated the request in a way that it's already prepared to account for instance, class and global variables as soon as we add support for it in Rubydex.

Automated Tests

Added a bunch of new tests trying to encode all sorts of behaviours that the previous ones didn't. For example:

  • Ensuring that the request only shows references if you have the cursor exactly on top of names
  • The placeholder behaviour for global, class and instance variables
  • Different variations of method references

@vinistock vinistock self-assigned this Apr 17, 2026
@vinistock vinistock requested a review from a team as a code owner April 17, 2026 13:46
@vinistock vinistock added server This pull request should be included in the server gem's release notes other Changes that aren't bugfixes, enhancements or breaking changes labels Apr 17, 2026
@vinistock vinistock requested review from alexcrocha and st0012 April 17, 2026 13:46
@vinistock vinistock mentioned this pull request Apr 20, 2026
19 tasks
@vinistock vinistock force-pushed the vs_fix_node_context_for_methods_with_receivers branch from 338eeee to 4c1f108 Compare April 21, 2026 15:02
@vinistock vinistock force-pushed the vs_migrate_find_references_to_rubydex branch from 3586cfd to d336389 Compare April 21, 2026 15:03
@vinistock vinistock force-pushed the vs_fix_node_context_for_methods_with_receivers branch from 4c1f108 to 9846ab4 Compare April 28, 2026 14:34
@vinistock vinistock force-pushed the vs_migrate_find_references_to_rubydex branch from d336389 to 9f72010 Compare April 28, 2026 14:35
Comment thread lib/ruby_lsp/requests/references.rb
@vinistock vinistock force-pushed the vs_fix_node_context_for_methods_with_receivers branch from 9846ab4 to c753e32 Compare April 30, 2026 13:34
@vinistock vinistock force-pushed the vs_migrate_find_references_to_rubydex branch from 9f72010 to dcc6d98 Compare April 30, 2026 13:35
@vinistock vinistock force-pushed the vs_fix_node_context_for_methods_with_receivers branch from c753e32 to 2cb06f8 Compare April 30, 2026 18:02
@vinistock vinistock force-pushed the vs_fix_node_context_for_methods_with_receivers branch from 2cb06f8 to b6b596d Compare April 30, 2026 18:03
@vinistock vinistock force-pushed the vs_migrate_find_references_to_rubydex branch from dcc6d98 to 1d78a19 Compare April 30, 2026 18:03
Base automatically changed from vs_fix_node_context_for_methods_with_receivers to rubydex_adoption_feature_branch April 30, 2026 19:00
@vinistock vinistock merged commit 2ace51b into rubydex_adoption_feature_branch Apr 30, 2026
52 of 53 checks passed
@vinistock vinistock deleted the vs_migrate_find_references_to_rubydex branch April 30, 2026 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

other Changes that aren't bugfixes, enhancements or breaking changes server This pull request should be included in the server gem's release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants