Consolidate event flow in solution manager

[edriouk]

Type: Feature Request

Simplify and streamline event processing by providing central place to orchestrate solution loading and RPC communication.

• provide consistent view on data
• synchronize event processing
• improve performance

In particular, it will allow to fix the following problems:
Problems with Loading Solution (cbuild setup integration) #338
CMSIS-View: shows "Loading..." on minimal debugger setup #486

Suggested Change

Track solution changes and RPC actions in dedicated services.
Track events only in Solution Manager, process them if relevant and fire own events for other clients.
Use a special dedicated Event Hub class to maintain events: register and fire them. Should not process events itself.
Event handlers connected the Hub events. This has two major advantages:

• less dependencies: events are originating from the same source
• possibility to implement asynchronous events exchanges between two classes, in particular Solution Manager and Solution Converter

Other managers and views only track Solution Manager events and ask Solution Manager for services.
Editor views notify Solution Manager to perform solution conversion.

Simplified sequence diagram:

sequenceDiagram
    participant Manager
    participant Converter
    participant FileWatcher
    participant GUI

    FileWatcher -->> Manager: Solution Changed (async)
    Manager ->> Manager: Load Solution (sync)
    Manager -->> GUI: Refresh (async)
    Manager -->> Converter: Convert Solution { updateRTE? } (async)
    Converter ->> Converter: Run Convert (sync)
    Converter -->> Manager: Convert Completed (async)
    Manager ->> Manager: Load Converted Files (sync)
    Manager -->> GUI: Refresh (async)

[ReinhardKeil]

Yes, I believe this makes sense. Suggest to schedule this for v1.68

Can you also take a look to the flickering (see #244 (comment))

[edriouk]

Implemented in https://github.com/Arm-Debug/vscode-cmsis-csolution/pull/2772
@ReinhardKeil , you can review the changes by looking at behavior of the extension, e.g. how CMSIS outline view gets updated, .
It seems fixed "CMSIS View shows "Loading..." when Zephyr SDK is installed". #447
But most important, this allows now to address problems mentioned in this issue .