(fix) remove preserveValueImports & isolatedModules from svelte.json

[dummdidumm]

It's deprecated, too
Leftover from #161

[mauvealerts]

Maybe remove isolatedModules too? I'd make a separate PR, but it'd probably conflict.

[dummdidumm]

You're right, isolatedModules is not deprecated but redundant.

[acidoxee]

My bad for leaving that deprecated preserveValueImports option in my PR 🤦‍♂️

Not sure for isolatedModules though @mauvealerts. I don't know anything about Svelte, does it force usage of tsc to build stuff? If not, possible usage of other compilers would require that setting to be present.

[dummdidumm]

According to the PR that introduced the verbatimModuleSyntax, isolatedModules is redundant and tsc will throw an error when it encounters it.

[acidoxee]

I was not aware of that, and it's not mentioned in the (upcoming) tsconfig docs in microsoft/TypeScript-Website#2759. I also don't remember seeing errors in my own codebase when using both those flags during the TS v5 beta and RC, is that new? 🤔

[dummdidumm]

This test indicated that it throws an error.

[orta]

👍🏻