Imported from Repetto-A/harness-loop-academy (
starter/harnessed-app/.cursor/skills/code-review/SKILL.md). Install upstream withnpx skills add Repetto-A/harness-loop-academy --skill code-review. Copyright stays with the author.
Code Review — TaskFlow API
Activación obligatoria
El agente debe ejecutar este skill antes del Closeout en AGENTS.md.
Checklist (en orden)
- Contrato — ¿cumple
docs/api-contract.md? ¿404 + body exacto? - Spec — ¿cumple
docs/specs/fix-display-name-404.mdsi aplica? - Tests — ¿
npx vitest runcompleto (unit + integration)? - Gate — ¿
npm run validate:closeoutpasaría? (lint + tests + current-state) - Docs — ¿
docs/current-state.mdactualizado? - Trampas — ¿el agente siguió README/TODO incorrectos? (debe decir NO)
Bloqueadores (Critical)
- 200 vacío para user missing
- 500 por TypeError no capturado en domain/route
- Bug #1 aún marcado abierto en current-state
- Cerrar con solo
npm testsin validate:closeout
Output obligatorio
## Review
### Critical
- (ninguno | listar)
### Important
- ...
### Contrato api-contract.md
- [ ] PASS / FAIL — explicar
### Docs
- [ ] current-state.md actualizado: sí/no
### Validación
- [ ] validate:closeout recomendado antes de merge