chore(root): Update biome to beta.5 (#2264)

gabrielmfern · gabrielmfern · commit 3e105702350a · 2025-06-04T11:00:59.000-03:00
diff --git a/apps/web/src/components/template.tsx b/apps/web/src/components/template.tsx
@@ -1,8 +1,8 @@
 'use client';
 
 import classNames from 'classnames';
-import Image from 'next/image';
 import type { ImageLoader } from 'next/image';
+import Image from 'next/image';
 import Link from 'next/link';
 import * as React from 'react';
 import { Heading } from './heading';
diff --git a/apps/web/src/utils/spam-assassin/parse-pointing-table-rows.ts b/apps/web/src/utils/spam-assassin/parse-pointing-table-rows.ts
@@ -27,7 +27,7 @@ export const parsePointingTableRows = (response: string) => {
   const responseFromTableStart = response.slice(
     tableStartMatch.index + tableStartMatch[0].length,
   );
-  let currentRow: Row | undefined = undefined;
+  let currentRow: Row | undefined;
   for (const line of responseFromTableStart.split(/\r\n|\n|\r/)) {
     if (line.trim().length === 0) break;
 
diff --git a/biome.json b/biome.json
@@ -1,5 +1,5 @@
 {
-  "$schema": "https://biomejs.dev/schemas/2.0.0-beta.1/schema.json",
+  "$schema": "https://biomejs.dev/schemas/2.0.0-beta.5/schema.json",
   "assist": {
     "actions": {
       "source": {
@@ -31,7 +31,6 @@
         "noCommaOperator": "error",
         "useNodejsImportProtocol": "error",
         "useAsConstAssertion": "error",
-        "useNumericLiterals": "error",
         "useEnumInitializers": "error",
         "useSelfClosingElements": "error",
         "useConst": "error",
@@ -54,6 +53,7 @@
         "noDelete": "off"
       },
       "a11y": {
+        "noStaticElementInteractions": "off",
         "noSvgWithoutTitle": "off",
         "noAutofocus": "off"
       },
@@ -67,6 +67,7 @@
         "useSortedClasses": "off"
       },
       "complexity": {
+        "useNumericLiterals": "error",
         "noUselessFragments": "off",
         "noForEach": "off"
       },
diff --git a/package.json b/package.json
@@ -15,7 +15,7 @@
     "test:watch": "turbo run test:watch"
   },
   "devDependencies": {
-    "@biomejs/biome": "2.0.0-beta.1",
+    "@biomejs/biome": "2.0.0-beta.5",
     "@changesets/cli": "2.29.2",
     "@types/node": "22.14.1",
     "@types/react": "19.0.1",
diff --git a/packages/preview-server/src/actions/email-validation/check-images.ts b/packages/preview-server/src/actions/email-validation/check-images.ts
@@ -103,7 +103,7 @@ export const checkImages = async (code: string, base: string) => {
             });
           }
 
-          let res: IncomingMessage | undefined = undefined;
+          let res: IncomingMessage | undefined;
           try {
             res = await quickFetch(url);
             const hasSucceeded =
@@ -132,7 +132,7 @@ export const checkImages = async (code: string, base: string) => {
             if (responseSizeBytes > 1_048_576 && result.status !== 'error') {
               result.status = 'warning';
             }
-          } catch (exception) {
+          } catch (_exception) {
             result.checks.push({
               type: 'fetch_attempt',
               passed: false,
@@ -142,7 +142,7 @@ export const checkImages = async (code: string, base: string) => {
             });
             result.status = 'error';
           }
-        } catch (exception) {
+        } catch (_exception) {
           result.checks.push({
             passed: false,
             type: 'syntax',
diff --git a/packages/preview-server/src/actions/email-validation/check-links.ts b/packages/preview-server/src/actions/email-validation/check-links.ts
@@ -68,7 +68,7 @@ export const checkLinks = async (code: string) => {
             });
           }
 
-          let res: IncomingMessage | undefined = undefined;
+          let res: IncomingMessage | undefined;
           try {
             res = await quickFetch(url);
             const hasSucceeded =
@@ -85,7 +85,7 @@ export const checkLinks = async (code: string) => {
                 ? 'warning'
                 : 'error';
             }
-          } catch (exception) {
+          } catch (_exception) {
             result.checks.push({
               type: 'fetch_attempt',
               passed: false,
@@ -95,7 +95,7 @@ export const checkLinks = async (code: string) => {
             });
             result.status = 'error';
           }
-        } catch (exception) {
+        } catch (_exception) {
           result.checks.push({
             passed: false,
             type: 'syntax',
diff --git a/packages/preview-server/src/actions/email-validation/quick-fetch.ts b/packages/preview-server/src/actions/email-validation/quick-fetch.ts
@@ -1,5 +1,5 @@
-import http from 'node:http';
 import type { IncomingMessage } from 'node:http';
+import http from 'node:http';
 import https from 'node:https';
 
 export const quickFetch = (url: URL) => {
diff --git a/packages/preview-server/src/app/preview/[...slug]/page.tsx b/packages/preview-server/src/app/preview/[...slug]/page.tsx
@@ -60,10 +60,9 @@ This is most likely not an issue with the preview server. Maybe there was a typo
 
   const serverEmailRenderingResult = await renderEmailByPath(emailPath);
 
-  let spamCheckingResult: SpamCheckingResult | undefined = undefined;
-  let lintingRows: LintingRow[] | undefined = undefined;
-  let compatibilityCheckingResults: CompatibilityCheckingResult[] | undefined =
-    undefined;
+  let spamCheckingResult: SpamCheckingResult | undefined;
+  let lintingRows: LintingRow[] | undefined;
+  let compatibilityCheckingResults: CompatibilityCheckingResult[] | undefined;
 
   if (isBuilding) {
     if ('error' in serverEmailRenderingResult) {
diff --git a/packages/preview-server/src/components/send.tsx b/packages/preview-server/src/components/send.tsx
@@ -31,7 +31,7 @@ export const Send = ({ markup }: { markup: string }) => {
       }
 
       toast.success('Email sent! Check your inbox.');
-    } catch (exception) {
+    } catch (_exception) {
       toast.error('Something went wrong. Please try again.');
     } finally {
       setIsSending(false);
diff --git a/packages/preview-server/src/components/toolbar/code-preview-line-link.tsx b/packages/preview-server/src/components/toolbar/code-preview-line-link.tsx
@@ -10,7 +10,6 @@ interface CodePreviewLineLinkProps {
 
 export const CodePreviewLineLink = ({
   line,
-  column,
   type,
 }: CodePreviewLineLinkProps) => {
   const searchParams = useSearchParams();
diff --git a/packages/preview-server/src/components/toolbar/results.tsx b/packages/preview-server/src/components/toolbar/results.tsx
@@ -7,6 +7,7 @@ export const Results = ({
 }: React.ComponentProps<'table'>) => {
   return (
     <table
+      {...props}
       className={cn(
         'group relative w-full border-collapse text-left text-slate-10 text-sm',
         className,
diff --git a/packages/preview-server/src/components/toolbar/use-cached-state.ts b/packages/preview-server/src/components/toolbar/use-cached-state.ts
@@ -1,13 +1,13 @@
 import { useSyncExternalStore } from 'react';
 
 export const useCachedState = <T>(key: string) => {
-  let value: T | undefined = undefined;
+  let value: T | undefined;
   if ('localStorage' in global) {
     const storedValue = global.localStorage.getItem(key);
     if (storedValue !== null && storedValue !== 'undefined') {
       try {
         value = JSON.parse(storedValue) as T;
-      } catch (exception) {
+      } catch (_exception) {
         console.warn(
           'Failed to load stored value for',
           key,
diff --git a/packages/preview-server/src/utils/caniemail/ast/get-object-variables.ts b/packages/preview-server/src/utils/caniemail/ast/get-object-variables.ts
@@ -1,5 +1,5 @@
-import traverse from '@babel/traverse';
 import type { Node } from '@babel/traverse';
+import traverse from '@babel/traverse';
 import type { AST } from '../../../actions/email-validation/check-compatibility';
 
 export interface Position {
diff --git a/packages/preview-server/src/utils/caniemail/tailwind/generate-tailwind-rules.ts b/packages/preview-server/src/utils/caniemail/tailwind/generate-tailwind-rules.ts
@@ -1,5 +1,5 @@
-import postcss from 'postcss';
 import type { Root, Rule } from 'postcss';
+import postcss from 'postcss';
 import evaluateTailwindFunctions from 'tailwindcss/lib/lib/evaluateTailwindFunctions';
 import { generateRules as rawGenerateRules } from 'tailwindcss/lib/lib/generateRules';
 import type { JitContext } from 'tailwindcss/lib/lib/setupContextUtils';
diff --git a/packages/preview-server/src/utils/caniemail/tailwind/get-tailwind-config.ts b/packages/preview-server/src/utils/caniemail/tailwind/get-tailwind-config.ts
@@ -1,7 +1,6 @@
-import fs from 'node:fs';
 import path from 'node:path';
-import traverse from '@babel/traverse';
 import type { Node } from '@babel/traverse';
+import traverse from '@babel/traverse';
 import * as esbuild from 'esbuild';
 import type { Config as TailwindOriginalConfig } from 'tailwindcss';
 import type { AST } from '../../../actions/email-validation/check-compatibility';
@@ -24,14 +23,6 @@ export type TailwindConfig = Pick<
   | 'plugins'
 >;
 
-const getFirstExistingFilepath = (filePaths: string[]) => {
-  for (const filePath of filePaths) {
-    if (fs.existsSync(filePath)) {
-      return filePath;
-    }
-  }
-};
-
 type ImportDeclaration = Node & { type: 'ImportDeclaration' };
 
 export const getTailwindConfig = async (
diff --git a/packages/preview-server/src/utils/result.ts b/packages/preview-server/src/utils/result.ts
@@ -32,7 +32,7 @@ export function mapResult<T, E, B>(
 
 export function ok<T, E>(value: NoInfer<T>): Ok<T, E>;
 // biome-ignore lint/suspicious/noConfusingVoidType: This is required for void return types on functions that can still error
-export function ok<T extends void = void, E = never>(value: void): Ok<void, E>;
+export function ok<_T extends void = void, E = never>(value: void): Ok<void, E>;
 export function ok<T, E>(value: NoInfer<T>): Ok<T, E> {
   return {
     value,
@@ -41,7 +41,7 @@ export function ok<T, E>(value: NoInfer<T>): Ok<T, E> {
 
 export function err<T, E>(error: NoInfer<E>): Error<T, E>;
 // biome-ignore lint/suspicious/noConfusingVoidType: This is required for void return types on functions that can still error
-export function err<T, E extends void = void>(error: void): Error<T, void>;
+export function err<T, _E extends void = void>(error: void): Error<T, void>;
 export function err<T, E>(error: NoInfer<E>): Error<T, E> {
   return {
     error,
diff --git a/packages/preview-server/src/utils/testing/js-email-test.js b/packages/preview-server/src/utils/testing/js-email-test.js
@@ -1,5 +1,5 @@
 // A simple JavaScript email component
-const React = require('react');
+const _React = require('react');
 const { Html, Button } = require('@react-email/components');
 
 function Email() {
diff --git a/packages/preview-server/src/utils/testing/request-response-email.tsx b/packages/preview-server/src/utils/testing/request-response-email.tsx
@@ -1,6 +1,6 @@
 /* eslint-disable @typescript-eslint/no-unused-vars */
-const req = new Request('https://react.email');
-const res = new Response('{}');
+const _req = new Request('https://react.email');
+const _res = new Response('{}');
 
 const Email = () => {
   return <div />;
diff --git a/packages/react-email/src/utils/get-preview-server-location.ts b/packages/react-email/src/utils/get-preview-server-location.ts
@@ -12,7 +12,7 @@ export const getPreviewServerLocation = async () => {
       url.parse(usersProject.esmResolve('@react-email/preview-server'), true)
         .path!,
     );
-  } catch (exception) {
+  } catch (_exception) {
     const response = await prompts({
       type: 'confirm',
       name: 'installPreviewServer',
diff --git a/packages/render/src/shared/options.ts b/packages/render/src/shared/options.ts
@@ -1,5 +1,4 @@
 import type { HtmlToTextOptions } from 'html-to-text';
-// biome-ignore lint/correctness/noUnusedImports: this is used in the deprecated doc
 import type { pretty } from './utils/pretty';
 
 export type Options = {
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ export const Send = ({ markup }: { markup: string }) => {`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`toast.success('Email sent! Check your inbox.');`
`34`		`- } catch (exception) {`
	`34`	`+ } catch (_exception) {`
`35`	`35`	`toast.error('Something went wrong. Please try again.');`
`36`	`36`	`} finally {`
`37`	`37`	`setIsSending(false);`