Use reflection to iterate over Node's properties

This had no visible affect on UI performance

Author: wenli-cai

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

@@ -31,22 +31,6 @@ internal data class Node(
   override fun hashCode(): Int {
     return id.hashCode()
   }
-
-  companion object {
-    fun getNodeField(node: Node, field: String): String {
-      return when (field.lowercase()) {
-        "name" -> node.name
-        "id" -> node.id
-        "parent" -> node.parent
-        "parentid" -> node.parentId
-        "props" -> node.props
-        "state" -> node.state
-        "rendering" -> node.rendering
-        "children" -> node.children.toString()
-        else -> throw IllegalArgumentException("Unknown field: $field")
-      }
-    }
-  }
 }
 
 internal fun Node.addChild(child: Node): Node {

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

@@ -12,7 +12,6 @@ import androidx.compose.foundation.layout.height
 import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.layout.size
 import androidx.compose.foundation.lazy.LazyColumn
-import androidx.compose.foundation.lazy.items
 import androidx.compose.material.Card
 import androidx.compose.material.Icon
 import androidx.compose.material.IconButton
@@ -35,6 +34,7 @@ import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.unit.dp
 import com.squareup.workflow1.traceviewer.model.Node
 import com.squareup.workflow1.traceviewer.model.NodeDiff
+import kotlin.reflect.full.memberProperties
 
 /**
  * A panel that displays information about the selected workflow node.
@@ -103,14 +103,18 @@ private fun NodePanelDetails(
         modifier = Modifier.padding(top = 8.dp, bottom = 8.dp)
       )
     }
-    val fields = listOf("Name", "Id", "Props", "State", "Rendering")
 
-    items(fields) { field ->
-      DetailCard(
-        label = field,
-        currValue = Node.getNodeField(node.current, field),
-        pastValue = if (node.previous != null) Node.getNodeField(node.previous, field) else null
-      )
+    val fields = Node::class.memberProperties
+    for (field in fields) {
+      val currVal = field.get(node.current).toString()
+      val pastVal = if (node.previous != null) field.get(node.previous).toString() else null
+      item {
+        DetailCard(
+          label = field.name,
+          currValue = currVal,
+          pastValue = pastVal
+        )
+      }
     }
   }
 }