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

Nuxt: Build Error with @nuxt/content #14057

Closed
s1gr1d opened this issue Oct 23, 2024 · 4 comments · Fixed by #14060
Closed

Nuxt: Build Error with @nuxt/content #14057

s1gr1d opened this issue Oct 23, 2024 · 4 comments · Fixed by #14060
Assignees
Labels
Package: nuxt Issues related to the Sentry Nuxt SDK

Comments

@s1gr1d
Copy link
Member

s1gr1d commented Oct 23, 2024

I'm creating this issue to better keep track of the problem commented here in the Nuxt tracking issue.

Which SDK are you using?

@sentry/nuxt

SDK Version

8.35.0

Reproduction Example/SDK Setup

Stackblitz

Steps to Reproduce

  • run npm run build

Expected Result

No error

Actual Result

Error during build:

[plugin inject] .nuxt/snapshot/cache/content/parsed/content/document.md: rollup-plugin-inject: failed to parse /home/projects/nuxt-starter-ye7qhs/.nuxt/snapshot/cache/content/parsed/content/document.md. Consider restricting the plugin to particular files via options.include
@github-actions github-actions bot added the Package: nuxt Issues related to the Sentry Nuxt SDK label Oct 23, 2024
@s1gr1d
Copy link
Member Author

s1gr1d commented Oct 23, 2024

@asokawotulo, I just tested your reproduction example in Stackblitz and locally as well and I could not reproduce the error you are getting. Can you confirm you are still getting this error?

Edit: I just saw your reproduction example includes dynamicImportForServerEntry: false. Without this, I can reproduce it 👍

@asokawotulo
Copy link

@s1gr1d, does this mean that the dynamicImportForServerEntry: false is no longer required as a workaround for @nuxt/content?

@s1gr1d
Copy link
Member Author

s1gr1d commented Oct 23, 2024

Exactly, this is no longer needed after version 8.36.0 (not yet released).

Copy link
Contributor

A PR closing this issue has just been released 🚀

This issue was referenced by PR #14060, which was included in the 8.36.0 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Package: nuxt Issues related to the Sentry Nuxt SDK
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants