chore(blockchain-link): Solana getAccountInfo - optimize for basic details

packages/blockchain-link/src/workers/solana/index.ts

@@ -44,6 +44,7 @@ type SignatureWithSlot = {
 const getAllSignatures = async (
     api: SolanaAPI,
     descriptor: MessageTypes.GetAccountInfo['payload']['descriptor'],
+    fullHistory = true,
 ) => {
     let lastSignature: SignatureWithSlot | undefined;
     let keepFetching = true;
@@ -61,7 +62,7 @@ const getAllSignatures = async (
             slot: info.slot,
         }));
         lastSignature = signatures[signatures.length - 1];
-        keepFetching = signatures.length === limit;
+        keepFetching = signatures.length === limit && fullHistory;
         allSignatures = [...allSignatures, ...signatures];
     }
 
@@ -144,42 +145,25 @@ const getAccountInfo = async (request: Request<MessageTypes.GetAccountInfo>) =>
 
     const publicKey = new PublicKey(payload.descriptor);
 
-    const accountInfo = await api.getAccountInfo(publicKey);
-
-    const getTransactionPage = async (
-        txIds: string[],
-        tokenAccountsInfos: SolanaTokenAccountInfo[],
-    ) => {
-        const transactionsPage = await fetchTransactionPage(api, txIds);
-
-        const tokenMetadata = await request.getTokenMetadata();
-
-        return transactionsPage
-            .filter(isValidTransaction)
-            .map(tx =>
-                solanaUtils.transformTransaction(
-                    tx,
-                    payload.descriptor,
-                    tokenAccountsInfos,
-                    tokenMetadata,
-                ),
-            )
-            .filter((tx): tx is Transaction => !!tx);
-    };
+    const accountInfo = details === 'basic' ? null : await api.getAccountInfo(publicKey);
 
     const tokenAccounts = await api.getParsedTokenAccountsByOwner(publicKey, {
         programId: new PublicKey(TOKEN_PROGRAM_PUBLIC_KEY),
     });
 
-    const allAccounts = [payload.descriptor, ...tokenAccounts.value.map(a => a.pubkey.toString())];
+    const sortedTokenAccountPubkeys = tokenAccounts.value.map(a => a.pubkey.toString()).sort();
+
+    const allAccounts = [payload.descriptor, ...sortedTokenAccountPubkeys];
 
     const allTxIds =
         details === 'basic' || details === 'txs' || details === 'txids'
             ? Array.from(
                   new Set(
                       (
                           await Promise.all(
-                              allAccounts.map(account => getAllSignatures(api, account)),
+                              allAccounts.map(account =>
+                                  getAllSignatures(api, account, details !== 'basic'),
+                              ),
                           )
                       )
                           .flat()
@@ -204,6 +188,26 @@ const getAccountInfo = async (request: Request<MessageTypes.GetAccountInfo>) =>
         decimals: a.account.data.parsed?.info?.tokenAmount?.decimals as number | undefined,
     }));
 
+    const getTransactionPage = async (
+        txIds: string[],
+        tokenAccountsInfos: SolanaTokenAccountInfo[],
+    ) => {
+        const transactionsPage = await fetchTransactionPage(api, txIds);
+
+        const tokenMetadata = await request.getTokenMetadata();
+
+        return transactionsPage
+            .filter(isValidTransaction)
+            .map(tx =>
+                solanaUtils.transformTransaction(
+                    tx,
+                    payload.descriptor,
+                    tokenAccountsInfos,
+                    tokenMetadata,
+                ),
+            )
+            .filter((tx): tx is Transaction => !!tx);
+    };
     const transactionPage =
         details === 'txs' ? await getTransactionPage(txIdPage, tokenAccountsInfos) : undefined;
 
@@ -217,9 +221,6 @@ const getAccountInfo = async (request: Request<MessageTypes.GetAccountInfo>) =>
 
     const balance = await api.getBalance(publicKey);
 
-    // https://solana.stackexchange.com/a/13102
-    const rent = await api.getMinimumBalanceForRentExemption(accountInfo?.data.byteLength || 0);
-
     const account: AccountInfo = {
         descriptor: payload.descriptor,
         balance: balance.toString(),
@@ -243,7 +244,10 @@ const getAccountInfo = async (request: Request<MessageTypes.GetAccountInfo>) =>
             ? {
                   misc: {
                       owner: accountInfo.owner.toString(),
-                      rent,
+                      // https://solana.stackexchange.com/a/13102
+                      rent: await api.getMinimumBalanceForRentExemption(
+                          accountInfo?.data.byteLength || 0,
+                      ),
                   },
               }
             : {}),

suite-common/wallet-utils/src/accountUtils.ts

@@ -808,6 +808,9 @@ export const isAccountOutdated = (account: Account, freshInfo: AccountInfo) => {
                 // changed stake pool
                 freshInfo.misc!.staking?.poolId !== account.misc.staking.poolId
             );
+        case 'solana':
+            // compare last transaction signature since the total number of txs may not be fetched fully
+            return freshInfo.history.txids?.[0] !== account.history.txids?.[0];
         default:
             return false;
     }