CompletionItem data property

[itsxallwater]

Hello!

In the lsp-sample repo there's some code in server.ts that builds the CompletionItem records for use with the extension. The sample includes a data property that doesn't seem to be defined in the CompletionItem docs and, in our localized testing, doesn't seem to make a difference when present or otherwise.

This is really just a question as to whether or not that property is necessary and if so, @brentlblair and I are looking for some clarity on what it impacts. He's finishing up a PR for a language server extension where we're doing some dynamic/custom IntelliSense construction.

Thanks :)

[jrieken]

The vscode-dts API keep instances around and therefore doesn't need a data-property