nova-elixir-ls icon indicating copy to clipboard operation
nova-elixir-ls copied to clipboard

Not all symbols are available

Open eahanson opened this issue 3 years ago • 1 comments

When I use "Open Quickly in Symbols…", I get an incomplete list of symbols. I can't see a pattern to which symbols are found vs. not found. I've tried removing my .elixir_ls directory but it doesn't help. My project isn't particularly big: 78 Elixir files and 86 Elixir test files.

I'm happy to dig in and try to figure out what's going on, but I could use a point in the right direction.

Do you have an open source Elixir project that you use to test that this plugin is working? I tried opening the Phoenix LiveView project in Nova, but after compiling and waiting until the ElixirLS messages stopped appearing in Nova's extension console, it wouldn’t list any modules.

Do you have any other troubleshooting/debugging hints? Thx!

eahanson avatar Nov 12 '22 00:11 eahanson

Hey @eahanson, thanks for taking the time to open an issue!

This sounds like an issue with how we are defining "symbols" in the the new TreeSittter query language. The queries are here. The relevant Nova documentation is here

The fix may be as simple as renaming the variables/captures, the parts of the query that start with @, to match the ones defined in the linked Nova documentation. Might be a few weeks before I can dive into this so if you are interested in taking a crack at it please do!

I don't have an open source project I use to test, although it would be nice to have one. I'll take a look at the LiveView project and see why it isn't working.

raulchedrese avatar Nov 13 '22 21:11 raulchedrese

I have not had as much time as I had hoped to look into this. I still hope to find the time, though if anyone else is interested in having a look, don't let me stop you 🙂.

eahanson avatar Jan 13 '23 03:01 eahanson

I updated the plugin on my computer to the new version and deleted my .elixir_ls directory, and then "Open Quickly in Symbols" started working. 🎉

eahanson avatar Feb 06 '23 19:02 eahanson