Skill

go/code-review

Go-specific code review checklist items. Extends core/code-review with Go-specific correctness, style, and performance checks.

Install

npx claudepluginhub deandum/claude-resources --plugin go-skills

Tool Access

This skill uses the workspace's default tool permissions.

Preview

Go-specific checklist items to supplement the core review process.

Supporting Assets

references/checklist.md

SKILL.md

Similar Skills

skill-lookup

Searches, retrieves, and installs Agent Skills from prompts.chat registry using MCP tools like search_skills and get_skill. Activates for finding skills, browsing catalogs, or extending Claude.

prompts.chat

159.9k

prompt-lookup

Searches prompts.chat for AI prompt templates by keyword or category, retrieves by ID with variable handling, and improves prompts via AI. Use for discovering or enhancing prompts.

prompts.chat

159.9k

Agent Development

6 files

Guides agent creation for Claude Code plugins with file templates, frontmatter specs (name, description, model), triggering examples, system prompts, and best practices.

plugin-dev

83.2k

Stats

Stars2

Forks1

Last CommitApr 15, 2026

Actions

View Source View Plugin View on GitHub View README

Go Code Review

Go-specific checklist items to supplement the core review process.

Correctness

Every error return checked — no _ for errors

defer for resource cleanup (files, rows, transactions, mutexes)

Goroutines have shutdown paths (context cancellation, errgroup)

No goroutine leaks (blocked channels, infinite loops without ctx check)

Nil pointer checks before dereferencing

Style

gofmt/goimports applied

Initialisms all caps (ID, URL, HTTP)

Interfaces defined at consumer, not provider

No init() functions for business logic — acceptable for side-effect registration (Cobra commands, flag parsers, test fixtures, pprof handlers). Prefer explicit initialization from main when possible.

got/want in tests, not actual/expected

Concurrency

No data races (go vet -race, shared state protected)

Channels properly closed (only by sender)

sync.Mutex not copied (pass by pointer)

errgroup or sync.WaitGroup for goroutine coordination

Performance

context.Context always first parameter

Connection pools configured (not default)

N+1 queries identified

sync.Pool only when allocation pressure measured

Security

SQL parameterized (? placeholders, never string concat)

No secrets in code or logs

Input validation at HTTP boundary

Full detailed checklist: checklist.md

Verification

All Go-specific checklist categories reviewed (correctness, style, concurrency, performance, security)

go vet ./... and golangci-lint run produce no findings

Race detector findings addressed (go test -race ./... passes)

Go Code Review

Go-specific checklist items to supplement the core review process.

Correctness

Every error return checked — no _ for errors
defer for resource cleanup (files, rows, transactions, mutexes)
Goroutines have shutdown paths (context cancellation, errgroup)
No goroutine leaks (blocked channels, infinite loops without ctx check)
Nil pointer checks before dereferencing

Style

gofmt/goimports applied
Initialisms all caps (ID, URL, HTTP)
Interfaces defined at consumer, not provider
No init() functions for business logic — acceptable for side-effect registration (Cobra commands, flag parsers, test fixtures, pprof handlers). Prefer explicit initialization from main when possible.
got/want in tests, not actual/expected

Concurrency

No data races (go vet -race, shared state protected)
Channels properly closed (only by sender)
sync.Mutex not copied (pass by pointer)
errgroup or sync.WaitGroup for goroutine coordination

Performance

context.Context always first parameter
Connection pools configured (not default)
N+1 queries identified
sync.Pool only when allocation pressure measured

Security

SQL parameterized (? placeholders, never string concat)
No secrets in code or logs
Input validation at HTTP boundary

Full detailed checklist: checklist.md

Verification

All Go-specific checklist categories reviewed (correctness, style, concurrency, performance, security)
go vet ./... and golangci-lint run produce no findings
Race detector findings addressed (go test -race ./... passes)