Feature/edit findings audit

[iTzGooDLife]

Descripción

Se realizaron los siguientes cambios:

• Se agrega la funcionalidad de editar findings dentro de las auditorias
• Se realiza una pequeña modificación de CVSSCalculator para compatibilizar con la feature mencionada antes, ahora no realizará el parseo si el vector está vacio (Se entiende por vacio si no tiene más de un elemento luego del '/', tipo: 'CVSS:3.1/').
• Se agrega un Check Icon al sidebar en su modalidad desplegado, para marcar si el finding está finalizado, este estado se maneja dentro del finding, en el switch con el label "Completed".
• Se agregan un Contexto para los Outlets dentro del componente padre "AuditRoot", esto para poder pasar estados/variables necesarias a componentes hijos. Además, se movió un poco de lógica desde el Sidebar hacia el AuditRoot.

OBSERVACIONES:

• No se agregaron los Custom Fields, se propone que quede como trabajo futuro.
• Hay un "TODO" pendiente, el cual es que si ocurren ciertos cambios dentro del componente de "general" de audits, entonces se debe reflejar en los findings, como el language, title, etc. No se abordó dentro de este PR debido a que afecta a muchos componentes extras.

Motivación y Contexto

Se finaliza la historia de usuario 11.

¿Cómo ha sido probado?

Pasos para comprobar:

• Editar findings con campos vacios o llenos y observar como los componentes cambian a sus respectivos valores
• Realizar recomendaciones de CVSS y CWE
• Comprobar el funcionamiento de la calculadora CVSS
• Actualizar el switch de un finding para observar como cambia el sidebar.

Capturas de pantalla (si es apropiado):

Tipos de cambios

• Bugfix (cambio que no interrumpe el funcionamiento y que soluciona un problema)
• New feature (cambio que no interrumpe el funcionamiento y que añade funcionalidad)
• Breaking change (corrección o funcionalidad que podría causar que la funcionalidad existente cambie)

Lista de verificación:

• Mi código sigue el estilo de código de este proyecto.
• Mi cambio requiere una modificación en la documentación.
• He actualizado la documentación en consecuencia.
• Requiere nuevos tests.

Summary by CodeRabbit

• Nuevas Funciones

  • Se ha añadido un nuevo componente DefinitionTab para editar hallazgos en un contexto de auditoría.
  • Se ha introducido un nuevo componente DetailsTab para la edición de hallazgos.
  • Se ha creado un nuevo componente ProofsTab para manejar información relacionada con pruebas en hallazgos de auditoría.
  • Se ha implementado un nuevo hook useAuditContext para acceder fácilmente al contexto de auditoría.
  • Se han añadido nuevas funciones para obtener, actualizar y eliminar hallazgos.
  • Se ha mejorado el componente AuditRoot para gestionar datos de auditoría y proporcionar contexto a los componentes secundarios.

• Mejoras de Usabilidad

  • Se ha añadido un icono visual para representar el estado de los hallazgos en el componente AuditSidebar.
  • Se ha mejorado el manejo de errores al eliminar hallazgos, proporcionando mensajes más claros al usuario.

• Correcciones de Errores

  • Se ha mejorado la gestión del estado de carga en varios componentes, asegurando una mejor experiencia de usuario.

• Refactorización

  • Se ha refactorizado la lógica de manejo de vulnerabilidades en el componente Add para mejorar la legibilidad y el manejo de errores.
  • Se ha simplificado la gestión de opciones de idioma en el componente General.

[iTzGooDLife]

Nada funcional, solo comentar que el contraste es fuerte aquí: Creo que sería más cómo a la vista algo así:

Efectivamente, queda mejor así, podrías aplicar directamente esos cambios. De igual manera, se implementó de la misma manera en que se implementó en añadir y editar vulnerabilidades (además de los custom fields y custom sections), considero que se deberia cambiar ahí también por temas de uniformidad.

[massi-ponce]

LGTM

[massi-ponce]

LGTM x2

[coderabbitai]

Actionable comments posted: 4

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Reviewing files that changed from the base of the PR and between e007929 and efdb0e1.

📒 Files selected for processing (2)

• frontend/src/routes/audits/edit/AuditRoot.tsx (6 hunks)
• frontend/src/routes/audits/edit/findings/edit/edit.tsx (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• frontend/src/routes/audits/edit/AuditRoot.tsx

🔇 Additional comments (2)

frontend/src/routes/audits/edit/findings/edit/edit.tsx (2)

236-262: Comentario omitido: Manejo de errores insuficiente

---

444-457: Comentario omitido: Problemas de accesibilidad

[Icksir]

Ahora funciona de pana 🥳

[jllanosg]

Hay un issue que documente el trabajo faltante del TODO que se menciona en el PR?

[iTzGooDLife]

Hay un issue que documente el trabajo faltante del TODO que se menciona en el PR?

Nop, tengo pensado hacerlos una vez se haga el merge, de igual manera en un rato más los puedo hacer 👍

[jllanosg]

LGTM! @caverav hacer el merge porfas