@exodus/solana-api 2.5.28 → 2.5.30-patch

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/solana-api",
-  "version": "2.5.28",
+  "version": "2.5.30-patch",
   "description": "Exodus internal Solana asset API wrapper",
   "main": "src/index.js",
   "files": [
@@ -34,5 +34,5 @@
   "devDependencies": {
     "@exodus/assets-testing": "file:../../../__testing__"
   },
-  "gitHead": "e46dbfdf19c725e5e7d4aa98a9e0eac913c44ed2"
+  "gitHead": "9ec7084bcdfe14f1c6f11df393351885773046a6"
 }

package/src/api.js

@@ -22,8 +22,9 @@ import { Connection } from './connection'
 
 // Doc: https://docs.solana.com/apps/jsonrpc-api
 
-const RPC_URL = 'https://solana.a.exodus.io' // https://vip-api.mainnet-beta.solana.com/, https://api.mainnet-beta.solana.com, https://solana-api.projectserum.com
-const WS_ENDPOINT = 'wss://solana.a.exodus.io/ws'
+const RPC_URL = 'https://solana.a.exodus.io' // https://vip-api.mainnet-beta.solana.com/, https://api.mainnet-beta.solana.com
+const WS_ENDPOINT = 'wss://solana.a.exodus.io/ws' // not standard across all node providers (we're compatible only with Quicknode)
+const FORCE_HTTP = true // use https over ws
 
 // Tokens + SOL api support
 export class Api {
@@ -63,6 +64,7 @@ export class Api {
     handleReconnect,
     reconnectDelay,
   }) {
+    if (FORCE_HTTP) return false
     const conn = new Connection({
       endpoint: this.wsUrl,
       address,
@@ -89,7 +91,7 @@ export class Api {
     if (handleTransfers) return handleTransfers(updates)
   }
 
-  async rpcCall(method, params = [], { address = '', forceHttp = false } = {}) {
+  async rpcCall(method, params = [], { address = '', forceHttp = FORCE_HTTP } = {}) {
     // ws request
     const connection = this.connections[address] || lodash.sample(Object.values(this.connections)) // pick random connection
     if (lodash.get(connection, 'isOpen') && !lodash.get(connection, 'shutdown') && !forceHttp) {
@@ -119,7 +121,7 @@ export class Api {
 
   async getRecentBlockHash(commitment?) {
     const result = await this.rpcCall(
-      'getRecentBlockhash',
+      'getLatestBlockhash',
       [{ commitment: commitment || 'finalized', encoding: 'jsonParsed' }],
       { forceHttp: true }
     )
@@ -802,7 +804,7 @@ export class Api {
    */
   broadcastTransaction = async (signedTx, options) => {
     console.log('Solana broadcasting TX:', signedTx) // base64
-    const defaultOptions = { encoding: 'base64', preflightCommitment: 'finalized' }
+    const defaultOptions = { encoding: 'base64', preflightCommitment: 'confirmed' }
 
     const params = [signedTx, { ...defaultOptions, ...options }]
     const errorMessagesToRetry = ['Blockhash not found']

package/src/get-balances.js

@@ -1,21 +1,30 @@
+import { TxSet } from '@exodus/models'
+
 // staking may be a feature that may not be available for a given wallet.
 // In this case, The wallet should exclude the staking balance from the general balance
 
-export const getBalancesFactory = ({ stakingFeatureAvailable }) => ({ asset, accountState }) => {
+export const getBalancesFactory = ({ stakingFeatureAvailable }) => ({
+  asset,
+  accountState,
+  txLog,
+}) => {
+  const zero = asset.currency.ZERO
+  const { balance, locked, withdrawable, pending } = fixBalances({
+    txLog,
+    balance: getBalanceFromAccountState({ asset, accountState }),
+    locked: accountState.mem?.locked || zero,
+    withdrawable: accountState.mem?.withdrawable || zero,
+    pending: accountState.mem?.pending || zero,
+    asset,
+  })
   if (asset.baseAsset.name !== asset.name) {
-    return { balance: accountState?.tokenBalances?.[asset.name], spendableBalance: null }
+    return { balance, spendableBalance: balance }
   }
 
-  const zero = asset.currency.ZERO
-
-  const balance = accountState.balance || zero
-
-  const { locked, withdrawable, pending } = accountState.mem || Object.create(null)
-
   const balanceWithoutStaking = balance
-    .sub(locked || zero)
-    .sub(withdrawable || zero)
-    .sub(pending || zero)
+    .sub(locked)
+    .sub(withdrawable)
+    .sub(pending)
     .clampLowerZero()
 
   return {
@@ -23,3 +32,46 @@ export const getBalancesFactory = ({ stakingFeatureAvailable }) => ({ asset, acc
     spendableBalance: balanceWithoutStaking.sub(asset.accountReserve || zero).clampLowerZero(),
   }
 }
+
+const fixBalances = ({ txLog = TxSet.EMPTY, balance, locked, withdrawable, pending, asset }) => {
+  for (const tx of txLog) {
+    if ((tx.sent || tx.data.staking) && tx.pending && !tx.error) {
+      if (tx.coinAmount.unitType.equals(tx.feeAmount.unitType)) {
+        balance = balance.sub(tx.feeAmount)
+      }
+
+      if (!tx.data.staking) {
+        // coinAmount is negative for sent tx
+        balance = balance.sub(tx.coinAmount.abs())
+      } else {
+        // staking tx
+        switch (tx.data.staking?.method) {
+          case 'delegate':
+            locked = locked.add(tx.coinAmount.abs())
+            break
+          case 'withdraw':
+            withdrawable = asset.currency.ZERO
+            break
+          case 'undelegate':
+            pending = pending.add(locked)
+            locked = asset.currency.ZERO
+            break
+        }
+      }
+    }
+  }
+  return {
+    balance: balance.clampLowerZero(),
+    locked: locked.clampLowerZero(),
+    withdrawable: withdrawable.clampLowerZero(),
+    pending: pending.clampLowerZero(),
+  }
+}
+
+const getBalanceFromAccountState = ({ asset, accountState }) => {
+  const isBase = asset.name === asset.baseAsset.name
+  return (
+    (isBase ? accountState?.balance : accountState?.tokenBalances?.[asset.name]) ||
+    asset.currency.ZERO
+  )
+}

package/src/index.js

@@ -9,6 +9,12 @@ export { default as SolanaFeeMonitor } from './fee-monitor'
 export { SolanaMonitor } from './tx-log'
 export { SolanaAccountState } from './account-state'
 export { getSolStakedFee, getStakingInfo, getUnstakingFee } from './staking-utils'
+export {
+  isSolanaStaking,
+  isSolanaUnstaking,
+  isSolanaWithdrawn,
+  isSolanaRewardsActivityTx,
+} from './txs-utils'
 export { createAndBroadcastTXFactory } from './tx-send'
 export { getBalancesFactory } from './get-balances'
 

package/src/tx-send.js

@@ -10,7 +10,6 @@ export const createAndBroadcastTXFactory = (api) => async (
 
   const {
     feeAmount,
-    shouldLog = true,
     method,
     stakeAddresses,
     seed,
@@ -33,21 +32,37 @@ export const createAndBroadcastTXFactory = (api) => async (
     reference,
     memo,
   } = options
-  let { recentBlockhash } = options
   const { baseAsset } = asset
   const from = await assetClientInterface.getReceiveAddress({
     assetName: baseAsset.name,
     walletAccount,
   })
-  const currentAccountState = await assetClientInterface.getAccountState({
-    assetName: baseAsset.name,
-    walletAccount,
-  })
 
-  recentBlockhash = recentBlockhash || (await api.getRecentBlockHash())
+  const isToken = asset.assetType === 'SOLANA_TOKEN'
+
+  // Check if receiver has address active when sending tokens.
+  if (isToken) {
+    // check address mint is the same
+    const targetMint = await api.getAddressMint(address) // null if it's a SOL address
+    if (targetMint && targetMint !== asset.mintAddress) {
+      const err = new Error('Wrong Destination Wallet')
+      err.reason = { mintAddressMismatch: true }
+      throw err
+    }
+  } else {
+    // sending SOL
+    const addressType = await api.getAddressType(address)
+    if (addressType === 'token') {
+      const err = new Error('Destination Wallet is a Token address')
+      err.reason = { wrongAddressType: true }
+      throw err
+    }
+  }
+
+  const recentBlockhash = options.recentBlockhash || (await api.getRecentBlockHash())
 
   let tokenParams = Object.create(null)
-  if (asset.assetType === 'SOLANA_TOKEN' || customMintAddress) {
+  if (isToken || customMintAddress) {
     const tokenMintAddress = customMintAddress || asset.mintAddress
     const tokenAddress = findAssociatedTokenAddress(address, tokenMintAddress)
     const [
@@ -95,7 +110,7 @@ export const createAndBroadcastTXFactory = (api) => async (
     creators,
   }
 
-  const unsignedTransaction = await createUnsignedTx({
+  const unsignedTransaction = createUnsignedTx({
     asset,
     from,
     to: address,
@@ -118,38 +133,36 @@ export const createAndBroadcastTXFactory = (api) => async (
   await baseAsset.api.broadcastTx(rawTx)
 
   const selfSend = from === address
-  const coinAmount = selfSend ? asset.currency.ZERO : amount.abs().negate()
-
-  if (shouldLog) {
-    const tx = {
-      txId,
-      confirmations: 0,
-      coinName: assetName,
-      coinAmount,
-      feeAmount,
-      feeCoinName: asset.feeAsset.name,
-      selfSend,
-      to: address,
-      data: {
-        // note
-      },
-      currencies: { [assetName]: asset.currency, [asset.feeAsset.name]: asset.feeAsset.currency },
-    }
-    await assetClientInterface.updateTxLogAndNotify({ assetName, walletAccount, txs: [tx] })
+  const isStakingTx = ['delegate', 'undelegate', 'withdraw'].includes(method)
+  const coinAmount = isStakingTx
+    ? amount.abs()
+    : selfSend
+    ? asset.currency.ZERO
+    : amount.abs().negate()
+
+  const data = isStakingTx
+    ? { staking: { ...stakingParams, stake: coinAmount.toBaseNumber() } }
+    : Object.create(null)
+
+  const tx = {
+    txId,
+    confirmations: 0,
+    coinName: assetName,
+    coinAmount,
+    feeAmount,
+    feeCoinName: asset.feeAsset.name,
+    selfSend,
+    to: address,
+    data,
+    currencies: { [assetName]: asset.currency, [asset.feeAsset.name]: asset.feeAsset.currency },
   }
+  await assetClientInterface.updateTxLogAndNotify({ assetName, walletAccount, txs: [tx] })
 
-  const changes = Object.create(null)
-  if (asset.assetType === 'SOLANA_TOKEN') {
-    Object.assign(changes, {
-      balance: currentAccountState.balance.sub(feeAmount), // solana balance
-      tokenBalances: {
-        ...currentAccountState.tokenBalances,
-        [assetName]: currentAccountState.tokenBalances[assetName].sub(coinAmount.abs()),
-      }, // SPL token balance
-    })
+  if (isToken) {
     // write tx entry in solana for token fee
-    const tx = {
+    const txForFee = {
       txId,
+      confirmations: 0,
       coinName: baseAsset.name,
       coinAmount: baseAsset.currency.ZERO,
       tokens: [assetName],
@@ -162,45 +175,12 @@ export const createAndBroadcastTXFactory = (api) => async (
         [baseAsset.feeAsset.name]: baseAsset.feeAsset.currency,
       },
     }
-    await assetClientInterface.updateTxLogAndNotify({ assetName, walletAccount, txs: [tx] })
-  } else if (customMintAddress) {
-    Object.assign(changes, {
-      balance: currentAccountState.balance.sub(feeAmount), // solana balance
-    })
-  } else if (method) {
-    // staking: no changes
-  } else {
-    // SOL transfer
-    Object.assign(changes, {
-      balance: currentAccountState.balance.sub(coinAmount.abs()).sub(feeAmount),
+    await assetClientInterface.updateTxLogAndNotify({
+      assetName: baseAsset.name,
+      walletAccount,
+      txs: [txForFee],
     })
   }
 
-  if (method) {
-    const stakingData = { ...currentAccountState.mem }
-    switch (method) {
-      case 'delegate':
-        stakingData.isDelegating = true
-        stakingData.locked = stakingData.locked.add(amount)
-        break
-      case 'undelegate':
-        stakingData.isDelegating = false
-        stakingData.pending = stakingData.pending.add(stakingData.locked)
-        stakingData.locked = asset.currency.ZERO
-        break
-      case 'withdraw':
-        stakingData.withdrawable = asset.currency.ZERO
-        break
-    }
-
-    Object.assign(changes, { mem: stakingData })
-  }
-
-  await assetClientInterface.updateAccountState({
-    assetName: baseAsset.name,
-    walletAccount,
-    newData: changes,
-  })
-
   return { txId }
 }

package/src/txs-utils.js

@@ -0,0 +1,11 @@
+import { get } from 'lodash'
+
+const isSolanaTx = (tx) => tx.coinName === 'solana'
+export const isSolanaStaking = (tx) =>
+  isSolanaTx(tx) && ['createAccountWithSeed', 'delegate'].includes(get(tx, 'data.staking.method'))
+export const isSolanaUnstaking = (tx) =>
+  isSolanaTx(tx) && get(tx, 'data.staking.method') === 'undelegate'
+export const isSolanaWithdrawn = (tx) =>
+  isSolanaTx(tx) && get(tx, 'data.staking.method') === 'withdraw'
+export const isSolanaRewardsActivityTx = (tx) =>
+  [isSolanaStaking, isSolanaUnstaking, isSolanaWithdrawn].some((fn) => fn(tx))