review-pr
Review a Fleet pull request for correctness, Go idioms, SQL safety, test coverage, and conventions. Use when asked to "review PR" or "review pull request".
Datos de origen
- Repositorio
- fleetdm/fleet
- Última actividad en el origen
- 6 de abril de 2026 a las 23:48
- Idioma detectado de SKILL.md
- inglés
- Estrellas
- 6894
- Forks
- 1024
Opciones de instalación
De forma predeterminada está seleccionado el prompt que primero revisa el origen. Puedes cambiar a un comando directo o descargar una copia local.
Revisa los archivos de origen
Lee SKILL.md y los archivos complementarios que muestra SkillsMP antes de decidir si quieres instalarlo.
Mostrando SKILL.md
SKILL.md
Instrucciones de origen · Vista previa de solo lectura- name
- review-pr
- description
- Review a Fleet pull request for correctness, Go idioms, SQL safety, test coverage, and conventions. Use when asked to "review PR" or "review pull request".
- context
- fork
- allowed-tools
- Bash(gh *), Read, Grep, Glob
- model
- opus
- effort
- high
Review the pull request: $ARGUMENTS
Use `gh pr view` and `gh pr diff` to get the full context.
Review the changes focusing on:
1. **Correctness** — logic errors, edge cases, nil pointer risks
2. **Go idioms** — error handling with ctxerr, proper context usage, slog logging
3. **SQL safety** — injection risks, missing indexes for new queries, migration correctness
4. **Test coverage** — are new code paths tested? Are integration tests needed?
5. **Fleet conventions** — matches patterns in surrounding code
For each issue found, cite the specific file and line. Categorize findings as:
- **Must fix** — bugs, security issues, data loss risks
- **Should fix** — convention violations, missing error handling
- **Nit** — style preferences, minor improvements
Be concise. Don't comment on things that are fine.
Ver en GitHub