Remove unnecessary frame tracking in main composable

Author: wenli-cai

workflow-trace-viewer/src/jvmMain/kotlin/com/squareup/workflow1/traceviewer/App.kt

@@ -33,7 +33,7 @@ internal fun App(
   modifier: Modifier = Modifier
 ) {
   var selectedNode by remember { mutableStateOf<NodeUpdate?>(null) }
-  val workflowFrames = remember { mutableStateListOf<Node>() }
+  var frameSize by remember { mutableIntStateOf(0) }
   var frameIndex by remember { mutableIntStateOf(0) }
   val sandboxState = remember { SandboxState() }
 
@@ -62,7 +62,7 @@ internal fun App(
       selectedTraceFile = null
       selectedNode = null
       frameIndex = 0
-      workflowFrames.clear()
+      frameSize = 0
     }
 
     // Main content
@@ -77,15 +77,15 @@ internal fun App(
         RenderTrace(
           traceSource = traceMode,
           frameInd = frameIndex,
-          onFileParse = { workflowFrames.addAll(it) },
+          onFileParse = { frameSize += it },
           onNodeSelect = { selectedNode = it },
           onNewFrame = { frameIndex += 1 }
         )
       }
     }
 
     FrameSelectTab(
-      frames = workflowFrames,
+      size = frameSize,
       currentIndex = frameIndex,
       onIndexChange = { frameIndex = it },
       modifier = Modifier.align(Alignment.TopCenter)

workflow-trace-viewer/src/jvmMain/kotlin/com/squareup/workflow1/traceviewer/ui/FrameSelectTab.kt

@@ -26,7 +26,7 @@ import kotlinx.coroutines.launch
  */
 @Composable
 internal fun FrameSelectTab(
-  frames: List<Node>,
+  size: Int,
   currentIndex: Int,
   onIndexChange: (Int) -> Unit,
   modifier: Modifier = Modifier
@@ -36,7 +36,7 @@ internal fun FrameSelectTab(
     if (currentIndex < 0) return@LaunchedEffect
     lazyListState.animateScrollToItem(currentIndex)
   }
-  
+
   Surface(
     modifier = modifier,
     color = Color.White,
@@ -61,7 +61,7 @@ internal fun FrameSelectTab(
           }
         },
     ) {
-      items(frames.size) { index ->
+      items(size) { index ->
         Text(
           text = "Frame ${index + 1}",
           color = if (index == currentIndex) Color.Black else Color.LightGray,

workflow-trace-viewer/src/jvmMain/kotlin/com/squareup/workflow1/traceviewer/ui/WorkflowTree.kt

@@ -47,7 +47,7 @@ import kotlinx.coroutines.withContext
 internal fun RenderTrace(
   traceSource: TraceMode,
   frameInd: Int,
-  onFileParse: (List<Node>) -> Unit,
+  onFileParse: (Int) -> Unit,
   onNodeSelect: (NodeUpdate) -> Unit,
   onNewFrame: () -> Unit,
   modifier: Modifier = Modifier
@@ -65,7 +65,7 @@ internal fun RenderTrace(
     fullTree.addAll(tree)
     affectedNodes.addAll(affected)
     isLoading = false
-    onFileParse(frame)
+    onFileParse(frame.size)
   }
 
   // Handles the result of parsing a trace, either from file or live. Live mode includes callback