Render progress for DAP progress events

[isidorn]

Related debug adapter protocol issue microsoft/debug-adapter-protocol#92

fyi @weinand

[weinand]

@isidorn I have added experimental DAP progress support to debugProtocol.d.ts.

I suggest that we start prototyping the debug progress UI for the use case "progress independent from a specific request". This is the important use case that @chuckries and @DanTup are looking for.

I will start this effort by adding this use case to Mock Debug.

[isidorn]

@weinand sounds good. Let me know when Mock Debug has adopted this.
Progress independent from a request: I suggest to render in as the spinning bar at the top of the Debug Viewlet.
Alternatives are:

• status bar entry (personally dislike this)
• progress badge in debug activity bar
• global workbench progress (too global imho)

[weinand]

@isidorn Yes, let's start by showing "unspecific progress" at the top of the Debug Viewlet.
If we will start showing the percentage we could show it in the status bar.

A shortcoming of the "top of the Debug Viewlet" location is that the user won't be able to cancel it, right?

[isidorn]

@weinand while we render progress we could also render a X or some other cnacel action in the debug viewlet title area. Pressing that would send a cancel. Check out search for long running search operations does somethign similar.

[weinand]

@isidorn I've added progress support to Mock Debug via this commit.
Running "progress" in the REPL starts a sequence of 100 progress events over a period of 50 seconds (but only if the client frontend sends a supportsProgressReporting capability).
The progress is cancellable.

[weinand]

@isidorn some random notes:

• yes, debug should have a similar cancelling UI as search (Mock Debug should have everything in place to support cancel)
• Mock debug provides the (mandatory) "title" attribute and the optional "message" attribute. We'll have to decide how they are displayed.
• overlapping progress: you can produce concurrent progress sequences with Mock debug by running multiple "progress" commands in the REPL within 50 seconds. I see two strategies for how to deal with concurrent progress sequences:
  • we could stack them and show only the top (newest) one (cleanest)
  • we could maintain just one progress and every new progress overwrites the current (kludge)
• next stage of the prototype: what to do with request-specific-progress. For now I suggest that we treat it identical as non-request-specific progress.

[isidorn]

@weinand thanks for adding the progess support for Mock debug. I plan to jump on this today.

1. Cool, will add a cancel title action
2. Maybe we show a hidden notification for every progress, and once the user clicks on the notification center he can se the title and the message
3. Idealy we would show progress as long as there is any progress event. Canceling cancels the last one
4. Yeah for now we will just render in the debug viewlet. Once there are specific use cases we can think more

[isidorn]

I have pushed the inital support for this.
When the progress is reported we show the progress bar in the debug viewlet and the progress can be canceled. Note the following:

• We use a "stop" icon which I think works for the ocassion, attached is an icon that search uses, but I do not think it would be necessery to "scope the icon to debug", since it is already in the viewlet fyi @misolori maybe instead of the X in the circle we should use the do not park sign that search uses
• When the user presses the Cancel we cancel the last progress that arrived
• We show progress as long as any progress is left
• If a session goes down we stop all the progress associated with that session
• Tooltip of the Cancel action shows the title of the progress

What is not covered by this work and we can tackle as seperate issues once clients adopt this:

• We ignore the ProgressUpdateEvent since VS Code progress UI does not support reporting progress status
• We do not show the message of the progress (could be done as a hidden progress notification)
• We only show progress in the debug viewlet, we do not show any scoped progress

Note that I had to do a slight change in mock debug such that it no longer waits after a progress has been canceled microsoft/vscode-mock-debug@29ec169
For reference here's the Cancel progress icon search is using

Try it out and let me know what you think.