Describe the bug

On the typescript documentation page, it seems that the before* hooks take in Options as the first input.

But I'm pretty sure that's not the correct typing, because 1. the dnsCache is always either missing, or is instantiated (whether you passed {dnsCache: true} or {dnsCache: new CacheableLookup(...)} during initialization), and 2. the url is always an instance of the URL (you can see the code doing new URL(...) before passing it as the options.url.

Actual behavior

I would expect there to be a separate "hook options" type, where the dnsCache is undefined | CacheableLookup and url to be URL

Expected behavior

N/A

Code to reproduce

N/A

url always getting instantiated:

• got/source/core/index.ts

  Line 763 in b1d61c1

  const redirectUrl = new URL(redirectBuffer, url);

• got/source/core/index.ts

  Line 1121 in b1d61c1

  const url = options.url as URL;

dnsCache always getting instantiated:

And I have extensive testing over at got-ssrf, where I would've seen tests fail if options.url wasn't a URL for either of the before* hooks.

Checklist

• I have read the documentation.
• I have tried my code with the latest version of Node.js and Got.