-
Notifications
You must be signed in to change notification settings - Fork 196
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
fix: unable to hit most inflight breakpoints while debugging #6217
Conversation
Thanks for opening this pull request! 🎉
|
Console preview environment is available at https://wing-console-pr-6217.fly.dev 🚀 Last Updated (UTC) 2024-04-14 21:10 |
BenchmarksComparison to Baseline 🟥⬜🟥⬜⬜⬜⬜⬜⬜⬜⬜⬜⬜
⬜ Within 1.5 standard deviations Benchmarks may vary outside of normal expectations, especially when running in GitHub Actions CI. Results
Last Updated (UTC) 2024-04-14 21:16 |
Thanks for contributing, @MarkMcCulloh! This PR will now be added to the merge queue, or immediately merged if |
Congrats! 🚀 This was released in Wing 0.70.11. |
Missed this change in #6217 Now vscode will automatically use the builtin debugger when you have a wing file open instead of asking you to pick one *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*.
Fixes #5988
Fixes #2546
Fixes #6036
Fixes #5986
#5988 was a symptom of a deeper issue. In
wing it
or any other command, breakpoints would only work sometimes and depended on the order of inflights in a file and how many there were. The underlying issue is that the sourcemaps are too slow and may not finish loading in time to help the debugger attach breakpoints. At first I made a change in this PR to fix a problem with how esbuild bundles our sourcemaps. On it's own this change made many scenarios work that previously didn't.That wasn't enough though, so I went through many other options. As part of this, I actually added debugger settings to vscode.
In the end I needed to add a hack where we simply have a small sleep before running inflight code.
TODO:
By submitting this pull request, I confirm that my contribution is made under the terms of the Wing Cloud Contribution License.