From cd8bfcfbdc8028991291b38ced311211ca70b212 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 9 Jun 2026 17:09:30 +0000
Subject: [PATCH] fix: make ComPortController shutdown deactivation idempotent

---
 .../Comm and IR/ComPortController.cs               | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/src/PepperDash.Essentials.Core/Comm and IR/ComPortController.cs b/src/PepperDash.Essentials.Core/Comm and IR/ComPortController.cs
index 8b28d23b..0bc71c62 100644
--- a/src/PepperDash.Essentials.Core/Comm and IR/ComPortController.cs	
+++ b/src/PepperDash.Essentials.Core/Comm and IR/ComPortController.cs	
@@ -38,6 +38,8 @@ namespace PepperDash.Essentials.Core
 
 		ComPort Port;
 		ComPort.ComPortSpec Spec;
+		private readonly object _deactivateLock = new object();
+		private bool _isDeactivated;
 
 		/// <summary>
 		/// Constructor
@@ -148,7 +150,9 @@ namespace PepperDash.Essentials.Core
 		void CrestronEnvironment_ProgramStatusEventHandler(eProgramStatusEventType programEventType)
 		{
 			if (programEventType == eProgramStatusEventType.Stopping)
+			{
 				Deactivate();
+			}
 		}
 
 		/// <summary>
@@ -157,6 +161,16 @@ namespace PepperDash.Essentials.Core
 		/// <inheritdoc />
 		public override bool Deactivate()
 		{
+			lock (_deactivateLock)
+			{
+				if (_isDeactivated)
+					return true;
+
+				_isDeactivated = true;
+			}
+
+			CrestronEnvironment.ProgramStatusEventHandler -= CrestronEnvironment_ProgramStatusEventHandler;
+
 			if (Port == null)
 				return true;