fix: refactor getClient method to return fetch client directly and update ApiOptions type

Author: PuruVJ

lib/index.js

@@ -142,10 +142,10 @@ class Api {
 
   /**
    * Get the HTTP client to use for requests
-   * @returns {Promise<FetchClient>} HTTP client
+   * @returns {FetchClient} HTTP client
    * @throws {Error} If no client is available
    */
-  async getClient () {
+  getClient () {
     if (this.options.mock) {
       console.warn('@beyonk/http: Using mocked http client')
       return this.options.mock
@@ -154,7 +154,7 @@ class Api {
     if (this.client) {
       return this.client
     } else if (config && config.fetch) {
-      return await config.fetch()
+      return config.fetch
     } else if (typeof window !== 'undefined') {
       return window.fetch.bind(window)
     }
@@ -210,7 +210,7 @@ class Api {
       this.config.overrides
     )
 
-    const client = await this.getClient()
+    const client = this.getClient()
     const ep = this.config.query ? `${endpoint}?${this.config.query}` : `${endpoint}`
 
     /** @type {QueryResult|undefined} */

lib/types.ts

@@ -8,8 +8,8 @@ export type ApiOptions = {
   baseUrl?: string;
   /** Mock client for testing */
   mock?: FetchClient;
-  /** Function that returns a fetch client */
-  fetch?: () => FetchClient | Promise<FetchClient>;
+  /** Fetch client for HTTP requests */
+  fetch?: FetchClient;
   /** Whether to retry failed requests */
   retry?: RetryOptions | false;
   /** Whether to parse error responses as JSON */