clean up imports and comments to ready PR

Author: wenli-cai

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

@@ -1,29 +1,14 @@
 package com.squareup.workflow1.traceviewer
 
-import androidx.compose.foundation.background
 import androidx.compose.foundation.layout.Box
-import androidx.compose.foundation.layout.fillMaxSize
-import androidx.compose.foundation.layout.padding
-import androidx.compose.foundation.layout.size
-import androidx.compose.foundation.shape.CircleShape
-import androidx.compose.material.Button
-import androidx.compose.material.ButtonDefaults.buttonColors
 import androidx.compose.material.Text
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
-import androidx.compose.ui.Alignment
-import androidx.compose.ui.graphics.Color
-import androidx.compose.ui.unit.dp
-import androidx.compose.ui.Modifier
-import androidx.compose.ui.unit.sp
 import io.github.vinceglb.filekit.PlatformFile
-import io.github.vinceglb.filekit.dialogs.FileKitType
-import io.github.vinceglb.filekit.dialogs.compose.rememberFilePickerLauncher
 import io.github.vinceglb.filekit.readString
 
-
 @Composable
 public fun App() {
   Box {

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

@@ -45,7 +45,8 @@ public fun SandboxBackground(
             event.changes.forEach { it.consume() }
           }
 
-          // panning: this tracks multiple events within one gesture to see what the user is doing, then calculates the offset and pans the screen accordingly
+          // panning: this tracks multiple events within one gesture to see what the user is doing,
+          // then calculates the offset and pans the screen accordingly
           val drag = event.changes.firstOrNull()
           if (drag != null && drag.pressed) {
             var prev = drag.position

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

@@ -1,22 +1,17 @@
 package com.squareup.workflow1.traceviewer
 
-import androidx.compose.foundation.background
 import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.padding
-import androidx.compose.foundation.layout.size
 import androidx.compose.foundation.shape.CircleShape
 import androidx.compose.material.Button
 import androidx.compose.material.ButtonDefaults.buttonColors
 import androidx.compose.material.Text
 import androidx.compose.runtime.Composable
-import androidx.compose.runtime.LaunchedEffect
-import androidx.compose.runtime.mutableStateOf
-import androidx.compose.runtime.remember
 import androidx.compose.ui.Alignment
+import androidx.compose.ui.Modifier
 import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.unit.dp
-import androidx.compose.ui.Modifier
 import androidx.compose.ui.unit.sp
 import io.github.vinceglb.filekit.PlatformFile
 import io.github.vinceglb.filekit.dialogs.FileKitType

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

@@ -7,8 +7,9 @@ import java.io.IOException
 
 /**
  * Parses a JSON string into [WorkflowNode] with Moshi adapters
- * All the caught exceptions should be handled by the caller, and appropriate UI feedback should be provided to user
  *
+ * All the caught exceptions should be handled by the caller, and appropriate UI feedback should be
+ * provided to user
  */
 public fun FetchRoot(json: String): WorkflowNode? {
   return try{

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

@@ -6,7 +6,6 @@ import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.Row
-import androidx.compose.foundation.layout.Spacer
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.padding
 import androidx.compose.material.Text
@@ -19,28 +18,17 @@ import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.unit.dp
 
 /**
- * Since the logic of Workflow is hierarchical (where each workflow may have parent workflows and/or children workflows,
- * a tree structure is most appropriate for representing the data rather than using flat data structures like an array.
+ * Since the logic of Workflow is hierarchical (where each workflow may have parent workflows and/or
+ * children workflows, a tree structure is most appropriate for representing the data rather than
+ * using flat data structures like an array.
  *
  * TBD what more metadata should be involved with each node, e.g. (props, states, # of render passes)
  */
 public data class WorkflowNode (
   val id: String,
   val name: String,
   val children: List<WorkflowNode>
-) {
-  // fun findParentForId(id: String): WorkflowNode? {
-  //   if (this.id == id) {
-  //     return null // This is the root node, so it has no parent
-  //   }
-  //   if (children.any { it.id == id }) {
-  //     return this
-  //   }
-  //   return children.firstNotNullOfOrNull {
-  //     it.findParentForId(id)
-  //   }
-  // }
-}
+)
 
 /**
  * Since the workflow nodes present a tree structure, we utilize a recursive function to draw the tree
@@ -73,8 +61,7 @@ public fun DrawWorkflowTree(
 }
 
 /**
- * Basic data, for now.
- * These can be designed to be clickable and be expanded to show more information.
+ * A basic box that represents a workflow node
  */
 @Composable
 private fun drawNode(