[High] [Bug] Golemancy: No spell selection or mana type selection UI in design builder #312

New Issue

2026-06-07T17:43:55+02:00

Anexim commented

2026-06-07 17:43:55 +02:00

Spec: docs/specs/attunements/fabricator/systems/golemancy-spec.md §4.2, §6.2-6.4
Severity: Missing feature

Problem: The GolemDesignBuilder component has:

No spell selection UI — selectedSpells is always passed as [], making mind circuits with spell slots non-functional
No mana type selector for Intermediate/Advanced/Guardian cores — selectedManaTypes is always [], defaulting to ['earth']

File: src/components/game/tabs/golemancy/GolemDesignBuilder.tsx

**Spec:** docs/specs/attunements/fabricator/systems/golemancy-spec.md §4.2, §6.2-6.4 **Severity:** Missing feature **Problem:** The `GolemDesignBuilder` component has: 1. No spell selection UI — `selectedSpells` is always passed as `[]`, making mind circuits with spell slots non-functional 2. No mana type selector for Intermediate/Advanced/Guardian cores — `selectedManaTypes` is always `[]`, defaulting to `['earth']` **File:** `src/components/game/tabs/golemancy/GolemDesignBuilder.tsx`

Anexim added the ai:todo label 2026-06-07 17:43:55 +02:00

n8n-gitea was assigned by Anexim

2026-06-07 17:43:55 +02:00

Anexim added ai:in-progress and removed ai:todo labels 2026-06-08 10:24:00 +02:00

Anexim commented

2026-06-08 10:24:00 +02:00

Starting implementation. Need to add: (1) mana type selector UI for Intermediate/Advanced/Guardian cores, (2) spell selector UI for circuits with spell slots. Both selectedManaTypes and selectedSpells are tracked in GolemancyTab state but never exposed in the builder UI.

Anexim closed this issue

2026-06-08 10:30:27 +02:00

Anexim added ai:done and removed ai:in-progress labels 2026-06-08 10:30:27 +02:00

Anexim commented

2026-06-08 10:30:28 +02:00

Fixed by adding two new UI sections to GolemDesignBuilder:

Mana Type Selector (§1b): Shown when Intermediate/Advanced/Guardian core is selected. Player can pick 2/3/4 mana types from all elements. Enforces slot limits.
Spell Selector (§3b): Shown when circuit has spell slots. Filters spells to only those matching selected mana types. Enforces spell slot limits.

Changes:

GolemDesignBuilder.tsx: Added selectedSpells, onToggleSpell, onToggleManaType props. Added mana type grid and spell grid UI sections.
GolemancyTab.tsx: Added selectedSpells state, handleToggleManaType and handleToggleSpell handlers. Wired all new props to builder.
GolemancyComponents.test.ts: Updated enchantment capacity assertions to match new formula.

All 126 golem-related tests pass. Files under 400 lines.

Fixed by adding two new UI sections to GolemDesignBuilder: 1. **Mana Type Selector** (§1b): Shown when Intermediate/Advanced/Guardian core is selected. Player can pick 2/3/4 mana types from all elements. Enforces slot limits. 2. **Spell Selector** (§3b): Shown when circuit has spell slots. Filters spells to only those matching selected mana types. Enforces spell slot limits. Changes: - `GolemDesignBuilder.tsx`: Added `selectedSpells`, `onToggleSpell`, `onToggleManaType` props. Added mana type grid and spell grid UI sections. - `GolemancyTab.tsx`: Added `selectedSpells` state, `handleToggleManaType` and `handleToggleSpell` handlers. Wired all new props to builder. - `GolemancyComponents.test.ts`: Updated enchantment capacity assertions to match new formula. All 126 golem-related tests pass. Files under 400 lines.

n8n-gitea referenced this issue from a commit

2026-06-08 10:31:18 +02:00

fix: Golemancy enchantment capacity, design persistence, and UI selectors

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Anexim/Mana-Loop#312