From 4bd862469736d6c58c1cd5cb6642d64ebb32f3af Mon Sep 17 00:00:00 2001
From: Ray Ryan <ray@squareup.com>
Date: Thu, 15 May 2025 12:02:13 -0700
Subject: [PATCH] Tweak to default `onNavigate()`

Log `keyFor(it)` by default instead of `it.toString()`. Latter is almost guaranteed to be noisy and inefficient.
---
 .../com/squareup/workflow1/ui/navigation/NavigationMonitor.kt   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/workflow-ui/core-common/src/main/java/com/squareup/workflow1/ui/navigation/NavigationMonitor.kt b/workflow-ui/core-common/src/main/java/com/squareup/workflow1/ui/navigation/NavigationMonitor.kt
index 2701f85aec..f77cb2ce7d 100644
--- a/workflow-ui/core-common/src/main/java/com/squareup/workflow1/ui/navigation/NavigationMonitor.kt
+++ b/workflow-ui/core-common/src/main/java/com/squareup/workflow1/ui/navigation/NavigationMonitor.kt
@@ -19,7 +19,7 @@ import kotlinx.coroutines.flow.onEach
  */
 public class NavigationMonitor(
   skipFirstScreen: Boolean = false,
-  private val onNavigate: (Any) -> Unit = { println(it) }
+  private val onNavigate: (Any) -> Unit = { println(Compatible.keyFor(it)) }
 ) {
   @Volatile
   private var lastKey: String? = if (skipFirstScreen) null else ""