Shopify / Shopify/ruby-lsp

Go to definition fails on the last character of a reference

Open
#3,308 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug help-wanted pinned server
Dominant language
Ruby
Stars
2k
Forks
281
Avg merge
2h 14m
Merged PRs (30d)
6

Description

Currently, we fail to locate the target for a go to definition request if the cursor is immediately after the reference the user is trying to jump to. For example:

Foo
#  ^ cursor position

Bar.something
#            ^ cursor position

Initially, I thought this implementation was correct, since the reference's location given to us by Prism really does not cover that one extra character. The cursor is in the next immediate character, not on top of the reference.

However, I checked what other LSPs do and apparently they do catch the reference even if the cursor is one character beyond.

I think we have two possible solutions for this:

  1. The more general option is to consider an inclusive range here. In other words, just change that line to be
# Remove one of the dots to make the range inclusive
next unless (loc_start_offset..loc_end_offset).cover?(char_position)

If this works properly, without causing any weirdness in one of our features, then this is the appropriate solution.

  1. If 1 is not viable, then we should perform some sort of off-by-one correction in the definition request when locating targets. This is not ideal, but would fix the scenario

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at lib/ruby_lsp/ruby_document.rb around the linked range check, then trace how go-to-definition locates references. Reproduce the Foo and Bar.something cursor positions and evaluate the inclusive-range option against existing behavior; done means both immediately-after-reference cases resolve without breaking other features.

Written by the indexing model from the issue text.

Assessment

Tech stack
ruby
Domain
devtools
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.