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

Debugger Step Over into skipFiles #118282

Closed
re4388 opened this issue Mar 6, 2021 · 7 comments
Closed

Debugger Step Over into skipFiles #118282

re4388 opened this issue Mar 6, 2021 · 7 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues verified Verification succeeded
Milestone

Comments

@re4388
Copy link

re4388 commented Mar 6, 2021

VS Code Version:

Version: 1.54.1 (user setup)
Commit: f30a9b7
Date: 2021-03-04T22:38:31.419Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: 8.7.220.31-electron.0
OS: Windows_NT x64 10.0.18363

OS Version:

  • win10 1909

  • Does this issue occur when all extensions are disabled?: Yes

Steps to Reproduce:

  1. npm install -g @angular/cli
  2. ng new my-dream-app
  3. cd my-dream-app
  4. ng serve

launch.json setting (both 2 settings are not working)

{
  "version": "0.2.0",
  "configurations": [
    {
      "type": "pwa-chrome",
      "request": "launch",
      "name": "Launch Chrome",
      "url": "http://localhost:4200",
      "webRoot": "${workspaceFolder}",
      "skipFiles": [
        "${workspaceFolder}/node_modules/**/*.js",
        "${workspaceFolder}/lib/**/*.js",
        "<node_internals>/**/*.js"
      ],
    }
  ]
}
{
  "version": "0.2.0",
  "configurations": [
    {
      "type": "pwa-chrome",
      "request": "launch",
      "name": "Launch Chrome",
      "url": "http://localhost:4200",
      "webRoot": "${workspaceFolder}",
      "skipFiles": [
        "${workspaceFolder}/node_modules/**/*.js",
        "${workspaceFolder}/lib/**/*.js",
        "<node_internals>/**/*.js"
      ],
      "sourceMaps": true,
      "sourceMapPathOverrides": {
        "webpack:/*": "${webRoot}/*",
        "/./*": "${webRoot}/*",
        "/src/*": "${webRoot}/*",
        "/*": "*",
        "/./~/*": "${webRoot}/node_modules/*"
      }
    }
  ]
}
  1. set breakpoint
    image

  2. Start debuggining by click the below panel
    image

7.Stop at breakpoint
image

  1. After 3 Step Over, we end up in the below screenshot (sometimes, it seems to skip, but sometimes not)
    On the right, we see those files are indeed skipped in the call stack? But the Step Over still stop inside the file.
    image

--

I apologize for my mistake if this is more about Angular specific not Vscode, but I think maybe Vscode debugger shall work as well?

@Gorthog
Copy link

Gorthog commented Mar 7, 2021

This is happening to me as well with nodejs backend with typescript.

@Gorthog
Copy link

Gorthog commented Mar 7, 2021

I have a breakpoint that is hit on the call site of an async function, and even though I have all js files added as skipped in launch.json file, when I press F11, debugger goes into the js file - I expect it to jump to line 9 in ts file.
index ts-debugger-not-skipping-WSL-Ubuntu_-Visual-Studio-Code-2021-03-07-21-21-37

here is the repro https://github.com/sinapis/vscode-118282

@weinand weinand assigned connor4312 and unassigned weinand Mar 7, 2021
@weinand weinand added the debug Debug viewlet, configurations, breakpoints, adapter issues label Mar 7, 2021
@connor4312 connor4312 added the bug Issue identified by VS Code Team member as probable bug label Mar 8, 2021
@connor4312 connor4312 added this to the On Deck milestone Mar 8, 2021
@connor4312
Copy link
Member

connor4312 commented Mar 30, 2021

Thank you for the good issue and repro steps! Turn out this bit of functionality never worked as it was intended. This'll be fixed in the nightly build.

@connor4312 connor4312 modified the milestones: On Deck, April 2021 Mar 30, 2021
@roblourens roblourens added the verified Verification succeeded label Apr 29, 2021
@roblourens
Copy link
Member

I press F11 and go into the promise helper in index.js, seems like both smartStep and skipFiles should avoid that

@roblourens roblourens reopened this Apr 29, 2021
@roblourens roblourens added verification-found Issue verification failed and removed verified Verification succeeded labels Apr 29, 2021
@connor4312
Copy link
Member

@roblourens can you share steps? I don't repro this https://memes.peet.io/img/21-04-ed599b74-21dd-48ae-8928-dbb50e256b9d.mp4

@roblourens
Copy link
Member

roblourens commented Apr 30, 2021

I was using the https://github.com/sinapis/vscode-118282 test repo, not the OP

image

@connor4312 connor4312 modified the milestones: April 2021, May 2021 Apr 30, 2021
@connor4312
Copy link
Member

That was actually a separate issue that affected attach without an --inspect-brk. If we parsed all scripts before the query for built-in modules returned and never parsed any additional scripts afterwards, skipfiles would never be sent.

@meganrogge meganrogge removed the verification-found Issue verification failed label Jun 7, 2021
@rzhao271 rzhao271 added the verified Verification succeeded label Jun 9, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Jun 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

7 participants