Don't mark local X = require() as a definition when doing Go To Definition

[JohnnyMorganz]

In an example like:

-- useFunction.luau
return function(...)
end

-- MyExample.luau
local useFunction = require(path.to.useFunction)

local x = useFunction(...)

If you perform go to definition on useFunction in the last line, it will give 2 results: the actual function definition from useFunction.luau, but also the line local useFunction = require(...).

The latter result is not helpful, and makes the go to definition a longer process than necessary (VSCode will pop up a lens displaying the 2 definitions, then you have to click through to the right one).

We should make go to definition only return the underlying definition. The local useFunction = require(...) should only show up when we want to find all references.