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

Breakpoints for worker bind after the worker has already started #1719

Closed
rchiodo opened this issue Jun 8, 2023 · 2 comments · Fixed by #1723
Closed

Breakpoints for worker bind after the worker has already started #1719

rchiodo opened this issue Jun 8, 2023 · 2 comments · Fixed by #1723
Assignees
Labels
author-verification-requested Issues potentially verifiable by issue author bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@rchiodo
Copy link

rchiodo commented Jun 8, 2023

Describe the bug
If your worker map file is too large, breakpoints won't bind until after the worker is already running

To Reproduce
The only steps I have to reproduce is to use our real product. I'm running a worker thread that contains our entire language server and the breakpoints for the worker thread bind on a time delay after the worker has actually started.

Meaning I can't debug startup of the worker thread.

The breakpoints do bind later, and I can debug messages sent to the server (as I can stick a breakpoint in the client).

Log File

vscode-debugadapter-d1089d59.json.gz

VS Code Version: 1.79.0-insiders

Additional context
Add any other context about the problem here.

@rchiodo rchiodo added the bug Issue identified by VS Code Team member as probable bug label Jun 8, 2023
connor4312 added a commit that referenced this issue Jun 9, 2023
Some requests can make the client application run slow. If the user
tries to resume multiple times when paused and "stuck", it's very easy
to get into a bad state.

With this change, coalesce multiple requests of the same type together
so that clicking resume while stalled doesn't tear the state.

I couldn't really get a consistent test case for this since it's
timing based :/

Found while investigating #1719
@connor4312 connor4312 added this to the June 2023 milestone Jun 9, 2023
connor4312 added a commit that referenced this issue Jun 9, 2023
* fix: sourcemap predictor not filtering nested session on windows

Fixes #1719

* fix flakiness in pretty print bps test
connor4312 added a commit that referenced this issue Jun 9, 2023
…nt (#1721)

* fix: possible bad state when resuming multiple times with a slow client

Some requests can make the client application run slow. If the user
tries to resume multiple times when paused and "stuck", it's very easy
to get into a bad state.

With this change, coalesce multiple requests of the same type together
so that clicking resume while stalled doesn't tear the state.

I couldn't really get a consistent test case for this since it's
timing based :/

Found while investigating #1719

* fix tests
@roblourens roblourens added verification-steps-needed Steps to verify are needed for verification author-verification-requested Issues potentially verifiable by issue author labels Jun 28, 2023
@roblourens
Copy link
Member

Hey @rchiodo, is this working for you now?

@rchiodo
Copy link
Author

rchiodo commented Jun 28, 2023

Yep, working good. I use it everyday now.

@connor4312 connor4312 added verified Verification succeeded and removed verification-steps-needed Steps to verify are needed for verification labels Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author-verification-requested Issues potentially verifiable by issue author bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants