avoid circular dependency in bigcommerce

framework/bigcommerce/api/index.ts

@@ -4,8 +4,8 @@ import {
   CommerceAPIConfig,
   getCommerceApi as commerceApi,
 } from '@commerce/api'
-import fetchGraphqlApi from './utils/fetch-graphql-api'
-import fetchStoreApi from './utils/fetch-store-api'
+import createFetchGraphqlApi from './utils/fetch-graphql-api'
+import createFetchStoreApi from './utils/fetch-store-api'
 
 import type { CartAPI } from './endpoints/cart'
 import type { CustomerAPI } from './endpoints/customer'
@@ -68,14 +68,14 @@ const config: BigcommerceConfig = {
   customerCookie: 'SHOP_TOKEN',
   cartCookie: process.env.BIGCOMMERCE_CART_COOKIE ?? 'bc_cartId',
   cartCookieMaxAge: ONE_DAY * 30,
-  fetch: fetchGraphqlApi,
+  fetch: createFetchGraphqlApi(() => getCommerceApi().getConfig()),
   applyLocale: true,
   // REST API only
   storeApiUrl: STORE_API_URL,
   storeApiToken: STORE_API_TOKEN,
   storeApiClientId: STORE_API_CLIENT_ID,
   storeChannelId: STORE_CHANNEL_ID,
-  storeApiFetch: fetchStoreApi,
+  storeApiFetch: createFetchStoreApi(() => getCommerceApi().getConfig()),
 }
 
 const operations = {

framework/bigcommerce/api/utils/fetch-graphql-api.ts

@@ -1,38 +1,36 @@
 import { FetcherError } from '@commerce/utils/errors'
 import type { GraphQLFetcher } from '@commerce/api'
-import { provider } from '..'
+import type { BigcommerceConfig } from '../index'
 import fetch from './fetch'
 
-const fetchGraphqlApi: GraphQLFetcher = async (
-  query: string,
-  { variables, preview } = {},
-  fetchOptions
-) => {
-  // log.warn(query)
-  const { config } = provider
-  const res = await fetch(config.commerceUrl + (preview ? '/preview' : ''), {
-    ...fetchOptions,
-    method: 'POST',
-    headers: {
-      Authorization: `Bearer ${config.apiToken}`,
-      ...fetchOptions?.headers,
-      'Content-Type': 'application/json',
-    },
-    body: JSON.stringify({
-      query,
-      variables,
-    }),
-  })
-
-  const json = await res.json()
-  if (json.errors) {
-    throw new FetcherError({
-      errors: json.errors ?? [{ message: 'Failed to fetch Bigcommerce API' }],
-      status: res.status,
+const fetchGraphqlApi: (getConfig: () => BigcommerceConfig) => GraphQLFetcher =
+  (getConfig) =>
+  async (query: string, { variables, preview } = {}, fetchOptions) => {
+    // log.warn(query)
+    const config = getConfig()
+    const res = await fetch(config.commerceUrl + (preview ? '/preview' : ''), {
+      ...fetchOptions,
+      method: 'POST',
+      headers: {
+        Authorization: `Bearer ${config.apiToken}`,
+        ...fetchOptions?.headers,
+        'Content-Type': 'application/json',
+      },
+      body: JSON.stringify({
+        query,
+        variables,
+      }),
     })
+
+    const json = await res.json()
+    if (json.errors) {
+      throw new FetcherError({
+        errors: json.errors ?? [{ message: 'Failed to fetch Bigcommerce API' }],
+        status: res.status,
+      })
+    }
+
+    return { data: json.data, res }
   }
 
-  return { data: json.data, res }
-}
-
 export default fetchGraphqlApi

framework/bigcommerce/api/utils/fetch-store-api.ts

@@ -1,56 +1,56 @@
 import type { RequestInit, Response } from '@vercel/fetch'
-import { provider } from '..'
+import type { BigcommerceConfig } from '../index'
 import { BigcommerceApiError, BigcommerceNetworkError } from './errors'
 import fetch from './fetch'
 
-export default async function fetchStoreApi<T>(
-  endpoint: string,
-  options?: RequestInit
-): Promise<T> {
-  const { config } = provider
-  let res: Response
+const fetchStoreApi =
+  <T>(getConfig: () => BigcommerceConfig) =>
+  async (endpoint: string, options?: RequestInit): Promise<T> => {
+    const config = getConfig()
+    let res: Response
 
-  try {
-    res = await fetch(config.storeApiUrl + endpoint, {
-      ...options,
-      headers: {
-        ...options?.headers,
-        'Content-Type': 'application/json',
-        'X-Auth-Token': config.storeApiToken,
-        'X-Auth-Client': config.storeApiClientId,
-      },
-    })
-  } catch (error) {
-    throw new BigcommerceNetworkError(
-      `Fetch to Bigcommerce failed: ${error.message}`
-    )
+    try {
+      res = await fetch(config.storeApiUrl + endpoint, {
+        ...options,
+        headers: {
+          ...options?.headers,
+          'Content-Type': 'application/json',
+          'X-Auth-Token': config.storeApiToken,
+          'X-Auth-Client': config.storeApiClientId,
+        },
+      })
+    } catch (error) {
+      throw new BigcommerceNetworkError(
+        `Fetch to Bigcommerce failed: ${error.message}`
+      )
+    }
+
+    const contentType = res.headers.get('Content-Type')
+    const isJSON = contentType?.includes('application/json')
+
+    if (!res.ok) {
+      const data = isJSON ? await res.json() : await getTextOrNull(res)
+      const headers = getRawHeaders(res)
+      const msg = `Big Commerce API error (${
+        res.status
+      }) \nHeaders: ${JSON.stringify(headers, null, 2)}\n${
+        typeof data === 'string' ? data : JSON.stringify(data, null, 2)
+      }`
+
+      throw new BigcommerceApiError(msg, res, data)
+    }
+
+    if (res.status !== 204 && !isJSON) {
+      throw new BigcommerceApiError(
+        `Fetch to Bigcommerce API failed, expected JSON content but found: ${contentType}`,
+        res
+      )
+    }
+
+    // If something was removed, the response will be empty
+    return res.status === 204 ? null : await res.json()
   }
-
-  const contentType = res.headers.get('Content-Type')
-  const isJSON = contentType?.includes('application/json')
-
-  if (!res.ok) {
-    const data = isJSON ? await res.json() : await getTextOrNull(res)
-    const headers = getRawHeaders(res)
-    const msg = `Big Commerce API error (${
-      res.status
-    }) \nHeaders: ${JSON.stringify(headers, null, 2)}\n${
-      typeof data === 'string' ? data : JSON.stringify(data, null, 2)
-    }`
-
-    throw new BigcommerceApiError(msg, res, data)
-  }
-
-  if (res.status !== 204 && !isJSON) {
-    throw new BigcommerceApiError(
-      `Fetch to Bigcommerce API failed, expected JSON content but found: ${contentType}`,
-      res
-    )
-  }
-
-  // If something was removed, the response will be empty
-  return res.status === 204 ? null : await res.json()
-}
+export default fetchStoreApi
 
 function getRawHeaders(res: Response) {
   const headers: { [key: string]: string } = {}