Review
/request-reviewRequest review while the decision is still reversible.
Review while choices remain reversible
A checkpoint is a planned pause
A review often arrives at one of two bad times. At the beginning, there may be little evidence to inspect. At the end, the approach may already be spread across code, data and documentation. The useful moment sits between them: enough has been built to expose the important choices, but changing direction is still affordable.
I call that moment a review checkpoint: a planned pause before completion where another person or agent examines one consequential decision against agreed success criteria. It is not a request for general encouragement, and it is not necessarily a release approval. Its purpose is to find errors while the work still has options.
People can become attached to work they have polished, but that is not a universal law and it is not the main control. Early review works because information has arrived while decisions remain reversible. The discipline of finishing a branch well begins by placing checkpoints before the expensive commitments, not by guessing how defensive its author will feel.
Ask for review while the decision can still move
Review value falls when commitment makes the important alternatives expensive to recover.
- Frame the consequential choice Name the decision, risk and exact question.
- Show the smallest evidence Provide context, criteria, alternatives and current proof.
- Invite challenge before commitment The reviewer can still change the approach.
- Record the decision Capture what changed and why.
- Continue with owned risk The request does not transfer responsibility to the reviewer.
A review checkpoint is useful when disagreement can still improve the work.
The request is the review interface
Give the reviewer a decision to test
“Looks good?” is difficult to answer well because it hides the decision. A practical request supplies five things: the context, the success criteria, the risk, the evidence already collected and one exact question. Success criteria are observable conditions the work must meet. Evidence might be a test result, browser recording, database query or before-and-after example.
Here is a complete hypothetical request made at a checkpoint for an automatic invoice-reminder feature:
Context: Finance currently sends reminders manually. The proposed job selects invoices seven days overdue and sends one reminder each morning.
Success criteria: paid, disputed and opted-out invoices receive nothing; an eligible invoice receives no duplicate reminder; every send is recorded.
Primary risk: a payment may arrive after the job selects an invoice but before it sends the message.
Evidence: the four eligibility tests pass; rerunning the job does not duplicate a reminder; a staging run records the expected audit event.
Exact question: “Could an invoice paid during that selection-to-send gap still receive a reminder? Please trace that path and name the test that would prove your answer.”
The reviewer traces the path and answers: “Yes. The duplicate guard prevents a second reminder, but eligibility is checked only when the invoice is selected. Recheck payment and dispute status immediately before sending, then add a test that inserts a payment between selection and send.” The author makes that change, the new test fails on the old version and passes on the revision, and the checkpoint closes with evidence. The review changed the design before the job was enabled for customers.
This is harness engineering in a modest form. The request shapes what the reviewer can inspect, and the evidence gives disagreement somewhere concrete to land. The exact question focuses attention without forbidding the reviewer from raising another serious risk.
The same structure helps a human reviewer or an AI verifier. An agent can inspect code and run evidence-producing tools, but a fluent answer is not itself evidence. For consequential work, the reviewer should report what it examined, what passed or failed and what remains uncertain.
Approval seeking is not review seeking
A useful request makes the reviewer able to disagree with one consequential claim.
Rubber-stamp request
- Looks good? No defined scope or decision.
- Large polished artifact The cost of substantive change is hidden.
- Social pressure to approve Confidence replaces evidence.
make challenge possible
Decision request
- One exact question For example: does this permission boundary match policy?
- Success criteria and evidence The reviewer can test the claim.
- Known risk and alternatives Disagreement has somewhere to go.
The author remains responsible for acting on the review and verifying the result.
Policy outranks preference
Early review supplements the mandatory gate
An early checkpoint does not replace a required security, privacy, legal, accessibility or release review. Team policy should decide which gates are mandatory based on impact, not whether an author feels confident. Payments, permissions, destructive data changes and safety-sensitive actions deserve named reviewers and explicit approval boundaries.
Policy should also prevent review theatre. Name the reviewer, the response time and the evidence required. Distinguish advice from approval. Allow small, reversible changes to use a lighter path so mandatory reviewers are not buried in noise. When an AI performs the first pass, keep human authority where the organisation has assigned accountability.
Requesting well only completes half the loop. The other half is receiving the review: test the comment, ask for clarification and update the work or explain the evidence for keeping it. As a professional, I want review placed where it can still change the outcome. As a student, I want the response to reveal which assumption failed. The practical rule is to ask when the evidence is real, the risk is named and the decision is still reversible.