fix(nuxt): Only wrap .mjs entry files in rollup (#14060)

s1gr1d · web-flow · commit 8a68fa90f518 · 2024-10-23T14:31:18.000+02:00
fixes #14057 `@nuxt/content` adds entry files to rollup. This fix adds a check for `.mjs` to ignore other files in the rollup plugin.
diff --git a/packages/nuxt/src/vite/addServerConfig.ts b/packages/nuxt/src/vite/addServerConfig.ts
@@ -120,7 +120,7 @@ function wrapEntryWithDynamicImport(resolvedSentryConfigPath: string): InputPlug
         return { id: source, moduleSideEffects: true, external: true };
       }
 
-      if (options.isEntry && !source.includes(`.mjs${SENTRY_WRAPPED_ENTRY}`)) {
+      if (options.isEntry && source.includes('.mjs') && !source.includes(`.mjs${SENTRY_WRAPPED_ENTRY}`)) {
         const resolution = await this.resolve(source, importer, options);
 
         // If it cannot be resolved or is external, just return it so that Rollup can display an error

Original file line number	Diff line number	Diff line change
`@@ -120,7 +120,7 @@ function wrapEntryWithDynamicImport(resolvedSentryConfigPath: string): InputPlug`
`120`	`120`	`return { id: source, moduleSideEffects: true, external: true };`
`121`	`121`	`}`
`122`	`122`
`123`		- if (options.isEntry && !source.includes(`.mjs${SENTRY_WRAPPED_ENTRY}`)) {
	`123`	+ if (options.isEntry && source.includes('.mjs') && !source.includes(`.mjs${SENTRY_WRAPPED_ENTRY}`)) {
`124`	`124`	`const resolution = await this.resolve(source, importer, options);`
`125`	`125`
`126`	`126`	`// If it cannot be resolved or is external, just return it so that Rollup can display an error`