Skip to content

Conversation

@muke78
Copy link
Owner

@muke78 muke78 commented Jul 1, 2025

✨ Refactor (optimization): Correct accommodation of user roles


🚀 Resumen general

Esta PR realiza una refactorización profunda y mejoras en la gestión de roles de usuario, optimizando la estructura del backend y simplificando el manejo de usuarios.
Se eliminaron dependencias no necesarias, se mejoró la configuración y se profesionalizaron los mensajes y logs.
Además, se modularizó el manejo de errores y se mejoró la conexión a la base de datos.


🗂️ Cambios principales por archivo

Nota: Por límite de espacio, sólo se muestran los primeros archivos y cambios más representativos.
Ver todos los detalles aquí

1. .husky/commit-msg y .husky/pre-commit

  • 🛡️ Mejor validación del mensaje de commit (añade soporte para tipo(scope): descripción).
  • 🚦 Pre-commit optimizado: se mejora el formateo con Prettier y se comenta la ejecución automática de tests.

2. package.json

  • 🧹 Limpieza: Se eliminaron paquetes no usados (chalk, commander).
  • ➕ Nuevo paquete agregado: standard para estandarizar el código.

3. server.js

  • 🔄 Refactor: Se eliminó la dependencia de commander y se centralizó la información del proyecto.
  • 🟢 Nueva ruta raíz / que devuelve la info y estado de la API.
  • 🛡️ Manejo global de errores con middleware dedicado.
  • ⚠️ Mejoras en los mensajes de error de puerto.

4. src/config/config.js

  • 🔄 Refactor total:
    • Se elimina el uso de inquirer y selección manual de base de datos.
    • Se centraliza la configuración de conexión MySQL usando variables de entorno estándar.
    • Mejor manejo de eventos y logs de conexión.

5. src/controllers/users/functions/*

  • 🔄 Simplificación:
    • Todos los métodos para usuarios (EliminarUsuario, EditarUsuario, InsertarUsuarios, etc.) ahora son funciones asíncronas limpias que devuelven resultados.
    • Se elimina lógica de respuesta HTTP (ahora se maneja en otro lugar).
    • Mejor separación de responsabilidades.

6. src/config/swaggerConfig.js

  • 📝 Corrige la versión de OpenAPI a 3.0.0.

🆕 Campos y mejoras destacadas

projectInfo en server.js:

const projectInfo = {
  name: "CRM Kinder Garden",
  description: "CRM para Gestión y Administración de una escuela",
  version: "1.0.0",
  authorName: "Erick Gonzalez",
  githubName: "https://github.com/muke78",
};

@muke78 muke78 self-assigned this Jul 1, 2025
@muke78 muke78 added the refactor Code improvement and modularity label Jul 1, 2025
@github-actions
Copy link

github-actions bot commented Jul 1, 2025

Dependency Review

The following issues were found:

  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ✅ 0 package(s) with unknown licenses.
  • ⚠️ 1 packages with OpenSSF Scorecard issues.

View full job summary

@muke78 muke78 merged commit a997ee9 into main Jul 3, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Code improvement and modularity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants