chore: reuse the shared biome config (#7763)

**Motivation**

Use consistent linting rules for all projects. 

**Description**

- Reuse the `@chainsafe/biome-config` package to have consistent 


**Steps to test or reproduce**

Run CI

---------

Co-authored-by: Cayman <caymannava@gmail.com>

Author: nazarhussain

biome.jsonc

@@ -1,32 +1,26 @@
 {
   "$schema": "./node_modules/@biomejs/biome/configuration_schema.json",
+  "extends": [
+    "@chainsafe/biomejs-config",
+    "@chainsafe/biomejs-config/recommended",
+    "@chainsafe/biomejs-config/nodejs",
+    "@chainsafe/biomejs-config/esm"
+  ],
   "vcs": {
-    "clientKind": "git",
-    "enabled": true,
-    "useIgnoreFile": true,
     "defaultBranch": "unstable"
   },
   "files": {
     "include": ["packages/*/src/**/*.ts", "packages/*/test/**/*.ts", "configs/**/*.ts", "./vitest.config.ts"]
   },
   "formatter": {
-    "enabled": true,
-    "formatWithErrors": true,
-    "useEditorconfig": true,
-    "lineWidth": 120,
-    "attributePosition": "auto",
-    "bracketSpacing": false,
     "ignore": ["**/lib", "**/.nyc_output", "./packages/*/spec-tests", "**/node_modules", "./packages/*/node_modules/**"]
   },
   "organizeImports": {
     "enabled": true
   },
   "linter": {
-    "enabled": true,
     "rules": {
-      "recommended": true,
       "correctness": {
-        "noUnusedVariables": "error",
         "useImportExtensions": {
           "level": "error",
           "options": {
@@ -37,11 +31,7 @@
               }
             }
           }
-        },
-        "useArrayLiterals": "error",
-        "noUndeclaredVariables": "error",
-        // TODO: We want this rule but it's not working properly with mono repos
-        "noUndeclaredDependencies": "off"
+        }
       },
       "performance": {
         // This rule should be enabled but with considerations and careful review
@@ -58,24 +48,16 @@
             "deniedGlobals": ["fetch"]
           }
         },
-        // We prefer to use `Math.pow` over `**` operator
-        "useExponentiationOperator": "off",
         // In some cases the enums are initialized with values of other enums
         "useLiteralEnumMembers": "off",
-        // We prefer to have multiple declarations lines
-        "useSingleVarDeclarator": "off",
         // 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",
-        // It's nice to use `Number` namespace but should be done in a separate PR
-        "useNumberNamespace": "off",
         // We prefer to auto-initialize enums
         "useEnumInitializers": "off",
-        // Namespaces are deprecated way to organize modules in TS
-        "noNamespace": "error",
         "useNamingConvention": {
           "level": "error",
           "options": {
@@ -101,12 +83,6 @@
                 },
                 "formats": ["camelCase", "snake_case", "PascalCase", "CONSTANT_CASE"]
               },
-              {
-                "selector": {
-                  "kind": "enum"
-                },
-                "formats": ["PascalCase"]
-              },
               {
                 "selector": {
                   "kind": "objectLiteralProperty"
@@ -167,77 +143,23 @@
         }
       },
       "suspicious": {
-        // `void` as type is useful in our case when used as generic constraint e.g. K extends number | void
-        "noConfusingVoidType": "off",
         // There is a lot of empty code blocks, should be enabled and clean up separately.
-        "noEmptyBlockStatements": "off",
-        "noConsoleLog": "error"
+        "noEmptyBlockStatements": "off"
       },
       "nursery": {
-        "useConsistentMemberAccessibility": {
-          "level": "error",
-          "options": {
-            "accessibility": "noPublic"
-          }
-        },
-        "noCommonJs": "error",
-        "noRestrictedImports": {
-          "level": "error",
-          "options": {
-            "paths": {
-              "child_process": "Please use node:child_process instead.",
-              "crypto": "Please use node:crypto instead.",
-              "fs": "Please use node:fs instead.",
-              "http": "Please use node:https instead.",
-              "net": "Please use node:net instead.",
-              "os": "Please use node:os instead.",
-              "path": "Please use node:path instead.",
-              "stream": "Please use node:stream instead.",
-              "util": "Please use node:util instead.",
-              "url": "Please use node:url instead.",
-              "worker_threads": "Please use node:worker_threads instead."
-            }
-          }
-        },
-        "noDuplicateElseIf": "error",
-        "noUselessEscapeInRegex": "error",
-        "noIrregularWhitespace": "error",
-        "noOctalEscape": "error",
         // Need to enable this rule with exception to anonymous functions
-        "useExplicitType": "off",
-        "useGuardForIn": "error",
-        "noDuplicateProperties": "error",
-        "useAtIndex": "error",
-        "useCollapsedIf": "error"
+        "useExplicitType": "off"
       }
     }
   },
-  "javascript": {
-    "formatter": {
-      "jsxQuoteStyle": "double",
-      "quoteProperties": "asNeeded",
-      "trailingCommas": "es5",
-      "semicolons": "always",
-      "arrowParentheses": "always",
-      "bracketSpacing": false,
-      "bracketSameLine": false,
-      "quoteStyle": "double",
-      "attributePosition": "auto",
-      "enabled": true
-    },
-    "linter": {
-      "enabled": true
-    },
-    "globals": ["BigInt"]
-  },
   "overrides": [
     // Code using console output
     {
       "include": ["packages/cli/src/", "packages/test-utils/src", "packages/flare/src"],
       "linter": {
         "rules": {
           "suspicious": {
-            "noConsoleLog": "off"
+            "noConsole": "off"
           }
         }
       }
@@ -257,7 +179,7 @@
             // During testing we use `any` type for quick assignments
             "noExplicitAny": "off",
             // Console logging is often used in tests
-            "noConsoleLog": "off"
+            "noConsole": "off"
           }
         }
       }

package.json

@@ -43,6 +43,7 @@
   "devDependencies": {
     "@actions/core": "^1.10.1",
     "@chainsafe/benchmark": "^1.2.3",
+    "@chainsafe/biomejs-config": "^0.1.2",
     "@biomejs/biome": "^1.9.4",
     "@types/node": "^20.12.8",
     "@vitest/browser": "^3.0.6",

packages/logger/src/browser.ts

@@ -71,6 +71,7 @@ class BrowserConsole extends Transport {
 
     if (val <= this.levels[this.level as LogLevel]) {
       // @ts-expect-error
+      // biome-ignore lint/suspicious/noConsole: We want the explicit usage of console
       console[mappedMethod](message);
     }
 

packages/prover/src/cli/index.ts

@@ -13,13 +13,14 @@ void prover
     if (msg?.includes("Not enough non-option arguments")) {
       // Show command help message when no command is provided
       yarg.showHelp();
-      // biome-ignore lint/suspicious/noConsoleLog: This code will run only in browser so console will be available.
+      // biome-ignore lint/suspicious/noConsole: This code will run only in browser so console will be available.
       console.log("\n");
     }
 
     const errorMessage =
       err !== undefined ? (err instanceof YargsError ? err.message : err.stack) : msg || "Unknown error";
 
+    // biome-ignore lint/suspicious/noConsole: We want to explicitly want to log error to console
     console.error(` ✖ ${errorMessage}\n`);
     process.exit(1);
   })

packages/utils/src/diff.ts

@@ -202,12 +202,12 @@ export function getDiffs(val1: Diffable, val2: Diffable, objectPath: string): Di
  */
 export function diff(val1: unknown, val2: unknown, outputValues = false, filename?: string): void {
   if (!isDiffable(val1)) {
-    // biome-ignore lint/suspicious/noConsoleLog: <explanation>
+    // biome-ignore lint/suspicious/noConsole: <explanation>
     console.log("val1 is not Diffable");
     return;
   }
   if (!isDiffable(val2)) {
-    // biome-ignore lint/suspicious/noConsoleLog: <explanation>
+    // biome-ignore lint/suspicious/noConsole: <explanation>
     console.log("val2 is not Diffable");
     return;
   }
@@ -227,7 +227,7 @@ export function diff(val1: unknown, val2: unknown, outputValues = false, filenam
     if (filename) {
       fs.writeFileSync(filename, output);
     } else {
-      // biome-ignore lint/suspicious/noConsoleLog: <explanation>
+      // biome-ignore lint/suspicious/noConsole: <explanation>
       console.log(output);
     }
   }

yarn.lock

@@ -473,6 +473,11 @@
     yaml "^2.7.0"
     yargs "^17.7.2"
 
+"@chainsafe/biomejs-config@^0.1.2":
+  version "0.1.2"
+  resolved "https://registry.yarnpkg.com/@chainsafe/biomejs-config/-/biomejs-config-0.1.2.tgz#e31c9a7f719a932ccf7b9b78bb5d8401a65587f7"
+  integrity sha512-WveZrCiFC+XxiUnzN2YH9pKxFOAUs3W0YxLDY8emktGR9U/Pa9pfQl4ZiZCD0uwRBQKZdWLKc0yoAtkVYqU9Jg==
+
 "@chainsafe/bls-hd-key@^0.3.0":
   version "0.3.0"
   resolved "https://registry.npmjs.org/@chainsafe/bls-hd-key/-/bls-hd-key-0.3.0.tgz"