Exclude NodeJS.ReadableStream from compat overloads (#5743)

* Exclude NodeJS.ReadableStream from compat overloads

* format fix

* Add changeset

Author: hsubox76

.changeset/famous-houses-retire.md

@@ -0,0 +1,6 @@
+---
+'@firebase/storage': patch
+'@firebase/storage-compat': patch
+---
+
+Fix typings for storage and storage-compat.

packages/storage/src/reference.ts

@@ -170,8 +170,8 @@ export function getBytesInternal(
     .then(bytes =>
       maxDownloadSizeBytes !== undefined
         ? // GCS may not honor the Range header for small files
-          bytes.slice(0, maxDownloadSizeBytes)
-        : bytes
+          (bytes as ArrayBuffer).slice(0, maxDownloadSizeBytes)
+        : (bytes as ArrayBuffer)
     );
 }
 
@@ -194,8 +194,8 @@ export function getBlobInternal(
     .then(blob =>
       maxDownloadSizeBytes !== undefined
         ? // GCS may not honor the Range header for small files
-          blob.slice(0, maxDownloadSizeBytes)
-        : blob
+          (blob as Blob).slice(0, maxDownloadSizeBytes)
+        : (blob as Blob)
     );
 }
 
@@ -236,7 +236,7 @@ export function getStreamInternal(
 
   ref.storage
     .makeRequestWithTokens(requestInfo, newStreamConnection)
-    .then(stream => stream.pipe(result))
+    .then(stream => (stream as NodeJS.ReadableStream).pipe(result))
     .catch(e => result.destroy(e));
   return result;
 }

scripts/build/create-overloads.ts

@@ -278,7 +278,8 @@ const BUILTIN_TYPES = [
   'Blob',
   'ServiceWorkerRegistration',
   'Record',
-  'Error'
+  'Error',
+  'NodeJS.ReadableStream'
 ];
 
 // find all types (except for the built-ins and primitives) referenced in the function declaration