orders.create(cart)First requestOrder 1042
RetryOrder 1043
Two orders for one cart.
Review the change.
Keep the reason.
Bring the diff, the discussion, and the decision into the same view.
Open the reviewMake checkout safe to retry#128
Sample pull requestcheckout/idempotencymain3 files changed
src/payments/checkout.tsSuggested change
42
export async function checkout(cart: Cart) {43
const order = await orders.create(cart);44
await payments.charge(order.total);45
return order;46
}Proposed replacement
43
const key = `cart:${cart.id}:${cart.version}`;44
const order = await orders.create(cart, { key });45
await payments.charge(order.total, { key });46
return order;42
export async function checkout(cart: Cart) {43
const key = `cart:${cart.id}:${cart.version}`;44
const order = await orders.create(cart, { key });45
await payments.charge(order.total, { key });46
return order;47
}src/auth/session.tsSuggested change
18
export function readSession(token: Token) {19
return cache.get(token.subject);20
}Proposed replacement
19
if (token.expiresAt <= Date.now()) return null;20
return cache.get(token.subject);18
export function readSession(token: Token) {19
if (token.expiresAt <= Date.now()) return null;20
return cache.get(token.subject);21
}src/catalog/search.tsSuggested change
27
export function search(query: string) {28
return products.find(query);29
}Proposed replacement
28
const term = query.trim();29
if (!term) return [];30
return products.find(term);27
export function search(query: string) {28
const term = query.trim();29
if (!term) return [];30
return products.find(term);31
}Inspect the suggested change, then apply it to this local example.
Same click. Same order.
Replay the edge case behind the review: one checkout request arrives twice.
orders.create(cart, { key })First requestOrder 1042
RetryOrder 1042
The retry returns the original order.
Start at the line that matters.
Nora attached one question to the order creation step. The reason lives beside the diff, where the next person can understand it.
orders.create(cart)A retry can create a second order.
A suggestion becomes a visible change.
The same key follows the order and the charge. Inspect the exact lines before accepting the suggestion.
const key = `cart:${cart.id}:${cart.version}`;Both operations share one idempotency key.
Test the edge case behind the comment.
The sample check repeats an identical request, then compares the returned order identifiers. It explains the condition being checked.
expect(first.id).toBe(retry.id)Sample check: one order from two requests.
Carry the decision into the merge.
The final record keeps the changed lines, the reason, and the reviewer together. A future reader does not need to reconstruct the conversation.
decision: "Keep retries idempotent"The review is ready for a human decision.
An illustrative review sequence. Sample checks do not execute project code.
Give the review
the right guardrails.
A checkout fix and a release change need different questions. Start with a working agreement, then inspect what it asks of the reviewer.
Sample team policies- Reviewers
- One teammate who knows the customer flow.
- Evidence
- The changed state, its empty state, and keyboard behavior.
- Before merge
- Every unresolved question has an owner or a recorded answer.
This agreement is a local review preview.
The reasoning stays with the change.
Three decisions in this sample review.
No decisions in this view.
Change the filter to continue the review.
src/payments/checkout.ts:43
Keep a retry from becoming a second order.
A slow connection can replay this request. Use the same idempotency key for the order and the charge, so a retry returns the original result.
const key = `cart:${cart.id}:${cart.version}`;src/auth/session.ts:19
An expired session should not outlive its token.
The cache can remain warm after a token expires. Check the token clock before returning cached user data.
if (token.expiresAt <= Date.now()) return null;src/catalog/search.ts:28
Search for a phrase, not the spaces around it.
Whitespace-only requests currently scan the entire catalogue. Normalize the phrase and return an empty result before querying.
const term = query.trim();Select a decision to inspect its source line and rationale.
Review the way your team works.
What stays attached to a decision?
The changed file, the exact source line, the reviewer’s reason, and its resolution. Applying the sample checkout suggestion updates the review sequence and its decision record below.
Can I undo a suggestion?
Yes. Revert restores the original lines for that file. Changes stay associated with their file when you switch between checkout, session, and search examples.
How do review agreements change the workflow?
Choose a product, infrastructure, or release context. Each agreement asks for different evidence and a different reviewer. The local preview records your choice alongside the final review stage.
Does the example run my code or connect a repository?
No. This is an editable frontend template. The diff, retry replay, and checks use illustrative local state. Connect your own repository and check runner before offering those services.
Leave a clearer
review behind.
Try the edge case. Inspect the suggestion. Make the decision.