Upgrade packages and fix linting rules (#135)

* Upgrade packages

* Fix linting issues and reformat code

* fix linter and unit test warnings

Author: devksingh4

.eslintrc

@@ -0,0 +1,81 @@
+import { defineConfig, globalIgnores } from "eslint/config";
+import _import from "eslint-plugin-import";
+import prettier from "eslint-plugin-prettier";
+import { fixupPluginRules } from "@eslint/compat";
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+import js from "@eslint/js";
+import { FlatCompat } from "@eslint/eslintrc";
+import mantine from "eslint-config-mantine";
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const compat = new FlatCompat({
+  baseDirectory: __dirname,
+  recommendedConfig: js.configs.recommended,
+  allConfig: js.configs.all,
+});
+
+export default defineConfig([
+  globalIgnores(["**/*.d.ts", "**/vite.config.ts"]),
+  {
+    extends: [
+      ...compat.extends(
+        "plugin:prettier/recommended",
+        "plugin:@typescript-eslint/recommended",
+      ),
+      ...mantine,
+    ],
+
+    plugins: { import: fixupPluginRules(_import), prettier },
+
+    settings: {
+      "import/parsers": {
+        "@typescript-eslint/parser": [".ts", ".tsx", ".js", ".jsx"],
+      },
+      "import/resolver": {
+        typescript: {
+          alwaysTryTypes: true,
+          project: ["src/api/tsconfig.json", "src/ui/tsconfig.json"],
+        },
+      },
+    },
+
+    rules: {
+      "import/no-unresolved": "error",
+      "no-param-reassign": "off",
+      "import/extensions": [
+        "error",
+        "ignorePackages",
+        { js: "never", jsx: "never", ts: "never", tsx: "never" },
+      ],
+      "no-unused-vars": "off",
+      "max-classes-per-file": "off",
+      "func-names": "off",
+      "no-case-declarations": "off",
+      "@typescript-eslint/no-unused-vars": [
+        "warn",
+        {
+          argsIgnorePattern: "^_",
+          varsIgnorePattern: "^_",
+          caughtErrorsIgnorePattern: "^_",
+        },
+      ],
+
+      "@typescript-eslint/no-explicit-any": "warn",
+      "@typescript-eslint/explicit-module-boundary-types": "off",
+    },
+  },
+  {
+    files: ["**/*.test.ts", "**/*.testdata.ts"],
+    rules: { "@typescript-eslint/no-explicit-any": "off" },
+  },
+  {
+    files: ["src/ui/*", "src/ui/**/*"],
+    rules: {
+      "@typescript-eslint/no-explicit-any": "off",
+      "@typescript-eslint/no-unused-vars": "off",
+      "no-console": ["error", { allow: ["warn", "error"] }],
+    },
+  },
+]);

eslint.config.mjs

@@ -27,58 +27,58 @@
   },
   "dependencies": {},
   "devDependencies": {
-    "@eslint/compat": "^1.1.1",
-    "@playwright/test": "^1.49.1",
-    "@tsconfig/node22": "^22.0.0",
-    "@types/node": "^22.1.0",
+    "@eslint/compat": "^1.2.8",
+    "@eslint/eslintrc": "^3.3.1",
+    "@eslint/js": "^9.25.1",
+    "@playwright/test": "^1.52.0",
+    "@tsconfig/node22": "^22.0.1",
+    "@types/node": "^22.15.2",
     "@types/pluralize": "^0.0.33",
-    "@types/react": "^18.3.3",
-    "@types/react-dom": "^18.3.0",
-    "@types/supertest": "^6.0.2",
+    "@types/react": "^19.1.2",
+    "@types/react-dom": "^19.1.2",
+    "@types/supertest": "^6.0.3",
     "@typescript-eslint/eslint-plugin": "^8.0.1",
     "@typescript-eslint/parser": "^8.0.1",
     "@vitejs/plugin-react": "^4.3.1",
-    "@vitest/coverage-istanbul": "2.1.9",
-    "@vitest/ui": "^2.0.5",
+    "@vitest/coverage-istanbul": "3.1.2",
+    "@vitest/ui": "^3.1.2",
     "aws-sdk-client-mock": "^4.1.0",
     "concurrently": "^9.1.2",
     "cross-env": "^7.0.3",
-    "esbuild": "^0.23.0",
+    "esbuild": "^0.25.3",
     "esbuild-plugin-copy": "^2.1.1",
-    "eslint": "^8.57.0",
+    "eslint": "^9.25.1",
     "eslint-config-airbnb": "^19.0.4",
     "eslint-config-airbnb-typescript": "^18.0.0",
     "eslint-config-esnext": "^4.1.0",
-    "eslint-config-mantine": "^3.2.0",
-    "eslint-config-prettier": "^9.1.0",
-    "eslint-import-resolver-typescript": "^3.6.1",
+    "eslint-config-mantine": "^4.0.3",
+    "eslint-config-prettier": "^10.1.2",
+    "eslint-import-resolver-typescript": "^4.3.4",
     "eslint-plugin-import": "^2.29.1",
     "eslint-plugin-jsx-a11y": "^6.9.0",
-    "eslint-plugin-prettier": "^5.2.1",
-    "eslint-plugin-react": "^7.35.0",
-    "eslint-plugin-react-hooks": "^4.6.2",
+    "eslint-plugin-prettier": "^5.2.6",
+    "eslint-plugin-react": "^7.37.5",
+    "eslint-plugin-react-hooks": "^5.2.0",
     "husky": "^9.1.4",
     "identity-obj-proxy": "^3.0.0",
-    "jsdom": "^24.1.1",
+    "jsdom": "^26.1.0",
     "node-ical": "^0.20.1",
-    "postcss": "^8.4.41",
+    "postcss": "^8.5.3",
     "postcss-preset-mantine": "^1.17.0",
     "postcss-simple-vars": "^7.0.1",
-    "prettier": "^3.3.3",
+    "prettier": "^3.5.3",
     "prop-types": "^15.8.1",
     "request": "^2.88.2",
-    "storybook": "^8.2.8",
-    "storybook-dark-mode": "^4.0.2",
-    "stylelint": "^16.8.1",
-    "stylelint-config-standard-scss": "^13.1.0",
-    "supertest": "^7.0.0",
+    "stylelint": "^16.19.1",
+    "stylelint-config-standard-scss": "^14.0.0",
+    "supertest": "^7.1.0",
     "synp": "^1.9.14",
-    "tsx": "^4.16.5",
-    "typescript": "^5.5.4",
-    "typescript-eslint": "^8.0.1",
+    "tsx": "^4.19.3",
+    "typescript": "^5.8.3",
+    "typescript-eslint": "^8.31.0",
     "vite-tsconfig-paths": "^5.0.1",
-    "vitest": "^2.0.5",
-    "yarn-upgrade-all": "^0.7.4"
+    "vitest": "^3.1.2",
+    "yarn-upgrade-all": "^0.7.5"
   },
   "resolutions": {
     "pdfjs-dist": "^4.8.69"

package.json

@@ -1,7 +1,6 @@
 import esbuild from "esbuild";
 import { resolve } from "path";
-import { copy } from 'esbuild-plugin-copy'
-
+import { copy } from "esbuild-plugin-copy";
 
 const commonParams = {
   bundle: true,
@@ -16,9 +15,22 @@ const commonParams = {
   target: "es2022", // Target ES2022
   sourcemap: false,
   platform: "node",
-  external: ["aws-sdk", "moment-timezone", "passkit-generator", "fastify", "zod", "zod-openapi", "@fastify/swagger", "@fastify/swagger-ui", "argon2"],
+  external: [
+    "aws-sdk",
+    "moment-timezone",
+    "passkit-generator",
+    "fastify",
+    "zod",
+    "zod-openapi",
+    "@fastify/swagger",
+    "@fastify/swagger-ui",
+    "argon2",
+  ],
   alias: {
-    'moment-timezone': resolve(process.cwd(), '../../node_modules/moment-timezone/builds/moment-timezone-with-data-10-year-range.js')
+    "moment-timezone": resolve(
+      process.cwd(),
+      "../../node_modules/moment-timezone/builds/moment-timezone-with-data-10-year-range.js",
+    ),
   },
   banner: {
     js: `
@@ -33,22 +45,22 @@ const commonParams = {
   }, // Banner for compatibility with CommonJS
   plugins: [
     copy({
-      resolveFrom: 'cwd',
+      resolveFrom: "cwd",
       assets: {
-        from: ['../../node_modules/@fastify/swagger-ui/static/*'],
-        to: ['../../dist/lambda/static'],
+        from: ["../../node_modules/@fastify/swagger-ui/static/*"],
+        to: ["../../dist/lambda/static"],
       },
     }),
     copy({
-      resolveFrom: 'cwd',
+      resolveFrom: "cwd",
       assets: {
-        from: ['./public/*'],
-        to: ['../../dist/lambda/public'],
+        from: ["./public/*"],
+        to: ["../../dist/lambda/public"],
       },
     }),
   ],
   inject: [resolve(process.cwd(), "./zod-openapi-patch.js")],
-}
+};
 esbuild
   .build({
     ...commonParams,
@@ -62,7 +74,7 @@ esbuild
     process.exit(1);
   });
 
-  esbuild
+esbuild
   .build({
     ...commonParams,
     entryPoints: ["api/sqs/index.js", "api/sqs/driver.js"],

src/api/build.js

@@ -47,7 +47,7 @@ export function withRoles<T extends FastifyZodOpenApiSchema>(
     "x-disable-api-key-auth": disableApiKeyAuth,
     description:
       roles.length > 0
-        ? `${disableApiKeyAuth ? "API key authentication is not permitted for this route.\n\n" : ""}Requires one of the following roles: ${roles.join(", ")}.${schema.description ? "\n\n" + schema.description : ""}`
+        ? `${disableApiKeyAuth ? "API key authentication is not permitted for this route.\n\n" : ""}Requires one of the following roles: ${roles.join(", ")}.${schema.description ? `\n\n${schema.description}` : ""}`
         : "Requires valid authentication but no specific role.",
     ...schema,
   };