fix: require the full skill(amount) format for xp values

Values like smithing(5)(x) were read as smithing(5). They are now
skipped with a warning, and only a plain skill name can reach the
console exp command.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

Author: ryanbarlow97

src/main/java/net/tfminecraft/advancedcrafting/objects/crafting/CraftingStation.java

@@ -7,6 +7,8 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
+import java.util.regex.Matcher;
+import java.util.regex.Pattern;
 
 import org.apache.commons.lang.WordUtils;
 import org.bukkit.Bukkit;
@@ -261,6 +263,9 @@ public StationFeedback craft(Player p, Double forcedQualityPercent) {
 		return f;
 	}
 
+	// skill(amount), e.g. crafter(2.0)
+	private static final Pattern XP_FORMAT = Pattern.compile("([A-Za-z0-9_-]+)\\((\\d+(?:\\.\\d+)?)\\)");
+
 	private void giveXP(Player p) {
 		// Map of skill name -> total XP to give
 		Map<String, Double> xpBySkill = new HashMap<>();
@@ -271,26 +276,31 @@ private void giveXP(Player p) {
 			String mId = split[1];
 			int amount = currentMaterials.get(s);
 
-			double xpPerUnit = 0.0;
-			String skill = null;
+			String raw = null;
 
 			if (type.equalsIgnoreCase("ingredient")) {
 				Ingredient ingredient = IngredientLoader.getByString(mId); // Assuming you have a method like this
 				if (ingredient != null && ingredient.getIngredientData().hasXP()) {
-					String raw = ingredient.getIngredientData().getXP();
-					xpPerUnit = parseXPAmount(raw);
-					skill = raw.split("\\(")[0]; // Assuming you store "agriculturist" here
+					raw = ingredient.getIngredientData().getXP();
 				}
 			} else if (type.equalsIgnoreCase("alloy")) {
 				Alloy alloy = AlloyManager.getAlloyById(mId); // Likewise for alloy
 				if (alloy != null && alloy.getData().hasXP()) {
-					String raw = alloy.getData().getXP();
-					xpPerUnit = parseXPAmount(raw);
-					skill = raw.split("\\(")[0]; // Assuming you store "agriculturist" here
+					raw = alloy.getData().getXP();
 				}
 			}
+			if (raw == null) continue;
+
+			// XP is paid after the item drops, so a bad value must be skipped rather than throw and leave the station uncleared.
+			Matcher m = XP_FORMAT.matcher(raw.trim());
+			if (!m.matches()) {
+				Bukkit.getLogger().warning("AC: Invalid xp value '" + raw + "', expected skill(amount)");
+				continue;
+			}
+			String skill = m.group(1);
+			double xpPerUnit = Double.parseDouble(m.group(2));
 
-			if (skill != null && xpPerUnit > 0) {
+			if (xpPerUnit > 0) {
 				double totalXP = xpPerUnit * amount;
 				xpBySkill.put(skill, xpBySkill.getOrDefault(skill, 0.0) + totalXP);
 			}
@@ -307,16 +317,6 @@ private void giveXP(Player p) {
 	}
 
 
-	// XP is paid after the item drops, so a bad value must not throw and leave the station uncleared.
-	private double parseXPAmount(String raw) {
-		try {
-			return Double.parseDouble(raw.split("\\(")[1].replace(")", ""));
-		} catch (RuntimeException e) {
-			Bukkit.getLogger().warning("AC: Invalid xp value '" + raw + "', expected skill(amount)");
-			return 0.0;
-		}
-	}
-
 	private boolean checkItems(Player p) {
 		boolean complete = true;
 		for(IngredientType t : types.keySet()) {