suggestion: Move OAuth2ClientConfig interface into src/types.ts about deno-oauth2-client HOT 3 OPEN

They all add up, and it's not so much about the size but the complexity of dependencies. Deno provides a very simple way to keep a dependency tree as clean as possible, by importing only exactly what you absolutely require, we should embrace it, it's the main advantage over the npm dependency hell.

If want to write functions that reference the OAuth2ClientConfig type, then I want just that, and not the whole implementation, and there is no reason I shouldn't be able to do that if types are just exported separated, it's a pretty easy thing for a library to do, and doesn't break compatibility.

from deno-oauth2-client.

This is a server-side module. A server downloading the whole module of 33 KB is negligible. This would be a different story if this were a client-side module where dependency sizes have a tangible effect. While this would be nice to have, this is not something to be concerned about.

from deno-oauth2-client.

Sounds reasonable

I'll add this change to the PR with the OIDC implementation (#39), since it already includes a refactoring of the OAuth2Client class.

from deno-oauth2-client.