-
Notifications
You must be signed in to change notification settings - Fork 268
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
Turbo sourcemaps not hitting TS breakpoints in imported modules #1554
Comments
I'm hitting something similar, in a debug configuration that debugs a VS Code extension built with webpack, and recursively attaches to a node process that the extension starts, and is built with tsup/esbuild, the breakpoints are no longer hit after the update, even with Sadly the code is proprietary but here is a debug log: vscode-debugadapter-e8aac7a4.json.gz, though I'm not sure it will help. The breakpoint in some TS file imported and bundled into |
MWE/reproduction: https://github.com/segevfiner/vscode-js-debug-bug Try breaking in app.ts (Code in the server) |
Here's another interesting datapoint: It seems like when there's a double So this doesn't work: "outFiles": [
"${workspaceFolder}/../**/dist/**/*.js",
"!**/node_modules/**"
], But this DOES work. "outFiles": [
"${workspaceFolder}/../lambda-api/dist/**/*.js",
"!**/node_modules/**"
], I wonder if |
After reading this (and after spending the afternoon trying to find the cause) I can now again debug into my Vue2 + Typescript application after rolling back VS Code to 1.74.0. |
@connor4312 Did you get a chance to try my reproduction? |
@segevfiner thanks for the repro. The issue you're hitting is specific to extension host debugging, and I'm not sure why it would have worked before. But that'll be fixed in the next nightly build. @MattReimer I'm not able to repro the issue with multiple glob stars in isolation. For others, please provide either similar repros, or log files using the instructions given in the "bug" issue template:
|
Argh. Built the nightly myself from commit 8d34776 and I was able to get it to debug the reproduction, but not the proprietery code I tried to debug, the "Debug Diagnostics" seems to say that app.js.map for app.js is not loaded for some reason, so any breakpoints in files bundled into it don't work, here is a log: vscode-debugadapter-d4acb6f6.json.gz vscode-debugadapter-13a567dc.json.gz where I can see the "sourceMapUrl" for app.js, but no "Creating sources from source map" for its source map. (The source map is there on disk right next to the file) |
Thanks for the log, that actually is an issue that regressed last month 😄 |
@connor4312 Tried with latest |
Sorry ... out of curiosity ... milestone 2 .... which release will that be? |
This was released in VS Code 1.76 |
... because I am on 1.76.1 and I still see the issue ... |
You probably have a different issue then. Please collect a trace log using the instructions I shared above and I can look into it. |
Sent an email ... |
I don't seem to have received anything, can you double check that it went through? |
Will send it again from my private mail account .... give me a minute .... |
Hope it arrived? |
I didn't get anything. You may also try my personal email [email protected] if the work email is not getting through for some reason... |
Oh boy, tried what you said. If it still doesn't go through, I will send you a OneDrive share link. |
Did it work? |
I believe this issue is in relation to the new debugger optimizations in patch 1.75 last week
When the patch hit last week we were unable to put breakpoints into '*.ts' files in imported modules and have them hit. We're running a lerna monorepo if that's useful.
/dist/
folders for both the original package and the imported ones did hit correctly."enableTurboSourcemaps": false
to ourlaunch.json
file fixes the issueI'm not sure what would be helpful to debug this but here is my profile from
launch.json
This worked flawlessly until VSCode updated last week. The note in the changelog said
enableTurboSourcemaps
will eventually be depprecated so obviously we'd like to see some kind of resolution to the issue before that happens.The text was updated successfully, but these errors were encountered: