saleor: update the provider structure

framework/saleor/cart/use-remove-item.tsx

@@ -1,25 +1,17 @@
 import { useCallback } from 'react'
-import type { MutationHookContext, HookFetcherContext } from '@commerce/utils/types'
-import { RemoveCartItemBody } from '@commerce/types'
-import { ValidationError } from '@commerce/utils/errors'
-import useRemoveItem, { RemoveItemInput as RemoveItemInputBase, UseRemoveItem } from '@commerce/cart/use-remove-item'
+import type { MutationHookContext, HookFetcherContext, MutationHook } from '@commerce/utils/types'
+import useRemoveItem, { UseRemoveItem } from '@commerce/cart/use-remove-item'
 import useCart from './use-cart'
 import * as mutation from '../utils/mutations'
 import { getCheckoutId, checkoutToCart } from '../utils'
-import { Cart, LineItem } from '../types'
 import { Mutation, MutationCheckoutLineDeleteArgs } from '../schema'
-
-export type RemoveItemFn<T = any> = T extends LineItem
-  ? (input?: RemoveItemInput<T>) => Promise<Cart | null>
-  : (input: RemoveItemInput<T>) => Promise<Cart | null>
-
-export type RemoveItemInput<T = any> = T extends LineItem ? Partial<RemoveItemInputBase> : RemoveItemInputBase
+import { LineItem, RemoveItemHook } from '../types/cart'
 
 export default useRemoveItem as UseRemoveItem<typeof handler>
 
 export const handler = {
   fetchOptions: { query: mutation.CheckoutLineDelete },
-  async fetcher({ input: { itemId }, options, fetch }: HookFetcherContext<RemoveCartItemBody>) {
+  async fetcher({ input: { itemId }, options, fetch }: HookFetcherContext<RemoveItemHook>) {
     const data = await fetch<Mutation, MutationCheckoutLineDeleteArgs>({
       ...options,
       variables: {
@@ -29,25 +21,19 @@ export const handler = {
     })
     return checkoutToCart(data.checkoutLineDelete)
   },
-  useHook:
-    ({ fetch }: MutationHookContext<Cart | null, RemoveCartItemBody>) =>
-    <T extends LineItem | undefined = undefined>(ctx: { item?: T } = {}) => {
-      const { item } = ctx
+  useHook: ({ fetch }: MutationHookContext<RemoveItemHook>) => <
+    T extends LineItem | undefined = undefined
+  > () => {
       const { mutate } = useCart()
-      const removeItem: RemoveItemFn<LineItem> = async (input) => {
-        const itemId = input?.id ?? item?.id
 
-        if (!itemId) {
-          throw new ValidationError({
-            message: 'Invalid input used for this operation',
-          })
-        }
+      return useCallback(
+        async function removeItem(input) {
+          const data = await fetch({ input: { itemId: input.id } })
+          await mutate(data, false)
 
-        const data = await fetch({ input: { itemId } })
-        await mutate(data, false)
-        return data
-      }
-
-      return useCallback(removeItem as RemoveItemFn<T>, [fetch, mutate])
+          return data
+        },
+        [fetch, mutate]
+      );
     },
 }