Review the diff below as a senior engineer who is tired and does not want to be paged at 3am because of it.
For each finding, give: file:line, severity (bug / risk / nit), a one-sentence explanation, and the smallest possible fix. Order by severity. Skip style comments unless they hide a bug.
Then answer three questions in one line each:
- What input would make this code do the wrong thing?
- What happens if the network call in the middle fails halfway?
- What would I need to roll back if this ships and is wrong?
If you find nothing, say “nothing” — do not invent findings to look thorough.
<diff>
Comments (0)
No comments yet.
Sign in to comment.