-
Notifications
You must be signed in to change notification settings - Fork 264
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
"npx shopify hydrogen debug cpu" is not working #2463
Comments
@DylanVann I am unable to reproduce this locally. Would you provide more information around node/npm version? |
Okay I can reproduce this. (The error doesn't show up when running cpu profiling in Hydrogen repo's skeleton template)
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What is the location of your example repository?
You can create a new Hydrogen project and it will have this issue
Which package or tool is having this issue?
Hydrogen
What version of that package or tool are you using?
2024.7.4
What version of Remix are you using?
2.10.1
Steps to Reproduce
npm create @shopify/hydrogen@latest
.npx shopify hydrogen debug cpu
Expected Behavior
CPU profiling to work.
Actual Behavior
I get an error that says
SourceMapConsumer is not a constructor
.I did some investigating and it looks like there's a couple issues, the first one seems to be related to CJS compat in the
cpu-profiler.ts
file where it importssource-map
. When the file is actually runSourceMapConsumer
is ondefault
from that module. After resolving that there seems to be an issue where Hydrogen is not finishing the build when profiling.The text was updated successfully, but these errors were encountered: