| name | verificar-arquitectura |
| description | Verifica que el código respete las reglas de Arquitectura Limpia del proyecto — que domain/ no importe framework, que los controladores no tengan lógica de negocio y que no se expongan entidades en la API. Úsala antes de abrir un Pull Request, al revisar código de otro, o cuando se agregue una clase nueva al backend. |
Verificar Arquitectura Limpia
Revisa el código del backend contra las reglas no negociables de AguaVigía CTG y reporta violaciones
con ubicación exacta y cómo corregirlas.
Las reglas
1. domain/ es Java puro — CERO framework
Ninguna clase bajo backend/src/main/java/com/aguavigia/ctg/domain/ puede importar:
org.springframework.*com.mongodb.* · org.bson.*jakarta.persistence.* · jakarta.validation.*com.fasterxml.jackson.*lombok.*
Es la regla más importante del proyecto. Si se rompe, la build falla por ArchUnit.
2. application/ no conoce tecnología
Puede importar domain.* y anotaciones mínimas de Spring para inyección. No puede importar
com.mongodb, org.bson, ni clases del paquete infrastructure.
3. Los controladores no tienen lógica de negocio
Un método de @RestController debe: recibir un DTO → llamar un caso de uso → devolver un DTO.
Señales de alarma: if con reglas de negocio, cálculos, bucles sobre entidades, llamadas directas a
repositorios.
4. No se exponen entidades de dominio en la API
Ningún método de controlador devuelve ni recibe una clase de domain.model. Siempre DTOs.
5. Los objetos de valor validan en el constructor
Todo record en domain/vo/ valida sus invariantes al construirse (una Coordenada fuera de rango
o una VentanaTiempo que termina antes de empezar deben lanzar excepción).
Cómo ejecutar la verificación
-
Busca violaciones de la regla 1 (la crítica):
Grep: pattern="^import (org\.springframework|com\.mongodb|org\.bson|jakarta|lombok)"
path="backend/src/main/java/com/aguavigia/ctg/domain"
output_mode="content"
Cualquier resultado es una violación crítica.
-
Busca violaciones de la regla 2:
Grep: pattern="^import com\.aguavigia\.ctg\.infrastructure"
path="backend/src/main/java/com/aguavigia/ctg/application"
-
Revisa los controladores (regla 3 y 4): lee cada archivo en api/rest/ y verifica que los
métodos solo deleguen.
-
Revisa los value objects (regla 5): lee domain/vo/ y confirma que cada record valide.
-
Si existe el test de ArchUnit, ejecútalo:
./mvnw test -Dtest=ArquitecturaTest
Formato del reporte
Para cada violación:
- Archivo y línea (
ruta:línea)
- Regla rota (número y nombre)
- Por qué importa — la consecuencia real, no la regla repetida
- Cómo se corrige — la acción concreta
Si no hay violaciones, dilo claramente y no inventes hallazgos menores para llenar el reporte.
Lo que NO es tarea de esta skill
- Estilo, formato o nombres de variables
- Buscar bugs de lógica
- Sugerir refactorizaciones que no estén relacionadas con las capas
