feat(definition): go to dependency - #171

tests pass for me locally

make sure dependencies don't show up in Find References

I don't think the existing tests handle this, so I wouldn't call it completed yet. same for workspace symbols

I think that this section of functionality might warrant it's own test module, as it will be testing workspace symbols, references, and definition.

that way you can restrict the proliferation of code that has to install a dependency to as few tests as possible, as literally downloading a dep during every test run is bound to be brittle and time consuming.

actually, it might make more sense to use a path dependency for these tests, and write those files in the same was as we do the normal tests.

actually running mix deps.get inside the tests seems like a red flag to me

actually, it might make more sense to use a path dependency for these tests, and write those files in the same was as we do the normal tests.

actually running mix deps.get inside the tests seems like a red flag to me

i think you’re right, but this got me thinking: don’t we want go to definition for path dependencies? does this include umbrella applications? how does this relate to workspaces? Let’s say I open a mono repo, it would be nice to have go to definition across projects.

Path deps should work the same as remote deps.

Umbrella apps should work already.

Monorepo project would most likely be a dep

@mhanberg what do you want to do actually for the workspace symbols: filter them in the query or avoid inserting them in the DB in te first place? If it's the latter: we do this in the sidecar then?

(this unpushed test is currently failing ^^)

I pushed the former, let me know if you want it differently :)

This PR is blocked on several things I am working on.

1. Improved schema migration strategy
2. Symbol indexing tracker
3. Add new column to symbols table

You should be able to rebase now and work on this again.

There is a new "source" column on the symbols table that defaults to "user".

Then, when you create a separate tracer for compiling the dependencies, you can have that one insert the symbols with the source being set to "dep"

then when you are filtering on them for workspace symbols, you can say WHERE source = 'user'

need to figure out how to handle this

CleanShot.2023-08-14.at.18.29.04.mp4

need to figure out how to handle this

I have a fix for this, just writing a test right now