go-code-review

SKILL.md

Systematic Go code review against community style standards and best practices.

Covers 15+ review categories: formatting, documentation, error handling, naming, concurrency, interfaces, data structures, security, declarations, functions, style, logging, imports, generics, and testing

Includes automated pre-review checks via gofmt, go vet, and golangci-lint to catch mechanical issues before manual review

Organizes findings by severity (must-fix, should-fix, nit) using a consistent template for clear communication

References specialized skills for deeper dives into error handling, naming conventions, concurrency patterns, and testing strategies

Go Code Review Checklist

Review Procedure

Use assets/review-template.md when formatting the output of a code review to ensure consistent structure with Must Fix / Should Fix / Nits severity grouping.

Run gofmt -d . and go vet ./... to catch mechanical issues first

Read the diff file-by-file; for each file, check the categories below in order

Flag issues with specific line references and the rule name

After reviewing all files, re-read flagged items to verify they're genuine issues

Summarize findings grouped by severity (must-fix, should-fix, nit)

Validation: After completing the review, re-read the diff once more to verify every flagged issue is real. Remove any finding you cannot justify with a specific line reference.

Formatting

 gofmt: Code is formatted with gofmt or goimports → go-linting