expressjs
convert package to commonjs
The package is being switched back to CommonJS since the new API is currently in ESM. For more information, see https://github.com/expressjs/discussions/pull/323
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
| Diff | Package | Supply Chain Security |
Vulnerability | Quality | Maintenance | License |
|---|---|---|---|---|---|---|
 | append-field@2.0.0 ⏵ 1.0.0 |  +1 | |  -1 |  +1 | |
![socket-security[bot] avatar](https://avatars.githubusercontent.com/in/156372?v=4 "Published by a pub.dev verified publisher"){kind=small}
@bjohansebas the reason I switched to ESM initially was in order to consume the file-type package (via stream-file-type)
Here is some more thoughts on it: https://github.com/expressjs/multer/issues/1042#issuecomment-973902270
Sorry for having been so slow to respond to things! Unfortunately I have had a very hectic time lately, and I no longer use Multer professionally so can't spend company time on it 😞
If you need any updates to stream-file-type I'd be happy to assist (could also lift it into appropriate org), or anything else you need here! Thank you for spending time on this 🙏
Hey, thanks @LinusU for the historical reason, i hadn’t found it before.
Given that it’s really complicated to do, the options I see are:
- Stick with ESM, we’d need to request an exception for this ADR
- Move support to Node >=22.12, which would go against the LTS plan
I think the best option will be to stay with ESM.