Track usage of internal snippet syntax

[jrieken]

With #3210 we are adding API to inserts snippets via completions. These snippets use the same syntax external, static snippets use which again is a subset of the TextMate snippet syntax. Eventually, we will not support the internal snippet syntax anymore.

This issue lists extensions that are known to be using the internal syntax and tracks their adoption of the new API.

• vscode.typescript - 0abf274
• lukehoban.Go -> @ramya-rao-a
• vscode.javascript
• rlovelett.vscode-swift-language -> Adopt SnippetString API RLovelett/vscode-swift#24
• vscode.css -> @aeschli
• HookyQR.JSDocTagComplete -> Adopt SnippetString API HookyQR/VSCodeJSDocTagComplete#2
• vscode.json -> @aeschli
• flowtype.flow-for-vscode -> Adopt SnippetString API flow/flow-for-vscode#51
• zeppelin.asp -> GONE, deleted?
• redhat.java -> Adopt SnippetString API redhat-developer/vscode-java#99
• DanTup.dart-code -> Adopt SnippetString API Dart-Code/Dart-Code#216
• vscode.html -> @aeschli
• twxs.cmake -> Adopt SnippetString API twxs/vs.language.cmake#21
• AureliaEffect.aurelia -> Adopt SnippetString API aurelia/vscode-extension#25
• msazurermtools.azurerm-vscode-tools -> not on GH
• mrmlnc.vscode-vash -> Adopt SnippetString API mrmlnc/vscode-vash#6

[jrieken]

Some extensions depend on the LSP to have support for this as well - #15577

[aeschli]

@jrieken Is there a spec for the new syntax?

[jrieken]

Yeah - this should be a more or less precise descriptions: https://github.com/Microsoft/vscode/blob/master/src/vs/editor/contrib/snippet/common/snippet.md

[aeschli]

@jrieken Can you spec the escaping rule in case a text contains $1 or ${... or the placeholder text contains a '}'

[aeschli]

I created #16056 for issues around escaping.

[DanTup]

FYI - Dart Code now uses the new API :-)

Our use of the old API seemed to break in 1.8 - underscores were being inserted during completion. Updating seemed like a faster option than waiting for a fix, but it does mean people upgrading to 1.8 need to then upgrade Dart Code (they can't do it before, since the new version had to have engine: 1.8) for things to work properly.

[ramya-rao-a]

This is done for the Go extension as well. Checking off the box for Go

microsoft/vscode-go#709

[jrieken]

Removal is happening in May