Apply modern Go syntax guidelines based on project's Go version. Use when user ask for modern Go code guidelines.
!grep -rh "^go " --include="go.mod" . 2>/dev/null | cut -d' ' -f2 | sort | uniq -c | sort -nr | head -1 | xargs | cut -d' ' -f2 | grep . || echo unknown
DO NOT search for go.mod files or try to detect the version yourself. Use ONLY the version shown above.
If version detected (not "unknown"):
If version is "unknown":
When writing Go code, use ALL features from this document up to the target version:
slices, maps, cmp) over legacy patternstime.Since: time.Since(start) instead of time.Now().Sub(start)time.Until: time.Until(deadline) instead of deadline.Sub(time.Now())errors.Is: errors.Is(err, target) instead of err == target (works with wrapped errors)any: Use any instead of interface{}bytes.Cut: before, after, found := bytes.Cut(b, sep) instead of Index+slicestrings.Cut: before, after, found := strings.Cut(s, sep)fmt.Appendf: buf = fmt.Appendf(buf, "x=%d", x) instead of []byte(fmt.Sprintf(...))atomic.Bool/atomic.Int64/atomic.Pointer[T]: Type-safe atomics instead of atomic.StoreInt32var flag atomic.Bool
flag.Store(true)
if flag.Load() { ... }
var ptr atomic.Pointer[Config]
ptr.Store(cfg)
strings.Clone: strings.Clone(s) to copy string without sharing memorybytes.Clone: bytes.Clone(b) to copy byte slicestrings.CutPrefix/CutSuffix: if rest, ok := strings.CutPrefix(s, "pre:"); ok { ... }errors.Join: errors.Join(err1, err2) to combine multiple errorscontext.WithCancelCause: ctx, cancel := context.WithCancelCause(parent) then cancel(err)context.Cause: context.Cause(ctx) to get the error that caused cancellationBuilt-ins:
min/max: max(a, b) instead of if/else comparisonsclear: clear(m) to delete all map entries, clear(s) to zero slice elementsslices package:
slices.Contains: slices.Contains(items, x) instead of manual loopsslices.Index: slices.Index(items, x) returns index (-1 if not found)slices.IndexFunc: slices.IndexFunc(items, func(item T) bool { return item.ID == id })slices.SortFunc: slices.SortFunc(items, func(a, b T) int { return cmp.Compare(a.X, b.X) })slices.Sort: slices.Sort(items) for ordered typesslices.Max/slices.Min: slices.Max(items) instead of manual loopslices.Reverse: slices.Reverse(items) instead of manual swap loopslices.Compact: slices.Compact(items) removes consecutive duplicates in-placeslices.Clip: slices.Clip(s) removes unused capacityslices.Clone: slices.Clone(s) creates a copymaps package:
maps.Clone: maps.Clone(m) instead of manual map iterationmaps.Copy: maps.Copy(dst, src) copies entries from src to dstmaps.DeleteFunc: maps.DeleteFunc(m, func(k K, v V) bool { return condition })sync package:
sync.OnceFunc: f := sync.OnceFunc(func() { ... }) instead of sync.Once + wrappersync.OnceValue: getter := sync.OnceValue(func() T { return computeValue() })context package:
context.AfterFunc: stop := context.AfterFunc(ctx, cleanup) runs cleanup on cancellationcontext.WithTimeoutCause: ctx, cancel := context.WithTimeoutCause(parent, d, err)context.WithDeadlineCause: Similar with deadline instead of durationLoops:
for i := range n: for i := range len(items) instead of for i := 0; i < len(items); i++cmp package:
cmp.Or: cmp.Or(flag, env, config, "default") returns first non-zero value// Instead of:
name := os.Getenv("NAME")
if name == "" {
name = "default"
}
// Use:
name := cmp.Or(os.Getenv("NAME"), "default")
reflect package:
reflect.TypeFor: reflect.TypeFor[T]() instead of reflect.TypeOf((*T)(nil)).Elem()net/http:
http.ServeMux patterns: mux.HandleFunc("GET /api/{id}", handler) with method and path paramsr.PathValue("id") to get path parametersmaps.Keys(m) / maps.Values(m) return iteratorsslices.Collect(iter) not manual loop to build slice from iteratorslices.Sorted(iter) to collect and sort in one stepkeys := slices.Collect(maps.Keys(m)) // not: for k := range m { keys = append(keys, k) }
sortedKeys := slices.Sorted(maps.Keys(m)) // collect + sort
for k := range maps.Keys(m) { process(k) } // iterate directly
time package
time.Tick: Use time.Tick freely — as of Go 1.23, the garbage collector can recover unreferenced tickers, even if they haven't been stopped. The Stop method is no longer necessary to help the garbage collector. There is no longer any reason to prefer NewTicker when Tick will do.t.Context() not context.WithCancel(context.Background()) in tests.
ALWAYS use t.Context() when a test function needs a context.Before:
func TestFoo(t *testing.T) {
ctx, cancel := context.WithCancel(context.Background())
defer cancel()
result := doSomething(ctx)
}
After:
func TestFoo(t *testing.T) {
ctx := t.Context()
result := doSomething(ctx)
}
omitzero not omitempty in JSON struct tags.
ALWAYS use omitzero for time.Duration, time.Time, structs, slices, maps.Before:
type Config struct {
Timeout time.Duration `json:"timeout,omitempty"` // doesn't work for Duration!
}
After:
type Config struct {
Timeout time.Duration `json:"timeout,omitzero"`
}
b.Loop() not for i := 0; i < b.N; i++ in benchmarks.
ALWAYS use b.Loop() for the main loop in benchmark functions.Before:
func BenchmarkFoo(b *testing.B) {
for i := 0; i < b.N; i++ {
doWork()
}
}
After:
func BenchmarkFoo(b *testing.B) {
for b.Loop() {
doWork()
}
}
strings.SplitSeq not strings.Split when iterating.
ALWAYS use SplitSeq/FieldsSeq when iterating over split results in a for-range loop.Before:
for _, part := range strings.Split(s, ",") {
process(part)
}
After:
for part := range strings.SplitSeq(s, ",") {
process(part)
}
Also: strings.FieldsSeq, bytes.SplitSeq, bytes.FieldsSeq.
wg.Go(fn) not wg.Add(1) + go func() { defer wg.Done(); ... }().
ALWAYS use wg.Go() when spawning goroutines with sync.WaitGroup.Before:
var wg sync.WaitGroup
for _, item := range items {
wg.Add(1)
go func() {
defer wg.Done()
process(item)
}()
}
wg.Wait()
After:
var wg sync.WaitGroup
for _, item := range items {
wg.Go(func() {
process(item)
})
}
wg.Wait()
new(val) not x := val; &x — returns pointer to any value.
Go 1.26 extends new() to accept expressions, not just types.
Type is inferred: new(0) → *int, new("s") → *string, new(T{}) → *T.
DO NOT use x := val; &x pattern — always use new(val) directly.
DO NOT use redundant casts like new(int(0)) — just write new(0).
Common use case: struct fields with pointer types.Before:
timeout := 30
debug := true
cfg := Config{
Timeout: &timeout,
Debug: &debug,
}
After:
cfg := Config{
Timeout: new(30), // *int
Debug: new(true), // *bool
}
errors.AsType[T](err) not errors.As(err, &target).
ALWAYS use errors.AsType when checking if error matches a specific type.Before:
var pathErr *os.PathError
if errors.As(err, &pathErr) {
handle(pathErr)
}
After:
if pathErr, ok := errors.AsType[*os.PathError](err); ok {
handle(pathErr)
}