`/// <reference lib="dom" />` in `api-call` package is globally overriding types about twurple HOT 3 CLOSED

While implementing this, I've found a few issues with passing fetch options, so I'm implementing a different solution now:

• The internal types don't depend on the dom lib, the @types/web package or anything else anymore. Instead, the smallest common denominator was extracted into and exported from the @d-fischer/cross-fetch package.
• If you need to pass fetch options that are specific to one platform, you can now augment the TwitchApiCallFetchOptions type (this is possible because it was changed to an interface)
• This is a breaking change for all current users of the fetchOptions config option (but I'm pretty sure your approach would be too in some obscure way), so it will only be implemented in version 7.0.

Stay tuned for a new prerelease version within the next week.

from twurple.

As the types used are inherently global, do you have a better solution for this?

from twurple.

@d-fischer Sure! One way we can accomplish this would be to use and import the @types/web package as a replacement for the triple slash lib directive to acquire the DOM types.

I've already tested this out with Twurple and it works perfectly! It no longer overrides the types globally. You can checkout how my reproduction is no longer showing any compile errors here. Just clone and switch branches with git checkout fix/using-@types/web.

To add the package, we just run the following:

cd packages/api-call

# it's fine to add this as a devDependency
yarn add @typescript/lib-dom@npm:@types/web --dev 

We then remove the directive /// <reference lib="dom" /> from both packages/api-call/src/apiCall.ts and packages/api-call/src/TwitchApiCallOptions.ts.

Finally, we just need to import the following at the top of the file (we only need to import it once):

// packages/api-call/src/TwitchApiCallOptions.ts

import '@typescript/lib-dom';

from twurple.