​

/go-review Command

​

What This Command Does

1. Identify Go Changes: Find modified .go files via git diff
2. Run Static Analysis: Execute go vet, staticcheck, golangci-lint
3. Security Scan: Check for SQL injection, command injection, race conditions
4. Concurrency Review: Analyze goroutine safety, channel usage, mutex patterns
5. Idiomatic Go Check: Verify code follows Go conventions and best practices
6. Generate Report: Categorize issues by severity

​

When to Use

• After writing or modifying Go code
• Before committing Go changes
• Reviewing pull requests with Go code
• Onboarding to a new Go codebase
• Learning idiomatic Go patterns

​

Review Categories

​

CRITICAL (Must Fix)

• SQL/Command injection vulnerabilities
• Race conditions without synchronization
• Goroutine leaks
• Hardcoded credentials
• Unsafe pointer usage
• Ignored errors in critical paths

​

HIGH (Should Fix)

• Missing error wrapping with context
• Panic instead of error returns
• Context not propagated
• Unbuffered channels causing deadlocks
• Interface not satisfied errors
• Missing mutex protection

​

MEDIUM (Consider)

• Non-idiomatic code patterns
• Missing godoc comments on exports
• Inefficient string concatenation
• Slice not preallocated
• Table-driven tests not used

​

Automated Checks Run

# Static analysis
go vet ./...

# Advanced checks (if installed)
staticcheck ./...
golangci-lint run

# Race detection
go build -race ./...

# Security vulnerabilities
govulncheck ./...

​

Command Syntax

/go-review

​

Approval Criteria

​

Related

• Agent: agents/go-reviewer.md
• Skills: skills/golang-patterns/, skills/golang-testing/
• Commands: /go-test, /go-build, /code-review