Prevent stack overflow with cyclic menu contributions

[dhuebner]

What it does

• Notifies about a menu item contribution where both a command and submenu is used. In such a case the command will be used and submenu entry ignored (vs-code behavior).
• Checks for menu recursion in MenuModelRegistry.linkSubmenu(). If detected a warning is logged and the contribution is ignored.

How to test

Follow the steps from #13036. After installing the extension several warnings should be logged:

2024-01-12T08:44:30.199Z root WARN Menu item robot.runSuite from plugin robocorp.robotframework-lsp contributed both submenu and command. Only command will be registered.
...
2024-01-12T08:44:30.199Z root WARN Recursive menu contribution detected: robotsubmenu is already in hierarchy of 1_run.

A "Maximum call stack size exceeded" should not be thrown and you should be able to see a new "Robot Framework" submenu in the explorer view.

Review checklist

• [x ] 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

[planger]

Thanks, code looks good to me, prevents call stack error and the context menu now opens with the correctly contributed items 👍

Tested with robocorp.robotframework-lsp-1.11.0.vsix in example browser app.

Thank you @dhuebner!

[JonasHelming]

@msujew Do you want to merge this?