wip: 🔥 Only three @deprecated remain, all worker related

artifacts.json

@@ -188,24 +188,6 @@
     "javaVersion": 17,
     "publicationName": "maven"
   },
-  {
-    "gradlePath": ":workflow-ui:container-android",
-    "group": "com.squareup.workflow1",
-    "artifactId": "workflow-ui-container-android",
-    "description": "Workflow UI Container Android",
-    "packaging": "aar",
-    "javaVersion": 17,
-    "publicationName": "maven"
-  },
-  {
-    "gradlePath": ":workflow-ui:container-common",
-    "group": "com.squareup.workflow1",
-    "artifactId": "workflow-ui-container-common-jvm",
-    "description": "Workflow UI Container",
-    "packaging": "jar",
-    "javaVersion": 17,
-    "publicationName": "maven"
-  },
   {
     "gradlePath": ":workflow-ui:core-android",
     "group": "com.squareup.workflow1",

samples/compose-samples/src/main/java/com/squareup/sample/compose/hellocomposebinding/HelloBindingActivity.kt

@@ -17,7 +17,6 @@ import com.squareup.workflow1.ui.ViewEnvironment
 import com.squareup.workflow1.ui.ViewRegistry
 import com.squareup.workflow1.ui.WorkflowLayout
 import com.squareup.workflow1.ui.WorkflowUiExperimentalApi
-import com.squareup.workflow1.ui.compose.composeViewFactory
 import com.squareup.workflow1.ui.compose.withCompositionRoot
 import com.squareup.workflow1.ui.container.withEnvironment
 import com.squareup.workflow1.ui.plus
@@ -31,7 +30,8 @@ private val viewEnvironment =
   }
 
 /**
- * Demonstrates how to create and display a view factory with [composeViewFactory].
+ * Demonstrates how to create and display a view factory with
+ * [composeScreenViewFactory][com.squareup.workflow1.ui.compose.composeScreenViewFactory].
  */
 class HelloBindingActivity : AppCompatActivity() {
   @OptIn(WorkflowUiExperimentalApi::class)

samples/containers/hello-back-button/build.gradle.kts

@@ -19,7 +19,6 @@ dependencies {
   implementation(libs.androidx.activity.ktx)
 
   implementation(project(":samples:containers:android"))
-  implementation(project(":workflow-ui:container-android"))
   implementation(project(":workflow-ui:core-android"))
   implementation(project(":workflow-ui:core-common"))
 }

samples/dungeon/app/build.gradle.kts

@@ -48,8 +48,6 @@ dependencies {
   implementation(project(":samples:dungeon:timemachine"))
   implementation(project(":samples:dungeon:timemachine-shakeable"))
   implementation(project(":workflow-tracing"))
-  implementation(project(":workflow-ui:container-android"))
-  implementation(project(":workflow-ui:container-common"))
   implementation(project(":workflow-ui:core-android"))
   implementation(project(":workflow-ui:core-common"))
 

samples/dungeon/app/src/main/java/com/squareup/sample/dungeon/DungeonAppWorkflow.kt

@@ -33,7 +33,7 @@ class DungeonAppWorkflow(
   data class DisplayBoardsListScreen(
     val boards: List<Board>,
     val onBoardSelected: (index: Int) -> Unit
-  ): Screen
+  ) : Screen
 
   override fun initialState(
     props: Props,

samples/dungeon/app/src/main/java/com/squareup/sample/dungeon/TimeMachineAppWorkflow.kt

@@ -7,10 +7,8 @@ import com.squareup.sample.timemachine.shakeable.ShakeableTimeMachineScreen
 import com.squareup.sample.timemachine.shakeable.ShakeableTimeMachineWorkflow
 import com.squareup.sample.timemachine.shakeable.ShakeableTimeMachineWorkflow.PropsFactory
 import com.squareup.workflow1.StatelessWorkflow
-import com.squareup.workflow1.mapRendering
 import com.squareup.workflow1.renderChild
 import com.squareup.workflow1.ui.WorkflowUiExperimentalApi
-import com.squareup.workflow1.ui.asScreen
 import kotlin.time.ExperimentalTime
 import kotlin.time.TimeSource
 
@@ -25,10 +23,9 @@ class TimeMachineAppWorkflow(
   context: Context
 ) : StatelessWorkflow<BoardPath, Nothing, ShakeableTimeMachineScreen>() {
 
-  @Suppress("DEPRECATION")
   private val timeMachineWorkflow =
     ShakeableTimeMachineWorkflow(
-      TimeMachineWorkflow(appWorkflow.mapRendering { asScreen(it) }, clock),
+      TimeMachineWorkflow(appWorkflow, clock),
       context
     )
 

samples/tictactoe/app/build.gradle.kts

@@ -40,7 +40,6 @@ dependencies {
   implementation(project(":samples:containers:android"))
   implementation(project(":samples:tictactoe:common"))
   implementation(project(":workflow-tracing"))
-  implementation(project(":workflow-ui:container-android"))
   implementation(project(":workflow-ui:core-android"))
   implementation(project(":workflow-ui:core-common"))
 }

settings.gradle.kts

@@ -73,8 +73,6 @@ include(
   ":workflow-ui:core-android",
   ":workflow-ui:internal-testing-android",
   ":workflow-ui:internal-testing-compose",
-  ":workflow-ui:container-common",
-  ":workflow-ui:container-android",
   ":workflow-ui:radiography"
 )
 

workflow-testing/src/test/java/com/squareup/workflow1/WorkerCompositionIntegrationTest.kt

@@ -192,7 +192,7 @@ internal class WorkerCompositionIntegrationTest {
       state += 1
     }
 
-    val workflow = Workflow.stateful<Int, Int, () -> Unit>(
+    val workflow = Workflow.stateful(
       initialState = 0,
       render = { _ ->
         runningWorker(triggerOutput) { action { setOutput(state) } }
@@ -221,7 +221,7 @@ internal class WorkerCompositionIntegrationTest {
 
   @Test fun `runningWorker throws when output emitted`() {
     @Suppress("UNCHECKED_CAST")
-    val worker = Worker.from { Unit } as Worker<Nothing>
+    val worker = Worker.from { } as Worker<Nothing>
     val workflow = Workflow.stateless<Unit, Unit, Unit> {
       // Suppress runningWorker usage because we want to make sure the deprecated
       // method still throws an exception as expected

workflow-testing/src/test/java/com/squareup/workflow1/testing/RealRenderTesterTest.kt

@@ -983,7 +983,7 @@ internal class RealRenderTesterTest {
 
   @Test fun `verifyAction verifies workflow output`() {
     val child = Workflow.stateless<Unit, String, Unit> {}
-    val workflow = Workflow.stateless<Unit, Nothing, Unit> {
+    val workflow = Workflow.stateless {
       renderChild(child) { TestAction(it) }
     }
     val testResult = workflow.testRender(Unit)
@@ -1002,7 +1002,7 @@ internal class RealRenderTesterTest {
 
   @Test fun `verifyAction verifies worker output`() {
     val worker = Worker.finished<String>()
-    val workflow = Workflow.stateless<Unit, Nothing, Unit> {
+    val workflow = Workflow.stateless {
       runningWorker(worker) { TestAction(it) }
     }
     val testResult = workflow.testRender(Unit)

workflow-ui/compose-tooling/api/compose-tooling.api

@@ -1,21 +1,10 @@
-public final class com/squareup/workflow1/ui/compose/tooling/ComposableSingletons$LegacyViewFactoriesKt {
-	public static final field INSTANCE Lcom/squareup/workflow1/ui/compose/tooling/ComposableSingletons$LegacyViewFactoriesKt;
-	public static field lambda-1 Lkotlin/jvm/functions/Function4;
-	public fun <init> ()V
-	public final fun getLambda-1$wf1_compose_tooling ()Lkotlin/jvm/functions/Function4;
-}
-
 public final class com/squareup/workflow1/ui/compose/tooling/ComposableSingletons$ViewFactoriesKt {
 	public static final field INSTANCE Lcom/squareup/workflow1/ui/compose/tooling/ComposableSingletons$ViewFactoriesKt;
 	public static field lambda-1 Lkotlin/jvm/functions/Function4;
 	public fun <init> ()V
 	public final fun getLambda-1$wf1_compose_tooling ()Lkotlin/jvm/functions/Function4;
 }
 
-public final class com/squareup/workflow1/ui/compose/tooling/LegacyViewFactoriesKt {
-	public static final fun Preview (Lcom/squareup/workflow1/ui/ViewFactory;Ljava/lang/Object;Landroidx/compose/ui/Modifier;Landroidx/compose/ui/Modifier;Lkotlin/jvm/functions/Function1;Landroidx/compose/runtime/Composer;II)V
-}
-
 public final class com/squareup/workflow1/ui/compose/tooling/ViewFactoriesKt {
 	public static final fun Preview (Lcom/squareup/workflow1/ui/ScreenViewFactory;Lcom/squareup/workflow1/ui/Screen;Landroidx/compose/ui/Modifier;Landroidx/compose/ui/Modifier;Lkotlin/jvm/functions/Function1;Landroidx/compose/runtime/Composer;II)V
 }