-
-
Notifications
You must be signed in to change notification settings - Fork 496
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
[FEATURE] Debugger support scripts with inspection tools #4982
Comments
going to try oracle/graal#5844 (comment) to see if it works |
I was able to use the lsp option and got it running, here is a capability reply from the lsp server, but in the initialize request it was not using the right "workspaceFolders": [
{
"uri": "file:///mnt/old-linux/home/commandblock2/git_repo/MultiMC5/build/instances/LBNG-Production/minecraft/LiquidBounce/scripts",
"name": "scripts"
}
] response from the lsp of graalvm {
"result": {
"capabilities": {
"hoverProvider": true,
"workspaceSymbolProvider": false,
"completionProvider": {
"resolveProvider": false
},
"referencesProvider": false,
"definitionProvider": false,
"documentHighlightProvider": true,
"codeActionProvider": true,
"signatureHelpProvider": {},
"textDocumentSync": 2,
"documentSymbolProvider": false,
"codeLensProvider": {
"resolveProvider": false
},
"executeCommandProvider": {
"commands": [
"dry_run",
"get_coverage"
]
}
}
},
"id": 0,
"jsonrpc": "2.0"
} |
Also the chromium browser comes with mcef does not seems to bundle a devtools:// you may found in a regular chromium. So if we want in-client debugging it's very likely that we will have to serve it ourself. |
Implement a part of issue #4982 Added a client sub command to enable placing break points in scripts. The command has following parameters: `.client debug <name> [<protocol>] [<suspendOnStart>] [<inspectInternals>] [<port>]`. Protocols includes [DAP](https://www.graalvm.org/latest/tools/dap/) and inspect([Chrome DevTools Protocol](https://chromedevtools.github.io/devtools-protocol/)).
LiquidBounce Branch
Nextgen
Describe your feature request.
Since we are using graalvm, we might have a chance to use its debugger support for our scripts. Using this will require one more external dependency, chromeinspector. Generally I am looking for possibility for placing breakpoints and having REPL. You can assign this to me.
Additional context
No response
The text was updated successfully, but these errors were encountered: