Migrate from the Node to the Web ReadableStream

[dlarocque]

In the past, we used node-fetch, which used the NodeJS.ReadableStream interface for streaming requests. When we moved to undici (#7705), we began using the Web ReadableStream interface under the hood, but did not update our usage of the interface, so users began seeing errors when using getStream (#8303).

To fix this issue, we need to migrate from the Node to the Web ReadableStream interface.

This will help us in the future when we migrate to native Node fetch, since that also uses the Web ReadableStream interface.

Fixes #8303

[changeset-bot]

🦋 Changeset detected

Latest commit: 85a432a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages

Name	Type
firebase	Minor
@firebase/storage	Minor
@firebase/storage-compat	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

Changeset File Check ✅

• No modified packages are missing from the changeset file.
• No changeset formatting errors detected.

[google-oss-bot]

Size Report ¹

Affected Products

• @firebase/firestore

  Type	Base (1601572)	Merge (6e4609c)	Diff
  browser	379 kB	381 kB	+2.16 kB (+0.6%)
  esm5	364 kB	366 kB	+1.94 kB (+0.5%)
  main	584 kB	587 kB	+3.37 kB (+0.6%)
  module	379 kB	381 kB	+2.16 kB (+0.6%)
  react-native	379 kB	382 kB	+2.16 kB (+0.6%)

• @firebase/firestore-lite

  Type	Base (1601572)	Merge (6e4609c)	Diff
  browser	109 kB	111 kB	+1.70 kB (+1.6%)
  esm5	106 kB	108 kB	+1.56 kB (+1.5%)
  main	150 kB	153 kB	+2.55 kB (+1.7%)
  module	109 kB	111 kB	+1.70 kB (+1.6%)
  react-native	109 kB	111 kB	+1.70 kB (+1.6%)

• @firebase/storage

  Type	Base (1601572)	Merge (6e4609c)	Diff
  main	59.4 kB	59.4 kB	-7 B (-0.0%)

• bundle

  15 size changes

  Type	Base (1601572)	Merge (6e4609c)	Diff
  firestore (CSI Auto Indexing Disable and Delete)	269 kB	270 kB	+934 B (+0.3%)
  firestore (CSI Auto Indexing Enable)	269 kB	270 kB	+934 B (+0.3%)
  firestore (Persistence)	304 kB	305 kB	+934 B (+0.3%)
  firestore (Query Cursors)	241 kB	242 kB	+1.38 kB (+0.6%)
  firestore (Query)	238 kB	240 kB	+1.38 kB (+0.6%)
  firestore (Read data once)	226 kB	228 kB	+1.09 kB (+0.5%)
  firestore (Read Write w Persistence)	323 kB	325 kB	+1.74 kB (+0.5%)
  firestore (Realtime updates)	229 kB	230 kB	+1.09 kB (+0.5%)
  firestore (Transaction)	205 kB	207 kB	+1.38 kB (+0.7%)
  firestore (Write data)	205 kB	207 kB	+1.12 kB (+0.5%)
  firestore-lite (Query Cursors)	89.8 kB	91.2 kB	+1.38 kB (+1.5%)
  firestore-lite (Query)	85.9 kB	87.3 kB	+1.38 kB (+1.6%)
  firestore-lite (Read data once)	62.1 kB	62.8 kB	+702 B (+1.1%)
  firestore-lite (Transaction)	87.1 kB	88.1 kB	+995 B (+1.1%)
  firestore-lite (Write data)	71.7 kB	72.4 kB	+730 B (+1.0%)

• firebase

  Type	Base (1601572)	Merge (6e4609c)	Diff
  firebase-compat.js	786 kB	788 kB	+1.65 kB (+0.2%)
  firebase-firestore-compat.js	342 kB	344 kB	+1.65 kB (+0.5%)
  firebase-firestore-lite.js	117 kB	119 kB	+1.69 kB (+1.4%)
  firebase-firestore.js	438 kB	440 kB	+2.15 kB (+0.5%)

Test Logs

• Base (1601572): https://github.com/firebase/firebase-js-sdk/actions/runs/10309971864
• Merge (6e4609c): https://github.com/firebase/firebase-js-sdk/actions/runs/10376785756

1. https://storage.googleapis.com/firebase-sdk-metric-reports/LWw7vFeYmV.html

[google-oss-bot]

Size Analysis Report ¹

This report is too large (631,171 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.

Test Logs

• Base (1601572): https://github.com/firebase/firebase-js-sdk/actions/runs/10309971864
• Merge (6e4609c): https://github.com/firebase/firebase-js-sdk/actions/runs/10376785756

1. https://storage.googleapis.com/firebase-sdk-metric-reports/6JHIJFgmS1.html

[hsubox76]

This is looking good, just have some questions on the new TransformStream syntax maybe just because I am unfamiliar with it. Maybe we can walk through quickly offline.

[egilmorez]

LGTM, thanks!

[markarndt]

LGTM.