CyclopsMC / IntegratedCrafting / 33948895090

67%

master-1.12: 27%

Pull #227

github

Raise the list recipe limit to a runaway guard

The attuned crafting interface already exposes every recipe of its target with
no limit at all, and indexing those 987 recipes of a crafting table costs it
the same as it costs a list interface (~95-100 ms for a network remove+add in
both cases). Limiting lists to 256 was therefore inconsistent: it withheld
something the mod already allows elsewhere.

The limit now defaults to 4096, which no regular machine reaches. It stays as a
guard against a computed list that runs away, since a list is not bounded by a
real machine the way an attuned interface is. Set it to 0 for no limit.

The machine reader game test asserted against the limit, which made it pass
for the wrong reason once the limit exceeded the machine's recipe count. It now
asserts that the interface exposes exactly the recipes its target holds.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Bn8brBLvFiJsfnBHpwf8ZN

1004 of 1740 branches covered (57.7%)

Branch coverage included in aggregate %.

3509 of 4918 relevant lines covered (71.35%)

3.66 hits per line