search

isaacs / rimraf

A `rm -rf` util for nodejs
ISC License
5.62k stars 250 forks source link

Dependency bloat #274

Closed jirutka closed 1 year ago

jirutka commented 1 year ago

It adds 25 dependencies to a project using Babel, TypeScript and ESLint. This is completely insane for a simple utility doing just rm -rf with globbing.

https://bundlephobia.com/package/rimraf@5.0.1: image

https://npmgraph.js.org/?q=rimraf: image

isaacs commented 1 year ago

Thanks for sharing your feelings.

Why are you bundling rimraf for web browsers? 25kb is not even a rounding error server-side.

jirutka commented 1 year ago

Why are you bundling rimraf for web browsers?

I’m not, this is just for illustration.

25kb is not even a rounding error server-side.

Size is not what I see as the problem here, the number of dependencies is.

isaacs commented 1 year ago

Ok, so are you suggesting that this lib should not have a cli, should not support globs, or that all those things should be inline instead of shared deps in node_modules?

isaacs commented 1 year ago

Size is not what I see as the problem here, the number of dependencies is.

You're using babel, typescript, and eslint and you're worried about my 25 deps? Lol

Just use fs.rm if that bothers you.