Fix ktlint issues

Author: wenli-cai

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

@@ -6,29 +6,32 @@ import androidx.compose.runtime.Composable
 import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
+import androidx.compose.ui.Modifier
 import io.github.vinceglb.filekit.PlatformFile
 import io.github.vinceglb.filekit.readString
 
 @Composable
-public fun App() {
+public fun App(
+  modifier: Modifier = Modifier
+) {
   Box {
-    val selectedFile = remember { mutableStateOf<PlatformFile?>(null)}
+    val selectedFile = remember { mutableStateOf<PlatformFile?>(null) }
 
-    if (selectedFile.value != null){
+    if (selectedFile.value != null) {
       SandboxBackground { WorkflowContent(selectedFile.value) }
     }
 
-    UploadFile{ selectedFile.value = it}
+    UploadFile(onFileSelect = { selectedFile.value = it })
   }
 }
 
 @Composable
 private fun WorkflowContent(file: PlatformFile?) {
   val jsonString = remember { mutableStateOf<String?>(null) }
-  LaunchedEffect(file){
+  LaunchedEffect(file) {
     jsonString.value = file?.readString()
   }
-  val root = jsonString.value?.let { FetchRoot(it) }
+  val root = jsonString.value?.let { fetchRoot(it) }
 
   if (root != null) {
     DrawWorkflowTree(root)

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

@@ -6,6 +6,7 @@ import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.wrapContentSize
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.getValue
+import androidx.compose.runtime.mutableFloatStateOf
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
 import androidx.compose.runtime.setValue
@@ -25,15 +26,17 @@ import androidx.compose.ui.input.pointer.pointerInput
  */
 @Composable
 public fun SandboxBackground(
-  content: @Composable () -> Unit
+  modifier: Modifier = Modifier,
+  content: @Composable () -> Unit,
 ) {
-  var scale by remember { mutableStateOf(1f) }
+  var scale by remember { mutableFloatStateOf(1f) }
   var offset by remember { mutableStateOf(Offset.Zero) }
 
-  Box (
-    modifier = Modifier
+  Box(
+    modifier
       .fillMaxSize()
-      .pointerInput(Unit) { // this allows for user's panning to view different parts of content
+      .pointerInput(Unit) {
+        // this allows for user's panning to view different parts of content
         awaitEachGesture {
           val event = awaitPointerEvent()
 
@@ -63,7 +66,7 @@ public fun SandboxBackground(
           }
         }
       }
-  ){
+  ) {
     Box(
       modifier = Modifier
         .wrapContentSize(unbounded = true, align = Alignment.Center)

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

@@ -18,16 +18,20 @@ import io.github.vinceglb.filekit.dialogs.FileKitType
 import io.github.vinceglb.filekit.dialogs.compose.rememberFilePickerLauncher
 
 @Composable
-public fun UploadFile(onFileSelected: (PlatformFile?) -> Unit) {
-  Box (modifier = Modifier
-    .padding(16.dp)
-    .fillMaxSize()
-  ){
+public fun UploadFile(
+  onFileSelect: (PlatformFile?) -> Unit,
+  modifier: Modifier = Modifier,
+) {
+  Box(
+    modifier
+      .padding(16.dp)
+      .fillMaxSize()
+  ) {
     val launcher = rememberFilePickerLauncher(
-      type = FileKitType.File(listOf("json","txt")),
+      type = FileKitType.File(listOf("json", "txt")),
       title = "Select Workflow Trace File"
     ) {
-      onFileSelected(it)
+      onFileSelect(it)
     }
     Button(
       onClick = { launcher.launch() },

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

@@ -11,8 +11,10 @@ import java.io.IOException
  * 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{
+public fun fetchRoot(
+  json: String
+): WorkflowNode? {
+  return try {
     val moshi = Moshi.Builder()
       .add(KotlinJsonAdapterFactory())
       .build()
@@ -21,8 +23,7 @@ public fun FetchRoot(json: String): WorkflowNode? {
     root
   } catch (e: JsonDataException) {
     throw JsonDataException("Failed to parse JSON: ${e.message}", e)
-  } catch (e: IOException){
+  } catch (e: IOException) {
     throw IOException("Malformed JSON: ${e.message}", e)
   }
 }
-

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

@@ -24,7 +24,7 @@ import androidx.compose.ui.unit.dp
  *
  * TBD what more metadata should be involved with each node, e.g. (props, states, # of render passes)
  */
-public data class WorkflowNode (
+public data class WorkflowNode(
   val id: String,
   val name: String,
   val children: List<WorkflowNode>
@@ -37,24 +37,25 @@ public data class WorkflowNode (
 @Composable
 public fun DrawWorkflowTree(
   node: WorkflowNode,
+  modifier: Modifier = Modifier,
 ) {
   Column(
-    modifier = Modifier
+    modifier
       .padding(5.dp)
-      .border(1.dp,Color.Black)
+      .border(1.dp, Color.Black)
       .fillMaxSize(),
     horizontalAlignment = Alignment.CenterHorizontally,
   ) {
     // draws itself
-    drawNode(node)
+    DrawNode(node)
 
     // draws children recursively
-    Row (
+    Row(
       horizontalArrangement = Arrangement.Center,
       verticalAlignment = Alignment.Top
     ) {
       node.children.forEach { childNode ->
-        DrawWorkflowTree (childNode)
+        DrawWorkflowTree(childNode)
       }
     }
   }
@@ -64,16 +65,16 @@ public fun DrawWorkflowTree(
  * A basic box that represents a workflow node
  */
 @Composable
-private fun drawNode(
+private fun DrawNode(
   node: WorkflowNode,
 ) {
   val open = remember { mutableStateOf(false) }
-  Box (
+  Box(
     modifier = Modifier
       .clickable { open.value = !open.value }
       .padding(10.dp)
-  ){
-    Column (horizontalAlignment = Alignment.CenterHorizontally) {
+  ) {
+    Column(horizontalAlignment = Alignment.CenterHorizontally) {
       Text(text = node.name)
       Text(text = "ID: ${node.id}")
       if (open.value) {