chore(BE)(lint): fix linter errors in server 1

[Muatasim-Aswad]

• Due to the rejected promises being auto-handled by express in the current version, and due to some linter errors, instead of just satisfying the linter, the whole try catch wrappers are removed where not necessary.

• The fixes related to validation will be part of a following PR.

• To see the exact errors and warnings that are solved here please refer to: previous linter result

[Copilot]

Pull Request Overview

This PR addresses linter errors and warnings in the backend server by upgrading ESLint configuration, updating type annotations, improving error handling, and implementing TypeScript best practices.

Key changes include:

• Upgrade from legacy ESLint to modern ESLint 9 with TypeScript ESLint configuration
• Replace any types with proper TypeScript types and add comprehensive type safety
• Remove unnecessary try-catch blocks and improve async/await handling

Reviewed Changes

Copilot reviewed 29 out of 31 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
server/src/utils/database.ts	Adds proper typing for MongoDB utilities and transforms
server/src/services/*.ts	Improves type safety, error handling, and async operations
server/src/repositories/TraineesRepository.ts	Adds type safety with helper methods and proper error handling
server/src/controllers/*.ts	Removes unnecessary error handling and improves async patterns
server/src/models/*.ts	Updates utility functions with better TypeScript patterns
server/eslint.config.js	Upgrades to modern ESLint configuration
client/package.json & eslint.config.js	Updates client ESLint to modern configuration
.github/workflows/quality-checks.yml	Adds CI workflow for linting both server and client

Files not reviewed (1)

• server/package-lock.json: Language not supported

Comments suppressed due to low confidence (2)

client/eslint.config.js:6

• The import { globalIgnores } from 'eslint/config' does not exist in ESLint. This should likely be removed as globalIgnores is being used as a function on line 9, but this import path is invalid.

import { globalIgnores } from 'eslint/config';

server/src/repositories/TraineesRepository.ts:92

• [nitpick] The parameter name _id suggests it's intentionally unused, but the original parameter name id was more descriptive. Since this is a stub method that throws 'Not implemented', consider keeping the original parameter name id for clarity.

  deleteTrainee(_id: string): Promise<void> {

[Muatasim-Aswad]

if the scripts need to be linted too, ts config will change to include them in the project but exclude them from the build