Ensure we apply feedback actions already on SetModel

[martin-fleck-at]

What it does

Ensure that feedback actions are already applied on set-model and not only on update-model.
Fixes eclipse-glsp/glsp#1239

How to test

In the workflow example, you can add a startup listener that adds feedback early on, e.g.,

import { IDiagramStartup, IFeedbackActionDispatcher, MaybePromise, SelectAction, TYPES } from '@eclipse-glsp/client';
import { inject, injectable } from 'inversify';

@injectable()
export class StartupFeedback implements IDiagramStartup {
    @inject(TYPES.IFeedbackActionDispatcher) protected feedbackDispatcher: IFeedbackActionDispatcher;

    preRequestModel(): MaybePromise<void> {
        this.feedbackDispatcher.registerFeedback(this, [SelectAction.setSelection(['task0'])]);
    }
}

With this (and the correct binding in the module) you should see that we have a select feedback registered and applied even if all you do is open the model.

Follow-ups

Changelog

• This PR should be mentioned in the changelog

• This PR introduces a breaking change (if yes, provide more details below for the changelog and the migration guide)

• Method FeedbackAwareUpdateModelCommand.getFeedbackCommands was move to IFeedbackEmitter for re-use, resulting in two new methods: getFeedbackCommands and applyFeedbackCommands.

• Method FeedbackAwareUpdateModelCommand.getPriority was replaced by a generic rank property and the Ranked namespace.

• The priority property (higher priority equals earlier execution) in FeedbackCommand was superseeded by a rank property (lower rank equals earlier execution).

[martin-fleck-at]

@tortmayr Locally I seem some exceptions I still need to have a look at. Will report back.

[martin-fleck-at]

@tortmayr I managed to resolve the dependency issue by using the contribution provider approach from Theia.

[tortmayr]

Thanks! Change seems to work great for the most part.
I only have two minor nitpicks, (the final ones I promise😉 )

[tortmayr]

LGTM! Thanks for your patience 😉
One minor suggestion, but nothing that should prevent merging this.