Eng Code Review Playbook

Review changes for correctness, safety, and regressions. Use when this capability is needed.

tomevault-io Updated

File contents

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.

Converted and distributed by TomeVault — claim your Tome and manage your conversions.

tomevault-io/skills-registry/tree/main/oodaris--autocodex--eng-code-review-playbook commit 37e584c75d

Frequently asked questions

npx skillmds@latest add tomevault-io/eng-code-review-playbook