refactor: remove unnecessary comments and improve code readability across multiple files

fix: ignore unused variable warnings in ESLint configuration

chore: add new line at the end of several EJS template files

refactor: simplify error handling functions in various controllers

refactor: remove redundant comments in repository and service files

style: standardize formatting in interfaces and repositories

Author: fox3000foxy

eslint.config.mjs

@@ -16,6 +16,7 @@ export default defineConfig([
   {
     rules: {
       "@typescript-eslint/no-explicit-any": "ignore",
+      "@typescript-eslint/no-unused-vars": "ignore",
     },
   },
   tseslint.configs.recommended,

mailTemplates/accountConfirmation.ejs

@@ -36,3 +36,4 @@
     </table>
 </body>
 </html>
+

mailTemplates/connectionNotification.ejs

@@ -36,3 +36,4 @@
     </table>
 </body>
 </html>
+

mailTemplates/passwordReset.ejs

@@ -44,3 +44,4 @@
     </table>
 </body>
 </html>
+

src/app.ts

@@ -36,7 +36,6 @@ server.setConfig(app => {
   app.use(express.static(path.join(__dirname, 'public')));
 });
 
-// 404 handler
 server.setErrorConfig(app => {
   app.use((req, res) => {
     res.status(404).json({ message: 'Not Found' });

src/controllers/AuthenticatorController.ts

@@ -9,7 +9,6 @@ import { IUserService } from '../services/UserService';
 import { genKey } from '../utils/GenKey';
 import { generateUserJwt } from '../utils/Jwt';
 
-// --- UTILS ---
 function handleError(res: Response, error: unknown, message: string, status = 500) {
   const msg = error instanceof Error ? error.message : String(error);
   res.status(status).send({ message, error: msg });

src/controllers/BuyOrderController.ts

@@ -6,7 +6,6 @@ import { IBuyOrderService } from '../services/BuyOrderService';
 import { IItemService } from '../services/ItemService';
 import { ILogService } from '../services/LogService';
 
-// --- UTILS ---
 function handleError(res: Response, error: unknown, message: string, status = 500) {
   const msg = error instanceof Error ? error.message : String(error);
   res.status(status).send({ message, error: msg });

src/controllers/GameController.ts

@@ -13,7 +13,6 @@ import { IUserService } from '../services/UserService';
 import { createGameBodySchema, gameIdParamSchema, updateGameBodySchema } from '../validators/GameValidator';
 const streamPipeline = promisify(pipeline);
 
-// --- UTILS ---
 function handleError(res: Response, error: unknown, message: string, status = 500) {
   const msg = error instanceof Error ? error.message : String(error);
   res.status(status).send({ message, error: msg });
@@ -57,7 +56,6 @@ export class Games {
     }
   }
 
-  // --- LISTING & SEARCH ---
   @httpGet('/')
   public async listGames(req: Request, res: Response) {
     try {
@@ -159,7 +157,6 @@ export class Games {
     }
   }
 
-  // --- CREATION & MODIFICATION ---
   @httpPost('/', LoggedCheck.middleware)
   public async createGame(req: AuthenticatedRequest, res: Response) {
     if (!(await validateOr400(createGameBodySchema, req.body, res))) {
@@ -209,7 +206,6 @@ export class Games {
     }
   }
 
-  // --- ACHAT ---
   @httpPost(':gameId/buy', LoggedCheck.middleware)
   public async buyGame(req: AuthenticatedRequest, res: Response) {
     const { gameId } = req.params;
@@ -255,7 +251,6 @@ export class Games {
     }
   }
 
-  // --- PROPRIÉTÉ ---
   @httpPost('/transfer-ownership/:gameId', LoggedCheck.middleware)
   public async transferOwnership(req: AuthenticatedRequest, res: Response) {
     const { gameId } = req.params;
@@ -359,7 +354,6 @@ export class Games {
       if (!link) return res.status(404).send({ message: 'No download link available' });
       const githubMatch = link.match(/^https:\/\/github.com\/([^/]+)\/([^/]+)(?:\.git)?$/i);
       if (githubMatch) {
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars
         const [_, owner, repo] = githubMatch;
         const zipUrl = `https://github.com/${owner}/${repo}/archive/refs/heads/main.zip`;
         return res.redirect(zipUrl);
@@ -373,7 +367,6 @@ export class Games {
       if (fileRes.body) {
         try {
           await streamPipeline(fileRes.body, res);
-          // eslint-disable-next-line @typescript-eslint/no-unused-vars
         } catch (err) {
           res.status(500).send({ message: 'Error streaming the file.' });
         }

src/controllers/GameGiftController.ts

@@ -7,7 +7,6 @@ import { IGameService } from '../services/GameService';
 import { ILogService } from '../services/LogService';
 import { IUserService } from '../services/UserService';
 
-// --- UTILS ---
 function handleError(res: Response, error: unknown, message: string, status = 500) {
   const msg = error instanceof Error ? error.message : String(error);
   res.status(status).send({ message, error: msg });
@@ -22,7 +21,6 @@ export class GameGifts {
     @inject('LogService') private logService: ILogService
   ) {}
 
-  // Helper pour créer des logs (signature uniforme)
   private async createLog(req: AuthenticatedRequest, action: string, tableName?: string, statusCode?: number, userId?: string) {
     try {
       await this.logService.createLog({