[ISSUE] Depot not following trading limit rules

[TheAnsuz]

Information:

Currently using the mod version 2.0.1 for minecraft forge 1.20.1.

The depot can indefinetly trade with villagers under the condition that it is during their working time (mid-day).
However as specified in the minecraft wiki villagers can only trade for a specific item a limited number of times, after that the trade gets disabled until the villager works on his choosen workstation (up to a maximum of 2 times per day).

Example:

If I have a Mason villager (using stonecutter) with the trade offer for 10 bricks for 1 emerald.  
I can trade up to 16 times before it gets disabled. I can repeat this two more times that day after hearing the "working sound". 
The trading floor however is capable of trading indefinetly, even if that trade is currently disabled for the villager
(As a bonus it does not add experience to the villager but that be unbalanced)

Desired behavior:

Trading floor should only be able to work for enabled trades and should disable the trade after reaching the limit.
This can be a configurable option so people who don't like this minecraft limit can just remove it or modify it (maybe a multiplier to allow larger trade amounts)

Reasoning:

I think it is to overpowered to allow limitless trades as this is meant to be used among create mod, as you will end up automating a lof of things and this kinda breaks the game very easily

[cakeGit]

Thanks for the very nicely done issue,

I do agree it should be an option, this is something I've wanted to do but never gotten round to due to being busy on other projects,

But when I do add the option, it is likely that the current state of it being unlimited stock will likely stay the default.

It wasn't a lightly made decision, but me and Tazer agreed (and from my opinion and experience in game), it generally isn't that powerful given the working hours and working times of the villager make a depot have for the most part slower speeds than a player (although I realise now trading a stack of emerald for stuff would break this rule, I'll add a limit next update), but it stays with create not really doing much with with scaling up machines, more of the focus is on automating the next item especially with the new update, so making a single process fairly quick is fine.

I'll leave this issue open until there's a max trade per work and trades out of stock option

[TheAnsuz]

I love your work on the mod, waiting to see what comes next!

I like the though of a max bulk trade amount, maybe block the result for being more than one stack per work? Or just limit to 4 bulk trades per work, that could balance those cheap trades.

About the decision I understand it, is just that after playing vanilla and needing like 4 villagers to get a decent amount of emeralds due to trade limits, needing just one and having more output feels a very large step in the production chain. Glad to see you have already planned a configuration for this case.

Thanks for your time!