fix: revert workaround for bug in nodejs 20.5 (#905)

This PR reverts the workaround for the
[bug](nodejs/node#48916) in nodejs 20.5 (and
below) which was [resolved](nodejs/node#48935)
in nodejs 20.6.

---------

Co-authored-by: Travis Vachon <[email protected]>

Author: Alan Shaw

packages/upload-client/src/car.js

@@ -42,16 +42,6 @@ export class BlockStream extends ReadableStream {
     let blocksPromise = null
     const getBlocksIterable = () => {
       if (blocksPromise) return blocksPromise
-      // FIXME: remove when resolved: https://github.com/nodejs/node/issues/48916
-      /* c8 ignore next 10 */
-      if (parseInt(globalThis.process?.versions?.node) > 18) {
-        blocksPromise = (async () => {
-          // @ts-expect-error
-          const bytes = await car.arrayBuffer()
-          return CarBlockIterator.fromBytes(new Uint8Array(bytes))
-        })()
-        return blocksPromise
-      }
       blocksPromise = CarBlockIterator.fromIterable(toIterable(car.stream()))
       return blocksPromise
     }

packages/upload-client/test/store.test.js

@@ -59,22 +59,22 @@ describe('Store.add', () => {
       channel: server,
     })
 
-    let progressStatusCalls = 0
+    let loaded = 0
     const carCID = await Store.add(
       { issuer: agent, with: space.did(), proofs, audience: serviceSigner },
       car,
       {
         connection,
         onUploadProgress: (status) => {
           assert(typeof status.loaded === 'number' && status.loaded > 0)
-          progressStatusCalls++
+          loaded = status.loaded
         },
       }
     )
 
     assert(service.store.add.called)
     assert.equal(service.store.add.callCount, 1)
-    assert.equal(progressStatusCalls, 1)
+    assert.equal(loaded, 225)
 
     assert(carCID)
     assert.equal(carCID.toString(), car.cid.toString())