fix: add return after t.Fatal for remaining SA5011 warnings

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: intel352

provider/openai_test.go

@@ -498,6 +498,7 @@ func TestOpenAIStreamWithToolCall(t *testing.T) {
 
 	if toolEvent == nil {
 		t.Fatal("expected tool_call event")
+		return
 	}
 	if toolEvent.Tool.ID != "call_xyz" {
 		t.Errorf("expected tool ID call_xyz, got %s", toolEvent.Tool.ID)

ratchetplugin/browser_manager_test.go

@@ -10,6 +10,7 @@ func TestNewBrowserManager(t *testing.T) {
 	bm := NewBrowserManager(true)
 	if bm == nil {
 		t.Fatal("expected non-nil BrowserManager")
+		return
 	}
 	if bm.browser != nil {
 		t.Error("expected browser to be nil before first GetPage call (lazy init)")

ratchetplugin/ratchetplugin_test.go

@@ -144,6 +144,7 @@ func TestNewSecretGuard(t *testing.T) {
 	sg := NewSecretGuard(p, "file")
 	if sg == nil {
 		t.Fatal("NewSecretGuard returned nil")
+		return
 	}
 	if sg.provider == nil {
 		t.Fatal("provider is nil")