go-error-review

# Go Error Review ## Purpose Audit Go code for error handling correctness, nil safety, and failure-path integrity. Core question for every function call: "What happens when it fails?" This skill merges error handling + API request/response correctness + database operation correctness because they share one review lens: "does this code handle failure correctly?" This skill does NOT cover: security vulnerabilities, concurrency safety, performance, code style, test quality, or business logic — those belong to sibling vertical skills. ## When To Use - Code contains error return values - Code uses `panic` / `recover` - Code involves `sql.Rows`, transactions, connection pools - Code involves HTTP request/response body handling - Code operates on `[]*T` pointer slices ## When NOT To Use - Security vulnerabilities → `go-security-review` - Concurrency safety → `go-concurrency-review` - Performance optimization → `go-performance-review` - Code style → `go-quality-review` - Business logic → `go-logic-review` ## Mandatory Gates ### 1) Execution Integrity Gate Never claim tests ran unless they actually did. If not run: state reason + exact command. ### 2) Go Version Gate Read `go.mod`. Key features: - `errors.Is` / `errors.As` (Go 1.13+) - `errors.Join` (Go 1.20+) - `fmt.Errorf` with multiple `%w` (Go 1.20+) ### 3) Anti-Example Suppression Gate MUST quote specific code evidence. Category match alone insufficient. Embedded anti-examples: - **"Missing error handling on json.Marsh...