fix: fix l1 to l2 conversion issues

packages/rewards-calculator/src/chain.ts

@@ -10,7 +10,7 @@ import {
   parseAbiItem,
 } from "viem";
 import { logger } from "./logger";
-import { bigSum, fromBase58, withCache } from "./utils";
+import { bigSum, fromBase58 } from "./utils";
 import { Rewards } from "./reward";
 import { Workers } from "./workers";
 import { fordefiRequest } from "./fordefi/request";
@@ -31,8 +31,8 @@ function getNitroGenesisBlock(chainId: number) {
 let lastKnowBlockPair: {l1Block: bigint, l2Block: bigint} | undefined = undefined
 
 // ref https://github.com/OffchainLabs/arbitrum-sdk/blob/5ef44308d3c89fd956c9dfdc59b6776b88afd251/src/lib/utils/lib.ts#L90
-export async function getFirstBlockForL1Block(targetL1Block: number | bigint | undefined): Promise<bigint> {
-  targetL1Block = targetL1Block == null ? await l1Client.getBlockNumber() : BigInt(targetL1Block)
+export async function getFirstBlockForL1Block(targetL1Block: number | bigint): Promise<bigint> {
+  targetL1Block = BigInt(targetL1Block)
 
   let start: bigint
   if (lastKnowBlockPair == null || lastKnowBlockPair.l1Block > targetL1Block) {
@@ -48,7 +48,9 @@ export async function getFirstBlockForL1Block(targetL1Block: number | bigint | u
     return lastKnowBlockPair.l2Block
   }
 
-  let end = await publicClient.getBlockNumber()
+  // for some reason .getBlockNumber() returns inconsistent result to .getBlock(),
+  // so, since we use .getBlock() further down the code, we should use .getBlock() here as well
+  let end = await publicClient.getBlock().then(block => block.number)
 
   let targetL2Block: bigint | undefined
   while (start <= end) {
@@ -127,9 +129,9 @@ export async function getLatestDistributionBlock() {
   return -1n
 }
 
-export const currentApy = withCache(_currentApy)
+// export const currentApy = withCache(_currentApy)
 
-async function _currentApy(l1BlockNumber: number | bigint) {
+export async function currentApy(l1BlockNumber: number | bigint) {
   assert (l1BlockNumber < 1000000000n, `${l1BlockNumber} should fit into number` )
 
   const l2blockNumber = await getFirstBlockForL1Block(l1BlockNumber)
@@ -172,8 +174,9 @@ export async function bond(blockNumber?: bigint) {
   return contracts.workerRegistration.read.bondAmount({ blockNumber });
 }
 
-export async function getBlockNumber() {
-  return Number(await l1Client.getBlockNumber());
+export async function getL1BlockNumber() {
+  const block = await publicClient.getBlock();
+  return Number.parseInt((block as any).l1BlockNumber, 16);
 }
 
 export async function lastRewardedBlock() {

packages/rewards-calculator/src/endpoints.ts

@@ -1,7 +1,7 @@
 import express from "express";
 import { epochStats } from "./reward";
 import { config, l1Client } from "./config";
-import { getBlockNumber, currentApy } from "./chain";
+import { getL1BlockNumber, currentApy } from "./chain";
 import { logger } from "./logger"
 
 const app = express();
@@ -106,22 +106,28 @@ app.get("/rewards/:fromBlock/:toBlock", async (req, res) => {
   await rewards(fromBlock, toBlock, res);
 });
 
-app.get("/currentApy/:atBlock", async (req, res) => {
-  const { atBlock } = req.params
-  let block
-  if (!isInteger(atBlock)) {
-    block = await getBlockNumber()
-  } else {
-    block = atBlock
+app.get("/currentApy/:atBlock?", async (req, res) => {
+  try {
+    const { atBlock } = req.params
+    let block: number
+    if (!atBlock || !isInteger(atBlock)) {
+      block = await getL1BlockNumber()
+      block = block - 1
+    } else {
+      block = Number(atBlock)
+    }
+    logger.log(`Block: ${block}`)
+    const apy = await currentApy(block);
+    res.jsonp({ block, apy})
+  } catch (e: any) {
+    console.error(e);
+    res.status(500).send(e.message);
   }
-  logger.log(`Block: ${block}`)
-  const apy = await currentApy(Number(block));
-  res.jsonp({ block, apy})
 });
 
 
 app.get("/rewards/:lastNBlocks", async (req, res) => {
-  const lastBlock = await getBlockNumber();
+  const lastBlock = await getL1BlockNumber();
   const fromBlock = lastBlock - Number(req.params.lastNBlocks);
   await rewards(fromBlock.toString(), lastBlock.toString(), res);
 });

packages/rewards-calculator/src/rewardBot.ts

@@ -3,7 +3,7 @@ import {
   canCommit,
   commitRewards,
   epochLength,
-  getBlockNumber,
+  getL1BlockNumber,
   getFirstBlockForL1Block,
   getLatestCommitment,
   getRegistrations,
@@ -164,7 +164,7 @@ export class RewardBot {
       );
     }
 
-    const currentBlock = await getBlockNumber();
+    const currentBlock = await getL1BlockNumber();
     const lastConfirmedBlock = currentBlock - config.epochConfirmationBlocks;
     if (lastConfirmedBlock - _lastRewardedBlock < epochLen) {
       return { fromBlock: 0, toBlock: 0, epochLen, chunkType: 'even' };

packages/rewards-calculator/src/testRPC.ts

@@ -66,9 +66,5 @@ export async function currentApyTest(blockNumber?: number) {
   console.log(`Approve Ranges: ${JSON.stringify(await approveRanges())}`)
   console.log(`APY: ${await currentApy(21279057n)}`)
   console.log(`APY: ${await currentApy(21279057)}`)
-  console.log(`APY: ${await currentApy(21279057n)}`)
-  console.log(`APY: ${await currentApy(21279057)}`)
-  console.log(`APY: ${await currentApy(21279057n)}`)
-  console.log(`APY: ${await currentApy(21279057)}`)
 })().then(() => console.log('Done'))
 

packages/rewards-calculator/src/workers.ts

@@ -41,7 +41,7 @@ export class Workers {
   private workers: Record<string, Worker> = {};
   private bond = new Decimal(0);
   private nextDistributionStartBlockNumber = 0n;
-
+  
   baseApr = new Decimal(0);
   stakeFactor = new Decimal(0);
   rAPR = new Decimal(0);
@@ -167,7 +167,7 @@ export class Workers {
     const baseApr = await currentApy(
       this.nextDistributionStartBlockNumber,
     );
-    this.baseApr = new Decimal(baseApr.toString());
+    this.baseApr = bigIntToDecimal(baseApr);
 
     this.stakeFactor = this.calculateStakeFactor();
 
@@ -309,10 +309,7 @@ export class Workers {
       dayjs(this.clickhouseClient.from),
       "second",
     );
-    const apy = bigIntToDecimal(
-      await currentApy(this.nextDistributionStartBlockNumber),
-    );
-    return apy.mul(this.totalSupply()).mul(duration).div(YEAR).div(10_000);
+    return this.baseApr.mul(this.totalSupply()).mul(duration).div(YEAR).div(10_000);
   }
 
   private calculateStakeFactor() {