Add an option to transform specific libraries inside node_modules

[efoken]

Describe the feature

I am working on a library leveraging WyW, and also have a Design System monorepo set up which uses that library. While working with Design System libraries, it's often the case that you want to transform those, also when installed as dependencies.

Therefore it would be nice to have an options like transformLibraries: ['your-design-lib'] that can be passed to WyW plugins.

Possible implementations

wyw({
  transformLibraries: ['your-design-lib'],
})

Would transform the your-design-lib package inside node_modules.

I already have implemented this by copying the whole Vite plugin, so I can contribute if you like.

[layershifter]

@efoken updating include & exclude patterns to include /node_modules/ should do it, no?

[efoken]

No, because you are checking if the path contains node_modules, for example in Vite plugin: https://github.com/Anber/wyw-in-js/blob/main/packages/vite/src/index.ts#L111 and esbuild: https://github.com/Anber/wyw-in-js/blob/main/packages/esbuild/src/index.ts#L99

EDIT: I changed Vite plugin and changed that lines to:

if (
  (url.includes('node_modules') &&
    !transformLibraries.some((libName: string) => url.includes(libName))) ||
  !filter(id) ||
  id in cssLookup
) {
  return;
}

[layershifter]

@efoken good point, I see. We need to allow transforming of node_modules for sure (I have the same requirement for Fluent UI) 👍

So, I would start with transformLibraries: boolean and leverage existing filters, WDYT?

[efoken]

@layershifter of course, boolean would be an option, just to bypass the node_modules filter. But maybe array is also a good idea, because then you can only allow specific modules to be transformed – I mean performance-wise it's better, no?

[layershifter]

But maybe array is also a good idea, because then you can only allow specific modules to be transformed – I mean performance-wise it's better, no?

@efoken my idea is that once we exclude a condition for node_modules (via a new option transformLibraries), we allow to use exclude & include filters and the flexibility will be already in place.

[efoken]

@layershifter you're right, good idea 👍