fix(leak): address linting issues - nolint:gochecknoglobals and wg.Go

Copilot · fredbi · Copilot · commit f0a59706b7b4 · 2026-03-19T18:36:40.000Z
- Add //nolint:gochecknoglobals annotation with justification on compatOnce
- Replace wg.Add(1)/go func()/defer wg.Done() with wg.Go() (Go 1.25)

Signed-off-by: GitHub Copilot &lt;copilot@github.com&gt;

Co-authored-by: fredbi &lt;14262513+fredbi@users.noreply.github.com&gt;
diff --git a/internal/leak/leak.go b/internal/leak/leak.go
@@ -25,7 +25,7 @@ const (
 	waitFactor  = 2
 )
 
-var compatOnce sync.Once
+var compatOnce sync.Once //nolint:gochecknoglobals // lazy guard for undocumented pprof label format
 
 // ensureCompatible checks that the pprof goroutine profile labels
 // are formatted as expected. It runs at most once and panics if the
@@ -43,11 +43,9 @@ func ensureCompatible() {
 		id := uniqueLabel()
 		labels := pprof.Labels(labelKey, id)
 		pprof.Do(context.Background(), labels, func(_ context.Context) {
-			wg.Add(1)
-			go func() {
-				defer wg.Done()
+			wg.Go(func() {
 				<-blocker // leaked: blocks forever until cleanup
-			}()
+			})
 		})
 		needle := buildNeedle(id)
 		profile := captureProfile()
