Barrera de revisión de un módulo de implementación de Rumbo. Prepara el pedido de review para Codex, arregla sus hallazgos confirmados en ramas fix/ (DEC-0019) y, con la aprobación del equipo, cierra el módulo con el tag mod-N. Usar cuando alguien pida cerrar o revisar el módulo MN, o diga que Codex ya terminó su review.
Install
npx skillscat add franluciano04/vault-rumbo/cerrar-modulo Install via the SkillsCat registry.
SKILL.md
Cerrar un módulo
Proceso en 06-Desarrollo/Flujo de implementación.md (DEC-0014). El módulo MN se cierra con el tag mod-N. Hay dos fases, según exista o no 06-Desarrollo/Reviews/MN — review Codex.md.
Fase A: preparar la review (todavía no hay review de Codex)
node 06-Desarrollo/herramientas/tickets.mjs modulo MN. Si hay tickets sin terminar (incluidos los e2e del módulo), listarlos y parar.- Escribir
06-Desarrollo/Reviews/MN — pedido de review.md:- Encabezado
tipo: referencia,estado: formalizado,responsable: equipo. - Tickets del módulo, con enlace y trazabilidad (CU y RF).
- Merges de cada ticket: hash y título, tal como los da el script.
- Archivos tocados por el módulo: la unión de
git diff --stat <merge>^1 <merge>de cada merge. - Puntos que conviene mirar entre tickets: contratos entre backend, web y mobile; permisos; transacciones; migraciones.
- Encabezado
- Commit
review: pedido de review de MNy push amain. - Decirle a la persona, textual, qué pedirle a Codex (desde la raíz del repo):
Leé
06-Desarrollo/Review de módulo — rol de Codex.mdy revisá el módulo MN según06-Desarrollo/Reviews/MN — pedido de review.md.
Fase B: procesar la review (Codex ya escribió MN — review Codex.md)
- Leer los hallazgos. Verificar cada uno contra el código. Si son muchos, lanzar
reviewer-ticketconmodel: sonnetsolo para confirmarlos. Anotar el resultado en la sección «Triage de Claude» del mismo archivo: confirmado o descartado, con el motivo. - Cada hallazgo bloqueante o importante confirmado se arregla directamente, sin crear un ticket (DEC-0019):
- En una rama
fix/MN-<hallazgo>desdemain, con los pasos de/implementar-ticket: test que reproduce la falla (test-writer), arreglo (implementadorde la capa, en modo fix), review con un modelo distinto, checks completos y PR. - Si toca auth, permisos, dinero, estados o el proceso diario, la review la hace
reviewer-seguridady el PR necesita la aprobación de otro integrante. - Se pueden agrupar varios hallazgos chicos de la misma zona en una misma rama.
- Los menores se le consultan a la persona.
- En una rama
- Anotar en «Triage de Claude» el PR de cada hallazgo arreglado. Commit
review: triage de Codex en MNy push amain. - Cierre, cuando todos los tickets del módulo estén
hechoy los PR de los hallazgos estén mergeados:- Pedir la aprobación humana del módulo (la del equipo, o la de quien lo pida en nombre del equipo).
- Con la aprobación,
git checkout main && git pull --ff-only origin main,git tag -a mod-N -m "Módulo MN cerrado"ygit push origin mod-N. tickets.mjs publicar-grafo.- Una entrada en
06-Desarrollo/Bitácora técnica.mdcon el resumen del módulo: qué quedó, tickets, hallazgos de Codex y decisiones tocadas. - Una línea en
00-Inicio/Estado del proyecto.md. Commit y push.
No hacer
- No cerrar un módulo sin la review de Codex ni sin la aprobación humana.
- No descartar un hallazgo sin haberlo verificado en el código.
- No editar la sección de hallazgos que escribió Codex: el triage va en su propia sección.