improvements

Author: beanbag44

src/main/kotlin/com/lambda/interaction/request/breaking/BreakManager.kt

@@ -508,13 +508,15 @@ object BreakManager : RequestHandler<BreakRequest>(
      *
      * If a primary [BreakInfo] is active, as long as the tick stage is valid, it is transformed
      * into a secondary break, so a new primary can be initialized. This means sending a
-     * [net.minecraft.network.packet.c2s.play.PlayerActionC2SPacket.Action] with action: [net.minecraft.network.packet.c2s.play.PlayerActionC2SPacket.Action.STOP_DESTROY_BLOCK]
+     * PlayerActionC2SPacket with action: STOP_DESTROY_BLOCK
      * packet to the server to start the automated breaking server side.
      *
      * If there is no way to keep both breaks, and the primary break hasn't been updated yet,
      * the primary break is canceled. Otherwise, the break cannot be started.
      *
      * @return the [BreakInfo], or null, if the break context wasn't accepted.
+     *
+     * @see net.minecraft.network.packet.c2s.play.PlayerActionC2SPacket
      */
     private fun AutomatedSafeContext.initNewBreak(
         requestCtx: BreakContext,
@@ -691,9 +693,11 @@ object BreakManager : RequestHandler<BreakRequest>(
         }
 
     /**
-     * A modified version of the vanilla [net.minecraft.client.network.ClientPlayerInteractionManager.updateBlockBreakingProgress] method.
+     * A modified version of the vanilla updateBlockBreakingProgress method.
      *
      * @return if the update was successful.
+     *
+     * @see net.minecraft.client.network.ClientPlayerInteractionManager.updateBlockBreakingProgress
      */
     private fun SafeContext.updateBreakProgress(info: BreakInfo): Unit = info.request.runSafeAutomated {
         val ctx = info.context
@@ -783,9 +787,11 @@ object BreakManager : RequestHandler<BreakRequest>(
     }
 
     /**
-     * A modified version of the minecraft [net.minecraft.client.network.ClientPlayerInteractionManager.attackBlock] method.
+     * A modified version of the minecraft attackBlock method.
      *
      * @return if the block started breaking successfully.
+     *
+     * @see net.minecraft.client.network.ClientPlayerInteractionManager.attackBlock
      */
     private fun AutomatedSafeContext.startBreaking(info: BreakInfo): Boolean {
         val ctx = info.context

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

@@ -39,8 +39,10 @@ import net.minecraft.util.math.BlockPos
  * A private constructor is used to force use of the cleaner [BreakRequestDsl] builder. This is
  * accessed through the [breakRequest] method.
  *
- * @param contexts A collection of [BreakContext]'s gathered from the [com.lambda.interaction.construction.simulation.BuildSimulator].
+ * @param contexts A collection of [BreakContext]'s gathered from the BuildSimulator.
  * @param pendingInteractions A mutable, concurrent list to store the pending actions.
+ *
+ * @see com.lambda.interaction.construction.simulation.BuildSimulator
  */
 data class BreakRequest private constructor(
     val contexts: Collection<BreakContext>,

src/main/kotlin/com/lambda/interaction/request/breaking/BrokenBlockHandler.kt

@@ -139,10 +139,12 @@ object BrokenBlockHandler : PostActionHandler<BreakInfo>() {
     }
 
     /**
-     * A modified version of the minecraft [net.minecraft.client.world.ClientWorld.breakBlock] method.
+     * A modified version of the minecraft breakBlock method.
      *
      * Performs the actions required to display breaking particles, sounds, texture overlay, etc.
      * based on the user's settings.
+     *
+     * @see net.minecraft.client.world.ClientWorld.breakBlock
      */
     fun SafeContext.destroyBlock(info: BreakInfo) {
         val ctx = info.context

src/main/kotlin/com/lambda/interaction/request/hotbar/HotbarManager.kt

@@ -124,8 +124,10 @@ object HotbarManager : RequestHandler<HotbarRequest>(
     }
 
     /**
-     * Sets the [activeRequest]. This also calls [net.minecraft.client.network.ClientPlayerInteractionManager.syncSelectedSlot] to
+     * Sets the [activeRequest]. This also calls syncSelectedSlot to
      * update the server now to keep predictability.
+     *
+     * @see net.minecraft.client.network.ClientPlayerInteractionManager.syncSelectedSlot
      */
     private fun AutomatedSafeContext.setActiveRequest(request: HotbarRequest) {
         maxSwapsThisTick = hotbarConfig.swapsPerTick

src/main/kotlin/com/lambda/interaction/request/inventory/InventoryManager.kt

@@ -157,7 +157,9 @@ object InventoryManager : RequestHandler<InventoryRequest>(
     private fun getStacks(slots: Collection<Slot>) = slots.map { it.stack.copy() }
 
     /**
-     * A modified version of the [net.minecraft.client.network.ClientPlayNetworkHandler.onInventory] method
+     * A modified version of the minecraft onInventory method
+     *
+     * @see net.minecraft.client.network.ClientPlayNetworkHandler.onInventory
      */
     @JvmStatic
     fun onInventoryUpdate(packet: InventoryS2CPacket, original: Operation<Void>){

src/main/kotlin/com/lambda/interaction/request/placing/PlaceManager.kt

@@ -207,8 +207,10 @@ object PlaceManager : RequestHandler<PlaceRequest>(
     }
 
     /**
-     * A modified version of the minecraft [net.minecraft.client.network.ClientPlayerInteractionManager.interactBlock] method,
+     * A modified version of the minecraft interactBlock method,
      * renamed to better suit its usage.
+     *
+     * @see net.minecraft.client.network.ClientPlayerInteractionManager.interactBlock
      */
     private fun AutomatedSafeContext.placeBlock(placeContext: PlaceContext, request: PlaceRequest, hand: Hand): ActionResult {
         interaction.syncSelectedSlot()
@@ -225,7 +227,9 @@ object PlaceManager : RequestHandler<PlaceRequest>(
     }
 
     /**
-     * A modified version of the minecraft [net.minecraft.client.network.ClientPlayerInteractionManager.interactBlockInternal] method.
+     * A modified version of the minecraft interactBlockInternal method.
+     *
+     * @see net.minecraft.client.network.ClientPlayerInteractionManager.interactBlockInternal
      */
     private fun AutomatedSafeContext.interactBlockInternal(
         placeContext: PlaceContext,
@@ -266,7 +270,9 @@ object PlaceManager : RequestHandler<PlaceRequest>(
     }
 
     /**
-     * A modified version of the minecraft [net.minecraft.item.Item.useOnBlock] method.
+     * A modified version of the minecraft useOnBlock method.
+     *
+     * @see net.minecraft.item.Item.useOnBlock
      */
     private fun AutomatedSafeContext.useOnBlock(
         placeContext: PlaceContext,
@@ -294,7 +300,9 @@ object PlaceManager : RequestHandler<PlaceRequest>(
     }
 
     /**
-     * A modified version of the minecraft [net.minecraft.item.BlockItem.place] method.
+     * A modified version of the minecraft place method.
+     *
+     * @see net.minecraft.item.BlockItem.place
      */
     private fun AutomatedSafeContext.place(
         placeContext: PlaceContext,