Not playing well with rmagatti/goto-preview

[joehannes]

Describe the bug
I initially reported it with goto-preview, but rmagatti was thinking maybe its due to aerial ...

System information

• OS: [windows/linux/mac] mac
• Neovim version:0.7 nightly (v0.7.0-dev+2267-gc46f7caad)
• AerialInfo: [output of :AerialInfo]
  Aerial Info

---

Filetype: typescriptreact
Configured backends:
lsp (supported) (attached)
treesitter (not supported)
Show symbols: all symbols

To Reproduce
Steps to reproduce the behavior:

1. install goto-preview alongside aerial (or use my setup github.com/joehannes-os/neovim-lua-setup
2. use default goto-preview shortcut to preview definition of a thing in a floating window "gpd" in normal mode with cursor on the thing (eg.: a React Component var ... but not the definition, haha)
3. see it fail with an aerial error msg

If possible, provide a minimal file that will trigger the issue:

sorry ...

Screenshots

Additional context
Add any other context about the problem here.

[stevearc]

I couldn't get goto-preview working to repro, so I'm just taking a stab here. I pushed up a change that should hopefully fix the crash. From the trace, it seems like we're getting a LSP symbols callback on a buffer that's invalid (usually means it's been deleted). Not sure why that would happen, but we can guard against it pretty easily. Give the update a try and let me know if that helps

[joehannes]

yeah @stevearc :-) ... that actually solved the issue!!

hurray