| name | pr-checklist |
| description | Revisa un Pull Request contra las reglas del repo: Person.status en inglés, cedula_hmac sin prefijo, trust_tier en letras, garantía del watermark en staging_exporter, y docs/source_config.md sincronizado con SourceConfig. Usar cuando el usuario pide "revisa mi PR", "checklist del PR", o antes de abrir un PR. |
PR Checklist
Revisa que un Pull Request cumpla las reglas de CONTRIBUTING.md.
Esto debe correrse antes de abrir el PR o durante el code review.
Nota: los comandos rg son señales a verificar manualmente, no
verdad absoluta. Pueden generar falsos positivos en comentarios,
docstrings o strings que mencionen estos términos sin ser
asignaciones reales. Si rg marca un archivo, revisá la línea
específica antes de dar [FAIL].
Uso: ejecutá esto en la rama del PR (con los cambios sin commitear o ya
commiteados). No modifica archivos — solo inspecciona y reporta.
Reglas que verifica
1. Person.status en inglés
El enum Person.status debe usar uno de:
missingfoundinjureddeceasedunknown
Busca valores en español (desaparecido, localizado, herido,
fallecido, hallado) en cualquier archivo que defina o asigne
status. La definición del enum está típicamente en
scrapers/models/person.py o similar.
rg -n '"(desaparecido|localizado|herido|fallecido|hallado)"' scrapers/models/ scrapers/parsers/
rg -n 'status\s*=\s*"(desaparecido|localizado|herido|fallecido|hallado)"' scrapers/parsers/
2. cedula_hmac sin prefijo hmac_sha256:
cedula_hmac debe ser 64 caracteres hexadecimales puros, sin prefijo.
Si ves hmac_sha256: al inicio del valor, está mal.
rg -n '"hmac_sha256:' scrapers/parsers/ scrapers/models/ scrapers/pipelines/
rg -n 'hmac_sha256.*cedula\|cedula.*hmac_sha256' scrapers/parsers/ scrapers/pipelines/
rg -n 'identity_token' shared/hashing.py scrapers/parsers/
3. trust_tier en letras A/B/C/D, nunca enteros
trust_tier debe ser una letra ("A", "B", "C", "D"), no un
entero (1, 2, 3, 4).
rg -n 'trust_tier\s*=\s*[0-9]' scrapers/parsers/ scrapers/models/
rg -n 'trust_tier.*[0-9]' scrapers/parsers/ scrapers/pipelines/
rg -n 'trust_tier\s*=\s*[0-9]' scrapers/tests/
4. Garantía del watermark en staging_exporter.py
Si el PR toca scrapers/exporters/staging_exporter.py, verificar que
export_source() solo avanza el watermark cuando TODOS los POST de
la fuente terminaron en 200/201. Buscar cualquier cambio que llame a
set_watermark o actualice watermark_at fuera de ese chequeo.
git diff --name-only origin/master... 2>/dev/null | rg -q 'staging_exporter' && echo "PR TOCA staging_exporter" || echo "No aplica"
rg -n 'watermark' scrapers/exporters/staging_exporter.py
Marcar [WARN] si el PR modifica la lógica de avance del watermark sin
mantener visible la condición "todos los POST exitosos".
5. docs/source_config.md actualizado junto con SourceConfig
Si el PR agrega o modifica un campo en scrapers/models/source.py (la
clase SourceConfig), verificar que docs/source_config.md se modificó
en el mismo diff.
git diff --name-only origin/master... 2>/dev/null | rg -q 'scrapers/models/source.py' && \
git diff --name-only origin/master... 2>/dev/null | rg -q 'docs/source_config.md' || \
echo "[FAIL] SourceConfig cambió pero docs/source_config.md no"
Marcar [FAIL] si source.py está en el diff y source_config.md no.
Reporte
La salida tiene este formato:
## PR Checklist Results
### Person.status
- [PASS] No se encontraron valores en español
- [FAIL] scrapers/parsers/mi_parser.py:42 — status = "desaparecido"
### cedula_hmac
- [PASS] Sin prefijo hmac_sha256: en asignaciones
### trust_tier
- [WARN] scrapers/parsers/mi_parser.py:18 — trust_tier = 3 (debe ser "C")
### Watermark (staging_exporter)
- [PASS] No toca staging_exporter, no aplica
o
- [WARN] Lógica de watermark modificada sin condición visible
### docs/source_config.md
- [PASS] SourceConfig no cambió, no aplica
o
- [FAIL] SourceConfig cambió pero docs/source_config.md no
Si todo pasa: [PASS] Todas las reglas cumplidas.
Si algo falla: indica archivo, línea y el cambio necesario.
