# Leer archivo a revisar
cat <path>

# Ver cambios si es git diff
git diff <path>
git diff HEAD~1 <path>

# Ver PR diff
gh pr diff <number>

# Ver archivos cambiados
gh pr view <number> --json files

# Ver comentarios existentes
gh api repos/<owner>/<repo>/pulls/<number>/comments

📍 **file.ts:42**

🔴 **CRÍTICO** - Security Issue
```typescript
// ❌ Actual
const query = `SELECT * FROM users WHERE id = ${userId}`;

// ✅ Sugerido
const query = `SELECT * FROM users WHERE id = $1`;
await db.query(query, [userId]);

// ❌ Actual
users.filter(u => u.active).map(u => u.name).forEach(...)

// ✅ Sugerido
for (const user of users) {
  if (user.active) {
    // process user.name
  }
}

// ❌ Actual
if (condition) {
  return true;
} else {
  return false;
}

// ✅ Sugerido
return condition;

### Severidad de Issues

| Emoji | Level | Description | Action |
|-------|-------|-------------|--------|
| 🔴 | CRÍTICO | Bugs, security issues, data loss | Must fix before merge |
| 🟠 | IMPORTANTE | Logic errors, missing validation | Should fix |
| 🟡 | MEJORA | Performance, readability | Nice to have |
| 🟢 | SUGERENCIA | Style, minor improvements | Optional |
| 💡 | INFO | Educational, tips | FYI |

## Patrones a Detectar

### TypeScript Anti-patterns
```typescript
// ❌ any
function process(data: any): any {}

// ❌ Type assertion sin validación
const user = data as User;

// ❌ Non-null assertion sin verificar
const name = user!.name;

// ❌ Implicit any en callbacks
array.map(item => item.value);

// ❌ Inline objects en props (re-render)
<Component style={{ color: 'red' }} />

// ❌ Index como key
{items.map((item, i) => <Item key={i} />)}

// ❌ State mutation
setState(prev => { prev.push(item); return prev; });

// ❌ useEffect sin deps correctas
useEffect(() => { fetchData(id); }, []);

// ❌ Eval
eval(userInput);

// ❌ dangerouslySetInnerHTML sin sanitizar
<div dangerouslySetInnerHTML={{ __html: userContent }} />

// ❌ Secrets en código
const API_KEY = "sk-abc123";

// ❌ Sin rate limiting
app.post('/api/login', handleLogin);

# Code Review Summary

## Files Reviewed
- `src/api/users.ts` - 5 issues
- `src/components/UserList.tsx` - 2 issues
- `src/utils/helpers.ts` - 1 issue

## Statistics
| Category | Count |
|----------|-------|
| 🔴 Critical | 1 |
| 🟠 Important | 3 |
| 🟡 Improvement | 2 |
| 🟢 Suggestion | 2 |

## Critical Issues (Must Fix)
1. **SQL Injection** in `users.ts:42`
   - Direct string interpolation in SQL query

## Important Issues (Should Fix)
1. **Missing error handling** in `users.ts:78`
2. **Type safety** - `any` type used in `helpers.ts:15`
3. **Memory leak** - Missing cleanup in useEffect

## Summary
Overall, the code is well-structured but has one critical security vulnerability that must be addressed before merging. The other issues are improvements that would enhance maintainability and type safety.

### Recommendation
🔴 **DO NOT MERGE** until critical issue is resolved.