@saulwade/swl-ses 2.4.3 → 2.5.2

package/habilidades/{filament-admin/SKILL.md → php-experto/recursos/filament-admin.md}

@@ -1,44 +1,28 @@
----
-name: filament-admin
-description: >
-  Laravel Filament v3: optimización de paneles admin, formularios con Tabs y
-  Grid, Infolist para vistas de detalle, Tables con filtros avanzados y bulk
-  actions, Resources con relaciones anidadas. Cargar cuando se construya o
-  rediseñe un panel Filament, se implementen formularios complejos, tablas con
-  filtros o Resources con relaciones.
-version: "1.0.0"
-herramientasPermitidas: [Read, Grep]
-evolvable: true  # default para skill estandar
-exclusiones:
-  - "No cargar para paneles admin genéricos sin Filament (Nova, Voyager, backpack) — cada panel tiene su propia API; cargar el skill del framework correspondiente."
-  - "No cargar para formularios frontend en la app pública de Laravel (Livewire sin Filament, Inertia/Vue) — para formularios de usuario final cargar `livewire-avanzado` o el skill del framework JS."
-  - "No cargar para migración de datos o seeds de base de datos en proyectos Laravel — para migraciones y seeds cargar `postgresql-experto` o el skill del ORM."
-  - "No cargar para autenticación y autorización en Laravel sin relación con paneles Filament — para auth cargar `auth-patrones` o `laravel-auth`."
----
 # Filament Admin v3 — Paneles, Formularios y Tablas
 
-Skill para construir paneles de administración con Laravel Filament v3.
-Cubre optimización de UX, formularios multi-tab, infolistas de detalle,
-tablas con filtros y acciones masivas.
-
----
-
-## Cuándo NO cargar
-
-- El proyecto usa un panel admin diferente (Nova, Voyager, Backpack): cargar el skill del framework correspondiente.
-- Los formularios son de la app pública del usuario final (Livewire sin Filament, Inertia/Vue): cargar `livewire-avanzado` o el skill del framework JS.
-- La tarea es migración de datos, seeds o esquema de BD sin relación con el panel: cargar `postgresql-experto`.
-- La tarea es autenticación y autorización sin relación con el panel Filament: cargar `auth-patrones`.
-
-## Cuándo cargar este skill
-
-- Al crear o modificar un `Resource` de Filament (formulario, tabla o infolist)
-- Al implementar formularios con más de 6 campos
-- Al configurar filtros, ordenamiento o bulk actions en tablas
-- Al separar modo vista del modo edición en un panel admin
-- Al implementar relaciones anidadas (`HasMany`, `BelongsToMany`) en Forms o Tables
-
----
+> Recurso de `php-experto` (absorbió al skill `filament-admin` el 2026-07-08).
+> Cargar php-experto y leer este archivo cuando se construya o rediseñe un
+> panel Laravel Filament v3: formularios con Tabs/Grid, Infolist, Tables con
+> filtros y bulk actions, Resources con relaciones anidadas.
+
+## Índice
+1. Jerarquía de optimización de UX
+2. Formulario con Tabs optimizados
+3. Infolist para modo vista
+4. Table con filtros y bulk actions
+5. Relaciones anidadas — HasMany con Repeater
+6. Reglas obligatorias
+7. Anti-patrones
+8. Checklist de verificación
+9. Referencias
++ Gotchas no obvios
+
+## Alcance — cuándo NO aplica este recurso
+
+- Panel admin distinto de Filament (Nova, Voyager, Backpack): usar el skill del framework correspondiente.
+- Formularios de la app pública (Livewire sin Filament, Inertia/Vue): no es territorio de Filament.
+- Migraciones/seeds sin relación con el panel: `postgresql-experto`.
+- Auth sin relación con el panel: `auth-patrones`.
 
 ## 1. Jerarquía de optimización de UX (en orden de impacto)
 

package/habilidades/prevencion-sobreingenieria/recursos/soluciones-nativas.md

@@ -1,166 +1,166 @@
-# Soluciones nativas de plataforma
-
-Recurso de `prevencion-sobreingenieria` — peldaño 4 de la escalera de decisión.
-La primera pregunta del ingeniero perezoso-eficiente: *¿la plataforma ya hace
-esto?* Antes de instalar un paquete o escribir un componente, revisar aquí.
-La plataforma viaja gratis con la app, no se rompe en updates, y la escribió
-gente cuyo trabajo es exactamente ese problema.
-
-Adaptado de [ponytail](https://github.com/DietrichGebert/ponytail)
-`docs/platform-native.md` (MIT, DietrichGebert).
-
----
-
-## Elementos HTML
-
-| Crees que necesitas | La plataforma ya tiene |
-|---|---|
-| Lib de date picker | `<input type="date">` |
-| Lib de time picker | `<input type="time">` |
-| Lib de color picker | `<input type="color">` |
-| Lib de range slider | `<input type="range">` |
-| Componente de progress bar | `<progress value="70" max="100">` |
-| Componente meter/gauge | `<meter value="0.7">` |
-| Lib de modal/diálogo | `<dialog>` + `dialog.showModal()` |
-| Componente acordeón/FAQ | `<details><summary>Título</summary>…</details>` |
-| Lib de tooltips | atributo `title` + CSS `::before`/`::after` |
-| Dropdown con búsqueda | `<input list="id"> <datalist id="id">` |
-| Textarea auto-crecible | `field-sizing: content` (CSS) |
-| Header sticky | `position: sticky; top: 0` (CSS) |
-
-## Capacidades CSS (donde se suele usar JS)
-
-| Crees que necesitas JS para | CSS ya tiene |
-|---|---|
-| Font size responsivo | `font-size: clamp(1rem, 2.5vw, 2rem)` |
-| Espaciado fluido | `padding: clamp(1rem, 5vw, 3rem)` |
-| Dark mode | `@media (prefers-color-scheme: dark)` |
-| Reduced motion | `@media (prefers-reduced-motion: reduce)` |
-| Layout responsivo sin breakpoints | `grid-template-columns: repeat(auto-fill, minmax(250px, 1fr))` |
-| Responsive a nivel componente | `@container` queries |
-| Design tokens / theming global | Custom properties (`--color-primario: #7c3aed`) |
-| Smooth scroll | `scroll-behavior: smooth` |
-| Carrusel con scroll-snap | `scroll-snap-type: x mandatory` + `scroll-snap-align: start` |
-| Aspect ratio | `aspect-ratio: 16 / 9` |
-| Truncar texto con elipsis | `overflow: hidden; text-overflow: ellipsis; white-space: nowrap` |
-| Clamp multilínea | `-webkit-line-clamp: 3` |
-| Aislamiento de estilos por capas | `@layer base, components, utilities` |
-| Selectores anidados | CSS nesting nativo (sin preprocesador) |
-| Selector de padre | `:has(input:checked)` |
-
-## JavaScript / APIs del navegador
-
-| Crees que necesitas | La plataforma ya tiene |
-|---|---|
-| `query-string` / `qs` | `new URLSearchParams(location.search)` |
-| `lodash.clonedeep` | `structuredClone(obj)` |
-| `lodash.groupby` | `Object.groupBy(arr, fn)` |
-| `numeral` / `accounting` | `new Intl.NumberFormat("es-MX", { style: "currency", currency: "MXN" })` |
-| `date-fns` format | `new Intl.DateTimeFormat("es-MX", { dateStyle: "long" }).format(fecha)` |
-| Tiempo relativo | `new Intl.RelativeTimeFormat("es", { numeric: "auto" }).format(-3, "day")` |
-| Plurales i18n | `new Intl.PluralRules("es-MX").select(cantidad)` |
-| `clipboard.js` | `navigator.clipboard.writeText(texto)` |
-| `uuid` (v4) | `crypto.randomUUID()` |
-| Lib de infinite scroll | `new IntersectionObserver(cb).observe(sentinela)` |
-| Lib de resize listener | `new ResizeObserver(cb).observe(elemento)` |
-| Watcher de mutaciones DOM | `new MutationObserver(cb).observe(el, opciones)` |
-| Check de conectividad | `navigator.onLine` + eventos `online`/`offline` |
-| Lib de share sheet | `navigator.share({ title, text, url })` |
-| `store.js` / `localForage` (caso simple) | `localStorage.setItem(clave, JSON.stringify(valor))` |
-| Abortar fetch por timeout | `AbortSignal.timeout(5000)` pasado a `fetch` |
-| Event bus custom | `new EventTarget()` / `dispatchEvent(new CustomEvent("x", { detail }))` |
-
-**Debounce en 3 líneas** (sin dependencia):
-
-```js
-// simplificado: debounce de una sola instancia; lib si se necesitan N independientes
-let t;
-const debounce = (fn, ms) => (...args) => { clearTimeout(t); t = setTimeout(() => fn(...args), ms); };
-```
-
-## Swift / SwiftUI
-
-| Crees que necesitas | La plataforma ya tiene |
-|---|---|
-| Lib de date/time picker | `DatePicker` |
-| Lib de color picker | `ColorPicker` |
-| Search bar + filtrado | `.searchable(text:)` |
-| Lib de pull-to-refresh | `.refreshable { }` |
-| Swipe-to-delete / acciones de fila | `.swipeActions { }` |
-| Carga async de imágenes + caché | `AsyncImage` |
-| Lib de gráficas | Swift Charts (`import Charts`) |
-| Render de Markdown | `Text(...)` markdown / `AttributedString(markdown:)` |
-| Share sheet | `ShareLink` |
-| Spinner de carga | `ProgressView()` |
-| Photo picker | `PhotosPicker` |
-| JSON (SwiftyJSON) | `Codable` + `JSONDecoder`/`JSONEncoder` |
-| HTTP simple (Alamofire) | `URLSession` async/await |
-| Crypto (CryptoSwift) | `CryptoKit` |
-| Persistencia / ORM | `SwiftData`, o `@AppStorage` para key-values |
-| Logging | `Logger` (`os.log`) |
-
-## Stdlib de Node.js
-
-| Crees que necesitas | Node ya tiene |
-|---|---|
-| `mkdirp` / `make-dir` | `fs.mkdirSync(ruta, { recursive: true })` |
-| `rimraf` | `fs.rmSync(ruta, { recursive: true, force: true })` |
-| `uuid` (v4) | `crypto.randomUUID()` |
-| `object-assign` | `Object.assign()` / spread |
-| `array-uniq` | `[...new Set(arr)]` |
-| `array-flatten` / `flat` | `arr.flat(profundidad)` |
-| `path-exists` | `fs.existsSync(ruta)` |
-| `load-json-file` | `JSON.parse(fs.readFileSync(ruta, "utf8"))` |
-| `write-json-file` | `fs.writeFileSync(ruta, JSON.stringify(obj, null, 2))` |
-| `pkg-dir` | `path.resolve(__dirname, "..")` / `import.meta.dirname` |
-
-## Stdlib de Python
-
-| Crees que necesitas | Python ya tiene |
-|---|---|
-| `python-dateutil` (parsing básico) | `datetime.fromisoformat()` (3.7+) |
-| `pytz` | `zoneinfo.ZoneInfo("America/Mexico_City")` (3.9+) |
-| `attrs` (data classes simples) | `@dataclass` |
-| `pathlib2` | `pathlib.Path` (3.4+) |
-| `simplejson` (uso básico) | `json` (stdlib) |
-| `requests` (GET simple) | `urllib.request.urlopen(url)`; `requests` para algo real |
-| `click` (comando único) | `argparse` (stdlib) |
-| `mergedeep` | `dict \| otro_dict` (3.9+) |
-| `more-itertools` (básico) | `itertools`: `chain`, `islice`, `groupby`, `product` |
-| `toolz` (básico) | `functools`: `lru_cache`, `partial`, `reduce` |
-
-## Base de datos
-
-| Crees que necesitas código de app para | La BD ya tiene |
-|---|---|
-| Paginación | `LIMIT 20 OFFSET 40` |
-| Totales acumulados | `SUM(...) OVER (ORDER BY fecha)` (window function) |
-| Rank por grupo | `RANK() OVER (PARTITION BY categoria ORDER BY score DESC)` |
-| Deduplicación | `SELECT DISTINCT` / `ON CONFLICT DO NOTHING` |
-| Recorrido de árboles | CTE recursivo (`WITH RECURSIVE`) |
-| Full-text search (básico) | `tsvector` / `MATCH AGAINST` / `FTS5` |
-| Almacenar + consultar JSON | `jsonb` (Postgres) / `JSON_EXTRACT` (SQLite/MySQL) |
-| Generación de UUID | `gen_random_uuid()` (Postgres) / `UUID()` (MySQL) |
-| Timestamps de insert/update | `DEFAULT now()` + trigger / `ON UPDATE CURRENT_TIMESTAMP` |
-| Unicidad | constraint `UNIQUE`, no checks en la app |
-| Integridad referencial | `FOREIGN KEY`, no checks en la app |
-| Rangos de valores | `CHECK (precio > 0)`, no validación en la app |
-
----
-
-## El patrón
-
-En todas las capas es el mismo ciclo:
-
-```
-El equipo de la plataforma pasa años resolviendo el problema.
-Alguien lo envuelve en un paquete.
-Instalas el wrapper.
-El wrapper queda sin mantenimiento.
-Terminas debugueando el wrapper.
-```
-
-Sáltate el wrapper. Cuando la solución nativa es genuinamente insuficiente
-(soporte de navegadores viejos, edge cases que no cubre, ergonomía que importa
-a escala), la librería se gana su lugar — instálala entonces, no antes
-(y verifica su vigencia con Context7, regla `usar-context7.md`).
+# Soluciones nativas de plataforma
+
+Recurso de `prevencion-sobreingenieria` — peldaño 4 de la escalera de decisión.
+La primera pregunta del ingeniero perezoso-eficiente: *¿la plataforma ya hace
+esto?* Antes de instalar un paquete o escribir un componente, revisar aquí.
+La plataforma viaja gratis con la app, no se rompe en updates, y la escribió
+gente cuyo trabajo es exactamente ese problema.
+
+Adaptado de [ponytail](https://github.com/DietrichGebert/ponytail)
+`docs/platform-native.md` (MIT, DietrichGebert).
+
+---
+
+## Elementos HTML
+
+| Crees que necesitas | La plataforma ya tiene |
+|---|---|
+| Lib de date picker | `<input type="date">` |
+| Lib de time picker | `<input type="time">` |
+| Lib de color picker | `<input type="color">` |
+| Lib de range slider | `<input type="range">` |
+| Componente de progress bar | `<progress value="70" max="100">` |
+| Componente meter/gauge | `<meter value="0.7">` |
+| Lib de modal/diálogo | `<dialog>` + `dialog.showModal()` |
+| Componente acordeón/FAQ | `<details><summary>Título</summary>…</details>` |
+| Lib de tooltips | atributo `title` + CSS `::before`/`::after` |
+| Dropdown con búsqueda | `<input list="id"> <datalist id="id">` |
+| Textarea auto-crecible | `field-sizing: content` (CSS) |
+| Header sticky | `position: sticky; top: 0` (CSS) |
+
+## Capacidades CSS (donde se suele usar JS)
+
+| Crees que necesitas JS para | CSS ya tiene |
+|---|---|
+| Font size responsivo | `font-size: clamp(1rem, 2.5vw, 2rem)` |
+| Espaciado fluido | `padding: clamp(1rem, 5vw, 3rem)` |
+| Dark mode | `@media (prefers-color-scheme: dark)` |
+| Reduced motion | `@media (prefers-reduced-motion: reduce)` |
+| Layout responsivo sin breakpoints | `grid-template-columns: repeat(auto-fill, minmax(250px, 1fr))` |
+| Responsive a nivel componente | `@container` queries |
+| Design tokens / theming global | Custom properties (`--color-primario: #7c3aed`) |
+| Smooth scroll | `scroll-behavior: smooth` |
+| Carrusel con scroll-snap | `scroll-snap-type: x mandatory` + `scroll-snap-align: start` |
+| Aspect ratio | `aspect-ratio: 16 / 9` |
+| Truncar texto con elipsis | `overflow: hidden; text-overflow: ellipsis; white-space: nowrap` |
+| Clamp multilínea | `-webkit-line-clamp: 3` |
+| Aislamiento de estilos por capas | `@layer base, components, utilities` |
+| Selectores anidados | CSS nesting nativo (sin preprocesador) |
+| Selector de padre | `:has(input:checked)` |
+
+## JavaScript / APIs del navegador
+
+| Crees que necesitas | La plataforma ya tiene |
+|---|---|
+| `query-string` / `qs` | `new URLSearchParams(location.search)` |
+| `lodash.clonedeep` | `structuredClone(obj)` |
+| `lodash.groupby` | `Object.groupBy(arr, fn)` |
+| `numeral` / `accounting` | `new Intl.NumberFormat("es-MX", { style: "currency", currency: "MXN" })` |
+| `date-fns` format | `new Intl.DateTimeFormat("es-MX", { dateStyle: "long" }).format(fecha)` |
+| Tiempo relativo | `new Intl.RelativeTimeFormat("es", { numeric: "auto" }).format(-3, "day")` |
+| Plurales i18n | `new Intl.PluralRules("es-MX").select(cantidad)` |
+| `clipboard.js` | `navigator.clipboard.writeText(texto)` |
+| `uuid` (v4) | `crypto.randomUUID()` |
+| Lib de infinite scroll | `new IntersectionObserver(cb).observe(sentinela)` |
+| Lib de resize listener | `new ResizeObserver(cb).observe(elemento)` |
+| Watcher de mutaciones DOM | `new MutationObserver(cb).observe(el, opciones)` |
+| Check de conectividad | `navigator.onLine` + eventos `online`/`offline` |
+| Lib de share sheet | `navigator.share({ title, text, url })` |
+| `store.js` / `localForage` (caso simple) | `localStorage.setItem(clave, JSON.stringify(valor))` |
+| Abortar fetch por timeout | `AbortSignal.timeout(5000)` pasado a `fetch` |
+| Event bus custom | `new EventTarget()` / `dispatchEvent(new CustomEvent("x", { detail }))` |
+
+**Debounce en 3 líneas** (sin dependencia):
+
+```js
+// simplificado: debounce de una sola instancia; lib si se necesitan N independientes
+let t;
+const debounce = (fn, ms) => (...args) => { clearTimeout(t); t = setTimeout(() => fn(...args), ms); };
+```
+
+## Swift / SwiftUI
+
+| Crees que necesitas | La plataforma ya tiene |
+|---|---|
+| Lib de date/time picker | `DatePicker` |
+| Lib de color picker | `ColorPicker` |
+| Search bar + filtrado | `.searchable(text:)` |
+| Lib de pull-to-refresh | `.refreshable { }` |
+| Swipe-to-delete / acciones de fila | `.swipeActions { }` |
+| Carga async de imágenes + caché | `AsyncImage` |
+| Lib de gráficas | Swift Charts (`import Charts`) |
+| Render de Markdown | `Text(...)` markdown / `AttributedString(markdown:)` |
+| Share sheet | `ShareLink` |
+| Spinner de carga | `ProgressView()` |
+| Photo picker | `PhotosPicker` |
+| JSON (SwiftyJSON) | `Codable` + `JSONDecoder`/`JSONEncoder` |
+| HTTP simple (Alamofire) | `URLSession` async/await |
+| Crypto (CryptoSwift) | `CryptoKit` |
+| Persistencia / ORM | `SwiftData`, o `@AppStorage` para key-values |
+| Logging | `Logger` (`os.log`) |
+
+## Stdlib de Node.js
+
+| Crees que necesitas | Node ya tiene |
+|---|---|
+| `mkdirp` / `make-dir` | `fs.mkdirSync(ruta, { recursive: true })` |
+| `rimraf` | `fs.rmSync(ruta, { recursive: true, force: true })` |
+| `uuid` (v4) | `crypto.randomUUID()` |
+| `object-assign` | `Object.assign()` / spread |
+| `array-uniq` | `[...new Set(arr)]` |
+| `array-flatten` / `flat` | `arr.flat(profundidad)` |
+| `path-exists` | `fs.existsSync(ruta)` |
+| `load-json-file` | `JSON.parse(fs.readFileSync(ruta, "utf8"))` |
+| `write-json-file` | `fs.writeFileSync(ruta, JSON.stringify(obj, null, 2))` |
+| `pkg-dir` | `path.resolve(__dirname, "..")` / `import.meta.dirname` |
+
+## Stdlib de Python
+
+| Crees que necesitas | Python ya tiene |
+|---|---|
+| `python-dateutil` (parsing básico) | `datetime.fromisoformat()` (3.7+) |
+| `pytz` | `zoneinfo.ZoneInfo("America/Mexico_City")` (3.9+) |
+| `attrs` (data classes simples) | `@dataclass` |
+| `pathlib2` | `pathlib.Path` (3.4+) |
+| `simplejson` (uso básico) | `json` (stdlib) |
+| `requests` (GET simple) | `urllib.request.urlopen(url)`; `requests` para algo real |
+| `click` (comando único) | `argparse` (stdlib) |
+| `mergedeep` | `dict \| otro_dict` (3.9+) |
+| `more-itertools` (básico) | `itertools`: `chain`, `islice`, `groupby`, `product` |
+| `toolz` (básico) | `functools`: `lru_cache`, `partial`, `reduce` |
+
+## Base de datos
+
+| Crees que necesitas código de app para | La BD ya tiene |
+|---|---|
+| Paginación | `LIMIT 20 OFFSET 40` |
+| Totales acumulados | `SUM(...) OVER (ORDER BY fecha)` (window function) |
+| Rank por grupo | `RANK() OVER (PARTITION BY categoria ORDER BY score DESC)` |
+| Deduplicación | `SELECT DISTINCT` / `ON CONFLICT DO NOTHING` |
+| Recorrido de árboles | CTE recursivo (`WITH RECURSIVE`) |
+| Full-text search (básico) | `tsvector` / `MATCH AGAINST` / `FTS5` |
+| Almacenar + consultar JSON | `jsonb` (Postgres) / `JSON_EXTRACT` (SQLite/MySQL) |
+| Generación de UUID | `gen_random_uuid()` (Postgres) / `UUID()` (MySQL) |
+| Timestamps de insert/update | `DEFAULT now()` + trigger / `ON UPDATE CURRENT_TIMESTAMP` |
+| Unicidad | constraint `UNIQUE`, no checks en la app |
+| Integridad referencial | `FOREIGN KEY`, no checks en la app |
+| Rangos de valores | `CHECK (precio > 0)`, no validación en la app |
+
+---
+
+## El patrón
+
+En todas las capas es el mismo ciclo:
+
+```
+El equipo de la plataforma pasa años resolviendo el problema.
+Alguien lo envuelve en un paquete.
+Instalas el wrapper.
+El wrapper queda sin mantenimiento.
+Terminas debugueando el wrapper.
+```
+
+Sáltate el wrapper. Cuando la solución nativa es genuinamente insuficiente
+(soporte de navegadores viejos, edge cases que no cubre, ergonomía que importa
+a escala), la librería se gana su lugar — instálala entonces, no antes
+(y verifica su vigencia con Context7, regla `usar-context7.md`).

package/habilidades/prevencion-sobreingenieria/recursos/variables-residuales-post-refactor.md

@@ -1,85 +1,85 @@
-# Variables residuales post-refactor son code smell
-
-Recurso de `prevencion-sobreingenieria`. Extraído del SKILL.md en v1.3.0 para
-respetar el límite de 300 líneas; el contenido es normativo sin cambios.
-
-## Regla
-
-Variables asignadas pero no usadas (F841 en ruff/flake8) **post-refactor** son
-casi siempre residuos de una versión anterior del código que el refactor no
-limpió. Se parecen a "variables temporales del desarrollador" pero delatan un
-refactor incompleto — la lógica que las usaba se movió o eliminó, pero las
-asignaciones quedaron.
-
-## Por qué son peligrosas
-
-- **Confunden al próximo lector**: asumirá que la variable se usa más abajo.
-- **Inflan el diff cognitivo** del refactor real.
-- **Ocultan bugs sutiles**: el valor asignado puede tener side effects (llamada
-  a función costosa o con efecto en estado) que ya no aporta nada al flujo.
-- **Invalidan la métrica de cobertura**: líneas "cubiertas" por tests que en
-  realidad no afectan el resultado.
-
-## Patrón operativo
-
-Después de **cualquier refactor** que modifique el flujo de una función (extraer
-método, inlinear función, cambiar el shape de un return), correr:
-
-```bash
-ruff check --select=F841 ruta/al/modulo.py
-# o para todo el proyecto
-ruff check --select=F841 .
-```
-
-Cada F841 detectado post-refactor es candidato 1 de 2:
-
-1. **Variable muerta** (caso común post-refactor): eliminarla.
-2. **Variable viva pero mal nombrada** (caso raro): si el valor se usa como
-   side effect intencional (ej: evaluar una propiedad para cachear), renombrar
-   con prefijo `_` y comentar el propósito.
-
-## Anti-patrón típico
-
-```python
-# Antes del refactor: había un if que usaba exito_key
-def procesar(data):
-    exito, exito_key = intentar(data)
-    if exito_key == "via_rapida":
-        return procesar_rapido(data)
-    return procesar_normal(data, exito_key)
-
-
-# Después de refactor que eliminó la rama "via_rapida"
-def procesar(data):
-    exito, exito_key = intentar(data)   # ← F841: exito_key asignado, no usado
-    return procesar_normal(data)         # ← también exito residual
-```
-
-## Patrón canónico
-
-```python
-# Post-refactor limpio
-def procesar(data):
-    _ = intentar(data)   # si el side effect de intentar() importa
-    return procesar_normal(data)
-
-# o mejor: si ni siquiera el side effect importa
-def procesar(data):
-    return procesar_normal(data)
-```
-
-## Reglas
-
-- **Gate pre-commit**: agregar `ruff check --select=F841` al pre-commit hook del proyecto. No bloquea en un repo grande donde F841 preexiste, pero sí bloquea en los archivos modificados por el commit actual.
-- **Revisión explícita post-refactor grande**: tras extraer 5+ métodos o mover 5+ bloques, correr `ruff --select=F401,F841` sobre los archivos tocados.
-- **NO silenciar con `# noqa: F841` sin justificación** — la excepción legítima (side effect intencional con nombre `_`) es tan rara que vale la pena documentarla; silenciar sin comentario es ocultar deuda.
-- **Incluir en checklist de PR**: "F841 limpio en archivos tocados" como item explícito.
-
-## Aplicabilidad
-
-Detectable en:
-
-- Python: `ruff --select=F841`, `flake8 --select=F841`.
-- TypeScript/JavaScript: `eslint --rule 'no-unused-vars: error'`.
-- Go: el compilador lo hace obligatorio (`declared but not used`).
-- Rust: warning `unused_variables`, promovido a error con `#![deny(unused_variables)]`.
+# Variables residuales post-refactor son code smell
+
+Recurso de `prevencion-sobreingenieria`. Extraído del SKILL.md en v1.3.0 para
+respetar el límite de 300 líneas; el contenido es normativo sin cambios.
+
+## Regla
+
+Variables asignadas pero no usadas (F841 en ruff/flake8) **post-refactor** son
+casi siempre residuos de una versión anterior del código que el refactor no
+limpió. Se parecen a "variables temporales del desarrollador" pero delatan un
+refactor incompleto — la lógica que las usaba se movió o eliminó, pero las
+asignaciones quedaron.
+
+## Por qué son peligrosas
+
+- **Confunden al próximo lector**: asumirá que la variable se usa más abajo.
+- **Inflan el diff cognitivo** del refactor real.
+- **Ocultan bugs sutiles**: el valor asignado puede tener side effects (llamada
+  a función costosa o con efecto en estado) que ya no aporta nada al flujo.
+- **Invalidan la métrica de cobertura**: líneas "cubiertas" por tests que en
+  realidad no afectan el resultado.
+
+## Patrón operativo
+
+Después de **cualquier refactor** que modifique el flujo de una función (extraer
+método, inlinear función, cambiar el shape de un return), correr:
+
+```bash
+ruff check --select=F841 ruta/al/modulo.py
+# o para todo el proyecto
+ruff check --select=F841 .
+```
+
+Cada F841 detectado post-refactor es candidato 1 de 2:
+
+1. **Variable muerta** (caso común post-refactor): eliminarla.
+2. **Variable viva pero mal nombrada** (caso raro): si el valor se usa como
+   side effect intencional (ej: evaluar una propiedad para cachear), renombrar
+   con prefijo `_` y comentar el propósito.
+
+## Anti-patrón típico
+
+```python
+# Antes del refactor: había un if que usaba exito_key
+def procesar(data):
+    exito, exito_key = intentar(data)
+    if exito_key == "via_rapida":
+        return procesar_rapido(data)
+    return procesar_normal(data, exito_key)
+
+
+# Después de refactor que eliminó la rama "via_rapida"
+def procesar(data):
+    exito, exito_key = intentar(data)   # ← F841: exito_key asignado, no usado
+    return procesar_normal(data)         # ← también exito residual
+```
+
+## Patrón canónico
+
+```python
+# Post-refactor limpio
+def procesar(data):
+    _ = intentar(data)   # si el side effect de intentar() importa
+    return procesar_normal(data)
+
+# o mejor: si ni siquiera el side effect importa
+def procesar(data):
+    return procesar_normal(data)
+```
+
+## Reglas
+
+- **Gate pre-commit**: agregar `ruff check --select=F841` al pre-commit hook del proyecto. No bloquea en un repo grande donde F841 preexiste, pero sí bloquea en los archivos modificados por el commit actual.
+- **Revisión explícita post-refactor grande**: tras extraer 5+ métodos o mover 5+ bloques, correr `ruff --select=F401,F841` sobre los archivos tocados.
+- **NO silenciar con `# noqa: F841` sin justificación** — la excepción legítima (side effect intencional con nombre `_`) es tan rara que vale la pena documentarla; silenciar sin comentario es ocultar deuda.
+- **Incluir en checklist de PR**: "F841 limpio en archivos tocados" como item explícito.
+
+## Aplicabilidad
+
+Detectable en:
+
+- Python: `ruff --select=F841`, `flake8 --select=F841`.
+- TypeScript/JavaScript: `eslint --rule 'no-unused-vars: error'`.
+- Go: el compilador lo hace obligatorio (`declared but not used`).
+- Rust: warning `unused_variables`, promovido a error con `#![deny(unused_variables)]`.

package/habilidades/proceso-debate-adversarial/recursos/personas.md

@@ -43,10 +43,11 @@ Presupuesto: `max_hallazgos_por_persona = presupuesto_total / num_personas`
 - **Red flags**: errores tragados, retries faltantes, sin circuit breakers, fallas silenciosas.
 
 ### 5. Abogado del Diablo
-- **Enfoque**: asunciones, casos límite, complejidad oculta, mantenibilidad.
-- **Preguntas guía**: ¿Qué asunciones son falsas? ¿Qué rompe esto? ¿Está sobre-ingenierizado?
-- **Evidencia exigida**: contraejemplos concretos, escenarios de caso límite.
-- **Red flags**: diseño solo-happy-path, asunciones sin probar, complejidad sin justificación.
+- **Enfoque**: asunciones, casos límite, complejidad oculta, mantenibilidad — y la legitimidad misma de la decisión.
+- **Preguntas guía**: ¿Qué asunciones son falsas? ¿Qué rompe esto? ¿Está sobre-ingenierizado? ¿Qué alternativa más simple se descartó sin articularla? Pre-mortem exprés: si esto fracasa en 6 meses, ¿cuál es la causa más probable y qué señal de HOY la anticipaba?
+- **Evidencia exigida**: contraejemplos concretos, escenarios de caso límite, y al menos 1 alternativa descartada con su tradeoff real (no de paja).
+- **Red flags**: diseño solo-happy-path, asunciones sin probar, complejidad sin justificación, "es la mejor práctica" sin fuente, "no hay otra opción razonable".
+- **Escalamiento**: si la duda es sobre HACER o NO el cambio (no sobre cómo), esta persona es insuficiente — usar `/swl:predecir --abogado-diablo` (agente `abogado-diablo-swl`: pre-mortem completo + doubt-driven review + dictamen que puede ser NO PROCEDER).
 
 ---