fix: throw error in getSmartAccountAddressFromInitialSigner if init…

…ial signer is undefined (#166)

* fix: throw error in `getSmartAccountAddressFromInitialSigner` if initial signer is undefined

* changeset

.changeset/nine-pears-run.md

@@ -0,0 +1,5 @@
+---
+'@abstract-foundation/agw-client': patch
+---
+
+Fix getSmartAccountAddressFromInitialSigner to throw if initial signer is undefined

packages/agw-client/src/utils.ts

@@ -49,6 +49,9 @@ export async function getSmartAccountAddressFromInitialSigner<
   initialSigner: Address,
   publicClient: PublicClient<Transport, chain>,
 ): Promise<Hex> {
+  if (initialSigner === undefined) {
+    throw new Error('Initial signer is required to get smart account address');
+  }
   // Generate salt based off address
   const addressBytes = toBytes(initialSigner);
   const salt = keccak256(addressBytes);

packages/agw-client/test/src/utils.test.ts

@@ -53,6 +53,17 @@ describe('convertBigIntToString', () => {
 });
 
 describe('getSmartAccountAddressFromInitialSigner', () => {
+  it('should throw an error if initial signer is undefined', async () => {
+    const publicClient = {
+      readContract: vi.fn().mockResolvedValue(address.smartAccountAddress),
+    };
+    expect(
+      getSmartAccountAddressFromInitialSigner(undefined!, publicClient as any),
+    ).rejects.toThrowError(
+      'Initial signer is required to get smart account address',
+    );
+  });
+
   it('should return the smart account address', () => {
     const initialSigner = address.signerAddress;
     const publicClient = {