Pr Review

---
description: 'Revisión sistemática de PR. Se activa con "revisar este PR", "code review", "revisar pull request".'
allowed-tools:
  - Bash(gh *)
  - Read
  - Grep
  - Glob
---

# Revisión sistemática de PR

Asegura la calidad del código y solidez arquitectónica a través de revisiones sistemáticas de Pull Request.

## Uso

```bash
# Revisión comprensiva de PR
gh pr view 123 --comments
"Revisar sistemáticamente este PR y proporcionar feedback desde perspectivas de calidad de código, seguridad y arquitectura"

# Revisión enfocada en seguridad
gh pr diff 123
"Enfocarse en revisar riesgos de seguridad y vulnerabilidades"

# Revisión desde perspectiva de arquitectura
gh pr checkout 123 && find . -name "*.js" | head -10
"Evaluar la arquitectura desde perspectivas de separación de capas, dependencias y principios SOLID"
```

## Ejemplos Básicos

```bash
# Evaluación cuantitativa de calidad de código
find . -name "*.js" -exec wc -l {} + | sort -rn | head -5
"Evaluar complejidad de código, tamaño de funciones y duplicación, y señalar mejoras"

# Verificación de vulnerabilidades de seguridad
grep -r "password\|secret\|token" . --include="*.js" | head -10
"Verificar riesgos de filtración de información sensible, hardcoding y bypass de autenticación"

# Detección de violaciones de arquitectura
grep -r "import.*from.*\.\./\.\." . --include="*.js"
"Evaluar violaciones de capas, dependencias circulares y problemas de acoplamiento"
```

## Sistema de Clasificación de Comentarios

```text
🔴 critical.must: Problemas críticos
├─ Vulnerabilidades de seguridad
├─ Problemas de integridad de datos
└─ Riesgos de falla del sistema

🟡 high.imo: Mejoras de alta prioridad
├─ Riesgo de mal funcionamiento
├─ Problemas de rendimiento
└─ Disminución significativa de mantenibilidad

🟢 medium.imo: Mejoras de prioridad media
├─ Mejora de legibilidad
├─ Mejora de estructura de código
└─ Mejora de calidad de pruebas

🟢 low.nits: Puntos menores
├─ Unificación de estilo
├─ Corrección de errores tipográficos
└─ Adición de comentarios

🔵 info.q: Preguntas/información
├─ Confirmación de intención de implementación
├─ Antecedentes de decisiones de diseño
└─ Compartir mejores prácticas
```

## Perspectivas de Revisión

### 1. Corrección de Código

- **Errores lógicos**: Valores límite, verificaciones de null, manejo de excepciones
- **Integridad de datos**: Seguridad de tipos, validación
- **Manejo de errores**: Completitud, procesamiento apropiado

### 2. Seguridad

- **Autenticación/autorización**: Verificaciones apropiadas, gestión de permisos
- **Validación de entrada**: Contramedidas SQL injection, XSS
- **Información sensible**: Restricciones de logging, cifrado

### 3. Rendimiento

- **Algoritmos**: Complejidad temporal, eficiencia de memoria
- **Base de datos**: Consultas N+1, optimización de índices
- **Recursos**: Fugas de memoria, utilización de caché

### 4. Arquitectura

- **Separación de capas**: Dirección de dependencias, separación apropiada
- **Acoplamiento**: Acoplamiento fuerte, utilización de interfaces
- **Principios SOLID**: Responsabilidad única, abierto-cerrado, inversión de dependencias

## Flujo de Revisión

1. **Pre-verificación**: Información de PR, diff de cambios, issues relacionados
2. **Verificaciones sistemáticas**: Seguridad → Corrección → Rendimiento → Arquitectura
3. **Feedback constructivo**: Sugerencias específicas de mejora y ejemplos de código
4. **Seguimiento**: Confirmación de correcciones, estado de CI, aprobación final

## Ejemplos de Comentarios Efectivos

### Problemas de Seguridad

**Formato:**

```text
**critical.must.** [Descripción del problema]

[Código propuesto]

[Explicación de la necesidad]
```

**Ejemplo:**

```text
**critical.must.** La contraseña se almacena en texto plano

// Corrección propuesta
const bcrypt = require('bcrypt');
const hashedPassword = await bcrypt.hash(password, 12);

Se requiere hashing para prevenir riesgos de seguridad.
```

### Mejora de Rendimiento

**Formato:**

```text
**high.imo.** [Descripción del problema]

[Código de mejora]

[Explicación del beneficio]
```

**Ejemplo:**

```text
**high.imo.** Ocurre problema de consulta N+1

// Mejora: Eager Loading
const users = await User.findAll({ include: [Post] });

Esto puede reducir significativamente el número de consultas.
```

### Violación de Arquitectura

**Formato:**

```text
**high.must.** [Descripción de la violación]

[Explicación detallada y solución recomendada]
```

**Ejemplo:**

```text
**high.must.** Ocurrió violación de capa

La capa de dominio depende directamente de la capa de infraestructura.
Por favor introducir una interfaz siguiendo el principio de inversión de dependencias.
```

## Notas

- **Tono constructivo**: Comunicación colaborativa en lugar de agresiva
- **Sugerencias específicas**: Proporcionar soluciones junto con señalar problemas
- **Priorización**: Abordar en orden de Crítico → Alto → Medio → Bajo
- **Mejora continua**: Documentar resultados de revisión en base de conocimiento