-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Build Error in SvelteKit with Cloudflare Pages Following Cloudflare SvelteKit Framework Guide #13976
Comments
Hi @onursoyer thanks for writing in! I can repro this as well with a freshly created SvelteKit app + Cloudflare. The root cause is us importing these stores in our |
Hey, so I did some digging and there seems to be a problem with our package's |
Any updates on this? |
Hey, no updates unfortunately. I'm still waiting on an update from the Svelte team in sveltejs/kit#12813 |
@Lms24 this error occurs even if we locally try to npm run build |
Yes this is to be expected if you use the cloudflare adapter. The build step within this adapter is causing the error. I'm currently reviewing #14672 which might fix this. |
Hi, the PR is on my list to review. I'll hopefully get to it soon. Sorry that this is taking so long but it's quite a complicated change |
Are there any workarounds for this? At the moment it looks like I have to go back to using Toucan-js instead of @sentry/cloudflare? Thanks. |
#14672 will get merged this week. We were just waiting on the v9 release and had to do a bit of bugfixing afterwards. Update: Release postponed to Monday due to an unexpected blocker in CI |
A PR closing this issue has just been released 🚀This issue was referenced by PR #14672, which was included in the 9.2.0 release. |
Is there an existing issue for this?
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/cloudflare
SDK Version
8.34.0
Framework Version
SvelteKit 2.7.0
Link to Sentry event
No response
Reproduction Example/SDK Setup
Steps to Reproduce
Expected Result
According to documentation I should be able to build the project without error.
Actual Result
You can mark the path "$app/stores" as external to exclude it from the bundle, which will remove this error and leave the unresolved path in the bundle.
error during build:
The text was updated successfully, but these errors were encountered: