TerraFirmaCraft / TerraFirmaCraft

Survival Mode as it Should've Been
https://discord.gg/PRuAKvY
European Union Public License 1.2
493 stars 221 forks source link

Support for NBT Tools #2658

Open AngellusMortis opened 4 months ago

AngellusMortis commented 4 months ago

It is hard to integrate TFC into other mods that use NBT tools (such as Tetra, Slient Gear and Tinker's Construct) since TFC heavily depends on item tags for tools.

It would be great if there was some method to extend TFC with datapack allow things like tree feeling, the anvil, and scraping with NBT tools.

LordFokas commented 1 month ago

I just want to point out that ItemStack NBT is going away in 1.21 (it has been removed in 1.20.5 already) so this may need to be reviewed. ( source: https://neoforged.net/news/20.5release/ )

Since the other modders are going to have to adapt to this change in the future, I'd recommend either waiting to see what happens before making a decision or taking the chance to "sit everyone" in a chat and agree on a standard.

alcatrazEscapee commented 1 month ago

@AngellusMortis does Tetra, Silent Gear, and/or Tinkers Construct provide Ingredients, or are their tools identifiable with existing ingredients? (i.e. I'm proposing a switch from a tag to a data-driven arbitrary ingredient.)

alcatrazEscapee commented 1 month ago

Also, it would be helpful to get a list of what actions are desirable to expose as "tools", because a lot of TFC internals just use, i.e. #tfc:hammers but that influences multiple actions.