fix: sanitize import filenames in generated imports

[danielroe]

🔗 Linked issue

nuxt/nuxt#12909

❓ 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 should consider whether we want to support the use case in the linked issue, although the PR is safe to merge and likely a good idea.

Note: vite and rollup themselves are not yet compatible with resolved ids containing ' so even with this PR, the original issue is not solved.

• fix(utils): exclude invalid URL chars rollup/rollup#4262
• fix: escape import filenames vitejs/vite#5896

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.
• pending feat: string-based codegen utils unjs/magicast#2

[netlify]

✔️ Deploy Preview for nuxt3-docs canceled.

🔨 Explore the source changes: 5090279

🔍 Inspect the deploy log: https://app.netlify.com/sites/nuxt3-docs/deploys/62011f624917db00083ef739

[pi0]

This seems nice initiative and refactors anyway even until rollup is not supporting 👍 But I guess we should move this fix to mlly for generation utils and refactor once.

[pi0]

@danielroe Would you please help resolving merge conflicts?

[danielroe]

Current status is we are creating a new library in unjs namespace to host these utils.

I suggest I wait to resolve merge conflicts until we have that repo created or a PR open in it.

[danielroe]

@pi0 It would be good to get this merged in even if we still have upstream issues with vite to resolve. (Unless you think there's something else we should fix here?)