search

liuchengxu / vista.vim

:cactus: Viewer & Finder for LSP symbols and tags
https://liuchengxu.github.io/vista.vim
MIT License
1.88k stars 89 forks source link

LSPs which do not support textDocument/documentSymbol, constantly receive errors. #436

Closed beauwilliams closed 1 year ago

beauwilliams commented 1 year ago

I have the marksman LSP installed and with this plugin I get loads of errors because the LSP does not support textDocument/documentSymbol

Now I currently load Aerial with all my LSP using on_attach method require('aerial').on_attach(client)

I can write some error handling code in my config but I think this is best handled upstream. I can look into PR when I have time. Should be quick fix to check if its supported or not

https://github.com/liuchengxu/vista.vim/blob/f9c8c8ad4135346a945e1ebfb00b723976d92dfc/autoload/vista/executive/nvim_lsp.vim#L68

beauwilliams commented 1 year ago

Ignore. I seem to be getting this bug again without vista attatched. Must be something else

beauwilliams commented 1 year ago

The error appears to have been caused by the marksman LSP server crashing with an unhandled exception.. after the textDocument/documentSymbol automations are registered. This is more of an edge case if anything then.

beauwilliams commented 1 year ago

Appears to be fixed with updated binary