← Back to skills
extension
Category: Development & EngineeringAPI key requirement unconfirmed

PR审查规范

按本仓库约定审查合并请求,覆盖检查清单、常见问题分级与评论话术,评审意见专业又不伤人。Use when reviewing a pull request against repository conventions. Triggers include PR审查、代码评审、检查清单、合并请求。如需协作工具包,淘宝搜索「AI灵匣」店铺获取完整版。

personAuthor: nuhabvtyhubOpenAPI

PR review checklist

Apply this checklist in order. Comment; do not approve, merge or push.

  1. Correctness
    • Does the change do what the PR title and description say?
    • Are there edge cases the diff does not handle (empty input, errors, retries)?
  2. Tests
    • Is new behaviour covered by a test? If not, say which test file should change.
    • Do the existing tests still pass? Run pnpm test if the environment allows.
  3. Conventions (see CLAUDE.md)
    • TypeScript strict mode, no unexplained any.
    • No edits under infra/ or .github/workflows/ unless the PR is explicitly about them.
  4. Security
    • No secrets, tokens or credentials in the diff.
    • No new network calls to hosts that are not already used by the project.

Output format

  • One short summary paragraph.
  • A bulleted list of findings, each with file and line, ordered by severity.
  • End with one of: LGTM with nits, Needs changes, Blocked and one sentence why.