fix(HoleFiller): share the remaining block budget between targets

[c8dhjp4tyv-bit]

iterateHoles() returns its original item budget after partially consuming it. In smart mode, three available blocks can therefore queue two separate two-block holes for different targets.

Return the remaining budget so later targets cannot spend items already assigned to a hole, while keeping creative mode exempt from the outer budget check. Adds a client regression with two independent player targets for competing holes, sufficient and insufficient budgets, duplicate holes and the creative-mode exemption.

Related to #9327, finding 7.

Validation: build and client game tests passed. The added regression also fails for its intended reason on unmodified sources.

[liquidbounce-support-bot]

Thanks for this — sharing the remaining budget instead of the original one looks right, and the regression test with a case that fails on unmodified sources is exactly what makes it easy to trust.

Related: this is finding 7 from the audit in #9327.

Two small notes for the reviewer:

• The gametest reaches collectHolesSmart through reflection (getDeclaredMethod + isAccessible = true), which ties it to private internals. Extracting the budget bookkeeping into an internal/testable function would be less brittle long-term.
• remainingItems vs. remainingItems1 is hard to read at the return site; renaming the mutated one (e.g. itemsAfterHoles) would make the fix self-evident.

No user-facing behaviour or settings change here, so nothing needed on the HoleFiller docs page.

---

🤖 Automated support reply — a human maintainer will review if this doesn't help.