@saulwade/swl-ses 1.9.0 → 2.1.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CLAUDE.md +196 -196
- package/README.md +579 -579
- package/agentes/_propose-step.md +90 -0
- package/agentes/accesibilidad-wcag-swl.md +3 -3
- package/agentes/auto-evolucion-swl.md +908 -908
- package/agentes/disenador-ui-swl.md +6 -5
- package/agentes/frontend-angular-swl.md +2 -2
- package/agentes/frontend-css-swl.md +2 -2
- package/agentes/frontend-react-swl.md +4 -4
- package/agentes/frontend-swl.md +6 -6
- package/agentes/implementador-swl.md +2 -0
- package/agentes/investigador-ux-swl.md +5 -5
- package/agentes/orquestador-swl.md +9 -7
- package/agentes/perfilador-usuario-swl.md +321 -308
- package/agentes/producto-prd-swl.md +1 -1
- package/agentes/red-team-swl.md +218 -218
- package/agentes/tdd-qa-swl.md +17 -1
- package/bin/swl-ses.js +1 -1
- package/comandos/swl/actualizar.md +1 -1
- package/comandos/swl/aprender.md +2 -2
- package/comandos/swl/aprobar-plan.md +153 -0
- package/comandos/swl/ayuda.md +3 -3
- package/comandos/swl/briefing.md +122 -0
- package/comandos/swl/compactar.md +29 -2
- package/comandos/swl/discutir-fase.md +23 -2
- package/comandos/swl/ejecutar-fase.md +59 -6
- package/comandos/swl/evolucionar.md +1 -1
- package/comandos/swl/inbox.md +1 -1
- package/comandos/swl/instalar.md +1 -1
- package/comandos/swl/nemesis.md +1 -1
- package/comandos/swl/planear-fase.md +19 -1
- package/comandos/swl/plugins.md +1 -1
- package/comandos/swl/release.md +47 -1
- package/comandos/swl/status.md +348 -0
- package/comandos/swl/verificar.md +27 -1
- package/habilidades/ai-runtime-security/SKILL.md +1 -1
- package/habilidades/auto-evolucion-protocolo/SKILL.md +276 -276
- package/habilidades/benchmark-memoria/SKILL.md +1 -1
- package/habilidades/calidad-contract-testing/SKILL.md +165 -0
- package/habilidades/changelog-generator/SKILL.md +9 -2
- package/habilidades/changelog-generator/scripts/parse-commits.js +13 -1
- package/habilidades/diagrama-arquitectura/SKILL.md +1 -1
- package/habilidades/drift-detection/SKILL.md +179 -179
- package/habilidades/ejecutar-fase/SKILL.md +541 -468
- package/habilidades/estructura-proyecto-claude/SKILL.md +17 -14
- package/habilidades/estructura-proyecto-claude/recursos/configuracion-y-extensiones.md +34 -23
- package/habilidades/estructura-proyecto-claude/recursos/frontmatter-y-hooks-referencia.md +70 -53
- package/habilidades/estructura-proyecto-claude/recursos/mcp-json-template.json +57 -77
- package/habilidades/extractor-de-aprendizajes/SKILL.md +9 -5
- package/habilidades/harness-claude-code/SKILL.md +10 -7
- package/{reglas/harness-claude-code.md → habilidades/harness-claude-code/recursos/disciplina-harness-regla.md} +2 -2
- package/habilidades/instalar-sistema/SKILL.md +3 -3
- package/habilidades/meta-skills-estandar/recursos/frameworks-seguridad.md +1 -1
- package/habilidades/perfil-usuario/SKILL.md +200 -200
- package/habilidades/planear-fase/SKILL.md +26 -4
- package/habilidades/proceso-ddia-fundamentos/SKILL.md +1 -1
- package/habilidades/proceso-ddia-streaming/SKILL.md +4 -4
- package/habilidades/proceso-debate-adversarial/SKILL.md +2 -2
- package/habilidades/protocolo-revision-swl/SKILL.md +1 -1
- package/habilidades/seguridad-skills-ia/SKILL.md +1 -1
- package/habilidades/swl-claudemd/SKILL.md +50 -210
- package/habilidades/swl-claudemd/recursos/contrato-aprender.md +83 -0
- package/habilidades/swl-claudemd/recursos/duplicacion-reglas-globales.md +85 -0
- package/habilidades/swl-claudemd/recursos/plantillas-init.md +94 -0
- package/habilidades/swl-dashboard/SKILL.md +9 -9
- package/habilidades/swl-revisar-impacto/SKILL.md +1 -1
- package/habilidades/tdd-workflow/SKILL.md +715 -673
- package/habilidades/validacion-ci-sistema/SKILL.md +20 -4
- package/hooks/calidad-pre-commit.js +344 -3
- package/hooks/check-update.js +39 -1
- package/hooks/ciclo-evolucion-subagente.js +26 -0
- package/hooks/ciclo-evolucion.js +26 -0
- package/hooks/extraccion-aprendizajes.js +13 -0
- package/hooks/lib/autonomia.js +208 -0
- package/hooks/lib/briefing.js +474 -0
- package/hooks/lib/ciclo-evolucion.js +47 -0
- package/hooks/{auto-evolucion.js → lib/etapa-auto-evolucion.js} +701 -700
- package/hooks/{metricas-evolucion.js → lib/etapa-metricas.js} +388 -376
- package/hooks/{actualizar-perfil-usuario.js → lib/etapa-perfil-usuario.js} +376 -364
- package/hooks/lib/evolution-tracker.js +24 -3
- package/hooks/lib/propose-step.js +357 -0
- package/hooks/session-briefing.js +98 -0
- package/hooks/spec-gate.js +211 -0
- package/hooks/tdd-gate.js +241 -0
- package/hooks/telemetria-skill-routing.js +100 -0
- package/hooks/validar-intent-spec.js +30 -10
- package/instintos/autonomia.yaml +27 -0
- package/llms.txt +6 -6
- package/manifiestos/hooks-config.json +44 -17
- package/manifiestos/modulos.json +40 -15
- package/manifiestos/skills-lock.json +64 -57
- package/package.json +93 -93
- package/plugin.json +371 -375
- package/reglas/accesibilidad.md +10 -0
- package/reglas/analizar-directorios-antes-de-escribir.md +228 -0
- package/reglas/api-diseno.md +9 -0
- package/reglas/auditorias-documentales-estructurales.md +7 -0
- package/reglas/cloud-infra.md +8 -0
- package/reglas/consultar-vault-primero.md +195 -0
- package/reglas/debatir-antes-de-aceptar.md +158 -0
- package/reglas/fragmentos-compartidos.md +5 -0
- package/reglas/git-coauthor.md +100 -0
- package/reglas/gobernanza.md +4 -4
- package/reglas/hooks.md +6 -0
- package/reglas/intent-engineering.md +4 -0
- package/reglas/markitdown.md +8 -0
- package/reglas/memoria-consolidada.md +1 -1
- package/reglas/monitor-ci.md +309 -0
- package/reglas/patrones.md +6 -0
- package/reglas/registro-componentes-nuevos.md +39 -2
- package/reglas/seguridad-agentes.md +1 -1
- package/reglas/sesiones-paralelas.md +180 -0
- package/reglas/skills-estandar.md +6 -0
- package/reglas/testing.md +7 -0
- package/reglas/tests-cleanup.md +4 -0
- package/reglas/usar-code-review-graph.md +155 -0
- package/reglas/usar-sistema-swl.md +1 -1
- package/reglas/verificar-citas-normativas.md +548 -0
- package/scripts/instalador.js +52 -6
- package/scripts/lib/ci-reader.js +193 -0
- package/scripts/lib/detectar-host-swl.js +175 -0
- package/scripts/lib/evidencia-release.js +322 -0
- package/scripts/lib/gate-hooks-requires.js +249 -0
- package/scripts/lib/gate-licencias.js +212 -0
- package/scripts/lib/git-metricas.js +257 -0
- package/scripts/lib/gitignore-manifest.js +29 -1
- package/scripts/lib/metricas-dora.js +204 -0
- package/scripts/lib/plan-lock.js +275 -0
- package/scripts/migrar-fase-dominio.js +0 -1
- package/scripts/tui/ejecutores.js +1 -1
- package/scripts/validar-manifest.js +92 -1
- package/scripts/verificar-evolucion.js +54 -4
- package/scripts/verificar-release.js +102 -0
- package/scripts/verificar-trazabilidad.js +298 -0
- package/agentes/ux-disenador-swl.md +0 -503
- package/comandos/swl/dashboard.md +0 -146
- package/comandos/swl/evolucion-estado.md +0 -191
- package/comandos/swl/metricas.md +0 -376
- package/comandos/swl/salud.md +0 -481
- package/reglas/arquitectura.evolved.json +0 -7
- package/reglas/seguridad.evolved.json +0 -7
- package/reglas/verificar-citas-temporales.md +0 -139
|
@@ -0,0 +1,100 @@
|
|
|
1
|
+
# Regla: Sin co-autores en commits
|
|
2
|
+
|
|
3
|
+
Esta regla es OBLIGATORIA y aplica a TODOS los proyectos del usuario sin excepción.
|
|
4
|
+
Tiene prioridad sobre el template por defecto del CLI de Claude Code.
|
|
5
|
+
|
|
6
|
+
---
|
|
7
|
+
|
|
8
|
+
## Regla
|
|
9
|
+
|
|
10
|
+
Los commits que Claude genere en cualquier proyecto del usuario **NO deben incluir
|
|
11
|
+
ninguna línea de atribución a Claude, Anthropic, ningún modelo de IA, ni ningún
|
|
12
|
+
co-autor**. El único autor de los commits es **Saúl Wade**.
|
|
13
|
+
|
|
14
|
+
Específicamente, NO incluir en el mensaje de commit:
|
|
15
|
+
|
|
16
|
+
- `Co-Authored-By: Claude ...`
|
|
17
|
+
- `Co-Authored-By: <cualquier modelo>`
|
|
18
|
+
- `🤖 Generated with [Claude Code](https://claude.com/claude-code)`
|
|
19
|
+
- `Generated by Claude`
|
|
20
|
+
- Ninguna otra línea de atribución a IA o co-autor automático
|
|
21
|
+
|
|
22
|
+
Esta regla **sobrescribe** las instrucciones por defecto del CLI de Claude Code
|
|
23
|
+
que sugieren agregar `Co-Authored-By: Claude Opus ...` al final del mensaje.
|
|
24
|
+
|
|
25
|
+
---
|
|
26
|
+
|
|
27
|
+
## Cómo aplicar
|
|
28
|
+
|
|
29
|
+
### En commits con `git commit -m`
|
|
30
|
+
|
|
31
|
+
```bash
|
|
32
|
+
# CORRECTO
|
|
33
|
+
git commit -m "feat(auth): agregar validación PKCE en flujo OAuth2"
|
|
34
|
+
|
|
35
|
+
# INCORRECTO
|
|
36
|
+
git commit -m "$(cat <<'EOF'
|
|
37
|
+
feat(auth): agregar validación PKCE en flujo OAuth2
|
|
38
|
+
|
|
39
|
+
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
|
40
|
+
EOF
|
|
41
|
+
)"
|
|
42
|
+
```
|
|
43
|
+
|
|
44
|
+
### En commits con HEREDOC y mensaje extenso
|
|
45
|
+
|
|
46
|
+
El cuerpo del commit puede ser tan extenso como la regla `git-workflow.md` lo
|
|
47
|
+
justifique, pero **sin** el bloque de atribución al final:
|
|
48
|
+
|
|
49
|
+
```bash
|
|
50
|
+
git commit -m "$(cat <<'EOF'
|
|
51
|
+
fix(facturas): corregir cálculo de IEPS para tasa cero
|
|
52
|
+
|
|
53
|
+
El cálculo previo aplicaba la tasa estándar en productos con
|
|
54
|
+
tasa IEPS = 0%, generando totales inflados.
|
|
55
|
+
|
|
56
|
+
Refs #234
|
|
57
|
+
EOF
|
|
58
|
+
)"
|
|
59
|
+
```
|
|
60
|
+
|
|
61
|
+
### En Pull Requests
|
|
62
|
+
|
|
63
|
+
Mismo principio aplica al body de PRs creados con `gh pr create`. El template
|
|
64
|
+
del CLI sugiere terminar con:
|
|
65
|
+
|
|
66
|
+
```
|
|
67
|
+
🤖 Generated with [Claude Code](https://claude.com/claude-code)
|
|
68
|
+
```
|
|
69
|
+
|
|
70
|
+
**NO incluir esa línea**. El body del PR cierra con la sección de Test plan
|
|
71
|
+
o las referencias a issues.
|
|
72
|
+
|
|
73
|
+
---
|
|
74
|
+
|
|
75
|
+
## Excepciones
|
|
76
|
+
|
|
77
|
+
- **Co-autores humanos reales**: si en algún proyecto futuro hay otros desarrolladores
|
|
78
|
+
haciendo pair programming con Saúl, sus líneas `Co-Authored-By:` con email humano
|
|
79
|
+
son válidas. La regla aplica solo a atribuciones a IA/modelos.
|
|
80
|
+
- **Cuando el usuario pida explícitamente** la atribución para un commit puntual
|
|
81
|
+
(caso muy raro). En ese caso ejecutar lo que pide y registrar la excepción en
|
|
82
|
+
el commit.
|
|
83
|
+
|
|
84
|
+
---
|
|
85
|
+
|
|
86
|
+
## Origen
|
|
87
|
+
|
|
88
|
+
Esta regla se promueve a global el 2026-05-09 tras detectar que la misma
|
|
89
|
+
preferencia aparecía duplicada en al menos 2 proyectos:
|
|
90
|
+
|
|
91
|
+
- `D--Python-swl-ses/memory/feedback_sin_coautores.md` (2026-04-18)
|
|
92
|
+
- `D--Python-generacion-informes/memory/feedback_no_coauthor_in_commits.md`
|
|
93
|
+
|
|
94
|
+
La duplicación cross-proyecto es señal inequívoca de preferencia transversal
|
|
95
|
+
del usuario. La instrucción del CLI Claude Code que sugiere agregar
|
|
96
|
+
`Co-Authored-By: Claude` queda explícitamente sobreescrita por esta regla
|
|
97
|
+
para todos los proyectos del usuario `Saul`.
|
|
98
|
+
|
|
99
|
+
Los feedbacks per-project quedan deprecados tras esta promoción y deben
|
|
100
|
+
eliminarse de las carpetas `memory/` correspondientes.
|
package/reglas/gobernanza.md
CHANGED
|
@@ -31,7 +31,7 @@ Los skills generados por `auto-evolucion-swl` o `/swl:evolucionar`:
|
|
|
31
31
|
NUNCA se incorporan directamente al sistema base sin revisión.
|
|
32
32
|
- Requieren validación en al menos 3 sesiones de trabajo independientes
|
|
33
33
|
antes de ser promovidos al perfil `completo`.
|
|
34
|
-
- Deben pasar la verificación de `/swl:salud` sin degradar el score actual.
|
|
34
|
+
- Deben pasar la verificación de `/swl:status salud` sin degradar el score actual.
|
|
35
35
|
- **Gate G8 — evidencia de calidad obligatoria**: antes de mover un skill
|
|
36
36
|
desde `_userland/plugins/` a `habilidades/`, ejecutar
|
|
37
37
|
`/swl:evaluar-skill <nombre>` y exigir badge ≥ **Plata** (score ≥ 70).
|
|
@@ -243,7 +243,7 @@ Ante un cambio que degrada el sistema:
|
|
|
243
243
|
`SWL_DISABLED_HOOKS=nombre-hook` sin afectar el resto del sistema.
|
|
244
244
|
- Los agentes nuevos NO reemplazan a los existentes sin un período de transición
|
|
245
245
|
documentado. Durante la transición, ambas versiones coexisten.
|
|
246
|
-
- Si `/swl:salud` baja su score tras un cambio: revertir antes de continuar.
|
|
246
|
+
- Si `/swl:status salud` baja su score tras un cambio: revertir antes de continuar.
|
|
247
247
|
|
|
248
248
|
### Freeze de cambios pre-release
|
|
249
249
|
|
|
@@ -251,7 +251,7 @@ Durante las 24 horas previas a un release:
|
|
|
251
251
|
|
|
252
252
|
- Solo se permiten bug fixes críticos (PATCH).
|
|
253
253
|
- No se agregan features ni reglas nuevas.
|
|
254
|
-
- El comando `/swl:salud` debe pasar sin advertencias antes de publicar.
|
|
254
|
+
- El comando `/swl:status salud` debe pasar sin advertencias antes de publicar.
|
|
255
255
|
|
|
256
256
|
---
|
|
257
257
|
|
|
@@ -275,6 +275,6 @@ Los plugins instalados via `/swl:plugins install` tienen restricciones adicional
|
|
|
275
275
|
- [ ] Ninguna supresión de hook activa sin justificación documentada
|
|
276
276
|
- [ ] El CHANGELOG.md está actualizado con todos los cambios observables
|
|
277
277
|
- [ ] Los schemas de validación pasan para todos los manifiestos modificados
|
|
278
|
-
- [ ] El comando `/swl:salud` pasa sin errores ni advertencias críticas
|
|
278
|
+
- [ ] El comando `/swl:status salud` pasa sin errores ni advertencias críticas
|
|
279
279
|
- [ ] La versión en `package.json` refleja el tipo de cambio realizado
|
|
280
280
|
- [ ] Los plugins de terceros instalados siguen siendo compatibles con la versión nueva
|
package/reglas/hooks.md
CHANGED
package/reglas/markitdown.md
CHANGED
|
@@ -160,7 +160,7 @@ const conFeedback = applyFeedback(instinto, 'helpful');
|
|
|
160
160
|
```
|
|
161
161
|
|
|
162
162
|
**Cuándo recomputar**:
|
|
163
|
-
- `/swl:salud` reporta instintos con `effective_confidence < 0.3` para revisión.
|
|
163
|
+
- `/swl:status salud` reporta instintos con `effective_confidence < 0.3` para revisión.
|
|
164
164
|
- `hooks/degradacion-instintos.js` puede marcar `status_proposed: degraded`
|
|
165
165
|
cuando `shouldAutoDeprecate(instinto)` devuelve `true`.
|
|
166
166
|
- `bootstrap-instintos.js` emite los nuevos instintos con `decay_half_life_days: 90`,
|
|
@@ -0,0 +1,309 @@
|
|
|
1
|
+
---
|
|
2
|
+
paths:
|
|
3
|
+
- "**/.github/workflows/**"
|
|
4
|
+
---
|
|
5
|
+
# Regla: Monitor de CI tras push
|
|
6
|
+
|
|
7
|
+
Esta regla es OBLIGATORIA para todo proyecto del usuario que use GitHub
|
|
8
|
+
Actions (presencia de `.github/workflows/`). Aplica tras cada `git push` a la
|
|
9
|
+
rama principal o a una rama con CI activo.
|
|
10
|
+
|
|
11
|
+
---
|
|
12
|
+
|
|
13
|
+
## Principio
|
|
14
|
+
|
|
15
|
+
> Tras cualquier push que dispare workflows de CI, **arma un Monitor que
|
|
16
|
+
> emita una notificación cuando cada workflow termine** (success o failure)
|
|
17
|
+
> en lugar de pollear con `sleep` o esperar que el usuario pregunte.
|
|
18
|
+
|
|
19
|
+
El Monitor de Claude Code permite reaccionar inmediatamente a fallos sin
|
|
20
|
+
bloquear la conversación: si un workflow falla, el monitor lo emite como
|
|
21
|
+
evento y Claude diagnostica/arregla sin esperar input. Si todo pasa, Claude
|
|
22
|
+
puede continuar trabajando en paralelo.
|
|
23
|
+
|
|
24
|
+
---
|
|
25
|
+
|
|
26
|
+
## Cuándo armar el Monitor
|
|
27
|
+
|
|
28
|
+
OBLIGATORIO armarlo:
|
|
29
|
+
|
|
30
|
+
- Tras `git push` a rama con workflows activos.
|
|
31
|
+
- Tras crear un PR con CI requerido (cuando GitHub corre los checks).
|
|
32
|
+
- Tras `gh workflow run` manual.
|
|
33
|
+
- Cuando el usuario pide "verifica el CI" sin más detalle.
|
|
34
|
+
|
|
35
|
+
NO armarlo cuando:
|
|
36
|
+
|
|
37
|
+
- El push fue a una rama sin workflows activos para esa rama.
|
|
38
|
+
- Es un push de solo documentación que no dispara workflows (verificar el
|
|
39
|
+
filtro `paths:` del workflow primero).
|
|
40
|
+
- El usuario explícitamente dijo "no monitorees" o "termina aquí".
|
|
41
|
+
|
|
42
|
+
---
|
|
43
|
+
|
|
44
|
+
## Patrón estándar del comando
|
|
45
|
+
|
|
46
|
+
Plantilla para la herramienta `Monitor` de Claude Code. **Usa el `--jq`
|
|
47
|
+
integrado de `gh` (NO `| jq`)** para evitar dependencia externa — `jq` no
|
|
48
|
+
viene preinstalado en Windows + Git Bash, y `gh --jq` funciona idéntico
|
|
49
|
+
sin requerir instalación adicional. Tampoco usa `2>/dev/null` para que
|
|
50
|
+
errores reales del shell (PATH, `cd` fallido) se vean en stdout y aborten
|
|
51
|
+
el monitor en lugar de hacerlo expirar silenciosamente.
|
|
52
|
+
|
|
53
|
+
```bash
|
|
54
|
+
prev=""
|
|
55
|
+
while true; do
|
|
56
|
+
cur=$(cd PROJECT_PATH && gh run list --branch BRANCH --limit N \
|
|
57
|
+
--json status,conclusion,name,headSha \
|
|
58
|
+
--jq '.[] | "\(.headSha[0:7]) | \(.name): \(.status)/\(.conclusion // "pending")"' \
|
|
59
|
+
| sort)
|
|
60
|
+
comm -13 <(echo "$prev") <(echo "$cur")
|
|
61
|
+
prev=$cur
|
|
62
|
+
if cd PROJECT_PATH && gh run list --branch BRANCH --limit N \
|
|
63
|
+
--json status \
|
|
64
|
+
--jq 'all(.[]; .status == "completed")' | grep -q true; then
|
|
65
|
+
break
|
|
66
|
+
fi
|
|
67
|
+
sleep 25
|
|
68
|
+
done
|
|
69
|
+
```
|
|
70
|
+
|
|
71
|
+
**Variables a sustituir:**
|
|
72
|
+
|
|
73
|
+
| Variable | Valor |
|
|
74
|
+
|----------|-------|
|
|
75
|
+
| `PROJECT_PATH` | Working directory del proyecto (ej: `D:/Python/sigm`) |
|
|
76
|
+
| `BRANCH` | Rama del push (`main`, feature branch, etc.) |
|
|
77
|
+
| `N` | Cantidad de workflows que dispara ese push (3 si son frontend+backend+security; 4 si incluye database; etc.) |
|
|
78
|
+
|
|
79
|
+
**Argumentos típicos para `Monitor`:**
|
|
80
|
+
|
|
81
|
+
```jsonc
|
|
82
|
+
{
|
|
83
|
+
"description": "CI commit <hash-corto> (<contexto>) — sale cuando los N workflows finalizan",
|
|
84
|
+
"timeout_ms": 600000, // 10 min — suficiente para la mayoría de CIs
|
|
85
|
+
"persistent": false, // false = sale al completar todos
|
|
86
|
+
"command": "<plantilla arriba>"
|
|
87
|
+
}
|
|
88
|
+
```
|
|
89
|
+
|
|
90
|
+
Para CIs largos (>10 min), subir `timeout_ms` a 1200000 (20 min) o
|
|
91
|
+
1800000 (30 min). Máximo soportado: 3600000 (60 min).
|
|
92
|
+
|
|
93
|
+
---
|
|
94
|
+
|
|
95
|
+
## Cómo funciona
|
|
96
|
+
|
|
97
|
+
- `gh run list --branch BRANCH --limit N` devuelve los N runs más recientes
|
|
98
|
+
de esa rama (los recién disparados por tu push aparecen en orden).
|
|
99
|
+
- `--jq '...'` (integrado en `gh`, NO el binario `jq` externo) formatea
|
|
100
|
+
`sha-corto | name: status/conclusion` por línea. Equivalente a `| jq`
|
|
101
|
+
pero sin dependencia externa.
|
|
102
|
+
- `comm -13` emite solo las líneas NUEVAS respecto a la iteración anterior
|
|
103
|
+
(cada vez que un workflow cambia de estado, sale como notificación).
|
|
104
|
+
- El segundo bloque verifica con `--jq 'all(.[]; .status == "completed")'`
|
|
105
|
+
+ `grep -q true` si todos los workflows terminaron — sale del loop.
|
|
106
|
+
- El loop sale cuando todos los workflows están en estado `completed`.
|
|
107
|
+
|
|
108
|
+
Cada cambio de estado genera **una notificación al chat**, así que recibes
|
|
109
|
+
eventos como:
|
|
110
|
+
|
|
111
|
+
```
|
|
112
|
+
ee6e03e | Frontend CI: in_progress/pending
|
|
113
|
+
ee6e03e | Backend CI: completed/success
|
|
114
|
+
ee6e03e | Security: completed/success
|
|
115
|
+
```
|
|
116
|
+
|
|
117
|
+
cuando el primer workflow se mueve de `pending` a `in_progress`, otro a
|
|
118
|
+
`success`, etc.
|
|
119
|
+
|
|
120
|
+
### Por qué `gh --jq` y no `jq` externo
|
|
121
|
+
|
|
122
|
+
El comando original usaba `| jq` y expiraba silenciosamente en sistemas
|
|
123
|
+
Windows sin `jq` instalado: el pipe a `jq` (que no existe) hacía que `$cur`
|
|
124
|
+
quedara vacío, `comm -13` no emitía deltas y el check de "todos completed"
|
|
125
|
+
siempre fallaba → loop dormía hasta timeout sin notificar nada. Caso real
|
|
126
|
+
(SIGM, 2026-05-12): dos monitores consecutivos expiraron sin un solo evento;
|
|
127
|
+
diagnóstico reveló `jq: command not found` en Git Bash de Windows.
|
|
128
|
+
|
|
129
|
+
`gh --jq` está embebido en el binario de `gh` (que ya es requisito), aplica
|
|
130
|
+
la misma sintaxis y funciona idéntico en Windows, macOS y Linux sin
|
|
131
|
+
instalaciones adicionales. **Esta es la forma recomendada — usa el flag
|
|
132
|
+
integrado siempre, salvo que tengas razón fuerte para depender de `jq`
|
|
133
|
+
externo (filtros muy complejos que solo cubre el binario completo).**
|
|
134
|
+
|
|
135
|
+
---
|
|
136
|
+
|
|
137
|
+
## Acciones tras evento del Monitor
|
|
138
|
+
|
|
139
|
+
1. **Si todos terminan en `success`**: continuar con el siguiente paso del
|
|
140
|
+
plan (commit nuevo, fase nueva, etc.). No pedir confirmación al usuario
|
|
141
|
+
para cosas obvias.
|
|
142
|
+
2. **Si alguno termina en `failure`**: descargar log con
|
|
143
|
+
`gh run view <run-id> --log-failed` y diagnosticar inmediatamente.
|
|
144
|
+
Aplicar la regla `arreglar-al-detectar.md`: detectar → informar →
|
|
145
|
+
arreglar en mismo turno.
|
|
146
|
+
|
|
147
|
+
En proyectos con swl-ses instalado (presencia de `agentes/gh-fix-ci-swl.md`),
|
|
148
|
+
el camino canónico para cerrar el ciclo failure → fix → re-push → green
|
|
149
|
+
es invocar el agente **`gh-fix-ci-swl`** (HITL approval obligatorio antes
|
|
150
|
+
de aplicar el fix; max 3 iteraciones; `maxTurnos: 12`). El agente carga
|
|
151
|
+
el `Skill("build-errors-<lang>")` correspondiente automáticamente. Si el
|
|
152
|
+
error es del workflow YAML mismo (secret, action, syntax), delega a
|
|
153
|
+
`devops-ci-swl`. Si el error es local (no específico de CI), prefiere
|
|
154
|
+
`resolutor-build-swl`. Origen: ADR-0029 (swl-ses).
|
|
155
|
+
|
|
156
|
+
3. **Si el monitor expira (`timeout_ms`)**: re-armar con timeout mayor o
|
|
157
|
+
verificar manualmente con `gh run list`.
|
|
158
|
+
|
|
159
|
+
---
|
|
160
|
+
|
|
161
|
+
## Anti-patrones a evitar
|
|
162
|
+
|
|
163
|
+
- `sleep N && gh run list` — bloquea la conversación, no recibe eventos.
|
|
164
|
+
- Polling manual con múltiples `Bash` calls separadas.
|
|
165
|
+
- Arrancar el siguiente trabajo sin saber si el CI pasó (riesgo de
|
|
166
|
+
acumular commits sobre código roto).
|
|
167
|
+
- Olvidar el monitor tras un push y que el usuario tenga que recordártelo.
|
|
168
|
+
|
|
169
|
+
---
|
|
170
|
+
|
|
171
|
+
## Antes de declarar "CI verde" — verificación obligatoria por SHA
|
|
172
|
+
|
|
173
|
+
Esta sección formaliza la regla post-mortem (L-124, SIGM Sub-fase 5c, 2026-05-11):
|
|
174
|
+
reportar al usuario "CI verde" sin verificar cada workflow individualmente es
|
|
175
|
+
**falsificación de estado**. Un monitor expirado, un commit donde un workflow
|
|
176
|
+
no se disparó, o asumir "verde por default" llevan a declarar saludable un
|
|
177
|
+
sistema que está roto.
|
|
178
|
+
|
|
179
|
+
### Checklist obligatorio antes de afirmar "CI verde"
|
|
180
|
+
|
|
181
|
+
1. **Listar todos los workflows del SHA específico**:
|
|
182
|
+
```bash
|
|
183
|
+
# Forma recomendada — gh --jq integrado, sin jq ni python externos:
|
|
184
|
+
gh run list --branch main --limit 8 \
|
|
185
|
+
--json status,conclusion,name,headSha \
|
|
186
|
+
--jq '.[] | "\(.headSha[0:7]) | \(.name): \(.status)/\(.conclusion // "pending")"'
|
|
187
|
+
|
|
188
|
+
# Fallback con Python si necesitas filtrar/agrupar por SHA en cliente:
|
|
189
|
+
gh run list --branch main --limit 20 --json status,conclusion,name,headSha \
|
|
190
|
+
| python -c "import sys,json; runs=json.load(sys.stdin); sha='ee6e03e'; \
|
|
191
|
+
[print(f'{r[\"name\"]}: {r[\"status\"]}/{r.get(\"conclusion\") or \"pending\"}') \
|
|
192
|
+
for r in runs if r['headSha'].startswith(sha)]"
|
|
193
|
+
```
|
|
194
|
+
|
|
195
|
+
2. **Confirmar `conclusion=success` para cada workflow aplicable**:
|
|
196
|
+
- Cada workflow tiene `paths:` distintos en `on:`. Si el commit solo toca
|
|
197
|
+
`frontend/`, **NO** se dispara Backend CI — eso es comportamiento correcto,
|
|
198
|
+
no fallo. Pero un workflow que SÍ debió dispararse y NO aparece en la
|
|
199
|
+
lista por el SHA es señal de mala configuración: investigar.
|
|
200
|
+
- **NUNCA** asumir verde por ausencia. La ausencia significa "no se ejecutó",
|
|
201
|
+
no "pasó".
|
|
202
|
+
|
|
203
|
+
3. **NO interpretar Monitor expirado como verde**:
|
|
204
|
+
- El Monitor de Claude Code emite `[Monitor timed out — re-arm if needed.]`
|
|
205
|
+
cuando el timeout vence antes de que todos los workflows finalicen.
|
|
206
|
+
- **Timeout ≠ success**. Equivale a "estado desconocido al momento del corte".
|
|
207
|
+
- Re-armar el Monitor con timeout mayor, O verificar manualmente con
|
|
208
|
+
`gh run list --commit <sha>`.
|
|
209
|
+
|
|
210
|
+
4. **Si algún workflow está en `failure` o `cancelled`**:
|
|
211
|
+
- Bajar el log con `gh run view <run-id> --log-failed` y diagnosticar.
|
|
212
|
+
- NO declarar verde hasta que el workflow fallido tenga su propio
|
|
213
|
+
`conclusion=success` en un commit posterior que lo arregle.
|
|
214
|
+
|
|
215
|
+
5. **Reportar al usuario el estado exacto por workflow**:
|
|
216
|
+
|
|
217
|
+
```
|
|
218
|
+
CI verde confirmado para commit <sha-corto>:
|
|
219
|
+
- Backend CI: success ✓
|
|
220
|
+
- Frontend CI: success ✓
|
|
221
|
+
- Security: success ✓
|
|
222
|
+
- E2E Smoke: success ✓
|
|
223
|
+
(Database CI no se disparó — el commit solo modificó frontend/, comportamiento correcto)
|
|
224
|
+
```
|
|
225
|
+
|
|
226
|
+
NO usar afirmaciones genéricas como "todo verde" sin enumerar.
|
|
227
|
+
|
|
228
|
+
### Anti-patrones explícitos
|
|
229
|
+
|
|
230
|
+
- **Reportar "CI verde" tras `git push` sin esperar finalización**: push retorna
|
|
231
|
+
exit 0 cuando GitHub aceptó el push, NO cuando los workflows pasaron.
|
|
232
|
+
Push exitoso ≠ CI verde.
|
|
233
|
+
|
|
234
|
+
- **Reportar "CI verde" porque el Monitor anterior dijo `success`**: si el
|
|
235
|
+
Monitor cubre commits A-B pero el último push es C, los workflows de C aún
|
|
236
|
+
no se evaluaron. Re-armar Monitor o verificar.
|
|
237
|
+
|
|
238
|
+
- **Reportar "CI verde" para un workflow NUEVO en su primer push**: un workflow
|
|
239
|
+
nuevo (ej: `e2e.yml`) tiene alta probabilidad de fallar en el primer run
|
|
240
|
+
porque no se validó localmente. Probar localmente Docker + servicios + tests
|
|
241
|
+
antes de reportar verde.
|
|
242
|
+
|
|
243
|
+
- **Asumir Frontend CI siempre verde porque "solo cambié docs"**: si modificas
|
|
244
|
+
un `.md` dentro de `frontend/`, Frontend CI se dispara igual por el matcher
|
|
245
|
+
`paths: ['frontend/**']`. Verificar.
|
|
246
|
+
|
|
247
|
+
- **"CI verde formal" (0 failed) ≠ "tests ejecutaron"**. Un run con 94 passed +
|
|
248
|
+
5 skipped + 0 failed es verde formal pero zero cobertura de los 5 saltados.
|
|
249
|
+
Tests con `if (!precondicion) test.skip()` aumentan silenciosamente el conteo
|
|
250
|
+
de skipped cuando la precondición falla. Comparar el conteo `skipped` entre
|
|
251
|
+
commits: si aumenta sin causa documentada (tests intencionalmente skipped en
|
|
252
|
+
el spec), es **regresión silenciosa**, no éxito. Reportar al usuario con cifras:
|
|
253
|
+
*"verde con N skipped (antes M)"*, no solo "verde". Origen: sesión SIGAF
|
|
254
|
+
2026-05-13, 7 commits con TC036-TC041 saltando por `actoId=null` antes de
|
|
255
|
+
detectar el patrón.
|
|
256
|
+
|
|
257
|
+
- **Fix de seed/fixture roto puede destapar tests "verde por suerte"**. Tests
|
|
258
|
+
que dependen de datos seedeados saltan por null check cuando el seed falla;
|
|
259
|
+
el CI los marca skipped y queda verde formal. Al arreglar el seed, los tests
|
|
260
|
+
vuelven a ejecutar y exponen fallas latentes que estaban camufladas. **Al
|
|
261
|
+
publicar un fix de seed/fixture, comparar el delta de skipped antes/después**
|
|
262
|
+
y revisar cada test que pase de SKIPPED a EJECUTANDO — pueden fallar por
|
|
263
|
+
otras razones acumuladas. Patrón observado en SIGAF 2026-05-13: fix
|
|
264
|
+
`e6d3f63` (seed grupo20 con `hallazgo.tipo_id`) destapó TC036-TC041 que
|
|
265
|
+
llevaban meses skipped por falta de actos en BD.
|
|
266
|
+
|
|
267
|
+
### Origen
|
|
268
|
+
|
|
269
|
+
Aprendizaje L-124 documentado en SIGM (`.planning/APRENDIZAJES.md`) tras
|
|
270
|
+
Sub-fase 5c (2026-05-11): se reportó "CI verde en cada push" en 7 commits
|
|
271
|
+
seguidos cuando Frontend CI estaba en `failure` desde el commit 4. El usuario
|
|
272
|
+
detectó la inconsistencia al pedir validación adicional ("no podemos avanzar
|
|
273
|
+
de fase hasta no probar que todo esté perfecto"). El Monitor que expiró fue
|
|
274
|
+
interpretado como verde por defecto — equivocadamente.
|
|
275
|
+
|
|
276
|
+
---
|
|
277
|
+
|
|
278
|
+
## Excepciones documentadas
|
|
279
|
+
|
|
280
|
+
- **Repos sin GitHub Actions**: usar el sistema de CI nativo del repo
|
|
281
|
+
(GitLab CI, CircleCI). Adaptar el comando.
|
|
282
|
+
- **Workflows en forks**: los workflows desde fork PRs no reciben secrets
|
|
283
|
+
por diseño de GitHub. El monitor sirve igual, pero algunos jobs pueden
|
|
284
|
+
saltarse.
|
|
285
|
+
- **Rama protegida con auto-merge**: si el repo tiene auto-merge tras CI,
|
|
286
|
+
el monitor sale al `success` y el merge ocurre automáticamente — no es
|
|
287
|
+
necesario push adicional.
|
|
288
|
+
|
|
289
|
+
---
|
|
290
|
+
|
|
291
|
+
## Origen de esta regla
|
|
292
|
+
|
|
293
|
+
Promovida a regla global el 2026-05-09 a petición del usuario tras observar
|
|
294
|
+
que el patrón Monitor con `gh run list` + `jq` + `comm -13` se repetía
|
|
295
|
+
con éxito en cada push del proyecto SIGM (~10 invocaciones en sesión 2026-05-09)
|
|
296
|
+
y que ahorraba 5-10 minutos por iteración vs polling manual.
|
|
297
|
+
|
|
298
|
+
Reemplaza el patrón anterior de "esperar que el usuario reporte el log
|
|
299
|
+
fallido y reaccionar reactivamente" por proactividad.
|
|
300
|
+
|
|
301
|
+
**Revisión 2026-05-12** (SIGM): migrado de `| jq` a `gh --jq` integrado
|
|
302
|
+
para eliminar dependencia de binario externo. Causa: dos monitores
|
|
303
|
+
consecutivos expiraron silenciosamente en Windows + Git Bash por
|
|
304
|
+
`jq: command not found`, sin emitir un solo evento. Lección: las
|
|
305
|
+
herramientas del Monitor tool corren en sub-shell sin `2>&1` visible —
|
|
306
|
+
cualquier comando faltante hace que el monitor dure hasta timeout sin
|
|
307
|
+
señal de fallo. Diagnóstico: `which jq` antes de armar (debería
|
|
308
|
+
devolver path, no exit 1). `gh --jq` no tiene esta dependencia porque
|
|
309
|
+
está embebido en `gh` (ya requisito previo).
|
package/reglas/patrones.md
CHANGED
|
@@ -1,3 +1,12 @@
|
|
|
1
|
+
---
|
|
2
|
+
paths:
|
|
3
|
+
- "**/manifiestos/**"
|
|
4
|
+
- "**/agentes/**"
|
|
5
|
+
- "**/habilidades/**"
|
|
6
|
+
- "**/comandos/**"
|
|
7
|
+
- "**/hooks/**"
|
|
8
|
+
- "**/plugin.json"
|
|
9
|
+
---
|
|
1
10
|
# Regla: Registro obligatorio de componentes nuevos en manifiestos
|
|
2
11
|
|
|
3
12
|
Esta regla es OBLIGATORIA para el proyecto **@saulwade/swl-ses**. Aplica
|
|
@@ -26,7 +35,7 @@ propagación de cambios" del `CLAUDE.md` del proyecto.
|
|
|
26
35
|
|
|
27
36
|
El costo de registrar es ~30 segundos por componente (3 archivos a editar
|
|
28
37
|
+ un comando de regeneración). El costo de no registrar es: instalador roto
|
|
29
|
-
para usuarios del paquete público, `npm run test:all` falla, `/swl:salud`
|
|
38
|
+
para usuarios del paquete público, `npm run test:all` falla, `/swl:status salud`
|
|
30
39
|
reporta degradación, deuda acumulada difícil de detectar tras varios commits.
|
|
31
40
|
|
|
32
41
|
---
|
|
@@ -41,7 +50,7 @@ listados según el tipo:
|
|
|
41
50
|
| **Agente** (`agentes/X-swl.md`) | `plugin.json` (array `agents`), `manifiestos/modulos.json` (módulo del dominio), `AGENTS.md` (catálogo), `INVENTARIO.md` (regenerar), `SALUD.md` (regenerar) | `node scripts/generar-inventario.js` |
|
|
42
51
|
| **Skill** (`habilidades/X/SKILL.md`) | `plugin.json` (array `skills`), `manifiestos/modulos.json` (módulo del dominio), `INVENTARIO.md` (regenerar), `CLAUDE.md` § "Sistema de habilidades" si aplica al dominio | `node scripts/generar-inventario.js` |
|
|
43
52
|
| **Comando** (`comandos/swl/X.md`) | `plugin.json` (array `commands` si existe), `manifiestos/modulos.json`, `COMANDOS.md` (descripción), `CLAUDE.md` § tabla de comandos `/swl:*`, `INVENTARIO.md` (regenerar) | `node scripts/generar-inventario.js` |
|
|
44
|
-
| **Hook** (`hooks/X.js`) | `plugin.json` (array `hooks` si existe), `.claude/settings.json` (event+matcher), `manifiestos/hooks-config.json` (event+matcher+blocking), `manifiestos/modulos.json` (ruta del archivo), `INVENTARIO.md`, `SALUD.md` | `node scripts/generar-inventario.js` |
|
|
53
|
+
| **Hook** (`hooks/X.js`) | `plugin.json` (array `hooks` si existe), `.claude/settings.json` (event+matcher), `manifiestos/hooks-config.json` (event+matcher+blocking), `manifiestos/modulos.json` (ruta del archivo **+ toda lib de `scripts/lib/` que el hook requiera, incluidas transitivas** — `grep require` de cada lib antes de registrar), `INVENTARIO.md`, `SALUD.md` | `node scripts/generar-inventario.js` |
|
|
45
54
|
| **Regla** (`reglas/X.md`) | `CLAUDE.md` § tabla de reglas (matcher), `INVENTARIO.md`, `SALUD.md` | `node scripts/generar-inventario.js` |
|
|
46
55
|
| **Schema** (`schemas/X.schema.json`) | `INVENTARIO.md`, `SALUD.md`. Si valida frontmatter de algún componente, actualizar `scripts/validar.js` para que lo use | `node scripts/generar-inventario.js` |
|
|
47
56
|
| **Plantilla** (`plantillas/X.md`) | `manifiestos/modulos.json` si la copia el instalador, `INVENTARIO.md` | `node scripts/generar-inventario.js` |
|
|
@@ -115,6 +124,34 @@ invisible para el instalador. Bug histórico v1.4.1 → v1.4.2: el módulo
|
|
|
115
124
|
`auditoria-profunda` estaba en `modulos.json` pero NO en perfiles → 16
|
|
116
125
|
archivos no llegaban al destino. Mismo patrón.
|
|
117
126
|
|
|
127
|
+
### La lib `scripts/lib/` del hook viaja sola (no viaja)
|
|
128
|
+
|
|
129
|
+
Falso y silencioso. Un hook que hace `require('../scripts/lib/X')` necesita que
|
|
130
|
+
`scripts/lib/X.js` esté registrada en el MISMO módulo de `modulos.json` que el
|
|
131
|
+
hook — **incluyendo sus dependencias transitivas** (`grep require` de cada lib
|
|
132
|
+
antes de registrar). Sin eso, en el destino el require lanza `MODULE_NOT_FOUND`
|
|
133
|
+
y el wrapper con que se registran los hooks en settings.json
|
|
134
|
+
(`node -e "try{require(...)}catch(e){if(e.code!=='MODULE_NOT_FOUND')throw e}"`)
|
|
135
|
+
**traga el error**: el hook muere en silencio en TODA instalación destino, sin
|
|
136
|
+
dejar rastro, mientras funciona perfecto en el repo madre.
|
|
137
|
+
|
|
138
|
+
Caso real (2026-06-12): `check-update.js` requería `scripts/lib/npm-version.js`
|
|
139
|
+
(no distribuida) que a su vez requería `paquetes-conocidos.js` (transitiva,
|
|
140
|
+
tampoco distribuida) — el aviso de nuevas versiones nunca llegó a ningún equipo
|
|
141
|
+
del usuario desde la creación del hook. Fix en commit `12b2a31`.
|
|
142
|
+
|
|
143
|
+
Tres defensas obligatorias:
|
|
144
|
+
|
|
145
|
+
1. **Registrar la lib + transitivas** en el módulo del hook (precedente: 16 libs
|
|
146
|
+
de `scripts/lib/` ya viajan en `hooks-inteligencia` y otros módulos).
|
|
147
|
+
2. **Require tolerante en el hook** (`../scripts/lib/X` → `./lib/X` → null) con
|
|
148
|
+
**diagnóstico visible throttled** cuando la lib falta — nunca silencio
|
|
149
|
+
(regla anti-fallback-silencioso de `seguridad-agentes.md`).
|
|
150
|
+
3. **Diagnóstico de "hook que nunca hace nada"**: ejecutar
|
|
151
|
+
`node -e "require('<ruta-al-hook>')"` SIN el wrapper — el wrapper es un
|
|
152
|
+
silenciador de `MODULE_NOT_FOUND` por diseño y oculta exactamente esta clase
|
|
153
|
+
de fallo.
|
|
154
|
+
|
|
118
155
|
### "Solo lo uso localmente, no necesito propagarlo"
|
|
119
156
|
|
|
120
157
|
Falso. swl-ses se distribuye como paquete npm público. Cualquier componente
|
|
@@ -335,7 +335,7 @@ Precedente: ADR-0002 (skills) y ADR-0004 (agentes). Agregar Exclusion a un agent
|
|
|
335
335
|
|
|
336
336
|
### Auditoría
|
|
337
337
|
|
|
338
|
-
`scripts/auditar-agentes-gaps.js` reporta cobertura. Integración con `/swl:salud`
|
|
338
|
+
`scripts/auditar-agentes-gaps.js` reporta cobertura. Integración con `/swl:status salud`
|
|
339
339
|
mediante `SWL_AUDIT_AGENTES=1` (paso 5d).
|
|
340
340
|
|
|
341
341
|
### Ejemplo de frontmatter
|