emaraschio Overview
Perform a quick quality pass on current diffs to surface risky areas, ensure polish, and flag follow-up actions for deeper review.
Steps
- Scan recent changes
- List open branches or pending commits requiring review
- Skim side-by-side diffs focusing on new or modified files
- Note files or modules with large or complex edits
- Assess quality signals
- Watch for TODOs, debug code, or commented blocks needing cleanup
- Verify naming, formatting, and imports follow project standards
- Check that tests or documentation were updated when behavior changed
- Flag next actions
- Mark sections that warrant full review or pair programming
- Capture questions or uncertainties to raise with the author
- Document any quick fixes you can apply immediately
Review Checklist
- High-risk files identified
- Debugging artifacts removed or flagged
- Style and conventions validated
- Tests/docs updates confirmed or requested
- Follow-up items recorded for deeper review
Guardrails
- Raise nitpicks as optional and non-blocking; keep the pass fast.
- Flag security issues even in a light review, no matter how quick the pass.
- Do not commit, push, merge, or run production scripts without consent.