@saulwade/swl-ses 1.4.2 → 1.5.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 +209 -208
- package/README.md +561 -560
- package/bin/swl-mcp-server.js +214 -187
- package/bin/swl-ses.js +74 -0
- package/comandos/swl/ejecutar-fase.md +33 -4
- package/comandos/swl/metricas.md +72 -0
- package/habilidades/discutir-fase/SKILL.md +50 -2
- package/habilidades/ejecutar-task-iterativo/SKILL.md +278 -0
- package/habilidades/meta-skills-estandar/SKILL.md +22 -1
- package/habilidades/node-experto/SKILL.md +13 -2
- package/habilidades/protocolo-revision-swl/SKILL.md +276 -0
- package/habilidades/tdd-workflow/SKILL.md +33 -4
- package/habilidades/verificar-trabajo/SKILL.md +54 -5
- package/manifiestos/modulos.json +1321 -1267
- package/package.json +3 -3
- package/plugin.json +351 -351
- package/scripts/derivar-feature-list.js +489 -0
- package/scripts/doctor.js +62 -8
- package/scripts/instalador.js +56 -5
- package/scripts/lib/detectar-runtime.js +75 -9
- package/scripts/lib/estado.js +13 -1
- package/scripts/lib/expandir-targets.js +71 -0
- package/scripts/lib/parsear-opciones.js +3 -0
- package/scripts/lib/toml-merge.js +204 -0
- package/scripts/lib/transformadores/base.js +43 -9
- package/scripts/lib/transformadores/codex.js +375 -115
- package/scripts/lib/transformadores/cursor.js +359 -0
- package/scripts/lib/transformadores/index.js +2 -0
- package/scripts/mcp-server/README.md +170 -128
- package/scripts/mcp-server/auth.js +105 -0
- package/scripts/mcp-server/cache.js +106 -0
- package/scripts/mcp-server/handlers.js +190 -10
- package/scripts/mcp-server/telemetry.js +78 -0
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: discutir-fase
|
|
3
|
-
description: Cuestionario adaptativo que se ejecuta antes de planear una fase de desarrollo. Extrae decisiones técnicas y de negocio del usuario, clasifica respuestas como "cerradas" (no negociables) o "abiertas" (decisión delegada al agente), y produce un archivo CONTEXTO.md que alimenta al planificador.
|
|
4
|
-
version: "1.
|
|
3
|
+
description: Cuestionario adaptativo que se ejecuta antes de planear una fase de desarrollo. Extrae decisiones técnicas y de negocio del usuario, clasifica respuestas como "cerradas" (no negociables) o "abiertas" (decisión delegada al agente), y produce un archivo CONTEXTO.md que alimenta al planificador. Inicia con discovery routing (5 paths) que evita el cuestionario completo cuando no aplica.
|
|
4
|
+
version: "1.1.0"
|
|
5
5
|
herramientasPermitidas: [Read, Write, Edit, Bash, Glob, Grep]
|
|
6
6
|
exclusiones:
|
|
7
7
|
- "No cargar si el usuario ya tiene un CONTEXTO.md poblado para la fase en curso — ir directo a `planear-fase`."
|
|
@@ -35,6 +35,54 @@ antipatrón de planear con suposiciones implícitas que luego se contradicen.
|
|
|
35
35
|
|
|
36
36
|
---
|
|
37
37
|
|
|
38
|
+
## Paso 0 — Discovery routing (obligatorio antes del cuestionario)
|
|
39
|
+
|
|
40
|
+
Antes de lanzar el cuestionario adaptativo, ejecuta **scan ligero** de
|
|
41
|
+
`.planning/` para identificar la ruta correcta. Esto evita gastar turnos en
|
|
42
|
+
un cuestionario cuando la petición del usuario no requiere una fase nueva.
|
|
43
|
+
|
|
44
|
+
Inspirado en `kiro-discovery` (cc-sdd) — ver `temp/cc-sdd-main/tools/cc-sdd/templates/agents/claude-code-skills/skills/kiro-discovery/SKILL.md`.
|
|
45
|
+
|
|
46
|
+
### Inventario rápido (solo metadata, NO leer contenido completo)
|
|
47
|
+
|
|
48
|
+
1. `Glob` sobre `.planning/fases/*.md` → lista de fases existentes y su estado.
|
|
49
|
+
2. `Read` de `.planning/HOJA-RUTA.md` (si existe) — para mapear la petición a roadmap.
|
|
50
|
+
3. `Read` del `.planning/ESTADO.md` (si existe) — para conocer fase activa.
|
|
51
|
+
|
|
52
|
+
Si no hay `.planning/`, marca **proyecto verde** y salta al cuestionario completo (Ruta C por defecto).
|
|
53
|
+
|
|
54
|
+
### Determinar la ruta
|
|
55
|
+
|
|
56
|
+
| Ruta | Cuándo aplica | Acción |
|
|
57
|
+
|------|--------------|--------|
|
|
58
|
+
| **RUTA_A** — Extiende fase existente | La petición es una extensión, fix o enhancement dentro del scope de una fase ya planeada. Cabe en `tasks` de un PLAN.md existente. | NO ejecutar cuestionario. Informar al usuario: "Esto encaja en la fase `<N>`. ¿Procedo a actualizar su PLAN.md o lanzar `/swl:ejecutar-fase <N>` directo?" |
|
|
59
|
+
| **RUTA_B** — Sin fase necesaria | Fix trivial (<5 archivos, 1 tarea, sin decisiones de scope). Bug puntual, typo, dependencia, refactor menor. | NO ejecutar cuestionario. Informar al usuario: "Esto no requiere fase formal. Procedo directo con `implementador-swl` o el agente de stack. ¿De acuerdo?" |
|
|
60
|
+
| **RUTA_C** — Nueva fase single-scope | Petición nueva, no encaja en fases existentes, single-domain. | Ejecutar cuestionario completo (Bloques 1-4). |
|
|
61
|
+
| **RUTA_D** — Decomposición multi-fase | Petición spans múltiples dominios o produciría >20 tareas en una sola fase. | Ejecutar cuestionario **resumido** (Bloque 1 + Bloque 4) para extraer decisiones de scope. Luego sugerir descomposición en N fases. Confirmar con usuario antes de proceder. |
|
|
62
|
+
| **RUTA_E** — Mixto | La petición combina: extensión de fase existente + nueva fase + posiblemente fix trivial. | Presentar al usuario la decomposición propuesta y confirmar antes de proceder. Para la parte "nueva fase" ejecutar cuestionario reducido. |
|
|
63
|
+
|
|
64
|
+
### Heurísticas de matching
|
|
65
|
+
|
|
66
|
+
- Si la petición menciona explícitamente "fase N" → considerar RUTA_A primero.
|
|
67
|
+
- Si la petición empieza con "arregla", "corrige", "fix", "elimina" → considerar RUTA_B primero.
|
|
68
|
+
- Si la petición empieza con "implementa", "crea", "agrega feature" → RUTA_C o RUTA_D.
|
|
69
|
+
- Si la petición tiene `y`, `,`, `también` enumerando >2 cosas distintas → RUTA_D o RUTA_E.
|
|
70
|
+
|
|
71
|
+
### Cómo presentar el veredicto
|
|
72
|
+
|
|
73
|
+
Tras el scan, emite un mensaje corto al usuario antes del cuestionario:
|
|
74
|
+
|
|
75
|
+
```
|
|
76
|
+
Discovery routing: RUTA_<X>
|
|
77
|
+
Justificación: <una oración explicando por qué>
|
|
78
|
+
Próximo paso: <qué se hará a continuación>
|
|
79
|
+
¿Procedo? [s/N o sugerir otra ruta]
|
|
80
|
+
```
|
|
81
|
+
|
|
82
|
+
Solo después de confirmar la ruta procede con el cuestionario (si aplica).
|
|
83
|
+
|
|
84
|
+
---
|
|
85
|
+
|
|
38
86
|
## Protocolo de entrevista adaptativa
|
|
39
87
|
|
|
40
88
|
### Regla 1 — Máximo 4 preguntas por mensaje
|
|
@@ -0,0 +1,278 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: ejecutar-task-iterativo
|
|
3
|
+
description: >
|
|
4
|
+
Modo iterativo de ejecución por tarea (opt-in via /swl:ejecutar-fase --iterative)
|
|
5
|
+
donde cada tarea atómica recibe contexto fresco, implementer dispatched como
|
|
6
|
+
sub-agente independiente, revisión task-local con protocolo-revision-swl,
|
|
7
|
+
auto-debug en BLOCKED, y commit por tarea. Alternativa al modo default
|
|
8
|
+
(oleadas paralelas con verificación al final de fase). Cargar cuando la fase
|
|
9
|
+
tiene tareas con dependencias secuenciales fuertes, módulos críticos donde
|
|
10
|
+
cada tarea merece revisión adversarial inmediata, o cuando la fase tiene
|
|
11
|
+
>12 tareas y el modo paralelo arriesga acumular deuda no detectada.
|
|
12
|
+
version: "1.0.0"
|
|
13
|
+
herramientasPermitidas: [Read, Write, Edit, Bash, Glob, Grep, Agent]
|
|
14
|
+
exclusiones:
|
|
15
|
+
- "No cargar si /swl:ejecutar-fase NO recibió el flag --iterative — el modo default sigue siendo oleadas paralelas."
|
|
16
|
+
- "No cargar para fases pequeñas (<5 tareas) — el overhead per-task supera el beneficio."
|
|
17
|
+
- "No cargar para tareas que no son atómicas o no tienen verificación clara — la iteración requiere boundary explícito por tarea."
|
|
18
|
+
- "No cargar si PLAN.md no tiene `estado: aprobado` — requirement freeze obligatorio antes de ejecutar."
|
|
19
|
+
evolvable: true
|
|
20
|
+
---
|
|
21
|
+
|
|
22
|
+
# Ejecución iterativa per-task
|
|
23
|
+
|
|
24
|
+
## Propósito
|
|
25
|
+
|
|
26
|
+
El modo default de `/swl:ejecutar-fase` ejecuta tareas en **oleadas paralelas**
|
|
27
|
+
y verifica al final de la fase con `verificar-trabajo` goal-backward. Esto es
|
|
28
|
+
eficiente para fases con tareas independientes y bajo riesgo.
|
|
29
|
+
|
|
30
|
+
Sin embargo, hay casos donde la granularidad task-por-task es superior:
|
|
31
|
+
|
|
32
|
+
- Tareas con dependencias secuenciales fuertes (cada una depende del resultado de la anterior).
|
|
33
|
+
- Módulos críticos (dinero, permisos, datos irreversibles) donde cada tarea merece revisión adversarial.
|
|
34
|
+
- Fases con >12 tareas, donde el modo paralelo arriesga acumular deuda silenciosa.
|
|
35
|
+
- El usuario explícitamente pide "una a una, revisando cada una".
|
|
36
|
+
|
|
37
|
+
Este skill implementa ese modo: 1 tarea por iteración, contexto fresco,
|
|
38
|
+
implementer dispatchado como sub-agente, review task-local con
|
|
39
|
+
`protocolo-revision-swl`, auto-debug en `BLOCKED`, y commit por tarea.
|
|
40
|
+
|
|
41
|
+
Adaptado del patrón `kiro-impl` de cc-sdd, manteniendo identidad SWL.
|
|
42
|
+
|
|
43
|
+
## Cuándo cargar
|
|
44
|
+
|
|
45
|
+
- `/swl:ejecutar-fase N --iterative` (flag explícito del usuario)
|
|
46
|
+
- El usuario pide "ejecuta el plan tarea por tarea con revisión en cada una"
|
|
47
|
+
- El PLAN.md declara en su frontmatter `modo_ejecucion: iterativo`
|
|
48
|
+
|
|
49
|
+
## Cuándo NO cargar
|
|
50
|
+
|
|
51
|
+
Ver `exclusiones` en frontmatter.
|
|
52
|
+
|
|
53
|
+
---
|
|
54
|
+
|
|
55
|
+
## Precondiciones (verificar antes de iterar)
|
|
56
|
+
|
|
57
|
+
1. `.planning/fases/0N-PLAN.md` con `estado: aprobado` (requirement freeze).
|
|
58
|
+
2. `.planning/ESTADO.md` inicializado o actualizado.
|
|
59
|
+
3. Working tree limpio (`git status --porcelain` vacío) o el usuario explícitamente autoriza cambios pendientes.
|
|
60
|
+
4. Tests del proyecto en estado "verde de base" — capturar baseline antes del primer task para detectar regresiones limpiamente.
|
|
61
|
+
5. Validation commands descubiertos del proyecto (test, build, lint, smoke):
|
|
62
|
+
|
|
63
|
+
```bash
|
|
64
|
+
node -e "const p=require('./package.json');console.log(JSON.stringify(p.scripts||{}))" 2>/dev/null
|
|
65
|
+
```
|
|
66
|
+
|
|
67
|
+
Si no se pueden derivar comandos canónicos → reportar al usuario y pedir
|
|
68
|
+
guía antes de iterar.
|
|
69
|
+
|
|
70
|
+
---
|
|
71
|
+
|
|
72
|
+
## El loop iterativo (1 task por iteración)
|
|
73
|
+
|
|
74
|
+
### Disciplina del loop
|
|
75
|
+
|
|
76
|
+
- **Una sola sub-tarea por iteración** (ej. 3.1, luego 3.2, luego 3.3 — no batch).
|
|
77
|
+
- **Contexto fresco**: al inicio de cada iteración, re-leer `PLAN.md` para
|
|
78
|
+
determinar la siguiente tarea actionable. NO confiar en memoria acumulada
|
|
79
|
+
de iteraciones previas.
|
|
80
|
+
- **Memoria residual mínima**: tras cada iteración, retener solo 1 línea
|
|
81
|
+
("3.1: APPROVED, 3 archivos cambiados"). Descartar status report completo
|
|
82
|
+
y detalles del reviewer.
|
|
83
|
+
- **Re-read tasks.md antes de cada nuevo dispatch** — el implementer previo
|
|
84
|
+
pudo agregar `## Implementation Notes` que aplican a la siguiente tarea.
|
|
85
|
+
|
|
86
|
+
### Iteración estándar
|
|
87
|
+
|
|
88
|
+
Para cada sub-tarea pendiente, en orden:
|
|
89
|
+
|
|
90
|
+
#### Paso 1 — Identificar la siguiente tarea
|
|
91
|
+
|
|
92
|
+
```bash
|
|
93
|
+
# Re-leer PLAN.md y encontrar primera tarea no marcada [x]
|
|
94
|
+
grep -E "^\s*- \[ \]" .planning/fases/0N-PLAN.md | head -1
|
|
95
|
+
```
|
|
96
|
+
|
|
97
|
+
- Saltear tareas con anotación `_Blocked:_`.
|
|
98
|
+
- Verificar dependencias declaradas en `_Depends:_` — si una dependencia no está `[x]`, ejecutarla primero o pedir resolución al usuario.
|
|
99
|
+
- Leer `_Boundary:_` para identificar los paths/módulos que esta tarea puede tocar.
|
|
100
|
+
|
|
101
|
+
#### Paso 2 — Dispatch del implementer (como sub-agente fresh)
|
|
102
|
+
|
|
103
|
+
Construir prompt con:
|
|
104
|
+
|
|
105
|
+
- Texto literal de la tarea (no parafrasear).
|
|
106
|
+
- Boundary scope.
|
|
107
|
+
- Paths a spec files (`PLAN.md`, `CONTEXTO.md`, otros).
|
|
108
|
+
- IDs literales de requirements/design references (NO inventar `REQ-*` aliases).
|
|
109
|
+
- Validation commands del proyecto (test, build, smoke) relevantes a la tarea.
|
|
110
|
+
- Implementation Notes previas del PLAN.md que apliquen al boundary actual.
|
|
111
|
+
- Indicación de si la tarea es behavioral (requiere feature flag opcional) o no-behavioral.
|
|
112
|
+
|
|
113
|
+
Invocar el sub-agente vía `Agent` tool con `subagent_type` apropiado:
|
|
114
|
+
|
|
115
|
+
- Backend Python → `backend-python-swl`
|
|
116
|
+
- Backend Node → `backend-node-swl`
|
|
117
|
+
- Frontend React → `frontend-react-swl`
|
|
118
|
+
- ... (matriz `usar-sistema-swl.md`)
|
|
119
|
+
- Fallback: `implementador-swl`
|
|
120
|
+
|
|
121
|
+
El sub-agente debe devolver un **Status Report estructurado** con campo
|
|
122
|
+
`STATUS: READY_FOR_REVIEW | BLOCKED | NEEDS_CONTEXT`.
|
|
123
|
+
|
|
124
|
+
#### Paso 3 — Manejar el status del implementer
|
|
125
|
+
|
|
126
|
+
Parsear el status solo del bloque exacto `## Status Report` con campo `STATUS:`.
|
|
127
|
+
Si el status no es parseable, re-dispatch UNA VEZ pidiendo el bloque estructurado.
|
|
128
|
+
Sin status parseable tras 2 dispatches → escalar.
|
|
129
|
+
|
|
130
|
+
| Status | Acción |
|
|
131
|
+
|--------|--------|
|
|
132
|
+
| `READY_FOR_REVIEW` | Continuar al Paso 4 (review) |
|
|
133
|
+
| `NEEDS_CONTEXT` | Re-dispatch UNA VEZ con el contexto adicional solicitado. Si persiste → Paso 5 (debug) |
|
|
134
|
+
| `BLOCKED` | Saltar a Paso 5 (debug). NO marcar la tarea como saltada |
|
|
135
|
+
|
|
136
|
+
#### Paso 4 — Review task-local
|
|
137
|
+
|
|
138
|
+
Invocar el revisor apropiado al stack:
|
|
139
|
+
|
|
140
|
+
- `revisor-codigo-swl` (general)
|
|
141
|
+
- `revisor-react-swl` / `revisor-angular-swl` (frontend)
|
|
142
|
+
- `revisor-typescript-swl` / `revisor-python-*` / etc.
|
|
143
|
+
- `revisor-seguridad-swl` (si la tarea toca auth, secretos, input externo)
|
|
144
|
+
|
|
145
|
+
El revisor carga `Skill("protocolo-revision-swl")` y emite veredicto:
|
|
146
|
+
|
|
147
|
+
| Verdict | Acción |
|
|
148
|
+
|---------|--------|
|
|
149
|
+
| `APPROVED` con score ≥9.0 | Continuar al Paso 6 (commit) |
|
|
150
|
+
| `REJECTED` (1er ciclo) | Re-dispatch del implementer con las findings del revisor. Volver al Paso 3 |
|
|
151
|
+
| `REJECTED` (2do ciclo) | Saltar al Paso 5 (debug) — la remediación no convergió |
|
|
152
|
+
|
|
153
|
+
Veredictos persistidos en `.planning/audit/reviews/<task-id>-<ts>.json`.
|
|
154
|
+
|
|
155
|
+
#### Paso 5 — Auto-debug (cuando BLOCKED o REJECTED persiste)
|
|
156
|
+
|
|
157
|
+
Invocar `depurador-swl` con contexto fresh:
|
|
158
|
+
|
|
159
|
+
- Síntoma exacto del blocker o de los findings que no se resolvieron.
|
|
160
|
+
- Stack trace, output del comando fallido, exit code.
|
|
161
|
+
- Reviewer feedback si aplica.
|
|
162
|
+
- `_Boundary:_` de la tarea.
|
|
163
|
+
- Implementation Notes relacionadas.
|
|
164
|
+
|
|
165
|
+
El depurador retorna:
|
|
166
|
+
|
|
167
|
+
```
|
|
168
|
+
ROOT_CAUSE: <descripción>
|
|
169
|
+
FIX_PLAN: <pasos concretos>
|
|
170
|
+
VERIFICATION: <cómo verificar el fix>
|
|
171
|
+
NEXT_ACTION: RETRY_TASK | BLOCK_TASK | STOP_FOR_HUMAN
|
|
172
|
+
CONFIDENCE: HIGH | MEDIUM | LOW
|
|
173
|
+
```
|
|
174
|
+
|
|
175
|
+
- `RETRY_TASK` con CONFIDENCE HIGH/MEDIUM → re-dispatch implementer con el fix plan. Volver al Paso 3.
|
|
176
|
+
- `BLOCK_TASK` → marcar tarea con `_Blocked:_<razón>_` en PLAN.md. Actualizar ESTADO.md. Continuar con la SIGUIENTE tarea (no esta).
|
|
177
|
+
- `STOP_FOR_HUMAN` → pausar el loop y escalar al usuario con el informe del depurador.
|
|
178
|
+
|
|
179
|
+
#### Paso 6 — Commit atómico de la tarea
|
|
180
|
+
|
|
181
|
+
Tras APPROVED:
|
|
182
|
+
|
|
183
|
+
```bash
|
|
184
|
+
git add <archivos del diff de esta tarea>
|
|
185
|
+
git commit -m "<tipo>(<scope>): <descripción imperativa de la tarea>
|
|
186
|
+
|
|
187
|
+
Tarea: <task-id> — <texto literal>
|
|
188
|
+
Plan: .planning/fases/0N-PLAN.md
|
|
189
|
+
Review: .planning/audit/reviews/<task-id>-<ts>.json"
|
|
190
|
+
```
|
|
191
|
+
|
|
192
|
+
Marcar la tarea como `[x]` en PLAN.md en el mismo commit (o en commit
|
|
193
|
+
inmediato siguiente atómico).
|
|
194
|
+
|
|
195
|
+
#### Paso 7 — Update de ESTADO.md y memoria
|
|
196
|
+
|
|
197
|
+
- Marcar tarea `[x]` en PLAN.md.
|
|
198
|
+
- Actualizar ESTADO.md con: tarea completada, commit hash, archivos cambiados, score del review.
|
|
199
|
+
- Agregar al final del PLAN.md, en sección `## Implementation Notes`, cualquier learning relevante para tareas futuras del mismo boundary o dependencia.
|
|
200
|
+
- Retener solo 1 línea en memoria de iteración para el siguiente loop.
|
|
201
|
+
|
|
202
|
+
#### Paso 8 — Próxima iteración o cierre
|
|
203
|
+
|
|
204
|
+
- Si quedan tareas pendientes: volver al Paso 1.
|
|
205
|
+
- Si todas las tareas están `[x]` o `_Blocked:_`: cerrar el loop e invocar
|
|
206
|
+
el flujo de cierre estándar de `ejecutar-fase` (RESUMEN.md, actualizar
|
|
207
|
+
HOJA-RUTA.md, verificación goal-backward final con `verificar-trabajo` —
|
|
208
|
+
ver Paso 6-8 del comando `/swl:ejecutar-fase`).
|
|
209
|
+
|
|
210
|
+
---
|
|
211
|
+
|
|
212
|
+
## Persistencia de aprendizajes entre tareas
|
|
213
|
+
|
|
214
|
+
A diferencia del modo paralelo (donde cada wave es relativamente
|
|
215
|
+
independiente), el modo iterativo propaga aprendizajes:
|
|
216
|
+
|
|
217
|
+
### En PLAN.md, sección `## Implementation Notes`:
|
|
218
|
+
|
|
219
|
+
```markdown
|
|
220
|
+
## Implementation Notes
|
|
221
|
+
|
|
222
|
+
- **Tarea 1.2 (POST /facturas)**: `Pydantic v2` cambia `parse_obj` → `model_validate`.
|
|
223
|
+
Aplica a futuras tareas que validen schemas con v2.
|
|
224
|
+
- **Tarea 2.1 (migración)**: `Alembic` no detecta cambios en `JSONB` automáticamente —
|
|
225
|
+
marcar con `nullable=False` explícito.
|
|
226
|
+
```
|
|
227
|
+
|
|
228
|
+
El implementer del paso 2 lee estas notes antes de empezar. Esto evita
|
|
229
|
+
repetir los mismos errores task-a-task.
|
|
230
|
+
|
|
231
|
+
---
|
|
232
|
+
|
|
233
|
+
## Diferencias clave con el modo default (paralelo por oleadas)
|
|
234
|
+
|
|
235
|
+
| Aspecto | Modo default (paralelo) | Modo iterativo (este skill) |
|
|
236
|
+
|---------|------------------------|----------------------------|
|
|
237
|
+
| Granularidad | Oleada (varias tareas en paralelo) | 1 tarea por iteración |
|
|
238
|
+
| Contexto del implementer | Acumulado entre tareas de la misma oleada | Fresco por tarea (sub-agente nuevo) |
|
|
239
|
+
| Review | Al final de fase con `verificar-trabajo` | Tras cada tarea con `protocolo-revision-swl` |
|
|
240
|
+
| Auto-debug | Manual o post-mortem | Automático en BLOCKED |
|
|
241
|
+
| Commit | Atómico por slice/tarea (similar) | Atómico estricto por tarea + review JSON |
|
|
242
|
+
| Implementation Notes | No requeridas | Propagadas entre tareas |
|
|
243
|
+
| Costo en tokens | Menor | Mayor (~1.5-2× por contexto fresh) |
|
|
244
|
+
| Cuándo preferir | Fases con tareas independientes, riesgo bajo | Tareas con dependencias fuertes, módulos críticos, fases grandes |
|
|
245
|
+
|
|
246
|
+
---
|
|
247
|
+
|
|
248
|
+
## Gotchas / Errores comunes no obvios
|
|
249
|
+
|
|
250
|
+
- **Re-usar contexto acumulado del implementer entre iteraciones**: el implementer del task 3.2 NO debe tener en su contexto el reporte completo del task 3.1 — solo el `## Implementation Notes` relevante. Causa: ahorro falso de tokens. Solución: dispatch fresh siempre, los notes son el canal explícito de propagación.
|
|
251
|
+
- **Saltar el Paso 4 (review) "porque la tarea es chica"**: el review per-task es lo que distingue este modo del default. Sin review per-task, el modo iterativo es solo "lento sin beneficio". Solución: si la tarea es realmente trivial, no usar modo iterativo.
|
|
252
|
+
- **Auto-debug que sugiere FIX y se aplica sin re-review**: tras debug → fix → COMMIT directo es anti-patrón. Solución: cada FIX debe volver al Paso 4 (review) antes de commit.
|
|
253
|
+
- **Boundary violations toleradas porque "el implementer tenía que tocar eso"**: si un archivo fuera del boundary aparece en el diff y el revisor lo marca como finding, no se puede aceptar sin extender el boundary EN EL PLAN.md (decisión documentada). Solución: si el boundary necesita extenderse, actualizar PLAN.md primero y volver a iterar.
|
|
254
|
+
- **`MERGED` en main sin marcar la tarea `[x]` en PLAN.md**: deja ESTADO.md y PLAN.md desincronizados. Solución: marcar `[x]` está en el MISMO commit del cambio (no commit aparte).
|
|
255
|
+
|
|
256
|
+
---
|
|
257
|
+
|
|
258
|
+
## Relación con otros componentes SWL
|
|
259
|
+
|
|
260
|
+
| Componente | Relación |
|
|
261
|
+
|-----------|----------|
|
|
262
|
+
| `ejecutar-fase` | Skill padre — el flag `--iterative` cede el control a este skill |
|
|
263
|
+
| `protocolo-revision-swl` | Invocado en Paso 4 por el revisor |
|
|
264
|
+
| `verificar-trabajo` | Invocado al cerrar el loop como verificación goal-backward final |
|
|
265
|
+
| `depurador-swl` | Invocado en Paso 5 cuando hay BLOCKED o REJECTED persistente |
|
|
266
|
+
| `implementador-swl` y agentes de stack | Dispatchados como sub-agentes en Paso 2 |
|
|
267
|
+
| `notificador-swl` | Invocado en STOP_FOR_HUMAN o ESCALATE |
|
|
268
|
+
| `reglas/seguridad-agentes.md` § Recovery Catalog | Marco de escalada (reprompt → reduce-autonomy → escalate → terminate) |
|
|
269
|
+
|
|
270
|
+
## Checklist de cierre del loop
|
|
271
|
+
|
|
272
|
+
- [ ] Todas las tareas pendientes están `[x]` o `_Blocked:_<razón>_`
|
|
273
|
+
- [ ] Cada tarea tiene su review JSON en `.planning/audit/reviews/`
|
|
274
|
+
- [ ] PLAN.md tiene `## Implementation Notes` con aprendizajes relevantes
|
|
275
|
+
- [ ] ESTADO.md actualizado con commit hashes y scores
|
|
276
|
+
- [ ] Verificación goal-backward final ejecutada (`verificar-trabajo`)
|
|
277
|
+
- [ ] RESUMEN.md generado (paso del flujo estándar de `ejecutar-fase`)
|
|
278
|
+
- [ ] Working tree limpio o con cambios pendientes documentados
|
|
@@ -9,7 +9,12 @@ description: >
|
|
|
9
9
|
idiomas de framework y mapeo a frameworks de seguridad. Cargar cuando se
|
|
10
10
|
esté escribiendo o auditando un SKILL.md y se necesiten patrones avanzados
|
|
11
11
|
que no están en la regla core.
|
|
12
|
-
version: "1.0.
|
|
12
|
+
version: "1.0.1"
|
|
13
|
+
evolved: true
|
|
14
|
+
evolved-from: "1.0.0"
|
|
15
|
+
evolved-at: "2026-05-15"
|
|
16
|
+
evolved-by: "aprender"
|
|
17
|
+
evolved-note: "Gotcha 'absorber naming externo al portar patrón valioso' agregado. Origen: instrucción explícita del usuario en sesión 2026-05-15 al renombrar 'kiro-review-protocol' → 'protocolo-revision-swl' durante absorción cc-sdd."
|
|
13
18
|
herramientasPermitidas: [Read, Write, Edit]
|
|
14
19
|
exclusiones:
|
|
15
20
|
- "No cargar si solo se necesita el frontmatter obligatorio, el límite de líneas o la estructura de directorio — eso está en la regla core `reglas/skills-estandar.md` que se carga siempre."
|
|
@@ -293,6 +298,22 @@ al caso en cuestión en lugar de los tres.
|
|
|
293
298
|
- **Cargar este skill cuando la regla core basta**: esto invierte el trade-off
|
|
294
299
|
de context budget. Cargar `meta-skills-estandar` solo cuando se necesita
|
|
295
300
|
contenido extendido, no por defecto.
|
|
301
|
+
- **Absorber naming externo al portar un patrón valioso**: cuando un skill
|
|
302
|
+
externo (de cc-sdd, harness-sdd, langchain, kiro, etc.) aporta un patrón
|
|
303
|
+
útil, la tentación es importar también el nombre original (`kiro-review`,
|
|
304
|
+
`kiro-impl`, `langchain-router`). Causa: economía cognitiva en la transición.
|
|
305
|
+
Costo: rompe la identidad del sistema receptor, confunde routing del
|
|
306
|
+
orquestador, viola la regla de idioma español MX y crea acoplamiento
|
|
307
|
+
conceptual con un sistema externo cuya API puede cambiar. Solución
|
|
308
|
+
obligatoria: **portar el patrón, adaptar el nombre al naming del sistema
|
|
309
|
+
receptor**. Para SWL: usar prefijo `swl-` o sufijo `-swl`, en español MX,
|
|
310
|
+
con el dominio del problema (no del sistema origen). Ejemplo real
|
|
311
|
+
(2026-05-15): el patrón `kiro-review` de cc-sdd se absorbió como
|
|
312
|
+
`protocolo-revision-swl` por instrucción explícita del usuario; `kiro-impl`
|
|
313
|
+
iterativo se absorbió como `ejecutar-task-iterativo`. Documentar el origen
|
|
314
|
+
en la sección "Referencias" o "Origen" del skill, NO en el nombre. Verificar
|
|
315
|
+
con `git log --grep="kiro\|otro-prefijo-externo"` que no quedan menciones
|
|
316
|
+
del naming externo en el codebase del sistema receptor.
|
|
296
317
|
|
|
297
318
|
## Referencias
|
|
298
319
|
|
|
@@ -4,9 +4,9 @@ description: Node.js y TypeScript backend moderno. Cubre patterns de Express/Fas
|
|
|
4
4
|
version: "1.0.1"
|
|
5
5
|
evolved: true
|
|
6
6
|
evolved-from: "1.0.0"
|
|
7
|
-
evolved-at: "2026-05-
|
|
7
|
+
evolved-at: "2026-05-15"
|
|
8
8
|
evolved-by: "aprender"
|
|
9
|
-
evolved-note: "Gotcha req.destroy() vs req.pause() en
|
|
9
|
+
evolved-note: "Gotcha req.destroy() vs req.pause() en HTTP nativos (PR #13). + CRLF regex breakage en Windows + serializar TOML con literal '''...''' (Sub-fase 11/11.5 v1.5.1)."
|
|
10
10
|
herramientasPermitidas: [Read, Write, Glob]
|
|
11
11
|
exclusiones:
|
|
12
12
|
- "No cargar para aplicaciones Next.js con App Router — Next.js tiene patrones de Server Components, SSR y caché que difieren del backend Node puro; cargar `nextjs-experto`."
|
|
@@ -272,6 +272,17 @@ if (require.main === module) {
|
|
|
272
272
|
|
|
273
273
|
**`req.destroy()` en un handler `http` cierra el socket antes de poder enviar la respuesta**: cuando se aborta la lectura del body (por ejemplo al exceder `maxPayloadBytes`), llamar `req.destroy()` en el listener `data` cierra el socket inmediatamente. El subsiguiente `res.writeHead(413); res.end()` del handler falla porque ya no hay socket — el cliente recibe `ECONNRESET` en lugar de 413. Causa: `destroy()` no espera al flush de la respuesta pendiente; equivale a un `RST` TCP. Fix: usar `req.pause()` + un flag `abortado = true` + rechazar la promesa de lectura; dejar que el handler envíe `res.writeHead(413); res.end(...)` normalmente. Node cierra el socket por sí solo tras `res.end()`. Aplica a servidores HTTP nativos sin Express/Fastify, donde el control del body es manual.
|
|
274
274
|
|
|
275
|
+
**Regex multi-line con `\n` falla con archivos CRLF de Windows**: un parser que usa `/^---\n([\s\S]*?)\n---\n/` para detectar frontmatter YAML rompe con archivos commiteados en Windows (donde git aplica `core.autocrlf=true` y convierte LF↔CRLF al checkout). Caso real (Sub-fase 11.5 v1.5.1, swl-ses): `parsearFrontmatter` en `scripts/lib/transformadores/base.js` devolvía `frontmatter:{}` y `cuerpo:<archivo entero>` para los 60 agentes `agentes/*.md` cuando se ejecutaba en Windows. Resultado downstream: `transformarAgente` de Codex generaba TOML con `description = ""` y `developer_instructions` con el frontmatter duplicado dentro del body. Bug latente que solo aparece al ejecutar contra archivos reales con CRLF. Fix: normalizar antes del match — `const norm = String(contenido).replace(/\r\n/g, '\n').replace(/\r/g, '\n');` y aplicar regex sobre `norm`. Siempre asumir que cualquier archivo leído de filesystem en Windows puede tener CRLF, incluso si git lo "almacena" como LF. Aplica a TODA función con regex multi-line que reciba contenido de filesystem.
|
|
276
|
+
|
|
277
|
+
**Serializar TOML zero-deps: preferir `'''literal'''` sobre `"""basic"""` para multi-line con backslashes**: TOML soporta dos sintaxis multi-line — basic `"""..."""` que procesa escapes (`\n`, `\t`, `\\`) y literal `'''...'''` que preserva el contenido tal cual. Caso real (Sub-fase 11 v1.5.1, swl-ses): al serializar el cuerpo Markdown de agentes SWL (con regex `\n`, paths Windows con `\`, etc.) a `developer_instructions` de `~/.codex/agents/*.toml`, usar basic `"""..."""` exigía escapar cada `\` a `\\` (laborioso y propenso a bugs). Solución: usar literal `'''...'''` por default (preserva Markdown crudo) y caer a basic `"""..."""` con escape SOLO si el cuerpo contiene `'''` literal (raro). Helper en `scripts/lib/transformadores/codex.js`:
|
|
278
|
+
```js
|
|
279
|
+
_tomlMultiline(s) {
|
|
280
|
+
if (!s.includes("'''")) return `'''\n${s}\n'''`;
|
|
281
|
+
return `"""\n${s.replace(/\\/g, '\\\\').replace(/"""/g, '\\"""')}\n"""`;
|
|
282
|
+
}
|
|
283
|
+
```
|
|
284
|
+
Aplica a cualquier serialización TOML manual sin librería (`@iarna/toml`, etc.) donde el contenido es texto rico con caracteres especiales.
|
|
285
|
+
|
|
275
286
|
```js
|
|
276
287
|
// MAL — ECONNRESET en cliente, nunca recibe 413
|
|
277
288
|
req.on('data', chunk => {
|