small hitbox and sink fix (#715)

* changed hardcoded sink limit to be dynamic

Changed hardcoded sink limit from `y < 5` to `y < (craft.getWorld().getMinHeight() + 5)` preserving the original 4 block buffer but accounting for the new minimum world heights of 1.18+

* Adjusted X, Y, Z lengths by +1

Adjusted X, Y, Z, lengths by +1 to compensate for coordinate vs block math.

* Fixed missing `. `

* fixed missing `)` too

Author: wolfebersahd

Movecraft/src/main/java/net/countercraft/movecraft/async/AsyncManager.java

@@ -293,7 +293,7 @@ private void processSinking() {
             if (!(craft instanceof SinkingCraft))
                 continue;
 
-            if (craft.getHitBox().isEmpty() || craft.getHitBox().getMinY() < 5) {
+            if (craft.getHitBox().isEmpty() || craft.getHitBox().getMinY() < (craft.getWorld().getMinHeight() +5 )) {
                 CraftManager.getInstance().release(craft, CraftReleaseEvent.Reason.SUNK, false);
                 continue;
             }

api/src/main/java/net/countercraft/movecraft/util/hitboxes/HitBox.java

@@ -23,21 +23,21 @@ default int getXLength() {
         if (isEmpty())
             return 0;
 
-        return Math.abs(getMaxX() - getMinX());
+        return Math.abs(getMaxX() - getMinX() +1 );
     }
 
     default int getYLength() {
         if (isEmpty())
             return 0;
 
-        return Math.abs(getMaxY() - getMinY());
+        return Math.abs(getMaxY() - getMinY() +1 );
     }
 
     default int getZLength() {
         if (isEmpty())
             return 0;
 
-        return Math.abs(getMaxZ() -getMinZ());
+        return Math.abs(getMaxZ() -getMinZ() +1 );
     }
 
     default boolean isEmpty() {