fix: [SDK-5616] make the composite login types internal

OneSignalUserProfile, LoginData, and OneSignalResult are only used by the profile login, which is no longer public.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: AR Abdul Azeez

OneSignalSDK/onesignal/core/api/core.api

@@ -89,15 +89,6 @@ public final class com/onesignal/InitData : com/onesignal/OneSignalResultData {
 	public fun toString ()Ljava/lang/String;
 }
 
-public final class com/onesignal/LoginData : com/onesignal/OneSignalResultData {
-	public final fun getEmailSubscriptionId ()Ljava/lang/String;
-	public final fun getExternalId ()Ljava/lang/String;
-	public final fun getOnesignalId ()Ljava/lang/String;
-	public final fun getSmsSubscriptionId ()Ljava/lang/String;
-	public fun toMap ()Ljava/util/Map;
-	public fun toString ()Ljava/lang/String;
-}
-
 public final class com/onesignal/LogoutData : com/onesignal/OneSignalResultData {
 	public fun toMap ()Ljava/util/Map;
 	public fun toString ()Ljava/lang/String;
@@ -169,45 +160,10 @@ public final class com/onesignal/OneSignalError$Detail {
 	public fun toString ()Ljava/lang/String;
 }
 
-public final class com/onesignal/OneSignalResult {
-	public synthetic fun <init> (Lcom/onesignal/OneSignalResultData;Lcom/onesignal/OneSignalError;Lkotlin/jvm/internal/DefaultConstructorMarker;)V
-	public final fun getData ()Lcom/onesignal/OneSignalResultData;
-	public final fun getError ()Lcom/onesignal/OneSignalError;
-	public final fun isSuccess ()Z
-	public final fun toMap ()Ljava/util/Map;
-	public fun toString ()Ljava/lang/String;
-}
-
 public abstract interface class com/onesignal/OneSignalResultData {
 	public abstract fun toMap ()Ljava/util/Map;
 }
 
-public final class com/onesignal/OneSignalUserProfile {
-	public static final field Companion Lcom/onesignal/OneSignalUserProfile$Companion;
-	public synthetic fun <init> (Lcom/onesignal/OneSignalUserProfile$Builder;Lkotlin/jvm/internal/DefaultConstructorMarker;)V
-	public static final fun builder ()Lcom/onesignal/OneSignalUserProfile$Builder;
-	public fun equals (Ljava/lang/Object;)Z
-	public final fun getAliases ()Ljava/util/Map;
-	public final fun getEmail ()Ljava/lang/String;
-	public final fun getSms ()Ljava/lang/String;
-	public final fun getTags ()Ljava/util/Map;
-	public fun hashCode ()I
-	public fun toString ()Ljava/lang/String;
-}
-
-public final class com/onesignal/OneSignalUserProfile$Builder {
-	public fun <init> ()V
-	public final fun build ()Lcom/onesignal/OneSignalUserProfile;
-	public final fun setAliases (Ljava/util/Map;)Lcom/onesignal/OneSignalUserProfile$Builder;
-	public final fun setEmail (Ljava/lang/String;)Lcom/onesignal/OneSignalUserProfile$Builder;
-	public final fun setSms (Ljava/lang/String;)Lcom/onesignal/OneSignalUserProfile$Builder;
-	public final fun setTags (Ljava/util/Map;)Lcom/onesignal/OneSignalUserProfile$Builder;
-}
-
-public final class com/onesignal/OneSignalUserProfile$Companion {
-	public final fun builder ()Lcom/onesignal/OneSignalUserProfile$Builder;
-}
-
 public final class com/onesignal/SyncJobService : android/app/job/JobService {
 	public fun <init> ()V
 	public fun onStartJob (Landroid/app/job/JobParameters;)Z

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/OneSignalResult.kt

@@ -4,34 +4,10 @@ import com.onesignal.common.toMap
 import org.json.JSONObject
 
 /**
- * The outcome of an asynchronous OneSignal call: either a [data] payload or an [error], never both
- * and never neither.
- *
- * Every SDK returns the same envelope, so a wrapper can handle results uniformly regardless of the
- * platform underneath it:
- *
- * ```json
- * { "success": true,  "data": { }, "error": null }
- * { "success": false, "data": null, "error": [ { "code": "STORAGE_LOCKED", ... } ] }
- * ```
- *
- * The presence of [error] is what defines the outcome; [isSuccess] and the wire-level `success`
- * flag are both derived from it, so the two can never disagree.
- *
- * Construct through [success], [failure], or [fromMap]. The internal companion is still reachable
- * from Java, so the either/or invariant is enforced in `init`, not by visibility.
- *
- * From Kotlin, inside a coroutine:
- * ```kotlin
- * if (result.isSuccess) println(result.data) else println(result.error?.first?.code)
- * ```
- *
- * From Java the generated accessors read naturally:
- * ```java
- * if (result.isSuccess()) { result.getData(); } else { result.getError().getDetails(); }
- * ```
+ * Either [data] or [error], never both. [isSuccess] follows [error].
+ * Build with [success], [failure], or [fromMap]. Java can reach the companion, so `init` enforces either/or.
  */
-class OneSignalResult<T : OneSignalResultData> private constructor(
+internal class OneSignalResult<T : OneSignalResultData> private constructor(
     /** The payload on success, `null` on failure. */
     val data: T?,
     /** The failure detail on failure, `null` on success. */

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/OneSignalResultData.kt

@@ -23,8 +23,8 @@ sealed interface OneSignalResultData {
  * Field names here are also the wire keys, so they are additive-only: never renamed, never removed.
  */
 
-/** The payload returned by a successful login. */
-class LoginData internal constructor(
+/** Payload from the internal profile login. */
+internal class LoginData internal constructor(
     /** The OneSignal ID the external ID is now associated with. */
     val onesignalId: String,
     /** The external ID that was logged in. */

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/OneSignalUserProfile.kt

@@ -5,11 +5,11 @@ import com.onesignal.common.hasNullByte
 import java.util.Collections
 
 /**
- * Profile applied at login. Email/SMS create or transfer that address onto this user; they do not replace other subscriptions.
- * Build one with [builder]. There is no public constructor, so adding a field never breaks compiled callers.
+ * Email, SMS, tags, and aliases for the internal profile login.
+ * Email and SMS create or transfer that address. They do not replace other subscriptions.
  */
 // Takes the Builder so the synthetic accessor's JVM signature stays fixed as fields are added.
-class OneSignalUserProfile private constructor(builder: Builder) {
+internal class OneSignalUserProfile private constructor(builder: Builder) {
     val email: String? = builder.email?.takeIf { it.isNotBlank() }
     val sms: String? = builder.sms?.takeIf { it.isNotBlank() }
     val tags: Map<String, String> = Collections.unmodifiableMap(builder.tags.toMap())
@@ -65,16 +65,16 @@ class OneSignalUserProfile private constructor(builder: Builder) {
         internal var aliases: Map<String, String> = emptyMap()
             private set
 
-        /** Email to create or transfer onto this user at login. */
+        /** Email to create or transfer onto this user. */
         fun setEmail(email: String?) = apply { this.email = email }
 
-        /** SMS to create or transfer onto this user at login. Must be E.164 (e.g. +14155552671). */
+        /** SMS to create or transfer onto this user. Must be E.164 (e.g. +14155552671). */
         fun setSms(sms: String?) = apply { this.sms = sms }
 
-        /** Sets tags to apply at login. The map is copied. */
+        /** Tags to include. The map is copied. */
         fun setTags(tags: Map<String, String>) = apply { this.tags = tags.toMap() }
 
-        /** Sets aliases to apply at login. The map is copied. */
+        /** Aliases to include. The map is copied. */
         fun setAliases(aliases: Map<String, String>) = apply { this.aliases = aliases.toMap() }
 
         /** Builds the profile. */

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/user/internal/operations/LoginUserOperation.kt

@@ -79,7 +79,7 @@ class LoginUserOperation() : Operation(LoginUserOperationExecutor.LOGIN_USER) {
     override val canStartExecute: Boolean get() = existingOnesignalId == null || !IDManager.isLocalId(existingOnesignalId!!)
     override val applyToRecordId: String get() = existingOnesignalId ?: onesignalId
 
-    constructor(
+    internal constructor(
         appId: String,
         onesignalId: String,
         externalId: String?,