🐛 Unused dependency when name matches node built-in
Prerequisites
- I'm using the latest version
- I've read the relevant documentation
- I've searched for existing issues
- I've checked the list of known issues
- I've read the issue reproduction guide
Reproduction url
https://codesandbox.io/p/devbox/2kk4rm?migrateFrom=fq39nt
Reproduction access
- I've made sure the reproduction is publicly accessible
Description of the issue
In packages that are meant to run in a non-node context (such as RN or the browser), it is common to depend on packages such as assert. The DependencyDuty (love the name btw) exits early when hitting a node built-in and doesn't consider it referenced. This results in false positives for unused dependencies being reported for packages that match node built-in module names.
knip/packages/knip/src/DependencyDeputy.ts
Line 193 in 4a3f84c
Our current workaround is to ignore those dependencies, but it might be more ideal to have a way to let knip know that node built-ins are not available.
How does funding with Polar work?
Pay now to fund the work behind this issue.
Get updates on progress being made.
Maintainer is rewarded once the issue is completed.
Backer
You're funding impactful open source efforts
Contributor
You want to contribute to this effort
Maintainer
You want to get funding like this too