Dispatch WorkflowInterceptor.onPropsChanged from Compose workflow nodes.

Author: zach-klippenstein

workflow-runtime/src/commonMain/kotlin/com/squareup/workflow1/internal/compose/ComposeWorkflowChildNode.kt

@@ -2,15 +2,19 @@ package com.squareup.workflow1.internal.compose
 
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.DisposableEffect
+import androidx.compose.runtime.NonRestartableComposable
+import androidx.compose.runtime.ReadOnlyComposable
 import androidx.compose.runtime.collection.MutableVector
 import androidx.compose.runtime.currentRecomposeScope
 import androidx.compose.runtime.getValue
 import androidx.compose.runtime.key
+import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
 import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.runtime.rememberUpdatedState
 import androidx.compose.runtime.saveable.LocalSaveableStateRegistry
 import androidx.compose.runtime.saveable.SaveableStateRegistry
+import androidx.compose.runtime.setValue
 import com.squareup.workflow1.ActionApplied
 import com.squareup.workflow1.ActionProcessingResult
 import com.squareup.workflow1.NoopWorkflowInterceptor
@@ -29,8 +33,8 @@ import com.squareup.workflow1.compose.WorkflowComposableRenderer
 import com.squareup.workflow1.identifier
 import com.squareup.workflow1.internal.IdCounter
 import com.squareup.workflow1.internal.WorkflowNodeId
-import com.squareup.workflow1.internal.requireSend
 import com.squareup.workflow1.internal.createId
+import com.squareup.workflow1.internal.requireSend
 import com.squareup.workflow1.workflowSessionToString
 import kotlinx.coroutines.CoroutineName
 import kotlinx.coroutines.CoroutineScope
@@ -96,6 +100,8 @@ internal class ComposeWorkflowChildNode<PropsT, OutputT, RenderingT>(
     }
   }
 
+  private var lastProps by mutableStateOf(initialProps)
+
   /**
    * Function invoked when [onNextAction] receives an output from [outputsChannel].
    */
@@ -149,6 +155,9 @@ internal class ComposeWorkflowChildNode<PropsT, OutputT, RenderingT>(
     // inside a renderChild call and renderChild does the keying.
     log("rendering workflow: props=$props")
     workflow as ComposeWorkflow
+
+    notifyInterceptorWhenPropsChanged(props)
+
     return withCompositionLocals(
       LocalSaveableStateRegistry provides saveableStateRegistry,
       LocalWorkflowComposableRenderer provides this
@@ -167,6 +176,22 @@ internal class ComposeWorkflowChildNode<PropsT, OutputT, RenderingT>(
     }
   }
 
+  @ReadOnlyComposable
+  @NonRestartableComposable
+  @Composable
+  private fun notifyInterceptorWhenPropsChanged(newProps: PropsT) {
+    if (lastProps != newProps) {
+      interceptor.onPropsChanged(
+        old = lastProps,
+        new = newProps,
+        state = ComposeWorkflowState,
+        session = this,
+        proceed = { _, _, _ -> ComposeWorkflowState },
+      )
+      lastProps = newProps
+    }
+  }
+
   @Composable
   override fun <ChildPropsT, ChildOutputT, ChildRenderingT> renderChild(
     childWorkflow: Workflow<ChildPropsT, ChildOutputT, ChildRenderingT>,