feat: migrates to eslint v9

Author: thedanchez

.eslintrc.json

@@ -18,7 +18,7 @@ jobs:
       - name: Setup Bun
         uses: oven-sh/setup-bun@v1
         with:
-          bun-version: 1.1.42
+          bun-version: 1.2.3
       - name: Install Dependencies
         run: bun install --frozen-lockfile
       - name: Type Check

.github/workflows/ci.yaml

@@ -0,0 +1,80 @@
+import js from "@eslint/js";
+import simpleImportSort from "eslint-plugin-simple-import-sort";
+import solid from "eslint-plugin-solid/configs/typescript";
+import globals from "globals";
+import tseslint from "typescript-eslint";
+
+export default tseslint.config(
+  {
+    // config with just ignores is the replacement for `.eslintignore`
+    ignores: ["**/build/**", "**/coverage/**", "**/dist/**", "**/node_modules/**"],
+  },
+  js.configs.recommended,
+  tseslint.configs.strict,
+  {
+    plugins: {
+      "simple-import-sort": simpleImportSort,
+    },
+    rules: {
+      "simple-import-sort/imports": "error",
+      "simple-import-sort/exports": "error",
+    },
+  },
+  {
+    files: ["**/*.ts", "**/*.tsx"],
+    languageOptions: {
+      parser: tseslint.parser,
+      parserOptions: {
+        sourceType: "module",
+        project: "./tsconfig.json",
+        ecmaFeatures: {
+          jsx: true,
+        },
+      },
+      globals: {
+        ...globals.browser,
+        ...globals.es2022,
+      },
+    },
+    plugins: solid.plugins,
+    rules: {
+      ...solid.rules,
+      "@typescript-eslint/no-non-null-assertion": "off",
+      "@typescript-eslint/no-namespace": "off",
+      "no-unused-vars": "off",
+      "@typescript-eslint/no-unused-vars": [
+        "error",
+        {
+          varsIgnorePattern: "^_",
+          argsIgnorePattern: "^_",
+          destructuredArrayIgnorePattern: "^_",
+        },
+      ],
+    },
+  },
+  {
+    files: ["server/**/*.ts"],
+    languageOptions: {
+      parser: tseslint.parser,
+      parserOptions: {
+        sourceType: "module",
+        project: "./tsconfig.json",
+      },
+      globals: {
+        Bun: true,
+      },
+    },
+    rules: {
+      "@typescript-eslint/no-non-null-assertion": "off",
+    },
+  },
+  {
+    files: ["__tests__/**/*.ts", "__tests__/**/*.tsx"],
+    languageOptions: {
+      globals: {
+        ...globals.node,
+        ...globals.vitest,
+      },
+    },
+  },
+);

bun.lock

@@ -9,12 +9,12 @@
     "dev": "vite",
     "format": "prettier . --check",
     "format:fix": "prettier . --write",
-    "lint": "eslint . --ext .ts,.tsx",
-    "lint:fix": "eslint . --ext .ts,.tsx --fix",
+    "lint": "eslint .",
+    "lint:fix": "eslint . --fix",
     "serve": "vite preview",
     "start": "vite",
     "start:msw": "VITE_MSW=true vite",
-    "start:server": "bun backend/server.ts",
+    "start:server": "bun server/index.ts",
     "test": "bun run test:ui && bun run test:server",
     "test:ui": "vitest run",
     "test:ui:cov": "vitest run --coverage",
@@ -25,29 +25,33 @@
   "devDependencies": {
     "@solidjs/testing-library": "^0.8.10",
     "@testing-library/jest-dom": "^6.6.3",
-    "@testing-library/user-event": "^14.5.2",
-    "@types/bun": "^1.1.16",
-    "@typescript-eslint/eslint-plugin": "^8.20.0",
-    "@typescript-eslint/parser": "^8.20.0",
-    "@vitest/coverage-istanbul": "^2.1.8",
-    "eslint": "^8.57.1",
+    "@testing-library/user-event": "^14.6.1",
+    "@types/bun": "^1.2.3",
+    "@vitest/coverage-istanbul": "^2.1.9",
+    "eslint": "^9.21.0",
     "eslint-plugin-simple-import-sort": "^12.1.1",
     "eslint-plugin-solid": "^0.14.5",
+    "globals": "^15.15.0",
+    "jiti": "^2.4.2",
     "jsdom": "^26.0.0",
-    "msw": "^2.7.0",
-    "prettier": "^3.4.2",
+    "msw": "^2.7.3",
+    "prettier": "^3.5.2",
     "typescript": "^5.7.3",
-    "vite": "^6.0.7",
-    "vite-plugin-solid": "^2.11.0",
-    "vitest": "^2.1.8"
+    "typescript-eslint": "^8.25.0",
+    "vite": "^6.2.0",
+    "vite-plugin-solid": "^2.11.2",
+    "vitest": "^2.1.9"
   },
   "dependencies": {
     "@elysiajs/eden": "^1.2.0",
     "@elysiajs/static": "^1.2.0",
-    "@solidjs/router": "^0.15.2",
-    "@tanstack/solid-query": "^5.64.1",
-    "elysia": "^1.2.10",
-    "solid-js": "^1.9.4"
+    "@solidjs/router": "^0.15.3",
+    "@tanstack/solid-query": "^5.66.9",
+    "elysia": "^1.2.22",
+    "solid-js": "^1.9.5"
+  },
+  "overrides": {
+    "cssstyle": "4.1.0"
   },
   "msw": {
     "workerDirectory": [

eslint.config.ts

@@ -8,7 +8,7 @@
  * - Please do NOT serve this file on production.
  */
 
-const PACKAGE_VERSION = '2.7.0'
+const PACKAGE_VERSION = '2.7.3'
 const INTEGRITY_CHECKSUM = '00729d72e3b82faf54ca8b9621dbb96f'
 const IS_MOCKED_RESPONSE = Symbol('isMockedResponse')
 const activeClientIds = new Set()

package.json

@@ -1,7 +1,7 @@
 import { treaty } from "@elysiajs/eden";
 import { describe, expect, it } from "vitest";
 
-import { app } from "../server";
+import { app } from "..";
 
 const api = treaty(app);
 

public/mockServiceWorker.js

@@ -1,6 +1,6 @@
 import { treaty } from "@elysiajs/eden";
 
-import type { Server } from "../backend/server";
+import type { Server } from "../server";
 
 const serverClient = treaty<Server>(window.location.origin);
 

backend/__tests__/server.test.ts renamed to server/__tests__/server.test.ts

@@ -30,5 +30,6 @@ if (import.meta.env.DEV && !(root instanceof HTMLElement)) {
 }
 
 enableMocking().then(() => {
+  // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
   render(() => <App />, root!);
 });

backend/api/index.ts renamed to server/api/index.ts

@@ -8,7 +8,7 @@ const TEST_EXCLUDES = [
   "src/index.tsx",
   "src/mocks",
   "public",
-  "backend",
+  "server",
 ];
 const COVERAGE_EXCLUDE = [...TEST_EXCLUDES, "**/*.test.{ts,tsx}"];