Cooldown being ruined by more than one item in slot

[Crescentine]

Hi there, I have a machine set up in my mod where the machine's 'intended' behavior is that when you put in more than one thing such as in this case, the recipe uses the cooldown still. 

Something's definitely missing, I just don't know what to add to fix this. I had no other idea of how to explain this properly, so I made a quick video demonstrating the issue.

I have alot of classes related to this and registries and all that, and I don't think it'd be proper to post them all here. 

Here is my github repositary , but I can share individual files too.

Thanks for all help you are able to give.

https://github.com/Crescentine/Trajans-Tanks-1.18

[Draco18s]

https://github.com/Crescentine/Trajans-Tanks-1.18/blob/main/src/main/java/com/crescentine/trajanstanks/block/steelmanufacturer/SteelManufacturerBlockEntity.java#L180
ItemStack merging would be better (inventory.Insert(...)). What if there's a stack in the output that doesn't match the output of the current recipe?

Also, having separate item stack handlers for input(s) and output(s) would simplify things massively. Rather than having to remember that the output slot is slot 10 everywhere you can just say "output."

Old 1.14 (MCP names) code, but same idea

https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/java/com/draco18s/harderores/entity/PackagerTileEntity.java#L191

 

{edit, scratch that, one mo'...}
Nope don't see what the issue is. The entity calls resetProgress from craftItem and that should prevent it from ending up back in that method next tick. You might have to use the debugger to figure it out.

Edited April 26, 2022 by Draco18s