fix: AI comments

Author: The-Code-Monkey

src/client/java/com/tcm/MineTale/block/workbenches/screen/FurnaceWorkbenchScreen.java

@@ -41,11 +41,7 @@ private static MineTaleRecipeBookComponent createRecipeBookComponent(FurnaceWork
         List<RecipeBookComponent.TabInfo> tabs = List.of(
             new RecipeBookComponent.TabInfo(tabIcon.getItem(), ModRecipeDisplay.FURNACE_T1_SEARCH)
         );
-//         List<RecipeBookComponent.TabInfo> tabs = List.of(
-//     new RecipeBookComponent.TabInfo(tabIcon.getItem(), ModRecipeDisplay.CAMPFIRE_SEARCH)
-// );
 
-        // Ensure the last argument is the RecipeBookType, NOT the RecipeType
         return new MineTaleRecipeBookComponent(menu, tabs, ModRecipes.FURNACE_T1_TYPE);
     }
 

src/client/java/com/tcm/MineTale/datagen/ModRecipeProvider.java

@@ -47,19 +47,17 @@ protected RecipeProvider createRecipeProvider(HolderLookup.Provider registryLook
 			public void buildRecipes() {
 				new WorkbenchRecipeBuilder(ModRecipes.CAMPFIRE_TYPE, ModRecipes.CAMPFIRE_SERIALIZER)
 					.input(Ingredient.of(Items.PORKCHOP))
-					// Note: Slot 1 is optional in our logic, so we just don't add a second input
 					.output(new ItemStack(Items.COOKED_PORKCHOP))
-					.time(10) // Campfires usually take longer (30 seconds)
+					.time(10)
 					.unlockedBy("has_porkchop", has(Items.PORKCHOP))
 					.category(CraftingBookCategory.MISC)
 					.bookCategory(ModRecipeDisplay.CAMPFIRE_SEARCH)
 					.save(exporter, Identifier.fromNamespaceAndPath(MineTale.MOD_ID, "campfire_pork_cooking"));
 
 				new WorkbenchRecipeBuilder(ModRecipes.FURNACE_T1_TYPE, ModRecipes.FURNACE_SERIALIZER)
 					.input(Ingredient.of(Items.PORKCHOP))
-					// Note: Slot 1 is optional in our logic, so we just don't add a second input
 					.output(new ItemStack(Items.ACACIA_BOAT))
-					.time(10) // Campfires usually take longer (30 seconds)
+					.time(10)
 					.unlockedBy("has_porkchop", has(Items.PORKCHOP))
 					.category(CraftingBookCategory.MISC)
 					.bookCategory(ModRecipeDisplay.FURNACE_T1_SEARCH)

src/main/java/com/tcm/MineTale/block/workbenches/CampfireWorkbench.java

@@ -107,24 +107,7 @@ public VoxelShape getShape(BlockState state, BlockGetter level, BlockPos pos, Co
     @Override
     public BlockEntity newBlockEntity(BlockPos pos, BlockState state) {
         // AbstractWorkbench logic ensures only the Master block gets the entity.
-        // We override it here to point specifically to our Furnace entity.
+        // We override it here to point specifically to our Campfire entity.
         return super.newBlockEntity(pos, state);
     }
-
-    // TODO: Check if we need this
-    // /**
-    //  * Compute the master (base) block position for this block based on its state.
-    //  *
-    //  * @param state the block state of the current block
-    //  * @param pos   the position of the current block
-    //  * @return the position of the master (base) block: if the block is the upper half, the block below is used; if the block's type is `RIGHT`, the position is offset one block counterclockwise from its facing direction; otherwise the original position
-    //  */
-    // public BlockPos getMasterPos(BlockState state, BlockPos pos) {
-    //     BlockPos master = pos;
-    //     Direction facing = state.getValue(FACING);
-    //     if (state.getValue(HALF) == DoubleBlockHalf.UPPER) master = master.below();
-    //     if (state.getValue(TYPE) == ChestType.RIGHT) master = master.relative(facing.getCounterClockWise());
-    //     return master;
-    // }
-    
 }

src/main/java/com/tcm/MineTale/block/workbenches/FurnaceWorkbench.java

@@ -67,9 +67,9 @@ public <T extends BlockEntity> BlockEntityTicker<T> getTicker(Level level, Block
     }
 
     /**
-     * The codec used to serialize and deserialize this CampfireWorkbench type.
+     * The codec used to serialize and deserialize this FurnaceWorkbench type.
      *
-     * @return the MapCodec for this CampfireWorkbench
+     * @return the MapCodec for this FurnaceWorkbench
      */
     @Override
     protected MapCodec<? extends FurnaceWorkbench> codec() {

src/main/java/com/tcm/MineTale/block/workbenches/entity/CampfireWorkbenchEntity.java

@@ -28,6 +28,7 @@ public class CampfireWorkbenchEntity extends AbstractWorkbenchEntity {
     private int cookTime;
     private int cookTimeTotal = 200; 
     private int fuelTime;
+    private int inputEnd = 3;
 
     protected final ContainerData data = new ContainerData() {
         /**
@@ -110,6 +111,10 @@ public void tick(Level level, BlockPos pos, BlockState state) {
         if (shiftQueueForward()) {
             changed = true;
         }
+
+        if (changed) {
+            setChanged();
+        }
     }
 
     /**
@@ -119,7 +124,7 @@ public void tick(Level level, BlockPos pos, BlockState state) {
     private boolean shiftQueueForward() {
         boolean moved = false;
         // Start from the front and pull from the back
-        for (int i = Constants.INPUT_START; i < Constants.INPUT_START + 1; i++) {
+        for (int i = Constants.INPUT_START; i < this.inputEnd; i++) {
             ItemStack current = inventory.getItem(i);
             ItemStack next = inventory.getItem(i + 1);