review-performancelisted

# Review: Performance (Code-Level Bottlenecks) ## Purpose Finds real, code-level performance problems introduced in the changed code of the current branch. Covers N+1 database queries, unnecessary React re-renders, missing memoization in hot paths, memory leaks, large bundle imports, synchronous blocking in async contexts, missing pagination, and redundant API calls. This skill focuses on **measurable or high-likelihood bottlenecks** only. Premature optimization, hypothetical slowness, and style preferences are out of scope. --- ## Input Contract | Field | Required | Description | |-------|----------|-------------| | Branch / diff range | No | Defaults to merge-base..HEAD + uncommitted changes | | Explicit commit hash/range | No | Review only that range when provided | | `JOB_NAME` | No | Job name when dispatched by orchestrator | | `CONTEXT_PATH` | No | Path to context doc when dispatched by orchestrator | --- ## Scope Boundaries | Concern | This skill | Use instead | |---------|-----------|-------------| | N+1 database queries, missing eager loading | YES | — | | Unnecessary React re-renders, missing memo/useCallback/useMemo | YES | — | | Expensive computations in hot paths (render loops, tight loops) | YES | — | | Missing database indexes (detectable from query patterns) | YES | — | | Memory leaks: unbounded collections, missing cleanup, forgotten subscriptions | YES | — | | Large bundle imports vs. named imports | YES | — | | Synchronous blocking in async context