Add CodeActionProvider#resolveCodeAction to plugin API

[lucas-koehler]

What it does

Fixes #9992 by extending the plugin API and implementing the corresponding code action resolvement.

Plugin-API:

• Add resolveCodeAction method to CodeActionProvider in theia.d.ts

RPC-API:

• Add $resolveCodeAction and $releaseCodeActions methods to LanguagesExt
• Extend CodeAction DTO to contain its cache id

Ext:

• Implement methods in LanguageExtImpl delegating to CodeActionAdapter
• Extend CodeActionAdapter
  • Cache provided code actions with an id
  • Handle resolving of code actions given
  • Handle disposing obsolete code actions

Main:

• Extend LanguagesMainImpl
  • Hand resolveCodeAction callback into quick fix providers registered with monaco
  • Dispose cached code actions on Ext side when monaco no longer needs them

Contributed on behalf of STMicroelectronics
Signed-off-by: Lucas Koehler [email protected]

How to test

• I adapted the VSCode sample code action extension to use code action resolvement. It will also log when an action is resolved. If resolvement wasn't working, the code actions (replacing :) with emojis) would fail. One of the provided actions (Learn more...) is a command to verify that these do not cause errors. The code actions are provided in .md files.
• The extension can be downloaded here: code-actions-sample-0.0.2.vsix
• Extension source code: code-actions-sample
• Add the extension to your theia instance
• Create a file example.md and add :) to it. the sample code actions trigger if your curser is right before a :)
• Use one or more of the convert actions
• Use the learn more action

Review checklist

• As an author, I have thoroughly tested my changes and carefully followed the review guidelines

Reminder for reviewers

• As a reviewer, I agree to behave in accordance with the review guidelines

[msujew]

The changes are looking good for me! I have found a minor issue which could in theory result in unexpected behavior during execution, but I wasn't able to exploit it, so I'll approve this already. Please address the comment below and I'll merge this 👍

• The example extension can be executed correctly
• Normal code actions continue to work as expected

[lucas-koehler]

@msujew Thanks for the review and test. I fixed the if(cacheId) as it can indeed be 0. Good catch! :)