fix: cancel signer and 1 wei precision for non accuring collaterals (#2796)

Author: mgrabina

src/components/transactions/CancelCowOrder/CancelCowOrderActions.tsx

@@ -6,9 +6,7 @@ import { useModalContext } from 'src/hooks/useModal';
 import { ActionName, SwapActionFields, TransactionHistoryItem } from 'src/modules/history/types';
 import { useRootStore } from 'src/store/root';
 import { getErrorTextFromError, TxAction } from 'src/ui-config/errorMapping';
-import { wagmiConfig } from 'src/ui-config/wagmiConfig';
 import { updateCowOrderStatus } from 'src/utils/swapAdapterHistory';
-import { getWalletClient } from 'wagmi/actions';
 
 import { COW_ENV, getCowAdapter } from '../Swap/helpers/cow';
 import { TxActionsWrapper } from '../TxActionsWrapper';
@@ -32,15 +30,12 @@ export const CancelCowOrderActions = ({ cowOrder, blocked }: CancelCowOrderActio
       const adapter = await getCowAdapter(cowOrder.chainId);
       AdapterContext.getInstance().setAdapter(adapter);
       const orderBookApi = new OrderBookApi({ chainId: cowOrder.chainId, env: COW_ENV });
-      const walletClient = await getWalletClient(wagmiConfig, { chainId: cowOrder.chainId });
-
-      if (!walletClient || !walletClient.account) {
-        throw new Error('Wallet not connected for signing');
-      }
+      const signer = adapter.signer;
+      if (!signer) throw new Error('Wallet not connected for signing');
       const { signature, signingScheme } = await OrderSigningUtils.signOrderCancellation(
         cowOrder.id,
         cowOrder.chainId,
-        walletClient
+        signer
       );
       await orderBookApi.sendSignedOrderCancellations({
         orderUids: [cowOrder.id],

src/components/transactions/Swap/modals/request/CollateralSwapModalContent.tsx

@@ -1,3 +1,4 @@
+import { valueToBigNumber } from '@aave/math-utils';
 import { SupportedChainId, WRAPPED_NATIVE_CURRENCIES } from '@cowprotocol/cow-sdk';
 import { useQueryClient } from '@tanstack/react-query';
 import {
@@ -158,14 +159,25 @@ const getTokensFrom = (
             )
           : undefined;
 
+        let balance = position.underlyingBalance;
+        if (position.reserve.supplyAPY === '0') {
+          // remove one wei from balance because is rounded up
+          const balanceBN = valueToBigNumber(balance);
+          const decimals = balanceBN.decimalPlaces();
+          if (decimals && decimals > 15) {
+            const oneWei = valueToBigNumber(1).dividedBy(10 ** (decimals ?? 18));
+            balance = balanceBN.minus(oneWei).toString();
+          }
+        }
+
         return {
           addressToSwap: position.reserve.aTokenAddress,
           addressForUsdPrice: position.reserve.aTokenAddress,
           underlyingAddress: position.reserve.underlyingAsset,
           decimals: baseToken.decimals,
           symbol: nativeToken?.symbol ?? baseToken.symbol,
           name: nativeToken?.name ?? baseToken.name,
-          balance: position.underlyingBalance,
+          balance,
           chainId,
           usdPrice: position.reserve.priceInUSD,
           supplyAPY: position.reserve.supplyAPY,