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

Protocol support for "Just my code" #87

Closed
roblourens opened this issue Nov 9, 2016 · 3 comments
Closed

Protocol support for "Just my code" #87

roblourens opened this issue Nov 9, 2016 · 3 comments
Assignees
Milestone

Comments

@roblourens
Copy link
Member

roblourens commented Nov 9, 2016

We need a protocol method to mark/unmark files as library code, that will for now be triggered by the context menu item on the call stack. We can either use one method that sends the full set on each change, kind of like how setBreakpoints works, or we can have a pair of methods that mark/unmark one stack frame (or Source). If we work in stack frames, then it would be easier to do this for functions later.

Thoughts?

Like

setLibrarySources
  sources: Source[]

or

markAsLibraryCode
  frameId: number

unmarkAsLibraryCode
  frameId: number

Later if vscode itself had UI to show library code sources or patterns, then it would need to communicate that with Sources or strings, so that's worth keeping in mind too.

We also need a property on StackFrame returned from the debug adapter, isLibraryCode, needed so vscode knows which message to send.

And we need supportsJustMyCode to enable this context menu item.

@gregg-miskelly
Copy link
Member

gregg-miskelly commented Nov 9, 2016

It might make sense to call this something other than supportsJustMyCode since a debug adapter can support various versions of Just-My-Code functionality without supporting this very script-centric set of API requests, and this might be confusing. Maybe supportsJustMyCodeDocuments?

@weinand
Copy link
Contributor

weinand commented Nov 17, 2016

Using a pair of methods (mark/unmark) makes sense here because the corresponding actions are tied to stack frames and they only exist if the debug adapter is running. So there is no need to buffer some state while the debug adapter is not running.

I will not introduce this addition as official protocol immediately for the reasons stated here. So I've moved this feature request to the backlog for now.

@weinand
Copy link
Contributor

weinand commented Nov 6, 2020

Since "Just my code" functionality could implemented in JS debuggers without DAP support I'm closing this.

@weinand weinand closed this as completed Nov 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants