Improve intellisense to handle structs and sigils

[jonatanklosko]

Improvements thanks to @josevalim's work on elixir-lang/elixir#11197.

completion_up.mp4

[josevalim]

Well done, This looks good to me!

there is just one thing to consider. In IEx we show private modules because it is likely you are using it to access private API in a lib. But this is very unlikely here. Should we simply hide modules with doc false? At least for structs but I would consider it in general.

[jonatanklosko]

@josevalim completing private modules may still be useful. For example, someone may use the Mix runtime instead of iex -S mix and then it's nice if we still complete their private modules. Perhaps we should change the text for entities with doc false to be something like "This is a private API"?

[josevalim]

I think that can be good for surround but for cursor context, I would rather optimize it for the majority of use cases. :) imagine using Ecto for the first time and getting all of the private modules listed :D

[jonatanklosko]

Right, makes sense! :D