-race в CI и go vet

-race в CI

# упрощённый шаг CI
run: go test -race ./...
env:
  GORACE: halt_on_error=1

GORACE=halt_on_error=1 останавливает тесты на первой же гонке — без этого go test -race продолжит прогон, и гонка утонет в выводе. Детектор в CI — это страховка: гонка, проскочившая локально, падает в пайплайне и блокирует merge.

Go vet — дешёвый сосед

func work(wg sync.WaitGroup) { // баг: WaitGroup по значению
    wg.Add(1)
    go func() { defer wg.Done(); task() }()
    wg.Wait()
}

go vet ловит копирование блокировок — copylocks: передачу sync.Mutex, WaitGroup и прочих noCopy-типов по значению. Заметь: copylocks ловит именно полный go vet, а go test исполняет только малую часть анализаторов vet, и copylocks в неё не входит. Это именно тот баг из, который вешает Wait навсегда. Vet ловит его на этапе компиляции — дёшево, до всяких прогонов.

Полный набор

Итого дешёвый стандартный набор: go vet./... на каждый коммит, go test -race./... в CI. Одно ловит структурные ошибки с блокировками, другое — реальные гонки на пройденных путях. Вместе они закрывают почти все «невидимые» баги конкурентности до продакшена.