5.0.0-next.214+ regression: $$_import_defaults is not defined

[jamesst20]

Describe the bug

I only have one exemple for now and it's Svelecte but since 5.0.0-next.214, it is no longer possible to use the library, as soon as you import it, you get this error.

Reproduction

This REPL used to work in the past: REPL

Minimal REPL reproductible: Minimal REPL

Logs

Severity

blocking svelte upgrade

[paoloricciuti]

Here is a reproduction for the actual bug. It happens when reassigning an import in the module context because the reactive import is defined in the scope of the component function instead. Either we should avoid transforming in the scope of the module or hoist the declaration of the dynamic import but both seems to have some problem...i'll see if i can investigate this.

[paoloricciuti]

Oh actually it's just creating a source...we can probably hoist it...let me PR and see what maintainers think.

[jamesst20]

Oh actually it's just creating a source...we can probably hoist it...let me PR and see what maintainers think.

Thanks for the investigation! That means this line is probably causing the issue https://github.com/mskocik/svelecte/blob/master/src/lib/Svelecte.svelte#L9

Let me know if I should open an issue over there ! :)

[paoloricciuti]

Oh actually it's just creating a source...we can probably hoist it...let me PR and see what maintainers think.

Thanks for the investigation! That means this line is probably causing the issue https://github.com/mskocik/svelecte/blob/master/src/lib/Svelecte.svelte#L9

Let me know if I should open an issue over there ! :)

Yup that's the incriminated line...also i'm not really sure why they are assigning there instead of just setting the factory in the settings.js file. But i think we can fix it.