fix(create-app): svelte templates use type module (fixes #3834)

[dominikg]

Description

fix #3834 use type module so imports in svelte.config.js work as expected

Additional context

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

Before submitting the PR, please make sure you do the following

• Read the Contributing Guidelines.
• Read the Pull Request Guidelines and follow the Commit Convention.
• Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
• Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
• Ideally, include relevant tests that fail without this PR but pass with it.

nvm

[patak-dev]

@dominikg we are not using type: 'module' in any of the others templates, will this be the standard for svelte from now on? Is SvelteKit also using type: 'module'?

[dominikg]

@dominikg we are not using type: 'module' in any of the others templates, will this be the standard for svelte from now on? Is SvelteKit also using type: 'module'?

sveltekit uses it
https://github.com/sveltejs/kit/blob/master/packages/create-svelte/templates/default/package.template.json#L13

vite-plugin-svelte still offers a cjs fallback so you can use commonjs if you really have to but it is discouraged.

[bluwy]

@dominikg Could this be related to this line in vite-plugin-svelte?

Since we're not using the typescript compiler in the build process there, reverting this line to use normal await import() could fix this. (Haven't tested this yet)

EDIT: Doesn't work :( Only works in node v14

[dominikg]

any chance this can get merged and ceate-app released with it? still causing issues #3907

[patak-dev]

@dominikg @vitejs/[email protected] has been released