unjs / unplugin

Unified plugin system for Vite, Rollup, Webpack, esbuild, Rolldown, and more
https://unplugin.unjs.io
MIT License
2.9k stars 104 forks source link

fix: adjust webpack rspack hooks sequence #326

Closed qiYuei closed 9 months ago

qiYuei commented 11 months ago

πŸ”— Linked issue

❓ Type of change

πŸ“š Description

webpack | Rspack, The load hook should be executed earlier than the transform hook

I'm not sure if I'm doing this as a Breaking change

image

minimal reproduction

πŸ“ Checklist

antfu commented 11 months ago

It seems to make sense. Possible to add some tests to it?

qiYuei commented 11 months ago

okay,I try to write it.But there seems to be something wrong with the rspack in my environmentπŸ˜….I'll fill in the Rspack test cases later

qiYuei commented 11 months ago

@antfu hello,I have submitted the test files, please check it!😁

qiYuei commented 10 months ago

@antfu Can this be merged? Or what else do I need to change 🧐

easonsolov commented 9 months ago

hi,I have been using unplugin recently and have encountered the same issue with a plugin. Is there any progress with this PR? Or is this a bug that belongs to unplugin? thanks @antfu @sxzz