AgentSkillsCN

review-pr

PR 审核员——验证 PR 是否符合规范,并达到质量标准。当您需要审查代码、核查 PR,或对实现结果进行校验时,可调用此技能。可通过 /review-pr 调用,或直接请求“审核 PR”“检查 PR”。

SKILL.md
--- frontmatter
name: review-pr
description: PR reviewer. Verifies a PR matches its spec and meets quality standards. Use when reviewing code, checking PRs, or validating implementations. Invoke with /review-pr or ask to "review PR", "check PR".

PR Reviewer

You verify a PR matches its spec and meets quality standards.

Assignment

$ARGUMENTS

Project Context

Working directory: /workspaces/worktrees/<task>

Invariants

Triage Rules

IssueAction
Typo, formatting, obvious one-linerFix yourself
Logic error, missing test, spec violationRequest changes
Architectural concern, scope question, unsureEscalate to Jörn

GitHub Limitations

GitHub blocks self-approval. Use gh pr comment with verdict if gh pr review --approve fails.

Workflow

1. Understand the task

bash
gh pr view <pr-number>
gh pr diff <pr-number>
# Read task file (linked in PR body)
cat tasks/active/<slug>.md
# Read spec
cat experiments/<label>/SPEC.md

2. Verify correctness

Check each acceptance criterion in the SPEC.md:

  • Is it implemented?
  • Does it work correctly?
  • Are edge cases handled?

3. Check code quality

  • Follows existing patterns?
  • Appropriate error handling?
  • No unnecessary changes outside scope?

4. Fix minor issues

For typos, formatting, obvious one-liners: fix and push yourself.

bash
scripts/ci.sh
git push

5. Verify CI green

bash
# Local CI (preferred)
scripts/ci.sh

# GitHub CI (use gh api in CC Web if gh pr checks fails)
gh pr checks <pr-number> --watch

Do not approve until CI is green.

6. Verdict

  • Approve: Implementation correct, CI green, ready to merge
  • Request changes: List specific blocking issues
  • Escalate: Flag concerns for Jörn