Thanks! These are great suggestions.

We definitely have plans to expose browser native APIs via both getFile and addFile, so your suggestions will help define what that will look like :)

The one catch at the moment is, we need the IPFS-Lite interface to match that of js-ipld, or at least, those core methods for now. So that's why we have the getFile and addFile as separate APIs in there (they aren't part of the IPLD interface).

Having said that, as is, addFile does not depend on node read streams, that's just an example. At the moment we are depending on https://github.com/ipfs/js-ipfs-unixfs-importer{/exporter}, which does indeed use Buffers and other less-nice things. This was mostly to avoid having to re-write everything from scratch... but I'd like to re-implement some of that anyway, as it is actually pretty heavy handed for what we need here for the most part. Good call.

I really like the Response suggestion for getFile... let's do that. Though not sure sure what your head API would be used for?

Similarly, addFile most certainly should support FormData and the native Files/Blob API. I'll create separate issues for those two (and certainly, PRs are welcome :)).

Building on this, I think I have a nice idea to make the API easier to use... see https://github.com/textileio/js-ipfs-lite/tree/carson/restructure/src/files which is where we could put all the nicer browser native files stuff... an example of how we'd do this in a browser/nodejs compatible way is here: https://github.com/textileio/js-ipfs-lite/tree/carson/restructure/src/setup

I'm working on web server which is based on WebAPI Streams, and there is one issue with the disturbed stream. You can read more here: whatwg/streams#1025. In a few words, there is no way to specify wether the current stream will provide an initial chunk of data on read in a browser. But there is a hack to do so:

function isStreamDisturbed(stream: ReadableStream): Boolean {
  try {   
      new Response(stream)
    
      return false
    }
    catch {
      return true
    }
}

const stream = new ReadableStream()
stream.cancel()

isStreamDisturbed(stream) // -> true