go-logic-review

# Go Logic Review ## Purpose Audit Go code for business logic correctness. Core question: **"Does the code do what it's supposed to do?"** Key distinction from other 6 vertical skills: they use **pattern matching** (see SQL → check injection, see goroutine → check race). This skill uses **semantic understanding** — understand the code's intent, then compare with its implementation. This skill relies primarily on AI's general reasoning ability, not heavy reference files. The checklist provides the review framework; AI provides the reasoning. This skill does NOT cover: security patterns, concurrency patterns, performance patterns, code style, test quality, or error handling patterns — those belong to sibling skills. ## When To Use - Any code change that modifies behavior - Code contains conditional branches (if/else, switch) - Code contains data transformation or processing - Code contains state management or state transitions - Default dispatch: always run (any code change can introduce logic errors) ## When NOT To Use - Pure refactoring with no behavior change - Config-only changes - Security vulnerability patterns → `go-security-review` - Concurrency patterns → `go-concurrency-review` - Code style → `go-quality-review` ## Mandatory Gates ### 1) Context Understanding Gate (unique to this skill) Before evaluating correctness, understand the INTENT: - Read function name, comments, docstring - Read caller context — who calls this function, what do they expect? - Read re...