What problem does it solve? Go code reviews often miss subtle correctness bugs like unwrapped errors, goroutine leaks, and unclear channel ownership while wasting time on style nits that linters already catch. This Skill runs a systematic Go-specific review that flags only correctness, performance, and maintainability issues with concrete file:line locations and suggested fixes. ## Core Features & Use Cases - Static Analysis Integration: Runs golangci-lint run --new-from-rev=HEAD~1 ./... against changed files to surface machine-detectable issues before the manual checklist pass. - Go-Specific Checklist: Walks a structured checklist covering error wrapping, goroutine termination, channel ownership, mutex discipline, slice pre-allocation, and interface design from references/go-review-checklist.md. - Structured Findings Report: Reports each issue as file:line with a category (correctness/performance/maintainability), severity (must-fix/should-fix/consider), and a suggested code fix. - Use Case: Before requesting human review on a Go PR, invoke the skill to catch error-handling and concurrency bugs first, or ask for a full-package review when auditing an entire package for concurrency issues. ## Quick Start Ask the assistant to review the Go code changed since the last commit for error handling, concurrency, and performance issues.
