7Fase 7 · SDD Lifecycle
Pull Request y revisión humana
La IA acelera, pero no sustituye el criterio · Tú validas la intención, no solo el output
9/14
Qué se hace aquí
El agente genera código, tests y documentación, pero la decisión final es humana. El PR debe estar trazado contra la historia y contra la spec. La revisión no es línea a línea: es una validación de intención, alcance, calidad y seguridad.
Ejemplo de PR bien estructurado
## Change
Implements OpenSpec change: add-transaction-filtering
## Related Story
JIRA-1234
## OpenSpec
openspec/changes/add-transaction-filtering
## Summary
- Added transaction filters (date, amount, category)
- Updated OpenAPI contract
- Added unit, integration and E2E tests
## Validation Evidence
- openspec validate OK · openapi validate OK
- unit tests OK · integration tests OK
- playwright OK · security scan OK
## Risks
- Query performance on large datasets (monitor post-release)
Comandos Git
git checkout -b feature/add-transaction-filtering
git add .
git commit -m "feat: add transaction filtering"
git push origin feature/add-transaction-filtering
Qué valida el revisor humano
Intención vs. código
¿La implementación cumple la spec? ¿Se salió el agente del alcance definido?
Seguridad
¿La seguridad está bien aplicada? ¿Hay riesgos de fuga de datos entre usuarios?
Patrones del proyecto
¿Se respetan las convenciones, el contrato API y la arquitectura?
Cobertura de tests
¿Los tests cubren todos los criterios de aceptación definidos en la spec?
✓ Gate 7 — Human Review
✓
PR vinculado a historia✓
PR vinculado a OpenSpec change✓
Código alineado con spec✓
Sin cambios fuera de alcance✓
Evidencias de tests visibles✓
Aprobación técnica obtenida