Go quality

Skill fxckcode/skills-kit/skills/go-quality

A collection of agent skills for repository setup, multi-CLI agent workflows, and intelligent context loading.

Install

npx -y skills add fxckcode/skills-kit --skill go-quality

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

0 stars0 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

What its author says it does

Copied from the file, not written here

Go project quality standards — linting, race detection, CI setup, error wrapping, context propagation, and test isolation. Use when setting up, reviewing, or fixing Go projects.

The file declares its own license as Apache-2.0. That is the author’s claim about this one file, and it is not the same thing as the license GitHub reports for the repository, which is listed with the other numbers below.

SKILL.md

2.7 KB, as published. Nobody here has run it

Go Quality Standards

What This Skill Owns

Verifying Go project layout (cmd/, internal/, single go.mod)
Enforcing quality gates: build, vet, format, race detector, lint
Error handling: wrapping with %w, no panic, no naked discards
Context propagation: every blocking operation accepts ctx
Config hygiene: YAML + CLI flags + env, sane defaults
Testing: table-driven, race-safe, no global state mutation
CI: GitHub Actions with gofmt, vet, test -race

Quality Gates (MUST DO)

Build & Format

go build ./... && go vet ./...
gofmt -s -l .  # must be empty

Race Detector

go test -race -count=1 ./...

MUST pass. Non-negotiable.

Error Handling

Wrap errors: fmt.Errorf("context: %w", err) — NEVER %s or %v for wrapping
No panic() for normal flow (log.Fatalf only in main())
No naked _ = fn() discards without justification

Context Propagation

Every blocking operation accepts ctx context.Context as first param
Use context.WithTimeout / context.WithCancel — never unbounded goroutines
Goroutines MUST check ctx.Done()

Logging

One log.SetPrefix() in main.go — never duplicate prefixes in format strings
Use log.Printf, not fmt.Fprintf to stderr

Config

YAML for persistence + CLI flags for overrides
Default config MUST produce a working baseline
CLI > env > YAML > default priority

Testing

Table-driven tests with t.Run subtests
Use t.TempDir() for filesystem — never override HOME in TestMain
Mock via interfaces, not global state
Use t.Helper() in test helpers

CI Setup

jobs:
  quality:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - uses: actions/setup-go@v5
        with:
          go-version: "1.26"
          check-latest: true
      - run: test -z "$(gofmt -l -s .)"
      - run: go vet ./...
      - run: go test -race -count=1 ./...

Common Pitfalls

Pitfall	Symptom	Fix
Duplicate log prefix	`[app] [app] listen`	Use log.SetPrefix once, omit from format strings
Config points to missing backend	YAML references backend that doesn't exist	Keep default config in sync with actual backends
TestMain overrides HOME	All tests share contaminated env	Use per-test t.TempDir() instead
Streaming delta has empty role	`"role":""` in SSE chunks	Use Delta struct with omitempty
Health bypass too broad	`/api/v1/foo/health` skips auth	Compare exact path with ==, not HasSuffix