Skip to content

Index instance variable references#757

Open
vinistock wants to merge 1 commit intomainfrom
04-13-index_instance_variable_references
Open

Index instance variable references#757
vinistock wants to merge 1 commit intomainfrom
04-13-index_instance_variable_references

Conversation

@vinistock
Copy link
Copy Markdown
Member

@vinistock vinistock commented Apr 20, 2026

Closes #517

This PR starts indexing and handling instance variable references, which is required to match the Ruby LSP's find references for ivars.

The approach is mostly the same as any other reference. We have a new reference struct that gets collected during indexing and then merged into the main graph.

Resolution is running as a separate step at the end, which reuses the same logic as instance variable definitions.

Copy link
Copy Markdown
Member Author

vinistock commented Apr 20, 2026

@vinistock vinistock changed the title WIP WIP: index instance variable references Apr 20, 2026
Base automatically changed from 04-13-ensure_singleton_class_ancestors_are_linearized_when_created_out_of_resolution_loop to main April 20, 2026 17:55
@vinistock vinistock force-pushed the 04-13-index_instance_variable_references branch from 3454bd4 to 9fa22d6 Compare April 22, 2026 13:20
@vinistock vinistock self-assigned this Apr 22, 2026
@vinistock vinistock added the enhancement New feature or request label Apr 22, 2026
@vinistock vinistock changed the title WIP: index instance variable references Index instance variable references Apr 22, 2026
@vinistock vinistock marked this pull request as ready for review April 22, 2026 13:23
@vinistock vinistock requested a review from a team as a code owner April 22, 2026 13:23
@vinistock vinistock force-pushed the 04-13-index_instance_variable_references branch from 9fa22d6 to 3351ec6 Compare April 24, 2026 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Index instance variable references

1 participant