Help us improve
Share bugs, ideas, or general feedback.
From code-review
Reviews Go code for idiomatic patterns, table-driven test conventions, error handling, and build commands like go test -race. Auto-loads for .go files in pre-commit-review.
npx claudepluginhub openshift-eng/ai-helpers --plugin code-reviewHow this skill is triggered — by the user, by Claude, or both
Slash command
/code-review:lang-goThe summary Claude sees in its skill listing — used to decide when to auto-load this skill
This skill provides Go-specific guidance for code quality reviews. It is loaded automatically by the `pre-commit-review` command when `--language go` is specified or when `.go` files are detected among the changed files.
Reviews Go code against community style standards for formatting, documentation, error handling, and naming using checklists from Go Wiki and Uber guide. Use before PRs or code reviews.
Reviews Go code against style guide, focusing on critical bugs, race conditions, and maintainability issues. Use for PRs, feature branches, or completed work reviews.
Reviews Go code for idiomatic patterns, error handling, concurrency safety, and common mistakes. Useful for .go files, goroutines, interfaces, generics (1.18+), and errors.Join/slog (1.20+/1.21+).
Share bugs, ideas, or general feedback.
This skill provides Go-specific guidance for code quality reviews. It is loaded automatically by the pre-commit-review command when --language go is specified or when .go files are detected among the changed files.
Use this skill when reviewing Go source code. Its sections are referenced during the unit test coverage, idiomatic code, and build verification review steps.
t.Run() for subtests to provide clear test case identificationt.Parallel() at the top of test functions and subtests where tests are independent-race flag — avoid shared mutable state between parallel tests_test.go suffixt.Helper() in test helper functions so failure messages report the caller's line numberFollow the conventions in Effective Go and the Go Code Review Comments wiki. Key reminders:
gofmt. Unformatted code is a blocking issueMixedCaps / mixedCaps. Avoid stuttering (http.Server not http.HTTPServer). Acronyms are all caps (URL, HTTP, ID)fmt.Errorf("context: %w", err) to preserve the chain for errors.Is() / errors.As()context.Context as the first parameter to functions that perform I/O or may be long-runningUse the following commands in priority order during build verification:
make test if a Makefile with a test target exists; otherwise go test -race ./...make verify if available; otherwise go vet ./...make build if available; otherwise go build ./...