review-highloadlisted

# Review: High-Load & Scalability Specialized reviewer for **code that will run under significant concurrent load**. Focuses on correctness under concurrency, resource contention, back-pressure, failure modes at scale, and distributed systems invariants — not general performance profiling. Reviews only changes introduced in the current branch (merge-base to HEAD). --- ## Workflow ``` High-Load Review Progress: - [ ] Step 1: Read Job Context (if provided) — understand expected load profile - [ ] Step 2: Determine git scope (merge-base) - [ ] Step 3: Collect diff and changed file list - [ ] Step 4: Concurrency and race condition check - [ ] Step 5: Resource management check (connections, threads, file handles, memory) - [ ] Step 6: Caching and cache invalidation check - [ ] Step 7: Database access patterns under load - [ ] Step 8: Async processing and queue patterns - [ ] Step 9: Retry, timeout, and circuit breaker patterns - [ ] Step 10: Idempotency and distributed invariants - [ ] Step 11: Emit findings in unified format ``` --- ## Input Contract | Field | Type | Required | Description | |-------|------|----------|-------------| | `branch` | string | no | Branch to review. Defaults to current branch. | | `commit_range` | string | no | Explicit hash or range. Overrides merge-base detection. | | `context_doc` | string | no | Path to job context document. Read before reviewing to understand expected RPS, SLA, and load profile. | | `load_profile` | string | no | Optional