Antes del
LGTM
- workshop online
- en directo
- 3–4 horas
Un protocolo para revisar código generado por IA en menos de 10 minutos por cambio: qué delegar, qué mirar y cómo decir que no. Venga como PR, como commit sobre main o como diff de tu agente.
Para seniors, leads y staff que deciden qué código entra en main.
Apuntarte no compromete a nada. La lista de espera accede primero a las 20 plazas, con precio early bird.
Imparte Emilio Carrión, Staff Engineer en Mercadona Tech · 28 ponencias en 6 países · 5.000+ ingenieros leen su newsletter
Tu método de review se diseñó para otro mundo
Se diseñó para uno o dos cambios al día, escritos por un humano que sufrió cada línea. Ahora recibes cinco o seis diarios que nadie sufrió. El código generado por IA no es peor: es más. Más volumen por hora de código mediocre-plausible, con formato impecable y tests en verde.
Y el código plausible es más peligroso que el incorrecto. El incorrecto lo caza el CI. El plausible entra, se sedimenta, y lo pagas en mantenimiento seis meses después. El enemigo no es el bug: es el sedimento.
Revisar más rápido no es la salida. Revisar menos cosas, mejor elegidas, sí.
La review cambió de trabajo
Ya no valida corrección; juzga necesidad. La corrección la validan los tests, el CI y los revisores automáticos, cada vez mejor. Lo que ninguna máquina responde por ti: ¿esto debería existir? ¿En esta forma? ¿En este repo?
El CI te dice si funciona. Tú decides si debe existir.
Esto no es un curso de prompting ni de ninguna herramienta. Es entrenamiento de criterio.
Este cambio pasa el CI. Y tiene cinco problemas.
Objetivo del ticket: «añadir un reintento al cobro fallido». El diff que llega: +412 −9
+ retry:
+ enabled: true
+ max_attempts: 3
+ backoff: "exponential"+ export interface RetryStrategy {
+ execute(charge: Charge): Promise<Result>
+ }
+ export class RetryStrategyFactory {
+ static create(type: string): RetryStrategy {
+ switch (type) {
+ case "exponential":
+ return new ExponentialRetry()
+ default:
+ throw new Error(`Unknown: ${type}`)
+ }
+ }
+ }+ it("retries the charge", async () => {
+ const s = { execute: vi.fn()
+ .mockResolvedValue(ok) }
+ await retryCharge(charge, s)
+ expect(s.execute).toHaveBeenCalled()
+ })-
Archivo nuevo con nombre de patrón, no de dominio
¿Qué concepto nuevo nombra RetryStrategyFactory? Ninguno. Patrón sin dominio = sospecha. La regla del archivo nuevo lo frena en 10 segundos.
-
Interface con una sola implementación
Generalizó con el primer ejemplo imaginado. Se generaliza con el segundo caso real, nunca con el primero imaginado.
-
Config para un valor que nadie va a cambiar
Tres reintentos, fijos desde el día uno. La configuración es código que no puedes testear: una superficie más de error por entorno.
-
El test verifica el mock
Comprueba que el stub devuelve lo que el propio test le puso. Cobertura en verde, protección cero. Teatro, no tests.
-
+412 −9 para «añadir un reintento»
La desproporción entre objetivo y diff es la señal más barata que existe. Se ve antes de leer una sola línea de código.
Ningún problema de esta lista es un bug, y ninguno lo va a parar tu CI. En el taller entrenas a ver estas señales a velocidad de diff, sin leer el código entero.
El protocolo de 10 minutos
Time-boxed a propósito: un protocolo de 40 minutos por cambio no sobrevive al lunes. Este sí.
- min 0–1
Procedencia y propósito
¿Quién o qué generó esto? ¿Qué problema resuelve? Un diff sin porqué no es revisable: es descifrable.
- min 1–3
Escaneo de forma
Antes de leer código: archivos nuevos, ratio diff/objetivo, zonas tocadas, proporción de tests. Aquí caza la mitad de las señales.
- min 3–8
Segunda lectura
Solo sobre los puntos calientes que el escaneo señaló. La primera lectura (correctness) ya la delegaste.
- min 8–10
Tres preguntas y veredicto
Aprobar, pedir cambios con dirección concreta, o rechazar y reencuadrar. Siempre con el porqué escrito.
Si a los 10 minutos no puedes emitir veredicto, el problema es del cambio, no tuyo.
Y te lo llevas todo por escrito
- La hoja del protocolo de 10 minutos (una página)
- La chuleta de los 7 principios que la IA viola: manifestación típica y pregunta que la desactiva
- Las plantillas de veredicto: los «no» sin fricción, listos para copiar
- Los prompts de dirección para regenerar la versión mínima
- El kit de equipo: 5 acuerdos redactados como propuesta de retro
- El repo de práctica con 4 cambios reales y soluciones comentadas
El martes revisas tu primer cambio con el protocolo. El viernes dices tu primer «no» con plantilla. En la próxima retro llevas los 5 acuerdos a tu equipo.
Para quién es (y para quién no)
Es para ti si…
- Decides qué código entra en main. Da igual si con PRs, trunk-based, en pairing o revisando el diff de tu agente antes de aplicar.
- Recibes más cambios generados por IA de los que puedes digerir con tu método actual.
- Eres senior, lead o staff y la review se está comiendo tu día (o tu criterio).
- Has aprobado código que te olía mal por no pagar el coste social del «no».
No es para ti si…
- Buscas un curso de prompting o de una herramienta concreta. Esto es entrenamiento de criterio, agnóstico de lenguaje y de flujo.
- No revisas código: ni el de tu equipo ni el que genera tu propio agente.
- Quieres argumentos para frenar la IA en tu equipo. Esto va de aprovechar su caudal sin tragarte su sedimento.
Emilio Carrión
Llevo años revisando código a escala real (el que escribe la gente y, cada vez más, el que generan las máquinas) y escribiendo sobre el oficio de ingeniería cuando el código ya no es lo escaso. Este taller es la versión entrenable de ese trabajo.
- Staff Engineer en Mercadona Tech
- Doctorando en la UPV en metodologías de producción de software
- 28 ponencias en 6 países: DjangoCon Europe, Nerdearla, Codemotion, T3chFest…
- Autor de «Producto, no código», la guía para seniors sobre el oficio cuando la IA escribe el código
- Newsletter semanal con más de 5.000 ingenieros suscritos
Más en emiliocarrion.com
La lista de espera paga menos
Precios con IVA incluido. Para empresas, factura con IVA desglosado.
Cada edición tiene 20 plazas. No es táctica de marketing: la práctica es por parejas y la revisión de tu cambio real la hago yo, una a una.
Lo que querrás saber antes de apuntarte
¿Cuándo es?
La fecha se anuncia primero a la lista de espera. Será online, en directo, en una franja pensada para que funcione tanto desde España como desde LATAM (tarde europea, mañana americana).
¿Habrá grabación?
Sí, para los inscritos. Pero el taller está diseñado para hacerse en directo: la práctica por parejas y la puesta en común no se pueden ver en diferido.
¿Necesito experiencia con herramientas de IA?
No. Necesitas revisar código, o decidir qué código entra en main. Da igual si la IA la usa tu equipo, tu agente o el compañero que te manda el cambio.
¿Sirve para mi stack y para mi flujo de trabajo?
El protocolo es agnóstico de lenguaje y de flujo. Funciona igual con PRs de GitHub, trunk-based development, pairing o revisión post-merge: lo que se revisa es el diff, no la ceremonia.
¿No puede hacer esta review otra IA?
Una parte sí, y el protocolo te enseña justo a delegársela: la IA revisora pasa primero y tú después, cada uno con su lista. Lo que no puede hacer por ti es la decisión: si una duplicación es deuda o es legítima, si una abstracción justifica su peso, si el cambio debería existir. La IA revisora hereda el mismo sesgo de añadir que la generadora. El taller va de saber qué te queda a ti.
¿Puedo pagarlo como empresa?
Sí, con factura. Y si vais varios del mismo equipo, escríbeme al apuntarte: tiene sentido hacer la práctica con vuestros propios cambios.
¿Y si hago el taller y no me convence?
Haz el taller entero. Si al terminar no tienes un protocolo que puedas aplicar el martes, escríbeme y te devuelvo el 100%. Sin letra pequeña: el riesgo es mío.
¿Por qué solo 20 plazas?
Porque la práctica es por parejas con puesta en común, y porque después del taller reviso yo un cambio real de cada asistente. Eso no escala a 200 personas, y no quiero que escale.
¿Qué pasa exactamente al apuntarme a la lista?
Quedas apuntado, y cuando haya fecha la lista recibe el acceso con precio early bird antes que nadie. Nada más: ni spam ni cadencia de ventas.
Apúntate a la lista de espera
Cuando haya fecha, la lista accede primero a las 20 plazas de la edición, con precio early bird. Apuntarte no compromete a nada: solo asegura que no te enteras tarde.
early bird para la lista · 20 plazas por edición · garantía del 100%
Solo te escribiré para la fecha y el acceso early bird. Ni spam ni cadencia de ventas.
La próxima vez que un cambio de 400 líneas te mire desde la cola, sabrás qué hacer con él en 10 minutos.