Removed old use-data lib

framework/commerce/cart/use-cart.tsx

@@ -6,7 +6,7 @@ import type {
   UseHookInput,
   UseHookResponse,
 } from '../utils/types'
-import useData from '../utils/use-data-2'
+import useData from '../utils/use-data'
 import { Provider, useCommerce } from '..'
 
 export type FetchCartInput = {

framework/commerce/customer/use-customer.tsx

@@ -6,7 +6,7 @@ import type {
   UseHookResponse,
 } from '../utils/types'
 import defaultFetcher from '../utils/default-fetcher'
-import useData from '../utils/use-data-2'
+import useData from '../utils/use-data'
 import { Provider, useCommerce } from '..'
 
 export type UseCustomerHandler<P extends Provider> = Prop<

framework/commerce/products/use-search.tsx

@@ -6,7 +6,7 @@ import type {
   UseHookResponse,
 } from '../utils/types'
 import defaultFetcher from '../utils/default-fetcher'
-import useData from '../utils/use-data-2'
+import useData from '../utils/use-data'
 import { Provider, useCommerce } from '..'
 import { BigcommerceProvider } from '@framework'
 

framework/commerce/utils/use-data-2.ts

@@ -1,84 +0,0 @@
-import useSWR, { responseInterface } from 'swr'
-import type {
-  HookHandler,
-  HookSwrInput,
-  HookFetchInput,
-  PickRequired,
-  Fetcher,
-  SwrOptions,
-} from './types'
-import defineProperty from './define-property'
-import { CommerceError } from './errors'
-
-export type ResponseState<Result> = responseInterface<Result, CommerceError> & {
-  isLoading: boolean
-}
-
-export type UseData = <
-  Data = any,
-  Input extends { [k: string]: unknown } = {},
-  FetchInput extends HookFetchInput = {},
-  Result = any,
-  Body = any
->(
-  options: PickRequired<
-    HookHandler<Data, Input, FetchInput, Result, Body>,
-    'fetcher'
-  >,
-  input: HookFetchInput | HookSwrInput,
-  fetcherFn: Fetcher,
-  swrOptions?: SwrOptions<Data, FetchInput, Result>
-) => ResponseState<Data>
-
-const useData: UseData = (options, input, fetcherFn, swrOptions) => {
-  const hookInput = Array.isArray(input) ? input : Object.entries(input)
-  const fetcher = async (
-    url: string,
-    query?: string,
-    method?: string,
-    ...args: any[]
-  ) => {
-    try {
-      return await options.fetcher({
-        options: { url, query, method },
-        // Transform the input array into an object
-        input: args.reduce((obj, val, i) => {
-          obj[hookInput[i][0]!] = val
-          return obj
-        }, {}),
-        fetch: fetcherFn,
-        normalize: options.normalizer,
-      })
-    } catch (error) {
-      // SWR will not log errors, but any error that's not an instance
-      // of CommerceError is not welcomed by this hook
-      if (!(error instanceof CommerceError)) {
-        console.error(error)
-      }
-      throw error
-    }
-  }
-  const response = useSWR(
-    () => {
-      const opts = options.fetchOptions
-      return opts
-        ? [opts.url, opts.query, opts.method, ...hookInput.map((e) => e[1])]
-        : null
-    },
-    fetcher,
-    swrOptions
-  )
-
-  if (!('isLoading' in response)) {
-    defineProperty(response, 'isLoading', {
-      get() {
-        return response.data === undefined
-      },
-      enumerable: true,
-    })
-  }
-
-  return response
-}
-
-export default useData

framework/commerce/utils/use-data.tsx

@@ -1,44 +1,54 @@
-import useSWR, { ConfigInterface, responseInterface } from 'swr'
-import type { HookSwrInput, HookFetcher, HookFetcherOptions } from './types'
+import useSWR, { responseInterface } from 'swr'
+import type {
+  HookHandler,
+  HookSwrInput,
+  HookFetchInput,
+  PickRequired,
+  Fetcher,
+  SwrOptions,
+} from './types'
 import defineProperty from './define-property'
 import { CommerceError } from './errors'
-import { useCommerce } from '..'
-
-export type SwrOptions<Data, Input = null, Result = any> = ConfigInterface<
-  Data,
-  CommerceError,
-  HookFetcher<Data, Input, Result>
->
 
 export type ResponseState<Result> = responseInterface<Result, CommerceError> & {
   isLoading: boolean
 }
 
-export type UseData = <Data = any, Input = null, Result = any>(
-  options: HookFetcherOptions | (() => HookFetcherOptions | null),
-  input: HookSwrInput,
-  fetcherFn: HookFetcher<Data, Input, Result>,
-  swrOptions?: SwrOptions<Data, Input, Result>
+export type UseData = <
+  Data = any,
+  Input extends { [k: string]: unknown } = {},
+  FetchInput extends HookFetchInput = {},
+  Result = any,
+  Body = any
+>(
+  options: PickRequired<
+    HookHandler<Data, Input, FetchInput, Result, Body>,
+    'fetcher'
+  >,
+  input: HookFetchInput | HookSwrInput,
+  fetcherFn: Fetcher,
+  swrOptions?: SwrOptions<Data, FetchInput, Result>
 ) => ResponseState<Data>
 
 const useData: UseData = (options, input, fetcherFn, swrOptions) => {
-  const { fetcherRef } = useCommerce()
+  const hookInput = Array.isArray(input) ? input : Object.entries(input)
   const fetcher = async (
-    url?: string,
+    url: string,
     query?: string,
     method?: string,
     ...args: any[]
   ) => {
     try {
-      return await fetcherFn(
-        { url, query, method },
+      return await options.fetcher({
+        options: { url, query, method },
         // Transform the input array into an object
-        args.reduce((obj, val, i) => {
-          obj[input[i][0]!] = val
+        input: args.reduce((obj, val, i) => {
+          obj[hookInput[i][0]!] = val
           return obj
         }, {}),
-        fetcherRef.current
-      )
+        fetch: fetcherFn,
+        normalize: options.normalizer,
+      })
     } catch (error) {
       // SWR will not log errors, but any error that's not an instance
       // of CommerceError is not welcomed by this hook
@@ -50,9 +60,9 @@ const useData: UseData = (options, input, fetcherFn, swrOptions) => {
   }
   const response = useSWR(
     () => {
-      const opts = typeof options === 'function' ? options() : options
+      const opts = options.fetchOptions
       return opts
-        ? [opts.url, opts.query, opts.method, ...input.map((e) => e[1])]
+        ? [opts.url, opts.query, opts.method, ...hookInput.map((e) => e[1])]
         : null
     },
     fetcher,

framework/commerce/wishlist/use-wishlist.tsx

@@ -6,7 +6,7 @@ import type {
   UseHookResponse,
 } from '../utils/types'
 import defaultFetcher from '../utils/default-fetcher'
-import useData from '../utils/use-data-2'
+import useData from '../utils/use-data'
 import { Provider, useCommerce } from '..'
 
 export type UseWishlistHandler<P extends Provider> = Prop<