🐛 Unused dependency when name matches node built-in

[sparten11740]

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

if (isBuiltin(packageName)) return true;

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.