Busca en todas las páginas de la documentación
En Solana, una restricción has_one omitida puede costar millones. La cultura de revisión de código equilibra la enseñanza de patrones de Sealevel a los juniors con la agilización del envío: comentarios de alta señal sobre seguridad y semántica de cuentas, bajo ruido en el formato (rustfmt se encarga de eso).
Tarjeta de receta de referencia rápida, lista para copiar y pegar.
## Lista de Revisión de PR de Solana
- [ ] Flags de propietario/firmante/escritura de cuenta correctos
- [ ] Semillas y bumps de PDA validados
- [ ] IDs de programa CPI codificados o en lista blanca
- [ ] Aritmética verificada (sin suma sin verificar)
- [ ] Pruebas: camino feliz + atacante (LiteSVM)
- [ ] Impacto en IDL/cliente anotado en la descripción del PRCuándo usar esto:
// Comentario de revisión BUENO - específico, enseña, sugiere corrección
// "El atacante puede pasar una bóveda externa como `user_vault` - añadir:
// constraint = user_vault.owner == user.key()
// Ver: ataques de sealevel - sustitución de cuenta"
// BAJA SEÑAL - evitar
// "Renombrar esta variable a `amt`"Plantilla de descripción de PR:
## Cambio
Añadir instrucción withdraw_v2
## Notas de seguridad
Nueva CPI a Jupiter; ID de programa en lista blanca en constantes
## Plan de pruebas
- anchor test
- Replay de exploit LiteSVM #42 de la auditoríaLo que esto demuestra:
| Tipo de PR | Primera respuesta | Objetivo de aprobación |
|---|---|---|
| Hotfix | 1 hora | Mismo día |
| Lógica del programa | 1 día hábil | 2 días |
| UI del cliente | 1 día hábil | 2 días |
#[account] - Modo de fallo #1. Solución: Revisión obligatoria de la sección de cuentas.[nit] opcional.| Alternativa | Usar cuándo | No usar cuándo |
|---|---|---|
| Revisión en grupo | Cambio complejo de CPI | Corrección de cliente rutinaria |
| Solo auditoría | Programa inmutable | Actualizaciones continuas |
| Solo lint automatizado | Errores tipográficos | Semántica de cuentas |
Dos para programs/ de producción; uno para solo documentación.
Clippy, comprobaciones de anchor, cargo audit en CI - los humanos se centran en la semántica.
Nunca ser el único aprobador en el propio PR para cambios de programa.
Escalar al líder técnico; ADR si es arquitectónico.
Enlazar documento interno o catálogo de Sealevel; mantener un tono colaborativo.
Mismo rigor que Rust - los clientes dependen de ello.
Video solo para RFC; PR asíncrono con SLA claro.
Tiempo hasta la primera revisión, tiempo de resolución de comentarios, tasa de defectos post-fusión.
Revisor senior emparejado; comentarios explicativos extra amables.
Revisión post-fusión dentro de las 24h; el comandante de incidentes puede fusionar con un senior.
Versiones de Stack: Esta página fue escrita para Agave 4.1.1, Solana CLI 3.0.10, Anchor 0.32.1, anchor-lang 0.32.1, Rust 1.91.1, @solana/kit 7.0.0, Surfpool 0.12.0, y LiteSVM 0.6.x.
Revisado por Chris St. John·Última actualización: 16 jul 2026