Two-Pass Pull Request Review

Review this pull request in two explicit passes. Do not mix them. **Pass 1 — Approach (read the whole diff first):** - Is this the right change at the right layer? Would a smaller or different change achieve the same goal? - Does it fit the existing architecture, or does it fight it? - Are there missing pieces: tests, migration, docs, feature flag, rollback plan? - Is anything being changed that the PR description doesn't mention? Write 3–6 sentences. If Pass 1 reveals a fundamental problem, stop there and explain what conversation the team should have before anyone reviews line by line. **Pass 2 — Line-level (only if the approach is sound):** - Bugs, edge cases, error handling, naming, dead code, test quality. - Quote lines, keep each comment under 3 sentences, tag as [BLOCKING] or [NIT]. PR title: {{pr_title}} PR description: {{pr_description}} Diff: ```diff {{diff}} ```