Adds more explicit type guard/null check to account for data sometimes coming back as "null"

Author: jonathanprozzi

bun.lock

@@ -19,8 +19,12 @@ export class TransferAction {
   constructor(private walletProvider: WalletProvider) {}
 
   async transfer(params: TransferParams): Promise<Transaction> {
-    if (!params.data) {
+    // Normalize empty or invalid data field to '0x'
+    if (!params.data || params.data === '0x') {
       params.data = '0x';
+    } else {
+      // Handle case where template returns 'null' string
+      params.data = (params.data as string) === 'null' ? '0x' : params.data;
     }
 
     const walletClient = this.walletProvider.getWalletClient(params.fromChain);
@@ -46,7 +50,8 @@ export class TransferAction {
         data: params.data as Hex,
       };
     } catch (error: unknown) {
-      const errorMessage = error instanceof Error ? error.message : String(error);
+      const errorMessage =
+        error instanceof Error ? error.message : String(error);
       throw new Error(`Transfer failed: ${errorMessage}`);
     }
   }
@@ -84,7 +89,9 @@ const buildTransferDetails = async (
   const parsedXml = parseKeyValueXml(xmlResponse);
 
   if (!parsedXml) {
-    throw new Error('Failed to parse XML response from LLM for transfer details.');
+    throw new Error(
+      'Failed to parse XML response from LLM for transfer details.'
+    );
   }
 
   const transferDetails = parsedXml as unknown as TransferParams;
@@ -128,7 +135,12 @@ export const transferAction: Action = {
     const action = new TransferAction(walletProvider);
 
     // Compose transfer context
-    const paramOptions = await buildTransferDetails(state, message, runtime, walletProvider);
+    const paramOptions = await buildTransferDetails(
+      state,
+      message,
+      runtime,
+      walletProvider
+    );
 
     try {
       const transferResp = await action.transfer(paramOptions);
@@ -146,7 +158,8 @@ export const transferAction: Action = {
       }
       return true;
     } catch (error: unknown) {
-      const errorMessage = error instanceof Error ? error.message : String(error);
+      const errorMessage =
+        error instanceof Error ? error.message : String(error);
       console.error('Error during token transfer:', errorMessage);
       if (callback) {
         callback({
@@ -179,5 +192,10 @@ export const transferAction: Action = {
       },
     ],
   ],
-  similes: ['EVM_TRANSFER', 'EVM_SEND_TOKENS', 'EVM_TOKEN_TRANSFER', 'EVM_MOVE_TOKENS'],
+  similes: [
+    'EVM_TRANSFER',
+    'EVM_SEND_TOKENS',
+    'EVM_TOKEN_TRANSFER',
+    'EVM_MOVE_TOKENS',
+  ],
 };