Vite: Rollup failed to resolve import "drizzle-orm"

[ansbar]

Hi, we still get the Rollup failed to resolve import "drizzle-orm" error when building our projects.

We use the fixed version 7.3.1. and aren't using RateLimiterDrizzle anywhere in our projects.

If I comment out the following in node_modules it works:

try { // Use dynamic import to prevent static analysis tools from detecting the import const drizzleOrm = await import('drizzle-orm'); const { and, or, gt, lt, eq, isNull, sql } = drizzleOrm.default || drizzleOrm; drizzleOperators = { and, or, gt, lt, eq, isNull, sql }; return drizzleOperators; } catch (error) { throw new RateLimiterDrizzleError( 'drizzle-orm is not installed. Please install drizzle-orm to use RateLimiterDrizzle.' ); }

We import like this:
import { RateLimiterMemory } from 'rate-limiter-flexible'

We actually havent got the problem in the application (a npm package) where we import it but we get the error in all applications that import that package..

[atetta]

@ansbar Hi, what is your building stack? Linters, webpack or any other bundler?
It looks like it analyzes dynamic import and throws error.

[ansbar]

Hi, we're using Vite with Eslint but it's in the build step its failing terminally.
(vue-tsc --noEmit && vite build)

error during build:
[vite]: Rollup failed to resolve import "drizzle-orm" from "C:/Repos/ids-web-ica/node_modules/rate-limiter-flexible/lib/RateLimiterDrizzle.js".
This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
build.rollupOptions.external
at viteLog (file:///C:/Repos/ids-web-ica/node_modules/vite/dist/node/chunks/dep-CMEinpL-.js:34362:57)
at onRollupLog (file:///C:/Repos/ids-web-ica/node_modules/vite/dist/node/chunks/dep-CMEinpL-.js:34396:9)
at onLog (file:///C:/Repos/ids-web-ica/node_modules/vite/dist/node/chunks/dep-CMEinpL-.js:34195:4)
at file:///C:/Repos/ids-web-ica/node_modules/rollup/dist/es/shared/node-entry.js:20866:32
at Object.logger [as onLog] (file:///C:/Repos/ids-web-ica/node_modules/rollup/dist/es/shared/node-entry.js:22748:9)
at ModuleLoader.handleInvalidResolvedId (file:///C:/Repos/ids-web-ica/node_modules/rollup/dist/es/shared/node-entry.js:21492:26)
at ModuleLoader.resolveDynamicImport (file:///C:/Repos/ids-web-ica/node_modules/rollup/dist/es/shared/node-entry.js:21550:58)
at async file:///C:/Repos/ids-web-ica/node_modules/rollup/dist/es/shared/node-entry.js:21438:32
at async Promise.all (index 4)

[atetta]

@ansbar As I understand, Vite tries to resolve lazy-loaded drizzle-orm during build step. Is that correct?
If so did you try to follow the suggested solution from the error message?

// vite.config.js
export default {
  build: {
    rollupOptions: {
      external: ['drizzle-orm']
    }
  }
}

That isn't ideal, but should work.
In the meantime, I'll think how to make import the right way not causing issues with different builders.

Alternatively, RateLimiterMemory can be imported directly:

import RateLimiterMemory from "rate-limiter-flexible/lib/RateLimiterMemory.js"

[ansbar]

Yes, you are correct and the external solution do work but we have to set it in 10 applications.
The direct import might be an excellent alternative, will try that, thanks.

[atetta]

Another alternative is to downgrade package version to 7.1.1.
There were changes in RateLimiterMongo and RateLimiterDrizzle from 7.1.1 to 7.3.1.
So you wouldn't miss anything.

[ansbar]

We actually run downgraded to 7.2.0 now

[animir]

@ansbar Ah, my bad. I didn't want to issue release notes for small fixes and edited the original. Sorry. I have just made release notes more clear in terms what and when was changed.

RateLimiterDrizzle was added in 7.2.0. So if you downgrade to 7.1.1 it wouldn't bother you.

[animir]

@ansbar Hey, could you help with testing an enhancement of lazy-loading?

Please, replace line 17

node-rate-limiter-flexible/lib/RateLimiterDrizzle.js

Line 17 in 11903a3

const drizzleOrm = await import('drizzle-orm');

in node_modules with

    function getPackageName() {
      return ['drizzle', 'orm'].join('-');
    }
    const drizzleOrm = await import(`${getPackageName()}`);

Thanks.

[ansbar]

@animir sure thing, tried now and it doesn't produce an error when building.

[animir]

@ansbar Cool, thanks. I hope nobody is going to implement AI-bundler to uncover that trick. :)

[ansbar]

A creative solution =)
Thanks for the quick fix