My library uses dynamic imports by design

[kravetsone]

Feature request

My library uses dynamic imports by design. It autoload routes from folder and import at runtime. Imports can't be staticly analyzable and its fine for us.

How we can ignore @rollup/plugin-dynamic-import-vars usage? Maybe you can introduce a flag to disable this plugin?

As a workaraound we pin 2.5.1 version

Related kravetsone/elysia-autoload#38

Motivations

My library uses dynamic imports by design

Alternatives

pin 2.5.1 version

Additional context

No response

Bugs are expected to be fixed by those affected by it

• I'm interested in working on this issue

Compensating engineering work financially will speed up resolution

• I'm willing to offer financial support

[privatenumber]

Is the issue that it bundles dynamically loaded modules when you don't want it to? If so, we could add an option to explicitly externalize them.

Or is the issue that it throws an error because it can't resolve your dynamic imports? If that's the case, we could suppress the errors and turn them into warnings instead.

[kravetsone]

Is the issue that it bundles dynamically loaded modules when you don't want it to? If so, we could add an option to explicitly externalize them.

Or is the issue that it throws an error because it can't resolve your dynamic imports? If that's the case, we could suppress the errors and turn them into warnings instead.

It's throw an error because it can't resolve dynamic imports, but i use dynamic imports in my library by design (autoload - imports things at runtime by glob pattern and uses it)

[privatenumber]

Gotcha, yeah we should not be throwing errors.

I might be able to find time for this in the next week or two, but I'm happy to accept a PR to address this if you'd like to work on it.

[kravetsone]

Gotcha, yeah we should not be throwing errors.

I might be able to find time for this in the next week or two, but I'm happy to accept a PR to address this if you'd like to work on it.

We should just enable it by default? Or introduce a new flag for control it?

[privatenumber]

Let's enable it by default and we can release it as a fix since this is an unexpected breakage