fix: make type really optional, add defaults for return type (#1338)

alchemyplatform · Feb 10, 2025 · 5afc25f · 5afc25f
1 parent aad98b8
commit 5afc25f
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/account-kit/react/src/hooks/useSmartAccountClient.ts b/account-kit/react/src/hooks/useSmartAccountClient.ts
@@ -1,5 +1,5 @@
 "use client";
-
+import { type OptionalFields } from "@aa-sdk/core";
 import type {
   GetSmartAccountClientParams,
   GetSmartAccountClientResult,
@@ -21,9 +21,12 @@ export type UseSmartAccountClientProps<
   TAccount extends SupportedAccountTypes | undefined =
     | SupportedAccountTypes
     | undefined
-> = GetSmartAccountClientParams<
-  TChain,
-  TAccount extends undefined ? "ModularAccountV2" : TAccount
+> = OptionalFields<
+  GetSmartAccountClientParams<
+    TChain,
+    TAccount extends undefined ? "ModularAccountV2" : TAccount
+  >,
+  "type"
 >;
 
 export type UseSmartAccountClientResult<
@@ -33,9 +36,7 @@ export type UseSmartAccountClientResult<
 
 export function useSmartAccountClient<
   TChain extends Chain | undefined = Chain | undefined,
-  TAccount extends SupportedAccountTypes | undefined =
-    | SupportedAccountTypes
-    | undefined
+  TAccount extends SupportedAccountTypes | undefined = "ModularAccountV2"
 >(
   args: UseSmartAccountClientProps<TChain, TAccount>
 ): UseSmartAccountClientResult<
@@ -66,7 +67,7 @@ export function useSmartAccountClient<
  */
 export function useSmartAccountClient({
   accountParams,
-  type,
+  type = "ModularAccountV2",
   ...clientParams
 }: UseSmartAccountClientProps): UseSmartAccountClientResult {
   const {