Always use refreshAccessTokenIfNeeded to refresh access token

ref DEV-3054

Author: louischan-oursky

packages/authgear-core/src/client.ts

@@ -103,10 +103,7 @@ export class _BaseAPIClient {
       throw new AuthgearError("missing delegate in api client");
     }
 
-    const shouldRefresh = this._delegate.shouldRefreshAccessToken();
-    if (shouldRefresh) {
-      await this._delegate.refreshAccessToken();
-    }
+    await this._delegate.refreshAccessTokenIfNeeded();
 
     const request = new this._Request(input, init);
 

packages/authgear-core/src/types.ts

@@ -151,16 +151,10 @@ export interface _APIClientDelegate {
    */
   getAccessToken(): string | undefined;
 
-  /**
-   * Called by the API Client before sending HTTP request.
-   * If true is returned, refreshAccessToken() is then called.
-   */
-  shouldRefreshAccessToken(): boolean;
-
   /**
    * Called by the API client to refresh the access token.
    */
-  refreshAccessToken(): Promise<void>;
+  refreshAccessTokenIfNeeded(): Promise<void>;
 }
 
 /**