File Storage API downloadRetrieve method returns ReadableStream

[pandektes-braedon]

Problem

The downloadRetrieve method is typed as returning a Readable node stream:

merge-node-client/src/api/resources/filestorage/resources/files/client/Client.ts

Line 330 in ffa041c

): Promise<stream.Readable> {

The api does not return a node stream, it returns a ReadableStream

Steps to Reproduce

    import { PassThrough, Readable } from 'node:stream'

    const readable = await merge.filestorage.files.downloadRetrieve(fileId)

    const isReadable = readable instanceof Readable // returns false
    const isReadableStream = readable instanceof ReadableStream // returns true

Would it be possible to update the API to return a node stream instead of the web stream?

[dsinghvi]

@pandektes-braedon we'll take a look here, did you notice this after upgrading the SDK to latest?

[pandektes-braedon]

@dsinghvi I'm currently running v1.0.11. I installed the SDK last week, however, it looks like the api is the same in latest.

Could you also confirm if the file is actually being streamed? Based on current current usage, it looks like your API is buffering the whole file in before returning it to me which defeats the purpose of the stream. I could be wrong but that's currently what it looks like while testing.

[dsinghvi]

It should be streamed -- i would use the .pipe method to test this