@saulwade/swl-ses 1.6.0 → 1.6.1
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 +32 -61
- package/README.md +2 -2
- package/agentes/datos-swl.md +1 -1
- package/agentes/frontend-angular-swl.md +7 -7
- package/agentes/frontend-css-swl.md +4 -4
- package/agentes/frontend-react-swl.md +7 -7
- package/agentes/frontend-swl.md +9 -9
- package/agentes/frontend-tailwind-swl.md +4 -4
- package/agentes/rendimiento-swl.md +2 -2
- package/comandos/swl/brainstorm.md +1 -0
- package/comandos/swl/compactar.md +1 -1
- package/comandos/swl/discutir-fase.md +15 -1
- package/comandos/swl/mapear-codebase.md +1 -1
- package/comandos/swl/nemesis.md +29 -0
- package/comandos/swl/planear-fase.md +2 -2
- package/comandos/swl/verificar.md +4 -4
- package/habilidades/aprendizaje-continuo/SKILL.md +7 -1
- package/habilidades/diseno-herramientas-agente/SKILL.md +1 -0
- package/habilidades/doc-sync/SKILL.md +441 -1
- package/habilidades/doubt-driven-review/SKILL.md +177 -171
- package/habilidades/feynman-auditor-swl/SKILL.md +129 -123
- package/habilidades/infra-github-actions/SKILL.md +172 -166
- package/habilidades/meta-skills-estandar/recursos/skills-as-agents.md +163 -163
- package/habilidades/nemesis-evaluacion-json/SKILL.md +5 -0
- package/habilidades/nemesis-redistribuir/SKILL.md +5 -0
- package/habilidades/node-experto/SKILL.md +197 -3
- package/habilidades/prevencion-racionalizacion/SKILL.md +1 -0
- package/habilidades/privacy-memoria/SKILL.md +1 -0
- package/habilidades/sre-patrones/SKILL.md +1 -1
- package/habilidades/state-inconsistency-auditor-swl/SKILL.md +172 -166
- package/habilidades/tdd-workflow/SKILL.md +178 -3
- package/habilidades/verificacion-evidencia/SKILL.md +1 -0
- package/habilidades/web-fetcher-routing/SKILL.md +81 -75
- package/habilidades/workflow-claude-code/SKILL.md +2 -2
- package/manifiestos/modulos.json +2 -1
- package/manifiestos/skills-lock.json +1142 -1142
- package/package.json +5 -3
- package/plugin.json +2 -2
- package/reglas/auditorias-documentales-estructurales.md +205 -0
- package/schemas/agent-frontmatter.schema.json +1 -1
- package/scripts/generar-inventario.js +450 -420
- package/scripts/verificar-docs-vs-codigo.js +654 -425
- package/scripts/verificar-evolucion.js +19 -3
|
@@ -1,166 +1,172 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: state-inconsistency-auditor-swl
|
|
3
|
-
description: >
|
|
4
|
-
Encuentra bugs donde una operación muta un fragmento de estado acoplado sin
|
|
5
|
-
actualizar su contraparte dependiente, causando corrupción silenciosa o fallos
|
|
6
|
-
en operaciones posteriores. Mapea sistemáticamente pares de estado acoplado,
|
|
7
|
-
todos sus paths de mutación, y los gaps donde un lado se actualiza sin el otro.
|
|
8
|
-
Language-agnostic (Python, TS, Go, Rust, Java, C#). Cargar cuando se sospeche
|
|
9
|
-
estado desincronizado: caché obsoleta, índices secundarios huérfanos, contadores
|
|
10
|
-
que no coinciden con la suma de sus partes, o permisos derivados que no reflejan
|
|
11
|
-
el estado de origen.
|
|
12
|
-
|
|
13
|
-
|
|
14
|
-
|
|
15
|
-
|
|
16
|
-
-
|
|
17
|
-
-
|
|
18
|
-
|
|
19
|
-
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
-
|
|
24
|
-
-
|
|
25
|
-
-
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
---
|
|
42
|
-
|
|
43
|
-
##
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
|
|
51
|
-
|
|
52
|
-
|
|
53
|
-
|
|
54
|
-
|
|
55
|
-
|
|
56
|
-
|
|
57
|
-
|
|
58
|
-
|
|
59
|
-
|
|
60
|
-
|
|
61
|
-
|
|
62
|
-
|
|
63
|
-
|
|
64
|
-
|
|
65
|
-
|
|
66
|
-
|
|
67
|
-
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
|
|
71
|
-
|
|
72
|
-
|
|
73
|
-
|
|
74
|
-
|
|
75
|
-
|
|
76
|
-
|
|
77
|
-
|
|
78
|
-
|
|
79
|
-
|
|
80
|
-
|
|
81
|
-
|
|
82
|
-
|
|
83
|
-
|
|
84
|
-
|
|
85
|
-
|
|
86
|
-
|
|
87
|
-
|
|
88
|
-
|
|
89
|
-
|
|
90
|
-
|
|
91
|
-
|
|
92
|
-
|
|
93
|
-
|
|
94
|
-
|
|
95
|
-
|
|
96
|
-
|
|
97
|
-
|
|
98
|
-
|
|
99
|
-
|
|
100
|
-
|
|
101
|
-
|
|
102
|
-
|
|
103
|
-
|
|
104
|
-
|
|
105
|
-
|
|
106
|
-
|
|
107
|
-
|
|
108
|
-
|
|
109
|
-
|
|
110
|
-
|
|
111
|
-
|
|
112
|
-
|
|
113
|
-
|
|
114
|
-
|
|
115
|
-
|
|
116
|
-
|
|
117
|
-
|
|
118
|
-
|
|
119
|
-
|
|
120
|
-
|
|
121
|
-
|
|
122
|
-
|
|
123
|
-
|
|
124
|
-
|
|
125
|
-
|
|
126
|
-
|
|
127
|
-
|
|
128
|
-
|
|
129
|
-
|
|
130
|
-
|
|
131
|
-
|
|
132
|
-
|
|
133
|
-
|
|
134
|
-
|
|
135
|
-
|
|
136
|
-
|
|
137
|
-
|
|
138
|
-
|
|
139
|
-
|
|
140
|
-
|
|
141
|
-
|
|
142
|
-
|
|
143
|
-
|
|
144
|
-
|
|
145
|
-
|
|
146
|
-
|
|
147
|
-
|
|
148
|
-
|
|
149
|
-
|
|
150
|
-
**
|
|
151
|
-
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
|
|
155
|
-
|
|
156
|
-
|
|
157
|
-
|
|
158
|
-
|
|
159
|
-
|
|
160
|
-
|
|
161
|
-
|
|
162
|
-
|
|
163
|
-
|
|
164
|
-
|
|
165
|
-
|
|
166
|
-
|
|
1
|
+
---
|
|
2
|
+
name: state-inconsistency-auditor-swl
|
|
3
|
+
description: >
|
|
4
|
+
Encuentra bugs donde una operación muta un fragmento de estado acoplado sin
|
|
5
|
+
actualizar su contraparte dependiente, causando corrupción silenciosa o fallos
|
|
6
|
+
en operaciones posteriores. Mapea sistemáticamente pares de estado acoplado,
|
|
7
|
+
todos sus paths de mutación, y los gaps donde un lado se actualiza sin el otro.
|
|
8
|
+
Language-agnostic (Python, TS, Go, Rust, Java, C#). Cargar cuando se sospeche
|
|
9
|
+
estado desincronizado: caché obsoleta, índices secundarios huérfanos, contadores
|
|
10
|
+
que no coinciden con la suma de sus partes, o permisos derivados que no reflejan
|
|
11
|
+
el estado de origen.
|
|
12
|
+
version: "1.0.0"
|
|
13
|
+
exclusiones:
|
|
14
|
+
- "No cargar para bugs de lógica en función individual — usar feynman-auditor-swl."
|
|
15
|
+
- "No cargar para auditoría de race conditions en código concurrente — eso requiere análisis específico de threading/async."
|
|
16
|
+
- "No cargar para módulos sin estado mutable o sin pares de estado acoplado — el patrón no aplica."
|
|
17
|
+
- "No cargar para revisión de estilo o legibilidad — usar revisor-codigo-swl."
|
|
18
|
+
---
|
|
19
|
+
|
|
20
|
+
# Cuándo cargar
|
|
21
|
+
|
|
22
|
+
- Sospechas que un caché, índice secundario, o contador agregado está desincronizado.
|
|
23
|
+
- Una función actualiza la tabla principal pero no el índice derivado.
|
|
24
|
+
- Hay paths de "emergencia" o "admin" que modifican estado sin pasar por el flujo normal.
|
|
25
|
+
- Nemesis está corriendo su Pasada 2.
|
|
26
|
+
|
|
27
|
+
# Cuándo NO cargar
|
|
28
|
+
|
|
29
|
+
- Para búsqueda de vulnerabilidades conocidas (inyección SQL, XSS) — usar `revisor-seguridad-swl`.
|
|
30
|
+
- Para revisión de lógica de función individual sin estado acoplado — usar `feynman-auditor-swl`.
|
|
31
|
+
- Para módulos sin persistencia de estado (transformaciones funcionales puras, utilería).
|
|
32
|
+
|
|
33
|
+
---
|
|
34
|
+
|
|
35
|
+
# State Inconsistency Auditor
|
|
36
|
+
|
|
37
|
+
Encuentra bugs donde una operación actualiza State A sin actualizar State B, cuando el invariante del sistema exige que ambos cambien juntos.
|
|
38
|
+
|
|
39
|
+
Los patrones detallados de estado acoplado están en [recursos/coupled-state-patterns.md](recursos/coupled-state-patterns.md).
|
|
40
|
+
|
|
41
|
+
---
|
|
42
|
+
|
|
43
|
+
## El patrón abstracto
|
|
44
|
+
|
|
45
|
+
Todo sistema mantiene **PARES DE ESTADO ACOPLADO**: dos o más valores de almacenamiento que deben mantener una relación (un invariante) entre sí. Cuando cualquier operación cambia un lado del par sin ajustar el otro, el invariante se rompe. Operaciones posteriores que leen ambos valores producen resultados incorrectos.
|
|
46
|
+
|
|
47
|
+
---
|
|
48
|
+
|
|
49
|
+
## Reglas del auditor
|
|
50
|
+
|
|
51
|
+
```
|
|
52
|
+
REGLA 0: MAPEAR ANTES DE CAZAR
|
|
53
|
+
Nunca empezar a revisar funciones sin tener el mapa completo de
|
|
54
|
+
dependencias de estado acoplado. No se puede encontrar una actualización
|
|
55
|
+
faltante si no se sabe qué actualizaciones son necesarias.
|
|
56
|
+
|
|
57
|
+
REGLA 1: TODOS LOS PATHS DE MUTACIÓN IMPORTAN
|
|
58
|
+
Una variable de estado puede modificarse desde 5 funciones distintas.
|
|
59
|
+
Las 5 deben actualizar el estado acoplado. Si 4 lo hacen y 1 no — ese es el bug.
|
|
60
|
+
|
|
61
|
+
REGLA 2: LAS OPERACIONES PARCIALES SON LA FUENTE #1
|
|
62
|
+
Las eliminaciones completas (eliminar todo) generalmente resetean todo el estado
|
|
63
|
+
correctamente. Las operaciones parciales (reducir en X) frecuentemente olvidan
|
|
64
|
+
reducir proporcionalmente el estado acoplado.
|
|
65
|
+
|
|
66
|
+
REGLA 3: COMPARAR PATHS PARALELOS
|
|
67
|
+
Si transferir() y eliminar() ambos reducen un balance, AMBOS deben actualizar
|
|
68
|
+
el mismo conjunto de estado acoplado. Si uno lo hace y el otro no — hallazgo.
|
|
69
|
+
|
|
70
|
+
REGLA 4: EL CÓDIGO DEFENSIVO ENMASCARA BUGS
|
|
71
|
+
Código como `valor > minimo ? valor - minimo : 0` o `min(calculado, disponible)`
|
|
72
|
+
oculta invariantes rotos silenciosamente. Son señales de alerta, no protecciones.
|
|
73
|
+
|
|
74
|
+
REGLA 5: SOLO HALLAZGOS BASADOS EN EVIDENCIA
|
|
75
|
+
Todo hallazgo debe incluir: el par acoplado, la operación que lo rompe,
|
|
76
|
+
una secuencia de trigger concreta, y la consecuencia observable.
|
|
77
|
+
```
|
|
78
|
+
|
|
79
|
+
---
|
|
80
|
+
|
|
81
|
+
## Proceso
|
|
82
|
+
|
|
83
|
+
### Fase 1: Mapear pares de estado acoplado
|
|
84
|
+
|
|
85
|
+
Para cada variable de almacenamiento, preguntar: **"¿Qué otros valores de almacenamiento deben cambiar cuando este cambia?"**
|
|
86
|
+
|
|
87
|
+
Construir un mapa de dependencias. Ver patrones comunes en `recursos/coupled-state-patterns.md`.
|
|
88
|
+
|
|
89
|
+
Salida de Fase 1: **Mapa de Dependencias de Estado Acoplado**.
|
|
90
|
+
|
|
91
|
+
### Fase 2: Matriz de mutación
|
|
92
|
+
|
|
93
|
+
Para cada variable identificada en Fase 1, listar TODAS las funciones y paths de código que la modifican. Incluir: escrituras directas, incrementos/decrementos, eliminaciones, mutaciones indirectas (llamadas internas), triggers externos (callbacks, hooks).
|
|
94
|
+
|
|
95
|
+
Marcar con `???` toda entrada donde no se ha confirmado si la contraparte acoplada también se actualiza.
|
|
96
|
+
|
|
97
|
+
Los `???` son los **objetivos primarios de auditoría**.
|
|
98
|
+
|
|
99
|
+
### Fase 3: Cross-check — el corazón de la auditoría
|
|
100
|
+
|
|
101
|
+
Para cada par (operación, variable de estado) de Fase 2:
|
|
102
|
+
|
|
103
|
+
> "Esta operación modifica State A. ¿También actualiza todo el estado acoplado que depende de A?"
|
|
104
|
+
|
|
105
|
+
Verificar específicamente:
|
|
106
|
+
- Eliminación completa (A → 0): ¿se resetea/limpia todo el estado acoplado?
|
|
107
|
+
- Eliminación parcial (A decrece): ¿se reduce proporcionalmente todo el estado acoplado?
|
|
108
|
+
- Incremento (A crece): ¿se incrementa proporcionalmente todo el estado acoplado?
|
|
109
|
+
- Transferencia (A se mueve entre entidades): ¿el estado acoplado se mueve también?
|
|
110
|
+
- Eliminación de entrada de colección: ¿la entrada del par también se elimina?
|
|
111
|
+
- Operación en lote: ¿el estado acoplado se actualiza por iteración o solo una vez?
|
|
112
|
+
|
|
113
|
+
**Si algún path actualiza A sin actualizar su estado acoplado → HALLAZGO.**
|
|
114
|
+
|
|
115
|
+
### Fase 4: Orden de operaciones dentro de funciones
|
|
116
|
+
|
|
117
|
+
Muchas funciones realizan múltiples cambios de estado en secuencia. Rastrear el orden exacto y preguntar en cada paso: "¿Después de este paso, todos los pares acoplados siguen siendo consistentes?"
|
|
118
|
+
|
|
119
|
+
Bugs de orden comunes:
|
|
120
|
+
- Calcular resultado ANTES de actualizar el índice → resultado usa estado obsoleto
|
|
121
|
+
- Leer caché DESPUÉS de modificar el origen → validación usa datos viejos
|
|
122
|
+
- Emitir evento con valores viejos DESPUÉS del cambio de estado → sistemas externos se desincronizarán
|
|
123
|
+
|
|
124
|
+
### Fase 5: Comparar paths paralelos
|
|
125
|
+
|
|
126
|
+
Encontrar operaciones que logran resultados similares por paths distintos. Para cada grupo, comparar: ¿TODOS los paths actualizan el mismo estado acoplado?
|
|
127
|
+
|
|
128
|
+
### Fase 6: Rastrear journeys multi-paso
|
|
129
|
+
|
|
130
|
+
Simular secuencias donde un actor interactúa múltiples veces:
|
|
131
|
+
|
|
132
|
+
1. Actor inicializa un recurso (estado inicializado)
|
|
133
|
+
2. Tiempo pasa / estado externo evoluciona
|
|
134
|
+
3. Actor hace modificación PARCIAL (el estado acoplado puede romperse aquí)
|
|
135
|
+
4. Más tiempo pasa
|
|
136
|
+
5. Actor hace otra operación que lee el estado acoplado
|
|
137
|
+
|
|
138
|
+
En el paso 5: ¿el estado acoplado sigue siendo válido dado el cambio parcial del paso 3?
|
|
139
|
+
|
|
140
|
+
### Fase 7: Patrones de enmascaramiento
|
|
141
|
+
|
|
142
|
+
El código defensivo puede ocultar invariantes rotos. Ver patrones en `recursos/coupled-state-patterns.md`.
|
|
143
|
+
|
|
144
|
+
Señal: si un ternario del tipo `a > b ? a - b : 0` está en un cálculo que involucra dos valores acoplados, preguntar: ¿por qué `a` podría ser menor que `b`? Si el invariante se mantuviera, no podría.
|
|
145
|
+
|
|
146
|
+
---
|
|
147
|
+
|
|
148
|
+
## Verificación (obligatoria para C/A/M)
|
|
149
|
+
|
|
150
|
+
**Método A: Rastreo de código** — leer la función exacta, rastrear todas las llamadas internas, confirmar que no existe actualización oculta al estado acoplado.
|
|
151
|
+
|
|
152
|
+
**Método B: Test de PoC** — escribir un test que ejecute la secuencia de trigger y afirme que el estado es inconsistente después de la operación.
|
|
153
|
+
|
|
154
|
+
**Método C: Híbrido** — rastreo + PoC para hallazgos que abarcan múltiples módulos.
|
|
155
|
+
|
|
156
|
+
**Falsos positivos frecuentes:**
|
|
157
|
+
- Reconciliación oculta en un hook `_before_save` / `_after_update` / middleware.
|
|
158
|
+
- Evaluación diferida intencional: el estado acoplado se reconcilia en la próxima lectura.
|
|
159
|
+
- Asimetría de diseño documentada: los dos estados no son realmente acoplados como se asumió.
|
|
160
|
+
|
|
161
|
+
---
|
|
162
|
+
|
|
163
|
+
## Adaptación por lenguaje
|
|
164
|
+
|
|
165
|
+
| Concepto | Python | TypeScript | Go | Rust | Java | C# |
|
|
166
|
+
|---------|--------|------------|-----|------|------|-----|
|
|
167
|
+
| Almacenamiento | atributos / BD | propiedades / BD | campos struct / BD | campos struct | campos clase | propiedades |
|
|
168
|
+
| Colección clave-valor | `dict` / `HashMap` Redis | `Map` / objeto | `map[K]V` | `HashMap` | `Map<K,V>` | `Dictionary<K,V>` |
|
|
169
|
+
| Eliminar entrada | `del d[k]` / `pop` | `delete obj[k]` | `delete(m, k)` | `map.remove(&k)` | `map.remove(k)` | `dict.Remove(k)` |
|
|
170
|
+
| Hook post-mutación | `@receiver` / signal | middleware / hook | middleware | `impl Drop` / hook | `@PostUpdate` | event handler |
|
|
171
|
+
|
|
172
|
+
<!-- Adaptado de nemesis-auditor-main bajo MIT License (https://github.com/0xiehnnkta/nemesis-auditor) -->
|
|
@@ -1,12 +1,12 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: tdd-workflow
|
|
3
3
|
description: Flujo completo de Test-Driven Development. Ciclo RED (el test falla) → GREEN (implementación mínima) → REFACTOR (limpieza). Incluye cobertura mínima obligatoria, tests de frontera, factories, fixtures y estrategias para diferentes tipos de código (APIs, services, componentes Angular).
|
|
4
|
-
version: "1.0.
|
|
4
|
+
version: "1.0.5"
|
|
5
5
|
evolved: true
|
|
6
|
-
evolved-from: "1.0.
|
|
6
|
+
evolved-from: "1.0.4"
|
|
7
7
|
evolved-at: "2026-05-16"
|
|
8
8
|
evolved-by: "aprender"
|
|
9
|
-
evolved-note: "v1.0.
|
|
9
|
+
evolved-note: "v1.0.5: gotcha 'Tests E2E de CLIs interactivos sin PTY real'. Origen M2 sesión 2026-05-16 — harness TTY mockeado en swl-ses v1.6.0 que evita instalar node-pty. v1.0.4: silenced tests por race en path único compartido + anti-patrón `if (X) assert(Y)` sin else. v1.0.3: gotcha cwd cacheado al require()."
|
|
10
10
|
herramientasPermitidas: [Read, Bash]
|
|
11
11
|
evolvable: true # default para skill estandar
|
|
12
12
|
exclusiones:
|
|
@@ -476,3 +476,178 @@ flag `swl-ses-update-check.json` compartido entre dos archivos `.test.js`
|
|
|
476
476
|
paralelos. El test "sin flag → debe advertir" pasaba en CI cuando otro
|
|
477
477
|
archivo creaba el flag, sin ejecutar ninguna assertion. Fix: env var
|
|
478
478
|
`SWL_UPDATE_FLAG_PATH` para aislamiento + assertions incondicionales.
|
|
479
|
+
|
|
480
|
+
---
|
|
481
|
+
|
|
482
|
+
## Tests E2E de CLIs interactivos sin PTY real
|
|
483
|
+
|
|
484
|
+
### El problema
|
|
485
|
+
|
|
486
|
+
Probar un CLI interactivo (TUI con `readline`, prompts, keypress events,
|
|
487
|
+
`process.stdin.isTTY`) en CI requiere normalmente un **pseudo-terminal
|
|
488
|
+
emulado** (PTY) — usualmente vía `node-pty`, una dependencia **nativa** que:
|
|
489
|
+
|
|
490
|
+
- Requiere compilación de extensiones C++ al instalar (puede fallar en
|
|
491
|
+
contenedores minimal o en Windows sin Visual Studio Build Tools).
|
|
492
|
+
- Agrega ~5 MB al `node_modules` por imagen.
|
|
493
|
+
- Hace que el test suite no corra en `npm test` sin setup extra.
|
|
494
|
+
|
|
495
|
+
Para CLIs interactivos donde no se necesita probar **el comportamiento
|
|
496
|
+
real del terminal** (escape codes, redibujado, scroll), sino solo la
|
|
497
|
+
**lógica del wizard** (¿qué pasa si el usuario presiona Esc en el paso 3?,
|
|
498
|
+
¿qué resuelve el promise tras Enter con default?), un harness TTY mockeado
|
|
499
|
+
cubre ~90% de los casos sin dep nativa.
|
|
500
|
+
|
|
501
|
+
### Patrón del harness
|
|
502
|
+
|
|
503
|
+
```javascript
|
|
504
|
+
// tests/harness-tty.js
|
|
505
|
+
'use strict';
|
|
506
|
+
|
|
507
|
+
const readline = require('readline');
|
|
508
|
+
|
|
509
|
+
function crearHarness() {
|
|
510
|
+
// 1. Capturar estado original para restauración
|
|
511
|
+
const stdoutOriginal = process.stdout.write.bind(process.stdout);
|
|
512
|
+
const isTtyStdoutOriginal = process.stdout.isTTY;
|
|
513
|
+
const isTtyStdinOriginal = process.stdin.isTTY;
|
|
514
|
+
const setRawModeOriginal = process.stdin.setRawMode
|
|
515
|
+
? process.stdin.setRawMode.bind(process.stdin) : null;
|
|
516
|
+
const emitKeypressOriginal = readline.emitKeypressEvents;
|
|
517
|
+
|
|
518
|
+
let capturado = '';
|
|
519
|
+
let listenersKeypress = [];
|
|
520
|
+
|
|
521
|
+
// 2. Forzar TTY antes de cargar módulos UI (que evalúan ES_TTY al require)
|
|
522
|
+
Object.defineProperty(process.stdout, 'isTTY', { value: true, configurable: true });
|
|
523
|
+
Object.defineProperty(process.stdin, 'isTTY', { value: true, configurable: true });
|
|
524
|
+
|
|
525
|
+
// 3. Capturar stdout en string buffer
|
|
526
|
+
process.stdout.write = (chunk) => {
|
|
527
|
+
capturado += typeof chunk === 'string' ? chunk : chunk.toString();
|
|
528
|
+
return true;
|
|
529
|
+
};
|
|
530
|
+
|
|
531
|
+
// 4. Mockear setRawMode/resume/pause como no-op (evita tomar control del terminal de test)
|
|
532
|
+
process.stdin.setRawMode = () => process.stdin;
|
|
533
|
+
process.stdin.resume = () => process.stdin;
|
|
534
|
+
process.stdin.pause = () => process.stdin;
|
|
535
|
+
|
|
536
|
+
// 5. Interceptar registros de 'keypress' para poder emitirlos a mano
|
|
537
|
+
const onListenerOriginal = process.stdin.on.bind(process.stdin);
|
|
538
|
+
process.stdin.on = (evento, listener) => {
|
|
539
|
+
if (evento === 'keypress') listenersKeypress.push(listener);
|
|
540
|
+
return onListenerOriginal(evento, listener);
|
|
541
|
+
};
|
|
542
|
+
|
|
543
|
+
// 6. Mockear readline.emitKeypressEvents (no necesita stdin real)
|
|
544
|
+
readline.emitKeypressEvents = (stream) => stream;
|
|
545
|
+
|
|
546
|
+
// 7. Limpiar require cache de módulos UI para que se evalúen con TTY=true
|
|
547
|
+
delete require.cache[require.resolve('../scripts/tui/lib/render')];
|
|
548
|
+
|
|
549
|
+
function cargarUI() {
|
|
550
|
+
return require('../scripts/tui/lib/render');
|
|
551
|
+
}
|
|
552
|
+
|
|
553
|
+
// 8. Emitir keypress programáticamente
|
|
554
|
+
function tecla(nombre, extras = {}) {
|
|
555
|
+
const key = { name: nombre, ctrl: false, meta: false, shift: false, ...extras };
|
|
556
|
+
const str = nombre.length === 1 ? nombre : '';
|
|
557
|
+
for (const listener of [...listenersKeypress]) {
|
|
558
|
+
try { listener(str, key); } catch (_) { /* swallow */ }
|
|
559
|
+
}
|
|
560
|
+
}
|
|
561
|
+
|
|
562
|
+
// 9. Esperar N ticks del event loop para promesas internas
|
|
563
|
+
function esperarTicks(n = 1) {
|
|
564
|
+
let p = Promise.resolve();
|
|
565
|
+
for (let i = 0; i < n; i++) p = p.then(() => undefined);
|
|
566
|
+
return p;
|
|
567
|
+
}
|
|
568
|
+
|
|
569
|
+
function captura(opts = {}) {
|
|
570
|
+
const valor = capturado;
|
|
571
|
+
if (opts.limpiar) capturado = '';
|
|
572
|
+
return valor;
|
|
573
|
+
}
|
|
574
|
+
|
|
575
|
+
function restaurar() {
|
|
576
|
+
Object.defineProperty(process.stdout, 'isTTY', { value: isTtyStdoutOriginal, configurable: true });
|
|
577
|
+
Object.defineProperty(process.stdin, 'isTTY', { value: isTtyStdinOriginal, configurable: true });
|
|
578
|
+
process.stdout.write = stdoutOriginal;
|
|
579
|
+
if (setRawModeOriginal) process.stdin.setRawMode = setRawModeOriginal;
|
|
580
|
+
process.stdin.on = onListenerOriginal;
|
|
581
|
+
readline.emitKeypressEvents = emitKeypressOriginal;
|
|
582
|
+
listenersKeypress = [];
|
|
583
|
+
// Limpiar require cache para no contaminar otros tests
|
|
584
|
+
delete require.cache[require.resolve('../scripts/tui/lib/render')];
|
|
585
|
+
}
|
|
586
|
+
|
|
587
|
+
return { cargarUI, tecla, esperarTicks, captura, restaurar };
|
|
588
|
+
}
|
|
589
|
+
|
|
590
|
+
module.exports = { crearHarness };
|
|
591
|
+
```
|
|
592
|
+
|
|
593
|
+
### Uso típico
|
|
594
|
+
|
|
595
|
+
```javascript
|
|
596
|
+
const test = require('node:test');
|
|
597
|
+
const assert = require('node:assert/strict');
|
|
598
|
+
const { crearHarness } = require('./harness-tty');
|
|
599
|
+
|
|
600
|
+
test('preguntarSiNo con harness: Enter resuelve con default true', async () => {
|
|
601
|
+
const h = crearHarness();
|
|
602
|
+
try {
|
|
603
|
+
const ui = h.cargarUI();
|
|
604
|
+
const promesa = ui.preguntarSiNo('test prompt', true);
|
|
605
|
+
|
|
606
|
+
await h.esperarTicks(2);
|
|
607
|
+
h.tecla('return');
|
|
608
|
+
|
|
609
|
+
const timeout = new Promise((_, reject) =>
|
|
610
|
+
setTimeout(() => reject(new Error('no resolvió en 500ms')), 500));
|
|
611
|
+
|
|
612
|
+
const r = await Promise.race([promesa, timeout]).catch(() => null);
|
|
613
|
+
// r === true si el harness simuló bien; null si readline real bloquea
|
|
614
|
+
// (caso esperado en Windows sin PTY real; documentar limitación)
|
|
615
|
+
} finally {
|
|
616
|
+
h.restaurar();
|
|
617
|
+
}
|
|
618
|
+
});
|
|
619
|
+
```
|
|
620
|
+
|
|
621
|
+
### Reglas operativas
|
|
622
|
+
|
|
623
|
+
- **`restaurar()` en `finally`**: el harness modifica state global
|
|
624
|
+
(process.stdout, process.stdin, readline, require.cache). Si un test
|
|
625
|
+
no restaura, contamina los siguientes.
|
|
626
|
+
- **Test de captura como smoke**: agregar un test "harness captura stdout"
|
|
627
|
+
que valida que `process.stdout.write('hola')` aparece en `captura()`.
|
|
628
|
+
Si falla, el harness está roto antes de testear el SUT.
|
|
629
|
+
- **Test "tecla() es no-op sin listeners"**: validar que emitir keypress
|
|
630
|
+
cuando nadie escucha NO rompe el harness ni propaga errores.
|
|
631
|
+
- **Limitación reconocida**: si `readline.createInterface()` real toma
|
|
632
|
+
control de stdin (en Windows con Git Bash sin PTY), el callback de
|
|
633
|
+
`rl.question()` no se invoca aunque el harness emita teclas. Usar
|
|
634
|
+
`Promise.race([promesa, timeout])` para que el test no cuelgue —
|
|
635
|
+
el test marca limitación, no falla.
|
|
636
|
+
|
|
637
|
+
### Cuándo NO usar este patrón
|
|
638
|
+
|
|
639
|
+
- Cuando necesitas probar **redibujado real del terminal** (alt screen
|
|
640
|
+
buffer, escape codes complejos, scrollback). Ahí sí necesitas PTY real
|
|
641
|
+
via `node-pty` o test manual.
|
|
642
|
+
- Cuando el SUT depende de **timing real del teclado** (input rates,
|
|
643
|
+
paste detection). El mock no replica latencia.
|
|
644
|
+
- Para CLIs sin lógica de control de flujo (solo `console.log` lineal) —
|
|
645
|
+
ahí basta capturar stdout sin mockear TTY.
|
|
646
|
+
|
|
647
|
+
### Origen
|
|
648
|
+
|
|
649
|
+
Aplicado en swl-ses v1.6.0 (`tests/scripts/tui/harness-tty.js`, ~180 LOC).
|
|
650
|
+
Validó el TUI completo de 5 fases sin instalar `node-pty`. Limitación
|
|
651
|
+
documentada: 1 test E2E "preguntarSiNo con harness" marca timeout en
|
|
652
|
+
Windows + Node 22+ porque readline real bloquea pese a stdin mockeado —
|
|
653
|
+
el harness emite la limitación sin fallar.
|
|
@@ -5,6 +5,7 @@ description: >
|
|
|
5
5
|
el comando de prueba, leer la salida completa y verificar antes de cualquier
|
|
6
6
|
afirmación positiva. Cargar SIEMPRE al terminar una tarea o slice, antes de
|
|
7
7
|
reportar éxito al usuario o al orquestador.
|
|
8
|
+
version: "1.0.0"
|
|
8
9
|
herramientasPermitidas: [Read, Write, Edit, Bash, Glob, Grep]
|
|
9
10
|
exclusiones:
|
|
10
11
|
- "No cargar como sustituto de `verificar-trabajo` — este skill es el gate de afirmaciones puntuales, no la verificación end-to-end de un plan completo."
|