[priority: 3] SteadyStream has identical behavior to EternalFlow — latent bug #82

New Issue

2026-05-19T09:11:19+02:00

Anexim commented

2026-05-19 09:11:19 +02:00

Severity: 3 — Medium
File: src/lib/game/effects/dynamic-compute.ts, lines 79-86

Description:
Both EternalFlow and SteadyStream return the same value:

if (hasSpecial(effects, SPECIAL_EFFECTS.ETERNAL_FLOW)) {
  return regen * effects.regenMultiplier;
}
if (hasSpecial(effects, SPECIAL_EFFECTS.STEADY_STREAM)) {
  return regen * effects.regenMultiplier;
}

SteadyStream should only block incursion, not other penalties. But there are no other penalties in the current code, so this is a latent bug — if other penalties are added, SteadyStream will incorrectly block them too.

Fix: SteadyStream should apply incursion immunity but still allow other penalties.

**Severity:** 3 — Medium **File:** `src/lib/game/effects/dynamic-compute.ts`, lines 79-86 **Description:** Both `EternalFlow` and `SteadyStream` return the same value: ```ts if (hasSpecial(effects, SPECIAL_EFFECTS.ETERNAL_FLOW)) { return regen * effects.regenMultiplier; } if (hasSpecial(effects, SPECIAL_EFFECTS.STEADY_STREAM)) { return regen * effects.regenMultiplier; } ``` `SteadyStream` should only block incursion, not other penalties. But there are no other penalties in the current code, so this is a **latent bug** — if other penalties are added, `SteadyStream` will incorrectly block them too. **Fix:** `SteadyStream` should apply incursion immunity but still allow other penalties.

Anexim added the ai:todo label 2026-05-19 09:11:19 +02:00

n8n-gitea was assigned by Anexim

2026-05-19 09:11:19 +02:00

Anexim commented

2026-05-19 11:05:04 +02:00

Starting work on SteadyStream latent bug fix.

Anexim closed this issue

2026-05-19 11:23:13 +02:00

Anexim commented

2026-05-19 11:23:14 +02:00

✅ Fixed. SteadyStream no longer returns early like EternalFlow. Now only skips the incursion penalty while still applying regenMultiplier at the end. This prevents future bugs if other penalties are added between the SteadyStream check and the final return. Regression tests added.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Anexim/Mana-Loop#82