PR Review

Core Workflow

1. Understand the change intent, affected surface, user impact, and stated test plan before judging the diff.
2. Review for correctness, edge cases, regressions, security, data handling, performance, compatibility, and maintainability.
3. Prioritize findings that could cause wrong behavior, user harm, production risk, or future maintenance cost.
4. Tie each finding to concrete evidence from code, tests, logs, or behavior.
5. Separate blocking findings from non-blocking suggestions and style nits.
6. Call out missing tests or validation when risk remains.

Safety Rules