Anexim/Mana-Loop
1
1
Fork 0
Code Issues 6 Pull Requests Actions Packages Projects Releases Wiki Activity

Task: Replace Arcane Shard with elementally attuned materials in Fabricator recipes #157

New Issue
Closed
opened 2026-05-27 10:22:56 +02:00 by Anexim · 2 comments
Anexim commented 2026-05-27 10:22:56 +02:00
Owner
Copy Link
Copy Source

Description

All 12 Fabricator recipes currently use a generic arcaneShard material. They should use elementally attuned mana crystals matching the recipe's element (e.g., earth attuned for earth recipes, metal attuned for metal recipes).

Current State

Every recipe in src/lib/game/data/fabricator-recipes.ts uses arcaneShard as a material. The arcaneShard is defined in loot-drops.ts lines 15-20.

Fix Required

  1. Create elementally attuned shard/crystal material types (e.g., earthShard, metalShard, crystalShard, sandShard)
  2. Update each Fabricator recipe to use the appropriate attuned material instead of arcaneShard
  3. Update loot-drops.ts with the new material definitions
  4. Update any test fixtures that reference arcaneShard

Affected Files

  • src/lib/game/data/fabricator-recipes.ts — all 12 recipes (lines 38, 51, 64, 79, 92, 105, 120, 133, 146, 161, 174, 187)
  • src/lib/game/data/loot-drops.ts — add new material types
  • src/lib/game/__tests__/crafting-utils-recipe.test.ts — update test fixtures
## Description All 12 Fabricator recipes currently use a generic `arcaneShard` material. They should use elementally attuned mana crystals matching the recipe's element (e.g., earth attuned for earth recipes, metal attuned for metal recipes). ## Current State Every recipe in `src/lib/game/data/fabricator-recipes.ts` uses `arcaneShard` as a material. The `arcaneShard` is defined in `loot-drops.ts` lines 15-20. ## Fix Required 1. Create elementally attuned shard/crystal material types (e.g., `earthShard`, `metalShard`, `crystalShard`, `sandShard`) 2. Update each Fabricator recipe to use the appropriate attuned material instead of `arcaneShard` 3. Update `loot-drops.ts` with the new material definitions 4. Update any test fixtures that reference `arcaneShard` ## Affected Files - `src/lib/game/data/fabricator-recipes.ts` — all 12 recipes (lines 38, 51, 64, 79, 92, 105, 120, 133, 146, 161, 174, 187) - `src/lib/game/data/loot-drops.ts` — add new material types - `src/lib/game/__tests__/crafting-utils-recipe.test.ts` — update test fixtures
Anexim added the ai:todo label 2026-05-27 10:22:56 +02:00
n8n-gitea was assigned by Anexim 2026-05-27 10:22:56 +02:00
Anexim commented 2026-05-27 12:50:51 +02:00
Author
Owner
Copy Link
Copy Source

Starting work on #157 — Replace Arcane Shard with elementally attuned materials in Fabricator recipes. Will read fabricator-recipes.ts, loot-drops.ts, and test fixtures first.

Starting work on #157 — Replace Arcane Shard with elementally attuned materials in Fabricator recipes. Will read fabricator-recipes.ts, loot-drops.ts, and test fixtures first.
Anexim commented 2026-05-27 12:56:43 +02:00
Author
Owner
Copy Link
Copy Source

Fixed #157 — Replaced arcaneShard with elementally attuned materials in all 12 Fabricator recipes.

Changes:

  1. loot-drops.ts — Added 4 new material types: earthShard, metalShard, crystalShard, sandShard (with appropriate rarities and drop chances)
  2. fabricator-recipes.ts — Replaced arcaneShard with the correct attuned shard per recipe:
    • Earth recipes (earthHelm, earthChest, earthBoots) → earthShard
    • Metal recipes (metalBlade, metalShield, metalGloves) → metalShard
    • Crystal recipes (crystalWand, crystalRing, crystalAmulet) → crystalShard
    • Sand recipes (sandBoots, sandGloves, sandVest) → sandShard
  3. crafting-utils-recipe.test.ts — Updated test fixtures to use earthShard instead of arcaneShard

All 902 tests pass. Committed and pushed as cbeb0b5.

Fixed #157 — Replaced arcaneShard with elementally attuned materials in all 12 Fabricator recipes. **Changes:** 1. `loot-drops.ts` — Added 4 new material types: earthShard, metalShard, crystalShard, sandShard (with appropriate rarities and drop chances) 2. `fabricator-recipes.ts` — Replaced arcaneShard with the correct attuned shard per recipe: - Earth recipes (earthHelm, earthChest, earthBoots) → earthShard - Metal recipes (metalBlade, metalShield, metalGloves) → metalShard - Crystal recipes (crystalWand, crystalRing, crystalAmulet) → crystalShard - Sand recipes (sandBoots, sandGloves, sandVest) → sandShard 3. `crafting-utils-recipe.test.ts` — Updated test fixtures to use earthShard instead of arcaneShard All 902 tests pass. Committed and pushed as cbeb0b5.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
ai:blocked
ai:done
ai:in-progress
ai:priority:high
priority
 ai:review
ai:todo
No Label ai:todo
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
n8n-gitea zhipu Anexim
No Assignees n8n-gitea
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Anexim/Mana-Loop#157
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?