feat: add tx polling timeout and comments

src/screens/staking/lib/staking_sdk/utils/coins.ts

@@ -35,7 +35,9 @@ const denomMap: Record<DenomToNormalise, [CoinDenom, number]> = {
   ADYM: [CoinDenom.DYM, aExp],
   AEVMOS: [CoinDenom.EVMOS, aExp],
   AISLM: [CoinDenom.ISLM, aExp],
-  inj: [CoinDenom.INJ, exp0], // This is handled when normalising the coin
+  // Because inj != INJ, this needs to keep the lower case, it is handled when
+  // normalising the coin
+  inj: [CoinDenom.INJ, exp0],
   PPICA: [CoinDenom.PICA, pExp],
   UAKT: [CoinDenom.AKT, uExp],
   UATOM: [CoinDenom.ATOM, uExp],

src/screens/staking/lib/staking_sdk/wallet_operations/cosmos.ts

@@ -131,14 +131,17 @@ const getCosmosError = (err: Error): CosmosError => {
 
 type PollTxOpts = {
   rpc: string;
+  timeout?: number;
   txHash: string;
 };
 
-const pollTx = async ({ rpc, txHash }: PollTxOpts) => {
+const pollTx = async ({ rpc, timeout, txHash }: PollTxOpts) => {
   const tmClient = await connectComet(rpc);
 
   const queryClient = QueryClient.withExtensions(tmClient, setupTxExtension);
 
+  let elapsed = 0;
+
   while (true) {
     try {
       const tx = await queryClient.tx.getTx(txHash);
@@ -149,7 +152,15 @@ const pollTx = async ({ rpc, txHash }: PollTxOpts) => {
       console.log("debug: cosmos.ts: err", err);
     }
 
-    await new Promise((resolve) => setTimeout(resolve, 5000));
+    const toWait = 5000;
+
+    elapsed += toWait;
+
+    if (timeout && elapsed >= timeout) {
+      throw new Error("Timeout");
+    }
+
+    await new Promise((resolve) => setTimeout(resolve, toWait));
   }
 };
 
@@ -175,6 +186,8 @@ const signAndBroadcastEthermint = async (
 
   const { account_number: accountNumber, sequence } = latestAccountInfo;
 
+  // The `sequence` can be `0` here if the account is new. But check that both
+  // values are set in case the types are not valid.
   if (typeof sequence !== "number" || !accountNumber) {
     throw new Error("Account number or sequence is missing");
   }
@@ -249,6 +262,9 @@ const signAndBroadcastEthermint = async (
 
       const txResult = await pollTx({
         rpc,
+        // Three minutes, since some transactions (e.g. for Injective) can take
+        // a long time
+        timeout: 1000 * 60 * 3,
         txHash,
       });