node-fetch/fetch-blob

Possibility to remove web-streams-polyfill dependency?

JeDaYoshi opened this issue · 1 comments

web-streams-polyfill is used as a dependency of this package, which is only used in streams.cjs as a polyfill fallback for users in <v16.5.0 for ReadableStreams.

However, I personally find it extremely overblown that, in this case, we're pulling a >7 MB dependency for something that might or might not be used, while we're also not using all the available po(l/n)yfills but ponyfill.es2018.js, which considerably (outside of depending of an external package) increases the size of the dependants, such as node-fetch.

Is it possible to include this file inside fetch-blob, or take any other possible solution (while also keeping compatibility with older Node.js versions)? I understand this is relatively low-priority, but I thought I'd at least raise it up for consideration.

Actually, I just noticed #116 is a thing. My bad!