From 6596ac56a54dfc87808e04a8cb48bb8fb097317c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Torjus=20H=C3=A5kestad?= Date: Wed, 4 Feb 2026 01:45:16 +0100 Subject: [PATCH] fix: resolve staticcheck nil pointer dereference warning Use t.Fatal instead of t.Error when retrieved session is nil to prevent subsequent nil pointer dereference on retrieved.ID. Co-Authored-By: Claude Opus 4.5 --- internal/mcp/session_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/mcp/session_test.go b/internal/mcp/session_test.go index 2760ab9..febb114 100644 --- a/internal/mcp/session_test.go +++ b/internal/mcp/session_test.go @@ -85,7 +85,7 @@ func TestSessionStoreCreate(t *testing.T) { // Verify we can retrieve it retrieved := store.Get(session.ID) if retrieved == nil { - t.Error("Should be able to retrieve created session") + t.Fatal("Should be able to retrieve created session") } if retrieved.ID != session.ID { t.Error("Retrieved session ID should match")