qa-reviewlisted

# Code Review ## Objective Identify quality, security, and maintainability issues BEFORE merge. ## Instructions ### 1. Overview ```bash # View the changes git diff main...HEAD --stat git log main...HEAD --oneline ``` ### 2. Review checklist #### Code quality - [ ] Readability (clear names, short functions) - [ ] DRY (no duplication) - [ ] SOLID (single responsibility) - [ ] Reasonable complexity #### Typing (TypeScript) - [ ] No `any` - [ ] Explicit types on public APIs - [ ] Well-defined interfaces #### Tests - [ ] Tests present and relevant - [ ] Edge cases covered - [ ] Mocks limited to I/O #### Security - [ ] Inputs validated - [ ] No hardcoded secrets - [ ] No injection possible #### Performance - [ ] No N+1 queries - [ ] No possible infinite loops - [ ] Memory managed correctly ### 3. Comment format ``` [TYPE] file:line - comment Types: - [CRITICAL] - Blocking, must be fixed - [IMPORTANT] - Should be fixed - [SUGGESTION] - Optional improvement - [QUESTION] - Clarification needed - [NITPICK] - Minor detail ``` ## Expected output ```markdown ## Review: [PR Title] ### Summary - **Files modified**: X - **Lines added**: +Y - **Lines removed**: -Z - **Verdict**: Approve / Request Changes / Comment ### Positive points - [Point 1] - [Point 2] ### Issues identified #### Critical - [CRITICAL] `file.ts:42` - Description #### Important - [IMPORTANT] `file.ts:87` - Description ### Suggestions - [SUGGESTION] `file.ts:123` - Description ### Final checklist - [ ]