What a useful review leaves behind
A useful review shows what was checked, what decision was made, why, and who owns the next step.Before you start
The Issue should have:- a named reviewer
- a result that can actually be opened
- clear acceptance criteria
- links to relevant Runs, checks, and known risks
Review the result
- Open the Issue and read the original request and acceptance criteria.
- Open the produced file, preview, report, screenshot, or link. Do not approve work from a summary alone.
- Check the important facts and tests. Open the Agent Run when you need to see which runtime ran, whether it finished, what it changed, or why it failed.
- Compare the result with each acceptance criterion.
- Record one decision with a specific comment:
- Accept when the result meets the request.
- Request changes when the assignee should revise it.
- Blocked when a required input, permission, or result is missing.
- Create a follow-up Issue for separate work instead of hiding it in a vague acceptance comment.
