♻️ refactor: update codebase #2

Author: pnguyen215

pkg/slogger/README.md

@@ -812,11 +812,10 @@ Rate-limit identical log messages to prevent log storms during error spikes:
 ```go
 log := slogger.NewLogger().
     WithLevel(slogger.InfoLevel).
-    WithSampling(slogger.SamplingOptions{
-        First:      10,              // log the first 10 identical messages per second
-        Period:     time.Second,     // sliding window duration
-        Thereafter: 100,             // then log every 100th message
-    })
+    WithSampling(slogger.NewSamplingOptions().
+        WithFirst(10). // log the first 10 identical messages per second
+        WithPeriod(time.Second). // sliding window duration
+        WithThereafter(100)) // then log every 100th message
 ```
 
 Sampling is keyed on the exact message string. Each unique message maintains an

pkg/slogger/logger.go

@@ -424,9 +424,9 @@ func (l *Logger) WithCallerSkip(skip int) *Logger {
 // Returns:
 //
 // the receiver *Logger, enabling method chaining.
-func (l *Logger) WithSampling(opts SamplingOptions) *Logger {
+func (l *Logger) WithSampling(opts *SamplingOptions) *Logger {
 	l.mu.Lock()
-	l.sampling = newSampler(opts)
+	l.sampling = newSampler(*opts)
 	l.mu.Unlock()
 	return l
 }

pkg/slogger/logger_internal_test.go

@@ -778,7 +778,7 @@ func TestLogger_WithSampling(t *testing.T) {
 		WithOutput(&buf).
 		WithLevel(TraceLevel).
 		WithFormatter(NewTextFormatter(&buf).WithDisableColor()).
-		WithSampling(SamplingOptions{first: 1, thereafter: 0, period: time.Hour})
+		WithSampling(NewSamplingOptions().WithFirst(1).WithThereafter(0).WithPeriod(time.Hour))
 
 	// First message should be logged
 	log.Info("test sampling")