Skip to content

Comments

[#6408] Generalize leveled conditions#6699

Open
krbz999 wants to merge 1 commit intofoundryvtt:5.3.xfrom
krbz999:6408-generalize-leveled-conditions
Open

[#6408] Generalize leveled conditions#6699
krbz999 wants to merge 1 commit intofoundryvtt:5.3.xfrom
krbz999:6408-generalize-leveled-conditions

Conversation

@krbz999
Copy link
Contributor

@krbz999 krbz999 commented Feb 6, 2026

Adds a "Condition" ActiveEffect subtype which is responsible for storing levels and other data related to that on the actor.

Generalizes most of the handling surrounding exhaustion. The attributes.exhaustion property remains for backwards compatibility and for the ability to update the Exhaustion condition via an actor update. For other conditions with levels, one can call actor.toggleStatusEffects(statusId, { levels: 1 }).

For example, adding a new condition,

DND5E.conditionTypes.contamination = {
  name: "Contamination",
  img: "modules/drakkenheim-scgd/assets/layout/meteor.svg",
  levels: 6,
  conditions: {
    4: ["weakened"],
    5: ["dazed", "incapacitated"],
  },
};

will add a new status with 6 levels, where "Weakened" (an unrelated example condition) is applied at 4 leves of Contamination or higher, and Dazed and Incapaciated are applied at 5 levels or higher.

The reduction property is also generalized and can be used by any condition, not just leveled ones.

@krbz999 krbz999 force-pushed the 6408-generalize-leveled-conditions branch from f09f33f to 75c3967 Compare February 6, 2026 15:54
@Fyorl Fyorl added system: active effects strategic Requires sign-off from a core Foundry VTT developer before merge. priority: medium labels Feb 6, 2026
@krbz999 krbz999 changed the title [#6508] Generalize leveled conditions [#6408] Generalize leveled conditions Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: medium strategic Requires sign-off from a core Foundry VTT developer before merge. system: active effects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants