agentsclimarketplace

Pr review

Skill bg-szy/TOP-SKILLS/skills/marketplace/pr-review

全球最大的 Claude Code 技能聚合库 · 收录 3900+ 来自 12+ 来源的技能，提供在线搜索与趋势分析看板 / The world's largest Claude Code skill aggregation hub — 3900+ skills from 12+ sources with online search and trend dashboard

Install
npx -y skills add bg-szy/TOP-SKILLS --skill pr-review

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

2 things to look at

  • no licenseNo license file was found in the repository. Code published without one is not open source by default, so using it at work is a question for whoever answers licensing questions where you are.
  • 4 stars4 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

What its author says it does

Copied from the file, not written here

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.

The file declares its own license as MIT. That is the author’s claim about this one file, and it is not the same thing as the license GitHub reports for the repository, which is listed with the other numbers below.

SKILL.md

2.4 KB, 446 tokens by cl100k_base, as published. Nobody here has run it

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.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 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 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)

Keep looking

Skills are one crate of 328,083. Ordering is by how many stacks a row turns up in, so the top of any crate is what has actually been picked rather than what has the most stars.