VIDSOL-211-chores adding paths aliases and config updates

- adding paths aliases
- removing eslint redundant rules
- adding type check on dev mode for both backend and frontend
- pre-commit husky now type-check, lint-check, run-prettier
- removed lint-staged totally unnecessary with husky

Author: johnny-quesada-developer

.husky/pre-commit

@@ -1,4 +1,7 @@
 #!/usr/bin/env sh
 . "$(dirname "$0")/_/husky.sh"
 
-yarn lint-staged
+# Exit on error and unset variables
+set -eu
+
+yarn --silent quality-check

.lintstagedrc

@@ -5,12 +5,12 @@
   "main": "index.js",
   "type": "module",
   "scripts": {
-    "dev": "tsx watch index.ts",
-    "start": "node --import tsx index.ts",
-    "debug": "node --inspect --watch --import tsx index.ts",
+    "dev": "nodemon --watch . --ext ts,tsx,json --ignore dist --exec \"sh -c 'yarn ts-check && node --import tsx index.ts'\"",
+    "start": "yarn ts-check && node --import tsx index.ts",
+    "debug": "nodemon --watch . --ext ts,tsx,json --ignore dist --exec \"sh -c 'yarn ts-check && node --inspect --import tsx index.ts'\"",
     "test": "NODE_OPTIONS=\"--experimental-vm-modules\" jest --maxWorkers=1 --coverage",
     "test:watch": "yarn test --watch",
-    "ts-check": "tsc -p tsconfig.json"
+    "ts-check": "tsc --noEmit -p tsconfig.json --pretty true"
   },
   "author": "",
   "license": "MIT",
@@ -41,6 +41,7 @@
     "@types/supertest": "^6.0.2",
     "@types/validator": "^13.15.2",
     "jest": "^29.7.0",
+    "nodemon": "^3.1.10",
     "supertest": "^7.0.0",
     "ts-jest": "^29.1.2"
   }

backend/package.json

@@ -19,5 +19,22 @@ module.exports = {
     'jsx-a11y/no-static-element-interactions': 'off',
     'jsx-a11y/click-events-have-key-events': 'off',
     'react/require-default-props': 'off',
+    
+    'jsdoc/require-jsdoc': 'off', // to keep code base consistency since const functions are not required to have jsdoc,
+
+    /*
+    * Functions and classes are hoisted, so they can be used before their declaration.
+    */
+    '@typescript-eslint/no-use-before-define': ['error', { 
+      functions: false,
+      classes: false,
+      variables: true,
+    }],
+
+    'no-redeclare': 'off', // duplicate rule, typescript already checks for this
+    '@typescript-eslint/no-shadow': 'off', // duplicate rule, typescript already checks for this
+    'consistent-return': 'off', // this rule goes against standard useEffect patterns,
+    'tailwindcss/classnames-order': 'off', // this rule forces the changes to be more disruptive than beneficial and prevent grouping classes logically
+    'no-dupe-class-members': 'off', // duplicate rule, typescript already checks for this
   },
 };

frontend/.eslintrc.cjs

@@ -5,16 +5,17 @@
   "version": "1.2.3",
   "type": "module",
   "scripts": {
-    "build": "vite build && yarn cp-build",
+    "build": "yarn  --silent ts-check && vite build && yarn cp-build",
     "cp-build": "mkdir -p ../backend/dist && cp -r dist ../backend",
-    "dev": "vite --host",
+    "dev": "concurrently 'yarn --silent ts-check:watch' 'vite --host'",
     "docs": "typedoc",
     "docs:watch": "typedoc --watch",
     "lint": "eslint src --ext ts,tsx --report-unused-disable-directives --max-warnings 0",
     "preview": "vite preview",
     "test": "vitest run --coverage",
     "test:watch": "vitest",
-    "ts-check": "tsc -p tsconfig.json"
+    "ts-check": "tsc -p tsconfig.json --pretty true",
+    "ts-check:watch": "yarn --silent ts-check --noEmit --watch --preserveWatchOutput"
   },
   "dependencies": {
     "@emotion/css": "^11.11.2",
@@ -64,6 +65,7 @@
     "@types/uuid": "^9.0.8",
     "@vitest/coverage-v8": "^1.3.0",
     "@vitest/ui": "^1.3",
-    "jsdom": "^22.1.0"
+    "jsdom": "^22.1.0",
+    "vite-plugin-checker": "^0.11.0"
   }
 }

frontend/package.json

@@ -17,7 +17,19 @@
     "strict": true,
     "noUnusedLocals": false,
     "noUnusedParameters": false,
-    "noFallthroughCasesInSwitch": true
+    "noFallthroughCasesInSwitch": true,
+    "paths": {
+      "@api/*": ["./src/api/*"],
+      "@components/*": ["./src/components/*"],
+      "@Context/*": ["./src/Context/*"],
+      "@hooks/*": ["./src/hooks/*"],
+      "@locales/*": ["./src/locales/*"],
+      "@pages/*": ["./src/pages/*"],
+      "@tests/*": ["./src/tests/*"],
+      "@app-types/*": ["./src/types/*"],
+      "@utils/*": ["./src/utils/*"],
+      "@test/*": ["./src/test/*"]
+    }
   },
   "include": [
     "./src",

frontend/tsconfig.json

@@ -4,6 +4,7 @@ import type { UserConfig as VitestUserConfigInterface } from 'vitest/config';
 import { defineConfig as defineVitestConfig } from 'vitest/config';
 import react from '@vitejs/plugin-react';
 import replace from '@rollup/plugin-replace';
+import checker from 'vite-plugin-checker';
 
 const vitestConfig: VitestUserConfigInterface = defineVitestConfig({
   test: {
@@ -43,6 +44,24 @@ export default defineConfig(({ mode }) => {
         'process.env.CI': process.env.CI,
         preventAssignment: true,
       }),
+      checker({
+        typescript: true,
+        terminal: true,
+      }),
     ],
+    resolve: {
+      alias: {
+        '@api': '/src/api',
+        '@components': '/src/components',
+        '@Context': '/src/Context',
+        '@hooks': '/src/hooks',
+        '@locales': '/src/locales',
+        '@pages': '/src/pages',
+        '@tests': '/src/tests',
+        '@app-types': '/src/types',
+        '@utils': '/src/utils',
+        '@test': '/src/test',
+      },
+    },
   });
 });

frontend/vite.config.ts

@@ -13,15 +13,18 @@
     "npm": "9.x"
   },
   "scripts": {
+    "quality-check": "yarn --silent ts-check && yarn --silent lint:fix",
     "build": "yarn workspace frontend build",
     "deploy-vcr": "vcr deploy",
-    "dev": "yarn && concurrently 'yarn workspace frontend dev' 'yarn workspace backend dev'",
+    "dev:frontend": "yarn workspace frontend dev",
+    "dev:backend": "yarn workspace backend dev",
+    "dev": "yarn && concurrently 'yarn dev:frontend' 'yarn dev:backend'",
     "docs": "yarn workspace frontend docs",
     "docs:watch": "yarn workspace frontend docs:watch",
     "lint": "ESLINT_USE_FLAT_CONFIG=false yarn eslint . --ext .ts,.tsx",
     "lint:filenames": "./scripts/lintFileNames.sh",
-    "lint:fix": "yarn prettier --write . && yarn lint --fix",
-    "postinstall": "husky",
+    "lint:fix": "prettier --log-level warn --write \"**/*.{ts,tsx,js,json,md,yml,yaml}\" && yarn lint --fix",
+    "prepare": "husky install",
     "run-server": "yarn workspace backend start",
     "start": "yarn workspace frontend build && yarn workspace backend start",
     "start:backend": "yarn workspace backend dev",
@@ -33,10 +36,10 @@
     "test:frontend": "yarn workspace frontend test",
     "test:frontend:watch": "yarn workspace frontend test:watch",
     "test:integration": "yarn workspace integration-tests test",
-    "ts-check": "yarn ts-check:backend && yarn ts-check:frontend",
+    "ts-check": "yarn --silent ts-check:backend && yarn --silent ts-check:frontend",
     "test:integrationUpdateScreenshots": "yarn workspace integration-tests updateScreenshots",
-    "ts-check:backend": "yarn workspace backend ts-check",
-    "ts-check:frontend": "yarn workspace frontend ts-check"
+    "ts-check:backend": "yarn --silent workspace backend ts-check",
+    "ts-check:frontend": "yarn --silent workspace frontend ts-check"
   },
   "repository": {
     "type": "git",
@@ -68,7 +71,6 @@
     "eslint-plugin-tailwindcss": "^3.18.0",
     "husky": "^9.0.11",
     "license-checker": "^25.0.1",
-    "lint-staged": "^15.2.2",
     "concurrently": "^9.1.2",
     "prettier": "^3.2.5",
     "typescript": "^5.8.3"