feat(tandomnness): improve performance and reability

Author: GabrielMartinezRodriguez

packages/contracts/src/randomness/Random.sol

@@ -11,7 +11,7 @@ contract Random is RandomCommitment, Drand {
     * This is necessary, because whenever a Drand value is generated at time T, it is not possible to guarantee it 
     * will be posted on a block with timestamp T (even if such a block exists) because of network delays.
     */
-    uint256 public constant DRAND_DELAY = 2;
+    uint256 public constant DRAND_DELAY = 4;
     /**
      * The minimum amount of time (in seconds) that commitments to future Drand randomness must be made
      * in advance. This delay is relative to Drand timestamp (so DRAND_DELAY must also be added).

packages/randomness-service/src/CustomGasEstimator.ts

@@ -18,6 +18,10 @@ export class CustomGasEstimator extends DefaultGasLimitEstimator {
             return ok(100000n)
         }
 
+        if (transaction.functionName === "postDrand") {
+            return ok(220000n)
+        }
+
         return this.simulateTransactionForGas(transactionManager, transaction)
     }
 }

packages/randomness-service/src/DrandService.ts

@@ -38,7 +38,7 @@ export class DrandService {
         }
 
         const url = `${env.EVM_DRAND_URL}/rounds/${round}`
-        const response = await ResultAsync.fromPromise(fetchWithRetry(url, {}, 2, 500), unknownToError)
+        const response = await ResultAsync.fromPromise(fetchWithRetry(url, {}, 2, 1000), unknownToError)
 
         if (response.isErr()) {
             return err(DrandError.NetworkError)

packages/randomness-service/src/index.ts

@@ -51,6 +51,10 @@ class RandomnessService {
             console.error(description)
         })
 
+        // Start fetching Drand beacons for past rounds. Subsequently, beacons will be fetched at regular intervals
+        // to maintain synchronization with the Drand network
+        this.handleNewDrandBeacons()
+
         // Synchronize the retrieval of new Drand beacons with the Drand network to request them as soon as they become available.
         const periodMs = Number(env.EVM_DRAND_PERIOD_SECONDS) * MS_IN_SECOND
         const drandGenesisTimestampMs = Number(env.EVM_DRAND_GENESIS_TIMESTAMP_SECONDS) * MS_IN_SECOND
@@ -255,22 +259,19 @@ class RandomnessService {
 
         const randomnessToReveal = this.randomnessRepository.getRandomnessForBlockNumber(nextBlockNumber)
 
-        if (!randomnessToReveal) {
-            console.warn("Not found randomness to reveal with block number", nextBlockNumber)
-            return transactions
-        }
-
-        const revealValueTransaction = this.transactionFactory.createRevealValueTransaction(randomnessToReveal)
+        if (randomnessToReveal) {
+            const revealValueTransaction = this.transactionFactory.createRevealValueTransaction(randomnessToReveal)
 
-        transactions.unshift(revealValueTransaction)
+            transactions.unshift(revealValueTransaction)
 
-        randomnessToReveal.addRevealTransactionIntentId(revealValueTransaction.intentId)
+            randomnessToReveal.addRevealTransactionIntentId(revealValueTransaction.intentId)
 
-        this.randomnessRepository.updateRandomness(randomnessToReveal).then((result) => {
-            if (result.isErr()) {
-                console.error("Failed to update randomness", result.error)
-            }
-        })
+            this.randomnessRepository.updateRandomness(randomnessToReveal).then((result) => {
+                if (result.isErr()) {
+                    console.error("Failed to update randomness", result.error)
+                }
+            })
+        }
 
         transactions.push(...this.pendingPostDrandTransactions)