Change Wrench checks in Create code to use a tag rather than a hardcoded item

src/generated/resources/.cache/5971ca79ecd127e7d1e65be1cd906ac939e6a245 → src/generated/resources/.cache/2d64935085b86659cb7857bad9701dbf9bab6e4c

@@ -1,4 +1,4 @@
-// 1.20.1	2025-09-10T22:15:14.76278042	Registrate Provider for create [Recipes, Advancements, Loot Tables, Tags (blocks), Tags (items), Tags (fluids), Tags (entity_types), generic_server_provider, Blockstates, Item models, Lang (en_us/en_ud), generic_client_provider]
+// 1.20.1	2025-09-17T09:54:58.774194	Registrate Provider for create [Recipes, Advancements, Loot Tables, Tags (blocks), Tags (items), Tags (fluids), Tags (entity_types), Blockstates, Item models, Lang (en_us/en_ud)]
 60bbdf92d2ac9824ea6144955c74043a6005f79d assets/create/blockstates/acacia_window.json
 6a67703c2697d81b7dc83e9d72a66f9c9ff08383 assets/create/blockstates/acacia_window_pane.json
 c3ae87b62e81d8e9476eccd793bb1548d74c66a1 assets/create/blockstates/adjustable_chain_gearshift.json
@@ -642,8 +642,8 @@ b0d8f08968763a5f74e5cd5644377a76a9f39753 assets/create/blockstates/yellow_toolbo
 fe8c497aacc641c2f01cec90bba9f19e59cc2ed2 assets/create/blockstates/yellow_valve_handle.json
 e819e93fdcbe9fd9c050a052d2718ff3b3539365 assets/create/blockstates/zinc_block.json
 64121dcb216381c83b4fe28aa361ea07c24c9ad0 assets/create/blockstates/zinc_ore.json
-30dd92866fb1516cc171825f1562109b3768eb96 assets/create/lang/en_ud.json
-77ece18f03c73bd494989335bcf319202b02fa10 assets/create/lang/en_us.json
+662e17c4a83c3685a1b11bbbad39cbabbaf8e93a assets/create/lang/en_ud.json
+c23658ca9e8bd29be2963fe6469ee9622854bd9b assets/create/lang/en_us.json
 a97e1060e00ae701a02e39cd4ef8054cf345fac4 assets/create/models/block/acacia_window.json
 103e032c0b1a0a6a27c67da8c91179a564bd281c assets/create/models/block/acacia_window_pane_noside.json
 fb00b627abda76ad4fea867ca57dbfadd24fffa3 assets/create/models/block/acacia_window_pane_noside_alt.json
@@ -4757,6 +4757,7 @@ a5b5711d1798473a9b25db5b7f749570ed0a2769 data/create/tags/items/upgrade_aquatic/
 da739ad2160e7df4e0e5cc89587670ce5e9450c3 data/create/tags/items/valve_handles.json
 c59c9fc0cdd45de659aa8023d36f9decb90f708c data/create/tags/items/vanilla_stripped_logs.json
 64441ac1daa64c81601b94b82b21c0ee862b6344 data/create/tags/items/vanilla_stripped_wood.json
+c741ba83b562ae3459d7c4f2db69537f549ea411 data/create/tags/items/wrench.json
 edf7390f09e622193bbc720ec51ba128ea1e96e6 data/curios/tags/items/head.json
 10d5f0c1763f80860a39398ecc33dbbe43099471 data/forge/tags/blocks/glass/colorless.json
 220b4c682f6cf536534e7cd6a79a9acc811bacd2 data/forge/tags/blocks/glass_panes.json

src/generated/resources/assets/create/lang/en_ud.json

@@ -3509,6 +3509,7 @@
   "tag.item.create.valve_handles": "sǝןpuɐH ǝʌןɐΛ",
   "tag.item.create.vanilla_stripped_logs": "sboꞀ pǝddıɹʇS ɐןןıuɐΛ",
   "tag.item.create.vanilla_stripped_wood": "pooM pǝddıɹʇS ɐןןıuɐΛ",
+  "tag.item.create.wrench": "sǝɥɔuǝɹM s,ǝʇɐǝɹƆ",
   "tag.item.forge.armors.boots": "sʇooᗺ",
   "tag.item.forge.armors.chestplates": "sǝʇɐןdʇsǝɥƆ",
   "tag.item.forge.armors.helmets": "sʇǝɯןǝH",

src/generated/resources/assets/create/lang/en_us.json

@@ -3509,6 +3509,7 @@
   "tag.item.create.valve_handles": "Valve Handles",
   "tag.item.create.vanilla_stripped_logs": "Vanilla Stripped Logs",
   "tag.item.create.vanilla_stripped_wood": "Vanilla Stripped Wood",
+  "tag.item.create.wrench": "Create's Wrenches",
   "tag.item.forge.armors.boots": "Boots",
   "tag.item.forge.armors.chestplates": "Chestplates",
   "tag.item.forge.armors.helmets": "Helmets",

src/generated/resources/data/create/tags/items/wrench.json

@@ -0,0 +1,5 @@
+{
+  "values": [
+    "create:wrench"
+  ]
+}

src/main/java/com/simibubi/create/AllItems.java

@@ -374,7 +374,7 @@ public class AllItems {
 	public static final ItemEntry<WrenchItem> WRENCH = REGISTRATE.item("wrench", WrenchItem::new)
 		.properties(p -> p.stacksTo(1))
 		.model(AssetLookup.itemModelWithPartials())
-		.tag(AllItemTags.WRENCH.tag)
+		.tag(AllItemTags.FORGE_WRENCH.tag, AllItemTags.CREATE_WRENCH.tag)
 		.register();
 
 	public static final ItemEntry<MinecartContraptionItem> MINECART_CONTRAPTION =

src/main/java/com/simibubi/create/AllTags.java

@@ -181,6 +181,7 @@ public enum AllItemTags {
 		CASING,
 		CONTRAPTION_CONTROLLED,
 		CREATE_INGOTS,
+		CREATE_WRENCH(MOD, "wrench"),
 		CRUSHED_RAW_MATERIALS,
 		INVALID_FOR_TRACK_PAVING,
 		DEPLOYABLE_DRINK,
@@ -210,13 +211,11 @@ public enum AllItemTags {
 		STRIPPED_WOOD(FORGE),
 
 		OBSIDIAN_DUST(FORGE, "dusts/obsidian"),
-
+		FORGE_WRENCH(FORGE, "tools/wrench"),
 		PLATES(FORGE),
 		OBSIDIAN_PLATES(FORGE, "plates/obsidian"),
 		CARDBOARD_PLATES(FORGE, "plates/cardboard"),
 
-		WRENCH(FORGE, "tools/wrench"),
-
 		ALLURITE(MOD, "stone_types/galosphere/allurite"),
 		AMETHYST(MOD, "stone_types/galosphere/amethyst"),
 		LUMIERE(MOD, "stone_types/galosphere/lumiere"),

src/main/java/com/simibubi/create/content/contraptions/AbstractContraptionEntity.java

@@ -10,11 +10,12 @@
 
 import javax.annotation.Nullable;
 
+import com.simibubi.create.AllTags.AllItemTags;
+
 import org.apache.commons.lang3.mutable.MutableInt;
 import org.apache.commons.lang3.tuple.MutablePair;
 
 import com.mojang.blaze3d.vertex.PoseStack;
-import com.simibubi.create.AllItems;
 import com.simibubi.create.AllPackets;
 import com.simibubi.create.AllSoundEvents;
 import com.simibubi.create.api.behaviour.movement.MovementBehaviour;
@@ -289,7 +290,7 @@ public boolean handlePlayerInteraction(Player player, BlockPos localPos, Directi
 										   InteractionHand interactionHand) {
 		int indexOfSeat = contraption.getSeats()
 			.indexOf(localPos);
-		if (indexOfSeat == -1 || AllItems.WRENCH.isIn(player.getItemInHand(interactionHand))) {
+		if (indexOfSeat == -1 || AllItemTags.CREATE_WRENCH.matches(player.getItemInHand(interactionHand))) {
 			if (contraption.interactors.containsKey(localPos))
 				return contraption.interactors.get(localPos)
 					.handlePlayerInteraction(player, interactionHand, localPos, this);

src/main/java/com/simibubi/create/content/contraptions/ContraptionHandlerClient.java

@@ -5,9 +5,10 @@
 
 import javax.annotation.Nullable;
 
+import com.simibubi.create.AllTags.AllItemTags;
+
 import org.apache.commons.lang3.mutable.MutableObject;
 
-import com.simibubi.create.AllItems;
 import com.simibubi.create.AllPackets;
 import com.simibubi.create.content.contraptions.sync.ContraptionInteractionPacket;
 import com.simibubi.create.content.trains.entity.CarriageContraptionEntity;
@@ -136,7 +137,7 @@ public static void rightClickingOnContraptionsGetsHandledLocally(InputEvent.Inte
 
 	private static boolean handleSpecialInteractions(AbstractContraptionEntity contraptionEntity, Player player,
 													 BlockPos localPos, Direction side, InteractionHand interactionHand) {
-		if (AllItems.WRENCH.isIn(player.getItemInHand(interactionHand))
+		if (AllItemTags.CREATE_WRENCH.matches(player.getItemInHand(interactionHand))
 			&& contraptionEntity instanceof CarriageContraptionEntity car)
 			return TrainRelocator.carriageWrenched(car.toGlobalVector(VecHelper.getCenterOf(localPos), 1), car);
 		return false;

src/main/java/com/simibubi/create/content/contraptions/actors/trainControls/ControlsInteractionBehaviour.java

@@ -3,7 +3,7 @@
 import java.util.UUID;
 
 import com.google.common.base.Objects;
-import com.simibubi.create.AllItems;
+import com.simibubi.create.AllTags.AllItemTags;
 import com.simibubi.create.api.behaviour.interaction.MovingInteractionBehaviour;
 import com.simibubi.create.content.contraptions.AbstractContraptionEntity;
 
@@ -19,7 +19,7 @@ public class ControlsInteractionBehaviour extends MovingInteractionBehaviour {
 	@Override
 	public boolean handlePlayerInteraction(Player player, InteractionHand activeHand, BlockPos localPos,
 		AbstractContraptionEntity contraptionEntity) {
-		if (AllItems.WRENCH.isIn(player.getItemInHand(activeHand)))
+		if (AllItemTags.CREATE_WRENCH.matches(player.getItemInHand(activeHand)))
 			return false;
 
 		UUID currentlyControlling = contraptionEntity.getControllingPlayer()

src/main/java/com/simibubi/create/content/contraptions/chassis/ChassisRangeDisplay.java

@@ -10,10 +10,11 @@
 import java.util.Set;
 
 import com.mojang.datafixers.util.Pair;
-import com.simibubi.create.AllItems;
 import com.simibubi.create.AllKeys;
 import com.simibubi.create.AllSpecialTextures;
 
+import com.simibubi.create.AllTags.AllItemTags;
+
 import net.createmod.catnip.outliner.Outliner;
 import net.minecraft.client.Minecraft;
 import net.minecraft.core.BlockPos;
@@ -89,7 +90,7 @@ protected Set<BlockPos> createSelection(ChassisBlockEntity chassis) {
 	public static void tick() {
 		Player player = Minecraft.getInstance().player;
 		Level world = Minecraft.getInstance().level;
-		boolean hasWrench = AllItems.WRENCH.isIn(player.getMainHandItem());
+		boolean hasWrench = AllItemTags.CREATE_WRENCH.matches(player.getMainHandItem());
 
 		for (Iterator<BlockPos> iterator = entries.keySet()
 			.iterator(); iterator.hasNext(); ) {

src/main/java/com/simibubi/create/content/contraptions/elevator/ElevatorContactBlock.java

@@ -6,7 +6,7 @@
 
 import com.simibubi.create.AllBlockEntityTypes;
 import com.simibubi.create.AllBlocks;
-import com.simibubi.create.AllItems;
+import com.simibubi.create.AllTags.AllItemTags;
 import com.simibubi.create.api.schematic.requirement.SpecialBlockItemRequirement;
 import com.simibubi.create.content.contraptions.elevator.ElevatorColumn.ColumnCoords;
 import com.simibubi.create.content.redstone.contact.RedstoneContactBlock;
@@ -229,7 +229,7 @@ public ItemRequirement getRequiredItems(BlockState state, BlockEntity be) {
 	@Override
 	public InteractionResult use(BlockState state, Level worldIn, BlockPos pos, Player player, InteractionHand handIn,
 		BlockHitResult hit) {
-		if (player != null && AllItems.WRENCH.isIn(player.getItemInHand(handIn)))
+		if (player != null && AllItemTags.CREATE_WRENCH.matches(player.getItemInHand(handIn)))
 			return InteractionResult.PASS;
 		DistExecutor.unsafeRunWhenOn(Dist.CLIENT,
 			() -> () -> withBlockEntityDo(worldIn, pos, be -> this.displayScreen(be, player)));

src/main/java/com/simibubi/create/content/contraptions/glue/SuperGlueHandler.java

@@ -5,6 +5,7 @@
 
 import com.simibubi.create.AllItems;
 import com.simibubi.create.AllPackets;
+import com.simibubi.create.AllTags.AllItemTags;
 import com.simibubi.create.api.contraption.BlockMovementChecks;
 
 import net.createmod.catnip.data.Iterate;
@@ -65,7 +66,7 @@ public static void glueInOffHandAppliesOnBlockPlace(EntityPlaceEvent event, Bloc
 		AttributeInstance reachAttribute = placer.getAttribute(ForgeMod.BLOCK_REACH.get());
 		if (!AllItems.SUPER_GLUE.isIn(itemstack) || reachAttribute == null)
 			return;
-		if (AllItems.WRENCH.isIn(placer.getMainHandItem()))
+		if (AllItemTags.CREATE_WRENCH.matches(placer.getMainHandItem()))
 			return;
 		if (event.getPlacedAgainst() == IPlacementHelper.ID)
 			return;

src/main/java/com/simibubi/create/content/contraptions/minecart/MinecartCouplingItem.java

@@ -1,6 +1,7 @@
 package com.simibubi.create.content.contraptions.minecart;
 
 import com.simibubi.create.AllItems;
+import com.simibubi.create.AllTags.AllItemTags;
 import com.simibubi.create.content.contraptions.minecart.capability.CapabilityMinecartController;
 import com.simibubi.create.content.contraptions.minecart.capability.MinecartController;
 
@@ -48,7 +49,7 @@ public static void handleInteractionWithMinecart(PlayerInteractEvent.EntityInter
 		if (AllItems.MINECART_COUPLING.isIn(heldItem)) {
 			if (!onCouplingInteractOnMinecart(event, minecart, player, controller))
 				return;
-		} else if (AllItems.WRENCH.isIn(heldItem)) {
+		} else if (AllItemTags.CREATE_WRENCH.matches(heldItem)) {
 			if (!onWrenchInteractOnMinecart(event, minecart, player, controller))
 				return;
 		} else

src/main/java/com/simibubi/create/content/contraptions/mounted/MinecartContraptionItem.java

@@ -4,6 +4,8 @@
 
 import javax.annotation.Nullable;
 
+import com.simibubi.create.AllTags.AllItemTags;
+
 import org.apache.commons.lang3.tuple.MutablePair;
 
 import com.simibubi.create.AllItems;
@@ -208,7 +210,7 @@ public static void wrenchCanBeUsedToPickUpMinecartContraptions(PlayerInteractEve
 			return;
 
 		ItemStack wrench = player.getItemInHand(event.getHand());
-		if (!AllItems.WRENCH.isIn(wrench))
+		if (!AllItemTags.CREATE_WRENCH.matches(wrench))
 			return;
 		if (entity instanceof AbstractContraptionEntity)
 			entity = entity.getVehicle();

src/main/java/com/simibubi/create/content/contraptions/wrench/RadialWrenchHandler.java

@@ -1,8 +1,9 @@
 package com.simibubi.create.content.contraptions.wrench;
 
-import com.simibubi.create.AllItems;
 import com.simibubi.create.AllKeys;
 
+import com.simibubi.create.AllTags.AllItemTags;
+
 import net.createmod.catnip.gui.ScreenOpener;
 import net.minecraft.client.Minecraft;
 import net.minecraft.client.player.LocalPlayer;
@@ -43,7 +44,7 @@ public static void onKeyInput(int key, boolean pressed) {
 		Level level = player.level();
 
 		ItemStack heldItem = player.getMainHandItem();
-		if (heldItem.getItem() != AllItems.WRENCH.get())
+		if (!AllItemTags.CREATE_WRENCH.matches(heldItem))
 			return;
 
 		HitResult objectMouseOver = mc.hitResult;

src/main/java/com/simibubi/create/content/decoration/girder/GirderBlock.java

@@ -4,9 +4,9 @@
 import static net.minecraft.world.level.block.state.properties.BlockStateProperties.WATERLOGGED;
 
 import com.simibubi.create.AllBlocks;
-import com.simibubi.create.AllItems;
 import com.simibubi.create.AllShapes;
 import com.simibubi.create.AllTags;
+import com.simibubi.create.AllTags.AllItemTags;
 import com.simibubi.create.content.decoration.bracket.BracketBlock;
 import com.simibubi.create.content.decoration.bracket.BracketedBlockEntityBehaviour;
 import com.simibubi.create.content.decoration.placard.PlacardBlock;
@@ -120,7 +120,7 @@ public InteractionResult use(BlockState pState, Level pLevel, BlockPos pPos, Pla
 			return InteractionResult.SUCCESS;
 		}
 
-		if (AllItems.WRENCH.isIn(itemInHand) && !pPlayer.isShiftKeyDown()) {
+		if (AllItemTags.CREATE_WRENCH.matches(itemInHand) && !pPlayer.isShiftKeyDown()) {
 			if (GirderWrenchBehavior.handleClick(pLevel, pPos, pState, pHit))
 				return InteractionResult.sidedSuccess(pLevel.isClientSide);
 			return InteractionResult.FAIL;

src/main/java/com/simibubi/create/content/decoration/girder/GirderWrenchBehavior.java

@@ -4,10 +4,11 @@
 import java.util.Collections;
 import java.util.List;
 
+import com.simibubi.create.AllTags.AllItemTags;
+
 import org.jetbrains.annotations.Nullable;
 
 import com.simibubi.create.AllBlocks;
-import com.simibubi.create.AllItems;
 
 import net.createmod.catnip.data.Iterate;
 import net.createmod.catnip.data.Pair;
@@ -51,7 +52,7 @@ public static void tick() {
 		if (!AllBlocks.METAL_GIRDER.has(world.getBlockState(pos)))
 			return;
 
-		if (!AllItems.WRENCH.isIn(heldItem))
+		if (!AllItemTags.CREATE_WRENCH.matches(heldItem))
 			return;
 
 		Pair<Direction, Action> dirPair = getDirectionAndAction(result, world, pos);

src/main/java/com/simibubi/create/content/equipment/blueprint/BlueprintEntity.java

@@ -7,6 +7,8 @@
 
 import javax.annotation.Nullable;
 
+import com.simibubi.create.AllTags.AllItemTags;
+
 import org.apache.commons.lang3.Validate;
 
 import com.simibubi.create.AllEntityTypes;
@@ -352,7 +354,7 @@ public InteractionResult interactAt(Player player, Vec3 vec, InteractionHand han
 		if (player instanceof FakePlayer)
 			return InteractionResult.PASS;
 
-		boolean holdingWrench = AllItems.WRENCH.isIn(player.getItemInHand(hand));
+		boolean holdingWrench = AllItemTags.CREATE_WRENCH.matches(player.getItemInHand(hand));
 		BlueprintSection section = getSectionAt(vec);
 		ItemStackHandler items = section.getItems();
 

src/main/java/com/simibubi/create/content/equipment/wrench/WrenchEventHandler.java

@@ -32,9 +32,9 @@ public static void useOwnWrenchLogicForCreateBlocks(PlayerInteractEvent.RightCli
 			return;
 		if (itemStack.isEmpty())
 			return;
-		if (AllItems.WRENCH.isIn(itemStack))
+		if (AllItemTags.CREATE_WRENCH.matches(itemStack))
 			return;
-		if (!AllItemTags.WRENCH.matches(itemStack.getItem()))
+		if (!AllItemTags.FORGE_WRENCH.matches(itemStack.getItem()))
 			return;
 
 		BlockState state = event.getLevel()

src/main/java/com/simibubi/create/content/equipment/wrench/WrenchItem.java

@@ -4,9 +4,9 @@
 
 import javax.annotation.Nonnull;
 
-import com.simibubi.create.AllItems;
 import com.simibubi.create.AllSoundEvents;
 import com.simibubi.create.AllTags;
+import com.simibubi.create.AllTags.AllItemTags;
 import com.simibubi.create.Create;
 import com.simibubi.create.foundation.item.render.SimpleCustomRenderer;
 
@@ -82,7 +82,7 @@ public static void wrenchInstaKillsMinecarts(AttackEntityEvent event) {
 			return;
 		Player player = event.getEntity();
 		ItemStack heldItem = player.getMainHandItem();
-		if (!AllItems.WRENCH.isIn(heldItem))
+		if (!AllItemTags.CREATE_WRENCH.matches(heldItem))
 			return;
 		if (player.isCreative())
 			return;

src/main/java/com/simibubi/create/content/kinetics/belt/BeltBlock.java

@@ -7,6 +7,8 @@
 import java.util.Set;
 import java.util.function.Consumer;
 
+import com.simibubi.create.AllTags.AllItemTags;
+
 import org.apache.commons.lang3.mutable.MutableBoolean;
 
 import com.simibubi.create.AllBlockEntityTypes;
@@ -257,7 +259,7 @@ public InteractionResult use(BlockState state, Level world, BlockPos pos, Player
 			return InteractionResult.PASS;
 		ItemStack heldItem = player.getItemInHand(handIn);
 
-		boolean isWrench = AllItems.WRENCH.isIn(heldItem);
+		boolean isWrench = AllItemTags.CREATE_WRENCH.matches(heldItem);
 		boolean isConnector = AllItems.BELT_CONNECTOR.isIn(heldItem);
 		boolean isShaft = AllBlocks.SHAFT.isIn(heldItem);
 		boolean isDye = heldItem.is(Tags.Items.DYES);

src/main/java/com/simibubi/create/content/kinetics/belt/BeltSlicer.java

@@ -7,6 +7,7 @@
 
 import com.simibubi.create.AllBlocks;
 import com.simibubi.create.AllItems;
+import com.simibubi.create.AllTags.AllItemTags;
 import com.simibubi.create.content.kinetics.base.KineticBlockEntity;
 import com.simibubi.create.content.kinetics.belt.BeltBlockEntity.CasingType;
 import com.simibubi.create.content.kinetics.belt.item.BeltConnectorItem;
@@ -468,7 +469,7 @@ public static void tickHoveringInformation() {
 		Feedback feedback = new Feedback();
 
 		// TODO: Populate feedback in the methods for clientside
-		if (AllItems.WRENCH.isIn(held) || AllItems.WRENCH.isIn(heldOffHand))
+		if (AllItemTags.CREATE_WRENCH.matches(held) || AllItemTags.CREATE_WRENCH.matches(heldOffHand))
 			useWrench(state, world, pos, mc.player, InteractionHand.MAIN_HAND, result, feedback);
 		else if (AllItems.BELT_CONNECTOR.isIn(held) || AllItems.BELT_CONNECTOR.isIn(heldOffHand))
 			useConnector(state, world, pos, mc.player, InteractionHand.MAIN_HAND, result, feedback);

src/main/java/com/simibubi/create/content/kinetics/crafter/MechanicalCrafterBlock.java

@@ -3,6 +3,7 @@
 import com.simibubi.create.AllBlockEntityTypes;
 import com.simibubi.create.AllBlocks;
 import com.simibubi.create.AllItems;
+import com.simibubi.create.AllTags.AllItemTags;
 import com.simibubi.create.content.kinetics.base.HorizontalKineticBlock;
 import com.simibubi.create.content.kinetics.base.KineticBlockEntity;
 import com.simibubi.create.content.kinetics.crafter.ConnectedInputHandler.ConnectedInput;
@@ -166,7 +167,7 @@ public InteractionResult use(BlockState state, Level worldIn, BlockPos pos, Play
 			return InteractionResult.PASS;
 
 		boolean isHand = heldItem.isEmpty() && handIn == InteractionHand.MAIN_HAND;
-		boolean wrenched = AllItems.WRENCH.isIn(heldItem);
+		boolean wrenched = AllItemTags.CREATE_WRENCH.matches(heldItem);
 
 		if (hit.getDirection() == state.getValue(HORIZONTAL_FACING)) {