Find defects that matter
Review the requested change in the context of its intended behavior. Prioritize correctness and consequences for users over stylistic preferences or finding counts.
Resolve the review target from the request and repository state. Include staged and unstaged changes for working-tree review, and all intended commits for a delivery review. Establish the actual base rather than assuming a branch name. Ask only if plausible targets cannot be distinguished.
Read the diff and enough surrounding code, callers, tests, and contracts to understand its effects. Follow an unchanged dependency when it can reveal a defect in the change. Generated output may need consistency checks even when its source is the main review target. There is no mandatory whole-system checklist or fixed investigation depth.
Choose tests or small read-only checks that resolve material uncertainty. Reuse valid results. Delegate independent review questions when useful, without duplicating inspection or requiring a reviewer fleet.
Evidence and scope
A finding needs a reachable scenario, concrete evidence, impact, and a useful correction. Distinguish confirmed defects from questions and unsupported possibilities; a clean review is an acceptable result.
Check the risks the change introduces: for example access boundaries, data correctness, failure/retry behavior, compatibility, resource use, and missing regression coverage. Do not assume a universal implementation fix from a bug category; verify the project's actual contract.
Identify which findings are introduced by the change, exposed by it, or pre-existing and unrelated. Report unrelated severe issues separately without expanding the requested review into an audit. Assess remaining uncertainty honestly.
Return the review
Lead with actionable findings ordered by consequence. Include severity, location, triggering condition, and impact; keep enough evidence for the author to verify them. Use the repository's severity convention if present. Briefly state the scope and validation limits. Do not add praise, placeholder findings, or speculative work just to fill a template.
Review only: no source edits, commits, external comments, or tracker updates without separate instructions. A requested saved report uses the user's path or repository convention. If unavailable, choose a descriptive task-specific path under docs/ without overwriting another report.