refactor: KitManagerImpl user-attribute forwarding and UserAttributeListener cleanup (#693)

* Add activeKits() helper and unit test

Introduce activeKits() to return non-disabled kit integrations from the
providers map. Add KitManagerImplTest.testActiveKitsExcludesDisabled to
verify disabled kits are omitted.

Made-with: Cursor

* Refactor KitManagerImpl user-attribute and identity forwarding

Use activeKits() for user attribute and identity paths; inline former
setUserAttribute helpers to avoid duplicate isDisabled checks. Flatten
syncUserIdentities with early returns and drop redundant null check on
identities map.

Made-with: Cursor

* UserAttributeListener: expose kit identity; use activeKits in onUserAttributesReceived

Declare getConfiguration() and getName() on UserAttributeListener so callers
can access kit metadata consistently. Route onUserAttributesReceived through
activeKits() and align variable naming with other active-kit paths.

Made-with: Cursor

* Remove FilteredMParticleUser from UserAttributeListener callbacks

Drop the user parameter from KitIntegration.UserAttributeListener methods;
update KitManagerImpl forwarding, instrumented/unit tests, and kit
implementations (Adobe, AppsFlyer, Apptentive, Braze, CleverTap, etc.).

Made-with: Cursor

* Refactor UserAttributeListener forwarding in KitManagerImpl

Add getConfiguration() and getName() to UserAttributeListener so callers
can use the listener type without casting. Introduce userAttributeListeners()
and iterate with variable name listener; keep activeKits() package-visible
for tests.

Made-with: Cursor

* test(appsflyer): align onConsentStateUpdated calls with UserAttributeListener API

Remove third argument and unused FilteredMParticleUser mock setup from unit tests.

Made-with: Cursor

* Fix DataplanBlockingUserTests onRemoveUserAttribute lambda arity

UserAttributeListenerTestKit exposes (String?) -> Unit; a two-parameter
lambda failed Kotlin type inference in compileDebugAndroidTestKotlin.

Made-with: Cursor

* Fix Braze kit tests for two-arg onConsentStateUpdated

KitIntegration.UserAttributeListener now passes only old and new
ConsentState. Drop the removed FilteredMParticleUser argument and
unused mock from AppboyKitTests across braze-38 through braze-41.

Made-with: Cursor

* Fix GA and GA4 kit tests for two-arg onConsentStateUpdated

Align GoogleAnalyticsFirebaseKitTest and GoogleAnalyticsFirebaseGA4KitTest
with KitIntegration: drop the removed FilteredMParticleUser argument and
unused mock from consent tests.

Made-with: Cursor

Author: denischilik

android-kit-base/src/androidTest/kotlin/com/mparticle/kits/DataplanBlockingUserTests.kt

@@ -73,7 +73,7 @@ class DataplanBlockingUserTests : BaseKitOptionsTest() {
             ),
         )
 
-        userAttributeListenerKitKit.onSetUserAttribute = { key, _, _ ->
+        userAttributeListenerKitKit.onSetUserAttribute = { key, _ ->
             assertTrue(allowedAttributes.containsKey(key))
             assertFalse(blockedAttributes.containsKey(key))
         }
@@ -146,7 +146,7 @@ class DataplanBlockingUserTests : BaseKitOptionsTest() {
                 count++
             }
         }
-        userAttributeListenerKitKit.onRemoveUserAttribute = { key, _ ->
+        userAttributeListenerKitKit.onRemoveUserAttribute = { key ->
             assertTrue(allowedAttributes.containsKey(key))
             assertFalse(blockedAttributes.containsKey(key))
             // make sure these are the attributes that are being removed
@@ -221,7 +221,7 @@ class DataplanBlockingUserTests : BaseKitOptionsTest() {
                 count++
             }
         }
-        userAttributeListenerKitKit.onSetUserAttributeList = { key, _, _ ->
+        userAttributeListenerKitKit.onSetUserAttributeList = { key, _ ->
             assertTrue(allowedAttributes.containsKey(key))
             assertFalse(blockedAttributes.containsKey(key))
             count++

android-kit-base/src/androidTest/kotlin/com/mparticle/kits/testkits/ModifyIdentityListenerTestKit.kt

@@ -2,7 +2,6 @@ package com.mparticle.kits.testkits
 
 import com.mparticle.MParticle
 import com.mparticle.consent.ConsentState
-import com.mparticle.kits.FilteredMParticleUser
 import com.mparticle.kits.KitIntegration.LogoutListener
 import com.mparticle.kits.KitIntegration.ModifyIdentityListener
 import com.mparticle.kits.KitIntegration.UserAttributeListener
@@ -30,7 +29,6 @@ open class ModifyIdentityListenerTestKit :
     override fun onSetUserAttributeList(
         attributeKey: String?,
         attributeValueList: List<String>?,
-        user: FilteredMParticleUser?,
     ) {
         setUserAttributeList?.invoke(attributeKey, attributeValueList)
         onAttributeReceived?.invoke(attributeKey, attributeValueList)
@@ -39,7 +37,6 @@ open class ModifyIdentityListenerTestKit :
     override fun onSetAllUserAttributes(
         userAttributes: Map<String, String>,
         userAttributeLists: Map<String, List<String>>,
-        user: FilteredMParticleUser,
     ) {
         setAllUserAttributes?.invoke(userAttributes, userAttributeLists)
         userAttributes.forEach { onAttributeReceived?.invoke(it.key, it.value) }
@@ -61,7 +58,6 @@ open class ModifyIdentityListenerTestKit :
 
     override fun onRemoveUserAttribute(
         key: String,
-        user: FilteredMParticleUser,
     ) {
         removeUserAttributeListener?.invoke(key)
         onAttributeReceived?.invoke(key, null)
@@ -70,7 +66,6 @@ open class ModifyIdentityListenerTestKit :
     override fun onSetUserAttribute(
         key: String?,
         value: Any?,
-        user: FilteredMParticleUser?,
     ) {
         if (key == null || value == null || value !is String) {
             return
@@ -83,20 +78,17 @@ open class ModifyIdentityListenerTestKit :
         key: String?,
         incrementedBy: Number?,
         value: String?,
-        user: FilteredMParticleUser?,
     ) {
     }
 
     override fun onSetUserTag(
         key: String?,
-        user: FilteredMParticleUser?,
     ) {
     }
 
     override fun onConsentStateUpdated(
         oldState: ConsentState?,
         newState: ConsentState?,
-        user: FilteredMParticleUser?,
     ) {
     }
 

android-kit-base/src/androidTest/kotlin/com/mparticle/kits/testkits/UserAttributeListenerTestKit.kt

@@ -1,95 +1,86 @@
 package com.mparticle.kits.testkits
 
 import com.mparticle.consent.ConsentState
-import com.mparticle.kits.FilteredMParticleUser
 import com.mparticle.kits.KitIntegration
 
 open class UserAttributeListenerTestKit :
     ListenerTestKit(),
     KitIntegration.UserAttributeListener {
-    var onIncrementUserAttribute: ((key: String?, incrementedBy: Number, value: String?, user: FilteredMParticleUser?) -> Unit)? =
+    var onIncrementUserAttribute: ((key: String?, incrementedBy: Number, value: String?) -> Unit)? =
         null
-    var onRemoveUserAttribute: ((key: String?, user: FilteredMParticleUser?) -> Unit)? = null
-    var onSetUserAttribute: ((key: String?, value: Any?, user: FilteredMParticleUser?) -> Unit)? =
+    var onRemoveUserAttribute: ((key: String?) -> Unit)? = null
+    var onSetUserAttribute: ((key: String?, value: Any?) -> Unit)? =
         null
-    var onSetUserTag: ((key: String?, user: FilteredMParticleUser?) -> Unit)? = null
-    var onSetUserAttributeList: ((attributeKey: String?, attributeValueList: List<String>?, user: FilteredMParticleUser?) -> Unit)? =
+    var onSetUserTag: ((key: String?) -> Unit)? = null
+    var onSetUserAttributeList: ((attributeKey: String?, attributeValueList: List<String>?) -> Unit)? =
         null
     var onSetAllUserAttributes: (
         (
             userAttributes: Map<String?, String?>?,
             userAttributeLists: Map<String?, List<String?>?>?,
-            user: FilteredMParticleUser?,
         ) -> Unit
     )? = null
     var supportsAttributeLists: (() -> Boolean)? = null
-    var onConsentStateUpdated: ((oldState: ConsentState?, newState: ConsentState?, user: FilteredMParticleUser?) -> Unit)? =
+    var onConsentStateUpdated: ((oldState: ConsentState?, newState: ConsentState?) -> Unit)? =
         null
 
     override fun onConsentStateUpdated(
         oldState: ConsentState?,
         newState: ConsentState?,
-        user: FilteredMParticleUser?,
     ) {
-        onConsentStateUpdated?.invoke(oldState, newState, user)
+        onConsentStateUpdated?.invoke(oldState, newState)
     }
 
     override fun onSetAllUserAttributes(
         userAttributes: Map<String?, String?>?,
         userAttributeLists: Map<String?, List<String>>?,
-        user: FilteredMParticleUser?,
     ) {
-        onSetAllUserAttributes?.invoke(userAttributes, userAttributeLists, user)
+        onSetAllUserAttributes?.invoke(userAttributes, userAttributeLists)
         userAttributes?.forEach { onAttributeReceived?.invoke(it.key, it.value) }
     }
 
     override fun onSetUserAttribute(
         key: String?,
         value: Any?,
-        user: FilteredMParticleUser?,
     ) {
-        onSetUserAttribute?.invoke(key, value, user)
+        onSetUserAttribute?.invoke(key, value)
         onAttributeReceived?.invoke(key, value)
-        onUserReceived?.invoke(user)
+        onUserReceived?.invoke(null)
     }
 
     override fun onSetUserTag(
         key: String?,
-        user: FilteredMParticleUser?,
     ) {
-        onSetUserTag?.invoke(key, user)
+        onSetUserTag?.invoke(key)
         onAttributeReceived?.invoke(key, null)
-        onUserReceived?.invoke(user)
+        onUserReceived?.invoke(null)
     }
 
     override fun onIncrementUserAttribute(
         key: String?,
         incrementedBy: Number,
         value: String?,
-        user: FilteredMParticleUser?,
     ) {
-        onIncrementUserAttribute?.invoke(key, incrementedBy, value, user)
+        onIncrementUserAttribute?.invoke(key, incrementedBy, value)
         onAttributeReceived?.invoke(key, value)
-        onUserReceived?.invoke(user)
+        onUserReceived?.invoke(null)
     }
 
     override fun onSetUserAttributeList(
         attributeKey: String?,
         attributeValueList: List<String>?,
-        user: FilteredMParticleUser?,
     ) {
-        onSetUserAttributeList?.invoke(attributeKey, attributeValueList, user)
+        onSetUserAttributeList?.invoke(attributeKey, attributeValueList)
         onAttributeReceived?.invoke(attributeKey, attributeValueList)
-        onUserReceived?.invoke(user)
+        onUserReceived?.invoke(null)
     }
 
     override fun onRemoveUserAttribute(
         key: String?,
-        user: FilteredMParticleUser?,
     ) {
-        onRemoveUserAttribute?.invoke(key, user)
+        onRemoveUserAttribute?.invoke(key)
         onAttributeReceived?.invoke(key, null)
-        onUserReceived?.invoke(user)
+        onUserReceived?.invoke(null)
     }
 
     override fun supportsAttributeLists() = supportsAttributeLists?.invoke() ?: true

android-kit-base/src/main/java/com/mparticle/kits/KitIntegration.java

@@ -539,50 +539,48 @@ public interface UserAttributeListener {
         /**
          * Called when a user attribute is removed for the current user.
          *
-         * @param key  attribute key
-         * @param user filtered user context for this kit
+         * @param key attribute key
          */
-        void onRemoveUserAttribute(String key, FilteredMParticleUser user);
+        void onRemoveUserAttribute(String key);
 
         /**
          * Called when a scalar user attribute is set for the current user.
          *
          * @param key   attribute key
          * @param value attribute value (may be non-String for some call paths)
-         * @param user  filtered user context for this kit
          */
-        void onSetUserAttribute(String key, Object value, FilteredMParticleUser user);
+        void onSetUserAttribute(String key, Object value);
 
         /**
          * Called when a list-valued user attribute is set and {@link #supportsAttributeLists()} returns true.
          *
          * @param attributeKey       attribute key (may be null)
          * @param attributeValueList attribute values (may be null)
-         * @param user               filtered user context for this kit (may be null)
          */
         void onSetUserAttributeList(
                 @Nullable String attributeKey,
-                @Nullable List<String> attributeValueList,
-                @Nullable FilteredMParticleUser user);
+                @Nullable List<String> attributeValueList);
 
         /**
          * Called when the full set of user attributes is synchronized for the current user.
          *
          * @param userAttributes     scalar user attributes
          * @param userAttributeLists list-valued user attributes when {@link #supportsAttributeLists()} is true;
          *                           otherwise list values may be merged into scalars by the framework
-         * @param user               filtered user context for this kit
          */
         void onSetAllUserAttributes(
                 Map<String, String> userAttributes,
-                Map<String, List<String>> userAttributeLists,
-                FilteredMParticleUser user);
+                Map<String, List<String>> userAttributeLists);
 
-        void onIncrementUserAttribute(String key, Number incrementedBy, String value, FilteredMParticleUser user);
+        void onIncrementUserAttribute(String key, Number incrementedBy, String value);
 
-        void onSetUserTag(String key, FilteredMParticleUser user);
+        void onSetUserTag(String key);
 
-        void onConsentStateUpdated(ConsentState oldState, ConsentState newState, FilteredMParticleUser user);
+        void onConsentStateUpdated(ConsentState oldState, ConsentState newState);
+
+        KitConfiguration getConfiguration();
+
+        String getName();
     }
 
     public interface BatchListener {