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".
Informations de source
- Dépôt
- fleetdm/fleet
- Dernière activité de la source
- 6 avril 2026 à 23:48
- Langue détectée de SKILL.md
- anglais
- Étoiles
- 6 865
- Forks
- 1 019
Options d'installation
Le prompt qui vérifie d'abord la source est sélectionné par défaut. Vous pouvez passer à une commande directe ou télécharger une copie locale.
Vérifiez les fichiers source
Lisez SKILL.md et les fichiers associés affichés par SkillsMP avant de décider de l'installer.
Affichage de SKILL.md
SKILL.md
Instructions source · Aperçu en lecture seule- 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.
Voir sur GitHub