inventoryIndexOrSelected over inventoryIndex within the break manager in order to avoid getSlotWithStack causing problems

Author: beanbag44

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

@@ -50,7 +50,7 @@ import com.lambda.threading.runSafe
 import com.lambda.util.BlockUtils.blockState
 import com.lambda.util.BlockUtils.calcItemBlockBreakingDelta
 import com.lambda.util.BlockUtils.instantBreakable
-import com.lambda.util.item.ItemStackUtils.inventoryIndex
+import com.lambda.util.item.ItemStackUtils.inventoryIndexOrSelected
 import com.lambda.util.math.distSq
 import com.lambda.util.world.raycast.RayCastUtils.blockResult
 import io.ktor.util.collections.*
@@ -169,7 +169,7 @@ class BreakChecker @SimCheckerDsl private constructor(simInfo: SimInfo)
                 val breakContext = BreakContext(
                     blockHit,
                     rotationRequest,
-                    swapStack.inventoryIndex,
+                    swapStack.inventoryIndexOrSelected,
                     stackSelection,
                     instant,
                     state,
@@ -228,7 +228,7 @@ class BreakChecker @SimCheckerDsl private constructor(simInfo: SimInfo)
         val breakContext = BreakContext(
             blockHit,
             rotationRequest,
-            swapStack.inventoryIndex,
+            swapStack.inventoryIndexOrSelected,
             stackSelection,
             instant,
             state,

src/main/kotlin/com/lambda/util/item/ItemStackUtils.kt

@@ -81,6 +81,12 @@ object ItemStackUtils {
 
     context(safeContext: SafeContext)
     val ItemStack.inventoryIndex get() = safeContext.player.inventory.getSlotWithStack(this)
+    context(safeContext: SafeContext)
+    val ItemStack.inventoryIndexOrSelected get() = with(safeContext) {
+        player.inventory.getSlotWithStack(this@inventoryIndexOrSelected).let {
+            if (it == -1) player.inventory.selectedSlot else it
+        }
+    }
 
     val List<ItemStack>.compressed: List<ItemStack>
         get() =