feat: support base64 strings in InputFile

[KnorpelSenf]

Title says it all. Should just be new InputFile({ base64 }).

[wojpawlik]

👎: tc39/proposal-arraybuffer-base64#51

Why doesn't fetchFile check response.ok?

Next time I touch InputFile, I'd love to drop

• Deno-only Deno.FsFile -- no advantage over new InputFile(fsFile.readable)
• URLLike -- has to parse and construct URL each time:

  grammY/src/core/input_file.ts

  Line 134 in ff4ebcb

  if ("url" in data) return await fetchFile(new URL(data.url));

[KnorpelSenf]

👎: tc39/proposal-arraybuffer-base64#51

Epic stuff, I expect it to land in Deno in a few weeks which is early enough for us to use it

Why doesn't fetchFile check response.ok?

It should from now on.

Next time I touch InputFile, I'd love to drop

• Deno-only Deno.FsFile -- no advantage over new InputFile(fsFile.readable)

The advantage is that users don't have to think about how to convert a Deno.FsFile to something grammY understands. We can do this work for them.

• URLLike -- has to parse and construct URL each time:

  grammY/src/core/input_file.ts

  Line 134 in ff4ebcb

  if ("url" in data) return await fetchFile(new URL(data.url));

It does not have to do that, you just chose to do it. We can still change it.

[wojpawlik]

{ base64 } (and URLLike) has to be parsed only once, in a conversion function called from the constructor.

Deno.FsFile stays for now. It might need to be replaced with { readable: AsyncIterable<Uint8Array> }. It could also be handled in the conversion function.

What are non-goals for InputFile? What should it not accept?