feat: add send transfer validation

src/app/common/error-formatters.ts

@@ -1,7 +1,7 @@
 import { Money } from '@shared/models/money.model';
 import { isFunction } from '@shared/utils';
 
-import { FormErrorMessages } from '@app/common/error-messages';
+import { FormErrorMessages } from '@shared/error-messages';
 
 export function formatPrecisionError(num?: Money) {
   if (!num) return FormErrorMessages.CannotDeterminePrecision;

src/app/common/transactions/bitcoin/use-generate-bitcoin-tx.ts

@@ -156,14 +156,14 @@ export function useGenerateUnsignedNativeSegwitMultipleRecipientsTx() {
           });
         }
 
-        outputs.forEach((output, index) => {
+        outputs.forEach(output => {
           // When coin selection returns output with no address we assume it is
           // a change output
           if (!output.address) {
             tx.addOutputAddress(signer.address, BigInt(output.value), networkMode);
             return;
           }
-          tx.addOutputAddress(values.recipients[index].address, BigInt(output.value), networkMode);
+          tx.addOutputAddress(output.address, BigInt(output.value), networkMode);
         });
 
         return { hex: tx.hex, fee, psbt: tx.toPSBT(), inputs };

src/app/common/validation/forms/address-validators.ts

@@ -4,7 +4,7 @@ import * as yup from 'yup';
 import { BitcoinNetworkModes, NetworkConfiguration } from '@shared/constants';
 import { isString } from '@shared/utils';
 
-import { FormErrorMessages } from '@app/common/error-messages';
+import { FormErrorMessages } from '@shared/error-messages';
 import { validateAddressChain, validateStacksAddress } from '@app/common/stacks-utils';
 
 function notCurrentAddressValidatorFactory(currentAddress: string) {

src/app/common/validation/forms/amount-validators.ts

@@ -14,7 +14,7 @@ import {
 import { UtxoResponseItem } from '@app/query/bitcoin/bitcoin-client';
 
 import { formatInsufficientBalanceError, formatPrecisionError } from '../../error-formatters';
-import { FormErrorMessages } from '../../error-messages';
+import { FormErrorMessages } from '../../../../shared/error-messages';
 import { currencyAmountValidator, stxAmountPrecisionValidator } from './currency-validators';
 
 const minSpendAmountInSats = 6000;

src/app/common/validation/forms/currency-validators.ts

@@ -3,7 +3,7 @@ import * as yup from 'yup';
 import { BTC_DECIMALS, STX_DECIMALS } from '@shared/constants';
 import { isNumber } from '@shared/utils';
 
-import { FormErrorMessages } from '@app/common/error-messages';
+import { FormErrorMessages } from '@shared/error-messages';
 import { countDecimals } from '@app/common/math/helpers';
 
 export function currencyAmountValidator() {

src/app/common/validation/forms/recipient-validators.ts

@@ -1,7 +1,7 @@
 import { NetworkConfiguration } from '@shared/constants';
 import { stacksChainIdToCoreNetworkMode } from '@shared/crypto/stacks/stacks.utils';
 
-import { FormErrorMessages } from '@app/common/error-messages';
+import { FormErrorMessages } from '@shared/error-messages';
 
 import {
   notCurrentAddressValidator,

src/app/pages/rpc-send-transfer/rpc-send-transfer-confirmation.tsx

@@ -121,7 +121,7 @@ export function RpcSendTransferConfirmation() {
 
   return (
     <>
-      <Stack gap={4}>
+      <Stack gap={4} width="100%">
         {recipients.map((recipient, index) => (
           <SendTransferConfirmationDetails
             key={index}

src/app/pages/send/ordinal-inscription/hooks/use-send-inscription-form.tsx

@@ -9,7 +9,7 @@ import { OrdinalSendFormValues } from '@shared/models/form.model';
 import { RouteUrls } from '@shared/route-urls';
 import { isError } from '@shared/utils';
 
-import { FormErrorMessages } from '@app/common/error-messages';
+import { FormErrorMessages } from '@shared/error-messages';
 import { useAnalytics } from '@app/common/hooks/analytics/use-analytics';
 import { formFeeRowValue } from '@app/common/send/utils';
 import { InsufficientFundsError } from '@app/common/transactions/bitcoin/coinselect/local-coin-selection';

src/app/pages/send/send-crypto-asset-form/components/recipient-fields/hooks/use-recipient-bns-name.tsx

@@ -5,7 +5,7 @@ import { useFormikContext } from 'formik';
 import { logger } from '@shared/logger';
 import { BitcoinSendFormValues, StacksSendFormValues } from '@shared/models/form.model';
 
-import { FormErrorMessages } from '@app/common/error-messages';
+import { FormErrorMessages } from '@shared/error-messages';
 import { StacksClient } from '@app/query/stacks/stacks-client';
 import { useStacksClient } from '@app/store/common/api-clients.hooks';
 import { useCurrentNetworkState } from '@app/store/networks/networks.hooks';

src/app/pages/send/send-crypto-asset-form/form/stacks/use-stacks-common-send-form.tsx

@@ -7,7 +7,7 @@ import { StacksSendFormValues } from '@shared/models/form.model';
 import { Money } from '@shared/models/money.model';
 import { isEmpty } from '@shared/utils';
 
-import { FormErrorMessages } from '@app/common/error-messages';
+import { FormErrorMessages } from '@shared/error-messages';
 import { stxMemoValidator } from '@app/common/validation/forms/memo-validators';
 import { stxRecipientValidator } from '@app/common/validation/forms/recipient-validators';
 import { nonceValidator } from '@app/common/validation/nonce-validators';

src/app/pages/swap/hooks/use-swap-form.tsx

@@ -6,7 +6,7 @@ import { FeeTypes } from '@shared/models/fees/fees.model';
 import { StacksTransactionFormValues } from '@shared/models/form.model';
 import { Money, createMoney } from '@shared/models/money.model';
 
-import { FormErrorMessages } from '@app/common/error-messages';
+import { FormErrorMessages } from '@shared/error-messages';
 import { convertAmountToFractionalUnit } from '@app/common/money/calculate-money';
 import { useNextNonce } from '@app/query/stacks/nonce/account-nonces.hooks';
 

src/shared/forms/amount-validators.ts

@@ -0,0 +1,3 @@
+export function checkIfDigitsOnly(value: string) {
+  return /^\d+$/.test(value);
+}

src/shared/forms/bitcoin-address-validators.ts

@@ -0,0 +1,53 @@
+import { AddressType, Network, getAddressInfo, validate } from 'bitcoin-address-validation';
+import * as yup from 'yup';
+
+import { BitcoinNetworkModes } from '@shared/constants';
+import { FormErrorMessages } from '@shared/error-messages';
+import { isString } from '@shared/utils';
+
+export function btcAddressValidator() {
+  return yup
+    .string()
+    .defined(FormErrorMessages.AddressRequired)
+    .test((input, context) => {
+      if (!input) return false;
+      if (!validate(input))
+        return context.createError({
+          message: FormErrorMessages.InvalidAddress,
+        });
+      return true;
+    });
+}
+
+// ts-unused-exports:disable-next-line
+export function btcTaprootAddressValidator() {
+  return yup.string().test((input, context) => {
+    if (!input || !validate(input)) return false;
+    if (getAddressInfo(input).type !== AddressType.p2tr)
+      return context.createError({
+        message: 'Only taproot addresses are supported',
+      });
+    return true;
+  });
+}
+
+function btcAddressNetworkValidatorFactory(network: BitcoinNetworkModes) {
+  function getAddressNetworkType(network: BitcoinNetworkModes): Network {
+    // Signet uses testnet address format, this parsing is to please the
+    // validation library
+    if (network === 'signet') return Network.testnet;
+    return network as Network;
+  }
+
+  return (value?: string) => {
+    if (!isString(value)) return false;
+    return validate(value, getAddressNetworkType(network));
+  };
+}
+
+export function btcAddressNetworkValidator(network: BitcoinNetworkModes) {
+  return yup.string().test({
+    test: btcAddressNetworkValidatorFactory(network),
+    message: FormErrorMessages.IncorrectNetworkAddress,
+  });
+}

src/shared/rpc/methods/send-transfer.spec.ts

@@ -24,11 +24,11 @@ describe('`sendTransfer` method', () => {
         recipients: [
           {
             address: 'bc1qar0srrr7xfkvy5l643lydnw9re59gtzzwf5mdq',
-            amount: '0.0001',
+            amount: '10000',
           },
           {
             address: 'bc1qar0srrr7xfkvy5l643lydnw9re59gtzzwf5mdq',
-            amount: '0.0001',
+            amount: '800',
           },
         ],
       };
@@ -50,7 +50,7 @@ describe('`sendTransfer` method', () => {
         network: 'mainnet',
         account: 0,
         address: 'bc1qar0srrr7xfkvy5l643lydnw9re59gtzzwf5mdq',
-        amount: '0.0001',
+        amount: '10000',
       };
 
       const newParams = {
@@ -59,7 +59,7 @@ describe('`sendTransfer` method', () => {
         recipients: [
           {
             address: 'bc1qar0srrr7xfkvy5l643lydnw9re59gtzzwf5mdq',
-            amount: '0.0001',
+            amount: '10000',
           },
         ],
       };

src/shared/rpc/methods/send-transfer.ts

@@ -1,7 +1,13 @@
 import type { SendTransferRequestParams } from '@btckit/types';
 import * as yup from 'yup';
 
-import { WalletDefaultNetworkConfigurationIds } from '@shared/constants';
+import { type BitcoinNetworkModes, WalletDefaultNetworkConfigurationIds } from '@shared/constants';
+import { FormErrorMessages } from '@shared/error-messages';
+import { checkIfDigitsOnly } from '@shared/forms/amount-validators';
+import {
+  btcAddressNetworkValidator,
+  btcAddressValidator,
+} from '@shared/forms/bitcoin-address-validators';
 
 import {
   accountSchema,
@@ -19,11 +25,27 @@ export const rpcSendTransferParamsSchemaLegacy = yup.object().shape({
 
 export const rpcSendTransferParamsSchema = yup.object().shape({
   account: accountSchema,
-  recipients: yup
-    .array()
-    .of(yup.object().shape({ address: yup.string().required(), amount: yup.string().required() }))
-    .required(),
-  network: yup.string().oneOf(Object.values(WalletDefaultNetworkConfigurationIds)),
+  network: yup.string().required().oneOf(Object.values(WalletDefaultNetworkConfigurationIds)),
+  recipients: yup.array().of(
+    yup.object().shape({
+      // check network is valid for address
+      address: btcAddressValidator().test(
+        'address-network-validation',
+        FormErrorMessages.IncorrectNetworkAddress,
+        (value, context) => {
+          const contextOptions = context.options as any;
+          const network = contextOptions.from[1].value.network as BitcoinNetworkModes;
+          return btcAddressNetworkValidator(network).isValidSync(value);
+        }
+      ),
+      amount: yup
+        .string()
+        .required()
+        .test('amount-validation', 'Sat denominated amounts only', value => {
+          return checkIfDigitsOnly(value);
+        }),
+    })
+  ),
 });
 
 export interface RpcSendTransferParamsLegacy extends SendTransferRequestParams {

test-app/src/components/bitcoin.tsx

@@ -417,7 +417,7 @@ export const Bitcoin = () => {
               },
               {
                 address: TEST_TESTNET_ACCOUNT_2_BTC_ADDRESS,
-                amount: '10000',
+                amount: '0.010000',
               },
             ],
             network: 'testnet',
@@ -432,6 +432,32 @@ export const Bitcoin = () => {
       >
         Send transfer to multiple addresses
       </styled.button>
+      <styled.button
+        mt={3}
+        onClick={() => {
+          console.log('requesting');
+          (window as any).LeatherProvider?.request('sendTransfer', {
+            recipients: [
+              {
+                address: TEST_TESTNET_ACCOUNT_2_BTC_ADDRESS,
+                amount: '10000',
+              },
+              {
+                address: TEST_TESTNET_ACCOUNT_2_BTC_ADDRESS,
+                amount: '10000',
+              },
+            ],
+          })
+            .then((resp: any) => {
+              console.log({ sucesss: resp });
+            })
+            .catch((error: Error) => {
+              console.log({ error });
+            });
+        }}
+      >
+        Send transfer validate error
+      </styled.button>
     </Box>
   );
 };

tests/specs/send/send-stx.spec.ts

@@ -9,7 +9,7 @@ import { getDisplayerAddress } from '@tests/utils';
 
 import { STX_DECIMALS } from '@shared/constants';
 
-import { FormErrorMessages } from '@app/common/error-messages';
+import { FormErrorMessages } from '@shared/error-messages';
 
 import { test } from '../../fixtures/fixtures';