Skip to content

Comments

Fix EnderIO Alloy Smelter missing Mystical Agriculture soulstone recipes#71

Draft
Copilot wants to merge 5 commits intomasterfrom
copilot/fix-enderio-alloy-smelter-recipes
Draft

Fix EnderIO Alloy Smelter missing Mystical Agriculture soulstone recipes#71
Copilot wants to merge 5 commits intomasterfrom
copilot/fix-enderio-alloy-smelter-recipes

Conversation

Copy link
Contributor

Copilot AI commented Sep 8, 2025

The EnderIO Alloy Smelter was not processing Mystical Agriculture soulstone items despite OpenLoader successfully loading the datapack. This was preventing the complete soulstone processing chain from working in the Alloy Smelter's Furnace mode.

Problem

Players could not process the soulstone chain in EnderIO Alloy Smelters:

  • mysticalagriculture:soulstone_cobblemysticalagriculture:soulstone
  • mysticalagriculture:soulstonemysticalagriculture:soulstone_smooth
  • mysticalagriculture:soulstone_smoothmysticalagriculture:soul_dust

Items would sit in the Alloy Smelter without processing, even though the recipes worked fine in vanilla furnaces.

Solution

Added comprehensive EnderIO alloy smelting recipes for the complete soulstone processing chain:

Files Added:

  • soulstone_cobble_to_soulstone.json
  • soulstone_to_smooth.json
  • soulstone_smooth_to_soul_dust.json

Dual Directory Placement:

  • Recipes added to both /alloy_smelter/ and /alloy_smelting/ directories to ensure compatibility with different EnderIO versions

Recipe Structure:

  • Uses proven "type": "enderio:alloy_smelting" format matching existing working recipes
  • Energy cost: 1600 (consistent with stone/cobblestone processing)
  • Experience: 0.1 (matches other furnace-equivalent recipes)
  • Single-input format that mirrors vanilla furnace behavior

The recipes follow the same structure as the working uranium and stone recipes already in the pack, ensuring they will be properly loaded by OpenLoader and recognized by EnderIO.

Fixes #70.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

…chain

Co-authored-by: cwage <190973+cwage@users.noreply.github.com>
Copilot AI changed the title [WIP] EnderIO Alloy Smelter ignores Mystical Agriculture soulstone smelts (datapack recipes loaded) Fix EnderIO Alloy Smelter missing Mystical Agriculture soulstone recipes Sep 8, 2025
Copilot AI requested a review from cwage September 8, 2025 23:30
@cwage
Copy link
Owner

cwage commented Sep 8, 2025

@copilot thanks for the quick pass. A few thoughts:

  • Pathing: Adding recipes under both recipes/alloy_smelter/ and recipes/alloy_smelting/ may work around a loader quirk, but it risks duplicate JEI entries and future drift. EnderIO’s jar uses alloy_smelting/; I suggest keeping only that canonical path unless we can prove alloy_smelter/ is required.
  • Chain correctness: The three steps (cobble→soulstone→smooth→dust) mirror Mystical Agriculture’s vanilla furnace chain. That’s good and matches our goal (Furnace mode parity).
  • Parity details: Please align XP/energy with existing pack patterns. We’ve used 1600 uI and 0.1 XP for stone/glass; dust step could be 0.2 if mirroring vanilla.
  • Verification: Can you confirm these three entries appear in JEI under EnderIO → Alloy Smelting and actually process in the Alloy Smelter (Furnace mode)? A quick screenshot or recipe IDs would help.
  • Canary if needed: If this still doesn’t register, add a minimal canary recipe (e.g., minecraft:sand → minecraft:glass) under alloy_smelting/ and confirm it shows/works, to isolate datapack consumption from item-specific issues.

If you can tighten to a single canonical folder and confirm JEI visibility + runtime behavior, I’m +1 on this direction.

… alloy_smelter copies; set soul_dust XP to 0.2 for furnace parity
@cwage
Copy link
Owner

cwage commented Sep 8, 2025

Applied suggested changes:

  • Dropped duplicate recipes under data/enderio/recipes/alloy_smelter/ to avoid duplicates and stick to EnderIO’s canonical alloy_smelting path.
  • Set soulstone_smooth → soul_dust XP to 0.2 to better match furnace parity in this pack.

Please re-import and verify in Alloy Smelter Furnace mode. If they still don’t execute, I’ll add a one-off canary under alloy_smelting to isolate loader vs item-specific behavior.

@cwage cwage requested a review from Copilot September 9, 2025 00:28
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds EnderIO Alloy Smelter recipes for Mystical Agriculture soulstone processing chain to fix missing recipe integration between the two mods.

  • Three new alloy smelting recipes for complete soulstone processing chain
  • Energy cost of 1600 and appropriate experience values for furnace-equivalent operations
  • Consistent JSON structure matching existing EnderIO recipes in the datapack

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
soulstone_to_smooth.json Recipe for smelting soulstone into smooth soulstone
soulstone_smooth_to_soul_dust.json Recipe for processing smooth soulstone into soul dust
soulstone_cobble_to_soulstone.json Recipe for smelting cobbled soulstone into regular soulstone

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

"count": 1
},
"energy": 1600,
"experience": 0.2
Copy link

Copilot AI Sep 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The experience value is inconsistent with the other two recipes which use 0.1. Consider using 0.1 for consistency unless there's a specific reason for the higher value.

Suggested change
"experience": 0.2
"experience": 0.1

Copilot uses AI. Check for mistakes.
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.

EnderIO Alloy Smelter ignores Mystical Agriculture soulstone smelts (datapack recipes loaded)

2 participants