review-stylelisted

# Review — Style, Naming & Readability Focused style reviewer. Covers naming conventions, dead code, readability, DRY violations, import organization, comment quality, file length, cyclomatic complexity, and console noise. Never touches logic correctness, architecture, security, or performance. --- ## Workflow ``` review-style Progress: - [ ] Step 1: Read Job Context (if CONTEXT_PATH provided) - [ ] Step 2: Determine git scope (merge-base) — see skills/shared/git-merge-base.md - [ ] Step 3: Collect diff and changed file list - [ ] Step 4: Naming conventions check - [ ] Step 5: Dead code and imports check - [ ] Step 6: Readability and complexity check - [ ] Step 7: DRY violations check - [ ] Step 8: Comment quality check - [ ] Step 9: File length and structure check - [ ] Step 10: Console usage check - [ ] Step 11: Emit findings in unified format, sorted by severity ``` --- ## Input Contract | Field | Type | Required | Description | |-------|------|----------|-------------| | `branch` | string | no | Branch to review. Defaults to current branch. | | `commit_range` | string | no | Explicit range (e.g., `abc123..HEAD`). Overrides merge-base detection. | | `context_doc` | string | no | Path to job context document (e.g., `<JOBS_ROOT>/<job>/ai/context.md`). | --- ## Scope Detection See shared script: `skills/shared/git-merge-base.md` Run that script to determine `BASE_SHA` before collecting the diff. ```bash # Default mode — all changes from merge-base to working tree