fix: nonce issue

Author: janek26

actions/transferAll/core.ts

@@ -11,6 +11,7 @@ import { Account } from "../../ui/pickAccounts";
 import TOKENS from "../../default-tokens.json";
 import { Ora } from "ora";
 import { oraLog } from "../../oraLog";
+import { getNonce } from "./getNonce";
 
 export async function transferAll(acc: Account, newAddress: string, ora: Ora) {
   const { privateKey } = acc;
@@ -52,7 +53,8 @@ export async function transferAll(acc: Account, newAddress: string, ora: Ora) {
     });
 
   if (calls.length) {
-    const { suggestedMaxFee } = await account.estimateFee(calls);
+    const nonce = await getNonce(acc.address, acc.networkId); // use this to get the nonce, as this covers old and new way of getting nonce
+    const { suggestedMaxFee } = await account.estimateFee(calls, { nonce });
 
     const callsWithFee = calls.map((c) => {
       const tokenDetails = tokens.find((t) => t.symbol === "ETH");

actions/transferAll/getNonce.ts

@@ -0,0 +1,25 @@
+import { SequencerProvider as NewProvider } from "starknet-490";
+import { SequencerProvider as OldProvider } from "starknet";
+
+export async function getNonce(address: string, network: string) {
+  const lowerCaseAddress = address.toLowerCase();
+  try {
+    const oldProvider = new OldProvider({ network: network as any });
+    const nonceResponse = await oldProvider
+      .callContract({
+        contractAddress: lowerCaseAddress,
+        entrypoint: "get_nonce",
+      })
+      .catch(() =>
+        oldProvider.callContract({
+          contractAddress: lowerCaseAddress,
+          entrypoint: "getNonce",
+        })
+      );
+    const nonce = nonceResponse.result[0];
+    return nonce;
+  } catch {
+    const newProvider = new NewProvider({ network: network as any });
+    return newProvider.getNonce(lowerCaseAddress);
+  }
+}

actions/transferAll/ui.ts

@@ -36,7 +36,7 @@ export async function showTransferAll(accounts: Account[]) {
 
   transferResults.forEach((result) => {
     if (result.status === "rejected") {
-      spinner.fail(result.reason);
+      spinner.fail(result.reason?.toString());
     }
   });
 

getImplementation.ts

@@ -6,7 +6,7 @@ export async function getImplementation(
   network: "mainnet-alpha" | "goerli-alpha"
 ) {
   const provider = new SequencerProvider({ network });
-  const multicallProvider = new Multicall(provider);
+  const multicallProvider = new Multicall(provider as any);
 
   const implementationAnswers = await Promise.allSettled(
     addresses.map((address) =>

getSigner.ts

@@ -6,7 +6,7 @@ export async function getSigners(
   network: "mainnet-alpha" | "goerli-alpha"
 ) {
   const provider = new SequencerProvider({ network });
-  const multicallProvider = new Multicall(provider);
+  const multicallProvider = new Multicall(provider as any);
 
   const signerAnswers = await Promise.allSettled(
     addresses.map((address) =>

getTokenBalance.ts

@@ -10,7 +10,7 @@ export async function getBalances(
   const tokens = TOKENS.filter((token) => token.network === network);
   const tokenAddresses = tokens.map((token) => token.address);
   const provider = new SequencerProvider({ network });
-  const multicallProvider = new Multicall(provider);
+  const multicallProvider = new Multicall(provider as any);
 
   const addressesTokensCombinations = tokenAddresses.flatMap((token) =>
     addresses.map((address) => ({ address, token }))

getVersion.ts

@@ -6,7 +6,7 @@ export async function getVersion(
   network: "mainnet-alpha" | "goerli-alpha"
 ) {
   const provider = new SequencerProvider({ network });
-  const multicallContract = new Multicall(provider);
+  const multicallContract = new Multicall(provider as any);
 
   const versionAnswers = await Promise.allSettled(
     addresses.map((address) =>

package.json

@@ -40,7 +40,8 @@
     "ora": "5",
     "prompts": "^2.4.2",
     "semver": "^7.3.7",
-    "starknet": "^4.1.0",
+    "starknet-490": "npm:[email protected]",
+    "starknet": "4.1.0",
     "starknet-390": "npm:[email protected]",
     "yup": "^1.0.0-beta.4"
   }