refactor: make isOAuthClientProvider check more explicit

Check typeof === 'function' on two required methods (tokens +
clientInformation) instead of bare 'in' operator. Slightly more robust
— verifies they're actually callable, not just properties with those
names. Same semantics, reads cleaner.

Author: felixweinberger

packages/client/src/client/auth.ts

@@ -84,9 +84,14 @@ export interface AuthProvider {
 /**
  * Type guard distinguishing `OAuthClientProvider` from a minimal `AuthProvider`.
  * Transports use this at construction time to classify the `authProvider` option.
+ *
+ * Checks for `tokens()` + `clientInformation()` — two required `OAuthClientProvider`
+ * methods that a minimal `AuthProvider` `{ token: ... }` would never have.
  */
 export function isOAuthClientProvider(provider: AuthProvider | OAuthClientProvider | undefined): provider is OAuthClientProvider {
-    return provider !== undefined && 'tokens' in provider && 'clientMetadata' in provider;
+    if (provider == null) return false;
+    const p = provider as OAuthClientProvider;
+    return typeof p.tokens === 'function' && typeof p.clientInformation === 'function';
 }
 
 /**