Merge pull request #1371 from PepperDash/temp-to-dev

Main -> Development

src/PepperDash.Essentials.Devices.Common/Displays/ScreenLiftController.cs

@@ -1,27 +1,15 @@
 using System;
 using System.Collections.Generic;
-using System.Timers;
 using Crestron.SimplSharp;
 using PepperDash.Core;
-using PepperDash.Core.Logging;
 using PepperDash.Essentials.Core;
 using PepperDash.Essentials.Core.Config;
 using PepperDash.Essentials.Core.CrestronIO;
 using PepperDash.Essentials.Core.DeviceTypeInterfaces;
-using PepperDash.Essentials.Devices.Common.Displays;
+using Serilog.Events;
 
 namespace PepperDash.Essentials.Devices.Common.Shades
 {
-    /// <summary>
-    /// Enumeration for requested state
-    /// </summary>
-    enum RequestedState
-    {
-        None,
-        Raise,
-        Lower
-    }
-
     /// <summary>
     /// Controls a single shade using three relays
     /// </summary>
@@ -32,16 +20,11 @@ namespace PepperDash.Essentials.Devices.Common.Shades
         readonly ScreenLiftRelaysConfig LowerRelayConfig;
         readonly ScreenLiftRelaysConfig LatchedRelayConfig;
 
-        DisplayBase DisplayDevice;
+        Displays.DisplayBase DisplayDevice;
         ISwitchedOutput RaiseRelay;
         ISwitchedOutput LowerRelay;
         ISwitchedOutput LatchedRelay;
 
-        private bool _isMoving;
-        private RequestedState _requestedState;
-        private RequestedState _currentMovement;
-        private Timer _movementTimer;
-
         /// <summary>
         /// Gets or sets the InUpPosition
         /// </summary>
@@ -97,11 +80,6 @@ namespace PepperDash.Essentials.Devices.Common.Shades
 
             IsInUpPosition = new BoolFeedback("isInUpPosition", () => _isInUpPosition);
 
-            // Initialize movement timer for reuse
-            _movementTimer = new Timer();
-            _movementTimer.Elapsed += OnMovementComplete;
-            _movementTimer.AutoReset = false;
-
             switch (Mode)
             {
                 case eScreenLiftControlMode.momentary:
@@ -151,25 +129,25 @@ namespace PepperDash.Essentials.Devices.Common.Shades
             {
                 case eScreenLiftControlMode.momentary:
                     {
-                        this.LogDebug("Getting relays for {mode}", Mode);
+                        Debug.LogMessage(LogEventLevel.Debug, this, $"Getting relays for {Mode}");
                         RaiseRelay = GetSwitchedOutputFromDevice(RaiseRelayConfig.DeviceKey);
                         LowerRelay = GetSwitchedOutputFromDevice(LowerRelayConfig.DeviceKey);
                         break;
                     }
                 case eScreenLiftControlMode.latched:
                     {
-                        this.LogDebug("Getting relays for {mode}", Mode);
+                        Debug.LogMessage(LogEventLevel.Debug, this, $"Getting relays for {Mode}");
                         LatchedRelay = GetSwitchedOutputFromDevice(LatchedRelayConfig.DeviceKey);
                         break;
                     }
             }
 
-            this.LogDebug("Getting display with key {displayKey}", DisplayDeviceKey);
+            Debug.LogMessage(LogEventLevel.Debug, this, $"Getting display with key {DisplayDeviceKey}");
             DisplayDevice = GetDisplayBaseFromDevice(DisplayDeviceKey);
 
             if (DisplayDevice != null)
             {
-                this.LogDebug("Subscribing to {displayKey} feedbacks", DisplayDeviceKey);
+                Debug.LogMessage(LogEventLevel.Debug, this, $"Subscribing to {DisplayDeviceKey} feedbacks");
 
                 DisplayDevice.IsWarmingUpFeedback.OutputChange += IsWarmingUpFeedback_OutputChange;
                 DisplayDevice.IsCoolingDownFeedback.OutputChange += IsCoolingDownFeedback_OutputChange;
@@ -185,49 +163,22 @@ namespace PepperDash.Essentials.Devices.Common.Shades
         {
             if (RaiseRelay == null && LatchedRelay == null) return;
 
-            this.LogDebug("Raise called for {type}", Type);
-
-            // If device is moving, bank the command
-            if (_isMoving)
-            {
-                this.LogDebug("Device is moving, banking Raise command");
-                _requestedState = RequestedState.Raise;
-                return;
-            }
-
-            this.LogDebug("Raising {type}", Type);
+            Debug.LogMessage(LogEventLevel.Debug, this, $"Raising {Type}");
 
             switch (Mode)
             {
                 case eScreenLiftControlMode.momentary:
                     {
                         PulseOutput(RaiseRelay, RaiseRelayConfig.PulseTimeInMs);
-
-                        // Set moving flag and start timer if movement time is configured
-                        if (RaiseRelayConfig.MoveTimeInMs > 0)
-                        {
-                            _isMoving = true;
-                            _currentMovement = RequestedState.Raise;
-                            if (_movementTimer.Enabled)
-                            {
-                                _movementTimer.Stop();
-                            }
-                            _movementTimer.Interval = RaiseRelayConfig.MoveTimeInMs;
-                            _movementTimer.Start();
-                        }
-                        else
-                        {
-                            InUpPosition = true;
-                        }
                         break;
                     }
                 case eScreenLiftControlMode.latched:
                     {
                         LatchedRelay.Off();
-                        InUpPosition = true;
                         break;
                     }
             }
+            InUpPosition = true;
         }
 
         /// <summary>
@@ -237,145 +188,59 @@ namespace PepperDash.Essentials.Devices.Common.Shades
         {
             if (LowerRelay == null && LatchedRelay == null) return;
 
-            this.LogDebug("Lower called for {type}", Type);
-
-            // If device is moving, bank the command
-            if (_isMoving)
-            {
-                this.LogDebug("Device is moving, banking Lower command");
-                _requestedState = RequestedState.Lower;
-                return;
-            }
-
-            this.LogDebug("Lowering {type}", Type);
+            Debug.LogMessage(LogEventLevel.Debug, this, $"Lowering {Type}");
 
             switch (Mode)
             {
                 case eScreenLiftControlMode.momentary:
                     {
                         PulseOutput(LowerRelay, LowerRelayConfig.PulseTimeInMs);
-
-                        // Set moving flag and start timer if movement time is configured
-                        if (LowerRelayConfig.MoveTimeInMs > 0)
-                        {
-                            _isMoving = true;
-                            _currentMovement = RequestedState.Lower;
-                            if (_movementTimer.Enabled)
-                            {
-                                _movementTimer.Stop();
-                            }
-                            _movementTimer.Interval = LowerRelayConfig.MoveTimeInMs;
-                            _movementTimer.Start();
-                        }
-                        else
-                        {
-                            InUpPosition = false;
-                        }
                         break;
                     }
                 case eScreenLiftControlMode.latched:
                     {
                         LatchedRelay.On();
-                        InUpPosition = false;
                         break;
                     }
             }
+            InUpPosition = false;
         }
 
-        private void DisposeMovementTimer()
+        void PulseOutput(ISwitchedOutput output, int pulseTime)
         {
-            if (_movementTimer != null)
-            {
-                _movementTimer.Stop();
-                _movementTimer.Elapsed -= OnMovementComplete;
-                _movementTimer.Dispose();
-                _movementTimer = null;
-            }
+            output.On();
+            CTimer pulseTimer = new CTimer(new CTimerCallbackFunction((o) => output.Off()), pulseTime);
         }
 
         /// <summary>
-        /// Called when movement timer completes
+        /// Attempts to get the port on teh specified device from config
         /// </summary>
-        private void OnMovementComplete(object sender, ElapsedEventArgs e)
+        /// <param name="relayKey"></param>
+        /// <returns></returns>
+        ISwitchedOutput GetSwitchedOutputFromDevice(string relayKey)
         {
-            this.LogDebug("Movement complete");
-
-            // Update position based on completed movement
-            if (_currentMovement == RequestedState.Raise)
-            {
-                InUpPosition = true;
-            }
-            else if (_currentMovement == RequestedState.Lower)
-            {
-                InUpPosition = false;
-            }
-
-            _isMoving = false;
-            _currentMovement = RequestedState.None;
-
-            // Execute banked command if one exists
-            if (_requestedState != RequestedState.None)
-            {
-                this.LogDebug("Executing next command: {command}", _requestedState);
-
-                var commandToExecute = _requestedState;
-                _requestedState = RequestedState.None;
-
-                // Check if current state matches what the banked command would do and execute if different
-                switch (commandToExecute)
-                {
-                    case RequestedState.Raise:
-                        Raise();
-                        break;
-
-                    case RequestedState.Lower:
-                        Lower();
-                        break;
-                }
-            }
-        }
-
-        private void PulseOutput(ISwitchedOutput output, int pulseTime)
-        {
-            output.On();
-
-            var timer = new Timer(pulseTime)
-            {
-                AutoReset = false
-            };
-
-            timer.Elapsed += (sender, e) =>
-                  {
-                      output.Off();
-                      timer.Dispose();
-                  };
-            timer.Start();
-        }
-
-        private ISwitchedOutput GetSwitchedOutputFromDevice(string relayKey)
-        {
-            var portDevice = DeviceManager.GetDeviceForKey<ISwitchedOutput>(relayKey);
+            var portDevice = DeviceManager.GetDeviceForKey(relayKey);
             if (portDevice != null)
             {
-                return portDevice;
+                return portDevice as ISwitchedOutput;
             }
             else
             {
-                this.LogWarning("Error: Unable to get relay device with key '{relayKey}'", relayKey);
+                Debug.LogMessage(LogEventLevel.Debug, this, "Error: Unable to get relay device with key '{0}'", relayKey);
                 return null;
             }
         }
 
-        private DisplayBase GetDisplayBaseFromDevice(string displayKey)
+        Displays.DisplayBase GetDisplayBaseFromDevice(string displayKey)
         {
-            var displayDevice = DeviceManager.GetDeviceForKey<DisplayBase>(displayKey);
+            var displayDevice = DeviceManager.GetDeviceForKey(displayKey);
             if (displayDevice != null)
             {
-                return displayDevice;
+                return displayDevice as Displays.DisplayBase;
             }
             else
             {
-                this.LogWarning("Error: Unable to get display device with key '{displayKey}'", displayKey);
+                Debug.LogMessage(LogEventLevel.Debug, this, "Error: Unable to get display device with key '{0}'", displayKey);
                 return null;
             }
         }
@@ -383,7 +248,7 @@ namespace PepperDash.Essentials.Devices.Common.Shades
     }
 
     /// <summary>
-    /// Factory for ScreenLiftController devices
+    /// Represents a ScreenLiftControllerFactory
     /// </summary>
     public class ScreenLiftControllerFactory : EssentialsDeviceFactory<RelayControlledShade>
     {
@@ -395,11 +260,14 @@ namespace PepperDash.Essentials.Devices.Common.Shades
             TypeNames = new List<string>() { "screenliftcontroller" };
         }
 
+        /// <summary>
+        /// BuildDevice method
+        /// </summary>
         /// <inheritdoc />
         public override EssentialsDevice BuildDevice(DeviceConfig dc)
         {
-            Debug.LogDebug("Factory Attempting to create new ScreenLiftController Device");
-            var props = dc.Properties.ToObject<ScreenLiftControllerConfigProperties>();
+            Debug.LogMessage(LogEventLevel.Debug, "Factory Attempting to create new Generic Comm Device");
+            var props = Newtonsoft.Json.JsonConvert.DeserializeObject<ScreenLiftControllerConfigProperties>(dc.Properties.ToString());
 
             return new ScreenLiftController(dc.Key, dc.Name, props);
         }

src/PepperDash.Essentials.Devices.Common/Displays/ScreenLiftRelaysConfig.cs

@@ -18,11 +18,5 @@ namespace PepperDash.Essentials.Devices.Common.Shades
     /// </summary>
     [JsonProperty("pulseTimeInMs")]
     public int PulseTimeInMs { get; set; }
-
-    /// <summary>
-    /// Gets or sets the MoveTimeInMs - time in milliseconds for the movement to complete
-    /// </summary>
-    [JsonProperty("moveTimeInMs")]
-    public int MoveTimeInMs { get; set; }
   }
 }

src/PepperDash.Essentials.MobileControl/MobileControlSystemController.cs

@@ -1748,7 +1748,7 @@ namespace PepperDash.Essentials
                 var clientNo = 1;
                 foreach (var clientContext in _directServer.UiClientContexts)
                 {
-                    var clients = _directServer.UiClients.Values.Where(c => c.TokenKey == clientContext.Key);
+                    var clients = _directServer.UiClients.Values.Where(c => c.Token == clientContext.Value.Token.Token);
 
                     CrestronConsole.ConsoleCommandResponse(
                         $"\r\nClient {clientNo}:\r\n" +

src/PepperDash.Essentials.MobileControl/Utilities.cs

@@ -1,4 +1,3 @@
-using System.Threading;
 using PepperDash.Core;
 using PepperDash.Core.Logging;
 using WebSocketSharp;
@@ -13,12 +12,13 @@ namespace PepperDash.Essentials
     private static int nextClientId = 0;
 
     /// <summary>
-    /// Get the next unique client ID (thread-safe)
+    /// Get the next unique client ID
     /// </summary>
     /// <returns>Client ID</returns>
     public static int GetNextClientId()
     {
-      return Interlocked.Increment(ref nextClientId);
+      nextClientId++;
+      return nextClientId;
     }
     /// <summary>
     /// Converts a WebSocketServer LogData object to Essentials logging calls.

src/PepperDash.Essentials.MobileControl/WebSocketServer/JoinResponse.cs

@@ -64,12 +64,6 @@ namespace PepperDash.Essentials.WebSocketServer
     [JsonProperty("userAppUrl")]
     public string UserAppUrl { get; set; }
 
-    /// <summary>
-    /// Gets or sets the WebSocketUrl with clientId query parameter
-    /// </summary>
-    [JsonProperty("webSocketUrl")]
-    public string WebSocketUrl { get; set; }
-
 
     /// <summary>
     /// Gets or sets the EnableDebug

src/PepperDash.Essentials.MobileControl/WebSocketServer/MobileControlWebsocketServer.cs

@@ -1,5 +1,4 @@
 using System;
-using System.Collections.Concurrent;
 using System.Collections.Generic;
 using System.ComponentModel;
 using System.IO;
@@ -60,24 +59,12 @@ namespace PepperDash.Essentials.WebSocketServer
         /// </summary>
         public Dictionary<string, UiClientContext> UiClientContexts { get; private set; }
 
-        private readonly ConcurrentDictionary<string, UiClient> uiClients = new ConcurrentDictionary<string, UiClient>();
-
-        /// <summary>
-        /// Stores pending client registrations using composite key: token-clientId
-        /// This ensures the correct client ID is matched even when connections establish out of order
-        /// </summary>
-        private readonly ConcurrentDictionary<string, string> pendingClientRegistrations = new ConcurrentDictionary<string, string>();
-
-        /// <summary>
-        /// Stores queues of pending client IDs per token for legacy clients (FIFO)
-        /// This ensures thread-safety when multiple legacy clients use the same token
-        /// </summary>
-        private readonly ConcurrentDictionary<string, ConcurrentQueue<string>> legacyClientIdQueues = new ConcurrentDictionary<string, ConcurrentQueue<string>>();
+        private readonly Dictionary<string, UiClient> uiClients = new Dictionary<string, UiClient>();
 
         /// <summary>
         /// Gets the collection of UI clients
         /// </summary>
-        public IReadOnlyDictionary<string, UiClient> UiClients => uiClients;
+        public ReadOnlyDictionary<string, UiClient> UiClients => new ReadOnlyDictionary<string, UiClient>(uiClients);
 
         private readonly MobileControlSystemController _parent;
 
@@ -736,95 +723,23 @@ namespace PepperDash.Essentials.WebSocketServer
 
         private UiClient BuildUiClient(string roomKey, JoinToken token, string key)
         {
-            // Dequeue the next clientId for legacy client support (FIFO per token)
-            // New clients will override this ID in OnOpen with the validated query parameter value
-            var clientId = "pending";
-            if (legacyClientIdQueues.TryGetValue(key, out var queue) && queue.TryDequeue(out var dequeuedId))
-            {
-                clientId = dequeuedId;
-                this.LogVerbose("Dequeued legacy clientId {clientId} for token {token}", clientId, key);
-            }
-            
-            var c = new UiClient($"uiclient-{key}-{roomKey}-{clientId}", clientId, token.Token, token.TouchpanelKey);
-            this.LogInformation("Constructing UiClient with key {key} and temporary ID (will be set from query param)", key);
+            var c = new UiClient($"uiclient-{key}-{roomKey}-{token.Id}", token.Id, token.Token, token.TouchpanelKey);
+            this.LogInformation("Constructing UiClient with key {key} and ID {id}", key, token.Id);
             c.Controller = _parent;
             c.RoomKey = roomKey;
-            c.TokenKey = key; // Store the URL token key for filtering
-            c.Server = this; // Give UiClient access to server for ID registration
 
-            // Don't add to uiClients yet - will be added in OnOpen after ID is set from query param
-            
-            c.ConnectionClosed += (o, a) => 
+            if (uiClients.ContainsKey(token.Id))
             {
-                uiClients.TryRemove(a.ClientId, out _);
-                // Clean up any pending registrations for this token
-                var keysToRemove = pendingClientRegistrations.Keys
-                    .Where(k => k.StartsWith($"{key}-"))
-                    .ToList();
-                foreach (var k in keysToRemove)
-                {
-                    pendingClientRegistrations.TryRemove(k, out _);
-                }
-                
-                // Clean up legacy queue if empty
-                if (legacyClientIdQueues.TryGetValue(key, out var legacyQueue) && legacyQueue.IsEmpty)
-                {
-                    legacyClientIdQueues.TryRemove(key, out _);
-                }
-            };
+                this.LogWarning("removing client with duplicate id {id}", token.Id);
+                uiClients.Remove(token.Id);
+            }
+            uiClients.Add(token.Id, c);
+            // UiClients[key].SetClient(c);
+            c.ConnectionClosed += (o, a) => uiClients.Remove(a.ClientId);
+            token.Id = null;
             return c;
         }
 
-        /// <summary>
-        /// Registers a UiClient with its validated client ID after WebSocket connection
-        /// </summary>
-        /// <param name="client">The UiClient to register</param>
-        /// <param name="clientId">The validated client ID</param>
-        /// <param name="tokenKey">The token key for validation</param>
-        /// <returns>True if registration successful, false if validation failed</returns>
-        public bool RegisterUiClient(UiClient client, string clientId, string tokenKey)
-        {
-            var registrationKey = $"{tokenKey}-{clientId}";
-            
-            // Verify this clientId was generated during a join request for this token
-            if (!pendingClientRegistrations.TryRemove(registrationKey, out _))
-            {
-                this.LogWarning("Client attempted to connect with unregistered or expired clientId {clientId} for token {token}", clientId, tokenKey);
-                return false;
-            }
-
-            // Registration is valid - add to active clients
-            uiClients.AddOrUpdate(clientId, client, (id, existingClient) =>
-            {
-                this.LogWarning("Replacing existing client with duplicate id {id}", id);
-                return client;
-            });
-            
-            this.LogInformation("Successfully registered UiClient with ID {clientId} for token {token}", clientId, tokenKey);
-            return true;
-        }
-
-        /// <summary>
-        /// Registers a UiClient using legacy flow (for backwards compatibility with older clients)
-        /// </summary>
-        /// <param name="client">The UiClient to register</param>
-        public void RegisterLegacyUiClient(UiClient client)
-        {
-            if (string.IsNullOrEmpty(client.Id))
-            {
-                this.LogError("Cannot register client with null or empty ID");
-                return;
-            }
-
-            uiClients.AddOrUpdate(client.Id, client, (id, existingClient) =>
-            {
-                this.LogWarning("Replacing existing client with duplicate id {id} (legacy flow)", id);
-                return client;
-            });
-            
-            this.LogInformation("Successfully registered UiClient with ID {clientId} using legacy flow", client.Id);
-        }
-
         /// <summary>
         /// Prints out the session data for each path
         /// </summary>
@@ -1131,22 +1046,10 @@ namespace PepperDash.Essentials.WebSocketServer
                 });
             }
 
-            // Generate a client ID for this join request
             var clientId = $"{Utilities.GetNextClientId()}";
-            
-            // Store in pending registrations for new clients that send clientId via query param
-            var registrationKey = $"{token}-{clientId}";
-            pendingClientRegistrations.TryAdd(registrationKey, clientId);
-            
-            // Also enqueue for legacy clients (thread-safe FIFO per token)
-            var queue = legacyClientIdQueues.GetOrAdd(token, _ => new ConcurrentQueue<string>());
-            queue.Enqueue(clientId);
+            clientContext.Token.Id = clientId;
 
-            this.LogVerbose("Assigning ClientId: {clientId} for token: {token}", clientId, token);
-
-            // Construct WebSocket URL with clientId query parameter
-            var wsProtocol = "ws";
-            var wsUrl = $"{wsProtocol}://{CrestronEthernetHelper.GetEthernetParameter(CrestronEthernetHelper.ETHERNET_PARAMETER_TO_GET.GET_CURRENT_IP_ADDRESS, 0)}:{Port}{_wsPath}{token}?clientId={clientId}";
+            this.LogVerbose("Assigning ClientId: {clientId}", clientId);
 
             // Construct the response object
             JoinResponse jRes = new JoinResponse
@@ -1161,7 +1064,6 @@ namespace PepperDash.Essentials.WebSocketServer
                 UserAppUrl = string.Format("http://{0}:{1}/mc/app",
                 CrestronEthernetHelper.GetEthernetParameter(CrestronEthernetHelper.ETHERNET_PARAMETER_TO_GET.GET_CURRENT_IP_ADDRESS, 0),
                 Port),
-                WebSocketUrl = wsUrl,
                 EnableDebug = false,
                 DeviceInterfaceSupport = deviceInterfaces
             };

src/PepperDash.Essentials.MobileControl/WebSocketServer/UiClient.cs

@@ -31,11 +31,6 @@ namespace PepperDash.Essentials.WebSocketServer
         /// </summary>
         public string Token { get; private set; }
 
-        /// <summary>
-        /// The URL token key used to connect (from UiClientContexts dictionary key)
-        /// </summary>
-        public string TokenKey { get; set; }
-
         /// <summary>
         /// Touchpanel Key associated with this client
         /// </summary>
@@ -46,11 +41,6 @@ namespace PepperDash.Essentials.WebSocketServer
         /// </summary>
         public MobileControlSystemController Controller { get; set; }
 
-        /// <summary>
-        /// Gets or sets the server instance for client registration
-        /// </summary>
-        public MobileControlWebsocketServer Server { get; set; }
-
         /// <summary>
         /// Gets or sets the room key that this client is associated with
         /// </summary>
@@ -109,50 +99,6 @@ namespace PepperDash.Essentials.WebSocketServer
             Log.Output = (data, message) => Utilities.ConvertWebsocketLog(data, message, this);
             Log.Level = LogLevel.Trace;
 
-            // Get clientId from query parameter
-            var queryString = Context.QueryString;
-            var clientId = queryString["clientId"];
-
-            if (!string.IsNullOrEmpty(clientId))
-            {
-                // New behavior: Validate and register with the server using provided clientId
-                if (Server == null || !Server.RegisterUiClient(this, clientId, TokenKey))
-                {
-                    this.LogError("Failed to register client with ID {clientId}. Invalid or expired registration.", clientId);
-                    Context.WebSocket.Close(CloseStatusCode.PolicyViolation, "Invalid or expired clientId");
-                    return;
-                }
-
-                // Update this client's ID to the validated one
-                Id = clientId;
-                Key = $"uiclient-{TokenKey}-{RoomKey}-{clientId}";
-
-                this.LogInformation("Client {clientId} successfully connected and registered (new flow)", clientId);
-            }
-            else
-            {
-                // Legacy behavior: Use clientId from Token.Id (generated in HandleJoinRequest)
-                this.LogInformation("Client connected without clientId query parameter. Using legacy registration flow.");
-
-                // Id is already set from Token in constructor, use it
-                if (string.IsNullOrEmpty(Id))
-                {
-                    this.LogError("Legacy client has no ID from token. Connection will be closed.");
-                    Context.WebSocket.Close(CloseStatusCode.PolicyViolation, "No client ID available");
-                    return;
-                }
-
-                Key = $"uiclient-{TokenKey}-{RoomKey}-{Id}";
-
-                // Register directly to active clients (legacy flow)
-                if (Server != null)
-                {
-                    Server.RegisterLegacyUiClient(this);
-                }
-
-                this.LogInformation("Client {clientId} registered using legacy flow", Id);
-            }
-
             if (Controller == null)
             {
                 Debug.LogMessage(LogEventLevel.Verbose, "WebSocket UiClient Controller is null");

src/PepperDash.Essentials/ControlSystem.cs

@@ -662,7 +662,6 @@ namespace PepperDash.Essentials
 
             if (jsonFiles.Length > 1)
             {
-                Debug.LogError("Multiple configuration files found in application directory: {@jsonFiles}", jsonFiles.Select(f => f.FullName).ToArray());
                 throw new Exception("Multiple configuration files found. Cannot continue.");
             }