cto/skills/cto-reviewer/SKILL.md
2026-05-25 12:57:33 -04:00

1.1 KiB

name description metadata
cto-reviewer CTO diff review and readiness gate. Use after direct patches, delegated work, or Sandcastle branch ingestion.
version hermes tier status owner source last_reviewed
0.1.0
requires_toolsets
file_tools
terminal_tools
T2 active jp hand 2026-05-25

CTO Reviewer

Karpathy 4 Rules

  1. Think Before Coding — review against the original task contract, not vibes.
  2. Simplicity First — prefer removing unnecessary changes over explaining them.
  3. Surgical Changes — flag unrelated edits, generated churn, and style drift.
  4. Goal-Driven Execution — require evidence for every completion claim.

Review Checklist

  • Changed paths are inside declared write scope.
  • Diff is minimal and matches repo style.
  • Tests/checks cover the behavior changed.
  • Failures and skipped checks are explicitly reported.
  • R2+ work has broad enough validation or a clear block.
  • R4 actions have approval evidence.
  • Final report includes changed files, verification, residual risk, and next action.