fix tx 0x00 (#233)

* fix(signTransaction): force undefined data to 0x so it isn't serialized to 0x00

* changeset

* update test

* update tests

.changeset/many-items-share.md

@@ -0,0 +1,5 @@
+---
+'@abstract-foundation/agw-client': patch
+---
+
+Fix undefined data in transaction so it doesn't serialize to 0x00

packages/agw-client/src/actions/signTransaction.ts

@@ -143,6 +143,12 @@ export async function signEip712TransactionInternal<
     customPaymasterHandler,
   );
 
+  if (transactionWithPaymaster.data === undefined) {
+    // serializer turns undefined into 0x00 which causes issues sending
+    // eth to contracts that don't have a fallback function
+    transactionWithPaymaster.data = '0x';
+  }
+
   const eip712Domain = chain?.custom.getEip712Domain({
     ...transactionWithPaymaster,
     type: 'eip712',

packages/agw-client/test/src/actions/signTransaction.test.ts

@@ -100,6 +100,7 @@ test('with useSignerAddress false', async () => {
       {
         chainId: anvilAbstractTestnet.chain.id,
         ...transaction,
+        data: '0x',
         from: address.smartAccountAddress,
         customSignature: signature,
         type: 'eip712' as any,
@@ -130,6 +131,7 @@ test('with useSignerAddress true', async () => {
       {
         chainId: anvilAbstractTestnet.chain.id,
         ...transaction,
+        data: '0x',
         from: address.signerAddress,
         customSignature: signature,
         type: 'eip712' as any,