Skip to content

Checklist: Code Review ​

Correctness ​

  • [ ] Change solves the stated problem
  • [ ] Acceptance criteria are met
  • [ ] Edge cases and error paths were considered

Safety ​

  • [ ] Authentication and authorization are correct
  • [ ] No secrets or sensitive data were introduced
  • [ ] Data integrity and migration impact were reviewed
  • [ ] Rollback or mitigation is clear for risky changes
  • [ ] Security review was performed for sensitive changes

Engineering Quality ​

  • [ ] Code follows existing standards or intentionally improves them
  • [ ] Tests are adequate for the risk of the change
  • [ ] CI passed
  • [ ] Logging, metrics, or alerts were updated if behavior changed

Operational Impact ​

  • [ ] Deployment steps are documented if needed
  • [ ] Runbooks or docs were updated if needed
  • [ ] Backward compatibility concerns were addressed
  • [ ] Dependencies and interfaces were reviewed for breakage risk
  • [ ] Change classification matches the actual release risk

AI-Assisted Changes ​

  • [ ] Agent-generated code was reviewed with the same rigor as human-written code
  • [ ] Hidden assumptions or undocumented conventions were removed