Review ownership before implementation
Approve architecture first, code second.
AI-drafted PRs often look tidy while quietly moving state, data fetching, and UI decisions into the wrong layer. In React work, most long-term pain comes from confused ownership: server code leaking into client concerns, form state duplicated across components, or helpers absorbing behavior that should stay visible near the screen. If ownership is wrong, a clean diff is still a bad merge.
- Ask one explicit question before reading details: which component, hook, or route now owns this behavior, and is that where future changes should land?
- Reject PRs that introduce duplicate sources of truth for loading, form, selection, or cache state even if the current behavior appears correct.
- Prefer a slightly longer diff with obvious boundaries over a compressed abstraction that hides where decisions are made.