For DocumentSymbol, allow passing also Location optionally, not only Range

[michalbali256]

Currently, DocumentSymbol, used in the result of Document Symbol request only contains Range, so it is only able to reference the file for which the request was sent.
I propose, that we would be able to reference other files as well, so the definition of properties range and selectionRange could change from:

range: Range;
selectionRange: Range;

to the following:

range: Range | Location;
selectionRange: Range | Location;

We are developing a language server for High Level Assembler, where we would quite benefit from such feature. There is a construct, which makes symbols that may be defined in different files related (imagine something like partial class in C#). It would be nice if the user could see all the related symbols listed in one place (the outline in VS Code), even when they are defined in different files.

What are your thoughts on this? Is there a reason why such feature was not allowed until now? Would it be worthwhile to create a pull request implementing these changes?

[dibarbet]

+1 - we'll eventually be moving from SymbolInformation (which does have a Location) to DocumentSymbol for some C# scenarios. As mentioned in the original post, having the Location is important for partial types and methods which are defined across documents. In VS, textdocument/documentSymbol is used for some navbar scenarios where, for example, we would want to show all methods in a partial type from all documents where that type is defined.

[michprev]

+1