UPDATE swagger docs

[SergioNR]

No description provided.

[kilo-code-bot]

✅ No Issues Found

2 files reviewed | Confidence: 95% | Recommendation: Merge

Review Details

Files:

• server/controllers/authController.js (loginLocal function)
• swagger.yaml (login endpoint documentation)

Changes Analyzed:

• Removed redundant success field from API responses
• Improved error handling in auth controller using Express middleware
• Updated swagger documentation to match actual API format
• Added 403 response documentation for participant login restriction

Quality Assessment:
✅ Security: No security concerns
✅ Performance: Slight improvement (smaller responses)
✅ Backward Compatibility: Maintained (only removes redundant fields)
✅ Code Quality: Improves consistency and follows REST conventions
✅ Documentation: Swagger docs now accurately reflect API responses

Key Improvements:

• Eliminates redundant success field (HTTP status codes indicate success/failure)
• Better error handling via Express middleware delegation
• Follows REST API best practices
• Reduces response payload size

[kilo-code-bot]

✅ GOOD: Improved error handling

Using Express middleware (next(err)) instead of manually formatting error responses is a clean approach that follows framework conventions and allows for centralized error handling.

[kilo-code-bot]

✅ GOOD: Documentation accuracy

Removing the redundant success fields from swagger responses aligns with REST conventions where HTTP status codes convey success/failure. This makes the API contract cleaner and more professional.