feat(cloudflare-r2-binding): allow specify raw type

[pi0]

(context: #517 by @MickL)

This PR allows getItemRaw(key, { type: "..." }) with specified return type based on user needs: object/stream/blob/bytes/arrayBuffer are possible (check docs)

[cloudflare-workers-and-pages]

Deploying unstorage with    Cloudflare Pages

Latest commit:	be4f18b
Status:	✅  Deploy successful!
Preview URL:	https://8b4dae62.unstorage.pages.dev
Branch Preview URL:	https://feat-cf-r2-type.unstorage.pages.dev

View logs

[MickL]

Maybe

• getItemRaw() should return .body
• getItem() should have the option to set return type

?

What I just saw from the docs is that they return .body and I guess this is exactly what is needed to return the correct object? But I am not very familiar with the differences of ArrayBuffer, Blob and ReadableStream.

Example from Cloudflare where tey return .body: https://developers.cloudflare.com/r2/examples/demo-worker/

[pi0]

Unstorage raw* methods return a resolved value and .body is a stream which is not resolved + getItem is text based. All usages are valid for CF R2 bindings and it depends on how we want it to be consumed. for unstorage resolved makes sense (but this PR allows you to use other possible ways).

[MickL]

I saw you marked type bytes to default for v2. Would then the following code work with H3? If yes, I am okay with the new implementation:

export default defineEventHandler(async (event) => {
  return useStorage('files').getItemRaw('some-key');
});

[pi0]

Yes it should. If you had time can you please make an issue + reproduction in h3 for supporting arrayBuffer too? (it looks an issue there)