Replace `go.uber.go/atomic` with Go standard library `sync/atomic`

Métriques du dépôt

Stars: (28 187 étoiles)
Métriques de merge PR: (Merge moyen 5j 20h) (522 PRs mergées en 30 j)

Description

Summary

We currently have a linter rule that blocks usage of sync/atomic and forces us to use go.uber.go/atomic instead. This rule was originally added because sync/atomic required atomic operations on primitive types, which was considered error-prone.

However, since Go 1.19, the standard library sync/atomic package now includes typed wrappers (e.g., atomic.Uint32, atomic.Int64, atomic.Bool, etc.) that provide the same safety guarantees as go.uber.go/atomic. This makes the uber package redundant.

Context

Prometheus is also making this transition:

Proposed Changes

Remove the linter rule that blocks sync/atomic
Migrate all go.uber.go/atomic usage to the standard library sync/atomic typed wrappers
Configure a linter rule to block the old-style atomic functions that operate on primitive types (e.g., atomic.AddInt64, atomic.LoadUint32) to ensure we only use the new typed wrappers and avoid accidental misuse

This approach gives us:

Fewer external dependencies
Alignment with the broader Go ecosystem
Same safety guarantees we had with uber/atomic

References

Go 1.19 release notes on atomic types: https://tip.golang.org/doc/go1.19#atomic_types
sync/atomic package docs: https://pkg.go.dev/sync/atomic

Guide contributeur

Direction de recherche: Recherchez toutes les importations et utilisations de 'go.uber.org/atomic' dans le code et remplacez les par les wrappers typés correspondants de 'sync/atomic' (par exemple, atomic.Int64, atomic.Uint32). Assurez vous de mettre à jour la règle du linter en conséquence.
Stack technique: go
Domaine: backend
Type d'issue: Refactorisation
Difficulté: 3
Temps estimé: 1-2 jours
Statut d'activité: Active
Clarté: Claire
Prérequis: Go
Accessibilité débutant: 65