@saulwade/swl-ses 2.2.0 → 2.2.3
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 +199 -196
- package/README.md +597 -579
- package/agentes/arquitecto-swl.md +0 -5
- package/agentes/backend-python-swl.md +0 -5
- package/agentes/implementador-swl.md +0 -5
- package/agentes/nemesis-auditor-swl.md +0 -5
- package/agentes/orquestador-swl.md +0 -5
- package/agentes/planificador-swl.md +0 -5
- package/agentes/revisor-codigo-swl.md +0 -5
- package/bin/swl-mcp-server.js +1 -1
- package/comandos/swl/adoptar-proyecto.md +253 -258
- package/comandos/swl/aprender.md +823 -828
- package/comandos/swl/claudemd.md +234 -239
- package/comandos/swl/ejecutar-fase.md +0 -5
- package/comandos/swl/nuevo-proyecto.md +200 -205
- package/comandos/swl/release.md +19 -5
- package/comandos/swl/revisar-impacto.md +0 -5
- package/habilidades/agent-browser/SKILL.md +0 -5
- package/habilidades/angular-moderno/SKILL.md +0 -5
- package/habilidades/api-rest-diseno/SKILL.md +0 -5
- package/habilidades/aprendizaje-continuo/SKILL.md +0 -5
- package/habilidades/auth-patrones/SKILL.md +0 -5
- package/habilidades/build-errors-nextjs/SKILL.md +0 -5
- package/habilidades/changelog-generator/SKILL.md +174 -179
- package/habilidades/checklist-seguridad/SKILL.md +0 -5
- package/habilidades/contenedores-docker/SKILL.md +0 -5
- package/habilidades/datos-etl/SKILL.md +0 -5
- package/habilidades/doc-sync/SKILL.md +0 -5
- package/habilidades/extractor-de-aprendizajes/SKILL.md +0 -5
- package/habilidades/fastapi-experto/SKILL.md +0 -5
- package/habilidades/frontend-avanzado/SKILL.md +0 -5
- package/habilidades/iam-secretos/SKILL.md +0 -5
- package/habilidades/manejo-errores/SKILL.md +0 -5
- package/habilidades/mapear-codebase/SKILL.md +0 -5
- package/habilidades/meta-skills-estandar/SKILL.md +0 -5
- package/habilidades/monitoring-alertas/SKILL.md +0 -5
- package/habilidades/nextjs-experto/SKILL.md +0 -5
- package/habilidades/nextjs-testing/SKILL.md +0 -5
- package/habilidades/node-experto/SKILL.md +0 -5
- package/habilidades/orquestacion-async/SKILL.md +0 -5
- package/habilidades/patrones-python/SKILL.md +227 -232
- package/habilidades/planear-fase/SKILL.md +336 -341
- package/habilidades/postgresql-experto/SKILL.md +0 -5
- package/habilidades/prevencion-sobreingenieria/SKILL.md +0 -5
- package/habilidades/protocolo-revision-swl/SKILL.md +0 -5
- package/habilidades/react-experto/SKILL.md +0 -5
- package/habilidades/release-semver/SKILL.md +0 -5
- package/habilidades/swl-claudemd/SKILL.md +0 -5
- package/habilidades/tdd-workflow/SKILL.md +710 -715
- package/habilidades/testing-python/SKILL.md +335 -340
- package/habilidades/verificar-trabajo/SKILL.md +0 -5
- package/hooks/lib/etapa-perfil-usuario.js +1 -1
- package/hooks/lib/evolution-tracker.js +191 -35
- package/hooks/resumen-sesion.js +4 -4
- package/llms.txt +1 -1
- package/manifiestos/canonical-hashes.json +1310 -0
- package/manifiestos/modulos.json +3 -0
- package/manifiestos/skills-lock.json +70 -70
- package/package.json +1 -1
- package/plugin.json +1 -1
- package/scripts/doctor.js +13 -0
- package/scripts/generar-canonical-hashes.js +147 -0
- package/scripts/instalador.js +140 -54
- package/scripts/lib/audit-evolved.js +76 -0
- package/scripts/lib/canonical-hash.js +94 -0
- package/scripts/lib/evolved-fuente.js +138 -0
- package/scripts/lib/manifiestos.js +1 -1
- package/scripts/publicar.js +42 -5
- package/scripts/remediar-evolved-instaladas.js +242 -0
- package/scripts/validar.js +14 -0
- package/scripts/vendor/claude-usage/__pycache__/scanner.cpython-314.pyc +0 -0
- package/scripts/verificar-evolucion.js +36 -0
- package/scripts/verificar-release.js +33 -0
- package/agentes/.evolved.json +0 -9
- package/comandos/swl/.evolved.json +0 -23
- package/habilidades/auth-patrones/.evolved.json +0 -9
- package/habilidades/extractor-de-aprendizajes/.evolved.json +0 -9
- package/habilidades/instalar-sistema/.evolved.json +0 -9
- package/habilidades/manejo-errores/.evolved.json +0 -9
- package/habilidades/node-experto/.evolved.json +0 -9
- package/habilidades/release-semver/.evolved.json +0 -9
|
@@ -1,232 +1,227 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: patrones-python
|
|
3
|
-
description: Idiomas pythonicos, PEP 8, type hints modernos, dataclasses, async/await, context managers, decorators y generators. Patrones de código limpio en Python.
|
|
4
|
-
version: "1.4.2"
|
|
5
|
-
|
|
6
|
-
|
|
7
|
-
|
|
8
|
-
|
|
9
|
-
|
|
10
|
-
|
|
11
|
-
|
|
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
|
-
import
|
|
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
|
-
|
|
167
|
-
|
|
168
|
-
|
|
169
|
-
|
|
170
|
-
|
|
171
|
-
|
|
172
|
-
|
|
173
|
-
|
|
174
|
-
|
|
175
|
-
|
|
176
|
-
|
|
177
|
-
|
|
178
|
-
|
|
179
|
-
|
|
180
|
-
|
|
181
|
-
|
|
182
|
-
|
|
183
|
-
|
|
184
|
-
|
|
185
|
-
|
|
186
|
-
|
|
187
|
-
|
|
188
|
-
|
|
189
|
-
#
|
|
190
|
-
resultado = ""
|
|
191
|
-
|
|
192
|
-
|
|
193
|
-
|
|
194
|
-
|
|
195
|
-
|
|
196
|
-
|
|
197
|
-
|
|
198
|
-
|
|
199
|
-
|
|
200
|
-
|
|
201
|
-
|
|
202
|
-
|
|
203
|
-
-
|
|
204
|
-
-
|
|
205
|
-
- **`
|
|
206
|
-
|
|
207
|
-
|
|
208
|
-
|
|
209
|
-
|
|
210
|
-
|
|
211
|
-
|
|
212
|
-
|
|
213
|
-
|
|
214
|
-
|
|
215
|
-
|
|
216
|
-
|
|
217
|
-
|
|
218
|
-
|
|
219
|
-
|
|
220
|
-
|
|
221
|
-
|
|
|
222
|
-
|
|
223
|
-
|
|
|
224
|
-
|
|
|
225
|
-
|
|
226
|
-
|
|
227
|
-
|
|
228
|
-
| 6 | Tracer/replicador paralelo del motor: marca SYNC obligatoria | duplicar lógica del motor para telemetría, explainability o shadow runs |
|
|
229
|
-
| 7 | Fixtures con datos pre-procesados NO ejercen el path crudo | el motor prefiere leer un campo crudo opcional sobre uno condensado, y los fixtures son condensados |
|
|
230
|
-
|
|
231
|
-
Cargar la sección específica del recurso cuando aplique al problema en mano —
|
|
232
|
-
no cargar el archivo completo si solo se necesita un patrón.
|
|
1
|
+
---
|
|
2
|
+
name: patrones-python
|
|
3
|
+
description: Idiomas pythonicos, PEP 8, type hints modernos, dataclasses, async/await, context managers, decorators y generators. Patrones de código limpio en Python.
|
|
4
|
+
version: "1.4.2"
|
|
5
|
+
herramientasPermitidas: [Read, Glob, Grep]
|
|
6
|
+
exclusiones:
|
|
7
|
+
- "No cargar para patrones de un framework específico (FastAPI, Django, Celery) — los idiomas generales de este skill aplican, pero los patrones de framework tienen restricciones adicionales; cargar el skill del framework correspondiente."
|
|
8
|
+
- "No cargar para errores de instalación o dependencias Python — si el problema es pip, virtualenv o incompatibilidad de versiones, cargar `build-errors-python`."
|
|
9
|
+
- "No cargar para patrones de concurrencia avanzados (TaskGroup, semáforos, event loops) — para esos cargar `async-python` que cubre asyncio en profundidad."
|
|
10
|
+
- "No cargar para testing pytest (fixtures, mocking, cobertura) — ese dominio está en `testing-python` que cubre la toolchain completa de tests."
|
|
11
|
+
evolvable: true # default para skill estandar
|
|
12
|
+
---
|
|
13
|
+
# Patrones Python
|
|
14
|
+
|
|
15
|
+
## Cuándo NO cargar
|
|
16
|
+
|
|
17
|
+
- La tarea es específica de FastAPI o Django — cargar el skill del framework que tiene restricciones adicionales a los patrones generales.
|
|
18
|
+
- El problema es de instalación de dependencias Python — cargar `build-errors-python`.
|
|
19
|
+
- La pregunta es sobre concurrencia avanzada con asyncio — cargar `async-python`.
|
|
20
|
+
|
|
21
|
+
## PEP 8 — Convenciones de estilo
|
|
22
|
+
|
|
23
|
+
### Nomenclatura
|
|
24
|
+
|
|
25
|
+
```python
|
|
26
|
+
# Módulos y paquetes: snake_case minúsculas
|
|
27
|
+
import mi_modulo
|
|
28
|
+
|
|
29
|
+
# Clases: PascalCase
|
|
30
|
+
class FacturaElectronica:
|
|
31
|
+
pass
|
|
32
|
+
|
|
33
|
+
# Funciones y variables: snake_case
|
|
34
|
+
def calcular_iva(monto: float) -> float:
|
|
35
|
+
tasa_iva = 0.16
|
|
36
|
+
return monto * tasa_iva
|
|
37
|
+
|
|
38
|
+
# Constantes: UPPER_SNAKE_CASE
|
|
39
|
+
MAX_REINTENTOS = 3
|
|
40
|
+
URL_BASE_API = "https://api.ejemplo.com/v1"
|
|
41
|
+
|
|
42
|
+
# Variables privadas de instancia: prefijo _
|
|
43
|
+
class Factura:
|
|
44
|
+
def __init__(self):
|
|
45
|
+
self._numero_interno = None # privado por convención
|
|
46
|
+
```
|
|
47
|
+
|
|
48
|
+
### Longitud de línea y formato
|
|
49
|
+
|
|
50
|
+
```python
|
|
51
|
+
# Máximo 88 caracteres (configuración Black estándar)
|
|
52
|
+
|
|
53
|
+
# Importaciones agrupadas:
|
|
54
|
+
# 1. Stdlib
|
|
55
|
+
import os
|
|
56
|
+
from pathlib import Path
|
|
57
|
+
|
|
58
|
+
# 2. Terceros
|
|
59
|
+
from fastapi import FastAPI, Depends
|
|
60
|
+
from sqlalchemy.orm import Session
|
|
61
|
+
|
|
62
|
+
# 3. Módulos locales
|
|
63
|
+
from app.models.usuario import Usuario
|
|
64
|
+
```
|
|
65
|
+
|
|
66
|
+
---
|
|
67
|
+
|
|
68
|
+
## Type Hints modernos (Python 3.10+)
|
|
69
|
+
|
|
70
|
+
```python
|
|
71
|
+
# Union con | en vez de Union[X, Y]
|
|
72
|
+
def procesar(valor: int | str | None) -> str:
|
|
73
|
+
return str(valor or "")
|
|
74
|
+
|
|
75
|
+
# list, dict, tuple nativos (no de typing)
|
|
76
|
+
def filtrar(items: list[str], lookup: dict[str, int]) -> list[str]:
|
|
77
|
+
return [i for i in items if i in lookup]
|
|
78
|
+
|
|
79
|
+
# TypeAlias explícito
|
|
80
|
+
type JSON = dict[str, "JSON"] | list["JSON"] | str | int | float | bool | None
|
|
81
|
+
|
|
82
|
+
# ParamSpec para decoradores que preservan firma
|
|
83
|
+
from typing import ParamSpec, Callable
|
|
84
|
+
P = ParamSpec("P")
|
|
85
|
+
|
|
86
|
+
def con_log(fn: Callable[P, T]) -> Callable[P, T]:
|
|
87
|
+
def wrapper(*args: P.args, **kwargs: P.kwargs) -> T:
|
|
88
|
+
print(f"Llamando {fn.__name__}")
|
|
89
|
+
return fn(*args, **kwargs)
|
|
90
|
+
return wrapper
|
|
91
|
+
```
|
|
92
|
+
|
|
93
|
+
---
|
|
94
|
+
|
|
95
|
+
## Dataclasses
|
|
96
|
+
|
|
97
|
+
```python
|
|
98
|
+
from dataclasses import dataclass, field
|
|
99
|
+
from datetime import datetime
|
|
100
|
+
|
|
101
|
+
@dataclass
|
|
102
|
+
class Producto:
|
|
103
|
+
id: str
|
|
104
|
+
nombre: str
|
|
105
|
+
precio: float
|
|
106
|
+
activo: bool = True
|
|
107
|
+
etiquetas: list[str] = field(default_factory=list)
|
|
108
|
+
|
|
109
|
+
def __post_init__(self) -> None:
|
|
110
|
+
if self.precio < 0:
|
|
111
|
+
raise ValueError(f"Precio no puede ser negativo: {self.precio}")
|
|
112
|
+
self.nombre = self.nombre.strip()
|
|
113
|
+
|
|
114
|
+
# Dataclass inmutable
|
|
115
|
+
@dataclass(frozen=True)
|
|
116
|
+
class Coordenada:
|
|
117
|
+
latitud: float
|
|
118
|
+
longitud: float
|
|
119
|
+
|
|
120
|
+
# Dataclass con slots (Python 3.10+) — ahorra memoria
|
|
121
|
+
@dataclass(slots=True)
|
|
122
|
+
class EventoAuditoria:
|
|
123
|
+
usuario_id: str
|
|
124
|
+
accion: str
|
|
125
|
+
timestamp: datetime
|
|
126
|
+
```
|
|
127
|
+
|
|
128
|
+
---
|
|
129
|
+
|
|
130
|
+
## Context Managers
|
|
131
|
+
|
|
132
|
+
```python
|
|
133
|
+
from contextlib import contextmanager, asynccontextmanager
|
|
134
|
+
|
|
135
|
+
# Context manager con generador
|
|
136
|
+
@contextmanager
|
|
137
|
+
def medir_tiempo(nombre: str):
|
|
138
|
+
inicio = time.perf_counter()
|
|
139
|
+
try:
|
|
140
|
+
yield
|
|
141
|
+
finally:
|
|
142
|
+
transcurrido = time.perf_counter() - inicio
|
|
143
|
+
print(f"{nombre}: {transcurrido:.4f}s")
|
|
144
|
+
|
|
145
|
+
# Context manager async
|
|
146
|
+
@asynccontextmanager
|
|
147
|
+
async def sesion_db(engine):
|
|
148
|
+
async with engine.begin() as conn:
|
|
149
|
+
try:
|
|
150
|
+
yield conn
|
|
151
|
+
await conn.commit()
|
|
152
|
+
except Exception:
|
|
153
|
+
await conn.rollback()
|
|
154
|
+
raise
|
|
155
|
+
```
|
|
156
|
+
|
|
157
|
+
---
|
|
158
|
+
|
|
159
|
+
## Anti-patrones Python Críticos
|
|
160
|
+
|
|
161
|
+
```python
|
|
162
|
+
# MAL: argumento mutable como default
|
|
163
|
+
def agregar(item, lista=[]): # La lista persiste entre llamadas
|
|
164
|
+
lista.append(item)
|
|
165
|
+
|
|
166
|
+
# BIEN: usar None como centinela
|
|
167
|
+
def agregar(item, lista=None):
|
|
168
|
+
if lista is None:
|
|
169
|
+
lista = []
|
|
170
|
+
lista.append(item)
|
|
171
|
+
|
|
172
|
+
# MAL: bare except
|
|
173
|
+
try:
|
|
174
|
+
operacion_riesgosa()
|
|
175
|
+
except:
|
|
176
|
+
pass # silencia KeyboardInterrupt y SystemExit
|
|
177
|
+
|
|
178
|
+
# BIEN: excepción específica
|
|
179
|
+
try:
|
|
180
|
+
operacion_riesgosa()
|
|
181
|
+
except ValueError as e:
|
|
182
|
+
logger.error(f"Valor inválido: {e}")
|
|
183
|
+
|
|
184
|
+
# MAL: concatenación en loop (O(n²))
|
|
185
|
+
resultado = ""
|
|
186
|
+
for parte in partes:
|
|
187
|
+
resultado += parte
|
|
188
|
+
|
|
189
|
+
# BIEN: join (O(n))
|
|
190
|
+
resultado = "".join(partes)
|
|
191
|
+
```
|
|
192
|
+
|
|
193
|
+
Para generators, decoradores avanzados, async/await y manejo de errores idiomático,
|
|
194
|
+
ver [recursos/referencia-completa.md](recursos/referencia-completa.md).
|
|
195
|
+
|
|
196
|
+
## Gotchas / Errores comunes no obvios
|
|
197
|
+
|
|
198
|
+
- **`dataclass` con campo mutable como default (`field: list = []`) produce un objeto compartido entre todas las instancias**: Python reutiliza el objeto por defecto en vez de crear uno nuevo. Causa: los defaults de dataclass se evalúan en tiempo de definición de la clase, no en cada instanciación — una lista mutable como default es el mismo objeto en memoria para todas las instancias. Solución: usar `field(default_factory=list)` para campos mutables: `from dataclasses import field; tags: list = field(default_factory=list)`.
|
|
199
|
+
- **Decorator que usa `functools.wraps` pero no preserva type hints si la función decorada tiene anotaciones genéricas**: el `@wraps` copia `__wrapped__`, `__doc__` y `__name__`, pero el tipo de retorno inferido por `mypy` es el del wrapper, no el del wrapped. Causa: `functools.wraps` no puede preservar el tipado estático del wrapped — mypy ve el tipo del wrapper `Callable[..., Any]`. Solución: usar `TypeVar` y tipado genérico en el decorator con `ParamSpec` (Python 3.10+): `P = ParamSpec('P'); T = TypeVar('T')` y tipar el wrapper como `Callable[P, T]`.
|
|
200
|
+
- **`__slots__` en clase Python produce `TypeError: multiple bases have instance lay-out conflict`** al heredar de otra clase con `__slots__`: las subclases con `__slots__` requieren que todos los ancestros también tengan `__slots__`, o que el ancestro directo sea `object`. Causa: si `ClaseBase` no tiene `__slots__`, tiene un `__dict__` implícito; si `ClaseHija` tiene `__slots__`, hay conflicto de layout de memoria. Solución: o agregar `__slots__ = ()` vacío a la clase base, o eliminar `__slots__` de la subclase — no mezclar clases con y sin `__slots__` en la misma jerarquía.
|
|
201
|
+
- **`property` setter que modifica un campo privado no refleja el cambio en `__repr__` generado por dataclass**: el `@property` en un dataclass crea un campo de clase que conflictúa con el campo de instancia del dataclass. Causa: `@dataclass` genera `__repr__` basado en los campos declarados en `__init__` — si el setter modifica un atributo con nombre diferente (ej: `_valor`), `__repr__` muestra el campo original sin la modificación. Solución: usar `field(init=False, repr=False)` para el campo interno y exponer solo la `property` en la interfaz pública.
|
|
202
|
+
- **`assert` no es guard de invariantes en producción con `PYTHONOPTIMIZE=1` o `python -O`**: el bytecode optimizado **elimina** todos los `assert` del módulo, por lo que `assert x is not None; return x` puede retornar `None` violando el contrato `-> dict` en producción aunque pase tests en desarrollo. El test runner por defecto NO usa `-O`, por lo que el bug es invisible hasta que alguien despliega con `PYTHONOPTIMIZE=1` (configuración común para reducir memoria en imágenes Docker production). Causa: `assert` está documentado en Python como herramienta de **debugging**, no de validación. Solución: para invariantes que DEBEN cumplirse en producción, usar guard explícito con raise: `if x is None: raise HTTPException(500, "Invariante violado")` o `if x is None: raise RuntimeError(...)`. Reservar `assert` solo para tests, scripts, o pre-condiciones triviales en código de desarrollo. Regla rápida: si el assert protege un caso que activa una respuesta del usuario o un side-effect, NO es assert — es validación y debe ser `if/raise`.
|
|
203
|
+
- **Pasar `None` explícito a un parámetro NO activa su valor por defecto en el callee**: si una función declara `def listar(activa: bool = True)` y el caller hace `listar(activa=valor)` donde `valor` puede ser `None` (típico con parámetros opcionales que viajan entre capas: `activa: bool | None = None`), el callee recibe `None`, NO `True`. Python aplica el default SOLO cuando el argumento se **omite literalmente** en la llamada, no cuando recibe `None` explícito. Causa: el binding del parámetro usa el valor pasado, sea cual sea — el default es fallback de *ausencia*, no de *nulidad*. El bug es silencioso: pasa los tests donde el caller omite el parámetro y falla en producción cuando el cliente omite el valor y una capa intermedia lo traduce a `None` antes de propagarlo. Caso real: un endpoint que documentaba "omitir devuelve solo activos" retornaba inactivos porque el router pasaba `activa=None` al service cuyo default era `True`. Solución: normalizar en la frontera — en el caller (`listar(activa=True if valor is None else valor)`) o tratando `None` como sentinel en el callee (`if activa is None: activa = True`). Regla: cuando un valor opcional cruza de una capa a otra y el callee tiene un default distinto de `None`, convertir `None → default` explícitamente en el punto de cruce.
|
|
204
|
+
- **`getattr(obj, "attr", default)` tampoco aplica `default` si el atributo existe con valor `None`**: misma raíz que el gotcha anterior, otra manifestación. `getattr` usa `default` SOLO cuando el atributo **no existe**; si existe con valor `None` (caso típico: campos opcionales de Pydantic/SQLAlchemy inicializados a `None`), devuelve `None`, no `default`. Así `getattr(modelo, "campo_opcional", "X")` retorna `None` cuando `campo_opcional is None`, no `"X"`. Solución sin ambigüedad: `raw = getattr(modelo, "campo_opcional", None); valor = raw if raw is not None else "X"` (el atajo `getattr(...) or "X"` solo sirve si `""` y `0` son aceptables como falsy). Regla unificada para ambos casos: en Python el `default` —de parámetro o de `getattr`— es fallback de **ausencia**, nunca de **nulidad**; si `None` es un valor posible, normalízalo en la frontera.
|
|
205
|
+
- **`except A, B:` sin paréntesis es sintaxis válida en Python ≥3.14 (PEP 758), NO un SyntaxError**: desde 3.14 los paréntesis en `except`/`except*` con múltiples tipos son opcionales (`except ValueError, TypeError:` ≡ `except (ValueError, TypeError):`). Bajo intérpretes <3.14 esa forma SÍ es SyntaxError, lo que confunde a herramientas y revisores que juzgan la sintaxis contra su conocimiento general del lenguaje en vez de contra la versión objetivo del proyecto (`requires-python`/`tool.ruff.target-version` en `pyproject.toml`). Causa: la validez de la sintaxis depende de la versión, no es absoluta. Efecto colateral a vigilar: `ruff format` con `target-version = "py314"` **quita** los paréntesis redundantes y, además, reformatea el archivo COMPLETO — puede tocar `except` ajenos a tu cambio (churn no intencional); `ruff check` (lo único que suele correr el CI) conserva la forma que encuentre. Solución: antes de marcar cualquier `except A, B:` como error, leer la versión objetivo del proyecto y verificar con evidencia ejecutable (`python -c "import <módulo>"` + `ruff check`). Si se prefieren los paréntesis por portabilidad/Pyright, restaurarlos a mano y NO correr `ruff format` sobre esas líneas. Caso real (2026-06-05): un proyecto con `requires-python >=3.14` recibió un falso positivo de SyntaxError CRÍTICO por esta forma; refutado con import + ruff + CI verde.
|
|
206
|
+
|
|
207
|
+
---
|
|
208
|
+
|
|
209
|
+
## Patrones avanzados (carga bajo demanda)
|
|
210
|
+
|
|
211
|
+
Los siguientes patrones aparecen al integrar Python con pipelines reales
|
|
212
|
+
(conversores de documentos, clientes heterogéneos, cachés determinísticos,
|
|
213
|
+
detectores de texto, duplicación deliberada de lógica). El detalle completo
|
|
214
|
+
está en [recursos/patrones-avanzados.md](recursos/patrones-avanzados.md).
|
|
215
|
+
|
|
216
|
+
| # | Patrón | Cargar cuando |
|
|
217
|
+
|---|--------|---------------|
|
|
218
|
+
| 1 | Normalizadores: colapsar al formato canónico del PROYECTO | escribir `_normalizar(texto)` después de un conversor externo (MarkItDown, Pandoc, mammoth) |
|
|
219
|
+
| 2 | kwargs opcionales entre clientes hermanos: try/except TypeError | propagar un kwarg nuevo a clientes con signature heterogénea sin romper la interfaz |
|
|
220
|
+
| 3 | Caché content-addressable por SHA256 | función pura costosa (>1 s) que se invoca repetidamente con los mismos inputs |
|
|
221
|
+
| 4 | F401 en archivos con soft imports es intencional | módulo importa una dependencia opcional dentro de `try/except ImportError` |
|
|
222
|
+
| 5 | Detectores regex multi-pattern: extender scope sin refinar genera FP | un detector con OR de patterns se va a aplicar a un texto más ruidoso que el original |
|
|
223
|
+
| 6 | Tracer/replicador paralelo del motor: marca SYNC obligatoria | duplicar lógica del motor para telemetría, explainability o shadow runs |
|
|
224
|
+
| 7 | Fixtures con datos pre-procesados NO ejercen el path crudo | el motor prefiere leer un campo crudo opcional sobre uno condensado, y los fixtures son condensados |
|
|
225
|
+
|
|
226
|
+
Cargar la sección específica del recurso cuando aplique al problema en mano —
|
|
227
|
+
no cargar el archivo completo si solo se necesita un patrón.
|