Clean up compose violations

Author: wenli-cai

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

@@ -71,7 +71,9 @@ internal fun App(
     ) {
       // if there is not a file selected and trace mode is live, then don't render anything.
       val readyForFileTrace = traceMode is TraceMode.File && selectedTraceFile != null
-      if (readyForFileTrace || traceMode is TraceMode.Live) {
+      val readyForLiveTrace = traceMode is TraceMode.Live
+
+      if (readyForFileTrace || readyForLiveTrace) {
         RenderTrace(
           traceSource = traceMode,
           frameInd = frameIndex,

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

@@ -28,7 +28,6 @@ import com.squareup.workflow1.traceviewer.util.ParseResult
 import com.squareup.workflow1.traceviewer.util.createMoshiAdapter
 import com.squareup.workflow1.traceviewer.util.parseFileTrace
 import com.squareup.workflow1.traceviewer.util.parseLiveTrace
-import io.github.vinceglb.filekit.dialogs.FileKitMode.Single.parseResult
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.withContext
 
@@ -175,7 +174,6 @@ private fun DrawNode(
     modifier = Modifier
       .background(if (isAffected) Color.Green else Color.Transparent)
       .clickable {
-        // Selecting a node will bubble back up to the main view to handle the selection
         onNodeSelect(node, previousNode)
       }
       .padding(10.dp)

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

@@ -48,7 +48,8 @@ internal suspend fun parseFileTrace(
   return ParseResult.Success(
     trace = parsedFrames,
     trees = frameTrees,
-    affectedNodes = parsedRenderPasses)
+    affectedNodes = parsedRenderPasses
+  )
 }
 
 /**