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".
Source facts
- Repository
- fleetdm/fleet
- Last source activity
- April 6, 2026 at 23:48
- Detected SKILL.md language
- English
- Stars
- 6,865
- Forks
- 1,019
Install options
The review-first prompt is selected by default. You can switch to a direct command or download a local copy.
Review the source files
Read SKILL.md and any companion files shown by SkillsMP before deciding whether to install.
Showing SKILL.md
SKILL.md
Source instructions ยท Read-only preview- 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.
View on GitHub