[Medium] [Task] [WIP/Incomplete] Mana conversion spec defines a complete new system but says "Manual conversion removed" — migration needed #279

Closed
opened 2026-06-05 02:12:52 +02:00 by Anexim · 3 comments
Owner

Type: Work in progress / Incomplete feature
Priority: Medium

mana-conversion-spec.md defines a complete unified mana conversion system that replaces the current fragmented approach. Key points:

  1. §10 says: "The existing convertMana action and processConvertAction are removed."
  2. §12 says: "Remove convertMana, processConvertAction, craftComposite" from manaStore
  3. §13 says: "Existing save data will need migration"

But the spec also says:

  • "The convertMana and craftComposite store actions are kept as no-ops for save compatibility but have no UI"

This is contradictory — it says both "removed" and "kept as no-ops."

The spec defines:

  • New files: element-distance.ts, conversion-rates.ts, conversion-costs.ts
  • Modified files: 10+ files
  • Removed: manual conversion, composite crafting via craftComposite

This is a major system overhaul that affects manaStore, gameStore, prestigeStore, attunements, disciplines, and UI components. The spec is detailed but it's unclear if any of this has been implemented.

Recommendation: Determine the current implementation status of the unified mana conversion system. If not implemented, create a plan for the migration.

**Type:** Work in progress / Incomplete feature **Priority:** Medium **mana-conversion-spec.md** defines a complete unified mana conversion system that replaces the current fragmented approach. Key points: 1. **§10** says: "The existing `convertMana` action and `processConvertAction` are **removed**." 2. **§12** says: "Remove `convertMana`, `processConvertAction`, `craftComposite`" from manaStore 3. **§13** says: "Existing save data will need migration" But the spec also says: - "The `convertMana` and `craftComposite` store actions are kept as no-ops for save compatibility but have no UI" This is contradictory — it says both "removed" and "kept as no-ops." The spec defines: - New files: `element-distance.ts`, `conversion-rates.ts`, `conversion-costs.ts` - Modified files: 10+ files - Removed: manual conversion, composite crafting via `craftComposite` This is a major system overhaul that affects manaStore, gameStore, prestigeStore, attunements, disciplines, and UI components. The spec is detailed but it's unclear if any of this has been implemented. **Recommendation:** Determine the current implementation status of the unified mana conversion system. If not implemented, create a plan for the migration.
Anexim added the ai:todo label 2026-06-05 02:12:52 +02:00
n8n-gitea was assigned by Anexim 2026-06-05 02:12:52 +02:00
Author
Owner

ultimate truth: follow the recommendation

ultimate truth: follow the recommendation
Author
Owner

RESOLVED — Created new Gitea issue #287. The old convertMana/craftComposite actions have been fully removed from manaStore.ts. The replacement unified passive conversion system from mana-conversion-spec.md is NOT yet implemented. The game currently has no mana conversion mechanism.

**RESOLVED** — Created new Gitea issue #287. The old `convertMana`/`craftComposite` actions have been fully removed from `manaStore.ts`. The replacement unified passive conversion system from `mana-conversion-spec.md` is NOT yet implemented. The game currently has no mana conversion mechanism.
Anexim added ai:done and removed ai:todo labels 2026-06-05 14:30:25 +02:00
Author
Owner

RESOLVED — Created new Gitea issue #287. The old convertMana/craftComposite actions have been fully removed from manaStore.ts. The replacement unified passive conversion system from mana-conversion-spec.md is NOT yet implemented.

✅ RESOLVED — Created new Gitea issue #287. The old convertMana/craftComposite actions have been fully removed from manaStore.ts. The replacement unified passive conversion system from mana-conversion-spec.md is NOT yet implemented.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Anexim/Mana-Loop#279