chore: replace eslint and prettier with biomejs

biome.jsonc

@@ -0,0 +1,52 @@
+{
+  "$schema": "./node_modules/@biomejs/biome/configuration_schema.json",
+  "vcs": {
+    "defaultBranch": "master"
+  },
+  "extends": [
+    "@chainsafe/biomejs-config",
+    "@chainsafe/biomejs-config/recommended",
+    "@chainsafe/biomejs-config/esm",
+    "@chainsafe/biomejs-config/erasable-syntax"
+  ],
+  "files": {
+    "include": ["src/**/*.ts", "test/**/*.ts"]
+  },
+  "linter": {
+    "rules": {
+      "style": {
+        // We use `+` operator for string concatenation a lot
+        "useTemplate": "off",
+        // We use to export types and object without differentiating
+        "useExportType": "off",
+        // We use to import types and object without differentiating
+        "useImportType": "off"
+      }
+    }
+  },
+  "overrides": [
+    // TODO: Due to enum change disabling this for now
+    // Will bring it back as we update the base config
+    {
+      "include": ["*"],
+      "linter": {
+        "rules": {
+          "style": {
+            "useNamingConvention": "off"
+          }
+        }
+      }
+    },
+    {
+      "include": ["src/utils/gaContext.ts", "src/github/octokit.ts"],
+      "linter": {
+        "rules": {
+          "style": {
+            // This file using `snake_case` a lot due to Github Api
+            "useNamingConvention": "off"
+          }
+        }
+      }
+    }
+  ]
+}

package.json

@@ -31,7 +31,8 @@
     "build:esm": "tsc -p tsconfig.build.esm.json && echo '{\"type\": \"module\"}' > ./lib/esm/package.json",
     "build:cjs": "tsc -p tsconfig.build.cjs.json && echo '{\"type\": \"commonjs\"}' > ./lib/cjs/package.json",
     "test:unit": "vitest run test/unit/**/*.test.ts",
-    "lint": "eslint --color src/ test/",
+    "lint": "biome check",
+    "lint:fix": "biome check --write",
     "prepublishOnly": "yarn build",
     "benchmark": "node --loader ts-node/esm ./src/cli/cli.ts 'test/perf/**/@(!(errors)).test.ts'",
     "writeDocs": "node --loader ts-node/esm scripts/writeOptionsMd.ts"
@@ -42,18 +43,13 @@
     "@types/yargs": "^17.0.33",
     "@types/debug": "^4.1.12",
     "dotenv": "^10.0.0",
-    "eslint": "^9.15.0",
-    "eslint-config-prettier": "^9.1.0",
-    "eslint-import-resolver-typescript": "^3.6.3",
-    "eslint-plugin-import": "^2.31.0",
-    "eslint-plugin-prettier": "^5.2.1",
-    "prettier": "^3.4.0",
     "rimraf": "^5.0.10",
     "ts-node": "^10.9.2",
     "typescript": "^5.8.2",
-    "typescript-eslint": "^8.16.0",
     "vitest": "^3.0.6",
-    "vitest-in-process-pool": "^2.0.0"
+    "vitest-in-process-pool": "^2.0.0",
+    "@biomejs/biome": "^1.9.4",
+    "@chainsafe/biomejs-config": "^0.1.1"
   },
   "dependencies": {
     "@actions/cache": "^4.0.0",

src/benchmark/benchmarkFn.ts

@@ -2,10 +2,10 @@ import fs from "node:fs";
 import path from "node:path";
 import {getCurrentSuite, setFn} from "@vitest/runner";
 import {createChainable} from "@vitest/runner/utils";
-import {store} from "./globalState.js";
 import {BenchApi, BenchmarkOpts, BenchmarkRunOptsWithFn, PartialBy} from "../types.js";
-import {runBenchFn} from "./runBenchmarkFn.js";
+import {store} from "./globalState.js";
 import {getBenchmarkOptionsWithDefaults} from "./options.js";
+import {runBenchFn} from "./runBenchmarkFn.js";
 
 export const bench: BenchApi = createBenchmarkFunction(function <T, T2>(
   this: Record<"skip" | "only", boolean | undefined>,
@@ -64,7 +64,6 @@ export const bench: BenchApi = createBenchmarkFunction(function <T, T2>(
   const cleanup = (): void => {
     store.removeOptions(task);
     // Clear up the assigned handler to clean the memory
-    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-expect-error
     setFn(task, null);
   };

src/benchmark/convergence/coefficientOfVariance.ts

@@ -1,11 +1,11 @@
 import Debug from "debug";
 import {BenchmarkOpts, ConvergenceCheckFn} from "../../types.js";
 import {
+  OutlierSensitivityEnum,
   calcMean,
   calcMedian,
   calcVariance,
   filterOutliers,
-  OutlierSensitivityEnum,
   sortData,
 } from "../../utils/math.js";
 
@@ -20,7 +20,7 @@ export function createCVConvergenceCriteria(
   const minSamples = Math.max(5, minRuns);
   const maxSamplesForCV = 1000;
 
-  return function canTerminate(runIdx: number, totalNs: bigint, runsNs: bigint[]): boolean {
+  return function canTerminate(runIdx: number, _totalNs: bigint, runsNs: bigint[]): boolean {
     const currentMs = Date.now();
     const elapsedMs = currentMs - startMs;
     const timeSinceLastCheck = currentMs - lastConvergenceSample;

src/benchmark/convergence/linearAverage.ts

@@ -12,7 +12,6 @@ export function createLinearConvergenceCriteria(
   let lastConvergenceSample = startMs;
   const sampleEveryMs = 100;
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   return function canTerminate(runIdx: number, totalNs: bigint, _runNs: bigint[]): boolean {
     const currentMs = Date.now();
     const elapsedMs = currentMs - startMs;

src/benchmark/format.ts

@@ -44,11 +44,13 @@ function getRatioRow(result: BenchmarkResult, prevResult: BenchmarkResult | null
 
   if (ratio > threshold) {
     return `\u001b[91m${str}\u001b[0m`; // red
-  } else if (ratio < 1 / threshold) {
+  }
+
+  if (ratio < 1 / threshold) {
     return `\u001b[92m${str}\u001b[0m`; // green
-  } else {
-    return str;
   }
+
+  return str;
 }
 
 function prettyTime(nanoSec: number): [number, string] {

src/benchmark/globalState.ts

@@ -1,6 +1,6 @@
-import Debug from "debug";
 import type {Suite, SuiteCollector, Task} from "@vitest/runner";
-import {BenchmarkResult, BenchmarkOpts, BenchmarkResults} from "../types.js";
+import Debug from "debug";
+import {BenchmarkOpts, BenchmarkResult, BenchmarkResults} from "../types.js";
 
 const debug = Debug("@chainsafe/benchmark/state");
 

src/benchmark/reporter.ts

@@ -1,8 +1,8 @@
-import {Task, Suite, File} from "@vitest/runner";
-import {color, consoleLog, symbols} from "../utils/output.js";
-import {store} from "./globalState.js";
+import {File, Suite, Task} from "@vitest/runner";
 import {Benchmark, BenchmarkOpts, BenchmarkResult} from "../types.js";
+import {color, consoleLog, symbols} from "../utils/output.js";
 import {formatResultRow} from "./format.js";
+import {store} from "./globalState.js";
 import {defaultBenchmarkOptions} from "./options.js";
 
 export class BenchmarkReporter {
@@ -129,7 +129,6 @@ export class BenchmarkReporter {
     }
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   onSuiteFinished(_suite: Suite): void {
     --this.indents;
 
@@ -138,7 +137,6 @@ export class BenchmarkReporter {
     }
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   onComplete(_files: File[]): void {
     consoleLog();
     this.indents += 2;

src/benchmark/runBenchmarkFn.ts

@@ -1,9 +1,9 @@
 import Debug from "debug";
-import {BenchmarkResult, BenchmarkOpts, Convergence, ConvergenceCheckFn, ConvergenceEnum} from "../types.js";
-import {calcSum, filterOutliers, OutlierSensitivityEnum} from "../utils/math.js";
-import {getBenchmarkOptionsWithDefaults} from "./options.js";
-import {createLinearConvergenceCriteria} from "./convergence/linearAverage.js";
+import {BenchmarkOpts, BenchmarkResult, Convergence, ConvergenceCheckFn, ConvergenceEnum} from "../types.js";
+import {OutlierSensitivityEnum, calcSum, filterOutliers} from "../utils/math.js";
 import {createCVConvergenceCriteria} from "./convergence/coefficientOfVariance.js";
+import {createLinearConvergenceCriteria} from "./convergence/linearAverage.js";
+import {getBenchmarkOptionsWithDefaults} from "./options.js";
 
 const debug = Debug("@chainsafe/benchmark/run");
 
@@ -121,14 +121,14 @@ Consider adjusting 'maxWarmUpMs' or 'maxWarmUpRuns' options orextend 'maxMs'
 if your function is very slow.
 `.trim()
       );
-    } else {
-      throw Error(
-        `
+    }
+
+    throw Error(
+      `
 No run was completed before 'maxMs' ${maxMs}. Consider extending the 'maxMs' time if
 either the before(), beforeEach() or fn() functions are too slow.
 `.trim()
-      );
-    }
+    );
   }
 
   let averageNs!: number;

src/benchmark/runner.ts

@@ -1,17 +1,17 @@
+import path from "node:path";
 import {
   File,
-  startTests,
   Suite,
   Task,
   VitestRunner,
   VitestRunnerConfig,
   VitestRunnerImportSource,
+  startTests,
 } from "@vitest/runner";
-import path from "node:path";
 import Debug from "debug";
 import {Benchmark, BenchmarkOpts, BenchmarkResults} from "../types.js";
-import {BenchmarkReporter} from "./reporter.js";
 import {store} from "./globalState.js";
+import {BenchmarkReporter} from "./reporter.js";
 
 const debug = Debug("@chainsafe/benchmark/runner");
 
@@ -63,8 +63,7 @@ export class BenchmarkRunner implements VitestRunner {
     }
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  async onAfterRunTask(task: Task): Promise<void> {
+  async onAfterRunTask(_task: Task): Promise<void> {
     // To help maintain consistent memory usage patterns
     // we trigger garbage collection manually
     if (this.triggerGC && global.gc) {
@@ -93,9 +92,9 @@ export class BenchmarkRunner implements VitestRunner {
     debug("starting tests %O", files);
     const res = await startTests(files, this);
 
-    const passed = res.filter((r) => r.result?.state == "pass");
-    const skipped = res.filter((r) => r.result?.state == "skip");
-    const failed = res.filter((r) => r.result?.state == "fail");
+    const passed = res.filter((r) => r.result?.state === "pass");
+    const skipped = res.filter((r) => r.result?.state === "skip");
+    const failed = res.filter((r) => r.result?.state === "fail");
 
     debug("finished tests. passed: %i, skipped: %i, failed: %i", passed.length, skipped.length, failed.length);