chore: drop ticket-id refs from comments and test names

Strip "SDK-4388" / "SDK-4474" prefixes from comments and test descriptions
in RefreshUserOperationExecutor and its test class. The descriptive context
that followed the ticket id already explains the why; the ticket id itself
is noise for future maintainers and is preserved in git history / commit
messages.

Author: sherwinski

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/user/internal/operations/impl/executors/RefreshUserOperationExecutor.kt

@@ -102,7 +102,7 @@ internal class RefreshUserOperationExecutor(
             propertiesModel.timezone = TimeUtils.getTimeZoneId()
 
             // Retrieve the push subscription ID from the backend configuration model store. Used
-            // both for re-attaching the locally-cached push model below and for the SDK-4474
+            // both for re-attaching the locally-cached push model below and for the push
             // self-heal divergence check inside the loop.
             val pushSubscriptionIdFromConfig = _configModelStore.model.pushSubscriptionId
 
@@ -136,10 +136,9 @@ internal class RefreshUserOperationExecutor(
                 if (subscriptionModel.type != SubscriptionType.PUSH) {
                     subscriptionModels.add(subscriptionModel)
                 } else if (subscription.id == pushSubscriptionIdFromConfig && pushSelfHealOperation == null) {
-                    // SDK-4474 self-heal for SDK-4388 victims. The buggy
-                    // SubscriptionOperationExecutor in older SDK builds dispatched the merged
-                    // create-subscription + update-subscription(SUBSCRIBED) batch as a POST
-                    // /subscriptions carrying the already-existing server-side id; the server
+                    // Self-heal for users stuck at "Never Subscribed". Older SDK builds dispatched
+                    // the merged create-subscription + update-subscription(SUBSCRIBED) batch as a
+                    // POST /subscriptions carrying the already-existing server-side id; the server
                     // responded 200 {} (no-op) and the local op queue was emptied, leaving the
                     // server stuck at enabled:false / notification_types:0. Once that user
                     // upgrades to a fixed SDK build, nothing in the queue triggers the fix path.
@@ -200,9 +199,9 @@ internal class RefreshUserOperationExecutor(
     }
 
     /**
-     * SDK-4474 self-heal builder. Returns an [UpdateSubscriptionOperation] iff the cached
-     * push subscription model resolves to enabled-and-opted-in but the server response
-     * recorded the same id as disabled. Returns null otherwise (no divergence — no-op).
+     * Push self-heal builder. Returns an [UpdateSubscriptionOperation] iff the cached push
+     * subscription model resolves to enabled-and-opted-in but the server response recorded
+     * the same id as disabled. Returns null otherwise (no divergence — no-op).
      *
      * Caller has already verified that [serverSubscription.id] matches the cached
      * `pushSubscriptionId`, so this method only inspects the local model + server flags.
@@ -226,7 +225,7 @@ internal class RefreshUserOperationExecutor(
                 "RefreshUserOperationExecutor: push subscription $pushSubscriptionId diverged from server " +
                     "(server enabled=${serverSubscription.enabled} notificationTypes=${serverSubscription.notificationTypes}; " +
                     "local opted-in and SUBSCRIBED). Enqueuing follow-up update-subscription op to re-assert local " +
-                    "truth via PATCH /subscriptions/{id}. See SDK-4388 / SDK-4474.",
+                    "truth via PATCH /subscriptions/{id}.",
             )
             UpdateSubscriptionOperation(
                 op.appId,

OneSignalSDK/onesignal/core/src/test/java/com/onesignal/user/internal/operations/RefreshUserOperationExecutorTests.kt

@@ -64,7 +64,7 @@ class RefreshUserOperationExecutorTests : FunSpec({
                 mapOf(IdentityConstants.ONESIGNAL_ID to remoteOneSignalId, "aliasLabel1" to "aliasValue1"),
                 PropertiesObject(country = remoteCountry, language = remoteLanguage, timezoneId = remoteTimeZone, tags = remoteTags),
                 listOf(
-                    // notificationTypes = 1 keeps server-side view healthy so the SDK-4474
+                    // notificationTypes = 1 keeps server-side view healthy so the push
                     // self-heal divergence check is a no-op for this happy-path test.
                     SubscriptionObject(existingSubscriptionId1, SubscriptionObjectType.ANDROID_PUSH, enabled = true, notificationTypes = 1, token = "on-backend-push-token"),
                     SubscriptionObject(remoteSubscriptionId1, SubscriptionObjectType.ANDROID_PUSH, enabled = true, notificationTypes = 1, token = "pushToken2"),
@@ -326,10 +326,11 @@ class RefreshUserOperationExecutorTests : FunSpec({
         }
     }
 
-    // SDK-4474 self-heal divergence detection. Verifies that when the device-cached push
+    // Push self-heal divergence detection. Verifies that when the device-cached push
     // subscription resolves to enabled-and-opted-in but the GET /users response returns the
-    // same subscription as disabled (the SDK-4388 stuck state), RefreshUserOperationExecutor
-    // emits a follow-up UpdateSubscriptionOperation to re-assert local truth via PATCH.
+    // same subscription as disabled (the "Never Subscribed" stuck state),
+    // RefreshUserOperationExecutor emits a follow-up UpdateSubscriptionOperation to re-assert
+    // local truth via PATCH.
     fun buildSelfHealHarness(
         serverPushEnabled: Boolean,
         serverNotificationTypes: Int?,
@@ -396,8 +397,8 @@ class RefreshUserOperationExecutorTests : FunSpec({
         return Triple(executor, cachedPushSubscriptionModel, mockUserBackendService)
     }
 
-    test("SDK-4474 self-heal: enqueues follow-up update-subscription op when server is stuck-disabled but local is enabled") {
-        // Given: server view says push is disabled (the SDK-4388 stuck state), local view says enabled
+    test("push self-heal: enqueues follow-up update-subscription op when server is stuck-disabled but local is enabled") {
+        // Given: server view says push is disabled (the stuck state), local view says enabled
         val (executor, _, _) =
             buildSelfHealHarness(
                 serverPushEnabled = false,
@@ -434,7 +435,7 @@ class RefreshUserOperationExecutorTests : FunSpec({
         }
     }
 
-    test("SDK-4474 self-heal: does NOT enqueue follow-up op when server matches local (healthy push subscription)") {
+    test("push self-heal: does NOT enqueue follow-up op when server matches local (healthy push subscription)") {
         // Given: server already enabled and notificationTypes=1, local also enabled
         val (executor, _, _) =
             buildSelfHealHarness(
@@ -453,7 +454,7 @@ class RefreshUserOperationExecutorTests : FunSpec({
         response.operations shouldBe null
     }
 
-    test("SDK-4474 self-heal: does NOT enqueue follow-up op when local is opted out (UNSUBSCRIBE is intentional)") {
+    test("push self-heal: does NOT enqueue follow-up op when local is opted out (UNSUBSCRIBE is intentional)") {
         // Given: server is disabled, but so is local (user explicitly opted out)
         val (executor, _, _) =
             buildSelfHealHarness(
@@ -472,7 +473,7 @@ class RefreshUserOperationExecutorTests : FunSpec({
         response.operations shouldBe null
     }
 
-    test("SDK-4474 self-heal: does NOT enqueue follow-up op when local has NO_PERMISSION (OS-level disable is real)") {
+    test("push self-heal: does NOT enqueue follow-up op when local has NO_PERMISSION (OS-level disable is real)") {
         // Given: server disabled, local also has no notification permission
         val (executor, _, _) =
             buildSelfHealHarness(