fix(webpack): output .mjs to use crossorigin preloads

[danielroe]

🔗 Linked issue

❓ Type of change

• 📖 Documentation (updates to the documentation or readme)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality like performance)
• ✨ New feature (a non-breaking change that adds functionality)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

We are outputting .js files for webpack, which means we also don't add the crossorigin attribute when rendering. Because of a typo in our tests we weren't pick up on this.

Bug shows up with following warnings (+ double fetches of all webpack JS assets):

A preload for 'http://localhost:3000/_nuxt/59e3a34.js' is found, but is not used because the request credentials mode does not match. Consider taking a look at crossorigin attribute.
The resource http://localhost:3000/_nuxt/59e3a34.js was preloaded using link preload but not used within a few seconds from the window's load event. Please make sure it has an appropriate `as` value and it is preloaded intentionally.

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[netlify]

✅ Deploy Preview for nuxt3-docs canceled.

Name	Link
🔨 Latest commit	910ce3b
🔍 Latest deploy log	https://app.netlify.com/sites/nuxt3-docs/deploys/62e3d75f8b0ea9000964a68b

[pi0]

Would it be fixed with upcoming vue-bundle-renderer? At some point we have to switch back to .js for client module assets. Considering CDN mime support.

[danielroe]

No, the vue-bundle-renderer PR doesn't change this behaviour when rendering crossorigin attribute. (We can change it there to default to module - do you think that's a better option?)

[pi0]

This PR seems good to go until we do this but yes I think it is a viable option to support type:module instead of depending on extension for module detection if possible.

[danielroe]

As a top-level option to createRenderer (defaulting to true)?

[pi0]

As a top-level option to createRenderer (defaulting to true)?

I believe could be in manifest.

[danielroe]

~> nuxt-contrib/vue-bundle-renderer#32.