pr-review
GitHub用于审查 MiniMax Skills 仓库的 PR,通过自动化脚本验证结构合规性,并人工检查技能范围、描述质量及 API 安全等规范。
Trigger Scenarios
Install
npx skills add MiniMax-AI/skills --skill pr-review -g -y
SKILL.md
Frontmatter
{
"name": "pr-review",
"license": "MIT",
"metadata": {
"version": "1.0",
"category": "tooling"
},
"description": "Review pull requests for the MiniMax Skills repository. Use when reviewing PRs, validating new skill submissions, or checking existing skills for compliance. Run the validation script first for hard checks, then apply quality guidelines for content review. Triggers: PR review, pull request, validate skill, check skill.\n"
}
PR Review Skill
Review pull requests against repository standards. Two-phase process: automated validation, then manual content review.
Phase 1: Automated Validation (Hard Rules)
Run the validation script to check structural requirements:
python .claude/skills/pr-review/scripts/validate_skills.py
The script checks:
SKILL.mdexists in every skill directory- YAML frontmatter is parseable
- Required fields present:
name,description namematches directory name- No hardcoded secrets detected
All ERROR-level checks must pass. WARNING-level items (missing license, metadata) should be flagged but are not blockers.
See references/structure-rules.md for the complete hard rules specification.
Phase 2: Content Review (Soft Guidelines)
After automated checks pass, review the PR against quality guidelines:
- Skill scope — Does it overlap with existing skills? Is the boundary clear?
- Description quality — Does the
descriptioninclude clear trigger conditions? - File size — Are reference docs reasonably sized for context window consumption?
- API key handling — If external APIs are used, are credentials read from environment variables?
- Script quality — Do scripts have shebang, requirements.txt, and error handling?
- Language — Are SKILL.md and code written in English?
- README sync — Are
README.mdandREADME_zh.mdupdated for new skills?
See references/quality-guidelines.md for soft guidelines details.
Review Checklist Summary
Must Pass (Blockers)
-
validate_skills.pyexits with code 0 - PR title follows conventional commit format
- One PR, one purpose
Should Pass (Flagged in Review)
- No functional overlap with existing skills
- Description includes trigger conditions
- Files are reasonably sized
- API keys via environment variables
- README tables updated for new skills (Source column set to
Community)
Version History
- 60aaae5 Current 2026-07-25 09:03


