decrease default BLOCK_GAS_LIMIT

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tx-manager",
-  "version": "0.3.0",
+  "version": "0.2.9",
   "description": "",
   "main": "index.js",
   "scripts": {
@@ -23,7 +23,7 @@
   "dependencies": {
     "async-mutex": "^0.2.4",
     "ethers": "^5.0.17",
-    "gas-price-oracle": "^0.3.1",
+    "gas-price-oracle": "^0.2.2",
     "web3-core-promievent": "^1.3.0"
   },
   "devDependencies": {

src/Transaction.js

@@ -96,16 +96,16 @@ class Transaction {
    * @private
    */
   async _execute() {
-    const mutexRelease = await this.manager._mutex.acquire()
+    await this.manager._mutex.acquire()
     try {
       await this._prepare()
       await this._send()
-      const receipt = await this._waitForConfirmations()
+      const receipt = this._waitForConfirmations()
       // we could have bumped nonce during execution, so get the latest one + 1
       this.manager._nonce = this.tx.nonce + 1
       return receipt
     } finally {
-      mutexRelease()
+      this.manager._mutex.release()
     }
   }
 
@@ -129,9 +129,7 @@ class Transaction {
       }
     }
     if (!this.tx.gasPrice) {
-      const fastGasPrice = BigNumber.from(await this._getGasPrice('fast'))
+      this.tx.gasPrice = await this._getGasPrice('fast')
-      const maxGasPrice = parseUnits(this.config.MAX_GAS_PRICE.toString(), 'gwei')
-      this.tx.gasPrice = min(fastGasPrice, maxGasPrice).toHexString()
     }
     if (!this.manager._nonce) {
       this.manager._nonce = await this._getLastNonce()
@@ -302,11 +300,8 @@ class Transaction {
         console.log(
           `Gas price ${formatUnits(this.tx.gasPrice, 'gwei')} gwei is too low, increasing and retrying`,
         )
-        if (this._increaseGasPrice()) {
+        this._increaseGasPrice()
-          return this._send()
+        return this._send()
-        } else {
-          throw new Error('Already at max gas price, but still not enough to submit the transaction')
-        }
       }
 
       if (this._hasError(message, sameTxErrors)) {
@@ -331,17 +326,17 @@ class Transaction {
   }
 
   _increaseGasPrice() {
-    const maxGasPrice = parseUnits(this.config.MAX_GAS_PRICE.toString(), 'gwei')
     const minGweiBump = parseUnits(this.config.MIN_GWEI_BUMP.toString(), 'gwei')
     const oldGasPrice = BigNumber.from(this.tx.gasPrice)
-    if (oldGasPrice.gte(maxGasPrice)) {
-      console.log('Already at max gas price, not bumping')
-      return false
-    }
     const newGasPrice = max(
       oldGasPrice.mul(100 + this.config.GAS_BUMP_PERCENTAGE).div(100),
       oldGasPrice.add(minGweiBump),
     )
+    const maxGasPrice = parseUnits(this.config.MAX_GAS_PRICE.toString(), 'gwei')
+    if (oldGasPrice.eq(maxGasPrice)) {
+      console.log('Already at max gas price, not bumping')
+      return false
+    }
     this.tx.gasPrice = min(newGasPrice, maxGasPrice).toHexString()
     console.log(`Increasing gas price to ${formatUnits(this.tx.gasPrice, 'gwei')} gwei`)
     return true

src/TxManager.js

@@ -18,14 +18,14 @@ const defaultConfig = {
 }
 
 class TxManager {
-  constructor({ privateKey, rpcUrl, broadcastNodes = [], config = {}, gasPriceOracleConfig = {} }) {
+  constructor({ privateKey, rpcUrl, broadcastNodes = [], config = {} }) {
     this.config = Object.assign({ ...defaultConfig }, config)
     this._privateKey = privateKey.startsWith('0x') ? privateKey : '0x' + privateKey
     this._provider = new ethers.providers.JsonRpcProvider(rpcUrl)
     this._wallet = new ethers.Wallet(this._privateKey, this._provider)
     this.address = this._wallet.address
     this._broadcastNodes = broadcastNodes
-    this._gasPriceOracle = new GasPriceOracle({ defaultRpc: rpcUrl, ...gasPriceOracleConfig })
+    this._gasPriceOracle = new GasPriceOracle({ defaultRpc: rpcUrl })
     this._mutex = new Mutex()
     this._nonce = null
   }

test/TxManager.test.js

@@ -101,23 +101,5 @@ describe('TxManager', () => {
 
       console.log('receipt', receipt)
     })
-    it.only('should send multiple txs', async () => {
-      const genTx = value => ({
-        value,
-        to: '0x0039F22efB07A647557C7C5d17854CFD6D489eF3',
-      })
-      await Promise.all([
-        manager.createTx(genTx(1)).send(),
-        manager.createTx(genTx(2)).send(),
-        manager.createTx(genTx(3)).send(),
-        manager.createTx(genTx(4)).send(),
-        manager.createTx(genTx(5)).send(),
-        manager.createTx(genTx(6)).send(),
-        manager.createTx(genTx(7)).send(),
-        manager.createTx(genTx(8)).send(),
-        manager.createTx(genTx(9)).send(),
-        manager.createTx(genTx(10)).send(),
-      ])
-    })
   })
 })

yarn.lock

@@ -1141,10 +1141,10 @@ functional-red-black-tree@^1.0.1:
   resolved "https://registry.yarnpkg.com/functional-red-black-tree/-/functional-red-black-tree-1.0.1.tgz#1b0ab3bd553b2a0d6399d29c0e3ea0b252078327"
   integrity sha1-GwqzvVU7Kg1jmdKcDj6gslIHgyc=
 
-gas-price-oracle@^0.3.1:
+gas-price-oracle@^0.2.2:
-  version "0.3.1"
+  version "0.2.2"
-  resolved "https://registry.yarnpkg.com/gas-price-oracle/-/gas-price-oracle-0.3.1.tgz#4977de5edfa49ae17defad0e0d06f9a67bdf2cc6"
+  resolved "https://registry.yarnpkg.com/gas-price-oracle/-/gas-price-oracle-0.2.2.tgz#32c57a9aa6bc69152be96812880232efebfecbc6"
-  integrity sha512-v82AEeQVnO4lGKoJbRqLI9YdOVoImBorYDSOMTGZZAF+RNC89fdTsbhLQVsI/1Zsduqcpiz4VqlQ/4I8wwCbAg==
+  integrity sha512-I4+rLbc7C1vgYXV+cYY0MKeqdZVna2hXpNfD2fcIvf/wIgvtIYmG9gsmhiaYGSgOE2RSPUs2xf/W4K2nJOoNuQ==
   dependencies:
     axios "^0.19.2"
     bignumber.js "^9.0.0"