Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maximum call stack size exceeded after install robot-lsp extension #13036

Closed
wss29 opened this issue Oct 27, 2023 · 2 comments · Fixed by #13264
Closed

Maximum call stack size exceeded after install robot-lsp extension #13036

wss29 opened this issue Oct 27, 2023 · 2 comments · Fixed by #13264
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help

Comments

@wss29
Copy link

wss29 commented Oct 27, 2023

Bug Description:

Steps to Reproduce:

1.download theia blueprint from https://github.com/eclipse-theia/theia-blueprint
2.execute yarn and yarn build,
3.download robotframework lsp extension from https://open-vsx.org/extension/robocorp/robotframework-lsp
4.copy downloaded robocorp.robotframework-lsp-1.11.0.vsix to plugins folder
image
5.select Launch Electron Backend & Fronted debug mode
image
6.start debug and the debug console will output the error message
image
7.open file explorer and right click any file, can not open context menu
image

Additional Information

  • Operating System:windows10
  • Theia Version:1.41.1
@msujew msujew added bug bugs found in the application help wanted issues meant to be picked up, require help labels Dec 6, 2023
@dhuebner
Copy link
Member

It comes because robot extension contributes a cyclic submenus.

"robotsubmenu": [
   {
       "command": "robot.runSuite",
       "group": "1_run@1",
       "submenu": "robotsubmenu",
       "when": "resourceExtname == .robot || explorerResourceIsFolder"
   },
]  

First it is not allowed to use command and submenu in the same item. Second, the contribution adds robotsubmenu as a submenu item to the robotsubmenu which cause the stack overflow while cleaning up the menu items.

@dhuebner
Copy link
Member

@msujew
This can be closed as fixed.

@msujew msujew closed this as completed Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants