Skip to content

fix(util): disassembler and stack equivalence (NEW-141, BUG-083, NEW-116, NEW-170)#65

Open
ximaks00-hue wants to merge 4 commits into
akki697222:dev-MC1.20from
ximaks00-hue:fix/new-141-170-item-inventory-utils
Open

fix(util): disassembler and stack equivalence (NEW-141, BUG-083, NEW-116, NEW-170)#65
ximaks00-hue wants to merge 4 commits into
akki697222:dev-MC1.20from
ximaks00-hue:fix/new-141-170-item-inventory-utils

Conversation

@ximaks00-hue

Copy link
Copy Markdown
Contributor

Summary

  • NEW-141: pass RegistryAccess to getResultItem in disassembler
  • BUG-083: harvest level -1 for unbreakable blocks
  • NEW-116: prefer block.getHarvestTool before tag heuristic
  • NEW-170: skip damage compare for non-damageable items

Test plan

  • Disassembler handles datapack recipes
  • Geolyzer reports -1 for bedrock harvest level

…(NEW-141, BUG-083, NEW-116, NEW-170)

- Pass RegistryAccess into getIngredients so getResultItem no longer receives null (NEW-141).

- Return harvest level -1 for unbreakable blocks; prefer block.getHarvestTool before tag heuristics (BUG-083, NEW-116).

- Skip damage comparison for non-damageable items in haveSameItemType, matching Original getHasSubtypes guard (NEW-170).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants