chore(deps): update dependency @biomejs/biome to v2.5.1

biome.jsonc

@@ -1,6 +1,4 @@
 {
-  "$schema": "https://biomejs.dev/schemas/2.4.2/schema.json",
-
   // Code formatting and organization suggestions
   "assist": {
     "enabled": true,
@@ -33,92 +31,22 @@
       "react": "all"
     },
     "rules": {
-      "a11y": "error",
-      "complexity": "error",
-      "correctness": "error",
-      "nursery": "off",
-      "performance": "error",
-      "recommended": true,
-      "security": "error",
+      "preset": "all",
+      "correctness": {
+        // Not a Qwik project - this rule produces false positives on ordinary
+        // React closures.
+        "useQwikValidLexicalScope": "off"
+      },
       "style": {
-        "recommended": true,
-        // Too strict - hardcoded strings acceptable in this project
+        // Hardcoded UI strings are acceptable in this example app; no i18n layer.
         "noJsxLiterals": "off"
       },
-      "suspicious": "error"
-    }
-  },
-
-  // Override settings
-  "overrides": [
-    {
-      "includes": ["**/*.tsx", "**/*.jsx"],
-      "linter": {
-        "rules": {
-          "performance": {
-            // Next.js specific rule - not applicable to Vite/React projects
-            "noImgElement": "off"
-          },
-          "style": {
-            // Allow default exports in React components
-            "noDefaultExport": "off",
-            // Allow class components (converting to function components is a larger refactor)
-            "useReactFunctionComponents": "off"
-          },
-          "suspicious": {
-            // React specific rule - not applicable to Vite/React projects
-            "noReactSpecificProps": "off"
-          }
-        }
-      }
-    },
-    {
-      "includes": [
-        "**/*.spec.ts",
-        "**/*.spec.tsx",
-        "**/*.test.ts",
-        "**/*.test.tsx"
-      ],
-      "linter": {
-        "rules": {
-          "complexity": {
-            // Test files can have longer functions with multiple test cases
-            "noExcessiveLinesPerFunction": "off"
-          },
-          "style": {
-            // HTTP headers follow spec casing (Authorization, not authorization)
-            "useNamingConvention": "off"
-          },
-          "suspicious": {
-            // Pact test executeTest callbacks are inherently async
-            "useAwait": "off"
-          }
-        }
-      }
-    },
-    {
-      "includes": ["*.config.ts", "*.config.js"],
-      "linter": {
-        "rules": {
-          "style": {
-            // Config files conventionally use default exports
-            "noDefaultExport": "off"
-          }
-        }
-      }
-    },
-    {
-      "includes": ["**/*.ts", "**/*.tsx"],
-      "linter": {
-        "rules": {
-          "correctness": {
-            // TypeScript handles this
-            "noUnresolvedImports": "off",
-            // False positive (we don't use Qwik)
-            "useQwikValidLexicalScope": "off"
-          }
-        }
+      "suspicious": {
+        // This is a React project - className/htmlFor are the correct props.
+        // The rule targets non-React JSX (e.g. Solid, Preact) where it would
+        // rewrite them to the HTML-standard names.
+        "noReactSpecificProps": "off"
       }
     }
-  ]
+  }
 }

package.json

@@ -29,7 +29,7 @@
     "check:fix": "biome check --write --unsafe ."
   },
   "devDependencies": {
-    "@biomejs/biome": "2.4.16",
+    "@biomejs/biome": "2.5.1",
     "@pact-foundation/pact": "16.5.0",
     "@testing-library/jest-dom": "6.9.1",
     "@types/node": "25.9.4",

src/App.tsx

@@ -1,11 +1,11 @@
-import { useEffect, useId, useState } from "react";
+import { useCallback, useEffect, useId, useState } from "react";
 import { Link } from "react-router-dom";
 import "spectre.css/dist/spectre.min.css";
 import "spectre.css/dist/spectre-icons.min.css";
 import "spectre.css/dist/spectre-exp.min.css";
-import API from "./api.ts";
-import Heading from "./Heading.tsx";
-import Layout from "./Layout.tsx";
+import { api } from "./api.ts";
+import { Heading } from "./Heading.tsx";
+import { Layout } from "./Layout.tsx";
 import type { Product } from "./types/index.ts";
 
 interface ProductTableRowProps {
@@ -53,15 +53,16 @@ function ProductTable(props: ProductTableProps) {
   );
 }
 
-function App() {
+export function App() {
   const [loading, setLoading] = useState(true);
   const [searchText, setSearchText] = useState("");
   const [products, setProducts] = useState<Product[]>([]);
   const [visibleProducts, setVisibleProducts] = useState<Product[]>([]);
   const inputId = useId();
 
   useEffect(() => {
-    API.getAllProducts()
+    api
+      .getAllProducts()
       .then((r) => {
         setLoading(false);
         setProducts(r);
@@ -83,12 +84,19 @@ function App() {
       );
     };
 
-    setVisibleProducts(searchText ? findProducts(searchText) : products);
+    let visible = products;
+    if (searchText) {
+      visible = findProducts(searchText);
+    }
+    setVisibleProducts(visible);
   }, [searchText, products]);
 
-  const onSearchTextChange = (e: React.ChangeEvent<HTMLInputElement>) => {
-    setSearchText(e.target.value);
-  };
+  const onSearchTextChange = useCallback(
+    (e: React.ChangeEvent<HTMLInputElement>) => {
+      setSearchText(e.target.value);
+    },
+    [],
+  );
 
   return (
     <Layout>
@@ -105,6 +113,7 @@ function App() {
           onChange={onSearchTextChange}
         />
       </div>
+      {/* biome-ignore lint/style/noTernary: idiomatic JSX conditional rendering with an else branch */}
       {loading ? (
         <div className="loading loading-lg centered" />
       ) : (
@@ -113,5 +122,3 @@ function App() {
     </Layout>
   );
 }
-
-export default App;

src/ErrorBoundary.tsx

@@ -1,7 +1,7 @@
 import type { ReactNode } from "react";
 import React from "react";
-import Heading from "./Heading.tsx";
-import Layout from "./Layout.tsx";
+import { Heading } from "./Heading.tsx";
+import { Layout } from "./Layout.tsx";
 
 interface ErrorBoundaryProps {
   children: ReactNode;
@@ -11,7 +11,7 @@ interface ErrorBoundaryState {
   hasError: boolean;
 }
 
-export default class ErrorBoundary extends React.Component<
+export class ErrorBoundary extends React.Component<
   ErrorBoundaryProps,
   ErrorBoundaryState
 > {
@@ -27,6 +27,7 @@ export default class ErrorBoundary extends React.Component<
         <Layout>
           <Heading text="Sad times :(" href="/" />
           <div className="columns">
+            {/* biome-ignore lint/performance/noImgElement: Vite/React project — next/image is not applicable */}
             <img
               className="column col-6"
               src="/sad_panda.gif"

src/Heading.tsx

@@ -5,7 +5,7 @@ interface HeadingProps {
   text: string;
 }
 
-function Heading(props: HeadingProps) {
+export function Heading(props: HeadingProps) {
   return (
     <div>
       <h1>
@@ -23,5 +23,3 @@ function Heading(props: HeadingProps) {
     </div>
   );
 }
-
-export default Heading;

src/Layout.tsx

@@ -4,7 +4,7 @@ interface LayoutProps {
   children: ReactNode;
 }
 
-function Layout(props: LayoutProps) {
+export function Layout(props: LayoutProps) {
   return (
     <div className="container">
       <div className="columns">
@@ -13,5 +13,3 @@ function Layout(props: LayoutProps) {
     </div>
   );
 }
-
-export default Layout;