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

🐛 MEDITATION: Mana Circulation cap perk has no observable effect without Void Meditation #165

New Issue
Closed
opened 2026-05-27 17:28:12 +02:00 by Anexim · 1 comment
Anexim commented 2026-05-27 17:28:12 +02:00
Owner
Copy Link
Copy Source

Bug

Max meditation multiplier from Mana Circulation perk (meditationCapBonus) doesn't increase the visible cap — still shows max 1.5x in ManaDisplay.

Root Cause

The Mana Circulation discipline's mana-circulation-meditation perk in src/lib/game/data/disciplines/base.ts:60-67 adds meditationCapBonus which correctly raises the maxMultiplier from 5.0 to 8.5. However:

  1. Without Void Meditation skill, the player's meditation is capped at 2.5x (Meditation Focus) or 3.0x (Deep Trance) — both far below the raised cap, making the discipline bonus irrelevant for 95% of the playthrough.
  2. The UI never shows the cap — ManaStatsSection.tsx:122-126 only shows the current multiplier with a cosmetic color threshold at 1.5x, but never displays the actual maximum.

Fix Suggestions

  • Make the Mana Circulation cap perk also raise the skill-specific caps (Meditation Focus, Deep Trance), not just Void Meditation's ceiling
  • OR add a dedicated discipline that improves meditation rate/effectiveness more broadly
  • Add cap display to the UI so players can see their max meditation multiplier

Files

  • src/lib/game/data/disciplines/base.ts:60-67 (perk definition)
  • src/lib/game/utils/mana-utils.ts:133-167 (getMeditationBonus)
  • src/components/game/tabs/StatsTab/ManaStatsSection.tsx:122-126 (display)
  • src/lib/game/effects/discipline-effects.ts:60-62 (cap routing)
## Bug Max meditation multiplier from Mana Circulation perk (`meditationCapBonus`) doesn't increase the visible cap — still shows max 1.5x in ManaDisplay. ## Root Cause The Mana Circulation discipline's `mana-circulation-meditation` perk in `src/lib/game/data/disciplines/base.ts:60-67` adds `meditationCapBonus` which correctly raises the `maxMultiplier` from 5.0 to 8.5. However: 1. **Without Void Meditation skill**, the player's meditation is capped at 2.5x (Meditation Focus) or 3.0x (Deep Trance) — both far below the raised cap, making the discipline bonus irrelevant for 95% of the playthrough. 2. **The UI never shows the cap** — `ManaStatsSection.tsx:122-126` only shows the current multiplier with a cosmetic color threshold at 1.5x, but never displays the actual maximum. ## Fix Suggestions - Make the Mana Circulation cap perk also raise the skill-specific caps (Meditation Focus, Deep Trance), not just Void Meditation's ceiling - OR add a dedicated discipline that improves meditation rate/effectiveness more broadly - Add cap display to the UI so players can see their max meditation multiplier ## Files - `src/lib/game/data/disciplines/base.ts:60-67` (perk definition) - `src/lib/game/utils/mana-utils.ts:133-167` (getMeditationBonus) - `src/components/game/tabs/StatsTab/ManaStatsSection.tsx:122-126` (display) - `src/lib/game/effects/discipline-effects.ts:60-62` (cap routing)
Anexim added the ai:todo label 2026-05-27 17:28:12 +02:00
n8n-gitea was assigned by Anexim 2026-05-27 17:28:12 +02:00
Anexim commented 2026-05-28 09:49:36 +02:00
Author
Owner
Copy Link
Copy Source

Fixed in commit 8cebea9. Added meditationCap display to ManaStatsSection UI so players can see their max meditation multiplier. Updated perk description to clarify it raises the Void Meditation ceiling.

Fixed in commit 8cebea9. Added meditationCap display to ManaStatsSection UI so players can see their max meditation multiplier. Updated perk description to clarify it raises the Void Meditation ceiling.
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#165
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?