NEW API + EVENT HOOKS (While_Leeching + Resource Tracker/Thresholds) #29

Dubledice · 2025-08-24T07:22:37Z

PR to Parent branch.

Commits summarized:

Addition of new Conditional Effects via HasExileEffectCondition.java
Introduce ResourceTracker to accumulate loss per resource and per “spec key”
Add SpendThresholdRegistry (global spec registration + resolution) and SpendThresholdManager
Clean up at ResourcesData.modify(...):
One additional Stats in ResourceStats.java
Two additional Stats in OffenseStats.java
EntityLeechData.java refactor for necessary logic improvements
New API + event hooks in ResourcesData.java @ line 118 -> 198
New event hook in DamageEvent.java for health tracking

General:

All new systems should be datapack-friendly and side-correct; debug is gated by flags and sent to the player only

IMPORTANT: The commits on Aug 19, 2025 were mandatory changes for me to access submodules, please review those changes carefully as I don't want them to cause issues for the rest of the team.

Contains (NEW) API + Event Hooks. Contains Refactored/Factored Code. Desc: Made for new "WHILE_LEECHING" system. Also houses new HasExileEffectCondition, e.g. allowing WHILE_(...) tags.

Added new More Damage While Leeching Magic Shield Offense Stat. Added new Leech At Full Health Conditional Resource Stat,

Implemented brand new Resource Threshold fetcher, tracking ALL resources for conditional stats/effects. Desc: Completely Datapack driven.

src/main/java/com/robertx22/mine_and_slash/capability/entity/ResourceTracker.java

src/main/java/com/robertx22/mine_and_slash/capability/entity/EntityData.java

src/main/java/com/robertx22/mine_and_slash/capability/entity/ResourceTracker.java

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/datapack/SpendThresholdDef.java

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/SpendThresholdRuntime.java

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/SpendThresholdSpec.java

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/ThresholdsInit.java

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/datapack/SpendThresholdDef.java

SaloEater · 2025-08-26T03:25:48Z

...ain/java/com/robertx22/mine_and_slash/aoe_data/database/exile_effects/adders/ModEffects.java

+    }
+
+    // Pretty names for resources (UI text)
+    private static final Map<ResourceType, String> RES_NAME = Map.of(


In next PR after you move thresholds generation into database it should be possible to use ResourceType to handle translations instead of raw text

I had figured that leeching was hardcoded and wouldn't need datapack functionality, unless you want me to use datadriven resource types.

SaloEater · 2025-08-26T03:27:34Z

...ain/java/com/robertx22/mine_and_slash/aoe_data/database/exile_effects/adders/ModEffects.java

+    );
+
+    // ---------- Generic flags ----------
+    public static final EffectCtx LEECHING_STATE = state(


In next PR after you move thresholds generation into database it should be possible to replace raw text with translatable placeholders similar to locNameForLangFile function

...ain/java/com/robertx22/mine_and_slash/aoe_data/database/exile_effects/adders/ModEffects.java

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/datapack/SpendThresholdDef.java

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/SpendThresholdSpec.java

mahjerion · 2025-08-28T20:37:47Z

I see a lot of conversations resolved, but where any changes actually made? I'd like some of the conventions to be standardized. If it helps I can comment which conversations you've resolved I want actioned to make it more clear.

Dubledice · 2025-08-28T20:41:55Z

I see a lot of conversations resolved, but where any changes actually made? I'd like some of the conventions to be standardized. If it helps I can comment which conversations you've resolved I want actioned to make it more clear.

Yeah, please do. Although for many if not all resolved commits, I simply implemented the suggested changes.

SaloEater · 2025-08-28T22:38:22Z

but where any changes actually made?

It's still an ongoing process on making it datapack-driven, I hope too that Dice will push everything after it's done

mahjerion · 2025-08-28T23:15:01Z

but where any changes actually made?

It's still an ongoing process on making it datapack-driven, I hope too that Dice will push everything after it's done

Oh yeah, thanks for working with duble on this. Yeah spoke with duble earlier, I will wait patiently

Suggested Fixes: Datapack Threshold

Dubledice · 2025-09-03T14:14:36Z

Added Suggested Fixes: Datapack Threshold to the PR. Contains all requested changes. Contains potentially some placeholder values like "on_expire" which requires the onExpire refactor to work, but it now is all datapack driven.

EDIT: Some of the PR is missing, I will add it named "Suggested Fixes: Extension". Apologies for the mistake.

EDIT 2: Suggested Fixes: Extension has been added.

Suggested Fixes: Extension

SaloEater · 2025-09-03T16:16:01Z

...ain/java/com/robertx22/mine_and_slash/aoe_data/database/exile_effects/adders/ModEffects.java

    public static int ESSENCE_OF_FROST_MAX_STACKS = 5;

    // ---------- Helper ----------
    private static EffectCtx state(String id, String name, Elements elem) {


This has to be done as method overload

Suggested change

private static EffectCtx state(String id, String name, Elements elem) {

private static EffectCtx state(String id, String name, Elements elem) {

return new EffectCtx(id, name, elem, EffectType.beneficial);

}

private static EffectCtx statePhysical(String id, String name) {

return state(id, name, Elements.Physical);

}

SaloEater · 2025-09-03T16:16:55Z

...ain/java/com/robertx22/mine_and_slash/aoe_data/database/exile_effects/adders/ModEffects.java


            LEECHING_STATE_BY_RES.put(
-                rt, state("leeching_" + rt.id + "_state", "Leeching " + nice, elem)
+                rt, state("leeching_" + rt.id + "_state", "Leeching " + nice, null)


It's always a good behavior to get rid of arguments that can be null

Suggested change

rt, state("leeching_" + rt.id + "_state", "Leeching " + nice, null)

rt, statePhysical("leeching_" + rt.id + "_state", "Leeching " + nice)

Yeah, same thing here. I can't remove null. It causes errors because of how EffectCtx is setup.

SaloEater · 2025-09-03T16:17:04Z

...ain/java/com/robertx22/mine_and_slash/aoe_data/database/exile_effects/adders/ModEffects.java

            );
            REGEN_STATE_BY_RES.put(
-                rt, state("regen_" + rt.id + "_state", "Regenerating " + nice, elem)
+                rt, state("regen_" + rt.id + "_state", "Regenerating " + nice, null)


Suggested change

rt, state("regen_" + rt.id + "_state", "Regenerating " + nice, null)

rt, statePhysical("regen_" + rt.id + "_state", "Regenerating " + nice)

Why statePhysical?

Also changing it to:

REGEN_STATE_BY_RES.put( rt, statePhysical("regen_" + rt.id + "_state", "Regenerating " + nice) );

causes errors. Only rt, state("regen_" + rt.id + "_state", "Regenerating " + nice, null) is accepted.

Because that null you had before meant Elements.Physical in your state method little higher here

Because that null you had before meant Elements.Physical in your state method little higher here

Sure, but I can not change it to anything else other than using physical or another element. If you want me to change it to rt, state("regen_" + rt.id + "_state", "Regenerating " + nice, Elements.Physical) I can do that.

Sure, but I can not change it to anything else other than using physical. If you want me to change it to rt, state("regen_" + rt.id + "_state", "Regenerating " + nice, Elements.Physical) I can do that.

Yea, either this or what I suggested before

Sure, but I can not change it to anything else other than using physical. If you want me to change it to rt, state("regen_" + rt.id + "_state", "Regenerating " + nice, Elements.Physical) I can do that.

Yea, either this or what I suggested before

I can't do what you suggested before because of how EffectCtx is defined. I will have to use a specific element from now on. Which I assume I can just default everything to phys if I have to.

SaloEater · 2025-09-03T16:20:32Z

...ain/java/com/robertx22/mine_and_slash/mechanics/thresholds/DataDrivenSpendThresholdSpec.java

+
+    public boolean showUi() { return showUi; }
+
+    // Perk lock is handled by callers (anonymous subclass) when needed.


Is this comment needed here?

Yes, this will be implemented later. It is placeholder for now.

Sure, just make sure next time no placeholders are added.
They just add extra lines and some complexity to the PR while doing literally nothing, I think everyone would appreciate such approach

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/SpendThresholdSpec.java

SaloEater · 2025-09-03T16:42:40Z

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/datapack/SpendThresholdDef.java

-        @SerializedName("effect_id") public String effectId;
-        @SerializedName("duration_ticks") public int durationTicks = 200;
+        @SerializedName("exile_potion_id") public String effectId;
+        @SerializedName("duration_seconds") public int durationSeconds = 0;


Can you return back to ticks instead of seconds everywhere?
It seems like your code needs ticks and MnS also usually uses ticks

I can revert this, but, the code still uses ticks. I just did some math to convert seconds to ticks because I am a neanderthal and cant count ticks. If it's an issue I can just deal with it and eventually I will remember the conversion.

Yes, please, let's use ticks instead of seconds

SaloEater · 2025-09-03T16:50:26Z

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/datapack/SpendThresholdDef.java

                for (ProcAction a : onProc) {
-                    if (!"apply_effect".equalsIgnoreCase(a.action) || a.effectId == null) continue;
+                    if (!"exile_effect".equalsIgnoreCase(a.action) || a.effectId == null) continue;
+                    var effect = com.robertx22.mine_and_slash.database.registry.ExileDB.ExileEffects().get(a.effectId);


add an import and

Suggested change

var effect = com.robertx22.mine_and_slash.database.registry.ExileDB.ExileEffects().get(a.effectId);

var effect = ExileDB.ExileEffects().get(a.effectId);

SaloEater · 2025-09-03T17:01:09Z

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/datapack/SpendThresholdDef.java

+
+                    // Attach on-expire duration overrides (convert seconds -> ticks)
+                    if (a.onExpire != null && !a.onExpire.isEmpty()) {
+                        if (inst.onExpireEffectDurationTicks == null) {


This PR doesn't have ExileEffectInstanceData.onExpireEffectDurationTicks, looks like this belongs to an another PR

Yes, this is apart of another PR. If it's causing errors just comment it out as it will be used later.

If you try to run your branch now, it won't even compile due to this error, so yea, just comment it out until next PR

Is it specifically lines 108-118 that is causing issues?

I mean, just try to compile the mod

Mine-And-Slash-Rework-new/src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/datapack/SpendThresholdDef.java:109: error: cannot find symbol if (inst.onExpireEffectDurationTicks == null) { ^ symbol: variable onExpireEffectDurationTicks location: variable inst of type ExileEffectInstanceData

Okay, got it. Thank you!

SaloEater · 2025-09-03T17:02:46Z

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/datapack/SpendThresholdDef.java

+                var unit  = com.robertx22.mine_and_slash.uncommon.datasaving.Load.Unit(sp);
                var store = unit.getStatusEffectsData();

                for (ProcAction a : onProc) {


Replace this loop with

for (ProcAction a : onProc) { if (!"exile_effect".equalsIgnoreCase(a.action) || a.effectId == null) continue; var effect = ExileDB.ExileEffects().get(a.effectId); if (effect == null) continue; int durTicks = SpendThresholdSpec.secondsToTicks(a.durationSeconds); int stacks = Math.max(1, a.stacks); var inst = EffectUtils.applyEffect(sp, effect, durTicks, stacks); // Attach on-expire duration overrides (convert seconds -> ticks) if (a.onExpire != null && !a.onExpire.isEmpty()) { if (inst.onExpireEffectDurationTicks == null) { inst.onExpireEffectDurationTicks = new java.util.HashMap<>(); } for (var e : a.onExpire.entrySet()) { int ticks = SpendThresholdSpec.secondsToTicks(Math.max(0, e.getValue())); if (ticks > 0) { inst.onExpireEffectDurationTicks.put(e.getKey(), ticks); } } } effect.onApply(sp); unit.sync.setDirty(); }

To make this work, replace EffectUtils with this:

public final class EffectUtils { private EffectUtils() {} /** * Apply/refresh a state effect on the player. * * @param sp target player * @param ctx effect context (ids defined in ModEffects) * @param durationTicks desired remaining lifetime (ticks); merged via MAX * @param stacks desired stacks; clamped to effect.max_stacks and merged via MAX * @return ExileEffectInstanceData for the applied effect, or null if resolve failed. */ public static ExileEffectInstanceData applyState(ServerPlayer sp, EffectCtx ctx, int durationTicks, int stacks) { final ExileEffect effect = resolveEffect(ctx); if (effect == null) return null; return applyEffect(sp, effect, durationTicks, stacks); } public static ExileEffectInstanceData applyEffect(ServerPlayer sp, ExileEffect effect, int durationTicks, int stacks) { if (effect == null) return null; var unit = Load.Unit(sp); var store = unit.getStatusEffectsData(); var inst = store.getOrCreate(effect); // persist if missing // Merge stacks/ticks: refresh semantics (never decrease on re-apply) final int wanted = Math.max(1, stacks); final int capped = (effect.max_stacks > 0) ? Math.min(wanted, effect.max_stacks) : wanted; inst.stacks = Math.max(inst.stacks, capped); inst.ticks_left = Math.max(inst.ticks_left, durationTicks); // Keep vanilla stats / one-of-a-kind cleanup in sync effect.onApply(sp); unit.sync.setDirty(); // network/state sync return inst; } /** Try both resourcePath (preferred) and id; some data uses either. */ private static ExileEffect resolveEffect(EffectCtx ctx) { ExileEffect eff = ExileDB.ExileEffects().get(ctx.resourcePath); if (eff == null) eff = ExileDB.ExileEffects().get(ctx.id); return eff; } }

What is wrong with the current version?

It's a good behavior to have no duplicated code and your for loop was almost a clean copy of EffectUtils.applyEffect so let's keep code clean

Replace this loop with

for (ProcAction a : onProc) { if (!"exile_effect".equalsIgnoreCase(a.action) || a.effectId == null) continue; var effect = ExileDB.ExileEffects().get(a.effectId); if (effect == null) continue; int durTicks = SpendThresholdSpec.secondsToTicks(a.durationSeconds); int stacks = Math.max(1, a.stacks); var inst = EffectUtils.applyEffect(sp, effect, durTicks, stacks); // Attach on-expire duration overrides (convert seconds -> ticks) if (a.onExpire != null && !a.onExpire.isEmpty()) { if (inst.onExpireEffectDurationTicks == null) { inst.onExpireEffectDurationTicks = new java.util.HashMap<>(); } for (var e : a.onExpire.entrySet()) { int ticks = SpendThresholdSpec.secondsToTicks(Math.max(0, e.getValue())); if (ticks > 0) { inst.onExpireEffectDurationTicks.put(e.getKey(), ticks); } } } effect.onApply(sp); unit.sync.setDirty(); }

To make this work, replace EffectUtils with this:

public final class EffectUtils { private EffectUtils() {} /** * Apply/refresh a state effect on the player. * * @param sp target player * @param ctx effect context (ids defined in ModEffects) * @param durationTicks desired remaining lifetime (ticks); merged via MAX * @param stacks desired stacks; clamped to effect.max_stacks and merged via MAX * @return ExileEffectInstanceData for the applied effect, or null if resolve failed. */ public static ExileEffectInstanceData applyState(ServerPlayer sp, EffectCtx ctx, int durationTicks, int stacks) { final ExileEffect effect = resolveEffect(ctx); if (effect == null) return null; return applyEffect(sp, effect, durationTicks, stacks); } public static ExileEffectInstanceData applyEffect(ServerPlayer sp, ExileEffect effect, int durationTicks, int stacks) { if (effect == null) return null; var unit = Load.Unit(sp); var store = unit.getStatusEffectsData(); var inst = store.getOrCreate(effect); // persist if missing // Merge stacks/ticks: refresh semantics (never decrease on re-apply) final int wanted = Math.max(1, stacks); final int capped = (effect.max_stacks > 0) ? Math.min(wanted, effect.max_stacks) : wanted; inst.stacks = Math.max(inst.stacks, capped); inst.ticks_left = Math.max(inst.ticks_left, durationTicks); // Keep vanilla stats / one-of-a-kind cleanup in sync effect.onApply(sp); unit.sync.setDirty(); // network/state sync return inst; } /** Try both resourcePath (preferred) and id; some data uses either. */ private static ExileEffect resolveEffect(EffectCtx ctx) { ExileEffect eff = ExileDB.ExileEffects().get(ctx.resourcePath); if (eff == null) eff = ExileDB.ExileEffects().get(ctx.id); return eff; } }

This ended up breaking my code in EffectUtils.java, but I will fix it since I see the vision.

Suggested Fixes: Cleanup

Dubledice · 2025-09-03T18:40:22Z

Added changes to new PR: Suggested Fixes: Cleanup. Awaiting review.

SaloEater · 2025-09-03T18:47:29Z

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/SpendThresholdSpec.java

+    public int priority()                       { return priority; }
+    public boolean showUi()                     { return showUi; }
+
+    // fluent config (for code-defined specs)


Methods from 73 to 111 still needed to be removed excluding withPriority method

@Dubledice this is still pending

SaloEater · 2025-09-03T18:50:43Z

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/SpendThresholdManager.java

+public final class SpendThresholdManager {
+    private SpendThresholdManager() {}
+
+    public static void registerDefaults() {


This still needs to be removed

I can't delete SpendThresholdManager.java as it is used for onExpire logic later. It also handles debugging.

Suggested Fixes: SpendThresholdManager.java

Dubledice · 2025-09-03T20:29:26Z

Suggested Fixes: SpendThresholdManager.java added to PR

Pr/suggested fixes

Dubledice · 2025-09-03T21:18:25Z

Added Suggested Fixes: SpendThresholdSpec.java to the PR.

SaloEater

Fingers crossed

Dubledice added 3 commits August 24, 2025 03:17

Conditional Leeching + Conditional Predicate

ff6fb59

Contains (NEW) API + Event Hooks. Contains Refactored/Factored Code. Desc: Made for new "WHILE_LEECHING" system. Also houses new HasExileEffectCondition, e.g. allowing WHILE_(...) tags.

New STATS

bf52165

Added new More Damage While Leeching Magic Shield Offense Stat. Added new Leech At Full Health Conditional Resource Stat,

NEW API + EVENT HOOKS

7336969

Implemented brand new Resource Threshold fetcher, tracking ALL resources for conditional stats/effects. Desc: Completely Datapack driven.

SaloEater suggested changes Aug 26, 2025

View reviewed changes

SaloEater reviewed Aug 26, 2025

View reviewed changes

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/datapack/SpendThresholdDef.java Outdated Show resolved Hide resolved

Merge branch '1.20-Forge' into 1.20-Forge

62bcd68

Dubledice requested a review from SaloEater August 26, 2025 19:19

SaloEater suggested changes Aug 26, 2025

View reviewed changes

AltimorTASDK reviewed Aug 26, 2025

View reviewed changes

src/main/java/com/robertx22/mine_and_slash/mechanics/thresholds/SpendThresholdSpec.java Outdated Show resolved Hide resolved

Dubledice added 4 commits August 28, 2025 22:34

Merge branch 'mahjerion:1.20-Forge' into 1.20-Forge

2559965

Merge branch 'mahjerion:1.20-Forge' into 1.20-Forge

211c0ff

Suggested Fixes: Datapack Threshold

6ec6525

Merge pull request #1 from Dubledice/pr/suggested-fixes

c0a0476

Suggested Fixes: Datapack Threshold

Dubledice requested a review from SaloEater September 3, 2025 14:11

Dubledice added 2 commits September 3, 2025 10:53

Suggested Fixes: Extension

5aece79

Merge pull request #2 from Dubledice/pr/suggested-fixes

1fdcb7f

Suggested Fixes: Extension

SaloEater suggested changes Sep 3, 2025

View reviewed changes

Dubledice added 2 commits September 3, 2025 14:38

Suggested Fixes: Cleanup

f06bf30

Merge pull request #3 from Dubledice/pr/suggested-fixes

e83314d

Suggested Fixes: Cleanup

Dubledice requested a review from SaloEater September 3, 2025 18:40

SaloEater reviewed Sep 3, 2025

View reviewed changes

SaloEater suggested changes Sep 3, 2025

View reviewed changes

Dubledice added 2 commits September 3, 2025 16:21

Suggested Fixes: SpendThresholdManager.java

0ee8eb6

Merge pull request #4 from Dubledice/pr/suggested-fixes

c364377

Suggested Fixes: SpendThresholdManager.java

Dubledice added 3 commits September 3, 2025 17:02

Suggested Fixes: SpendThresholdSpec.java

26310f1

Suggested Fixes: SpendThresholdSpec.java

5976286

Merge pull request #5 from Dubledice/pr/suggested-fixes

308fd77

Pr/suggested fixes

Dubledice requested a review from SaloEater September 3, 2025 21:18

SaloEater approved these changes Sep 3, 2025

View reviewed changes

-    private static EffectCtx state(String id, String name, Elements elem) {
+    private static EffectCtx state(String id, String name, Elements elem) {
+        return new EffectCtx(id, name, elem, EffectType.beneficial);
+    }
+    private static EffectCtx statePhysical(String id, String name) {
+        return state(id, name, Elements.Physical);
+    }

	rt, state("leeching_" + rt.id + "_state", "Leeching " + nice, null)
	rt, statePhysical("leeching_" + rt.id + "_state", "Leeching " + nice)

	rt, state("regen_" + rt.id + "_state", "Regenerating " + nice, null)
	rt, statePhysical("regen_" + rt.id + "_state", "Regenerating " + nice)


		public boolean showUi() { return showUi; }

		// Perk lock is handled by callers (anonymous subclass) when needed.

	var effect = com.robertx22.mine_and_slash.database.registry.ExileDB.ExileEffects().get(a.effectId);
	var effect = ExileDB.ExileEffects().get(a.effectId);

NEW API + EVENT HOOKS (While_Leeching + Resource Tracker/Thresholds) #29

Are you sure you want to change the base?

NEW API + EVENT HOOKS (While_Leeching + Resource Tracker/Thresholds) #29

Uh oh!

Conversation

Dubledice commented Aug 24, 2025

PR to Parent branch.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mahjerion commented Aug 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Dubledice commented Aug 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

SaloEater commented Aug 28, 2025

Uh oh!

mahjerion commented Aug 28, 2025

Uh oh!

Dubledice commented Sep 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

SaloEater Sep 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Dubledice Sep 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Dubledice Sep 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

mahjerion commented Aug 28, 2025 •

edited

Loading

Dubledice commented Aug 28, 2025 •

edited

Loading

Dubledice commented Sep 3, 2025 •

edited

Loading

SaloEater Sep 3, 2025 •

edited

Loading

Dubledice Sep 3, 2025 •

edited

Loading

Dubledice Sep 3, 2025 •

edited

Loading

SaloEater Sep 3, 2025 •

edited

Loading