pr-review

What the source does

• FilesystemNone observedNo reads or writes in the source
• NetworkNone observedNo outbound calls in the source
• ShellNone observedNo commands run in the source
• DependenciesPinnedNothing floats
• SecretsNone foundNo keys in source

Five audit layers

Gates applied: no behavioural pass.

4. **API key handling** — If external APIs are used, are credentials read from environment variables?

Audit 0.4.1 · source sha ceddde63e832 · 31 Oct 2026Report an issue or request a re-scan

• Publisherminimax-ai GitHub organizationOwner of the repository this skill is read from
• Source folderminimax-ai/skills/.claude/skills/pr-reviewThe exact folder we audited
• LicenceMITAs the repository states it
• Commit pinned for this verdictceddde63e832A new audit runs when the source changes
• Repository statusNot archivedLast push 195 days ago
• Sandboxed runNot runSkills are read, not executed; layer L4 is skipped

Its 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…

---
name: pr-review
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.
license: MIT
metadata:
  version: "1.0"
  category: tooling
---

# 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:

```bash
python .claude/skills/pr-review/scripts/validate_skills.py
```

The script checks:
- `SKILL.md` exists in every skill directory
- YAML frontmatter is parseable
- Required fields present: `name`, `description`
- `name` matches 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](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:

1. **Skill scope** — Does it overlap with existing skills? Is the boundary clear?
2. **Description quality** — Does the `description` include clear trigger conditions?
3. **File size** — Are reference docs reasonably sized for context window consumption?
4. **API key handling** — If external APIs are used, are credentials read from environment variables?
5. **Script quality** — Do scripts have shebang, requirements.txt, and error handling?
6. **Language** — Are SKILL.md and code written in English?
7. **README sync** — Are `README.md` and `README_zh.md` updated for new skills?

See [references/quality-guidelines.md](references/quality-guidelines.md) for soft guidelines details.

## Review Checklist Summary

### Must Pass (Blockers)
- [ ] `validate_skills.py` exits 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`)

Excerpt, first 2,461 characters of SKILL.md, verbatim from the audited commit. It is text from someone else’s repository, quoted so you can judge it before your agent reads it.

git clone https://github.com/minimax-ai/skills
git -C skills checkout ceddde63e832

Works in

1. 2026-10-31
   !CAUTION B 89 / 100

   first audit · source ceddde63e832

Questions about pr-review

• github.com/minimax-ai/skills/tree/main/.claude/skills/pr-review — the audited source at commit ceddde63e832 and every finding aboveObserved · 31 Oct 2026

Spotted something wrong? Report a correction · Methodology