feat: Prepare project for Render deployment

[Mosleh92]

User description

هدف از این Pull Request

این PR ساختار پروژه را به طور کامل تمیزکاری کرده و یک پایه اولیه و استاندارد برای توسعه ایجاد می‌کند. مشکلات قبلی مانند مخزن تو در تو و فایل‌های اضافه برطرف شده است.

تغییرات اصلی

• پاکسازی فایل‌های غیرضروری و ساختار پروژه.
• نصب وابستگی‌های اصلی مانند React Router و TailwindCSS.
• پیکربندی صحیح TailwindCSS و فعال‌سازی آن در پروژه.
• ایجاد ساختار پوشه‌بندی استاندارد (pages, components, layout).
• پیاده‌سازی کامپوننت‌های پایه Header, Footer و صفحه HomePage.
• راه‌اندازی مسیریابی (Routing) اولیه در App.jsx.

---

PR Type

Bug fix, Enhancement

---

Description

• Fix ESLint configuration and critical errors

• Enhance backend seed script with environment variables

• Resolve XSS handling and security middleware issues

• Clean up unused imports and variables across codebase

---

Changes diagram

flowchart LR
  A["ESLint Config"] --> B["Jest Environment"]
  C["Seed Script"] --> D["Environment Variables"]
  E["Security Middleware"] --> F["XSS Clean"]
  G["Code Cleanup"] --> H["Remove Unused Imports"]
  I["Controller Fixes"] --> J["Function Definitions"]

Loading

---

Changes walkthrough 📝

	Relevant files
Enhancement	1 files seed.js Add environment variables for superuser credentials            +7/-3
Bug fix	15 files security.js Fix XSS handling to use xss-clean middleware                          +7/-43    currencyTransaction.controller.js Add missing mongoose import and remove unused                        +2/-1      customerBanking.controller.js Add mongoose import and remove unused User                              +2/-1      receipt.controller.js Add mongoose import and fix duplicate key                                +2/-1      remittance.controller.js Remove unused imports and fix variable usage                          +12/-12  tenantSettings.controller.js Fix case declarations and remove unused imports                    +21/-20  InterBranchTransfer.js Fix method name and mark unused parameter                                +1/-1      Receipt.js Remove duplicate status field definition                                  +1/-1      accounting.js Fix import to use tenantAccess instead of authorize            +1/-1      health.js Fix unused variable in disk check                                                +2/-1      server.js Fix middleware imports and add xss-clean                                  +34/-30  CustomerService.js Add mongoose import and remove unused variable                      +2/-1      HoldManager.js Add missing mongoose import                                                            +1/-0      exchangeRateService.js Add ExchangeRate import and comment unused function            +11/-10  i18n.js Fix duplicate key in translations object                                  +4/-2
Miscellaneous	28 files auth-fixed.controller.js Remove unused imports and variables                                            +3/-3      auth.controller.js Clean up unused imports and commented functions                    +11/-11  debt.controller.js Remove unused User import                                                                +1/-1      interBranchTransfer.controller.js Remove unused User import                                                                +1/-1      p2p.controller.js Remove unused imports                                                                        +2/-2      payment.controller.js Remove unused User import                                                                +1/-1      transaction.controller.js Remove multiple unused imports                                                      +8/-8      auth-fixed.js Remove unused variable                                                                      +1/-1      validation.js Remove unused i18n import                                                                +1/-1      validators.js Remove unused query import                                                              +1/-1      Account.js Mark unused parameter in static method                                      +1/-1      ExchangeRate.js Remove unused variables in updateRate method                          +2/-2      P2PChat.js Mark unused parameters in methods                                                +2/-2      PaymentOrder.js Mark unused parameter in static method                                      +1/-1      VIPCustomer.js Mark unused parameters and variables                                          +4/-4      customerBanking.routes.js Remove unused i18n import                                                                +1/-1      customers.js Remove unused tenantAccess import                                                +1/-1      receipt.routes.js Remove unused i18n import                                                                +1/-1      P2PService.js Remove unused NotificationService import                                  +1/-1      PaymentService.js Remove unused imports                                                                        +3/-3      ReceiptService.js Remove unused User import                                                                +1/-1      RemittanceService.js Remove unused imports                                                                        +2/-2      SubscriptionService.js Remove unused Plan import                                                                +1/-1      TransactionWorkflowService.js Mark unused parameters in static method                                    +1/-1      planLimitService.js Remove unused Plan import                                                                +1/-1      settlementService.js Mark unused parameters and imports                                              +2/-2      socket-auth.js Remove unused Tenant import                                                            +1/-1      remittanceUtils.js Remove unused crypto import                                                            +1/-1
Formatting	1 files report.controller.js Convert exports to function declarations for consistency  +28/-19
Tests	8 files accounting.test.js Remove unused variables in test setup                                        +8/-8      customerValidation.test.js Mark unused user variable                                                                +2/-2      payment.integration.test.js Remove unused model imports                                                            +3/-3      tenant-isolation.integration.test.js Remove unused variables and imports                                            +9/-9      report.test.js Remove unused User import                                                                +1/-1      securityAdvanced.test.js Mark unused variables in test setup                                            +3/-3      securityIsolation.test.js Mark unused staff variable                                                              +1/-1      setup.js Remove unused redis import                                                              +1/-1
Error handling	1 files cache.js Fix empty catch block                                                                        +4/-1
Configuration changes	1 files .eslintrc.json Add Jest environment configuration                                              +2/-1

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

[Mosleh92]

هدف از این Pull Request

این PR ساختار پروژه را به طور کامل تمیزکاری کرده و یک پایه اولیه و استاندارد برای توسعه ایجاد می‌کند. مشکلات قبلی مانند مخزن تو در تو و فایل‌های اضافه برطرف شده است.

تغییرات اصلی

• پاکسازی فایل‌های غیرضروری و ساختار پروژه.
• نصب وابستگی‌های اصلی مانند React Router و TailwindCSS.
• پیکربندی صحیح TailwindCSS و فعال‌سازی آن در پروژه.
• ایجاد ساختار پوشه‌بندی استاندارد (pages, components, layout).
• پیاده‌سازی کامپوننت‌های پایه Header, Footer و صفحه HomePage.
• راه‌اندازی مسیریابی (Routing) اولیه در App.jsx.

[qodo-code-review]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 4 🔵🔵🔵🔵⚪

🧪 No relevant tests

🔒 Security concerns Sensitive information exposure: The seed script uses environment variables with hardcoded fallback values for superuser credentials (admin@exchangeplatform.com and admin123). If environment variables are not set in production, these weak default credentials will be used, creating a significant security vulnerability. Additionally, the XSS protection middleware was removed and replaced with xss-clean, but the implementation may be incomplete, potentially leaving the application vulnerable to XSS attacks.

⚡ Recommended focus areas for review Security Risk Environment variables for superuser credentials are used with fallback to hardcoded defaults. This could lead to production systems using weak default credentials if environment variables are not properly set. const superuserEmail = process.env.SUPERUSER_EMAIL || 'admin@exchangeplatform.com'; const superuserPassword = process.env.SUPERUSER_PASSWORD || 'admin123'; Incomplete Implementation XSS protection middleware was removed and replaced with a comment indicating xss-clean will handle it in server.js, but the implementation appears incomplete and may leave security gaps. // XSS protection will be handled by xss-clean middleware in server.js }; }; // Helper function to sanitize objects - REMOVED as xss-clean will be used as middleware /* const sanitizeObject = (obj) => { // ... (previous implementation) ... }; */ Import Inconsistency Multiple controller imports are commented out as unused but the server setup methods still reference them, which could cause runtime errors when those routes are accessed. // const exchangeAccountController = require('./controllers/exchangeAccount.controller'); // Unused // const interBranchTransferController = require('./controllers/interBranchTransfer.controller'); // Unused

[qodo-code-review]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Fix undefined property access The code attempts to access settings.migration.source.connection but the connection property was removed from the source object. This will cause a runtime error when the migration process starts. backend/src/controllers/tenantSettings.controller.js [387] -startMigrationProcess(settings.tenantId, sourceSystem, settings.migration.source.connection); +startMigrationProcess(settings.tenantId, sourceSystem, null); [To ensure code accuracy, apply this suggestion manually] Suggestion importance[1-10]: 9 __ Why: The suggestion correctly identifies that settings.migration.source.connection is accessed after being removed, which would cause a runtime error.	High
	Preserve original method name The method name was changed from generateTransferNumber to generateTransferId, but this could break existing code that calls the original method name. Verify that all callers have been updated accordingly. backend/src/models/InterBranchTransfer.js [181-185] -interBranchTransferSchema.statics.generateTransferId = function(_tenantId) { // tenantId marked as unused +interBranchTransferSchema.statics.generateTransferNumber = function(_tenantId) { // Keep original method name const timestamp = Date.now().toString().slice(-8); const random = Math.floor(Math.random() * 1000).toString().padStart(3, '0'); return `TFR${timestamp}${random}`; }; Apply / Chat Suggestion importance[1-10]: 7 __ Why: The suggestion correctly notes the method rename from generateTransferNumber to generateTransferId and raises a valid concern about breaking changes, which is a good practice for code review.	Medium
General	Enable error logging for debugging The empty catch block silently ignores all cache errors, making debugging difficult. Consider logging errors or at least providing a minimal error indication for monitoring purposes. backend/src/utils/cache.js [25-28] } catch (err) { // Errors in caching should not break the application flow. Log them if necessary. - // console.error('Cache set error:', err); // Optional: log the error + console.error('Cache set error:', err); // Log the error for debugging } [To ensure code accuracy, apply this suggestion manually] Suggestion importance[1-10]: 5 __ Why: The suggestion correctly points out that swallowing errors makes debugging harder; however, the PR already improved this by adding comments, so the suggestion's impact is moderate.	Low
More