feat: add go to definition for imports

[PeterCardenas]

Adds the go to definition capability for navigating to import files. This functionality already exists via document link, but I think it's more/also appropriate as a definition.
I can also imagine going to definition for flag aliases as well as targets in the future.

[salesforce-cla]

Thanks for the contribution! Before we can merge this, we need @PeterCardenas to sign the Salesforce Inc. Contributor License Agreement.

[vogelsgesang]

At least for my personal neo-vim configuration (where I only have keybindings for go-to-definition), this is certainly an improvement. Thanks for opening this Pull Request!

Did you double-check what the behavior in VS-Code is, if there is both a link and a "go-to-definition" available on the same line?

[vogelsgesang]

Oh, and I do actually need you to Salesforce CLA before I can merge this

[PeterCardenas]

that's weird, it says i signed it already (i did try before when i put up the pr but there was some error i didn't get back to)

[vogelsgesang]

🤔 maybe the check only re-triggers when something in the Pull Request gets changed. Let's see if this gets auto-resolved after updating the code

[PeterCardenas]

ope looks like that was it, i'll check vscode when i get a chance to set it up. i have neovim setup with document link support with lsplinks and both still work fine tho

[vogelsgesang]

I verified that this also works for VS-Code