Breakpoint not catching when upgrading from Node 8 to Node 10.
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 30/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- javascript, node.js
- Domain
- devtools
Research direction
Reproduce the breakpoint behavior in a new Node.js project using Windows 10, Visual Studio 2017 15.9.16, and Node.js 10.16.3, following the listed upgrade steps. Compare breakpoints set before startup with the debugger statement and the Node 8.16.1 behavior; done means global-scope breakpoints are caught without removing and re-adding them.
Written by the indexing model from the issue text.
Description
Breakpoints are not catching right away after I upgraded from Node 8 to Node 10.
When Node 8 was installed I could set a breakpoint on a global statement and it would hit. Now, AFTER node runs first, I have to remove and re-add a breakpoint. Once doing so all other breakpoints on the opened code window start working also. It seems none of the breakpoints get registered properly BEFORE node runs the app.

Here you can see the debugger statement causes the debugger to hit there only, while all breakpoints do not work. Removing and adding any one breakpoint fixes all of them.
Rolling back to Node 8.16.1 fixes the issue and everything works as expected.
Expected Behavior
Global scope breakpoint hit.
Actual Behavior
Breakpoint is not hit.
- OS: Windows 10
- Visual Studio Version: VS 2017 15.9.16
- Node.js Version: 10.16.3
Steps to Reproduce
- Create new node project in Visual Studio.
- Download node installer to get the latest stable release of node, which is 10.16.3. Upgrade node 8 to node 10.
- Upgrade npm to the latest: npm install -g npm stable
- Upgrade node installed via NPM to match the downloaded version (if necessary): npm install -g node@10.16.3
- Create a module with a global statement and set a breakpoint on it.
- Breakpoint is never hit.
- Dominant language
- C#
- Stars
- 1.8k
- Forks
- 355
- PR merge metrics
- No merged PRs in 30d
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from microsoft/nodejstools
-
microsoft/nodejstools#2523 · 1 reaction · 1 assignee ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
microsoft/nodejstools#2519 · 1 comment ·
-
Difficulty 5/5 Over a week Newbie friendliness 15/100
microsoft/nodejstools#2517 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 25/100
microsoft/nodejstools#2514 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
microsoft/nodejstools#2513 · 5 comments ·
All issues in microsoft/nodejstools
Similar issues
-
type/automation type/tech-debt
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
t/bug
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
ci-failure-cause test-failure
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
area:auth FE mvp P3
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
klasolsson81/jobbliggaren#1788 ·