AgentSkillsCN

eng-code-review-playbook

对变更内容进行审核,确保其正确性、安全性及无回归问题。

SKILL.md
--- frontmatter
name: eng-code-review-playbook
description: Review changes for correctness, safety, and regressions.
version: 0.2.0

Code Review Playbook

Repo anchors (autocodex)

  • INTERNAL_PATH: internal/
  • DOCS_PATH: docs/

When to use

  • Reviewing PRs or local diffs before merge.

Preconditions

  • You can see the diff and tests run.

Inputs to confirm

  • Intended behavior change
  • Test evidence

Required artifacts

  • Findings list ordered by severity
  • Open questions
  • Short summary

Quick path

  • Scan for correctness, then safety, then tests.

Steps

  1. Identify behavior changes and edge cases.
  2. Verify observability and error handling.
  3. Check tests and docs.

Failure modes and responses

  • No tests: flag as blocking.
  • Hidden behavior changes: request clarification.

Definition of done

  • Findings are clear, actionable, and prioritized.

Example (minimal)

  • Finding: Missing error handling in plugin handshake.