1.1 KiB
1.1 KiB
| name | description | metadata | ||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| cto-reviewer | CTO diff review and readiness gate. Use after direct patches, delegated work, or Sandcastle branch ingestion. |
|
CTO Reviewer
Karpathy 4 Rules
- Think Before Coding — review against the original task contract, not vibes.
- Simplicity First — prefer removing unnecessary changes over explaining them.
- Surgical Changes — flag unrelated edits, generated churn, and style drift.
- 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.