[lsp] make source action kinds more specific

[rchl]

LSP server advertises support for the following code action kinds:

codeActionKinds: [
    "quickfix",
    "source.organizeImports",
    "source.removeUnusedImports",
    "source.sortImports",
    "source.fixAll"
]

For example: https://github.com/microsoft/typescript-go/blob/092b34f534182baf2875887c20ffed2177f14d92/internal/lsp/lsproto/lsp.go#L313-L314

It should ideally advertise more specific kinds with a .ts suffix (for example). So for example source.removeUnusedImports.ts. That would allow the LSP client to target ts-server-specific code action from "code actions on save" functionality without triggering actions of other servers.

So for example:

    "editor.codeActionsOnSave": {
        "source.removeUnusedImports.ts": "explicit"  // Trigger for typescript server but not other servers that might be running on the file
    },

It will still be possible for the user to specify just source.removeUnusedImports so it should be a backward-compatible change.

[TorinAsakura]

Jake Bailey (@jakebailey) Ryan Cavanaugh (@RyanCavanaugh)

Guys, hey again. Let's get this task moving again.

Last time Jake and I got stuck on what the .ts handling should actually look like. Before I open a new PR under tsc/, can we just agree on that part?

Otherwise we're going to spend another round arguing about names and where a helper lives instead of finishing the thing. No disrespect, guys, I just want to get it done.

[rchl]

I've looked at your closed PR and I think the suffix should actually be tsgo instead of js or ts or js/ts since those code actions are bound to a server and not file type (as was also mentioned in that PR).

EDIT: Hmm, actually, tsgo might be kind of an artifact of the development approach. Maybe it's not really relevant anymore after merging the code to the main TypeScript repo. In that case ts might actually be the way to go (where ts doesn't represent the file extension but abbreviated server name). Could also be just a full typescript word.

Don't take my opinion as decision since I'm not part of this project though.