From 3286d27898ef0378e27d4e7cef862bb0544489eb Mon Sep 17 00:00:00 2001
From: "anthropic-code-agent[bot]" <242468646+Claude@users.noreply.github.com>
Date: Fri, 26 Jun 2026 20:09:32 +0000
Subject: [PATCH 1/6] Initial plan


From af5611e403dc7fa8651b595bc493550db7be6361 Mon Sep 17 00:00:00 2001
From: "anthropic-code-agent[bot]" <242468646+Claude@users.noreply.github.com>
Date: Fri, 26 Jun 2026 20:11:48 +0000
Subject: [PATCH 2/6] Mark IMobileControlMessengerWithSubscriptions and
 EnableMessengerSubscriptions as obsolete

All messengers are now subscription based in v3.x, making these
constructs no longer necessary.

Closes #1435

Agent-Logs-Url: https://github.com/PepperDash/Essentials/sessions/bda64c9c-5343-412b-801f-5e60816bc38d

Co-authored-by: ndorin <18535240+ndorin@users.noreply.github.com>
---
 .../IMobileControlMessengerWithSubscriptions.cs               | 2 ++
 .../MobileControlConfig.cs                                    | 4 +++-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/PepperDash.Essentials.Core/DeviceTypeInterfaces/IMobileControlMessengerWithSubscriptions.cs b/src/PepperDash.Essentials.Core/DeviceTypeInterfaces/IMobileControlMessengerWithSubscriptions.cs
index 887f1789..e6365571 100644
--- a/src/PepperDash.Essentials.Core/DeviceTypeInterfaces/IMobileControlMessengerWithSubscriptions.cs
+++ b/src/PepperDash.Essentials.Core/DeviceTypeInterfaces/IMobileControlMessengerWithSubscriptions.cs
@@ -1,3 +1,4 @@
+using System;
 using PepperDash.Core;
 
 namespace PepperDash.Essentials.Core.DeviceTypeInterfaces
@@ -5,6 +6,7 @@ namespace PepperDash.Essentials.Core.DeviceTypeInterfaces
   /// <summary>
   /// Defines the contract for IMobileControlMessenger
   /// </summary>
+  [Obsolete("This interface is obsolete and will be removed in a future version. All messengers are now subscription based.")]
   public interface IMobileControlMessengerWithSubscriptions : IMobileControlMessenger
   {
     /// <summary>
diff --git a/src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs b/src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs
index ec7219a3..9beed963 100644
--- a/src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs
+++ b/src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs
@@ -1,4 +1,5 @@
-using System.Collections.Generic;
+using System;
+using System.Collections.Generic;
 using Newtonsoft.Json;
 using Newtonsoft.Json.Converters;
 
@@ -43,6 +44,7 @@ namespace PepperDash.Essentials
         /// Enable subscriptions for Messengers
         /// </summary>
         [JsonProperty("enableMessengerSubscriptions")]
+        [Obsolete("This property is obsolete and will be removed in a future version. All messengers are now subscription based.")]
         public bool EnableMessengerSubscriptions { get; set; }
     }
 

From 2fac0ca926b752d96b8bb777e923a5830ffc5baa Mon Sep 17 00:00:00 2001
From: Neil Dorin <ndorin@pepperdash.com>
Date: Fri, 26 Jun 2026 14:32:22 -0600
Subject: [PATCH 3/6] Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
---
 .../IMobileControlMessengerWithSubscriptions.cs                 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/PepperDash.Essentials.Core/DeviceTypeInterfaces/IMobileControlMessengerWithSubscriptions.cs b/src/PepperDash.Essentials.Core/DeviceTypeInterfaces/IMobileControlMessengerWithSubscriptions.cs
index e6365571..8603d5b6 100644
--- a/src/PepperDash.Essentials.Core/DeviceTypeInterfaces/IMobileControlMessengerWithSubscriptions.cs
+++ b/src/PepperDash.Essentials.Core/DeviceTypeInterfaces/IMobileControlMessengerWithSubscriptions.cs
@@ -4,7 +4,7 @@ using PepperDash.Core;
 namespace PepperDash.Essentials.Core.DeviceTypeInterfaces
 {
   /// <summary>
-  /// Defines the contract for IMobileControlMessenger
+  /// Obsolete: messengers are subscription based by default; use IMobileControlMessenger instead.
   /// </summary>
   [Obsolete("This interface is obsolete and will be removed in a future version. All messengers are now subscription based.")]
   public interface IMobileControlMessengerWithSubscriptions : IMobileControlMessenger

From 640bd7a8a781a33f734a5eb03086a79550ac344a Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Fri, 26 Jun 2026 21:55:49 +0000
Subject: [PATCH 4/6] Update XML summary for EnableMessengerSubscriptions to
 reflect obsolete status

---
 src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs b/src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs
index 9beed963..27c9d31e 100644
--- a/src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs
+++ b/src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs
@@ -41,7 +41,7 @@ namespace PepperDash.Essentials
         public bool EnableApiServer { get; set; } = true;
 
         /// <summary>
-        /// Enable subscriptions for Messengers
+        /// Retained for backward compatibility only. This property is obsolete; all messengers are now subscription based.
         /// </summary>
         [JsonProperty("enableMessengerSubscriptions")]
         [Obsolete("This property is obsolete and will be removed in a future version. All messengers are now subscription based.")]

From 0240887d93e890a56c42ce8c8e78f794d5406215 Mon Sep 17 00:00:00 2001
From: Neil Dorin <ndorin@pepperdash.com>
Date: Fri, 26 Jun 2026 15:57:52 -0600
Subject: [PATCH 5/6] Clarify summary for EnableMessengerSubscriptions property

Updated the summary comment for EnableMessengerSubscriptions property to clarify its purpose.
---
 .../MobileControlConfig.cs                                    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs b/src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs
index 27c9d31e..963e7fd5 100644
--- a/src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs
+++ b/src/PepperDash.Essentials.MobileControl/MobileControlConfig.cs
@@ -41,7 +41,7 @@ namespace PepperDash.Essentials
         public bool EnableApiServer { get; set; } = true;
 
         /// <summary>
-        /// Retained for backward compatibility only. This property is obsolete; all messengers are now subscription based.
+        /// Enables subscriptions for messengers
         /// </summary>
         [JsonProperty("enableMessengerSubscriptions")]
         [Obsolete("This property is obsolete and will be removed in a future version. All messengers are now subscription based.")]
@@ -290,4 +290,4 @@ namespace PepperDash.Essentials
         /// </summary>
         NEO
     }
-}
\ No newline at end of file
+}

From 8ac4eb75843f1267f410443e15c1a267792213a2 Mon Sep 17 00:00:00 2001
From: "anthropic-code-agent[bot]" <242468646+Claude@users.noreply.github.com>
Date: Fri, 26 Jun 2026 22:32:56 +0000
Subject: [PATCH 6/6] refactor: marked mobile control subscription items as
 obsolete

Co-authored-by: ndorin <18535240+ndorin@users.noreply.github.com>