copy dependency stack when nesting simulation to prevent concurrency issues

Author: beanbag44

src/main/kotlin/com/lambda/context/AutomationConfig.kt

@@ -62,7 +62,7 @@ object AutomationConfig : Configurable(LambdaConfig), Automated {
     val showAllEntries by setting("Show All Entries", false, "Show all entries in the task tree").group(Group.Debug)
     val shrinkFactor by setting("Shrink Factor", 0.001, 0.0..1.0, 0.001).group(Group.Debug)
     val ignoreItemDropWarnings by setting("Ignore Drop Warnings", false, "Hides the item drop warnings from the break manager").group(Group.Debug)
-    val maxSimDependencies by setting("Max Sim Dependencies", 10, 0..25, 1, "Maximum dependency build results").group(Group.Debug)
+    val maxSimDependencies by setting("Max Sim Dependencies", 3, 0..10, 1, "Maximum dependency build results").group(Group.Debug)
 
     @Volatile
     var drawables = listOf<Drawable>()

src/main/kotlin/com/lambda/interaction/construction/simulation/ISimInfo.kt

@@ -47,7 +47,7 @@ interface ISimInfo : Automated {
             targetState: TargetState,
             pov: Vec3d,
             concurrentResults: MutableSet<BuildResult>,
-            simBuilder: suspend ISimInfo.() -> Unit
+            simBuilder: suspend SimInfo.() -> Unit
         ) {
             SimInfo(
                 pos, state, targetState,
@@ -63,12 +63,12 @@ interface ISimInfo : Automated {
             state: BlockState = this.state,
             targetState: TargetState = this.targetState,
             pov: Vec3d = this.pov,
-            simBuilder: suspend ISimInfo.() -> Unit
+            simBuilder: suspend SimInfo.() -> Unit
         ) {
             SimInfo(
                 pos, state, targetState,
                 targetState.getProcessingInfo(pos) ?: return,
-                pov, dependencyStack, concurrentResults, this
+                pov, Stack<Sim<*>>().apply { addAll(dependencyStack) }, concurrentResults, this
             ).takeIf { it.hasBasicRequirements() }?.run { simBuilder() }
         }
     }

src/main/kotlin/com/lambda/interaction/construction/simulation/checks/BreakSim.kt

@@ -26,6 +26,7 @@ import com.lambda.interaction.construction.simulation.ISimInfo
 import com.lambda.interaction.construction.simulation.ISimInfo.Companion.sim
 import com.lambda.interaction.construction.simulation.Sim
 import com.lambda.interaction.construction.simulation.SimDsl
+import com.lambda.interaction.construction.simulation.SimInfo
 import com.lambda.interaction.construction.simulation.checks.PlaceSim.Companion.simPlacement
 import com.lambda.interaction.construction.verify.TargetState
 import com.lambda.interaction.material.ContainerSelection.Companion.selectContainer
@@ -74,7 +75,7 @@ class BreakSim private constructor(simInfo: ISimInfo)
     companion object {
         @SimDsl
         context(automatedSafeContext: AutomatedSafeContext, dependent: Sim<*>)
-        suspend fun ISimInfo.simBreak() =
+        suspend fun SimInfo.simBreak() =
             BreakSim(this).run {
                 withDependent(dependent) {
                     automatedSafeContext.simBreaks()

src/main/kotlin/com/lambda/interaction/construction/simulation/checks/PlaceSim.kt

@@ -26,6 +26,7 @@ import com.lambda.interaction.construction.simulation.ISimInfo
 import com.lambda.interaction.construction.simulation.ISimInfo.Companion.sim
 import com.lambda.interaction.construction.simulation.Sim
 import com.lambda.interaction.construction.simulation.SimDsl
+import com.lambda.interaction.construction.simulation.SimInfo
 import com.lambda.interaction.construction.simulation.checks.BreakSim.Companion.simBreak
 import com.lambda.interaction.construction.simulation.checks.PlaceSim.RotatePlaceTest.Companion.rotatePlaceTest
 import com.lambda.interaction.construction.verify.TargetState
@@ -77,7 +78,7 @@ class PlaceSim private constructor(simInfo: ISimInfo)
     companion object {
         context(automatedSafeContext: AutomatedSafeContext, dependent: Sim<*>)
         @SimDsl
-        suspend fun ISimInfo.simPlacement() =
+        suspend fun SimInfo.simPlacement() =
             PlaceSim(this).run {
                 withDependent(dependent) {
                     automatedSafeContext.simPlacements()

src/main/kotlin/com/lambda/interaction/construction/simulation/checks/PostProcessingSim.kt

@@ -23,8 +23,10 @@ import com.lambda.interaction.construction.result.BuildResult
 import com.lambda.interaction.construction.result.results.GenericResult
 import com.lambda.interaction.construction.result.results.InteractResult
 import com.lambda.interaction.construction.simulation.ISimInfo
+import com.lambda.interaction.construction.simulation.ISimInfo.Companion.sim
 import com.lambda.interaction.construction.simulation.Sim
 import com.lambda.interaction.construction.simulation.SimDsl
+import com.lambda.interaction.construction.simulation.SimInfo
 import com.lambda.interaction.construction.simulation.checks.PlaceSim.Companion.simPlacement
 import com.lambda.interaction.construction.verify.TargetState
 import com.lambda.interaction.material.ContainerSelection.Companion.selectContainer
@@ -52,7 +54,7 @@ class PostProcessingSim private constructor(simInfo: ISimInfo)
     companion object {
         context(automatedSafeContext: AutomatedSafeContext, dependent: Sim<*>)
         @SimDsl
-        suspend fun ISimInfo.simPostProcessing() =
+        suspend fun SimInfo.simPostProcessing() =
             PostProcessingSim(this).run {
                 withDependent(dependent) {
                     automatedSafeContext.simPostProcessing()
@@ -96,7 +98,7 @@ class PostProcessingSim private constructor(simInfo: ISimInfo)
                     simInteraction(expectedState)
                 }
 
-                Properties.SLAB_TYPE -> simPlacement()
+                Properties.SLAB_TYPE -> sim { simPlacement() }
             }
         }
     }