chore(lint): resolve Biome lint errors and apply auto-fix formatting

Suppress `useNullishCoalescing` on the intentional `||` fallback for empty
string and `useNamingConvention` on the `__OPTIONAL_CATCH_ALL__*` keys which
are part of the library's public API contract. Also includes `biome check
--write` auto-fix for import ordering and formatting.

Author: sectsect

.vscode/tasks.json

@@ -2,18 +2,20 @@
   // See https://go.microsoft.com/fwlink/?LinkId=733558
   // for the documentation about the tasks.json format
   "version": "2.0.0",
-  "tasks": [{
-    "label": "Project wide type checking with TypeScript",
-    "type": "npm",
-    "script": "type-check",
-    "problemMatcher": ["$tsc"],
-    "group": {
-      "kind": "build",
-      "isDefault": true
-    },
-    "presentation": {
-      "clear": true,
-      "reveal": "never"
+  "tasks": [
+    {
+      "label": "Project wide type checking with TypeScript",
+      "type": "npm",
+      "script": "type-check",
+      "problemMatcher": ["$tsc"],
+      "group": {
+        "kind": "build",
+        "isDefault": true
+      },
+      "presentation": {
+        "clear": true,
+        "reveal": "never"
+      }
     }
-  }]
+  ]
 }

eslint.config.mjs

@@ -2,29 +2,28 @@
 /* eslint-disable no-underscore-dangle */
 /* eslint-disable import/no-anonymous-default-export */
 
+import { FlatCompat } from '@eslint/eslintrc';
 // ESLint core and compatibility utilities
 import js from '@eslint/js';
-import { FlatCompat } from '@eslint/eslintrc';
-// import { fixupPluginRules } from '@eslint/compat';
-import globals from 'globals';
-
-// ESLint plugins for various technologies and best practices
-import react from 'eslint-plugin-react';
-import reactHooks from 'eslint-plugin-react-hooks';
-import jsxA11Y from 'eslint-plugin-jsx-a11y';
+// import nextPlugin from '@next/eslint-plugin-next';
+import pluginQuery from '@tanstack/eslint-plugin-query';
 import typescriptEslint from '@typescript-eslint/eslint-plugin';
-// import tseslint from 'typescript-eslint';
-import unusedImports from 'eslint-plugin-unused-imports';
-import tailwindcss from 'eslint-plugin-tailwindcss';
-import tsdoc from 'eslint-plugin-tsdoc';
+import vitest from '@vitest/eslint-plugin';
+import deprecation from 'eslint-plugin-deprecation';
 import _import from 'eslint-plugin-import';
+import jsxA11Y from 'eslint-plugin-jsx-a11y';
 import prettier from 'eslint-plugin-prettier';
 import eslintPluginPrettierRecommended from 'eslint-plugin-prettier/recommended';
+// ESLint plugins for various technologies and best practices
+import react from 'eslint-plugin-react';
+import reactHooks from 'eslint-plugin-react-hooks';
+import tailwindcss from 'eslint-plugin-tailwindcss';
 import testingLibrary from 'eslint-plugin-testing-library';
-import vitest from '@vitest/eslint-plugin';
-import deprecation from 'eslint-plugin-deprecation';
-// import nextPlugin from '@next/eslint-plugin-next';
-import pluginQuery from '@tanstack/eslint-plugin-query';
+import tsdoc from 'eslint-plugin-tsdoc';
+// import tseslint from 'typescript-eslint';
+import unusedImports from 'eslint-plugin-unused-imports';
+// import { fixupPluginRules } from '@eslint/compat';
+import globals from 'globals';
 
 const compat = new FlatCompat({
   baseDirectory: import.meta.dirname, // Added in: v21.2.0, v20.11.0 @ https://nodejs.org/api/esm.html#importmetadirname

src/__tests__/index.test.ts

@@ -1,6 +1,6 @@
 import { renderHook } from '@testing-library/react';
-import { usePathname, useParams } from 'next/navigation';
-import { describe, expect, vi, beforeEach } from 'vitest';
+import { useParams, usePathname } from 'next/navigation';
+import { beforeEach, describe, expect, vi } from 'vitest';
 
 import usePlaceholderPath from '../index';
 
@@ -103,6 +103,7 @@ describe('usePlaceholderPath', () => {
   test('should handle optional catch-all segments with one value', () => {
     vi.mocked(usePathname).mockReturnValue('/shop/category');
     vi.mocked(useParams).mockReturnValue({
+      // biome-ignore lint/style/useNamingConvention: library internal prefix (see getPlaceholder in src/index.ts) is part of the public API contract for optional catch-all routes
       __OPTIONAL_CATCH_ALL__slug: ['category'],
     });
 
@@ -113,6 +114,7 @@ describe('usePlaceholderPath', () => {
   test('should handle optional catch-all segments with two values', () => {
     vi.mocked(usePathname).mockReturnValue('/shop/category/product');
     vi.mocked(useParams).mockReturnValue({
+      // biome-ignore lint/style/useNamingConvention: library internal prefix (see getPlaceholder in src/index.ts) is part of the public API contract for optional catch-all routes
       __OPTIONAL_CATCH_ALL__slug: ['category', 'product'],
     });
 
@@ -123,6 +125,7 @@ describe('usePlaceholderPath', () => {
   test('should handle optional catch-all segments with three values', () => {
     vi.mocked(usePathname).mockReturnValue('/shop/category/product/variant');
     vi.mocked(useParams).mockReturnValue({
+      // biome-ignore lint/style/useNamingConvention: library internal prefix (see getPlaceholder in src/index.ts) is part of the public API contract for optional catch-all routes
       __OPTIONAL_CATCH_ALL__slug: ['category', 'product', 'variant'],
     });
 

src/index.ts

@@ -1,4 +1,4 @@
-import { usePathname, useParams } from 'next/navigation';
+import { useParams, usePathname } from 'next/navigation';
 
 interface UsePlaceholderPathOptions {
   optionalCatchAllSegments?: string;
@@ -80,7 +80,8 @@ const usePlaceholderPath = (options: UsePlaceholderPathOptions = {}) => {
 
   // Handle top-level optional catch-all segments
   if (isTopLevelOptionalCatchAll) {
-    const catchAllSegment = optionalCatchAllSegments || 'slug'; // Use 'slug' as default if empty string
+    // biome-ignore lint/nursery/useNullishCoalescing: intentional `||` to treat empty string as missing and fall back to 'slug'
+    const catchAllSegment = optionalCatchAllSegments || 'slug';
     return `/${segments[0]}/[[...${catchAllSegment}]]`;
   }
 

vitest.setup.ts

@@ -1,5 +1,6 @@
 /* eslint-disable import/no-extraneous-dependencies */
 import '@testing-library/jest-dom/vitest';
+
 import { cleanup } from '@testing-library/react';
 import { afterEach } from 'vitest';
 import 'intersection-observer';