code-review-response
$
npx mdskill add 686f6c61/alfred-dev/code-review-responseEvaluate code review feedback and propose technical solutions.
- Helps developers address reviewer comments and implement changes.
- Depends on PR comments and code repository access.
- Decides recommendations by classifying feedback as actionable or stylistic.
- Delivers responses with evidence and alternative solutions.
SKILL.md
.github/skills/code-review-responseView on GitHub ↗
--- name: code-review-response description: "Usar al recibir feedback de code review para responder técnicamente. Activar cuando el usuario quiera responder a comentarios de PR, gestionar feedback de code review, resolver comentarios de un revisor, o cuando el revisor pide cambios en el código." --- # Responder a code review ## Resumen Este skill gestiona la respuesta técnica a comentarios de code review. El objetivo no es aceptar todo el feedback ciegamente ni rechazarlo por ego, sino evaluarlo técnicamente y responder con evidencia. Un buen proceso de code review mejora el código; un mal proceso genera fricción y resentimiento. La clave es tratar cada comentario como una oportunidad de mejorar el código o de enriquecer la discusión técnica del equipo. ## Proceso 1. **Leer todos los comentarios antes de responder a ninguno.** Obtener una visión global del feedback. A veces un comentario individual cobra sentido diferente cuando se ve junto con los demás. Agrupar mentalmente por temática: estilo, lógica, arquitectura, rendimiento. 2. **Clasificar cada comentario:** - **Correcto y accionable:** el revisor ha detectado un problema real. Implementar el cambio. - **Correcto pero discutible:** el revisor tiene razón en el diagnóstico pero la solución propuesta no es la mejor. Contraargumentar con alternativa. - **Cuestión de estilo:** no hay bien o mal objetivo, es preferencia. Si el proyecto tiene guía de estilo, seguirla. Si no, aceptar a menos que haya buena razón para no hacerlo. - **Incorrecto:** el revisor ha malinterpretado el código o el contexto. Explicar con datos, no con autoridad. - **Fuera de alcance:** el comentario es válido pero no pertenece a este PR. Crear un issue para abordarlo después. 3. **Verificar si el comentario es correcto.** Para cada comentario técnico: - Leer el código señalado con ojos frescos. - Si el comentario reporta un bug, intentar reproducirlo. - Si sugiere un cambio de rendimiento, medir antes de aceptar. - Si propone un refactoring, verificar que los tests cubren el área afectada. 4. **Responder con evidencia, no con opiniones.** Si se acepta el cambio, implementarlo y responder con el commit que lo resuelve. Si se rechaza, explicar por qué con datos: métricas de rendimiento, referencia a una decisión de diseño documentada, test que demuestra que el comportamiento es correcto. 5. **Implementar los cambios aceptados.** Cada cambio derivado de code review se hace en un commit separado y descriptivo que referencie el comentario original cuando sea posible. 6. **No tomárselo como algo personal.** El code review es sobre el código, no sobre la persona. Si un comentario resulta brusco, responder al contenido técnico e ignorar el tono. ## Relación con otros skills Este skill guía la respuesta técnica a comentarios de review. Para hacer la revisión de código en sí (como revisor), usar `calidad/code-review`. ## Qué NO hacer - **No aceptar todas las sugerencias sin evaluar su mérito técnico.** Aceptar ciegamente genera código inconsistente y erosiona la confianza en el proceso de review. Cada cambio debe tener sentido técnico. - **No responder de forma defensiva.** El code review es sobre el código, no sobre la persona. Una respuesta defensiva corta la discusión técnica y empeora el resultado. - **No ignorar comentarios sin explicar por qué.** Si se decide no implementar un cambio sugerido, documentar la razón con evidencia técnica. Un comentario sin respuesta es peor que un rechazo argumentado. ## Criterios de éxito - Todos los comentarios del code review tienen una respuesta (aceptación, rechazo argumentado o discusión). - Los cambios aceptados están implementados y commiteados. - Los rechazos están argumentados con evidencia técnica, no con opiniones. - Los comentarios fuera de alcance se han registrado como issues para seguimiento. - El tono de las respuestas es profesional y constructivo.
More from 686f6c61/alfred-dev
- acceptance-criteriaGenerar criterios de aceptación en formato Given/When/Then. Activar cuando el usuario quiera definir criterios de aceptacion, usar formato Given When Then, escribir en Gherkin, saber como determinar que algo esta terminado o establecer una definicion de hecho.
- architecture-docsUsar para documentar la arquitectura del sistema. Activar ante: documentar arquitectura, diagrama del sistema, como funciona el proyecto, vision general tecnica
- bundle-sizeAnalizar y reducir el tamaño de bundles frontend. Activar cuando el bundle sea grande, se quiera reducir tamaño, aplicar tree shaking, configurar lazy loading, usar webpack analyzer o analizar el peso de la aplicacion.
- choose-stackUsar para evaluar y elegir tecnologías con matriz de decisión ponderada. Activar cuando el usuario quiera elegir tecnología, comparar frameworks, decidir entre alternativas técnicas, construir una matriz de decisión, evaluar stack, seleccionar base de datos, elegir lenguaje o comparar herramientas.
- ci-cd-pipelineConfigurar pipeline CI/CD adaptado al proyecto. Activar cuando el usuario quiera configurar CI, crear GitHub Actions, configurar GitLab CI, montar un pipeline de despliegue, automatizar tests o implementar integracion continua.
- compliance-checkUsar para verificar cumplimiento RGPD, NIS2 y CRA. También: verificar RGPD, cumplimiento normativo, NIS2, CRA, Cyber Resilience Act, protección de datos, regulación europea.
- copy-reviewRevisar textos publicos: claridad, tono, ortografia y CTAs. Activar ante: revisar textos, mejorar copy, tono de comunicacion, textos de la web, landing page copy
- dependency-strategyEstrategia integral de gestion de dependencias: inventario, evaluacion de riesgo, politica de actualizaciones y documentacion. Usar para auditar el estado global de las dependencias del proyecto.
- dependency-updateRevisar dependencias desactualizadas, con CVEs o end-of-life, y proponer actualizaciones seguras. También: actualizar paquetes, actualizar dependencias, Dependabot, Renovate, versión desactualizada, breaking changes.
- deploy-configConfigurar despliegue según hosting. Activar cuando el usuario quiera desplegar en Vercel, Railway, AWS, configurar hosting, preparar para produccion o gestionar variables de entorno de despliegue.