From 3d0175a3bd667bcd40c68970eaec5fed66480e8d Mon Sep 17 00:00:00 2001
From: Pulse Monitor <pulse@pulse-monitor.dev>
Date: Wed, 27 Aug 2025 13:50:39 +0000
Subject: [PATCH] fix: remove mock support from production builds

---
 internal/monitoring/mock_stub.go | 10 ----------
 internal/monitoring/monitor.go   | 13 -------------
 2 files changed, 23 deletions(-)
 delete mode 100644 internal/monitoring/mock_stub.go

diff --git a/internal/monitoring/mock_stub.go b/internal/monitoring/mock_stub.go
deleted file mode 100644
index 2186e8efe..000000000
--- a/internal/monitoring/mock_stub.go
+++ /dev/null
@@ -1,10 +0,0 @@
-// +build production
-
-package monitoring
-
-import "github.com/rcourtman/pulse-go-rewrite/internal/models"
-
-// getMockState returns nil in production builds
-func getMockState() *models.StateSnapshot {
-	return nil
-}
\ No newline at end of file
diff --git a/internal/monitoring/monitor.go b/internal/monitoring/monitor.go
index e215e6c68..e887bc8f5 100644
--- a/internal/monitoring/monitor.go
+++ b/internal/monitoring/monitor.go
@@ -2106,19 +2106,6 @@ func (m *Monitor) pollPBSInstance(ctx context.Context, instanceName string, clie
 
 // GetState returns the current state
 func (m *Monitor) GetState() models.StateSnapshot {
-	// Check if mock mode is enabled
-	if mockEnabled := os.Getenv("PULSE_MOCK_MODE") == "true"; mockEnabled {
-		// Import is handled at package level, use fully qualified name
-		if mockState := getMockState(); mockState != nil {
-			log.Debug().
-				Int("nodes", len(mockState.Nodes)).
-				Int("vms", len(mockState.VMs)).
-				Int("containers", len(mockState.Containers)).
-				Msg("Returning mock state")
-			return *mockState
-		}
-		log.Warn().Msg("Mock mode enabled but getMockState returned nil")
-	}
 	return m.state.GetSnapshot()
 }