refactor: remove WebRTCRelayEnabled config, always register endpoint

The relay is lazy-started on first request so there's no cost to
always registering the /display/webrtc endpoint. No need for a
config toggle.

Made-with: Cursor

Author: rgarcia

server/cmd/api/main.go

@@ -141,21 +141,17 @@ func main() {
 	// the VP8 video stream to external WebRTC clients via a single
 	// WebSocket signaling endpoint. The Neko connection is lazy —
 	// it only starts when the first client connects.
-	if config.WebRTCRelayEnabled {
-		relay, err := webrtcscreen.NewRelay(ctx, webrtcscreen.RelayConfig{
-			NekoBaseURL: "http://127.0.0.1:8080",
-			NekoUser:    "admin",
-			NekoPass:    adminPassword,
-			Logger:      slogger,
-		})
-		if err != nil {
-			slogger.Error("failed to create webrtc relay", "err", err)
-			os.Exit(1)
-		}
-
-		r.Get("/display/webrtc", relay.HandleWebSocket)
-		slogger.Info("webrtc relay endpoint registered at /display/webrtc (lazy start)")
+	relay, err := webrtcscreen.NewRelay(ctx, webrtcscreen.RelayConfig{
+		NekoBaseURL: "http://127.0.0.1:8080",
+		NekoUser:    "admin",
+		NekoPass:    adminPassword,
+		Logger:      slogger,
+	})
+	if err != nil {
+		slogger.Error("failed to create webrtc relay", "err", err)
+		os.Exit(1)
 	}
+	r.Get("/display/webrtc", relay.HandleWebSocket)
 
 	srv := &http.Server{
 		Addr:    fmt.Sprintf(":%d", config.Port),

server/cmd/config/config.go

@@ -31,10 +31,6 @@ type Config struct {
 	// DevTools proxy address passed to ChromeDriver as goog:chromeOptions.debuggerAddress.
 	// If empty, it is derived from DevToolsProxyPort as 127.0.0.1:<port>.
 	DevToolsProxyAddr string `envconfig:"DEVTOOLS_PROXY_ADDR" default:""`
-
-	// WebRTC relay: connects to Neko internally and re-serves the video
-	// stream to external WebRTC clients via /display/webrtc.
-	WebRTCRelayEnabled bool `envconfig:"WEBRTC_RELAY_ENABLED" default:"true"`
 }
 
 // Load loads configuration from environment variables