Pass unknown options to fetch

[kdelmonte]

This PR implements option number 2 of this comment by @sholladay.

Ky starts passing unknown options directly to fetch instead of relying on the Request class to do it for us. This is the right thing for Ky to do but might prove to be a little tricky. We would need to either pass all options to fetch, which could inadvertently override some of our options processing logic that's done with the Request class, or have some kind of filtering system to only pass unknown options to fetch, which would presumably involve some kind of hardcoded list of known options (which would have to include all options that the Request class understands) and that doesn't sound fun.

Fixes #531
Fixes #516

Also, tested this against Next.js v13.5.5 patched fetch and I can verify that it works.

[sholladay]

Great implementation. It's about as simple as you could make it, given the silly dance we have to do here.

I was originally thinking we would maintain a list of known request options to use for filtering, which I really did not want to do. But I see that the request class exposes most (all?) of its constructor options as instance properties and you are using that to avoid hardcoding the list. Nice work.

Technically, we could allow the Ky options to be passed on to fetch() and it likely wouldn't cause any trouble, at least in browsers. Although, there could be environments where fetch() throws for unexpected options. So doing it this way is certainly safer. 👍

[doroved]

Thanks, everything works, but now how to deal with TS? In bun fetch the type for proxy is there, how to use it in ky?

[doroved]

Type in bun

[kdelmonte]

This PR caused #539 which is being resolved with #540.

[doroved]

This PR caused #539 which is being resolved with #540.

So what to do with TS types? I provided information above

[kdelmonte]

@doroved , please create a separate issue and provide all the details of what you think needs to be done.

[loxK]

This PR caused #539 which is being resolved with #540.

A couple of hours of debugging an app to finally find out that regression....