FEAT: Add version of withSentryObservableEffect that has better inter…

sentry-compose/src/androidMain/kotlin/io/sentry/compose/SentryNavigationIntegration.kt

@@ -48,30 +48,23 @@ internal class SentryLifecycleObserver(
  * A [DisposableEffect] that captures a [Breadcrumb] and starts an [ITransaction] and sends
  * them to Sentry for every navigation event when being attached to the respective [NavHostController].
  *
- * @param enableNavigationBreadcrumbs Whether the integration should capture breadcrumbs for
- * navigation events.
- * @param enableNavigationTracing Whether the integration should start a new [ITransaction]
- * with [SentryOptions.idleTimeout] for navigation events.
+ * @param navListener An instance of a [SentryNavigationListener] that is shared with other sentry integrations, like
+ * the fragment navigation integration. 
  */
 @Composable
 @NonRestartableComposable
-public fun NavHostController.withSentryObservableEffect(
-    enableNavigationBreadcrumbs: Boolean = true,
-    enableNavigationTracing: Boolean = true
+fun NavHostController.withSentryObservableEffect(
+    navListener: SentryNavigationListener
 ): NavHostController {
-    val enableBreadcrumbsSnapshot by rememberUpdatedState(enableNavigationBreadcrumbs)
-    val enableTracingSnapshot by rememberUpdatedState(enableNavigationTracing)
+
+    val navListenerSnapshot by rememberUpdatedState(navListener)
 
     // As described in https://developer.android.com/codelabs/jetpack-compose-advanced-state-side-effects#6
     val lifecycle = LocalLifecycleOwner.current.lifecycle
     DisposableEffect(lifecycle, this) {
         val observer = SentryLifecycleObserver(
             this@withSentryObservableEffect,
-            navListener = SentryNavigationListener(
-                enableNavigationBreadcrumbs = enableBreadcrumbsSnapshot,
-                enableNavigationTracing = enableTracingSnapshot,
-                traceOriginAppendix = TRACE_ORIGIN_APPENDIX
-            )
+            navListener = navListenerSnapshot
         )
 
         lifecycle.addObserver(observer)
@@ -84,6 +77,32 @@ public fun NavHostController.withSentryObservableEffect(
     return this
 }
 
+/**
+ * A [DisposableEffect] that captures a [Breadcrumb] and starts an [ITransaction] and sends
+ * them to Sentry for every navigation event when being attached to the respective [NavHostController].
+ * This version of withSentryObservableEffect should be used if you are working purely with Compose. 
+ *
+ * @param enableNavigationBreadcrumbs Whether the integration should capture breadcrumbs for
+ * navigation events.
+ * @param enableNavigationTracing Whether the integration should start a new [ITransaction]
+ * with [SentryOptions.idleTimeout] for navigation events.
+ */
+@Composable
+@NonRestartableComposable
+fun NavHostController.withSentryObservableEffect(
+    enableNavigationBreadcrumbs: Boolean = true,
+    enableNavigationTracing: Boolean = true
+): NavHostController {
+    val enableBreadcrumbsSnapshot by rememberUpdatedState(enableNavigationBreadcrumbs)
+    val enableTracingSnapshot by rememberUpdatedState(enableNavigationTracing)
+
+    return withSentryObservableEffect(navListener = SentryNavigationListener(
+        enableNavigationBreadcrumbs = enableBreadcrumbsSnapshot,
+        enableNavigationTracing = enableTracingSnapshot,
+        traceOriginAppendix = TRACE_ORIGIN_APPENDIX
+    ))
+}
+
 /**
  * A [DisposableEffect] that captures a [Breadcrumb] and starts an [ITransaction] and sends
  * them to Sentry for every navigation event when being attached to the respective [NavHostController].