Use toHex(0n) in translatePayment for better clarity

- Use bigint internally (0n) and convert to hex at JSON-RPC boundary
- Add comment explaining the conversion for JSON-RPC compatibility
- This follows modern best practices while maintaining compatibility

Author: spencerstock

packages/account-sdk/src/interface/payment/pay.test.ts

@@ -43,7 +43,7 @@ describe('pay', () => {
         {
           to: '0x833589fCD6eDb6E08f4c7C32D4f71b54bdA02913',
           data: '0xabcdef',
-          value: 0n,
+          value: '0x0',
         },
       ],
       capabilities: {},
@@ -103,7 +103,7 @@ describe('pay', () => {
         {
           to: '0x833589fCD6eDb6E08f4c7C32D4f71b54bdA02913',
           data: '0xabcdef',
-          value: 0n,
+          value: '0x0',
         },
       ],
       capabilities: {},
@@ -148,7 +148,7 @@ describe('pay', () => {
         {
           to: '0x833589fCD6eDb6E08f4c7C32D4f71b54bdA02913',
           data: '0xabcdef',
-          value: 0n,
+          value: '0x0',
         },
       ],
       capabilities: {},
@@ -234,7 +234,7 @@ describe('pay', () => {
         {
           to: '0x833589fCD6eDb6E08f4c7C32D4f71b54bdA02913',
           data: '0xabcdef',
-          value: 0n,
+          value: '0x0',
         },
       ],
       capabilities: {},
@@ -271,7 +271,7 @@ describe('pay', () => {
         {
           to: '0x833589fCD6eDb6E08f4c7C32D4f71b54bdA02913',
           data: '0xabcdef',
-          value: 0n,
+          value: '0x0',
         },
       ],
       capabilities: {},
@@ -352,7 +352,7 @@ describe('pay', () => {
         {
           to: '0x036CbD53842c5426634e7929541eC2318f3dCF7e',
           data: '0xabcdef',
-          value: 0n,
+          value: '0x0',
         },
       ],
       capabilities: {
@@ -417,7 +417,7 @@ describe('pay', () => {
         {
           to: '0x833589fCD6eDb6E08f4c7C32D4f71b54bdA02913',
           data: '0xabcdef',
-          value: 0n,
+          value: '0x0',
         },
       ],
       capabilities: {

packages/account-sdk/src/interface/payment/utils/translatePayment.test.ts

@@ -2,9 +2,9 @@ import { describe, expect, it } from 'vitest';
 import { CHAIN_IDS, TOKENS } from '../constants.js';
 import type { PayerInfo } from '../types.js';
 import {
-  buildSendCallsRequest,
-  encodeTransferCall,
-  translatePaymentToSendCalls,
+    buildSendCallsRequest,
+    encodeTransferCall,
+    translatePaymentToSendCalls,
 } from './translatePayment.js';
 
 describe('translatePayment', () => {

packages/account-sdk/src/interface/payment/utils/translatePayment.ts

@@ -1,4 +1,4 @@
-import { encodeFunctionData, parseUnits, type Address, type Hex } from 'viem';
+import { encodeFunctionData, parseUnits, toHex, type Address, type Hex } from 'viem';
 import { CHAIN_IDS, ERC20_TRANSFER_ABI, TOKENS } from '../constants.js';
 import type { PayerInfo } from '../types.js';
 
@@ -35,7 +35,7 @@ export function buildSendCallsRequest(transferData: Hex, testnet: boolean, payer
   const call = {
     to: usdcAddress as Address,
     data: transferData,
-    value: '0x0' as Hex, // No ETH value for ERC20 transfer
+    value: toHex(0n), // No ETH value for ERC20 transfer
   };
 
   // Build the capabilities object