👁️ fix: update .gitignore for improved file management and add missing entries; refactor UsersController to use query function and simplify user data handling; enhance web route middleware typing

Author: pphatdev

.gitignore

@@ -1,10 +1,89 @@
-dist
-node_modules
+# Dependencies
+node_modules/
+
+# Build outputs
+dist/
+build/
+*.js.map
+*.d.ts.map
+
+# Environment & Configuration
 env.json
 .cache-*/
 .vscode
 
 storage/*
 !storage/example/
 src/data/database.json
-src/server/data/database.json
+
+# Database files
+src/data/*.db
+src/data/*.db-journal
+src/data/*.db-shm
+src/data/*.db-wal
+src/data/*.sqlite
+src/data/*.sqlite3
+
+# Storage & Uploads (keep example folder)
+storage/*
+!storage/example/
+!storage/.gitkeep
+uploads/
+temp/
+
+# Cache & Temporary files
+.cache/
+.cache-*/
+.temp/
+*.tmp
+*.log
+logs/
+*.pid
+*.seed
+
+# IDE & Editor files
+.vscode/
+.idea/
+*.swp
+*.swo
+*~
+.DS_Store
+Thumbs.db
+*.suo
+*.ntvs*
+*.njsproj
+*.sln
+*.sw?
+
+# Testing
+coverage/
+.nyc_output/
+*.lcov
+test-results/
+playwright-report/
+
+# OS files
+.DS_Store
+.DS_Store?
+._*
+.Spotlight-V100
+.Trashes
+ehthumbs.db
+Desktop.ini
+$RECYCLE.BIN/
+
+# TypeScript
+*.tsbuildinfo
+
+# Optional npm cache directory
+.npm
+
+# Optional eslint cache
+.eslintcache
+
+# Backup files
+*.backup
+*.bak
+*.old
+db-backup/
+

src/client/controller/users.ts

@@ -1,6 +1,6 @@
 import { NextFunction, Request, Response } from "express";
 import { Controller } from "./controller.js";
-import { getDbClient } from "../../server/utils/db.js";
+import { query } from "../../server/utils/db.js";
 
 export class UsersController extends Controller {
 
@@ -23,35 +23,21 @@ export class UsersController extends Controller {
                 return;
             }
 
-            const sql = getDbClient();
-            const usersResult = await sql`
+            const usersResult = await query<any>(`
                 SELECT
                     id, username, email, name, avatar, role,
                     is_active, last_login_at, created_at, updated_at
                 FROM users
                 ORDER BY created_at DESC
-            `;
-
-            const users = usersResult.map((userRow: any) => ({
-                id: userRow.id,
-                username: userRow.username,
-                email: userRow.email,
-                name: userRow.name,
-                avatar: userRow.avatar,
-                role: userRow.role,
-                is_active: userRow.is_active,
-                last_login_at: userRow.last_login_at,
-                created_at: userRow.created_at,
-                updated_at: userRow.updated_at
-            }));
+            `);
 
             // Render users page
             const usersData = {
                 ...Controller.defaultConfig,
                 title: `Users Management - ${Controller.defaultConfig.title}`,
                 page: 'users',
                 currentPath,
-                users,
+                users: usersResult,
                 usersError: ""
             };
 

src/client/routes/web.ts

@@ -1,4 +1,4 @@
-import express, { Request, Response } from 'express';
+import express, { Request, Response, NextFunction } from 'express';
 import path from 'path';
 import { fileURLToPath } from 'url';
 import { DashboardController } from '../controller/dashboard.js';
@@ -45,7 +45,7 @@ app.get('/upload/history', UploadHistoryController.get);
 app.get('/users', UsersController.get);
 
 // Catch-all 404 handler - render not-found page with default config
-app.use((req, res, next) => {
+app.use((req: Request, res: Response, next: NextFunction) => {
     // Skip API routes - let them be handled by API router
     if (req.path.startsWith('/api')) {
         return next();