From 80baf3ad95e6183ff021de7e460cc9004591ff9f Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Tue, 2 Dec 2025 14:56:35 +0000
Subject: [PATCH] refactor: Remove unreachable dead code branches

- checkOrigin: Remove redundant same-origin check at line 155 that was
  already handled at line 116 (origin == requestOrigin)

Function improved from 95.1% to 97.4% coverage.
---
 internal/websocket/hub.go | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/internal/websocket/hub.go b/internal/websocket/hub.go
index 4190493da..d423cc679 100644
--- a/internal/websocket/hub.go
+++ b/internal/websocket/hub.go
@@ -151,11 +151,7 @@ func (h *Hub) checkOrigin(r *http.Request) bool {
 			return true
 		}
 
-		// Still check for exact same-origin match
-		if origin == requestOrigin {
-			return true
-		}
-
+		// Note: same-origin match already handled above (line 116)
 		log.Warn().
 			Str("origin", origin).
 			Str("requestOrigin", requestOrigin).