chore: remove cacheOnly concept, match iOS behavior for ON_APP_START

ON_APP_START auto-captured location now goes through the normal path
(cache + sync filter + track event if filter passes) instead of
skipping the track event. This aligns with iOS SDK behavior where
all location updates are treated uniformly regardless of source.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: Shahroz16

location/src/main/kotlin/io/customer/location/LocationOrchestrator.kt

@@ -17,7 +17,7 @@ internal class LocationOrchestrator(
     private val locationProvider: LocationProvider
 ) {
 
-    suspend fun requestLocationUpdate(cacheOnly: Boolean = false) {
+    suspend fun requestLocationUpdate() {
         if (!config.isEnabled) {
             logger.debug("Location tracking is disabled, ignoring requestLocationUpdate.")
             return
@@ -34,7 +34,7 @@ internal class LocationOrchestrator(
                 granularity = LocationGranularity.DEFAULT
             )
             logger.debug("Tracking location: lat=${snapshot.latitude}, lng=${snapshot.longitude}")
-            locationTracker.onLocationReceived(snapshot.latitude, snapshot.longitude, cacheOnly)
+            locationTracker.onLocationReceived(snapshot.latitude, snapshot.longitude)
         } catch (e: CancellationException) {
             logger.debug("Location request was cancelled.")
             throw e

location/src/main/kotlin/io/customer/location/LocationTracker.kt

@@ -74,23 +74,16 @@ internal class LocationTracker(
     }
 
     /**
-     * Processes an incoming location: caches in memory, persists coordinates,
-     * and optionally attempts to send a location track event.
-     *
-     * @param cacheOnly when true, updates the in-memory cache and persists
-     *   coordinates but does NOT send a "Location Update" track event.
-     *   Used by ON_APP_START auto-requests to refresh cached location for
-     *   identify context enrichment without generating a discrete event.
+     * Processes an incoming location: caches in memory, persists
+     * coordinates, and attempts to send a location track event.
      */
-    fun onLocationReceived(latitude: Double, longitude: Double, cacheOnly: Boolean = false) {
-        logger.debug("Location update received: lat=$latitude, lng=$longitude, cacheOnly=$cacheOnly")
+    fun onLocationReceived(latitude: Double, longitude: Double) {
+        logger.debug("Location update received: lat=$latitude, lng=$longitude")
 
         lastLocation = LocationCoordinates(latitude = latitude, longitude = longitude)
         locationPreferenceStore.saveCachedLocation(latitude, longitude)
 
-        if (!cacheOnly) {
-            trySendLocationTrack(latitude, longitude)
-        }
+        trySendLocationTrack(latitude, longitude)
     }
 
     /**

location/src/main/kotlin/io/customer/location/ModuleLocation.kt

@@ -117,12 +117,12 @@ class ModuleLocation @JvmOverloads constructor(
         }
 
         // ON_APP_START: auto-capture location on cold start.
-        // Fire-and-forget — refreshes cached location so subsequent identify() calls
-        // carry fresh data. If permissions are denied or services disabled, orchestrator
-        // silently no-ops.
+        // Fire-and-forget — captures location and sends through the normal path
+        // (cache + sync filter + track event if filter passes).
+        // If permissions are denied or services disabled, orchestrator silently no-ops.
         if (moduleConfig.trackingMode == LocationTrackingMode.ON_APP_START) {
             locationScope.launch {
-                orchestrator.requestLocationUpdate(cacheOnly = true)
+                orchestrator.requestLocationUpdate()
             }
         }
     }

location/src/test/java/io/customer/location/LocationServicesImplTest.kt

@@ -27,7 +27,7 @@ class LocationServicesImplTest {
         val services = LocationServicesImpl(config, logger, tracker, orchestrator, scope)
         services.setLastKnownLocation(37.7749, -122.4194)
 
-        verify(exactly = 0) { tracker.onLocationReceived(any(), any(), any()) }
+        verify(exactly = 0) { tracker.onLocationReceived(any(), any()) }
     }
 
     @Test

location/src/test/java/io/customer/location/LocationTrackerTest.kt

@@ -214,44 +214,6 @@ class LocationTrackerTest {
         verify(exactly = 0) { dataPipeline.track(any(), any()) }
     }
 
-    // -- onLocationReceived with cacheOnly --
-
-    @Test
-    fun givenCacheOnly_expectPersistsToStore() {
-        tracker.onLocationReceived(37.7749, -122.4194, cacheOnly = true)
-
-        verify { store.saveCachedLocation(37.7749, -122.4194) }
-    }
-
-    @Test
-    fun givenCacheOnly_expectUpdatesIdentifyContext() {
-        tracker.onLocationReceived(37.7749, -122.4194, cacheOnly = true)
-
-        val context = tracker.getIdentifyContext()
-        context.shouldNotBeEmpty()
-        context["location_latitude"] shouldBeEqualTo 37.7749
-        context["location_longitude"] shouldBeEqualTo -122.4194
-    }
-
-    @Test
-    fun givenCacheOnly_expectTrackNotCalled() {
-        tracker.onLocationReceived(37.7749, -122.4194, cacheOnly = true)
-
-        verify(exactly = 0) { dataPipeline.track(any(), any()) }
-    }
-
-    @Test
-    fun givenCacheOnlyFalse_expectTrackCalled() {
-        tracker.onLocationReceived(37.7749, -122.4194, cacheOnly = false)
-
-        verify {
-            dataPipeline.track(
-                name = EventNames.LOCATION_UPDATE,
-                properties = mapOf("latitude" to 37.7749, "longitude" to -122.4194)
-            )
-        }
-    }
-
     // -- resetContext (synchronous, called by analytics.reset during clearIdentify) --
 
     @Test