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".
معلومات المصدر
- المستودع
- fleetdm/fleet
- آخر نشاط في المصدر
- ٦ أبريل ٢٠٢٦ في ٢٣:٤٨
- لغة SKILL.md المكتشفة
- الإنجليزية
- النجوم
- ٦٬٨٩٤
- التفرعات
- ١٬٠٢٤
خيارات التثبيت
يُحدَّد Prompt الذي يراجع المصدر أولًا بشكل افتراضي. يمكنك التبديل إلى أمر مباشر أو تنزيل نسخة محلية.
مراجعة ملفات المصدر
اقرأ SKILL.md وأي ملفات مرافقة يعرضها SkillsMP قبل أن تقرر التثبيت.
عرض SKILL.md
SKILL.md
تعليمات المصدر · معاينة للقراءة فقط- 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.
عرض على GitHub