.nvmrc

@@ -1 +1 @@
-v14.21.3
+v20.3.0

README.md

@@ -6,19 +6,23 @@ This is a library providing convenient and fast access to oracles for Tornado-sp
 
 <hr>
 
-- Create `.npmrc` file in project root with content `@tornado:registry=https://git.tornado.ws/api/packages/tornado-packages/npm/`
-- Run `npm i @tornado/tornado-oracles`
+* Create `.npmrc` file in project root with content `@tornado:registry=https://git.tornado.ws/api/packages/tornado-packages/npm/`
+* Run `npm i @tornado/tornado-oracles`
+
+
 
 ### Import
 
 <hr>
 
 ```typescript
-const { TornadoFeeOracleV5, TornadoFeeOracleV5, TokenPriceOracle } = require('@tornado/tornado-oracles');
+const {TornadoFeeOracleV5, TornadoFeeOracleV5, TokenPriceOracle} = require('@tornado/tornado-oracles')
 or
-import { TornadoFeeOracleV5, TornadoFeeOracleV5, TokenPriceOracle } from '@tornado/tornado-oracles';
+import {TornadoFeeOracleV5, TornadoFeeOracleV5, TokenPriceOracle} from '@tornado/tornado-oracles'
 ```
 
+
+
 ### Usage
 
 <hr>
@@ -26,31 +30,31 @@ import { TornadoFeeOracleV5, TornadoFeeOracleV5, TokenPriceOracle } from '@torna
 ##### Estimate withdrawal gas costs
 
 ```typescript
-import { TornadoFeeOracleV5 } from '@tornado/tornado-oracles';
+import { TornadoFeeOracleV5 } from '@tornado/tornado-oracles'
 
 const tx: TransactionData = {
     to: tornadoProxyLightAddress,
     data: poolInstance.methods.withdraw(...params).encodeABI(),
-  value: withdrawalProofArgs[5],
-};
+    value: withdrawalProofArgs[5]
+}
 
-const feeOracle = new TornadoFeeOracleV5(1, 'https://eth.llamarpc.com'); // First parameter - chain ID
-const withdrawalGas = await feeOracle.getGas(tx, 'relayer_withdrawal');
+const feeOracle = new TornadoFeeOracleV5(1, "https://eth.llamarpc.com"); // First parameter - chain ID
+const withdrawalGas = await feeOracle.getGas(tx, "relayer_withdrawal");
 ```
 
 ##### Estimate gas price and gas limit to send transaction
 
 ```typescript
-import { TornadoFeeOracleV5 } from '@tornado/tornado-oracles';
+import { TornadoFeeOracleV5 } from '@tornado/tornado-oracles'
 
 const incompleteTx: TransactionData = {
     to: tornadoProxyLightAddress,
     data: poolInstance.methods.withdraw(...params).encodeABI(),
-  value: withdrawalProofArgs[5],
-};
+    value: withdrawalProofArgs[5]
+}
 
-const feeOracle = new TornadoFeeOracleV5(1, 'https://eth.llamarpc.com');
-const transactionType: TxType = 'relayer_withdrawal';
+const feeOracle = new TornadoFeeOracleV5(1, "https://eth.llamarpc.com");
+const transactionType: TxType = "relayer_withdrawal";
 const gasPrice = await feeOracle.getGasPriceInHex(transactionType);
 const gasLimit = await feeOracle.getGasLimit(incompleteTx, transactionType);
 
@@ -60,9 +64,9 @@ const tx: TransactionData = Object.assign({ gasPrice, gasLimit }, incompleteTx);
 ##### Get token prices (rate to ETH) for tokens that used in Tornado
 
 ```typescript
-import { TokenPriceOracle } from '@tornado/tornado-oracles';
+import { TokenPriceOracle } from '@tornado/tornado-oracles'
 
-const priceOracle = new TokenPriceOracle('https://eth.llamarpc.com');
+const priceOracle = new TokenPriceOracle("https://eth.llamarpc.com");
 const tokenPrices = await priceOracle.fetchPrices();
 
 console.log(tokenPrices); // All prices in WEI

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tornado/tornado-oracles",
-  "version": "0.3.0",
+  "version": "0.2.3",
   "description": "Gas oracle for Tornado-specific transactions",
   "main": "./lib/index.js",
   "types": "./lib/index.d.ts",
@@ -30,12 +30,12 @@
   "dependencies": {
     "@tornado/gas-price-oracle": "^0.5.3",
     "@tornado/tornado-config": "^2.0.0",
-    "@types/node": "^20.5.1",
     "bignumber.js": "^9.1.1",
     "ethers": "5.7"
   },
   "devDependencies": {
     "@typechain/ethers-v5": "^11.1.1",
+    "@types/node": "^20.4.10",
     "nodemon": "^3.0.1",
     "prettier": "^3.0.1",
     "ts-node": "^10.9.1",

src/feeOracle.ts

@@ -2,14 +2,14 @@ import { GasPriceOracle } from '@tornado/gas-price-oracle';
 import { BigNumber, BigNumberish, ethers } from 'ethers';
 import { parseUnits } from 'ethers/lib/utils';
 import { TransactionData, TxType, ITornadoFeeOracle, GasPrice, LegacyGasPriceKey } from './types';
-import { JsonRpcProvider } from '@ethersproject/providers';
+import { Provider } from '@ethersproject/abstract-provider';
 import { ChainId, defaultGasPrices } from './config';
-import { bump, calculateGasPriceInWei, convertETHToToken, fromGweiToWeiHex, serializeTx } from './utils';
+import { bump, calculateGasPriceInWei, fromGweiToWeiHex, serializeTx } from './utils';
 import { getOptimismL1FeeOracle } from './contracts/factories';
 import { AvailableTokenSymbols } from '@tornado/tornado-config';
 
 export abstract class TornadoFeeOracle implements ITornadoFeeOracle {
-  protected provider: JsonRpcProvider;
+  protected provider: Provider;
 
   public constructor(
     protected chainId: ChainId,
@@ -94,7 +94,7 @@ export abstract class TornadoFeeOracle implements ITornadoFeeOracle {
   /**
    * Estimates gas limit for transaction (or basic gas limit, if no tx data provided)
    * @param {TransactionData} tx Transaction data (object in web3 / ethers format)
-   * @param {TxType} type Tornado transaction type: withdrawal by user, withdrawal by relayer, relayer fee check or 'other'
+   * @param {TxType} type Tornado transaction type: withdrawal by user, withdrawal by relayer or 'other'
    * @returns {Promise<BigNumber>} Gas limit
    */
   abstract getGasLimit(tx?: TransactionData, type?: TxType): Promise<BigNumber>;
@@ -105,30 +105,10 @@ export abstract class TornadoFeeOracle implements ITornadoFeeOracle {
    * and if the relayer pays a commission and the transfer of tokens fails, this commission will remain to the relayer.
    *
    * Refund needed that recipient can use tokens after withdrawal (covers gas fee for send/swap)
-   * @param {TransactionData} [tx] Transaction data (object in web3 / ethers format)
-   * @param {TxType} [type] Tornado transaction type: withdrawal by user, withdrawal by relayer, relayer fee check or 'other'
-   * @returns {Promise<BigNumber>} Refund amount in WEI
+   * @returns {Promise<BigNumber>} Refund amount
    */
-  async calculateRefundInETH(tx?: TransactionData, type?: TxType): Promise<BigNumber> {
-    return (await this.getGas(tx, type)).mul(2);
-  }
-
-  /**
-   * Get refund amount on ETH or Goerli in non-native token
-   * @param {BigNumberish} tokenPriceInEth Token price in WEI in native currency
-   * @param {string | number} tokenDecimals Token (currency) decimals
-   * @param {TransactionData} [tx] Transaction data (object in web3 / ethers format)
-   * @param {TxType} [type] Tornado transaction type: withdrawal by user, withdrawal by relayer, relayer fee check or 'other'
-   * @returns {Promise<BigNumber>} Refund amount in WEI in selected token
-   */
-  async calculateRefundInToken(
-    tokenPriceInEth: BigNumberish,
-    tokenDecimals: string | number,
-    tx?: TransactionData,
-    type?: TxType,
-  ): Promise<BigNumber> {
-    const refundInEth = await this.calculateRefundInETH(tx, type);
-    return convertETHToToken(refundInEth, tokenDecimals, tokenPriceInEth);
+  async calculateRefundInETH(): Promise<BigNumber> {
+    return (await this.getGas()).mul(2);
   }
 
   /**
@@ -138,26 +118,28 @@ export abstract class TornadoFeeOracle implements ITornadoFeeOracle {
    * @param {TxType} type Tornado transaction type: withdrawal costs calculation from user side or from relayer side
    * @param {TransactionData} tx Transaction data (object in web3 / ethers format)
    * @param {number} relayerFeePercent Relayer fee percent from the transaction amount (for example, 0.15 for BNB or 0.4 for ETH Mainnet)
-   * @param {AvailableTokenSymbols | Uppercase<AvailableTokenSymbols>} currency Currency symbol
-   * @param {number | string } amount Withdrawal amount in selected currency
-   * @param {number | string } decimals Token (currency) decimals
-   * @param {BigNumberish} [refund=0] Refund in ETH, if withdrawed other tokens on Mainnet (not ETH)
-   * @param {BigNumberish} [tokenPriceInEth] If withdrawing other token on Mainnet or Goerli, need to provide token price in ETH (in WEI)
+   * @param {string} currency Currency symbol
+   * @param {number} amount Withdrawal amount in selected currency
+   * @param {number} decimals Token (currency) decimals
+   * @param {BigNumberish} refund Refund in ETH, if withdrawed other tokens on Mainnet (not ETH)
+   * @param {BigNumberish} tokenPriceInEth If withdrawing other token on Mainnet or Goerli, need to provide token price in ETH (in WEI)
    * @returns {Promise<BigNumber>} Fee in WEI
    */
   async calculateWithdrawalFeeViaRelayer(
     type: TxType,
     tx: TransactionData,
     relayerFeePercent: number,
-    currency: AvailableTokenSymbols | Uppercase<AvailableTokenSymbols>,
-    amount: string | number,
-    decimals: string | number,
+    currency: AvailableTokenSymbols,
+    amount: string,
+    decimals: number,
     refund: BigNumberish = 0,
     tokenPriceInEth?: BigNumberish,
   ): Promise<BigNumber> {
+    let withdrawalFee = BigNumber.from(0);
     const gasCosts = await this.getGas(tx, type);
+    withdrawalFee = gasCosts;
 
-    const relayerFee = parseUnits(amount.toString(), decimals)
+    const relayerFee = parseUnits(amount, decimals)
       .mul(`${relayerFeePercent * 1e10}`)
       .div(`${100 * 1e10}`);
 
@@ -166,11 +148,10 @@ export abstract class TornadoFeeOracle implements ITornadoFeeOracle {
         console.error('Token price is required argument, if not native chain token is withdrawed');
         return BigNumber.from(0);
       }
-
-      const feeInEth = gasCosts.add(refund);
-      return convertETHToToken(feeInEth, decimals, tokenPriceInEth).add(relayerFee);
+      const tokenDecimals = BigNumber.from(10).pow(decimals);
+      return withdrawalFee.add(refund).mul(tokenDecimals).div(tokenPriceInEth).add(relayerFee);
     }
 
-    return gasCosts.add(relayerFee);
+    return withdrawalFee.add(relayerFee);
   }
 }

src/feeOracleV5.ts

@@ -4,11 +4,8 @@ import { ITornadoFeeOracle, GasPrice, TransactionData, TxType, LegacyGasPrices,
 import { GasPriceOracle } from '@tornado/gas-price-oracle';
 import { BigNumber } from 'ethers';
 import { bump } from './utils';
-import { TornadoFeeOracleV4 } from './feeOracleV4';
 
 export class TornadoFeeOracleV5 extends TornadoFeeOracle implements ITornadoFeeOracle {
-  private fallbackFeeOracle: TornadoFeeOracleV4;
-
   public constructor(chainId: number, rpcUrl: string, defaultGasPrices?: LegacyGasPrices) {
     const oracleConfig = {
       chainId,
@@ -21,12 +18,10 @@ export class TornadoFeeOracleV5 extends TornadoFeeOracle implements ITornadoFeeO
     const gasPriceOracle = new GasPriceOracle(oracleConfig);
 
     super(chainId, rpcUrl, gasPriceOracle);
-
-    this.fallbackFeeOracle = new TornadoFeeOracleV4(chainId, rpcUrl, defaultGasPrices);
   }
 
   async getGasLimit(tx?: TransactionData, type: TxType = 'other', bumpPercent: number = 20): Promise<BigNumber> {
-    if (!tx || Object.keys(tx).length === 0) return this.fallbackFeeOracle.getGasLimit(tx, type, bumpPercent);
+    if (!tx) return BigNumber.from(21_000);
 
     /* Relayer gas limit must be lower so that fluctuations in gas price cannot lead to the fact that
      * the relayer will actually pay for gas more than the money allocated for this by the user

src/types.ts

@@ -36,13 +36,7 @@ export interface ITornadoFeeOracle {
   getGasPrice: (type?: TxType, speed?: LegacyGasPriceKey, bumpPercent?: number) => Promise<GasPrice>;
   getGasPriceInHex: (type?: TxType, speed?: LegacyGasPriceKey, bumpPercent?: number) => Promise<string>;
   getGasLimit: (tx?: TransactionData, type?: TxType, bumpPercent?: number) => Promise<BigNumber>;
-  calculateRefundInETH: (tx?: TransactionData, type?: TxType) => Promise<BigNumber>;
-  calculateRefundInToken: (
-    tokenPriceInEth: BigNumberish,
-    tokenDecimals: string | number,
-    tx?: TransactionData,
-    type?: TxType,
-  ) => Promise<BigNumber>;
+  calculateRefundInETH: () => Promise<BigNumber>;
   calculateWithdrawalFeeViaRelayer: (
     type: TxType,
     tx: TransactionData,

src/utils.ts

@@ -32,12 +32,3 @@ export function bump(value: BigNumberish, percent: number): BigNumber {
 export function fromGweiToWeiHex(value: number | string): BigNumberish {
   return BigNumber.from(BigNumberFloat(value).times(GWEI).toString()).toHexString();
 }
-
-export function convertETHToToken(
-  amountInWEI: BigNumberish,
-  tokenDecimals: number | string,
-  tokenPriceInWei: BigNumberish,
-): BigNumber {
-  const tokenDecimalsMultiplier = BigNumber.from(10).pow(tokenDecimals);
-  return BigNumber.from(amountInWEI).mul(tokenDecimalsMultiplier).div(tokenPriceInWei);
-}

yarn.lock

@@ -411,10 +411,10 @@
     lodash "^4.17.15"
     ts-essentials "^7.0.1"
 
-"@types/node@^20.5.1":
-  version "20.5.1"
-  resolved "https://registry.yarnpkg.com/@types/node/-/node-20.5.1.tgz#178d58ee7e4834152b0e8b4d30cbfab578b9bb30"
-  integrity sha512-4tT2UrL5LBqDwoed9wZ6N3umC4Yhz3W3FloMmiiG4JwmUJWpie0c7lcnUNd4gtMKuDEO4wRVS8B6Xa0uMRsMKg==
+"@types/node@^20.4.10":
+  version "20.4.10"
+  resolved "https://registry.yarnpkg.com/@types/node/-/node-20.4.10.tgz#73c9480791e3ddeb4887a660fc93a7f59353ad45"
+  integrity sha512-vwzFiiy8Rn6E0MtA13/Cxxgpan/N6UeNYR9oUu6kuJWxu6zCk98trcDp8CBhbtaeuq9SykCmXkFr2lWLoPcvLg==
 
 "@types/prettier@^2.1.1":
   version "2.7.3"
@@ -959,9 +959,9 @@ prettier@^2.3.1:
   integrity sha512-tdN8qQGvNjw4CHbY+XXk0JgCXn9QiF21a55rBe5LJAU+kDyC4WQn4+awm2Xfk2lQMk5fKup9XgzTZtGkjBdP9Q==
 
 prettier@^3.0.1:
-  version "3.0.2"
-  resolved "https://registry.yarnpkg.com/prettier/-/prettier-3.0.2.tgz#78fcecd6d870551aa5547437cdae39d4701dca5b"
-  integrity sha512-o2YR9qtniXvwEZlOKbveKfDQVyqxbEIWn48Z8m3ZJjBjcCmUy3xZGIv+7AkaeuaTr6yPXJjwv07ZWlsWbEy1rQ==
+  version "3.0.1"
+  resolved "https://registry.yarnpkg.com/prettier/-/prettier-3.0.1.tgz#65271fc9320ce4913c57747a70ce635b30beaa40"
+  integrity sha512-fcOWSnnpCrovBsmFZIGIy9UqK2FaI7Hqax+DIO0A9UxeVoY4iweyaFjS5TavZN97Hfehph0nhsZnjlVKzEQSrQ==
 
 pstree.remy@^1.1.8:
   version "1.1.8"