Discusión de code review
Code review discussion
Has abierto un PR para un nuevo prompt de Claude. Diego, tu lead técnico, ha dejado tres comentarios: uno sobre el tamaño del system prompt, otro sobre falta de tests y otro sobre nombres de variables. Tenéis una llamada rápida para resolverlo.
Tu objetivo
Entender los comentarios, defender tus decisiones cuando tienen sentido y aceptar cambios cuando son justos.
Para conseguirlo
- Defendiste al menos una decisión con un argumento técnico.
- Aceptaste al menos un comentario sin ponerte a la defensiva.
- Pediste ayuda concreta donde te faltaba conocimiento.
- Cerraste acordando próximos pasos.
Thanks for the review. Let me explain my reasoning.
Gracias por la review. Te explico mi razonamiento.
Al empezar.
I made it longer because the model was missing edge cases.
Lo hice más largo porque el modelo se saltaba casos límite.
Para defender una decisión.
That's a fair point, I'll shorten it.
Es un buen punto, lo acortaré.
Para aceptar un cambio.
I agree with the variable names. I'll rename them.
Estoy de acuerdo con los nombres. Los renombro.
Para aceptar otro cambio.
About the tests, I wasn't sure how to mock the API.
Sobre los tests, no sabía cómo mockear la API.
Para pedir ayuda concreta.
Could you point me to an example I can follow?
¿Me puedes pasar un ejemplo que pueda seguir?
Para pedir referencia.
Let me push the changes and ping you again.
Hago los cambios y te aviso otra vez.
Para cerrar.
Hey Jorge, thanks for the PR. I left a few comments. Want to walk me through your reasoning on the system prompt size first?