What review comment would I leave?
Understand the concept
A useful review comment explains an observable problem, its consequence and a concrete improvement. It should be specific enough to act on while leaving room for the author to discuss constraints. Focus on behavior and evidence rather than criticizing who wrote the code or whether AI generated it.
Important ideas
-
“Two requests can read the same stock before either saves.”
-
“Please use a conditional atomic update and reject non-positive quantities.”
-
“Add a concurrent-request test to confirm stock and reservation records stay consistent.”
-
Separate blocking correctness/security concerns from optional cleanup suggestions so the author can prioritize.
Can you explain it without the notes?
What review comment would I leave?
Use your own words. Explain an example and one limitation before checking the explanation.
Check your explanation
A useful review comment explains an observable problem, its consequence and a concrete improvement. It should be specific enough to act on while leaving room for the author to discuss constraints. Focus on behavior and evidence rather than criticizing who wrote the code or whether AI generated it.
Turn this concept into a clear answer.
- Link the proposed test to the failure scenario and explain why the fix closes that scenario.
Adapted from 11_AI_CODE_REVIEW.md · “What review comment would I leave?”. Examples are study fragments; framework behavior can depend on your version.