@saulwade/swl-ses 1.4.0 → 1.4.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 +4 -3
- package/README.md +15 -14
- package/agentes/nemesis-auditor-swl.md +161 -0
- package/comandos/swl/nemesis.md +122 -0
- package/comandos/swl/salud.md +34 -0
- package/comandos/swl/verificar.md +45 -0
- package/habilidades/feynman-auditor-swl/SKILL.md +123 -0
- package/habilidades/feynman-auditor-swl/recursos/preguntas-language-agnostic.md +108 -0
- package/habilidades/state-inconsistency-auditor-swl/SKILL.md +166 -0
- package/habilidades/state-inconsistency-auditor-swl/recursos/coupled-state-patterns.md +147 -0
- package/habilidades/web-fetcher-routing/SKILL.md +75 -0
- package/hooks/lib/security-net.js +201 -0
- package/manifiestos/modulos.json +30 -0
- package/manifiestos/skills-lock.json +1114 -1093
- package/package.json +2 -2
- package/plugin.json +2 -2
- package/scripts/audit-tools/audit-history.js +330 -0
- package/scripts/audit-tools/bundle-tracker.js +290 -0
- package/scripts/audit-tools/canary-monitor.js +352 -0
- package/scripts/audit-tools/code-profiler.js +605 -0
- package/scripts/audit-tools/dep-doctor.js +320 -0
- package/scripts/audit-tools/env-validator.js +206 -0
- package/scripts/audit-tools/lib/fs-walk.js +48 -0
- package/scripts/audit-tools/lib/output.js +23 -0
- package/scripts/audit-tools/migration-checker.js +392 -0
- package/scripts/audit-tools/pentest-scanner.js +1436 -0
|
@@ -0,0 +1,108 @@
|
|
|
1
|
+
# Preguntas Feynman — Language-Agnostic
|
|
2
|
+
|
|
3
|
+
28 preguntas organizadas en 7 categorías. Aplicar a cada función no trivial del módulo bajo auditoría.
|
|
4
|
+
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
## Q1 — Propósito
|
|
8
|
+
|
|
9
|
+
**Q1.1** ¿Qué hace exactamente esta función y solo esta función? Si no puedo describirla en una oración sin usar "y", ¿tiene demasiadas responsabilidades?
|
|
10
|
+
|
|
11
|
+
**Q1.2** ¿Cuál es el invariante que esta función garantiza al terminar? ¿El código lo garantiza de verdad o solo en el camino feliz?
|
|
12
|
+
|
|
13
|
+
**Q1.3** ¿Hay alguna condición en que esta función debería rechazar la operación pero no lo hace? ¿Qué pasa si se la llama dos veces seguidas con los mismos argumentos?
|
|
14
|
+
|
|
15
|
+
**Q1.4** ¿Esta función asume que el estado del sistema está en alguna condición particular antes de ejecutarse? ¿Dónde se verifica esa condición?
|
|
16
|
+
|
|
17
|
+
---
|
|
18
|
+
|
|
19
|
+
## Q2 — Orden de operaciones
|
|
20
|
+
|
|
21
|
+
**Q2.1** ¿Importa el orden en que estas líneas se ejecutan? Si cambio el orden de dos líneas adyacentes, ¿el resultado es el mismo?
|
|
22
|
+
|
|
23
|
+
**Q2.2** ¿Esta función lee un valor y luego lo modifica? Si algo cambia el valor entre la lectura y la escritura, ¿qué pasa?
|
|
24
|
+
|
|
25
|
+
**Q2.3** ¿El cálculo de recompensa, crédito, o resultado se hace ANTES o DESPUÉS de actualizar el estado base? ¿Debería ser al revés?
|
|
26
|
+
|
|
27
|
+
**Q2.4** ¿Si esta función llama a un sistema externo (BD, API, caché), el estado local ya está actualizado en ese momento? ¿El sistema externo puede ver un estado inconsistente?
|
|
28
|
+
|
|
29
|
+
---
|
|
30
|
+
|
|
31
|
+
## Q3 — Consistencia cross-función
|
|
32
|
+
|
|
33
|
+
**Q3.1** ¿Qué tienen en común esta función y otra que hace algo similar? ¿Una tiene un guard o una actualización que la otra no tiene?
|
|
34
|
+
|
|
35
|
+
**Q3.2** ¿Existe una función de "camino normal" y una de "camino de emergencia/admin"? ¿Ambas actualizan exactamente el mismo conjunto de estado?
|
|
36
|
+
|
|
37
|
+
**Q3.3** ¿Esta función inicializa todos los campos que otras funciones esperan encontrar inicializados? ¿Hay algún campo que se lee antes de ser escrito?
|
|
38
|
+
|
|
39
|
+
**Q3.4** ¿El path que crea un recurso y el path que lo elimina dejan el sistema en un estado simétrico? ¿O el delete deja huérfanos?
|
|
40
|
+
|
|
41
|
+
---
|
|
42
|
+
|
|
43
|
+
## Q4 — Asunciones implícitas
|
|
44
|
+
|
|
45
|
+
**Q4.1** ¿Qué asume este código sobre el tipo o el rango del argumento que recibe? ¿Hay alguna validación que debería existir pero no existe?
|
|
46
|
+
|
|
47
|
+
**Q4.2** ¿El código asume que el actor que llama esta función tiene permiso para hacerlo? ¿Dónde se verifica ese permiso exactamente?
|
|
48
|
+
|
|
49
|
+
**Q4.3** ¿El código asume que algún recurso externo (BD, archivo, servicio) está disponible y en un estado consistente? ¿Qué pasa si no lo está?
|
|
50
|
+
|
|
51
|
+
**Q4.4** ¿El código asume que nunca se llamará con valor cero, lista vacía, o string vacío? ¿Se validó eso?
|
|
52
|
+
|
|
53
|
+
---
|
|
54
|
+
|
|
55
|
+
## Q5 — Límites y casos de borde
|
|
56
|
+
|
|
57
|
+
**Q5.1** ¿Qué pasa si el valor es exactamente cero? ¿Y si es el máximo representable? ¿Y si es negativo?
|
|
58
|
+
|
|
59
|
+
**Q5.2** ¿Qué pasa si la lista o colección está vacía? ¿Y si tiene exactamente un elemento? ¿Y si tiene millones?
|
|
60
|
+
|
|
61
|
+
**Q5.3** ¿Qué pasa si el recurso ya existe cuando se intenta crear? ¿Y si no existe cuando se intenta modificar o eliminar?
|
|
62
|
+
|
|
63
|
+
**Q5.4** ¿Qué pasa si dos actores ejecutan esta función al mismo tiempo? ¿Hay condición de carrera? ¿El resultado es determinista?
|
|
64
|
+
|
|
65
|
+
---
|
|
66
|
+
|
|
67
|
+
## Q6 — Returns y manejo de errores
|
|
68
|
+
|
|
69
|
+
**Q6.1** ¿Qué devuelve esta función cuando el input es inválido? ¿Lanza excepción, devuelve null, devuelve un error tipado, o silencia el problema?
|
|
70
|
+
|
|
71
|
+
**Q6.2** ¿Si una operación parcial falla a mitad de camino, el estado queda consistente? ¿Hay rollback? ¿O el sistema queda a medias?
|
|
72
|
+
|
|
73
|
+
**Q6.3** ¿El caller de esta función verifica el valor de retorno? ¿Un error no verificado puede causar daño silencioso más adelante?
|
|
74
|
+
|
|
75
|
+
**Q6.4** ¿El código captura excepciones de forma demasiado amplia (`except Exception`, `catch (e)` vacío)? ¿Qué errores legítimos podría estar silenciando?
|
|
76
|
+
|
|
77
|
+
---
|
|
78
|
+
|
|
79
|
+
## Q7 — Interacciones externas
|
|
80
|
+
|
|
81
|
+
**Q7.1** ¿Esta función llama a un sistema externo (BD, API, caché, cola)? ¿La llamada puede fallar? ¿Qué pasa con el estado local si falla?
|
|
82
|
+
|
|
83
|
+
**Q7.2** ¿Esta función persiste datos en más de un lugar (ej: BD + caché + índice)? ¿Qué pasa si la segunda escritura falla después de que la primera tuvo éxito?
|
|
84
|
+
|
|
85
|
+
**Q7.3** ¿El acumulador o índice que esta función usa se actualiza ANTES de modificar el valor base, o DESPUÉS? ¿El orden importa para la correctitud del cálculo?
|
|
86
|
+
|
|
87
|
+
**Q7.4** ¿Esta función envía una notificación, evento, o mensaje a otro sistema? ¿Lo hace antes o después de confirmar la operación? ¿El receptor puede actuar sobre un estado que aún no se confirmó?
|
|
88
|
+
|
|
89
|
+
**Q7.5** ¿Esta función lee un valor de caché y lo trata como fresco? ¿Hay algún path donde la caché puede estar desactualizada respecto a la fuente de verdad?
|
|
90
|
+
|
|
91
|
+
**Q7.6** ¿Esta función modifica datos de otro actor además de los del actor que la llama? ¿Tiene permiso para hacerlo en todos los casos?
|
|
92
|
+
|
|
93
|
+
**Q7.7** ¿Existe un patrón de acumulador donde el total acumulado se calcula con base en el estado anterior? Si el estado base cambia (insert/update/delete de otro registro), ¿el acumulador refleja el cambio correctamente?
|
|
94
|
+
|
|
95
|
+
**Q7.8** ¿Esta función delega a otra función que tiene sus propios efectos secundarios? ¿El caller sabe qué estado modifica el callee internamente?
|
|
96
|
+
|
|
97
|
+
---
|
|
98
|
+
|
|
99
|
+
## Cómo usar estas preguntas
|
|
100
|
+
|
|
101
|
+
1. Leer la función completa una vez para entender el propósito.
|
|
102
|
+
2. Para cada pregunta de las categorías Q1–Q7, marcar:
|
|
103
|
+
- ✓ Respondida claramente por el código
|
|
104
|
+
- ? Requiere rastrear el caller o el sistema externo para responder
|
|
105
|
+
- ✗ El código no la responde — sospechoso de bug
|
|
106
|
+
3. Todo `✗` o `?` sin resolver es un **sospechoso** para Fase 3 de síntesis.
|
|
107
|
+
|
|
108
|
+
<!-- Adaptado de nemesis-auditor-main bajo MIT License (https://github.com/0xiehnnkta/nemesis-auditor) -->
|
|
@@ -0,0 +1,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
|
+
---
|
|
13
|
+
|
|
14
|
+
# Cuándo cargar
|
|
15
|
+
|
|
16
|
+
- Sospechas que un caché, índice secundario, o contador agregado está desincronizado.
|
|
17
|
+
- Una función actualiza la tabla principal pero no el índice derivado.
|
|
18
|
+
- Hay paths de "emergencia" o "admin" que modifican estado sin pasar por el flujo normal.
|
|
19
|
+
- Nemesis está corriendo su Pasada 2.
|
|
20
|
+
|
|
21
|
+
# Cuándo NO cargar
|
|
22
|
+
|
|
23
|
+
- Para búsqueda de vulnerabilidades conocidas (inyección SQL, XSS) — usar `revisor-seguridad-swl`.
|
|
24
|
+
- Para revisión de lógica de función individual sin estado acoplado — usar `feynman-auditor-swl`.
|
|
25
|
+
- Para módulos sin persistencia de estado (transformaciones funcionales puras, utilería).
|
|
26
|
+
|
|
27
|
+
---
|
|
28
|
+
|
|
29
|
+
# State Inconsistency Auditor
|
|
30
|
+
|
|
31
|
+
Encuentra bugs donde una operación actualiza State A sin actualizar State B, cuando el invariante del sistema exige que ambos cambien juntos.
|
|
32
|
+
|
|
33
|
+
Los patrones detallados de estado acoplado están en [recursos/coupled-state-patterns.md](recursos/coupled-state-patterns.md).
|
|
34
|
+
|
|
35
|
+
---
|
|
36
|
+
|
|
37
|
+
## El patrón abstracto
|
|
38
|
+
|
|
39
|
+
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.
|
|
40
|
+
|
|
41
|
+
---
|
|
42
|
+
|
|
43
|
+
## Reglas del auditor
|
|
44
|
+
|
|
45
|
+
```
|
|
46
|
+
REGLA 0: MAPEAR ANTES DE CAZAR
|
|
47
|
+
Nunca empezar a revisar funciones sin tener el mapa completo de
|
|
48
|
+
dependencias de estado acoplado. No se puede encontrar una actualización
|
|
49
|
+
faltante si no se sabe qué actualizaciones son necesarias.
|
|
50
|
+
|
|
51
|
+
REGLA 1: TODOS LOS PATHS DE MUTACIÓN IMPORTAN
|
|
52
|
+
Una variable de estado puede modificarse desde 5 funciones distintas.
|
|
53
|
+
Las 5 deben actualizar el estado acoplado. Si 4 lo hacen y 1 no — ese es el bug.
|
|
54
|
+
|
|
55
|
+
REGLA 2: LAS OPERACIONES PARCIALES SON LA FUENTE #1
|
|
56
|
+
Las eliminaciones completas (eliminar todo) generalmente resetean todo el estado
|
|
57
|
+
correctamente. Las operaciones parciales (reducir en X) frecuentemente olvidan
|
|
58
|
+
reducir proporcionalmente el estado acoplado.
|
|
59
|
+
|
|
60
|
+
REGLA 3: COMPARAR PATHS PARALELOS
|
|
61
|
+
Si transferir() y eliminar() ambos reducen un balance, AMBOS deben actualizar
|
|
62
|
+
el mismo conjunto de estado acoplado. Si uno lo hace y el otro no — hallazgo.
|
|
63
|
+
|
|
64
|
+
REGLA 4: EL CÓDIGO DEFENSIVO ENMASCARA BUGS
|
|
65
|
+
Código como `valor > minimo ? valor - minimo : 0` o `min(calculado, disponible)`
|
|
66
|
+
oculta invariantes rotos silenciosamente. Son señales de alerta, no protecciones.
|
|
67
|
+
|
|
68
|
+
REGLA 5: SOLO HALLAZGOS BASADOS EN EVIDENCIA
|
|
69
|
+
Todo hallazgo debe incluir: el par acoplado, la operación que lo rompe,
|
|
70
|
+
una secuencia de trigger concreta, y la consecuencia observable.
|
|
71
|
+
```
|
|
72
|
+
|
|
73
|
+
---
|
|
74
|
+
|
|
75
|
+
## Proceso
|
|
76
|
+
|
|
77
|
+
### Fase 1: Mapear pares de estado acoplado
|
|
78
|
+
|
|
79
|
+
Para cada variable de almacenamiento, preguntar: **"¿Qué otros valores de almacenamiento deben cambiar cuando este cambia?"**
|
|
80
|
+
|
|
81
|
+
Construir un mapa de dependencias. Ver patrones comunes en `recursos/coupled-state-patterns.md`.
|
|
82
|
+
|
|
83
|
+
Salida de Fase 1: **Mapa de Dependencias de Estado Acoplado**.
|
|
84
|
+
|
|
85
|
+
### Fase 2: Matriz de mutación
|
|
86
|
+
|
|
87
|
+
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).
|
|
88
|
+
|
|
89
|
+
Marcar con `???` toda entrada donde no se ha confirmado si la contraparte acoplada también se actualiza.
|
|
90
|
+
|
|
91
|
+
Los `???` son los **objetivos primarios de auditoría**.
|
|
92
|
+
|
|
93
|
+
### Fase 3: Cross-check — el corazón de la auditoría
|
|
94
|
+
|
|
95
|
+
Para cada par (operación, variable de estado) de Fase 2:
|
|
96
|
+
|
|
97
|
+
> "Esta operación modifica State A. ¿También actualiza todo el estado acoplado que depende de A?"
|
|
98
|
+
|
|
99
|
+
Verificar específicamente:
|
|
100
|
+
- Eliminación completa (A → 0): ¿se resetea/limpia todo el estado acoplado?
|
|
101
|
+
- Eliminación parcial (A decrece): ¿se reduce proporcionalmente todo el estado acoplado?
|
|
102
|
+
- Incremento (A crece): ¿se incrementa proporcionalmente todo el estado acoplado?
|
|
103
|
+
- Transferencia (A se mueve entre entidades): ¿el estado acoplado se mueve también?
|
|
104
|
+
- Eliminación de entrada de colección: ¿la entrada del par también se elimina?
|
|
105
|
+
- Operación en lote: ¿el estado acoplado se actualiza por iteración o solo una vez?
|
|
106
|
+
|
|
107
|
+
**Si algún path actualiza A sin actualizar su estado acoplado → HALLAZGO.**
|
|
108
|
+
|
|
109
|
+
### Fase 4: Orden de operaciones dentro de funciones
|
|
110
|
+
|
|
111
|
+
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?"
|
|
112
|
+
|
|
113
|
+
Bugs de orden comunes:
|
|
114
|
+
- Calcular resultado ANTES de actualizar el índice → resultado usa estado obsoleto
|
|
115
|
+
- Leer caché DESPUÉS de modificar el origen → validación usa datos viejos
|
|
116
|
+
- Emitir evento con valores viejos DESPUÉS del cambio de estado → sistemas externos se desincronizarán
|
|
117
|
+
|
|
118
|
+
### Fase 5: Comparar paths paralelos
|
|
119
|
+
|
|
120
|
+
Encontrar operaciones que logran resultados similares por paths distintos. Para cada grupo, comparar: ¿TODOS los paths actualizan el mismo estado acoplado?
|
|
121
|
+
|
|
122
|
+
### Fase 6: Rastrear journeys multi-paso
|
|
123
|
+
|
|
124
|
+
Simular secuencias donde un actor interactúa múltiples veces:
|
|
125
|
+
|
|
126
|
+
1. Actor inicializa un recurso (estado inicializado)
|
|
127
|
+
2. Tiempo pasa / estado externo evoluciona
|
|
128
|
+
3. Actor hace modificación PARCIAL (el estado acoplado puede romperse aquí)
|
|
129
|
+
4. Más tiempo pasa
|
|
130
|
+
5. Actor hace otra operación que lee el estado acoplado
|
|
131
|
+
|
|
132
|
+
En el paso 5: ¿el estado acoplado sigue siendo válido dado el cambio parcial del paso 3?
|
|
133
|
+
|
|
134
|
+
### Fase 7: Patrones de enmascaramiento
|
|
135
|
+
|
|
136
|
+
El código defensivo puede ocultar invariantes rotos. Ver patrones en `recursos/coupled-state-patterns.md`.
|
|
137
|
+
|
|
138
|
+
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.
|
|
139
|
+
|
|
140
|
+
---
|
|
141
|
+
|
|
142
|
+
## Verificación (obligatoria para C/A/M)
|
|
143
|
+
|
|
144
|
+
**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.
|
|
145
|
+
|
|
146
|
+
**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.
|
|
147
|
+
|
|
148
|
+
**Método C: Híbrido** — rastreo + PoC para hallazgos que abarcan múltiples módulos.
|
|
149
|
+
|
|
150
|
+
**Falsos positivos frecuentes:**
|
|
151
|
+
- Reconciliación oculta en un hook `_before_save` / `_after_update` / middleware.
|
|
152
|
+
- Evaluación diferida intencional: el estado acoplado se reconcilia en la próxima lectura.
|
|
153
|
+
- Asimetría de diseño documentada: los dos estados no son realmente acoplados como se asumió.
|
|
154
|
+
|
|
155
|
+
---
|
|
156
|
+
|
|
157
|
+
## Adaptación por lenguaje
|
|
158
|
+
|
|
159
|
+
| Concepto | Python | TypeScript | Go | Rust | Java | C# |
|
|
160
|
+
|---------|--------|------------|-----|------|------|-----|
|
|
161
|
+
| Almacenamiento | atributos / BD | propiedades / BD | campos struct / BD | campos struct | campos clase | propiedades |
|
|
162
|
+
| Colección clave-valor | `dict` / `HashMap` Redis | `Map` / objeto | `map[K]V` | `HashMap` | `Map<K,V>` | `Dictionary<K,V>` |
|
|
163
|
+
| Eliminar entrada | `del d[k]` / `pop` | `delete obj[k]` | `delete(m, k)` | `map.remove(&k)` | `map.remove(k)` | `dict.Remove(k)` |
|
|
164
|
+
| Hook post-mutación | `@receiver` / signal | middleware / hook | middleware | `impl Drop` / hook | `@PostUpdate` | event handler |
|
|
165
|
+
|
|
166
|
+
<!-- Adaptado de nemesis-auditor-main bajo MIT License (https://github.com/0xiehnnkta/nemesis-auditor) -->
|
|
@@ -0,0 +1,147 @@
|
|
|
1
|
+
# Patrones de Estado Acoplado — Language-Agnostic
|
|
2
|
+
|
|
3
|
+
9 patrones de estado acoplado frecuentes en sistemas Python, TypeScript, Go, Rust, Java y C#.
|
|
4
|
+
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
## Patrón 1: Balance ↔ Caché derivada
|
|
8
|
+
|
|
9
|
+
**Descripción**: un valor base (balance de cuenta, stock de inventario, crédito disponible) tiene una caché calculada a partir de él (porcentaje disponible, categoría de riesgo, tier de cliente).
|
|
10
|
+
|
|
11
|
+
**Invariante**: la caché debe reflejar el balance actual, no el del momento en que se calculó.
|
|
12
|
+
|
|
13
|
+
**Paths de mutación riesgosos**: cualquier operación que modifica el balance directamente sin pasar por el flujo que actualiza la caché.
|
|
14
|
+
|
|
15
|
+
**Trigger típico**: función de ajuste manual, importación masiva, o corrección administrativa que escribe el balance sin llamar al servicio que recalcula la caché.
|
|
16
|
+
|
|
17
|
+
**Consecuencia**: decisiones que leen la caché (¿aplica descuento? ¿otorgar crédito?) usan datos obsoletos.
|
|
18
|
+
|
|
19
|
+
---
|
|
20
|
+
|
|
21
|
+
## Patrón 2: Registro principal ↔ Índice secundario
|
|
22
|
+
|
|
23
|
+
**Descripción**: una tabla o colección tiene un índice secundario (índice invertido, tabla de búsqueda rápida, lista de IDs por categoría) que debe estar sincronizado con la tabla principal.
|
|
24
|
+
|
|
25
|
+
**Invariante**: toda entrada en la tabla principal tiene exactamente una entrada correspondiente en el índice secundario (o exactamente ninguna, según el diseño).
|
|
26
|
+
|
|
27
|
+
**Paths de mutación riesgosos**: delete de la tabla principal sin delete del índice; insert sin insert correspondiente; update que cambia la clave del índice sin actualizar el índice.
|
|
28
|
+
|
|
29
|
+
**Trigger típico**: función de "purga" o "limpieza" que borra de la tabla principal pero olvida el índice. O migración que popula la tabla principal pero no regenera el índice.
|
|
30
|
+
|
|
31
|
+
**Consecuencia**: búsquedas por índice devuelven resultados incorrectos (IDs que ya no existen, o registros que no aparecen en búsqueda aunque existan).
|
|
32
|
+
|
|
33
|
+
---
|
|
34
|
+
|
|
35
|
+
## Patrón 3: Contador agregado ↔ Suma de partes
|
|
36
|
+
|
|
37
|
+
**Descripción**: un contador de totales (total de pedidos, total de puntos, total de items activos) debe ser igual a la suma de los contadores individuales.
|
|
38
|
+
|
|
39
|
+
**Invariante**: `total == sum(individuales)` siempre.
|
|
40
|
+
|
|
41
|
+
**Paths de mutación riesgosos**: operaciones que modifican un contador individual sin ajustar el total. Frecuente en operaciones de batch donde el total se actualiza una sola vez al final pero un error parcial deja los individuales en estado inconsistente.
|
|
42
|
+
|
|
43
|
+
**Trigger típico**: transacción fallida a mitad que actualiza N de M elementos pero no revierte el total; o función que incrementa directamente el conteo individual sin notificar al agregador.
|
|
44
|
+
|
|
45
|
+
**Consecuencia**: reportes y dashboards muestran cifras incorrectas; límites y cuotas se aplican con base en datos erróneos.
|
|
46
|
+
|
|
47
|
+
---
|
|
48
|
+
|
|
49
|
+
## Patrón 4: Sesión / Permiso derivado ↔ Recurso de origen
|
|
50
|
+
|
|
51
|
+
**Descripción**: un objeto de sesión, token, o permiso derivado almacena un snapshot de los permisos o atributos del actor al momento de su creación. El actor puede cambiar después.
|
|
52
|
+
|
|
53
|
+
**Invariante**: depende del diseño. Si los permisos se evalúan en tiempo de solicitud, el objeto derivado debe reflejar el estado actual del actor. Si se usan permisos fijados al momento de creación, el invariante es documentado explícitamente.
|
|
54
|
+
|
|
55
|
+
**Paths de mutación riesgosos**: revocar permisos al actor sin invalidar sesiones activas; cambiar el rol del actor sin forzar re-autenticación; eliminar al actor sin expirar sus tokens.
|
|
56
|
+
|
|
57
|
+
**Trigger típico**: función de "deshabilitar cuenta" que setea `is_active=False` en el usuario pero no invalida los tokens JWT o sesiones existentes.
|
|
58
|
+
|
|
59
|
+
**Consecuencia**: actor deshabilitado sigue operando hasta que el token expira naturalmente.
|
|
60
|
+
|
|
61
|
+
---
|
|
62
|
+
|
|
63
|
+
## Patrón 5: Posición ↔ Factor de salud cacheado
|
|
64
|
+
|
|
65
|
+
**Descripción**: un recurso con múltiples atributos (posición de deuda: monto + colateral + fecha + valor de mercado del colateral) tiene un factor calculado (ratio LTV, score de riesgo, estado de salud) que se cachea por rendimiento.
|
|
66
|
+
|
|
67
|
+
**Invariante**: el factor cacheado debe recalcularse cada vez que cualquiera de sus inputs cambia.
|
|
68
|
+
|
|
69
|
+
**Paths de mutación riesgosos**: actualizar cualquier input del cálculo sin invalidar o recalcular el factor cacheado.
|
|
70
|
+
|
|
71
|
+
**Trigger típico**: función que actualiza el colateral sin llamar a `recalcular_factor()`. O función de ajuste de precio que modifica el valor de mercado sin propagar la invalidación de caché.
|
|
72
|
+
|
|
73
|
+
**Consecuencia**: decisiones de riesgo (¿emitir alerta? ¿bloquear operación?) usan un factor obsoleto.
|
|
74
|
+
|
|
75
|
+
---
|
|
76
|
+
|
|
77
|
+
## Patrón 6: Replica / Proyección ↔ Origen CDC
|
|
78
|
+
|
|
79
|
+
**Descripción**: un sistema mantiene una replica o proyección de datos de origen para acelerar lecturas (tabla denormalizada, proyección CQRS, replica de lectura, caché materializada). El origen puede cambiar vía CDC (Change Data Capture), eventos de dominio, o sincronización periódica.
|
|
80
|
+
|
|
81
|
+
**Invariante**: la replica debe converger al estado del origen dentro de la ventana de latencia acordada.
|
|
82
|
+
|
|
83
|
+
**Paths de mutación riesgosos**: escrituras directas al origen que no emiten el evento de dominio necesario para actualizar la replica; fallos en el consumer de eventos que quedan silenciosos; migraciones que modifican el origen sin reprocesar los eventos para actualizar la replica.
|
|
84
|
+
|
|
85
|
+
**Trigger típico**: endpoint de "corrección de datos" que escribe directo a la BD origen sin publicar el evento de dominio correspondiente.
|
|
86
|
+
|
|
87
|
+
**Consecuencia**: lecturas de la replica devuelven datos obsoletos indefinidamente.
|
|
88
|
+
|
|
89
|
+
---
|
|
90
|
+
|
|
91
|
+
## Patrón 7: Inventario total ↔ Items en almacén (por ubicación)
|
|
92
|
+
|
|
93
|
+
**Descripción**: el inventario total de un SKU es la suma de los items en cada almacén o ubicación. Se mantiene desnormalizado por rendimiento.
|
|
94
|
+
|
|
95
|
+
**Invariante**: `inventario_total[SKU] == sum(items_por_ubicacion[ubicacion][SKU] for ubicacion in todas)`
|
|
96
|
+
|
|
97
|
+
**Paths de mutación riesgosos**: movimiento entre almacenes que decrementa una ubicación sin incrementar la otra; recepción en almacén sin incrementar el total; ajuste de inventario físico que corrige la ubicación sin propagar al total.
|
|
98
|
+
|
|
99
|
+
**Trigger típico**: función de "ajuste de cierre" que actualiza el stock físico de un almacén sin pasar por la capa de negocio que mantiene el total sincronizado.
|
|
100
|
+
|
|
101
|
+
**Consecuencia**: órdenes de compra se emiten sobre stock inexistente; o stock disponible no se vende por parecer agotado.
|
|
102
|
+
|
|
103
|
+
---
|
|
104
|
+
|
|
105
|
+
## Patrón 8: Estado de workflow ↔ Timestamps de auditoría
|
|
106
|
+
|
|
107
|
+
**Descripción**: una entidad con workflow (pedido, solicitud, tarea) almacena su estado actual y los timestamps de cada transición. El timestamp de la última transición debe corresponder al estado actual.
|
|
108
|
+
|
|
109
|
+
**Invariante**: `estado_actual` siempre tiene un timestamp asociado que refleja cuándo se alcanzó ese estado.
|
|
110
|
+
|
|
111
|
+
**Paths de mutación riesgosos**: funciones que cambian el estado sin registrar el timestamp; o funciones que registran el timestamp sin actualizar el estado.
|
|
112
|
+
|
|
113
|
+
**Trigger típico**: función de "forzar estado" en panel de administración que escribe `estado = 'COMPLETADO'` directamente sin pasar por la máquina de estados que registra `completado_en = now()`.
|
|
114
|
+
|
|
115
|
+
**Consecuencia**: reportes de tiempo de ciclo son incorrectos; SLAs no se calculan bien; auditorías de cumplimiento fallan.
|
|
116
|
+
|
|
117
|
+
---
|
|
118
|
+
|
|
119
|
+
## Patrón 9: Configuración global ↔ Snapshot por instancia
|
|
120
|
+
|
|
121
|
+
**Descripción**: hay una configuración global (tasa de impuesto, precio de tarifa, límite de política) y objetos que almacenan el valor que tenía la configuración en el momento de su creación (tasa de impuesto aplicada a esta factura, precio de tarifa al momento de la reserva).
|
|
122
|
+
|
|
123
|
+
**Invariante**: el snapshot por instancia debe preservar el valor histórico, nunca actualizarse cuando cambia la configuración global. Pero si hay un bug en la creación, el snapshot puede haber capturado un valor incorrecto.
|
|
124
|
+
|
|
125
|
+
**Paths de mutación riesgosos**: función que no captura el snapshot al crear la instancia y usa la configuración global en tiempo de lectura; o función de retroactiva que sobreescribe snapshots históricos con el valor actual de la configuración.
|
|
126
|
+
|
|
127
|
+
**Trigger típico**: migración de datos que "normaliza" snapshots históricos al valor actual de la configuración global, destruyendo la trazabilidad.
|
|
128
|
+
|
|
129
|
+
**Consecuencia**: recálculos retroactivos producen cifras incorrectas; auditorías de facturación no reproducen los valores originales.
|
|
130
|
+
|
|
131
|
+
---
|
|
132
|
+
|
|
133
|
+
## Patrones de enmascaramiento
|
|
134
|
+
|
|
135
|
+
Código defensivo que oculta invariantes rotos en lugar de detectarlos:
|
|
136
|
+
|
|
137
|
+
| Patrón | Código típico | Por qué es señal de alerta |
|
|
138
|
+
|--------|--------------|---------------------------|
|
|
139
|
+
| Ternario de clamp | `a > b ? a - b : 0` | Si el invariante se mantuviera, `a` nunca sería menor que `b` |
|
|
140
|
+
| Try/catch vacío | `try { ... } catch {}` | El revert por estado roto se captura y se ignora |
|
|
141
|
+
| Early exit en cero | `if value == 0: return` | Omite el cómputo cuando el estado roto produce cero |
|
|
142
|
+
| Cap con min() | `min(calculado, disponible)` | El over-counting es el bug; `min()` solo evita el error visible |
|
|
143
|
+
| Fallback a default | `valor = d.get(k, 0)` | Si la key debería existir pero fue eliminada sin limpiar el par, el default enmascara el dato faltante |
|
|
144
|
+
|
|
145
|
+
Cuando se encuentra cualquiera de estos patrones en código que involucra dos valores acoplados, rastrear si el patrón existe porque el invariante ya estaba roto en algún path de mutación.
|
|
146
|
+
|
|
147
|
+
<!-- Adaptado de nemesis-auditor-main bajo MIT License (https://github.com/0xiehnnkta/nemesis-auditor) -->
|
|
@@ -0,0 +1,75 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: web-fetcher-routing
|
|
3
|
+
description: >
|
|
4
|
+
Routing inteligente para fetching de URLs según dominio y formato:
|
|
5
|
+
GitHub raw, PDF, sitios JS-heavy y default. Reduce tokens consumidos
|
|
6
|
+
evitando WebFetch cuando hay alternativa más eficiente. Cargar antes
|
|
7
|
+
de hacer WebFetch a URLs externas, especialmente repositorios GitHub,
|
|
8
|
+
PDFs públicos, o sitios con paywall/Cloudflare/JS heavy.
|
|
9
|
+
---
|
|
10
|
+
|
|
11
|
+
# web-fetcher-routing
|
|
12
|
+
|
|
13
|
+
Selecciona la herramienta correcta para cada URL antes de intentar el fetch.
|
|
14
|
+
Un WebFetch al HTML de GitHub UI consume 10-30× más tokens que leer el raw.
|
|
15
|
+
Un PDF vía Read devuelve basura binaria. La elección equivocada cuesta tokens
|
|
16
|
+
reales; este skill la automatiza.
|
|
17
|
+
|
|
18
|
+
## Cuándo cargar
|
|
19
|
+
|
|
20
|
+
- Antes de hacer `WebFetch` a cualquier URL externa no trivial.
|
|
21
|
+
- Cuando la URL apunta a GitHub (repositorio, archivo o blob).
|
|
22
|
+
- Cuando la URL termina en `.pdf` o el contexto indica documento PDF.
|
|
23
|
+
- Cuando el fetch previo falló con 402, página de login o contenido vacío
|
|
24
|
+
(señal de JS-heavy o Cloudflare).
|
|
25
|
+
|
|
26
|
+
## Cuándo NO cargar
|
|
27
|
+
|
|
28
|
+
- URLs internas del proyecto o localhost — usar `Read` directamente.
|
|
29
|
+
- Contenido ya cargado en el contexto de esta sesión — no re-fetchar.
|
|
30
|
+
- Cuando el usuario dictó explícitamente la herramienta a usar
|
|
31
|
+
("usa WebFetch", "lee con curl").
|
|
32
|
+
|
|
33
|
+
## Tabla de routing
|
|
34
|
+
|
|
35
|
+
| Patrón de URL | Herramienta | Razón |
|
|
36
|
+
|---|---|---|
|
|
37
|
+
| `github.com/[user]/[repo]/blob/[branch]/[path]` | Reescribir a `raw.githubusercontent.com/[user]/[repo]/[branch]/[path]` y hacer `WebFetch` con la URL reescrita | Evita el HTML de la UI de GitHub; 10-30× menos tokens |
|
|
38
|
+
| `raw.githubusercontent.com/...` | `WebFetch` directo | Ya es raw content |
|
|
39
|
+
| URL termina en `.pdf` | Invocar `Skill("swl-markitdown")` para conversión con `pdftotext` o `markitdown` | `Read` no soporta PDF; `WebFetch` devuelve binario o HTML de visor |
|
|
40
|
+
| `x.com`, `twitter.com`, cualquier host con CAPTCHA o Cloudflare detectado | Invocar `Skill("agent-browser")` con headless Chrome | `WebFetch` devuelve 402 o página de login; `agent-browser` usa accessibility tree (~82% menos tokens que screenshots) |
|
|
41
|
+
| `mp.weixin.qq.com`, `feishu.cn`, `larksuite.com` | Invocar `Skill("agent-browser")` | Plataformas chinas con autenticación o JS requerido |
|
|
42
|
+
| Todo lo demás | `WebFetch` directo | El caso común; probar primero antes de escalar |
|
|
43
|
+
|
|
44
|
+
## Algoritmo de decisión
|
|
45
|
+
|
|
46
|
+
1. Parsear la URL: extraer esquema, dominio y extensión del path.
|
|
47
|
+
2. Comparar con los patrones de la tabla, en orden de arriba hacia abajo.
|
|
48
|
+
3. Si el dominio es `github.com` con segmento `/blob/` en el path:
|
|
49
|
+
reescribir la URL antes de hacer el fetch.
|
|
50
|
+
4. Invocar la herramienta asignada al patrón que coincide.
|
|
51
|
+
5. Si el resultado tiene señales de paywall o error (primeras 10 líneas
|
|
52
|
+
contienen "Subscribe", "Sign in", "403", página vacía): escalar al
|
|
53
|
+
patrón JS-heavy con `Skill("agent-browser")`.
|
|
54
|
+
6. Reportar qué método se usó y por qué, en una línea antes del contenido.
|
|
55
|
+
|
|
56
|
+
## Ejemplo de reescritura GitHub
|
|
57
|
+
|
|
58
|
+
```
|
|
59
|
+
# URL original
|
|
60
|
+
https://github.com/tw93/Waza/blob/main/skills/read/SKILL.md
|
|
61
|
+
|
|
62
|
+
# URL reescrita para raw
|
|
63
|
+
https://raw.githubusercontent.com/tw93/Waza/main/skills/read/SKILL.md
|
|
64
|
+
```
|
|
65
|
+
|
|
66
|
+
## Señales de fallo que activan escalado
|
|
67
|
+
|
|
68
|
+
| Señal en la respuesta | Acción |
|
|
69
|
+
|---|---|
|
|
70
|
+
| HTTP 402 | Escalar a `agent-browser` |
|
|
71
|
+
| Primeras líneas con "Sign in" o "Subscribe" | Detener, avisar al usuario |
|
|
72
|
+
| Contenido HTML con menos de 200 caracteres | Reintentar con método alternativo |
|
|
73
|
+
| Binario o caracteres ilegibles | Verificar si es PDF; si sí, usar `swl-markitdown` |
|
|
74
|
+
|
|
75
|
+
<!-- Adaptado de Waza/skills/read bajo MIT License (tw93/Waza) -->
|