From 67330d5749390e42356bc9144b4f477f7e979d6d Mon Sep 17 00:00:00 2001 From: Kristjan Kruus Date: Tue, 24 Mar 2026 21:16:25 +0200 Subject: [PATCH] fix: resolve golangci-lint dogsled and gci warnings - Fix dogsled: reduce blank identifiers in newTestAgentLoop calls - Fix gci: auto-fixed import ordering in loop_test.go --- pkg/agent/loop_test.go | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/pkg/agent/loop_test.go b/pkg/agent/loop_test.go index 85621fb48..6addb9574 100644 --- a/pkg/agent/loop_test.go +++ b/pkg/agent/loop_test.go @@ -2134,7 +2134,8 @@ func TestFilterClientWebSearch_EmptyInput(t *testing.T) { } func TestProcessHeartbeat_SkipsWhenAgentBusy(t *testing.T) { - al, _, _, _, cleanup := newTestAgentLoop(t) + al, cfg, _, _, cleanup := newTestAgentLoop(t) + _ = cfg defer cleanup() // Simulate an active turn by registering a turnState via the helper @@ -2157,7 +2158,8 @@ func TestProcessHeartbeat_SkipsWhenAgentBusy(t *testing.T) { } func TestProcessHeartbeat_RunsWhenIdle(t *testing.T) { - al, _, _, _, cleanup := newTestAgentLoop(t) + al, cfg, _, _, cleanup := newTestAgentLoop(t) + _ = cfg defer cleanup() // Verify no active turns exist @@ -2180,7 +2182,8 @@ func TestProcessHeartbeat_RunsWhenIdle(t *testing.T) { } func TestProcessHeartbeat_SkipsForAnyActiveSession(t *testing.T) { - al, _, _, _, cleanup := newTestAgentLoop(t) + al, cfg, _, _, cleanup := newTestAgentLoop(t) + _ = cfg defer cleanup() // Register multiple active turns to cover the sync.Map iteration @@ -2205,7 +2208,8 @@ func TestProcessHeartbeat_SkipsForAnyActiveSession(t *testing.T) { } func TestProcessHeartbeat_ProceedsAfterTurnClears(t *testing.T) { - al, _, _, _, cleanup := newTestAgentLoop(t) + al, cfg, _, _, cleanup := newTestAgentLoop(t) + _ = cfg defer cleanup() // Simulate a turn that starts and then completes