Improvement: Use a sequence in the build task for sim result handling (#232)

Author: beanbag44

src/main/kotlin/com/lambda/interaction/managers/breaking/BreakRequest.kt

@@ -143,7 +143,16 @@ data class BreakRequest private constructor(
 			nowOrNothing: Boolean = false,
 			builder: (BreakRequestBuilder.() -> Unit)? = null
 		) = asSequence()
-			.map { if (it is Dependent) it.lastDependency else it }
+			.breakRequest(pendingInteractions, nowOrNothing, builder)
+
+		@BreakRequestDsl
+		@JvmName("breakRequest3")
+		context(automated: Automated)
+		fun Sequence<BuildResult>.breakRequest(
+			pendingInteractions: MutableCollection<BuildContext>,
+			nowOrNothing: Boolean = false,
+			builder: (BreakRequestBuilder.() -> Unit)? = null
+		) = map { if (it is Dependent) it.lastDependency else it }
 			.filterIsInstance<BreakResult.Break>()
 			.sorted()
 			.map { it.context }
@@ -152,7 +161,7 @@ data class BreakRequest private constructor(
 			?.let { automated.breakRequest(it, pendingInteractions, nowOrNothing, builder) }
 
 		@BreakRequestDsl
-		@JvmName("breakRequest3")
+		@JvmName("breakRequest4")
 		fun Automated.breakRequest(
 			contexts: Collection<BreakContext>,
 			pendingInteractions: MutableCollection<BuildContext>,

src/main/kotlin/com/lambda/interaction/managers/interacting/InteractRequest.kt

@@ -79,7 +79,16 @@ data class InteractRequest private constructor(
 			nowOrNothing: Boolean = false,
 			builder: (PlaceRequestBuilder.() -> Unit)? = null
 		) = asSequence()
-			.map { if (it is Dependent) it.lastDependency else it }
+			.interactRequest(pendingInteractions, nowOrNothing, builder)
+
+		@PlaceRequestDsl
+		@JvmName("interactRequest2")
+		context(automated: Automated)
+		fun Sequence<BuildResult>.interactRequest(
+			pendingInteractions: MutableCollection<BuildContext>,
+			nowOrNothing: Boolean = false,
+			builder: (PlaceRequestBuilder.() -> Unit)? = null
+		) = map { if (it is Dependent) it.lastDependency else it }
 			.filterIsInstance<InteractResult.Interact>()
 			.sorted()
 			.map { it.context }
@@ -88,7 +97,7 @@ data class InteractRequest private constructor(
 			?.let { automated.interactRequest(it, pendingInteractions, nowOrNothing, builder) }
 
 		@PlaceRequestDsl
-		@JvmName("interactRequest2")
+		@JvmName("interactRequest3")
 		fun Automated.interactRequest(
 			contexts: Collection<InteractContext>,
 			pendingInteractions: MutableCollection<BuildContext>,

src/main/kotlin/com/lambda/task/tasks/BuildTask.kt

@@ -119,11 +119,15 @@ class BuildTask private constructor(
     }
 
     private fun AutomatedSafeContext.simulateAndProcess() {
-        val results = runSafeAutomated { blueprint.structure.simulate() }
+        val results =
+            blueprint.structure
+                .simulate()
+                .asSequence()
 
         DEFAULT.drawables = results
             .filterIsInstance<Drawable>()
             .plus(pendingInteractions.toList())
+            .toList()
 
         val viableResults = results
             .filter { result ->
@@ -141,7 +145,7 @@ class BuildTask private constructor(
         handleResult(bestResult, viableResults)
     }
 
-    private fun AutomatedSafeContext.handleResult(result: BuildResult, allResults: List<BuildResult>) {
+    private fun AutomatedSafeContext.handleResult(result: BuildResult, allResults: Sequence<BuildResult>) {
         if (result !is Dependent && result !is Contextual && pendingInteractions.isNotEmpty()) return
 
         when (result) {