Merge branch 'main' into v13

This reverts commit c5308617e1.

README.md

@@ -2,17 +2,11 @@
 
 ## Table of Contents
 
-- [Foundryborne Daggerheart](#foundryborne-daggerheart)
-  - [Table of Contents](#table-of-contents)
 - [Overview](#overview)
-  - [User Install](#user-install)
+- [User Install Guide](#user-install)
 - [Documentation](#documentation)
-  - [Development Setup](#development-setup)
+- [Developer Setup](#development-setup)
-    - [Available Scripts](#available-scripts)
+- [Contribution Info](#contributing)
-    - [Notes](#notes)
-  - [Contributing](#contributing)
-  - [AI Policy](#ai-policy)
-  - [Disclaimer](#disclaimer)
 
 ## Overview
 
@@ -78,7 +72,7 @@ Looking to contribute to the project? Look no further, check out our [contributi
 
 The Foundryborne Daggerheart system does not make use of AI (generative or otherwise) for any area of its implementation. We expect all contributors to follow this same policy when contributing with a pull request; contributions made using AI will be rejected outright.
 
-## Disclaimer
+## Disclaimer:
 
 **Daggerheart System**
 Daggerheart is a trademark of Darrington Press LLC. All original content, mechanics, and intellectual property related to the Daggerheart roleplaying game are © Darrington Press LLC.

daggerheart.d.ts

@@ -4,18 +4,6 @@ import '@common/primitives/global.mjs';
 import Canvas from '@client/canvas/board.mjs';
 import { ResourceUpdateMap } from './module/data/action/baseAction.mjs';
 
-import * as applications from './module/applications/_module.mjs';
-import * as data from './module/data/_module.mjs';
-import * as models from './module/data/_module.mjs';
-import * as documents from './module/documents/_module.mjs';
-import { macros } from './module/_module.mjs';
-import * as dice from './module/dice/_module.mjs';
-import * as fields from './module/data/fields/_module.mjs';
-import { gameSettings } from './module/config/settingsConfig.mjs';
-import DhCountdowns from './module/data/countdowns.mjs';
-import DhAutomation from './module/data/settings/Automation.mjs';
-
-
 // Foundry's use of `Object.assign(globalThis) means many globally available objects are not read as such
 // This declare global hopefully fixes that
 // Note: eslint is not aware of these, whatever is added here should go in the eslint's globals list
@@ -92,27 +80,3 @@ declare global {
         damageOptions: object;
     }
 }
-
-declare module '@client/packages/system.mjs' {
-    export default interface System {
-        api: {
-            applications: typeof applications,
-            data: typeof data,
-            models: typeof models,
-            documents: typeof documents,
-            macros: typeof macros,
-            dice: typeof dice,
-            fields: typeof fields
-        };
-    }
-}
-
-declare module '@client/helpers/client-settings.mjs' {
-    // Add explicit typed overrides for auto complete. These require /** @type {"string"} on the vars themselves to work */
-    export default interface ClientSettings {
-        get(namespace: 'daggerheart', key: typeof gameSettings.Automation): DhAutomation;
-        get(namespace: 'daggerheart', key: typeof gameSettings.Homebrew): DhHomebrew;
-        get(namespace: 'daggerheart', key: typeof gameSettings.Countdowns): DhCountdowns;
-        get(namespace: 'daggerheart', key: string): unknown;
-    }
-}

daggerheart.mjs

@@ -6,7 +6,6 @@ import * as documents from './module/documents/_module.mjs';
 import { macros } from './module/_module.mjs';
 import * as collections from './module/documents/collections/_module.mjs';
 import * as dice from './module/dice/_module.mjs';
-import * as die from './module/dice/die/_module.mjs';
 import * as fields from './module/data/fields/_module.mjs';
 import RegisterHandlebarsHelpers from './module/helpers/handlebarsHelper.mjs';
 import { enricherConfig, enricherRenderSetup } from './module/enrichers/_module.mjs';
@@ -24,7 +23,7 @@ import TokenManager from './module/documents/tokenManager.mjs';
 CONFIG.DH = SYSTEM;
 CONFIG.TextEditor.enrichers.push(...enricherConfig);
 
-CONFIG.Dice.rolls = [Roll = BaseRoll, DHRoll, DualityRoll, D20Roll, DamageRoll, FateRoll];
+CONFIG.Dice.rolls = [BaseRoll, DHRoll, DualityRoll, D20Roll, DamageRoll, FateRoll];
 CONFIG.Dice.daggerheart = {
     DHRoll: DHRoll,
     DualityRoll: DualityRoll,
@@ -39,8 +38,6 @@ CONFIG.RegionBehavior.dataModels = {
 };
 
 Object.assign(CONFIG.Dice.termTypes, dice.diceTypes);
-CONFIG.Dice.terms.d = die.BaseDie;
-CONFIG.Dice.types = [die.BaseDie, CONFIG.Dice.terms.f];
 
 CONFIG.Actor.documentClass = documents.DhpActor;
 CONFIG.Actor.dataModels = models.actors.config;
@@ -270,10 +267,6 @@ Hooks.on('i18nInit', () => {
 });
 
 Hooks.on('setup', () => {
-    if (game.user.isGM) {
-        document.body.dataset.gm = true;
-    }
-
     CONFIG.statusEffects = [
         ...CONFIG.statusEffects.filter(x => !['dead', 'unconscious'].includes(x.id)),
         ...Object.values(SYSTEM.GENERAL.conditions()).map(x => ({
@@ -397,7 +390,7 @@ const updateActorsRangeDependentEffects = async token => {
     ).rangeMeasurement;
 
     for (let effect of token.actor?.allApplicableEffects() ?? []) {
-        if (!effect.system.rangeDependence) continue;
+        if (!effect.system.rangeDependence?.enabled) continue;
         const { target, range, type } = effect.system.rangeDependence;
 
         // If there are no targets, assume false. Otherwise, start with the effect enabled.

lang/en.json

@@ -84,7 +84,6 @@
                     "transformActorMissing": "The assigned actor to transform into does not exist. It was probably deleted or moved in/out of a compendium",
                     "canvasError": "There is no active scene.",
                     "prototypeError": "You can only use a transform action from a Token",
-                    "linkedSelectedError": "To transform a linked actor there either needs to be only a single token of it on the canvas, or you need to left-click select only one of them.",
                     "actorLinkError": "You cannot transform a token with Actor Link set to true"
                 }
             },
@@ -104,10 +103,8 @@
                     "startCountdown": "Start Countdown"
                 },
                 "damage": {
-                    "addResource": "Add Resource",
                     "multiplier": "Multiplier",
-                    "flatMultiplier": "Flat Multiplier",
+                    "flatMultiplier": "Flat Multiplier"
-                    "markResources": "Mark Resources"
                 },
                 "general": {
                     "customFormula": "Custom Formula",
@@ -116,8 +113,7 @@
                 "area": {
                     "sectionTitle": "Areas",
                     "shape": "Shape",
-                    "size": "Size",
+                    "size": "Size"
-                    "hasHole": "Area Hole"
                 },
                 "displayInChat": "Display in chat",
                 "deleteTriggerTitle": "Delete Trigger",
@@ -126,15 +122,6 @@
                 "damageOnSave": "Damage on Save",
                 "useDefaultItemValues": "Use default Item values"
             },
-            "Reload": {
-                "checkReload": "Check Reload",
-                "reloadRequired": "Reload Required!",
-                "notRolled": "Not Rolled",
-                "checkFailed": "Check Failed",
-                "checkPassed": "Check Passed",
-                "rerollConfirmationTitle": "Reroll Reload Check",
-                "rerollConfirmationText": "Are you sure you want to reload the reload check?"
-            },
             "RollField": {
                 "diceRolling": {
                     "compare": "Should be",
@@ -178,7 +165,9 @@
         },
         "ACTIVEEFFECT": {
             "Config": {
-                "rangeDependence": { "title": "Range Dependence" },
+                "rangeDependence": {
+                    "title": "Range Dependence"
+                },
                 "stacking": { "title": "Stacking" },
                 "targetDispositions": "Affected Dispositions"
             },
@@ -783,7 +772,7 @@
                 "removeRoll": "Remove Roll",
                 "resultsHint": "Results will appear when characters roll",
                 "selectLeaderHint": "Select one Character to be the leader",
-                "selectParticipantsHint": "Select all Characters who will participate in the roll",
+                "selectParticipantsHint": "Select one Character to be the leader",
                 "startGroupRoll": "Start Group Roll",
                 "title": "Group Roll"
             },
@@ -881,10 +870,6 @@
                 "bruiser": "for each Bruiser adversary.",
                 "solo": "for each Solo adversary."
             },
-            "AreaTypes": {
-                "attached": { "label": "Attached To Token" },
-                "placed": { "label": "Placed In Scene" }
-            },
             "ArmorInteraction": {
                 "none": { "label": "Ignores Armor" },
                 "active": { "label": "Active w/ Armor" },
@@ -1331,11 +1316,6 @@
                     "short": "V. Far"
                 }
             },
-            "ReloadChoices": {
-                "off": { "label": "Don't Use" },
-                "manual": { "label": "Manual" },
-                "auto": { "label": "Automatic" }
-            },
             "RollTypes": {
                 "trait": {
                     "name": "Trait"
@@ -2235,9 +2215,7 @@
             },
             "Resource": {
                 "single": "Resource",
-                "plural": "Resources",
+                "plural": "Resources"
-                "unloaded": "The weapon is not loaded",
-                "loaded": "The weapon is loaded"
             },
             "Roll": {
                 "attack": "Attack Roll",
@@ -2263,7 +2241,7 @@
                         "hint": "A used armor slot normally reduces damage by one step. This value increases the number of steps damage is reduced by."
                     },
                     "magical": {
-                        "label": "Damage Reduction: Only Magical",
+                        "label": "Daamge Reduction: Only Magical",
                         "hint": "Armor can only be used to reduce magical damage"
                     },
                     "maxArmorMarkedBonus": "Max Armor Used",
@@ -2501,6 +2479,7 @@
             "reroll": "Reroll",
             "rerolled": "Rerolled",
             "rerollThing": "Reroll {thing}",
+            "resource": "Resource",
             "result": {
                 "single": "Result",
                 "plural": "Results"
@@ -2567,9 +2546,6 @@
                 },
                 "identifier": {
                     "label": "Identifier"
-                },
-                "gmNotes": {
-                    "label": "GM Notes"
                 }
             },
             "Ancestry": {
@@ -2585,9 +2561,6 @@
                     "severe": "Severe Threshold"
                 }
             },
-            "Base": {
-                "addGMNote": "Add GM Note"
-            },
             "Beastform": {
                 "FIELDS": {
                     "beastformType": { "label": "Beastform Type" },
@@ -2704,9 +2677,6 @@
                     "displayFear": {
                         "label": "Display Fear"
                     },
-                    "fearPosition": {
-                        "label": "Fear Position"
-                    },
                     "displayCountdownUI": {
                         "label": "Display Countdown UI"
                     },
@@ -2758,13 +2728,6 @@
                     "token": "Tokens",
                     "bar": "Bar",
                     "hide": "Hide"
-                },
-                "fearPosition": {
-                    "free": "Free",
-                    "topCenter": "Top + Center",
-                    "bottomCenter": "Bottom + Center",
-                    "rightTop": "Right + Top",
-                    "leftBottom": "Left + Bottom"
                 }
             },
             "Automation": {
@@ -2823,10 +2786,6 @@
                             "hint": "Effects with defined range dependency will automatically turn on/off depending on range"
                         }
                     },
-                    "reload": {
-                        "label": "Reload Checking",
-                        "hint": "If the system should automatically roll reload checks or wait for the manual press of a button in chat"
-                    },
                     "resourceScrollTexts": {
                         "label": "Show Resource Change Scrolltexts",
                         "hint": "When a character is damaged, uses armor etc, a scrolling text will briefly appear by the token to signify this."
@@ -3270,7 +3229,7 @@
                 "beastformToManyAdvantages": "You cannot select any more advantages.",
                 "beastformToManyFeatures": "You cannot select any more features.",
                 "beastformEquipWeapon": "You cannot use weapons while in a Beastform.",
-                "loadoutMaxReached": "You've reached the maximum loadout size. To add more, move at least one domain card to the vault.",
+                "loadoutMaxReached": "You've reached maximum loadout. Move atleast one domain card to the vault, or increase the limit in homebrew settings if desired.",
                 "domainMaxReached": "You've reached the maximum domains for the class. Increase the limit in homebrew settings if desired.",
                 "insufficientResources": "You don't have enough resources to use that action.",
                 "actionNoUsesRemaining": "That action doesn't have remaining uses.",
@@ -3288,8 +3247,7 @@
                 "knowTheTide": "Know The Tide gained a token",
                 "lackingItemTransferPermission": "User {user} lacks owner permission needed to transfer items to {target}",
                 "noTokenTargeted": "No token is targeted",
-                "behaviorRegionRequiresGM": "Creating a Region with an attached Behavior requires an online GM",
+                "behaviorRegionRequiresGM": "Creating a Region with an attached Behavior requires an online GM"
-                "reloadRequired": "The {weapon} must be reloaded to be used!"
             },
             "Progress": {
                 "migrationLabel": "Performing system migration. Please wait and do not close Foundry."

module/applications/dialogs/beastformDialog.mjs

@@ -315,15 +315,15 @@ export default class BeastformDialog extends HandlebarsApplicationMixin(Applicat
 
                     const beastformEffect = selected.effects.find(x => x.type === 'beastform');
                     for (const traitBonus of app.modifications.traitBonuses) {
-                        const existingChange = beastformEffect.system.changes.find(
+                        const existingChange = beastformEffect.changes.find(
                             x => x.key === `system.traits.${traitBonus.trait}.value`
                         );
                         if (existingChange) {
                             existingChange.value = Number.parseInt(existingChange.value) + traitBonus.bonus;
                         } else {
-                            beastformEffect.system.changes.push({
+                            beastformEffect.changes.push({
                                 key: `system.traits.${traitBonus.trait}.value`,
-                                type: 'add',
+                                mode: 2,
                                 priority: null,
                                 value: traitBonus.bonus
                             });

module/applications/dialogs/damageDialog.mjs

@@ -51,11 +51,7 @@ export default class DamageDialog extends HandlebarsApplicationMixin(Application
         const context = await super._prepareContext(_options);
         context.config = CONFIG.DH;
         context.title = this.config.title ?? this.title;
-
+        context.formula = this.roll.constructFormula(this.config);
-        const { damageFormula, resourceFormulas } = this.roll.constructFormulas(this.config); 
-        context.damageFormula = damageFormula;
-        context.resourceFormulas = resourceFormulas;
-
         context.hasHealing = this.config.hasHealing;
         context.directDamage = this.config.directDamage;
         context.selectedMessageMode = this.config.selectedMessageMode;
@@ -77,11 +73,7 @@ export default class DamageDialog extends HandlebarsApplicationMixin(Application
 
     static updateRollConfiguration(_event, _, formData) {
         const data = foundry.utils.expandObject(formData.object);
-
+        foundry.utils.mergeObject(this.config.roll, data.roll);
-        if (this.config.damageFormula) 
-            foundry.utils.mergeObject(this.config.damageFormula, data.damageFormula);
-        
-        foundry.utils.mergeObject(this.config.resourceFormulas, data.resourceFormulas);
         foundry.utils.mergeObject(this.config.modifiers, data.modifiers);
         this.config.selectedMessageMode = data.selectedMessageMode;
 

module/applications/dialogs/deathMove.mjs

@@ -139,16 +139,14 @@ export default class DhDeathMove extends HandlebarsApplicationMixin(ApplicationV
                 name: game.i18n.localize('DAGGERHEART.CONFIG.DeathMoves.blazeOfGlory.name'),
                 description: game.i18n.localize('DAGGERHEART.CONFIG.DeathMoves.blazeOfGlory.description'),
                 img: CONFIG.DH.GENERAL.deathMoves.blazeOfGlory.img,
-                system: {
                 changes: [
                     {
                         key: 'system.rules.roll.guaranteedCritical',
-                            type: 'add',
+                        mode: 2,
                         value: 'true'
                     }
                 ]
             }
-            }
         ]);
 
         await this.actor.setDeathMoveDefeated(CONFIG.DH.GENERAL.defeatedConditionChoices.dead.id);
@@ -185,6 +183,8 @@ export default class DhDeathMove extends HandlebarsApplicationMixin(ApplicationV
 
         if (result === undefined) return;
 
+        const autoExpandDescription = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.appearance)
+            .expandRollMessage?.desc;
         const cls = getDocumentClass('ChatMessage');
 
         const msg = {
@@ -200,6 +200,7 @@ export default class DhDeathMove extends HandlebarsApplicationMixin(ApplicationV
                     img: this.selectedMove.img,
                     description: game.i18n.localize(this.selectedMove.description),
                     result: result,
+                    open: autoExpandDescription ? 'open' : '',
                     showRiskItAllButton: this.showRiskItAllButton,
                     riskItAllButtonLabel: this.riskItAllButtonLabel,
                     riskItAllHope: this.riskItAllHope

module/applications/dialogs/downtime.mjs

@@ -196,6 +196,9 @@ export default class DhpDowntime extends HandlebarsApplicationMixin(ApplicationV
             .filter(x => x.testUserPermission(game.user, 'LIMITED'))
             .filter(x => x.uuid !== this.actor.uuid);
 
+        const autoExpandDescription = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.appearance)
+            .expandRollMessage?.desc;
+
         const cls = getDocumentClass('ChatMessage');
         const msg = {
             user: game.user.id,
@@ -216,7 +219,8 @@ export default class DhpDowntime extends HandlebarsApplicationMixin(ApplicationV
                     actor: { name: this.actor.name, img: this.actor.img },
                     moves: moves,
                     characters: characters,
-                    selfId: this.actor.uuid
+                    selfId: this.actor.uuid,
+                    open: autoExpandDescription ? 'open' : ''
                 }
             ),
             flags: {

module/applications/dialogs/groupRollDialog.mjs

@@ -1,7 +1,7 @@
 import { ResourceUpdateMap } from '../../data/action/baseAction.mjs';
 import { shouldUseHopeFearAutomation } from '../../helpers/utils.mjs';
 import { emitGMUpdate, GMUpdateEvent, RefreshType, socketEvent } from '../../systemRegistration/socket.mjs';
-import PartySheet from '../sheets/actors/party.mjs';
+import Party from '../sheets/actors/party.mjs';
 
 const { HandlebarsApplicationMixin, ApplicationV2 } = foundry.applications.api;
 
@@ -11,7 +11,7 @@ export default class GroupRollDialog extends HandlebarsApplicationMixin(Applicat
 
         this.party = party;
         this.partyMembers = party.system.partyMembers
-            .filter(x => PartySheet.DICE_ROLL_ACTOR_TYPES.includes(x.type))
+            .filter(x => Party.DICE_ROLL_ACTOR_TYPES.includes(x.type))
             .map(member => ({
                 ...member.toObject(),
                 uuid: member.uuid,

module/applications/dialogs/tagTeamDialog.mjs

@@ -1,9 +1,8 @@
 import { ResourceUpdateMap } from '../../data/action/baseAction.mjs';
-import { ChatDamageData } from '../../data/chat-message/chatDamageData.mjs';
 import { MemberData } from '../../data/tagTeamData.mjs';
 import { getCritDamageBonus, shouldUseHopeFearAutomation } from '../../helpers/utils.mjs';
 import { emitGMUpdate, GMUpdateEvent, RefreshType, socketEvent } from '../../systemRegistration/socket.mjs';
-import PartySheet from '../sheets/actors/party.mjs';
+import Party from '../sheets/actors/party.mjs';
 
 const { HandlebarsApplicationMixin, ApplicationV2 } = foundry.applications.api;
 
@@ -14,7 +13,7 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
         this.usesTagTeamHopeCost = true;
         this.party = party;
         this.partyMembers = party.system.partyMembers
-            .filter(x => PartySheet.DICE_ROLL_ACTOR_TYPES.includes(x.type))
+            .filter(x => Party.DICE_ROLL_ACTOR_TYPES.includes(x.type))
             .map(member => ({
                 ...member.toObject(),
                 uuid: member.uuid,
@@ -141,7 +140,7 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
             const hasRolled = Boolean(data.rollData);
             if (!hasRolled) return false;
 
-            return !data.rollData.options.hasDamage || data.damageRollData.active;
+            return !data.rollData.options.hasDamage || Boolean(data.rollData.options.damage);
         });
 
         return context;
@@ -182,7 +181,7 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
                 const selectedRoll = Object.values(this.party.system.tagTeam.members).find(member => member.selected);
                 const critSelected = !selectedRoll
                     ? undefined
-                    : (selectedRoll?.roll?.isCritical ?? false);
+                    : (selectedRoll?.rollData?.options?.roll?.isCritical ?? false);
 
                 partContext.hintText = await this.getInfoTexts(this.party.system.tagTeam.members);
                 partContext.joinedRoll = await this.getJoinedRoll({
@@ -209,16 +208,6 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
 
         const rollOptions = [];
         const damageRollOptions = [];
-
-        if (actor?.system.usedUnarmed) {
-            damageRollOptions.push({
-                value: actor.system.attack.uuid,
-                label: actor.system.usedUnarmed.name,
-                group: actor.name,
-                baseAction: actor.system.attack
-            });
-        }
-
         for (const item of actor?.items ?? []) {
             if (!item.system.metadata.hasActions) continue;
             const actions = [...item.system.actions, ...(item.system.attack ? [item.system.attack] : [])];
@@ -236,7 +225,8 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
         }
 
         const selectedRoll = Object.values(this.party.system.tagTeam.members).find(member => member.selected);
-        const critSelected = !selectedRoll ? undefined : (selectedRoll?.roll?.isCritical ?? false);
+        const critSelected = !selectedRoll ? undefined : (selectedRoll?.rollData?.options?.roll?.isCritical ?? false);
+        const damage = data.rollData?.options?.damage;
 
         return {
             ...data,
@@ -247,9 +237,9 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
             hasRolled: Boolean(data.rollData),
             rollOptions,
             damageRollOptions,
-            damage: data.damageRollData,
+            damage: damage,
-            critDamage: await this.getCriticalDamage(data.damageRollData),
+            critDamage: await this.getCriticalDamage(damage),
-            useCritDamage: critSelected || (critSelected === undefined && data.roll?.isCritical)
+            useCritDamage: critSelected || (critSelected === undefined && data.rollData?.options?.roll?.isCritical)
         };
     }
 
@@ -379,7 +369,7 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
         let rollIsSelected = false;
         for (const member of Object.values(members)) {
             const rollFinished = Boolean(member.rollData);
-            const damageFinished = member.rollData?.options?.hasDamage ? member.damageRollData.active : true;
+            const damageFinished = member.rollData?.options?.hasDamage ? Boolean(member.rollData.options.damage) : true;
 
             rollsAreFinished = rollsAreFinished && rollFinished && damageFinished;
             rollIsSelected = rollIsSelected || member.selected;
@@ -541,9 +531,16 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
         await action.workflow.get('damage').execute(config, null, true);
         if (!config.damage) return;
 
+        const current = this.party.system.tagTeam.members[memberKey].rollData;
         await this.updatePartyData(
             {
-                [`system.tagTeam.members.${memberKey}.damageRollData`]: config.damage
+                [`system.tagTeam.members.${memberKey}.rollData`]: {
+                    ...current,
+                    options: {
+                        ...current.options,
+                        damage: config.damage
+                    }
+                }
             },
             this.getUpdatingParts(button)
         );
@@ -551,11 +548,15 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
 
     static async #removeDamageRoll(_, button) {
         const { memberKey } = button.dataset;
+        const current = this.party.system.tagTeam.members[memberKey].rollData;
         this.updatePartyData(
             {
-                [`system.tagTeam.members.${memberKey}.damageRollData`]: {
+                [`system.tagTeam.members.${memberKey}.rollData`]: {
-                    main: null,
+                    ...current,
-                    resources: _replace({})
+                    options: {
+                        ...current.options,
+                        damage: null
+                    }
                 }
             },
             this.getUpdatingParts(button)
@@ -563,35 +564,81 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
     }
 
     static async #rerollDamageDice(_, button) {
-        const { isResource, memberKey, damageKey, diceIndex, resultIndex } = button.dataset;
+        const { memberKey, damageKey, part, dice } = button.dataset;
         const memberData = this.party.system.tagTeam.members[memberKey];
-        await memberData.damageRollData.rerollDamageDie(isResource, damageKey, diceIndex, resultIndex);
+        const partData = memberData.rollData.options.damage[damageKey].parts[part];
+        const activeDiceResultKey = Object.keys(partData.dice[dice].results).find(
+            index => partData.dice[dice].results[index].active
+        );
+        const { parsedRoll, rerolledDice } = await game.system.api.dice.DamageRoll.reroll(
+            partData,
+            dice,
+            activeDiceResultKey
+        );
+
+        const rollData = this.party.system.tagTeam.members[memberKey].rollData;
+        rollData.options.damage[damageKey].parts = rollData.options.damage[damageKey].parts.map((damagePart, index) => {
+            if (index !== Number.parseInt(part)) return damagePart;
+
+            return {
+                ...damagePart,
+                total: parsedRoll.total,
+                dice: rerolledDice
+            };
+        });
+        rollData.options.damage[damageKey].total = rollData.options.damage[damageKey].parts.reduce((acc, part) => {
+            acc += part.total;
+            return acc;
+        }, 0);
 
-        const basePath = `system.tagTeam.members.${memberKey}.damageRollData`;
-        const updatePath = isResource ? `${basePath}.resources.${damageKey}` : `${basePath}.main`;
-        const updateValue = isResource ? 
-            memberData.damageRollData.resources[damageKey] : memberData.damageRollData.main;
         this.updatePartyData(
             {
-                [updatePath]: updateValue.toJSON()
+                [`system.tagTeam.members.${memberKey}.rollData`]: rollData
             },
             this.getUpdatingParts(button)
         );
     }
 
-    async getCriticalDamage(origDamage) {
+    async getCriticalDamage(damage) {
-        const newDamage = origDamage ? ChatDamageData.fromJSON(JSON.stringify(origDamage)) : null;
+        const newDamage = foundry.utils.deepClone(damage);
-        if (newDamage?.main) {
+        for (let key in newDamage) {
-            const criticalDamage = await getCritDamageBonus(newDamage.main.formula);
+            var damage = newDamage[key];
+            damage.formula = '';
+            damage.total = 0;
+
+            for (let part of damage.parts) {
+                const criticalDamage = await getCritDamageBonus(part.formula);
                 if (criticalDamage) {
-                const criticalTerm = new foundry.dice.terms.NumericTerm({ number: criticalDamage, evaluated: true });
+                    part.modifierTotal += criticalDamage;
-                criticalTerm.evaluate();
+                    part.total += criticalDamage;
-                newDamage.main = await Roll.fromTerms([
+                    part.formula = `${part.dice.map(x => x.formula).join(' + ')} + ${part.modifierTotal}`;
-                    ...origDamage.main.terms,
+                    part.roll = new Roll(part.formula);
-                    new foundry.dice.terms.OperatorTerm({ operator: '+' }),
+                }
-                    criticalTerm
+
-                ]);
+                damage.formula = [damage.formula, part.formula].filter(x => x).join(' + ');
-                newDamage.main.options = foundry.utils.deepClone(origDamage.main.options);
+                damage.total += part.total;
+            }
+        }
+
+        return newDamage;
+    }
+
+    async getNonCriticalDamage(config) {
+        const newDamage = foundry.utils.deepClone(config.damage);
+        for (let key in newDamage) {
+            var damage = newDamage[key];
+            damage.formula = '';
+            damage.total = 0;
+
+            for (let part of damage.parts) {
+                const critDamageBonus = await getCritDamageBonus(part.formula);
+                part.modifierTotal -= critDamageBonus;
+                part.total -= critDamageBonus;
+                part.formula = `${part.dice.map(x => x.formula).join(' + ')} + ${part.modifierTotal}`;
+                part.roll = new Roll(part.formula);
+
+                damage.formula = [damage.formula, part.formula].filter(x => x).join(' + ');
+                damage.total += part.total;
             }
         }
 
@@ -600,7 +647,6 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
 
     static async #selectRoll(_, button) {
         const { memberKey } = button.dataset;
-        
         this.updatePartyData(
             {
                 [`system.tagTeam.members`]: Object.entries(this.party.system.tagTeam.members).reduce(
@@ -611,12 +657,7 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
                     {}
                 )
             },
-            /* Selecting a roll must update all member sections hbs to display the correct damage information incase of a critical */
+            this.getUpdatingParts(button)
-            [ 
-                ...Object.keys(this.party.system.tagTeam.members),
-                this.constructor.PARTS.rollSelection.id,
-                this.constructor.PARTS.result.id
-            ]
         );
     }
 
@@ -634,65 +675,29 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
             if (!baseMainRoll?.rollData || !baseSecondaryRoll) return null;
 
             const mainRoll = new MemberData(baseMainRoll.toObject());
-            mainRoll.damageRollData = baseMainRoll.damageRollData ? 
+            const secondaryRollData = new MemberData(baseSecondaryRoll.toObject()).rollData;
-                ChatDamageData.fromJSON(JSON.stringify(baseMainRoll.damageRollData)) : null;
+            const systemData = mainRoll.rollData.options;
-            const secondaryRoll = new MemberData(baseSecondaryRoll.toObject());
+            const isCritical = overrideIsCritical ?? systemData.roll.isCritical;
-            secondaryRoll.damageRollData = baseSecondaryRoll.damageRollData ? 
+            if (isCritical) systemData.damage = await this.getCriticalDamage(systemData.damage);
-                ChatDamageData.fromJSON(JSON.stringify(baseSecondaryRoll.damageRollData)) : null;
 
-            const isCritical = overrideIsCritical ?? mainRoll.roll.isCritical;
+            if (secondaryRollData?.options.hasDamage) {
-            if (isCritical) mainRoll.damageRollData = await this.getCriticalDamage(mainRoll.damageRollData);
-
-            if (secondaryRoll.damageRollData) {
                 const secondaryDamage = (displayVersion ? overrideIsCritical : isCritical)
-                    ? await this.getCriticalDamage(secondaryRoll.damageRollData)
+                    ? await this.getCriticalDamage(secondaryRollData.options.damage)
-                    : secondaryRoll.damageRollData;
+                    : secondaryRollData.options.damage;
-                if (mainRoll.damageRollData) {
+                if (systemData.damage) {
-                    if (secondaryDamage.main) {
+                    for (const [key, damage] of Object.entries(secondaryDamage ?? {})) {
-                        if (mainRoll.damageRollData.main) {
+                        if (key in systemData.damage) {
-                            mainRoll.damageRollData.main = Roll.fromTerms([
+                            systemData.damage[key].formula = [systemData.damage[key]?.formula, damage.formula]
-                                ...baseMainRoll.damageRollData.main.terms,
+                                .filter(x => x)
-                                new foundry.dice.terms.OperatorTerm({ operator: '+' }),
+                                .join(' + ');
-                                ...baseSecondaryRoll.damageRollData.main.terms
+                            systemData.damage[key].total += damage.total;
-                            ]);
+                            systemData.damage[key].parts.push(...damage.parts);
-
-                            /* Joining the roll.options of both rolls */
-                            const joinedDamageTypes = new Set([
-                                ...baseMainRoll.damageRollData.main.options.damageTypes,
-                                ...baseSecondaryRoll.damageRollData.main.options.damageTypes
-                            ]);
-                            mainRoll.damageRollData.main.options = {
-                                ...baseMainRoll.damageRollData.main.options,
-                                damageTypes: [...joinedDamageTypes]
-                            };
                         } else {
-                            mainRoll.damageRollData.main = secondaryDamage.main;
+                            systemData.damage[key] = damage;
-                        }
-                    }
-
-                    for (const [key, damage] of Object.entries(secondaryDamage.resources ?? {})) {
-                        if (key in mainRoll.damageRollData.resources) {
-                            mainRoll.damageRollData.resources[key] = Roll.fromTerms([
-                                ...baseMainRoll.damageRollData.resources[key].terms,
-                                new foundry.dice.terms.OperatorTerm({ operator: '+' }),
-                                ...baseSecondaryRoll.damageRollData.resources[key].terms
-                            ]);
-
-                            /* Joining the roll.options of both rolls */
-                            const joinedDamageTypes = new Set([
-                                ...baseMainRoll.damageRollData.resources[key].options.damageTypes,
-                                ...baseSecondaryRoll.damageRollData.resources[key].options.damageTypes
-                            ]);
-                            mainRoll.damageRollData.resources[key].options = {
-                                ...baseMainRoll.damageRollData.resources[key].options,
-                                damageTypes: [...joinedDamageTypes]
-                            };
-                        } else {
-                            mainRoll.damageRollData.resources[key] = damage;
                         }
                     }
                 } else {
-                    mainRoll.damageRollData = secondaryDamage;
+                    systemData.damage = secondaryDamage;
                 }
             }
 
@@ -747,27 +752,13 @@ export default class TagTeamDialog extends HandlebarsApplicationMixin(Applicatio
 
         const mainActor = this.party.system.partyMembers.find(x => x.uuid === mainRoll.options.source.actor);
         mainRoll.options.title = game.i18n.localize('DAGGERHEART.APPLICATIONS.TagTeamSelect.chatMessageRollTitle');
-        
-        /* This could assumably be done better. For some reason rolls don't get correctly done through rollData.toJSON */
-        const systemData = {
-            ...mainRoll.options,
-            damage: joinedRoll.damageRollData?.toJSON()
-        };
-
-        if (joinedRoll.damageRollData.main) {
-            systemData.damage.main = joinedRoll.damageRollData.toJSON();
-        }
-        for (const type of Object.keys(joinedRoll.damageRollData?.resources ?? {})) {
-            systemData.damage.resources[type] = joinedRoll.damageRollData.resources[type].toJSON();
-        }
-
         const cls = getDocumentClass('ChatMessage'),
             msgData = {
                 type: 'dualityRoll',
                 user: game.user.id,
                 title: game.i18n.localize('DAGGERHEART.APPLICATIONS.TagTeamSelect.title'),
                 speaker: cls.getSpeaker({ actor: mainActor }),
-                system: systemData,
+                system: mainRoll.options,
                 rolls: [JSON.stringify(joinedRoll.roll)],
                 sound: null,
                 flags: { core: { RollTable: true } }

module/applications/sheets-configs/action-base-config.mjs

@@ -1,4 +1,4 @@
-import { DHDamageData } from '../../data/fields/action/damageField.mjs';
+import { getUnusedDamageTypes } from '../../helpers/utils.mjs';
 import DaggerheartSheet from '../sheets/daggerheart-sheet.mjs';
 
 const { ApplicationV2 } = foundry.applications.api;
@@ -31,10 +31,8 @@ export default class DHActionBaseConfig extends DaggerheartSheet(ApplicationV2)
             removeElement: this.removeElement,
             removeTransformActor: this.removeTransformActor,
             editEffect: this.editEffect,
-            addDamage: this.#onAddDamage,
+            addDamage: this.addDamage,
-            removeDamage: this.#onRemoveDamage,
+            removeDamage: this.removeDamage,
-            addDamageResource: this.#onAddDamageResource,
-            removeDamageResource: this.#onRemoveDamageResource,
             editDoc: this.editDoc,
             addTrigger: this.addTrigger,
             removeTrigger: this.removeTrigger,
@@ -159,9 +157,9 @@ export default class DHActionBaseConfig extends DaggerheartSheet(ApplicationV2)
         context.tabs = this._getTabs(this.constructor.TABS);
         context.config = CONFIG.DH;
         if (this.action.damage) {
-            const allKeys = Object.keys(CONFIG.DH.GENERAL.healingTypes);
+            context.allDamageTypesUsed = !getUnusedDamageTypes(this.action.damage.parts).length;
-            context.allDamageTypesUsed = allKeys.every(k => k in this.action._source.damage.resources);
+
-            if (this.action.damage?.main?.hasOwnProperty('includeBase') && this.action.type === 'attack')
+            if (this.action.damage.hasOwnProperty('includeBase') && this.action.type === 'attack')
                 context.hasBaseDamage = !!this.action.parent.attack;
         }
 
@@ -255,7 +253,7 @@ export default class DHActionBaseConfig extends DaggerheartSheet(ApplicationV2)
         const submitData = this._prepareSubmitData(event, formData);
 
         const data = foundry.utils.mergeObject(this.action.toObject(), submitData);
-        this.action = (await this.action.update(data)) ?? this.action;
+        this.action = await this.action.update(data);
 
         this.sheetUpdate?.(this.action);
         this.render();
@@ -301,70 +299,53 @@ export default class DHActionBaseConfig extends DaggerheartSheet(ApplicationV2)
         this.constructor.updateForm.bind(this)(null, null, { object: foundry.utils.flattenObject(data) });
     }
 
-    /** @this DHActionBaseConfig */
+    static addDamage(_event) {
-    static #onAddDamage() {
+        if (!this.action.damage.parts) return;
-        if (!this.action.damage || this.action.damage?.main) return;
 
-        const data = this.action.toObject();
+        const choices = getUnusedDamageTypes(this.action._source.damage.parts);
-        data.damage.main = {
-            ...DHDamageData.schema.getInitialValue(),
-            applyTo: 'hitPoints',
-            type: 'physical'
-        };
-        this.constructor.updateForm.bind(this)(null, null, { object: foundry.utils.flattenObject(data) });
-    }
-
-    /** @this DHActionBaseConfig */
-    static #onRemoveDamage() {
-        if (!this.action.damage?.main) return;
-        const data = this.action.toObject();
-        data.damage.main = null;
-        this.constructor.updateForm.bind(this)(null, null, { object: foundry.utils.flattenObject(data) });
-    }
-
-    /** @this DHActionBaseConfig */
-    static #onAddDamageResource(_event) {
-        if (!this.action.damage) return;
-
-        const allKeys = Object.keys(CONFIG.DH.GENERAL.healingTypes);
-        const unused = allKeys.filter(k => !(k in this.action._source.damage.resources));
-        const choices = unused.map(k => ({ value: k, label: _loc(CONFIG.DH.GENERAL.healingTypes[k].label) }));
         const content = new foundry.data.fields.StringField({
-            label: _loc('DAGGERHEART.GENERAL.Resource.single'),
+            label: game.i18n.localize('Damage Type'),
             choices,
             required: true
-        }).toFormGroup({}, {
+        }).toFormGroup(
+            {},
+            {
                 name: 'type',
                 localize: true,
                 nameAttr: 'value',
                 labelAttr: 'label'
-        }).outerHTML;
+            }
+        ).outerHTML;
 
         const callback = (_, button) => {
             const data = this.action.toObject();
             const type = choices[button.form.elements.type.value].value;
-            data.damage.resources[type] = {
+            const part = this.action.schema.fields.damage.fields.parts.element.getInitialValue();
-                ...this.action.schema.fields.damage.fields.resources.element.getInitialValue(),
+            part.applyTo = type;
-                applyTo: type
+            if (type === CONFIG.DH.GENERAL.healingTypes.hitPoints.id)
-            };
+                part.type = this.action.schema.fields.damage.fields.parts.element.fields.type.element.initial;
+
+            data.damage.parts[type] = part;
             this.constructor.updateForm.bind(this)(null, null, { object: foundry.utils.flattenObject(data) });
         };
 
         const typeDialog = new foundry.applications.api.DialogV2({
             buttons: [
+                foundry.utils.mergeObject(
                     {
                         action: 'ok',
                         label: 'Confirm',
                         icon: 'fas fa-check',
-                    default: true,
+                        default: true
-                    callback
+                    },
-                }
+                    { callback: callback }
+                )
             ],
             content: content,
             rejectClose: false,
             modal: false,
             window: {
-                title: _loc('DAGGERHEART.ACTIONS.Config.damage.addResource')
+                title: game.i18n.localize('Add Damage')
             },
             position: { width: 300 }
         });
@@ -372,12 +353,12 @@ export default class DHActionBaseConfig extends DaggerheartSheet(ApplicationV2)
         typeDialog.render(true);
     }
 
-    /** @this DHActionBaseConfig */
+    static removeDamage(_event, button) {
-    static #onRemoveDamageResource(_event, button) {
+        if (!this.action.damage.parts) return;
-        if (!this.action.damage?.resources) return;
         const data = this.action.toObject();
         const key = button.dataset.key;
-        data.damage.resources[key] = _del;
+        delete data.damage.parts[key];
+        data.damage.parts[`${key}`] = _del;
         this.constructor.updateForm.bind(this)(null, null, { object: foundry.utils.flattenObject(data) });
     }
 

module/applications/sheets-configs/activeEffectConfig.mjs

@@ -155,10 +155,6 @@ export default class DhActiveEffectConfig extends foundry.applications.sheets.Ac
             .querySelector('.stacking-change-checkbox')
             ?.addEventListener('change', this.stackingChangeToggle.bind(this));
 
-        htmlElement
-            .querySelector('.range-dependence-change-checkbox')
-            ?.addEventListener('change', this.rangeDependenceChangeToggle.bind(this));
-
         htmlElement
             .querySelector('.armor-change-checkbox')
             ?.addEventListener('change', this.armorChangeToggle.bind(this));
@@ -228,20 +224,6 @@ export default class DhActiveEffectConfig extends foundry.applications.sheets.Ac
         return this.submit({ updateData: { system: systemData } });
     }
 
-    rangeDependenceChangeToggle(event) {
-        const rangeFields = this.document.system.schema.fields.rangeDependence.fields;
-        const systemData = {
-            rangeDependence: event.target.checked
-                ? _replace({ 
-                    type: rangeFields.type.initial, 
-                    target: rangeFields.target.initial,
-                    range: rangeFields.range.initial 
-                })
-                : null
-        };
-        return this.submit({ updateData: { system: systemData } });
-    }
-
     armorChangeToggle(event) {
         if (event.target.checked) {
             this.addArmorChange();

module/applications/sheets-configs/adversary-settings.mjs

@@ -1,4 +1,3 @@
-import { DHDamageData } from '../../data/fields/action/damageField.mjs';
 import DHBaseActorSettings from '../sheets/api/actor-setting.mjs';
 
 /**@typedef {import('@client/applications/_types.mjs').ApplicationClickAction} ApplicationClickAction */
@@ -9,10 +8,8 @@ export default class DHAdversarySettings extends DHBaseActorSettings {
         classes: ['adversary-settings'],
         position: { width: 455, height: 'auto' },
         actions: {
-            addExperience: this.#onAddExperience,
+            addExperience: DHAdversarySettings.#addExperience,
-            removeExperience: this.#onRemoveExperience,
+            removeExperience: DHAdversarySettings.#removeExperience
-            addDamage: this.#onAddDamage,
-            removeDamage: this.#onRemoveDamage
         }
     };
 
@@ -74,7 +71,7 @@ export default class DHAdversarySettings extends DHBaseActorSettings {
      * Adds a new experience entry to the actor.
      * @type {ApplicationClickAction}
      */
-    static async #onAddExperience() {
+    static async #addExperience() {
         const newExperience = {
             name: 'Experience',
             modifier: 0
@@ -86,7 +83,7 @@ export default class DHAdversarySettings extends DHBaseActorSettings {
      * Removes an experience entry from the actor.
      * @type {ApplicationClickAction}
      */
-    static async #onRemoveExperience(_, target) {
+    static async #removeExperience(_, target) {
         const experience = this.actor.system.experiences[target.dataset.experience];
         const confirmed = await foundry.applications.api.DialogV2.confirm({
             window: {
@@ -101,28 +98,4 @@ export default class DHAdversarySettings extends DHBaseActorSettings {
 
         await this.actor.update({ [`system.experiences.${target.dataset.experience}`]: _del });
     }
-
-    /**
-     * @this DHAdversarySettings 
-     * @type {ApplicationClickAction}
-     */
-    static #onAddDamage() {
-        this.actor.update({
-            'system.attack.damage.main': {
-                ...DHDamageData.schema.getInitialValue(),
-                applyTo: 'hitPoints',
-                type: 'physical'
-            }
-        });
-    }
-    
-    /**
-     * @this DHAdversarySettings 
-     * @type {ApplicationClickAction}
-     */
-    static #onRemoveDamage() {
-        this.actor.update({
-            'system.attack.damage.main': null
-        });
-    }
 }

module/applications/sheets/actors/_types.d.ts

@@ -1,18 +1,8 @@
 import DhCompanion from '../../../data/actor/companion.mjs';
-import DhParty from '../../../data/actor/party.mjs';
 import DhpActor from '../../../documents/actor.mjs';
 
 declare module './companion.mjs' {
     export default interface DhCompanionSheet {
         actor: DhpActor<DhCompanion>;
-        document: DhpActor<DhCompanion>;
     }
 }
-
-declare module './party.mjs' {
-    export default interface PartySheet {
-        actor: DhpActor<DhParty>;
-        document: DhpActor<DhParty>;
-    }
-}
-

module/applications/sheets/actors/adversary.mjs

@@ -7,7 +7,7 @@ export default class AdversarySheet extends DHBaseActorSheet {
     /** @inheritDoc */
     static DEFAULT_OPTIONS = {
         classes: ['adversary'],
-        position: { width: 645, height: 750 },
+        position: { width: 660, height: 766 },
         window: { resizable: true },
         actions: {
             toggleHitPoints: AdversarySheet.#toggleHitPoints,
@@ -58,13 +58,12 @@ export default class AdversarySheet extends DHBaseActorSheet {
             template: 'systems/daggerheart/templates/sheets/actors/adversary/features.hbs',
             scrollable: ['.feature-section']
         },
+        notes: {
+            template: 'systems/daggerheart/templates/sheets/actors/adversary/notes.hbs'
+        },
         effects: {
             template: 'systems/daggerheart/templates/sheets/actors/adversary/effects.hbs',
             scrollable: ['.effects-sections']
-        },
-        notes: {
-            template: 'systems/daggerheart/templates/sheets/actors/adversary/notes.hbs',
-            scrollable: ['.editor-content']
         }
     };
 

module/applications/sheets/actors/character.mjs

@@ -3,7 +3,7 @@ import DhDeathMove from '../../dialogs/deathMove.mjs';
 import { CharacterLevelup, LevelupViewMode } from '../../levelup/_module.mjs';
 import DhCharacterCreation from '../../characterCreation/characterCreation.mjs';
 import FilterMenu from '../../ux/filter-menu.mjs';
-import { getArmorSources, getDocFromElement, getDocFromElementSync, itemAbleRollParse, sortBy } from '../../../helpers/utils.mjs';
+import { getArmorSources, getDocFromElement, getDocFromElementSync, sortBy } from '../../../helpers/utils.mjs';
 
 /**@typedef {import('@client/applications/_types.mjs').ApplicationClickAction} ApplicationClickAction */
 
@@ -29,7 +29,6 @@ export default class CharacterSheet extends DHBaseActorSheet {
             toggleResourceDice: CharacterSheet.#toggleResourceDice,
             handleResourceDice: CharacterSheet.#handleResourceDice,
             advanceResourceDie: CharacterSheet.#advanceResourceDie,
-            toggleItemReload: CharacterSheet.#onToggleItemReload,
             cancelBeastform: CharacterSheet.#cancelBeastform,
             toggleResourceManagement: CharacterSheet.#toggleResourceManagement,
             useDowntime: this.useDowntime,
@@ -915,7 +914,7 @@ export default class CharacterSheet extends DHBaseActorSheet {
         const doc = await getDocFromElement(button);
         const { available } = this.document.system.loadoutSlot;
         if (doc.system.inVault && !available && !doc.system.loadoutIgnore) {
-            return ui.notifications.warn('DAGGERHEART.UI.Notifications.loadoutMaxReached', { localize: true });
+            return ui.notifications.warn(game.i18n.localize('DAGGERHEART.UI.Notifications.loadoutMaxReached'));
         }
 
         await doc?.update({ 'system.inVault': !doc.system.inVault });
@@ -955,21 +954,11 @@ export default class CharacterSheet extends DHBaseActorSheet {
         });
     }
 
+    /** */
     static #advanceResourceDie(_, target) {
         this.updateResourceDie(target, true);
     }
 
-    static async #onToggleItemReload(_, target) {
-        const item = await getDocFromElement(target);
-        if (!item || !item.system.resource?.max) 
-            return;
-
-        await item.update({ 
-            'system.resource.value': item.system.needsReload ? 
-                itemAbleRollParse(item.system.resource.max, this.document, item) : 0
-        })
-    }
-
     lowerResourceDie(event) {
         event.preventDefault();
         event.stopPropagation();
@@ -1068,7 +1057,7 @@ export default class CharacterSheet extends DHBaseActorSheet {
             direction: 'DOWN'
         });
 
-        html.querySelectorAll('.armor .slot').forEach(element => {
+        html.querySelectorAll('.armor-slot').forEach(element => {
             element.addEventListener('click', CharacterSheet.armorSourcePipUpdate);
         });
     }
@@ -1083,7 +1072,7 @@ export default class CharacterSheet extends DHBaseActorSheet {
 
     /** Update specific armor source */
     static async armorSourcePipUpdate(event) {
-        const target = event.target.closest('.slot');
+        const target = event.target.closest('.armor-slot');
         const { uuid, value } = target.dataset;
         const document = await foundry.utils.fromUuid(uuid);
 
@@ -1111,7 +1100,7 @@ export default class CharacterSheet extends DHBaseActorSheet {
         }
 
         const container = target.closest('.slot-bar');
-        for (const armorSlot of container.querySelectorAll('.armor .slot i')) {
+        for (const armorSlot of container.querySelectorAll('.armor-slot i')) {
             const index = Number.parseInt(armorSlot.dataset.index);
             if (decreasing && index >= newCurrent) {
                 armorSlot.classList.remove('fa-shield');
@@ -1217,22 +1206,10 @@ export default class CharacterSheet extends DHBaseActorSheet {
             if (!confirmed) return;
         }
 
-        // Check for same actor drag/drop attempts
+        if (this.document.uuid === item.parent?.uuid) {
-        const isSameActor = this.document.uuid === item.parent?.uuid;
-        const loadoutFieldset = event.target.closest('[data-in-vault]');
-        const vaulting = loadoutFieldset?.dataset.inVault === 'true';
-        if (isSameActor && loadoutFieldset && item.type === 'domainCard' && vaulting !== item.system.inVault) {
-            // This is likely an attempt to vault or unvault an item
-            const { available } = this.document.system.loadoutSlot;
-            if (!vaulting && !available && !item.system.loadoutIgnore) {
-                return ui.notifications.warn('DAGGERHEART.UI.Notifications.loadoutMaxReached', { localize: true });
-            }
-            return item.update({ 'system.inVault': vaulting });
-        } else if (isSameActor) {
             return super._onDropItem(event, item);
         }
 
-        // Handle beastforms
         if (item.type === 'beastform') {
             if (this.document.effects.find(x => x.type === 'beastform')) {
                 return ui.notifications.warn(

module/applications/sheets/actors/party.mjs

@@ -6,9 +6,7 @@ import DaggerheartMenu from '../../sidebar/tabs/daggerheartMenu.mjs';
 import { socketEvent } from '../../../systemRegistration/socket.mjs';
 import DhpActor from '../../../documents/actor.mjs';
 
-/**@typedef {import('@client/applications/_types.mjs').ApplicationClickAction} ApplicationClickAction */
+export default class Party extends DHBaseActorSheet {
-
-export default class PartySheet extends DHBaseActorSheet {
     constructor(options) {
         super(options);
 
@@ -26,17 +24,17 @@ export default class PartySheet extends DHBaseActorSheet {
             resizable: true
         },
         actions: {
-            openDocument: PartySheet.#onOpenDocument,
+            openDocument: Party.#openDocument,
-            deletePartyMember: PartySheet.#onDeletePartyMember,
+            deletePartyMember: Party.#deletePartyMember,
-            toggleHope: PartySheet.#onToggleHope,
+            toggleHope: Party.#toggleHope,
-            toggleHitPoints: PartySheet.#onToggleHitPoints,
+            toggleHitPoints: Party.#toggleHitPoints,
-            toggleStress: PartySheet.#onToggleStress,
+            toggleStress: Party.#toggleStress,
-            toggleArmorSlot: PartySheet.#onToggleArmorSlot,
+            toggleArmorSlot: Party.#toggleArmorSlot,
-            tempBrowser: PartySheet.#onTempBrowser,
+            tempBrowser: Party.#tempBrowser,
-            refreshActions: PartySheet.#onRefreshActions,
+            refeshActions: Party.#refeshActions,
-            triggerRest: PartySheet.#onTriggerRest,
+            triggerRest: Party.#triggerRest,
-            tagTeamRoll: PartySheet.#onTagTeamRoll,
+            tagTeamRoll: Party.#tagTeamRoll,
-            groupRoll: PartySheet.#onGroupRoll
+            groupRoll: Party.#groupRoll
         },
         dragDrop: [{ dragSelector: '[data-item-id]', dropSelector: null }]
     };
@@ -74,23 +72,6 @@ export default class PartySheet extends DHBaseActorSheet {
         this._createSearchFilter();
     }
 
-    /** @inheritdoc */
-    _toggleDisabled(disabled) {
-        // Overriden to only disable text inputs by default if the user is a member
-        const isMember = this.actor.system.partyMembers.some(
-            m => m.testUserPermission(game.user, CONST.DOCUMENT_OWNERSHIP_LEVELS.OWNER)
-        );
-        if (!isMember) return super._toggleDisabled(disabled);
-
-        const form = this.form;
-        for (const input of form.querySelectorAll('input:not([type=search]), .editor.prosemirror')) {
-            input.disabled = disabled;
-        }
-        for (const element of form.querySelectorAll('.input[contenteditable]')) {
-            element.classList.toggle('disabled', disabled);
-        }
-    }
-
     /* -------------------------------------------- */
     /*  Prepare Context                             */
     /* -------------------------------------------- */
@@ -218,15 +199,7 @@ export default class PartySheet extends DHBaseActorSheet {
         }
     }
 
-    /* -------------------------------------------- */
+    static async #openDocument(_, target) {
-    /*  Event handlers                          */
-    /* -------------------------------------------- */
-
-    /**
-     * @this PartySheet 
-     * @type {ApplicationClickAction}
-     */
-    static async #onOpenDocument(_, target) {
         const uuid = target.dataset.uuid;
         const document = await foundry.utils.fromUuid(uuid);
         document?.sheet?.render(true);
@@ -234,10 +207,9 @@ export default class PartySheet extends DHBaseActorSheet {
 
     /**
      * Toggles a hope resource value.
-     * @this PartySheet
      * @type {ApplicationClickAction}
      */
-    static async #onToggleHope(_, target) {
+    static async #toggleHope(_, target) {
         const hopeValue = Number.parseInt(target.dataset.value);
         const actor = await foundry.utils.fromUuid(target.dataset.actorId);
         const newValue = actor.system.resources.hope.value >= hopeValue ? hopeValue - 1 : hopeValue;
@@ -247,10 +219,9 @@ export default class PartySheet extends DHBaseActorSheet {
 
     /**
      * Toggles a hp resource value.
-     * @this PartySheet
      * @type {ApplicationClickAction}
      */
-    static async #onToggleHitPoints(_, target) {
+    static async #toggleHitPoints(_, target) {
         const hitPointsValue = Number.parseInt(target.dataset.value);
         const actor = await foundry.utils.fromUuid(target.dataset.actorId);
         const newValue = actor.system.resources.hitPoints.value >= hitPointsValue ? hitPointsValue - 1 : hitPointsValue;
@@ -260,10 +231,9 @@ export default class PartySheet extends DHBaseActorSheet {
 
     /**
      * Toggles a stress resource value.
-     * @this PartySheet
      * @type {ApplicationClickAction}
      */
-    static async #onToggleStress(_, target) {
+    static async #toggleStress(_, target) {
         const stressValue = Number.parseInt(target.dataset.value);
         const actor = await foundry.utils.fromUuid(target.dataset.actorId);
         const newValue = actor.system.resources.stress.value >= stressValue ? stressValue - 1 : stressValue;
@@ -273,10 +243,9 @@ export default class PartySheet extends DHBaseActorSheet {
 
     /**
      * Toggles a armor slot resource value.
-     * @this PartySheet
      * @type {ApplicationClickAction}
      */
-    static async #onToggleArmorSlot(_, target) {
+    static async #toggleArmorSlot(_, target) {
         const actor = await foundry.utils.fromUuid(target.dataset.actorId);
         const { value, max } = actor.system.armorScore;
         const inputValue = Number.parseInt(target.dataset.value);
@@ -289,19 +258,12 @@ export default class PartySheet extends DHBaseActorSheet {
 
     /**
      * Opens Compedium Browser
-     * @this PartySheet
-     * @type {ApplicationClickAction}
      */
-    static async #onTempBrowser(_, target) {
+    static async #tempBrowser(_, target) {
         new ItemBrowser().render({ force: true });
     }
 
-    /** 
+    static async #refeshActions() {
-     * @todo Is this unused?
-     * @this PartySheet
-     * @type {ApplicationClickAction}
-     */
-    static async #onRefreshActions() {
         const confirmed = await foundry.applications.api.DialogV2.confirm({
             window: {
                 title: 'New Section',
@@ -319,11 +281,7 @@ export default class PartySheet extends DHBaseActorSheet {
         if (!confirmed) return;
     }
 
-    /** 
+    static async #triggerRest(_, button) {
-     * @this PartySheet
-     * @type {ApplicationClickAction}
-     */
-    static async #onTriggerRest(_, button) {
         const confirmed = await foundry.applications.api.DialogV2.confirm({
             window: {
                 title: game.i18n.localize(`DAGGERHEART.APPLICATIONS.Downtime.${button.dataset.type}.title`),
@@ -346,30 +304,6 @@ export default class PartySheet extends DHBaseActorSheet {
         });
     }
 
-    /** 
-     * @this PartySheet
-     * @type {ApplicationClickAction}
-     */
-    static async #onDeletePartyMember(event, target) {
-        const doc = await foundry.utils.fromUuid(target.closest('[data-uuid]')?.dataset.uuid);
-        if (!event.shiftKey) {
-            const confirmed = await foundry.applications.api.DialogV2.confirm({
-                window: {
-                    title: game.i18n.format('DAGGERHEART.ACTORS.Party.RemoveConfirmation.title', {
-                        name: doc.name
-                    })
-                },
-                content: game.i18n.format('DAGGERHEART.ACTORS.Party.RemoveConfirmation.text', { name: doc.name })
-            });
-
-            if (!confirmed) return;
-        }
-
-        const currentMembers = this.document.system.partyMembers.map(x => x.uuid);
-        const newMembersList = currentMembers.filter(uuid => uuid !== doc.uuid);
-        await this.document.update({ 'system.partyMembers': newMembersList });
-    }
-
     static async downtimeMoveQuery({ actorId, downtimeType }) {
         const actor = await foundry.utils.fromUuid(actorId);
         if (!actor || !actor?.isOwner) return;
@@ -378,19 +312,11 @@ export default class PartySheet extends DHBaseActorSheet {
         });
     }
 
-    /** 
+    static async #tagTeamRoll() {
-     * @this PartySheet
-     * @type {ApplicationClickAction}
-     */
-    static async #onTagTeamRoll() {
         new game.system.api.applications.dialogs.TagTeamDialog(this.document).render({ force: true });
     }
 
-    /** 
+    static async #groupRoll(_params) {
-     * @this PartySheet
-     * @type {ApplicationClickAction}
-     */
-    static async #onGroupRoll(_params) {
         new game.system.api.applications.dialogs.GroupRollDialog(this.document).render({ force: true });
     }
 
@@ -534,10 +460,11 @@ export default class PartySheet extends DHBaseActorSheet {
         }
     }
 
-    /** @inheritdoc */
+    /* -------------------------------------------- */
+
     async _onDropActor(event, document) {
         const data = foundry.applications.ux.TextEditor.implementation.getDragEventData(event);
-        if (document instanceof DhpActor && PartySheet.ALLOWED_ACTOR_TYPES.includes(document.type)) {
+        if (document instanceof DhpActor && Party.ALLOWED_ACTOR_TYPES.includes(document.type)) {
             const currentMembers = this.document.system.partyMembers.map(x => x.uuid);
             if (currentMembers.includes(data.uuid)) {
                 return ui.notifications.warn(game.i18n.localize('DAGGERHEART.UI.Notifications.duplicateCharacter'));
@@ -550,4 +477,24 @@ export default class PartySheet extends DHBaseActorSheet {
 
         return null;
     }
+
+    static async #deletePartyMember(event, target) {
+        const doc = await foundry.utils.fromUuid(target.closest('[data-uuid]')?.dataset.uuid);
+        if (!event.shiftKey) {
+            const confirmed = await foundry.applications.api.DialogV2.confirm({
+                window: {
+                    title: game.i18n.format('DAGGERHEART.ACTORS.Party.RemoveConfirmation.title', {
+                        name: doc.name
+                    })
+                },
+                content: game.i18n.format('DAGGERHEART.ACTORS.Party.RemoveConfirmation.text', { name: doc.name })
+            });
+
+            if (!confirmed) return;
+        }
+
+        const currentMembers = this.document.system.partyMembers.map(x => x.uuid);
+        const newMembersList = currentMembers.filter(uuid => uuid !== doc.uuid);
+        await this.document.update({ 'system.partyMembers': newMembersList });
+    }
 }

module/applications/sheets/api/application-mixin.mjs

@@ -305,7 +305,7 @@ export default function DHApplicationMixin(Base) {
         _preSyncPartState(partId, newElement, priorElement, state) {
             super._preSyncPartState(partId, newElement, priorElement, state);
             for (const el of priorElement.querySelectorAll('.extensible.extended')) {
-                const { actionId, itemUuid } = el.closest('[data-item-uuid], [data-action-id]').dataset;
+                const { actionId, itemUuid } = el.parentElement.dataset;
                 const selector = `${actionId ? `[data-action-id="${actionId}"]` : `[data-item-uuid="${itemUuid}"]`} .extensible`;
                 const newExtensible = newElement.querySelector(selector);
                 newExtensible?.classList.add('extended');

module/applications/sheets/api/base-item.mjs

@@ -30,8 +30,7 @@ export default class DHBaseItemSheet extends DHApplicationMixin(ItemSheetV2) {
             addFeature: DHBaseItemSheet.#addFeature,
             deleteFeature: DHBaseItemSheet.#deleteFeature,
             addResource: DHBaseItemSheet.#addResource,
-            removeResource: DHBaseItemSheet.#removeResource,
+            removeResource: DHBaseItemSheet.#removeResource
-            editGMNote: DHBaseItemSheet.#onEditGMNote
         },
         dragDrop: [
             { dragSelector: null, dropSelector: '.drop-section' },
@@ -77,16 +76,10 @@ export default class DHBaseItemSheet extends DHApplicationMixin(ItemSheetV2) {
     /**@inheritdoc */
     async _preparePartContext(partId, context, options) {
         await super._preparePartContext(partId, context, options);
-        const TextEditor = foundry.applications.ux.TextEditor.implementation;
 
         switch (partId) {
             case 'description':
-                context.enrichedDescription = await this.document.system.getEnrichedDescription({ gmNotes: false });
+                context.enrichedDescription = await this.document.system.getEnrichedDescription();
-                context.enrichedGMNotes = await TextEditor.implementation.enrichHTML(this.item.system.gmNotes, {
-                    relativeTo: this.item,
-                    rollData: this.item.getRollData(),
-                    secrets: this.item.isOwner
-                })
                 break;
             case 'effects':
                 await this._prepareEffectsContext(context, options);
@@ -338,45 +331,4 @@ export default class DHBaseItemSheet extends DHApplicationMixin(ItemSheetV2) {
             }
         }
     }
-
-    /** 
-     * Handles the Add GM Note button being pressed. This is only used when an item has no GM notes.
-     * Later edits to a GM note instead go through the normal editor toggle workflow.
-     * @this DHBaseItemSheet
-     */
-    static #onEditGMNote() {
-        // Open the editor, which might be hidden. We remove the css class to hide temporarily
-        // so that menu auto resizing functions properly.
-        const editor = this.element.querySelector('prose-mirror[name="system.gmNotes"]');
-        const wasHidden = editor.classList.contains('hide-if-inactive');
-        editor.classList.remove('hide-if-inactive');
-        editor.open = true;
-        window.setTimeout(() => {
-            if (wasHidden) editor.classList.add('hide-if-inactive');
-        }, 0);
-    }
-
-    /** @inheritdoc */
-    async _onRender(context, options) {
-        await super._onRender(context, options);
-
-        // Render an add gmnotes button if there are no set GM notes.
-        // We need to re-render on close since its possible to prosemirror to close *without* triggering a full re-render
-        if (game.user.isGM && !this.item.system.gmNotes) {
-            const description = this.element.querySelector('[name="system.description"]');
-            const addButton = () => {
-                if (description.querySelector('[data-action=editGMNote]')) return;
-
-                const button = document.createElement('button');
-                button.type = 'button';
-                button.classList.add('icon', 'toggle', 'fa-regular', 'fa-note-medical');
-                button.dataset.action = 'editGMNote';
-                button.dataset.tooltip = 'DAGGERHEART.ITEMS.Base.addGMNote';
-                description.appendChild(button);
-            }
-            
-            addButton();
-            description.addEventListener('close', () => addButton());
-        }
-    }
 }

module/applications/ui/chatLog.mjs

@@ -1,7 +1,6 @@
 import { enrichedDualityRoll } from '../../enrichers/DualityRollEnricher.mjs';
 import { enrichedFateRoll, getFateTypeData } from '../../enrichers/FateRollEnricher.mjs';
 import { getCommandTarget, rollCommandToJSON } from '../../helpers/utils.mjs';
-import FearTracker from './fearTracker.mjs';
 
 export default class DhpChatLog extends foundry.applications.sidebar.tabs.ChatLog {
     constructor(options) {
@@ -111,7 +110,7 @@ export default class DhpChatLog extends foundry.applications.sidebar.tabs.ChatLo
                     const message = game.messages.get(li.dataset.messageId);
                     return message.system.hasRoll && (game.user.isGM || message.isAuthor);
                 },
-                onClick: async (_event, li) => {
+                callback: async li => {
                     const message = game.messages.get(li.dataset.messageId);
                     const reroll = await message.rolls[0].reroll({ liveRoll: true });
                     message.update({ rolls: [reroll] });
@@ -127,7 +126,7 @@ export default class DhpChatLog extends foundry.applications.sidebar.tabs.ChatLo
                         : false;
                     return (game.user.isGM || message.isAuthor) && hasRolledDamage;
                 },
-                onClick: async (_event, li) => {
+                callback: async li => {
                     const message = game.messages.get(li.dataset.messageId);
                     const update = await message.system.getRerolledDamage();
                     message.update(update);
@@ -153,9 +152,6 @@ export default class DhpChatLog extends foundry.applications.sidebar.tabs.ChatLo
         html.querySelectorAll('.risk-it-all-button').forEach(element =>
             element.addEventListener('click', event => this.riskItAllClearStressAndHitPoints(event, data))
         );
-        for (const element of html.querySelectorAll('.roll-reload-check')) {
-            element.addEventListener('click', event => this.onRollReloadCheck(event, message));
-        }
     };
 
     setupHooks() {
@@ -183,15 +179,28 @@ export default class DhpChatLog extends foundry.applications.sidebar.tabs.ChatLo
     }
 
     async onRollSimple(event, message) {
-        const buttonType = event.target.dataset.type ?? 'damage';
+        const buttonType = event.target.dataset.type ?? 'damage',
-        const total = message.rolls.reduce((a, c) => a + Roll.fromJSON(c).total, 0);
+            total = message.rolls.reduce((a, c) => a + Roll.fromJSON(c).total, 0),
-        const targets = Array.from(game.user.targets);
+            damages = {
+                hitPoints: {
+                    parts: [
+                        {
+                            applyTo: 'hitPoints',
+                            damageTypes: [],
+                            total
+                        }
+                    ]
+                }
+            },
+            targets = Array.from(game.user.targets);
+
         if (targets.length === 0)
             return ui.notifications.info(game.i18n.localize('DAGGERHEART.UI.Notifications.noTargetsSelected'));
-        for (const target of targets) {
+
-            if (buttonType === 'healing') target.actor.takeHealing({ hitPoints: total });
+        targets.forEach(target => {
-            else target.actor.takeDamage({ total });
+            if (buttonType === 'healing') target.actor.takeHealing(damages);
-        }
+            else target.actor.takeDamage(damages);
+        });
     }
 
     async abilityUseButton(event, message) {
@@ -247,17 +256,26 @@ export default class DhpChatLog extends foundry.applications.sidebar.tabs.ChatLo
         }
 
         const message = game.messages.get(messageData._id);
-        const target = event.target.closest('[data-result]');
+        const target = event.target.closest('[data-die-index]');
 
         if (target.dataset.type === 'damage') {
-            const { isResource, damageType, dice, result } = target.dataset;
+            const { damageType, part, dice, result } = target.dataset;
-            await message.system.damage.rerollDamageDie(isResource, damageType, dice, result);
+            const damagePart = message.system.damage[damageType].parts[part];
-
+            const { parsedRoll, rerolledDice } = await game.system.api.dice.DamageRoll.reroll(damagePart, dice, result);
-            const updatePath = isResource ? `system.damage.resources.${damageType}` : 'system.damage.main';
+            const damageParts = message.system.damage[damageType].parts.map((damagePart, index) => {
-            const updateValue = isResource ? 
+                if (index !== Number(part)) return damagePart;
-                message.system.damage.resources[damageType] : message.system.damage.main;
+                return {
-            await message.update({
+                    ...damagePart,
-                [updatePath]: updateValue.toJSON()
+                    total: parsedRoll.total,
+                    dice: rerolledDice
+                };
+            });
+            const updateMessage = game.messages.get(message._id);
+            await updateMessage.update({
+                [`system.damage.${damageType}`]: {
+                    total: parsedRoll.total,
+                    parts: damageParts
+                }
             });
         } else {
             const rerollDice = message.system.roll.dice[target.dataset.dieIndex];
@@ -279,27 +297,4 @@ export default class DhpChatLog extends foundry.applications.sidebar.tabs.ChatLo
         const actor = game.actors.get(event.target.dataset.actorId);
         new game.system.api.applications.dialogs.RiskItAllDialog(actor, resourceValue).render({ force: true });
     }
-
-    _toggleNotifications({ closing = false } = {}) {
-        super._toggleNotifications(closing)
-        FearTracker.handleOffSet();
-    }
-
-    async onRollReloadCheck(_event, messageData) {
-        const message = game.messages.get(messageData._id);
-
-        if (message.system.reloadCheckValue) {
-            const confirmed = await foundry.applications.api.DialogV2.confirm({
-                window: {
-                    title: _loc('DAGGERHEART.ACTIONS.Reload.rerollConfirmationTitle')
-                },
-                content: _loc('DAGGERHEART.ACTIONS.Reload.rerollConfirmationText')
-            });
-
-            if (!confirmed) return;
-        }
-
-        const { rollValue } = await message.system.action.handleReload?.({ awaitRoll: true });
-        await message.update({ 'system.reloadCheckValue': rollValue });
-    }
 }

module/applications/ui/countdownEdit.mjs

@@ -153,6 +153,7 @@ export default class CountdownEdit extends HandlebarsApplicationMixin(Applicatio
             action: socketEvent.Refresh,
             data: { refreshType: RefreshType.Countdown }
         });
+        Hooks.callAll(socketEvent.Refresh, { refreshType: RefreshType.Countdown });
     }
 
     static #addCountdown() {

module/applications/ui/countdowns.mjs

@@ -12,11 +12,14 @@ const { HandlebarsApplicationMixin, ApplicationV2 } = foundry.applications.api;
 
 export default class DhCountdowns extends HandlebarsApplicationMixin(ApplicationV2) {
     previousCountdownData = null;
+    changedCountdownsForAnimation = new Set();
 
     constructor(options = {}) {
         super(options);
+
         this.previousCountdownData = 
             game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Countdowns).countdowns;
+        this.setupHooks();
     }
 
     /** @inheritDoc */
@@ -77,15 +80,7 @@ export default class DhCountdowns extends HandlebarsApplicationMixin(Application
         return frame;
     }
 
-    /** @inheritdoc */
-    async _onFirstRender(context, options) {
-        await super._onFirstRender(context, options);
-        this._createContextMenu(this._getCountdownContextOptions, '.countdown-container[data-countdown]', {
-            parentClassHooks: false, fixed: true
-        });
-    }
     
-    /** @inheritdoc */
     async _onRender(context, options) {
         await super._onRender(context, options);
 
@@ -100,7 +95,7 @@ export default class DhCountdowns extends HandlebarsApplicationMixin(Application
 
         /* Handle animations to draw attention to countdown values changing */
         const typesToAnimate = new Set();
-        for (const countdownKey of options.animate ?? []) {
+        for (const countdownKey of this.changedCountdownsForAnimation) {
             const shimmerAnimation = [
                 { backgroundPositionX: '98%' },
                 { backgroundPositionX: '0%' }
@@ -132,6 +127,8 @@ export default class DhCountdowns extends HandlebarsApplicationMixin(Application
             const element = this.element.querySelector(`.header-type-toggles .header-type[data-type="${type}"]`);
             element?.animate(pulseAnimation, pulseTiming);
         }
+        
+        this.changedCountdownsForAnimation.clear();
     }
 
     /** Returns countdown data filtered by ownership */
@@ -140,15 +137,17 @@ export default class DhCountdowns extends HandlebarsApplicationMixin(Application
         const values = Object.entries(setting.countdowns).map(([key, countdown]) => ({
             key,
             countdown,
-            ownership: countdown.getUserLevel(game.user)
+            ownership: DhCountdowns.#getPlayerOwnership(game.user, setting, countdown)
         }));
         return values.filter(v => v.ownership !== CONST.DOCUMENT_OWNERSHIP_LEVELS.NONE);
     }
 
     _getCountdownData() {
+        const setting = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Countdowns);
+
         return this.#getCountdowns().reduce((acc, { key, countdown, ownership }) => {
             const playersWithAccess = game.users.reduce((acc, user) => {
-                const ownership = countdown.getUserLevel(user);
+                const ownership = DhCountdowns.#getPlayerOwnership(user, setting, countdown);
                 if (!user.isGM && ownership && ownership !== CONST.DOCUMENT_OWNERSHIP_LEVELS.NONE) {
                     acc.push(user);
                 }
@@ -214,6 +213,19 @@ export default class DhCountdowns extends HandlebarsApplicationMixin(Application
         return context;
     }
 
+    static #getPlayerOwnership(user, setting, countdown) {
+        if (user.isGM) return CONST.DOCUMENT_OWNERSHIP_LEVELS.OWNER;
+
+        const playerOwnership = countdown.ownership[user.id];
+        return playerOwnership === undefined || playerOwnership === CONST.DOCUMENT_OWNERSHIP_LEVELS.INHERIT
+            ? setting.defaultOwnership
+            : playerOwnership;
+    }
+
+    cooldownRefresh = ({ refreshType }) => {
+        if (refreshType === RefreshType.Countdown) this.render();
+    };
+
     static canPerformEdit() {
         if (game.user.isGM) return true;
 
@@ -311,12 +323,18 @@ export default class DhCountdowns extends HandlebarsApplicationMixin(Application
             action: socketEvent.Refresh,
             data: { refreshType: RefreshType.Countdown }
         });
+        Hooks.callAll(socketEvent.Refresh, { refreshType: RefreshType.Countdown });
+    }
+
+    setupHooks() {
+        Hooks.on(socketEvent.Refresh, this.cooldownRefresh.bind());
     }
 
     async close(options) {
         /* Opt out of Foundry's standard behavior of closing all application windows marked as UI when Escape is pressed */
         if (options.closeKey) return;
 
+        Hooks.off(socketEvent.Refresh, this.cooldownRefresh);
         return super.close(options);
     }
 
@@ -359,30 +377,4 @@ export default class DhCountdowns extends HandlebarsApplicationMixin(Application
             refreshType: RefreshType.Countdown
         });
     }
-
-    /**
-     * @returns {import('@client/applications/ux/context-menu.mjs').ContextMenuEntry[]}
-     */
-    _getCountdownContextOptions() {
-        /** @param {HTMLElement} element */
-        const getCountdownFromElement = element => {
-            const id = element.closest('[data-countdown]').dataset.countdown;
-            if (!id) return null;
-            const setting = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Countdowns);
-            return setting.countdowns[id ?? ''];
-        }
-
-        return [
-            {
-                label: 'CONTROLS.CommonDelete',
-                icon: 'fa-solid fa-trash',
-                visible: element => {
-                    return getCountdownFromElement(element)?.isOwner;
-                },
-                onClick: (_, target) => {
-                    getCountdownFromElement(target)?.delete();
-                }
-            }
-        ];
-    }
 }

module/applications/ui/effectsDisplay.mjs

@@ -39,12 +39,6 @@ export default class DhEffectsDisplay extends HandlebarsApplicationMixin(Applica
         }
     };
 
-    /**
-     * Debounce and slightly delayed request to re-render this panel. Necessary for situations where it is not possible
-     * to properly wait for promises to resolve before refreshing the UI.
-     */
-    refresh = foundry.utils.debounce(this.render.bind(this), 50);
-
     get element() {
         return document.body.querySelector('.daggerheart.dh-style.effects-display');
     }

module/applications/ui/fearTracker.mjs

@@ -13,14 +13,6 @@ const { HandlebarsApplicationMixin, ApplicationV2 } = foundry.applications.api;
 export default class FearTracker extends HandlebarsApplicationMixin(ApplicationV2) {
     constructor(options = {}) {
         super(options);
-
-        this._dragData = {
-            isDragging: false,
-            startX: 0,
-            startY: 0,
-            startLeft: 0,
-            startTop: 0
-        }
     }
 
     /** @inheritDoc */
@@ -29,20 +21,19 @@ export default class FearTracker extends HandlebarsApplicationMixin(ApplicationV
         classes: [],
         tag: 'div',
         window: {
-            frame: false,
+            frame: true,
             title: 'DAGGERHEART.GENERAL.fear',
             positioned: true,
             resizable: true,
             minimizable: false
         },
-        classes: ['daggerheart', 'dh-style', 'fear-tracker'],
         actions: {
             setFear: FearTracker.setFear,
             increaseFear: FearTracker.increaseFear
         },
         position: {
-            width: 540,
+            width: 222,
-            height: 'auto'
+            height: 222
         }
     };
 
@@ -62,10 +53,6 @@ export default class FearTracker extends HandlebarsApplicationMixin(ApplicationV
         return game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Homebrew).maxFear;
     }
 
-    get fearPosition() {
-        return game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.appearance).fearPosition;
-    }
-
     /* -------------------------------------------- */
     /*  Rendering                                   */
     /* -------------------------------------------- */
@@ -76,48 +63,15 @@ export default class FearTracker extends HandlebarsApplicationMixin(ApplicationV
             current = this.currentFear,
             max = this.maxFear,
             percent = (current / max) * 100,
-            isGM = game.user.isGM,
+            isGM = game.user.isGM;
-            locked = false,
-            isFree = this.fearPosition == 'free';
 
-        return { display, current, max, percent, isGM, locked, isFree };
+        return { display, current, max, percent, isGM };
     }
 
     /** @override */
-    async _onRender(context, options) {
+    async _preFirstRender(context, options) {
-        await super._onRender(context, options);
+        options.position =
-
-        this.#setupDragging();
-        this.#setupResizing();
-
-        const fearPosition = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.appearance).fearPosition;
-
-        if (options.isFirstRender) FearTracker.handleOffSet();
-        if (!options.force) return;
-        
-        this.handleStyleElement(fearPosition);
-
-        switch (fearPosition) {
-            case 'topCenter':
-                document.getElementById('ui-top')?.appendChild(this.element);
-                break;
-            case 'bottomCenter':
-                document.getElementById('ui-bottom')?.prepend(this.element);
-                break;
-            case 'rightTop':
-                document.getElementById('ui-right-column-1')?.appendChild(this.element);
-                break;
-            case 'leftBottom':
-                document.getElementById('ui-left-column-1')?.insertBefore(this.element, document.getElementById('players'));
-                break;
-                
-            default:
-                document.body?.appendChild(this.element);
-                const position =
             game.user.getFlag(CONFIG.DH.id, 'app.resources.position') ?? FearTracker.DEFAULT_OPTIONS.position;
-                this.setPosition(position);
-                break;
-        }
     }
 
     /** @override */
@@ -126,31 +80,15 @@ export default class FearTracker extends HandlebarsApplicationMixin(ApplicationV
             await game.settings.set(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Resources.Fear, this.maxFear);
     }
 
-    handleStyleElement(fearPosition) {
-        for (const position of Object.values(CONFIG.DH.GENERAL.fearPosition)) {
-            this.element.classList.remove(position.value); 
-        }
-
-        this.element.classList.add(fearPosition);
-    }
-
-    static handleOffSet() {
-        const fearTracker = document.getElementById('resources');
-        const hotbar = document.getElementById('hotbar');
-
-        if (!fearTracker) return;
-
-        const offset = Number(hotbar.style.getPropertyValue('--offset').replace(/px$/, '')) || 0;
-
-        if (offset > 0) return;
-
-        fearTracker.style.setProperty('--offset', `${offset - 13}px`);
-    }
-
     _onPosition(position) {
         game.user.setFlag(CONFIG.DH.id, 'app.resources.position', position);
     }
 
+    async close(options = {}) {
+        if (!options.allowed) return;
+        else super.close(options);
+    }
+
     static async setFear(event, target) {
         if (!game.user.isGM) return;
         const fearCount = Number(target.dataset.index ?? 0);
@@ -172,119 +110,4 @@ export default class FearTracker extends HandlebarsApplicationMixin(ApplicationV
             value
         );
     }
-
-    // TODO: Remove methods later to use Foundry's dragger and resize methods 
-    /* -------------------------------------------- */
-    /*  Dragging handlers                           */
-    /* -------------------------------------------- */
-    #setupDragging() {
-        const dragHandle = this.element.querySelector('.drag-handle');
-        if (!dragHandle) return;
-        dragHandle.addEventListener('mousedown', this.#onDragStart.bind(this));
-    }
-
-    #onDragStart(event) {
-        if (event.button !== 0) return;
-        this._dragData.isDragging = true;
-        this._dragData.startX = event.clientX;
-        this._dragData.startY = event.clientY;
-        const rect = this.element.getBoundingClientRect();
-        this._dragData.startLeft = rect.left;
-        this._dragData.startTop = rect.top;
-        this.element.style.cursor = 'grabbing';
-
-        this._dragHandler = this.#onDragging.bind(this);
-        this._dragEndHandler = this.#onDragEnd.bind(this);
-        window.addEventListener('mousemove', this._dragHandler);
-        window.addEventListener('mouseup', this._dragEndHandler);
-    }
-
-    #onDragging(event) {
-        if (!this._dragData.isDragging) return;
-
-        const dragX = event.clientX - this._dragData.startX;
-        const dragY = event.clientY - this._dragData.startY;
-
-        this.element.style.left = `${this._dragData.startLeft + dragX}px`;
-        this.element.style.top = `${this._dragData.startTop + dragY}px`;
-    }
-
-    #onDragEnd() {
-        if (!this._dragData.isDragging) return;
-        this._dragData.isDragging = false;
-        this.element.style.cursor = '';
-
-        if (this._dragHandler) window.removeEventListener('mousemove', this._dragHandler);
-        if (this._dragEndHandler) window.removeEventListener('mouseup', this._dragEndHandler);
-
-        const rect = this.element.getBoundingClientRect();
-        const pos = { top: rect.top, left: rect.left };
-        
-        this.setPosition(pos);
-    }
-
-    /* -------------------------------------------- */
-    /*  Resize handlers                             */
-    /* -------------------------------------------- */
-
-    #setupResizing() {
-        const resizeHandle = this.element.querySelector('.resize-handle');
-        if (!resizeHandle) return;
-        resizeHandle.addEventListener('mousedown', this.#onResizeStart.bind(this));
-    }
-
-    #onResizeStart(e) {
-        if (e.button !== 0) return;
-        e.stopPropagation();
-
-        let maxAllowedWidth = 10000;
-
-        this._resizeData = {
-            isResizing: true,
-            startX: e.clientX,
-            startY: e.clientY,
-            startWidth: this.element.offsetWidth,
-            startHeight: this.element.offsetHeight,
-            maxAllowedWidth: Math.max(50, maxAllowedWidth)
-        };
-
-        this._resizeHandler = this.#onResizing.bind(this);
-        this._resizeEndHandler = this.#onResizeEnd.bind(this);
-        window.addEventListener('mousemove', this._resizeHandler);
-        window.addEventListener('mouseup', this._resizeEndHandler);
-    }
-
-    #onResizing(e) {
-        if (!this._resizeData?.isResizing) return;
-
-        const currentDx = e.clientX - this._resizeData.startX;
-        const potentialWidth = Math.max(50, this._resizeData.startWidth + currentDx);
-
-        const width = Math.min(potentialWidth, this._resizeData.maxAllowedWidth);
-
-        this.element.style.width = `${width}px`;
-
-        if (width < 100) {
-            this.element.classList.add('narrow');
-        } else {
-            this.element.classList.remove('narrow');
-        }
-    }
-
-    #onResizeEnd() {
-        if (!this._resizeData?.isResizing) return;
-        this._resizeData.isResizing = false;
-
-        if (this._resizeHandler) window.removeEventListener('mousemove', this._resizeHandler);
-        if (this._resizeEndHandler) window.removeEventListener('mouseup', this._resizeEndHandler);
-
-        let width = parseFloat(this.element.style.width);
-
-
-        if (isNaN(width)) {
-            width = this.element.getBoundingClientRect().width;
-        }
-
-        this.setPosition({ width: width });
-    }
 }

module/canvas/placeables/regionLayer.mjs

@@ -96,7 +96,7 @@ export default class DhRegionLayer extends foundry.canvas.layers.RegionLayer {
         return inBounds.length === 1 ? inBounds[0] : null;
     }
 
-    static getTemplateShape({ shapeType, angle, range, direction, hasHole } = {}) {
+    static getTemplateShape({ type, angle, range, direction } = {}) {
         const { line, rectangle, inFront, cone, circle, emanation } = CONFIG.DH.GENERAL.templateTypes;
 
         /* Length calculation */
@@ -112,11 +112,11 @@ export default class DhRegionLayer extends foundry.canvas.layers.RegionLayer {
 
         const shapeData = {
             ...canvas.mousePosition,
-            type: shapeType,
+            type: type,
             direction: direction ?? 0
         };
 
-        switch (shapeType) {
+        switch (type) {
             case rectangle.id:
                 shapeData.width = length;
                 shapeData.height = length;
@@ -145,8 +145,7 @@ export default class DhRegionLayer extends foundry.canvas.layers.RegionLayer {
                     y: 0,
                     width: 1,
                     height: 1,
-                    shape: game.canvas.grid.isHexagonal ? CONST.TOKEN_SHAPES.ELLIPSE_1 : CONST.TOKEN_SHAPES.RECTANGLE_1,
+                    shape: game.canvas.grid.isHexagonal ? CONST.TOKEN_SHAPES.ELLIPSE_1 : CONST.TOKEN_SHAPES.RECTANGLE_1
-                    hole: hasHole
                 };
                 break;
         }

module/config/actionConfig.mjs

@@ -119,10 +119,6 @@ export const advantageState = {
 export const areaTypes = {
     placed: {
         id: 'placed',
-        label: 'DAGGERHEART.CONFIG.AreaTypes.placed.label'
+        label: 'Placed Area'
-    },
-    attached: {
-        id: 'attached',
-        label: 'DAGGERHEART.CONFIG.AreaTypes.attached.label'
     }
 };

module/config/encounterConfig.mjs

@@ -90,21 +90,19 @@ export const BPModifiers = {
                     name: 'DAGGERHEART.CONFIG.BPModifiers.increaseDamage.effect.name',
                     description: 'DAGGERHEART.CONFIG.BPModifiers.increaseDamage.effect.description',
                     img: 'icons/magic/control/buff-flight-wings-red.webp',
-                    system: {
                     changes: [
                         {
                             key: 'system.bonuses.damage.physical.dice',
-                                type: 'add',
+                            mode: 2,
                             value: '1d4'
                         },
                         {
                             key: 'system.bonuses.damage.magical.dice',
-                                type: 'add',
+                            mode: 2,
                             value: '1d4'
                         }
                     ]
                 }
-                }
             ]
         }
     },

module/config/generalConfig.mjs

@@ -825,7 +825,7 @@ export const refreshTypes = {
 export const itemAbilityCosts = {
     resource: {
         id: 'resource',
-        label: 'DAGGERHEART.GENERAL.Resource.single',
+        label: 'DAGGERHEART.GENERAL.resource',
         group: 'Global'
     },
     quantity: {
@@ -924,14 +924,6 @@ export const fearDisplay = {
     hide: { value: 'hide', label: 'DAGGERHEART.SETTINGS.Appearance.fearDisplay.hide' }
 };
 
-export const fearPosition = {
-    free: { value: 'free', label: 'DAGGERHEART.SETTINGS.Appearance.fearPosition.free' },
-    topCenter: { value: 'topCenter', label: 'DAGGERHEART.SETTINGS.Appearance.fearPosition.topCenter' },
-    bottomCenter: { value: 'bottomCenter', label: 'DAGGERHEART.SETTINGS.Appearance.fearPosition.bottomCenter' },
-    rightTop: { value: 'rightTop', label: 'DAGGERHEART.SETTINGS.Appearance.fearPosition.rightTop' },
-    leftBottom: { value: 'leftBottom', label: 'DAGGERHEART.SETTINGS.Appearance.fearPosition.leftBottom' }
-};
-
 export const basicOwnershiplevels = {
     0: { value: 0, label: 'OWNERSHIP.NONE' },
     2: { value: 2, label: 'OWNERSHIP.OBSERVER' },

module/config/itemConfig.mjs

@@ -37,16 +37,14 @@ export const armorFeatures = {
                 name: 'DAGGERHEART.CONFIG.ArmorFeature.channeling.effects.channeling.name',
                 description: 'DAGGERHEART.CONFIG.ArmorFeature.channeling.effects.channeling.description',
                 img: 'icons/magic/symbols/rune-sigil-horned-blue.webp',
-                system: {       
                 changes: [
                     {
                         key: 'system.bonuses.roll.spellcast',
-                            type: 'add',
+                        mode: 2,
                         value: '1'
                     }
                 ]
             }
-            }
         ]
     },
     difficult: {
@@ -57,46 +55,44 @@ export const armorFeatures = {
                 name: 'DAGGERHEART.CONFIG.ArmorFeature.difficult.effects.difficult.name',
                 description: 'DAGGERHEART.CONFIG.ArmorFeature.difficult.effects.difficult.description',
                 img: 'icons/magic/control/buff-flight-wings-red.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.traits.agility.value',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     },
                     {
                         key: 'system.traits.strength.value',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     },
                     {
                         key: 'system.traits.finesse.value',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     },
                     {
                         key: 'system.traits.instinct.value',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     },
                     {
                         key: 'system.traits.presence.value',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     },
                     {
                         key: 'system.traits.knowledge.value',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     },
                     {
                         key: 'system.evasion',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     }
                 ]
             }
-            }
         ]
     },
     flexible: {
@@ -107,16 +103,14 @@ export const armorFeatures = {
                 name: 'DAGGERHEART.CONFIG.ArmorFeature.flexible.effects.flexible.name',
                 description: 'DAGGERHEART.CONFIG.ArmorFeature.flexible.effects.flexible.description',
                 img: 'icons/magic/movement/abstract-ribbons-red-orange.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.evasion',
-                            type: 'add',
+                        mode: 2,
                         value: '1'
                     }
                 ]
             }
-            }
         ]
     },
     fortified: {
@@ -127,16 +121,14 @@ export const armorFeatures = {
                 name: 'DAGGERHEART.CONFIG.ArmorFeature.fortified.effects.fortified.name',
                 description: 'DAGGERHEART.CONFIG.ArmorFeature.fortified.effects.fortified.description',
                 img: 'icons/magic/defensive/shield-barrier-glowing-blue.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.rules.damageReduction.increasePerArmorMark',
-                            type: 'override',
+                        mode: 5,
                         value: '2'
                     }
                 ]
             }
-            }
         ]
     },
     gilded: {
@@ -147,16 +139,14 @@ export const armorFeatures = {
                 name: 'DAGGERHEART.CONFIG.ArmorFeature.gilded.effects.gilded.name',
                 description: 'DAGGERHEART.CONFIG.ArmorFeature.gilded.effects.gilded.description',
                 img: 'icons/magic/control/control-influence-crown-gold.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.traits.presence.value',
-                            type: 'add',
+                        mode: 2,
                         value: '1'
                     }
                 ]
             }
-            }
         ]
     },
     heavy: {
@@ -167,16 +157,14 @@ export const armorFeatures = {
                 name: 'DAGGERHEART.CONFIG.ArmorFeature.heavy.effects.heavy.name',
                 description: 'DAGGERHEART.CONFIG.ArmorFeature.heavy.effects.heavy.description',
                 img: 'icons/commodities/metal/ingot-worn-iron.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.evasion',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     }
                 ]
             }
-            }
         ]
     },
     hopeful: {
@@ -224,16 +212,14 @@ export const armorFeatures = {
                 name: 'DAGGERHEART.CONFIG.ArmorFeature.magical.effects.magical.name',
                 description: 'DAGGERHEART.CONFIG.ArmorFeature.magical.effects.magical.description',
                 img: 'icons/magic/defensive/barrier-shield-dome-blue-purple.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.rules.damageReduction.magical',
-                            type: 'override',
+                        mode: 5,
                         value: 1
                     }
                 ]
             }
-            }
         ]
     },
     painful: {
@@ -263,16 +249,14 @@ export const armorFeatures = {
                 name: 'DAGGERHEART.CONFIG.ArmorFeature.physical.effects.physical.name',
                 description: 'DAGGERHEART.CONFIG.ArmorFeature.physical.effects.physical.description',
                 img: 'icons/commodities/stone/ore-pile-tan.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.rules.damageReduction.physical',
-                            type: 'override',
+                        mode: 5,
                         value: 1
                     }
                 ]
             }
-            }
         ]
     },
     quiet: {
@@ -296,21 +280,19 @@ export const armorFeatures = {
                 name: 'DAGGERHEART.CONFIG.ArmorFeature.reinforced.effects.reinforced.name',
                 description: 'DAGGERHEART.CONFIG.ArmorFeature.reinforced.effects.reinforced.description',
                 img: 'icons/magic/defensive/shield-barrier-glowing-triangle-green.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.bunuses.damageThresholds.major',
-                            type: 'add',
+                        mode: 2,
                         value: '2'
                     },
                     {
                         key: 'system.bunuses.damageThresholds.severe',
-                            type: 'add',
+                        mode: 2,
                         value: '2'
                     }
                 ]
             }
-            }
         ]
     },
     resilient: {
@@ -344,21 +326,19 @@ export const armorFeatures = {
                 name: 'DAGGERHEART.CONFIG.ArmorFeature.sharp.effects.sharp.name',
                 description: 'DAGGERHEART.CONFIG.ArmorFeature.sharp.effects.sharp.description',
                 img: 'icons/magic/defensive/shield-barrier-glowing-triangle-green.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.bonuses.damage.primaryWeapon.dice',
-                            type: 'add',
+                        mode: 2,
                         value: '1d4'
                     },
                     {
                         key: 'system.bonuses.damage.secondaryWeapon.dice',
-                            type: 'add',
+                        mode: 2,
                         value: '1d4'
                     }
                 ]
             }
-            }
         ]
     },
     shifting: {
@@ -428,21 +408,19 @@ export const armorFeatures = {
                 name: 'DAGGERHEART.CONFIG.ArmorFeature.veryHeavy.effects.veryHeavy.name',
                 description: 'DAGGERHEART.CONFIG.ArmorFeature.veryHeavy.effects.veryHeavy.description',
                 img: 'icons/commodities/metal/ingot-stamped-steel.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.evasion',
-                            type: 'add',
+                        mode: 2,
                         value: '-2'
                     },
                     {
                         key: 'system.traits.agility.value',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     }
                 ]
             }
-            }
         ]
     },
     warded: {
@@ -453,17 +431,15 @@ export const armorFeatures = {
                 name: 'DAGGERHEART.CONFIG.ArmorFeature.warded.effects.warded.name',
                 description: 'DAGGERHEART.CONFIG.ArmorFeature.warded.effects.warded.description',
                 img: 'icons/magic/defensive/barrier-shield-dome-pink.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.resistance.magical.reduction',
-                            type: 'add',
+                        mode: 2,
                         value: '@system.armorScore',
                         priority: 21
                     }
                 ]
             }
-            }
         ]
     }
 };
@@ -512,11 +488,10 @@ export const weaponFeatures = {
                 name: 'DAGGERHEART.CONFIG.WeaponFeature.barrier.effects.barrier.name',
                 description: 'DAGGERHEART.CONFIG.WeaponFeature.barrier.effects.barrier.description',
                 img: 'icons/skills/melee/shield-block-bash-blue.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.evasion',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     },
                     {
@@ -529,7 +504,6 @@ export const weaponFeatures = {
                     }
                 ]
             }
-            }
         ]
     },
     bonded: {
@@ -540,16 +514,14 @@ export const weaponFeatures = {
                 name: 'DAGGERHEART.CONFIG.WeaponFeature.bonded.effects.damage.name',
                 description: 'DAGGERHEART.CONFIG.WeaponFeature.bonded.effects.damage.description',
                 img: 'icons/magic/symbols/chevron-elipse-circle-blue.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.bonuses.damage.primaryWeapon.bonus',
-                            type: 'add',
+                        mode: 2,
                         value: '@system.levelData.level.current'
                     }
                 ]
             }
-            }
         ]
     },
     bouncing: {
@@ -581,21 +553,19 @@ export const weaponFeatures = {
                 name: 'DAGGERHEART.CONFIG.WeaponFeature.brave.effects.brave.name',
                 description: 'DAGGERHEART.CONFIG.WeaponFeature.brave.effects.brave.description',
                 img: 'icons/magic/life/heart-cross-strong-flame-purple-orange.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.evasion',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     },
                     {
                         key: 'system.damageThresholds.severe',
-                            type: 'add',
+                        mode: 2,
                         value: 'ITEM.@system.tier'
                     }
                 ]
             }
-            }
         ]
     },
     brutal: {
@@ -648,16 +618,14 @@ export const weaponFeatures = {
                         name: 'DAGGERHEART.CONFIG.WeaponFeature.charged.name',
                         description: 'DAGGERHEART.CONFIG.WeaponFeature.charged.description',
                         img: 'icons/magic/lightning/claws-unarmed-strike-teal.webp',
-                        system: {
                         changes: [
                             {
                                 key: 'system.proficiency',
-                                    type: 'add',
+                                mode: 2,
                                 value: '1'
                             }
                         ]
                     }
-                    }
                 ]
             }
         ]
@@ -692,16 +660,14 @@ export const weaponFeatures = {
                 name: 'DAGGERHEART.CONFIG.WeaponFeature.cumbersome.effects.cumbersome.name',
                 description: 'DAGGERHEART.CONFIG.WeaponFeature.cumbersome.effects.cumbersome.description',
                 img: 'icons/commodities/metal/mail-plate-steel.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.traits.finesse.value',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     }
                 ]
             }
-            }
         ]
     },
     deadly: {
@@ -741,17 +707,15 @@ export const weaponFeatures = {
                         name: 'DAGGERHEART.CONFIG.WeaponFeature.deflecting.effects.deflecting.name',
                         description: 'DAGGERHEART.CONFIG.WeaponFeature.deflecting.effects.deflecting.description',
                         img: 'icons/skills/melee/hand-grip-sword-strike-orange.webp',
-                        system: {
                         changes: [
                             {
                                 key: 'system.evasion',
-                                    type: 'add',
+                                mode: 2,
                                 value: '@system.armorScore',
                                 priority: 21
                             }
                         ]
                     }
-                    }
                 ]
             }
         ]
@@ -790,16 +754,14 @@ export const weaponFeatures = {
                 name: 'DAGGERHEART.CONFIG.WeaponFeature.destructive.name',
                 description: 'DAGGERHEART.CONFIG.WeaponFeature.destructive.effects.agility',
                 img: 'icons/skills/melee/strike-flail-spiked-pink.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.traits.agility.value',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     }
                 ]
             }
-            }
         ]
     },
     devastating: {
@@ -829,14 +791,14 @@ export const weaponFeatures = {
                 name: 'DAGGERHEART.CONFIG.WeaponFeature.doubleDuty.effects.doubleDuty.name',
                 description: 'DAGGERHEART.CONFIG.WeaponFeature.doubleDuty.effects.doubleDuty.description',
                 img: 'icons/skills/melee/sword-shield-stylized-white.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.bonuses.damage.primaryWeapon.bonus',
-                            type: 'add',
+                        mode: 2,
                         value: '1'
                     }
                 ],
+                system: {
                     rangeDependence: {
                         enabled: true,
                         range: 'melee',
@@ -940,16 +902,14 @@ export const weaponFeatures = {
                         name: 'DAGGERHEART.CONFIG.WeaponFeature.greedy.actions.greed.name',
                         description: 'DAGGERHEART.CONFIG.WeaponFeature.greedy.actions.greed.description',
                         img: 'icons/commodities/currency/coins-crown-stack-gold.webp',
-                        system: {
                         changes: [
                             {
                                 key: 'system.proficiency',
-                                    type: 'add',
+                                mode: 2,
                                 value: '1'
                             }
                         ]
                     }
-                    }
                 ]
             }
         ]
@@ -991,16 +951,14 @@ export const weaponFeatures = {
                 name: 'DAGGERHEART.CONFIG.WeaponFeature.heavy.effects.heavy.name',
                 description: 'DAGGERHEART.CONFIG.WeaponFeature.heavy.effects.heavy.description',
                 img: 'icons/commodities/metal/ingot-worn-iron.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.evasion',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     }
                 ]
             }
-            }
         ]
     },
     hooked: {
@@ -1108,16 +1066,14 @@ export const weaponFeatures = {
                 name: 'DAGGERHEART.CONFIG.WeaponFeature.massive.effects.massive.name',
                 description: 'DAGGERHEART.CONFIG.WeaponFeature.massive.effects.massive.description',
                 img: 'icons/skills/melee/strike-flail-destructive-yellow.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.evasion',
-                            type: 'add',
+                        mode: 2,
                         value: '-1'
                     }
                 ]
             }
-            }
         ]
     },
     painful: {
@@ -1147,14 +1103,14 @@ export const weaponFeatures = {
                 name: 'DAGGERHEART.CONFIG.WeaponFeature.paired.effects.paired.name',
                 description: 'DAGGERHEART.CONFIG.WeaponFeature.paired.effects.paired.description',
                 img: 'icons/skills/melee/weapons-crossed-swords-yellow-teal.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.bonuses.damage.primaryWeapon.bonus',
-                            type: 'add',
+                        mode: 2,
                         value: 'ITEM.@system.tier + 1'
                     }
                 ],
+                system: {
                     rangeDependence: {
                         enabled: true,
                         range: 'melee',
@@ -1202,16 +1158,14 @@ export const weaponFeatures = {
                         name: 'DAGGERHEART.CONFIG.WeaponFeature.persuasive.effects.persuasive.name',
                         description: 'DAGGERHEART.CONFIG.WeaponFeature.persuasive.effects.persuasive.description',
                         img: 'icons/magic/control/hypnosis-mesmerism-eye.webp',
-                        system: {
                         changes: [
                             {
                                 key: 'system.traits.presence.value',
-                                    type: 'add',
+                                mode: 2,
                                 value: '2'
                             }
                         ]
                     }
-                    }
                 ]
             }
         ]
@@ -1249,7 +1203,6 @@ export const weaponFeatures = {
                 name: 'DAGGERHEART.CONFIG.WeaponFeature.protective.effects.protective.name',
                 description: 'DAGGERHEART.CONFIG.WeaponFeature.protective.effects.protective.description',
                 img: 'icons/skills/melee/shield-block-gray-orange.webp',
-                system: {
                 changes: [
                     {
                         key: 'Armor',
@@ -1262,8 +1215,6 @@ export const weaponFeatures = {
                     }
                 ]
             }
-
-            }
         ]
     },
     quick: {
@@ -1293,16 +1244,14 @@ export const weaponFeatures = {
                 name: 'DAGGERHEART.CONFIG.WeaponFeature.reliable.effects.reliable.name',
                 description: 'DAGGERHEART.CONFIG.WeaponFeature.reliable.effects.reliable.description',
                 img: 'icons/skills/melee/strike-sword-slashing-red.webp',
-                system: {
                 changes: [
                     {
                         key: 'system.bonuses.roll.primaryWeapon.bonus',
-                            type: 'add',
+                        mode: 2,
                         value: 1
                     }
                 ]
             }
-            }
         ]
     },
     reloading: {
@@ -1392,7 +1341,7 @@ export const weaponFeatures = {
                 changes: [
                     {
                         key: 'system.bonuses.damage.primaryWeapon.bonus',
-                        type: 'add',
+                        mode: 2,
                         value: '@system.traits.agility.value',
                         priority: 21
                     }

module/config/settingsConfig.mjs

@@ -27,10 +27,8 @@ export const menu = {
 };
 
 export const gameSettings = {
-    /** @type {'Automation'} */
     Automation: 'Automation',
     Metagaming: 'Metagaming',
-    /** @type {'Homebrew'} */
     Homebrew: 'Homebrew',
     appearance: 'Appearance',
     GlobalOverrides: 'GlobalOverrides',
@@ -39,7 +37,6 @@ export const gameSettings = {
         Fear: 'ResourcesFear'
     },
     LevelTiers: 'LevelTiers',
-    /** @type {'Countdowns'} */
     Countdowns: 'Countdowns',
     LastMigrationVersion: 'LastMigrationVersion',
     SpotlightRequestQueue: 'SpotlightRequestQueue',
@@ -62,18 +59,3 @@ export const actionAutomationChoices = {
         label: 'DAGGERHEART.CONFIG.ActionAutomationChoices.always'
     }
 };
-
-export const reloadChoices = {
-    off: {
-        id: 'off',
-        label: 'DAGGERHEART.CONFIG.ReloadChoices.off.label'
-    },
-    manual: {
-        id: 'manual',
-        label: 'DAGGERHEART.CONFIG.ReloadChoices.manual.label'
-    },
-    auto: {
-        id: 'auto',
-        label: 'DAGGERHEART.CONFIG.ReloadChoices.auto.label'
-    }
-};

module/config/system.mjs

@@ -12,7 +12,6 @@ import * as HOOKS from './hooksConfig.mjs';
 import * as TRIGGER from './triggerConfig.mjs';
 import * as ITEMBROWSER from './itemBrowserConfig.mjs';
 
-/** @type {"daggerheart"} */
 export const SYSTEM_ID = 'daggerheart';
 
 export const SYSTEM = {

module/data/_types.d.ts

@@ -1,7 +0,0 @@
-import { DhCountdown } from './countdowns.mjs'
-
-declare module './countdowns.mjs' {
-    export default interface DhCountdowns {
-        countdowns: Record<string, DhCountdown>;
-    }
-}

module/data/action/attackAction.mjs

@@ -13,19 +13,18 @@ export default class DHAttackAction extends DHDamageAction {
             if (this.damage.includeBase) {
                 const baseDamage = this.getParentHitPointDamage();
                 if (baseDamage) {
-                    if (!this.damage.main) {
+                    if (!this.damage.parts.hitPoints) {
-                        this.damage.main = baseDamage;
+                        this.damage.parts.hitPoints = baseDamage;
                     } else {
-                        for (const type of baseDamage.type) this.damage.main.type.add(type);
+                        for (const type of baseDamage.type) this.damage.parts.hitPoints.type.add(type);
 
-                        this.damage.main.value.custom = {
+                        this.damage.parts.hitPoints.value.custom = {
                             enabled: true,
-                            formula: `${baseDamage.value.getFormula()} + ${this.damage.main.value.getFormula()}`
+                            formula: `${baseDamage.value.getFormula()} + ${this.damage.parts.hitPoints.value.getFormula()}`
                         };
                     }
                 }
             }
-            
             if (this.roll.useDefault) {
                 this.roll.trait = this.item.system.attack.roll.trait;
                 this.roll.type = 'attack';
@@ -34,28 +33,24 @@ export default class DHAttackAction extends DHDamageAction {
     }
 
     getParentHitPointDamage() {
-        return this.item?.system?.attack.damage.main;
+        return this.item?.system?.attack.damage.parts.hitPoints;
     }
 
     get damageFormula() {
-        const hitPointsPart = this.damage.main;
+        const hitPointsPart = this.damage.parts.hitPoints;
         if (!hitPointsPart) return '0';
 
         return hitPointsPart.value.getFormula();
     }
 
     get altDamageFormula() {
-        const hitPointsPart = this.damage.main;
+        const hitPointsPart = this.damage.parts.hitPoints;
         if (!hitPointsPart) return '0';
 
         return hitPointsPart.valueAlt.getFormula();
     }
 
     async use(event, options) {
-        if (this.item?.system.needsReload) {
-            return ui.notifications.error(_loc('DAGGERHEART.UI.Notifications.reloadRequired', { weapon: this.item.name }));
-        }
-
         const result = await super.use(event, options);
 
         if (result?.message?.system.action?.roll?.type === 'attack') {
@@ -66,23 +61,6 @@ export default class DHAttackAction extends DHDamageAction {
         return result;
     }
 
-    async handleReload(options = { awaitRoll: false }) {
-        const roll = await new Roll('1d6').evaluate();
-        if (game.modules.get('dice-so-nice')?.active) {
-            if (options.awaitRoll)
-                await game.dice3d.showForRoll(roll, game.user, true);
-            else
-                game.dice3d.showForRoll(roll, game.user, true);    
-        }
-        
-        const needsReload = roll.total === 1;
-        if (needsReload) {
-            this.item.update({ 'system.resource.value': 0 });
-        }
-
-        return { needsReload, rollValue: roll.total };
-    }
-
     /**
      * Generate a localized label array for this item subtype.
      * @returns {(string | { value: string, icons: string[] })[]} An array of localized strings and damage label objects.
@@ -95,7 +73,7 @@ export default class DHAttackAction extends DHDamageAction {
         if (range) labels.push(game.i18n.localize(`DAGGERHEART.CONFIG.Range.${range}.short`));
 
         const useAltDamage = this.actor?.effects?.find(x => x.type === 'horde')?.active;
-        for (const { value, valueAlt, type } of [damage.main, ...damage.resources].filter(d => !!d)) {
+        for (const { value, valueAlt, type } of damage.parts) {
             const usedValue = useAltDamage ? valueAlt : value;
             const damageString = Roll.replaceFormulaData(usedValue.getFormula(), this.actor?.getRollData() ?? {});
             const str = damageString
@@ -104,7 +82,7 @@ export default class DHAttackAction extends DHDamageAction {
                     x: game.i18n.localize('DAGGERHEART.GENERAL.damage')
                 });
 
-            const icons = Array.from(type ?? [])
+            const icons = Array.from(type)
                 .map(t => CONFIG.DH.GENERAL.damageTypes[t]?.icon)
                 .filter(Boolean);
 

module/data/action/baseAction.mjs

@@ -114,10 +114,7 @@ export default class DHBaseAction extends ActionMixin(foundry.abstract.DataModel
         return this._id;
     }
 
-    /** 
+    /** Returns true if the current user is the owner of the containing item */
-     * Returns true if the current user is the owner of the containing item.
-     * @returns {boolean}
-     */
     get isOwner() {
         return this.item?.isOwner ?? true;
     }
@@ -146,7 +143,6 @@ export default class DHBaseAction extends ActionMixin(foundry.abstract.DataModel
 
     /**
      * Return the first Actor parent found.
-     * @returns {DhpActor | null}
      */
     get actor() {
         return this.item instanceof DhpActor
@@ -159,7 +155,6 @@ export default class DHBaseAction extends ActionMixin(foundry.abstract.DataModel
     /**
      * Returns true if the action is usable.
      * An action is usable on any actor type. For example, an adversary might have a base attack action.
-     * @returns {boolean}
      */
     get usable() {
         const actor = this.actor;
@@ -256,8 +251,7 @@ export default class DHBaseAction extends ActionMixin(foundry.abstract.DataModel
 
         if (Hooks.call(`${CONFIG.DH.id}.postUseAction`, this, config) === false) return;
 
-        if (this.chatDisplay && !config.skips.createMessage && !config.actionChatMessageHandled) 
+        if (this.chatDisplay && !config.skips.createMessage && !config.actionChatMessageHandled) await this.toChat();
-            await this.toChat(null, config);
 
         return config;
     }
@@ -289,6 +283,7 @@ export default class DHBaseAction extends ActionMixin(foundry.abstract.DataModel
             hasEffect: this.hasEffect,
             hasSave: this.hasSave,
             onSave: this.save?.damageMod,
+            isDirect: !!this.damage?.direct,
             selectedMessageMode: game.settings.get('core', 'messageMode'),
             data: this.getRollData(),
             evaluate: this.hasRoll,
@@ -306,20 +301,20 @@ export default class DHBaseAction extends ActionMixin(foundry.abstract.DataModel
         };
 
         if (this.damage) {
-            config.isDirect = !!this.damage.main?.direct;
+            config.isDirect = this.damage.direct;
 
-            const groupAttackTokens = this.damage.main?.groupAttack
+            const groupAttackTokens = this.damage.groupAttack
                 ? game.system.api.fields.ActionFields.DamageField.getGroupAttackTokens(
                     this.actor.id,
-                    this.damage.main.groupAttack
+                    this.damage.groupAttack
                 )
                 : null;
 
             config.damageOptions = {
-                groupAttack: this.damage.main?.groupAttack
+                groupAttack: this.damage.groupAttack
                     ? {
                         numAttackers: Math.max(groupAttackTokens.length, 1),
-                        range: this.damage.main.groupAttack
+                        range: this.damage.groupAttack
                     }
                     : null
             };
@@ -429,11 +424,11 @@ export default class DHBaseAction extends ActionMixin(foundry.abstract.DataModel
     }
 
     get hasDamage() {
-        return this.type !== 'healing' && (Boolean(this.damage?.main) || !foundry.utils.isEmpty(this.damage?.resources));
+        return Boolean(Object.keys(this.damage?.parts ?? {}).length) && this.type !== 'healing';
     }
 
     get hasHealing() {
-        return this.type === 'healing' && !foundry.utils.isEmpty(this.damage?.resources);
+        return Boolean(Object.keys(this.damage?.parts ?? {}).length) && this.type === 'healing';
     }
 
     get hasSave() {
@@ -469,25 +464,6 @@ export default class DHBaseAction extends ActionMixin(foundry.abstract.DataModel
                 return acc;
             }, {});
         }
-
-        if (source.damage?.parts && !source.damage.resources && !source.damage.main) {
-            source.damage.main = null;
-            source.damage.resources = {};
-            for (const [partKey, part] of Object.entries(source.damage.parts)) {
-                if (partKey === 'hitPoints' && source.type !== 'healing') {
-                    source.damage.main = {
-                        ...part,
-                        includeBase: source.damage.includeBase,
-                        direct: source.damage.direct,
-                        groupAttack: source.damage.groupAttack
-                    };
-                } else {
-                    source.damage.resources[partKey] = part;
-                }
-            }
-
-            delete source.damage.parts;
-        }
     }
 }
 

module/data/action/damageAction.mjs

@@ -8,8 +8,11 @@ export default class DHDamageAction extends DHBaseAction {
      * @returns Formula string
      */
     getDamageFormula() {
-        if (!this.damage.main) return '';
+        const strings = [];
+        for (const { value } of this.damage.parts) {
+            strings.push(Roll.replaceFormulaData(value.getFormula(), this.actor?.getRollData() ?? {}));
+        }
 
-        return Roll.replaceFormulaData(this.damage.main.value.getFormula(), this.actor?.getRollData() ?? {});
+        return strings.join(' + ');
     }
 }

module/data/activeEffect/baseEffect.mjs

@@ -57,6 +57,11 @@ export default class BaseEffect extends foundry.data.ActiveEffectTypeDataModel {
                 description: new fields.HTMLField({ label: 'DAGGERHEART.GENERAL.description' })
             }),
             rangeDependence: new fields.SchemaField({
+                enabled: new fields.BooleanField({
+                    required: true,
+                    initial: false,
+                    label: 'DAGGERHEART.GENERAL.enabled'
+                }),
                 type: new fields.StringField({
                     required: true,
                     choices: CONFIG.DH.GENERAL.rangeInclusion,
@@ -75,7 +80,7 @@ export default class BaseEffect extends foundry.data.ActiveEffectTypeDataModel {
                     initial: CONFIG.DH.GENERAL.range.melee.id,
                     label: 'DAGGERHEART.GENERAL.range'
                 })
-            }, { nullable: true, initial: null }),
+            }),
             stacking: new fields.SchemaField(
                 {
                     value: new fields.NumberField({
@@ -141,8 +146,14 @@ export default class BaseEffect extends foundry.data.ActiveEffectTypeDataModel {
             description: '',
             transfer: options.transfer,
             statuses: [],
+            changes: [],
             system: {
-                changes: []
+                rangeDependence: {
+                    enabled: false,
+                    type: CONFIG.DH.GENERAL.rangeInclusion.withinRange.id,
+                    target: CONFIG.DH.GENERAL.otherTargetTypes.hostile.id,
+                    range: CONFIG.DH.GENERAL.range.melee.id
+                }
             }
         };
     }
@@ -174,12 +185,4 @@ export default class BaseEffect extends foundry.data.ActiveEffectTypeDataModel {
         if (this.parent.actor && options.scrollingTextData)
             this.parent.actor.queueScrollText(options.scrollingTextData);
     }
-
-    static migrateData(source) {
-        if (source.rangeDependence?.enabled === false) {
-            source.rangeDependence = null;
-        }
-
-        return super.migrateData(source);
-    }
 }

module/data/actor/adversary.mjs

@@ -84,7 +84,8 @@ export default class DhpAdversary extends DhCreature {
                         type: 'attack'
                     },
                     damage: {
-                        main: {
+                        parts: {
+                            hitPoints: {
                                 type: ['physical'],
                                 applyTo: 'hitPoints',
                                 value: {
@@ -93,6 +94,7 @@ export default class DhpAdversary extends DhCreature {
                             }
                         }
                     }
+                }
             }),
             experiences: new fields.TypedObjectField(
                 new fields.SchemaField({

module/data/actor/character.mjs

@@ -104,7 +104,8 @@ export default class DhCharacter extends DhCreature {
                         trait: 'strength'
                     },
                     damage: {
-                        main: {
+                        parts: {
+                            hitPoints: {
                                 type: ['physical'],
                                 applyTo: 'hitPoints',
                                 value: {
@@ -116,6 +117,7 @@ export default class DhCharacter extends DhCreature {
                             }
                         }
                     }
+                }
             }),
             levelData: new fields.EmbeddedDataField(DhLevelData),
             bonuses: new fields.SchemaField({
@@ -836,7 +838,7 @@ export default class DhCharacter extends DhCreature {
             isReversed: true
         };
 
-        this.attack.damage.main.value.custom.formula = `@prof${this.basicAttackDamageDice}${this.rules.attack.damage.bonus ? ` + ${this.rules.attack.damage.bonus}` : ''}`;
+        this.attack.damage.parts.hitPoints.value.custom.formula = `@prof${this.basicAttackDamageDice}${this.rules.attack.damage.bonus ? ` + ${this.rules.attack.damage.bonus}` : ''}`;
 
         // Clamp resources (must be done last to ensure all updates occur)
         this.resources.clamp();

module/data/actor/tierAdjustment.mjs

@@ -2,14 +2,8 @@ import { calculateExpectedValue, parseTermsFromSimpleFormula } from '../../helpe
 import { adversaryExpectedDamage, adversaryScalingData } from '../../config/actorConfig.mjs';
 import { parseInlineParams } from '../../enrichers/parser.mjs';
 
-/** 
- * Accepts source data for an adversary and a target tier, and returns new source data
- * @type {object} source
- * @type {number} tier
- * @returns {object} adjusted source data
- */
 export function getTierAdjustedAdversary(source, tier) {
-    const currentTier = source.system.tier ?? 1;
+    const currentTier = source.tier ?? 1;
 
     /** @type {(2 | 3 | 4)[]} */
     const tiers = new Array(Math.abs(tier - currentTier))
@@ -40,20 +34,20 @@ export function getTierAdjustedAdversary(source, tier) {
 
     // Store initial attack damage for abilities that have you deal a "standard attack"
     const initialAttack = {
-        type: source.system.attack.damage?.main?.type?.toSorted(),
+        type: source.system.attack.damage?.parts.hitPoints?.type?.toSorted(),
-        value: getFormula(source.system.attack.damage?.main?.value)
+        value: getDamagePartsFormula(source.system.attack.damage?.parts.hitPoints?.value)
     };
 
     // Update damage of base attack.
     try {
         const damage = source.system.attack.damage;
-        if (!damage?.main) throw new Error('Unexpected missing damage in adversary');
+        if (!damage?.parts.hitPoints) throw new Error('Unexpected missing attack in adversary');
 
         for (const property of ['value', 'valueAlt']) {
-            const data = damage.main[property];
+            const data = damage.parts.hitPoints[property];
-            const previousFormula = getFormula(data);
+            const previousFormula = getDamagePartsFormula(data);
-            const value = calculateAdjustedDamage(previousFormula, 'attack', damageMeta);
+            const { value, formula } = calculateAdjustedDamage(previousFormula, 'attack', damageMeta);
-            applyAdjustedDamage(data, value);
+            applyAdjustedDamage(data, value, formula);
         }
     } catch (err) {
         ui.notifications.warn('Failed to convert attack damage of adversary');
@@ -71,7 +65,7 @@ export function getTierAdjustedAdversary(source, tier) {
                 if (!formula) return match;
 
                 try {
-                    const newFormula = getFormula(calculateAdjustedDamage(formula, 'action', damageMeta));
+                    const newFormula = calculateAdjustedDamage(formula, 'action', damageMeta)?.formula;
                     descriptionFormulas.push(formula);
                     return match.replace(formula, newFormula);
                 } catch {
@@ -82,21 +76,21 @@ export function getTierAdjustedAdversary(source, tier) {
 
         // Update damage in item actions and convert all formula matches in the descriptions to the new damage
         for (const action of Object.values(item.system.actions)) {
-            if (!action.damage?.main) continue;
+            if (!action.damage?.parts.hitPoints) continue;
             try {
                 // Apply conversions and save a record. If it matches attack damage *and* Its not in the description, use attack conversion instead
                 const result = [];
                 for (const property of ['value', 'valueAlt']) {
-                    const { [property]: data, type: damageType } = action.damage.main;
+                    const { [property]: data, type: damageType } = action.damage.parts.hitPoints;
-                    const previousFormula = getFormula(data);
+                    const previousFormula = getDamagePartsFormula(data);
                     const isActuallyAttack =
                         previousFormula === initialAttack.value &&
                         foundry.utils.equals(damageType.toSorted(), initialAttack.type) &&
                         !descriptionFormulas.includes(previousFormula);
                     const type = isActuallyAttack ? 'attack' : 'action';
-                    const value = calculateAdjustedDamage(previousFormula, type, damageMeta);
+                    const { value, formula } = calculateAdjustedDamage(previousFormula, type, damageMeta);
-                    applyAdjustedDamage(data, value);
+                    applyAdjustedDamage(data, value, formula);
-                    result.push({ previousFormula, formula: getFormula(value) });
+                    result.push({ previousFormula, formula });
                 }
 
                 // Override text in the description with those values
@@ -195,30 +189,24 @@ function calculateAdjustedDamage(formula, type, { currentDamageRange, newDamageR
         value.bonus = Math.round(expected - getBaseAverage());
     }
 
-    return value;
+    const newFormula = [value.diceQuantity ? `${value.diceQuantity}d${value.faces}` : null, value.bonus]
+        .filter(p => !!p)
+        .join('+');
+    return { value, formula: newFormula };
 }
 
-/** 
+function getDamagePartsFormula(data) {
- * Get formula from either damage data *or* a simple formula object.
+    return data.custom.enabled
- * @returns {string} the new formula data
+        ? data.custom.formula
- */
+        : [data.flatMultiplier ? `${data.flatMultiplier}${data.dice}` : 0, data.bonus ?? 0].filter(p => !!p).join('+');
-function getFormula(data) {
-    if (data.custom?.enabled) {
-        return data.custom.formula;
-    }
-
-    const diceQuantity = data.flatMultiplier ?? data.diceQuantity;
-    const dice = data.faces ? `d${data.faces}` : data.dice;
-    const mod = data.bonus;
-    return [diceQuantity ? `${diceQuantity}${dice}` : 0, mod].filter(p => !!p).join('+');
 }
 
 /**
  * Updates damage to reflect a specific value.
- * @param {object} diceData
+ * @throws if damage structure is invalid for conversion
- * @param {object} value 
+ * @returns the converted formula and value as a simplified term, or null if it doesn't deal HP damage
  */
-function applyAdjustedDamage(diceData, value) {
+function applyAdjustedDamage(diceData, value, formula) {
     if (value.diceQuantity) {
         diceData.custom.enabled = false;
         diceData.bonus = value.bonus;
@@ -226,6 +214,6 @@ function applyAdjustedDamage(diceData, value) {
         diceData.flatMultiplier = value.diceQuantity;
     } else if (!value.diceQuantity) {
         diceData.custom.enabled = true;
-        diceData.custom.formula = getFormula(value);
+        diceData.custom.formula = formula;
     }
 }

module/data/chat-message/actorRoll.mjs

@@ -1,5 +1,4 @@
 import { triggerChatRollFx } from '../../helpers/utils.mjs';
-import { ChatDamageData } from './chatDamageData.mjs';
 
 const fields = foundry.data.fields;
 
@@ -42,7 +41,6 @@ export default class DHActorRoll extends foundry.abstract.TypeDataModel {
             hasEffect: new fields.BooleanField({ initial: false }),
             hasSave: new fields.BooleanField({ initial: false }),
             hasTarget: new fields.BooleanField({ initial: false }),
-            reloadCheckValue: new fields.NumberField({ integer: true, nullable: true, initial: null }),
             isDirect: new fields.BooleanField({ initial: false }),
             onSave: new fields.StringField(),
             source: new fields.SchemaField({
@@ -51,7 +49,7 @@ export default class DHActorRoll extends foundry.abstract.TypeDataModel {
                 originItem: originItemField(),
                 action: new fields.StringField()
             }),
-            damage: new fields.EmbeddedDataField(ChatDamageData),
+            damage: new fields.ObjectField(),
             damageOptions: new fields.ObjectField(),
             costs: new fields.ArrayField(new fields.ObjectField()),
             successConsumed: new fields.BooleanField({ initial: false })
@@ -76,14 +74,10 @@ export default class DHActorRoll extends foundry.abstract.TypeDataModel {
         return fromUuidSync(this.source.actor);
     }
 
-    get item() {
+    get actionItem() {
         const actionActor = this.actionActor;
         if (!actionActor || !this.source.item) return null;
 
-        return actionActor.items.get(this.source.item);
-    }
-
-    get actionItem() {
         switch (this.source.originItem.type) {
             case CONFIG.DH.ITEM.originItemType.restMove:
                 const restMoves = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Homebrew).restMoves;
@@ -91,18 +85,11 @@ export default class DHActorRoll extends foundry.abstract.TypeDataModel {
                     this.source.originItem.actionIndex
                 ];
             default:
-                return this.item?.system.actionsList?.find(a => a.id === this.source.action);
+                const item = actionActor.items.get(this.source.item);
+                return item ? item.system.actionsList?.find(a => a.id === this.source.action) : null;
         }
     }
 
-    get hasReload() {
-        return this.item?.system.hasReload;
-    }
-
-    get reloadCheckFailed() {
-        return this.reloadCheckValue === 1;
-    }
-
     get action() {
         const { actionActor, actionItem: itemAction } = this;
         if (!this.source.action) return null;
@@ -145,21 +132,28 @@ export default class DHActorRoll extends foundry.abstract.TypeDataModel {
         });
     }
 
+    /* TODO: Change how damage data is stored somehow to enable better rerolling */
     async getRerolledDamage() {
-        if (!this.damage.active) return;
+        if (!this.damage) return;
 
         const rerolls = [];
-        const update = { system: { damage: { main: null, resources: _replace({}) } } };
+        const update = { system: { damage: {} } };
-        if (this.damage.main) {
+        for (const partKey in this.damage) {
-            const reroll = await this.damage.main.reroll();
+            const part = this.damage[partKey];
-            rerolls.push(reroll);
+            const testRoll = Roll.fromData(part.parts[0].roll);
-            update.system.damage.main = reroll.toJSON();
+            const rerolled = await testRoll.reroll();
-        }
+            rerolls.push(rerolled);
 
-        for (const key of Object.keys(this.damage.resources)) {
+            if (!update.system.damage[partKey]) update.system.damage[partKey] = { parts: [part.parts[0]] };
-            const reroll = await this.damage.resources[key].reroll();
+            const partData = update.system.damage[partKey].parts[0];
-            rerolls.push(reroll);
+            update.system.damage[partKey].total = rerolled.total;
-            update.system.damage.resources[key] = reroll.toJSON();
+            partData.modifierTotal = rerolled.terms.reduce((acc, x) => {
+                if (x.isDeterministic && !x.operator) acc += x.total;
+                return acc;
+            }, 0);
+            partData.dice = rerolled.dice.map(d => ({ ...d.toJSON(), dice: d.denomination }));
+            partData.total = rerolled.total;
+            partData.roll = rerolled.toJSON();
         }
 
         await triggerChatRollFx(rerolls);
@@ -204,43 +198,6 @@ export default class DHActorRoll extends foundry.abstract.TypeDataModel {
         this.isGM = game.user.isGM; //temp
     }
 
-    static migrateData(source) {
-        const { main, resources, ...flatDamageKeys } = source.damage ?? {};
-        if (source.damage && !main && !resources) {
-            source.damage.main = null;
-            source.damage.resources = {};
-
-            const getRoll = key => {
-                const damageData = source.damage[key];
-                const oldRoll = damageData.parts[0]?.roll;
-                return oldRoll ? JSON.stringify({
-                    ...oldRoll,
-                    class: 'BaseRoll',
-                    options: {
-                        ...oldRoll.options,
-                        damageTypes: damageData.parts[0].damageTypes ?? []
-                    }
-                }) : null;
-            };
-
-            for (const key of Object.keys(flatDamageKeys)) {
-                if (key === 'hitPoints' && source.hasDamage && !source.hasHealing) {
-                    source.damage.main = getRoll('hitPoints');
-                } 
-                else {
-                    source.damage.resources[key] = getRoll(key);
-                }
-            }
-        }
-
-        for (const key of Object.keys(flatDamageKeys)) {
-            delete source.damage[key];
-        }
-        
-        return source;
-    }
-
-
     getTargetList() {
         const targets =
                 this.targetMode && this.parent.isAuthor

module/data/chat-message/chatDamageData.mjs

@@ -1,58 +0,0 @@
-import { triggerChatRollFx } from '../../helpers/utils.mjs';
-
-export class ChatDamageData extends foundry.abstract.DataModel {
-    constructor(data = {}, options = {}) {
-        super(data, options);
-        
-        this._prepareRolls();
-    }
-
-    static defineSchema() {
-        const fields = foundry.data.fields;
-        
-        return {
-            main: new fields.JSONField({ nullable: true, validate: ChatDamageData.#validateRoll}),
-            resources: new fields.TypedObjectField(new fields.JSONField({validate: ChatDamageData.#validateRoll}))
-        };
-    }
-
-    get active() {
-        return !!this.main || Boolean(Object.keys(this.resources).length);
-    }
-
-    static #validateRoll(rollJSON) {
-        if (rollJSON) {
-            const roll = JSON.parse(rollJSON);
-            if (!roll.evaluated) throw new Error('Roll objects added to ChatMessage documents must be evaluated');
-        }
-    }
-
-    _prepareRolls() {
-        this.main &&= Roll.fromData(this.main);
-        for (const key of Object.keys(this.resources)) {
-            this.resources[key] = Roll.fromData(this.resources[key]);
-        }
-    }
-
-    async rerollDamageDie(isResource, damageType, dice, resultIndex) {
-        const reroll = isResource ? this.resources[damageType] : this.main;
-        const rerollDice = reroll.dice[dice];
-        await rerollDice.rerollResult(resultIndex);
-        await reroll._evaluate();
-    
-        const rerolledResult = rerollDice.results[rerollDice.results.length - 1];
-        if (rerolledResult) {
-            const fakeRoll = {
-                _evaluated: true,
-                dice: [new foundry.dice.terms.Die({
-                    ...rerollDice,
-                    results: [rerolledResult],
-                    total: rerolledResult.value,
-                    faces: rerollDice.faces
-                })],
-                options: { appearance: {} }
-            };
-            await triggerChatRollFx([fakeRoll]);
-        }
-    }
-}

module/data/countdowns.mjs

@@ -1,4 +1,3 @@
-import { RefreshType, socketEvent } from '../systemRegistration/socket.mjs';
 import FormulaField from './fields/formulaField.mjs';
 
 export default class DhCountdowns extends foundry.abstract.DataModel {
@@ -15,36 +14,19 @@ export default class DhCountdowns extends foundry.abstract.DataModel {
         };
     }
 
-    /** @inheritdoc */
+    handleChange() {
-    _initialize(options) {
-        super._initialize(options);
-        for (const [id, countdown] of Object.entries(this.countdowns)) {
-            countdown.id = id;
-        }
-    }
-
-    async handleChange() {
         const previousCountdowns = foundry.ui.countdowns.previousCountdownData;
         const changedCountdowns = Object.entries(this.countdowns).reduce((acc, [key, countdown]) => {
-            const previous = previousCountdowns[key];
+            const previousCountdown = previousCountdowns[key];
-            const currentChanged = !previous || (previous.progress.current !== countdown.progress.current);
+            if (!previousCountdown || (previousCountdown.progress.current !== countdown.progress.current)) {
-            if (currentChanged && previous?.progress.start === countdown.progress.start) {
                 acc.push(key);
             }
+
             return acc;
         }, []);
 
-        // Re-render countdowns applications. When the change is due to an actual update, resync the editor
+        for (const countdownKey of changedCountdowns)
-        if (!foundry.utils.equals(previousCountdowns, this.countdowns)) {
+            foundry.ui.countdowns.changedCountdownsForAnimation.add(countdownKey);
-            await foundry.ui.countdowns.render({ animate: changedCountdowns });
-            for (const instance of game.system.api.applications.ui.CountdownEdit.instances()) {
-                instance.data = this;
-                await instance.render();
-            }
-        }
-
-        // Inform modules of updates
-        Hooks.callAll(socketEvent.Refresh, { refreshType: RefreshType.Countdown });
     }
 
     static migrateData(source) {
@@ -178,14 +160,6 @@ export class DhCountdown extends foundry.abstract.DataModel {
         }, {});
     }
 
-    /**
-     * A boolean indicator for whether the current game User has ownership rights for this countdown
-     * @returns {boolean}
-     */
-    get isOwner() {
-        return this.getUserLevel(game.user) === CONST.DOCUMENT_OWNERSHIP_LEVELS.OWNER;
-    }
-
     /** @inheritDoc */
     static migrateData(source) {
         if (source.progress.max) {
@@ -196,29 +170,4 @@ export class DhCountdown extends foundry.abstract.DataModel {
 
         return super.migrateData(source);
     }
-
-    /**
-     * Get the explicit permission level that a User has over this Document, a value in CONST.DOCUMENT_OWNERSHIP_LEVELS.
-     * Compendium content ignores the ownership field in favor of User role-based ownership. Otherwise, Documents use
-     * granular per-User ownership definitions and Embedded Documents defer to their parent ownership.
-     *
-     * @param {BaseUser} [user=game.user] The User being tested
-     * @returns {DocumentOwnershipNumber} A numeric permission level from {@link CONST.DOCUMENT_OWNERSHIP_LEVELS}
-     */
-    getUserLevel(user) {
-        if (user.isGM) return CONST.DOCUMENT_OWNERSHIP_LEVELS.OWNER;
-
-        const setting = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Countdowns);
-        const playerOwnership = this.ownership[user.id];
-        return playerOwnership === undefined || playerOwnership === CONST.DOCUMENT_OWNERSHIP_LEVELS.INHERIT
-            ? setting.defaultOwnership
-            : playerOwnership;
-    }
-
-    async delete() {
-        const setting = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Countdowns);
-        const data = foundry.utils.deepClone(setting._source);
-        delete data.countdowns[this.id];
-        await game.settings.set(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Countdowns, data);
-    }
 }

module/data/fields/action/areasField.mjs

@@ -33,10 +33,6 @@ export default class AreasField extends fields.ArrayField {
                 initial: CONFIG.DH.GENERAL.range.veryClose.id,
                 label: 'DAGGERHEART.ACTIONS.Config.area.size'
             }),
-            hasHole: new fields.BooleanField({ 
-                initial: false,
-                label: 'DAGGERHEART.ACTIONS.Config.area.hasHole' 
-            }),
             effects: new fields.ArrayField(new fields.DocumentIdField())
         });
         super(element, options, context);

module/data/fields/action/damageField.mjs

@@ -12,10 +12,20 @@ export default class DamageField extends fields.SchemaField {
 
     /** @inheritDoc */
     constructor(options, context = {}) {
-        super({
+        const damageFields = {
-            main: new fields.EmbeddedDataField(DHDamageData, { nullable: true }),
+            parts: new IterableTypedObjectField(DHDamageData),
-            resources: new IterableTypedObjectField(DHResourceData)
+            includeBase: new fields.BooleanField({
-        }, options, context);
+                initial: false,
+                label: 'DAGGERHEART.ACTIONS.Settings.includeBase.label'
+            }),
+            direct: new fields.BooleanField({ initial: false, label: 'DAGGERHEART.CONFIG.DamageType.direct.name' }),
+            groupAttack: new fields.StringField({
+                choices: CONFIG.DH.GENERAL.groupAttackRange,
+                blank: true,
+                label: 'DAGGERHEART.ACTIONS.Settings.groupAttack.label'
+            })
+        };
+        super(damageFields, options, context);
     }
 
     /**
@@ -31,23 +41,25 @@ export default class DamageField extends fields.SchemaField {
             this.hasRoll &&
             DamageField.getAutomation() === CONFIG.DH.SETTINGS.actionAutomationChoices.never.id &&
             !force
-        ) {
+        )
             return;
-        }
 
-        const damageFormula = this.damage.main ? 
+        let formulas = this.damage.parts.map(p => ({
-            DamageField.formatFormulas.call(this, [this.damage.main], config)[0] : null;
+            formula: DamageField.getFormulaValue.call(this, p, config).getFormula(this.actor),
-        const resourceFormulas = DamageField.formatFormulas.call(this, this.damage.resources, config);
+            damageTypes: p.applyTo === 'hitPoints' && !p.type.size ? new Set(['physical']) : p.type,
+            applyTo: p.applyTo
+        }));
 
-        if (!damageFormula && !resourceFormulas.length) return false;
+        if (!formulas.length) return false;
+
+        formulas = DamageField.formatFormulas.call(this, formulas, config);
 
         messageId = config.message?._id ?? messageId;
         const message = game.messages.get(messageId);
         const damageConfig = {
             dialog: {},
             ...config,
-            damageFormula,
+            roll: formulas,
-            resourceFormulas, 
             data: this.getRollData(),
             isCritical: Boolean(message?.system.roll?.isCritical)
         };
@@ -80,7 +92,7 @@ export default class DamageField extends fields.SchemaField {
 
         const targetDamage = [];
         const damagePromises = [];
-        for (const target of targets) {
+        for (let target of targets) {
             const actor = foundry.utils.fromUuidSync(target.actorId);
             if (!actor) continue;
             if (!config.hasHealing && config.onSave && target.saved?.success === true) {
@@ -102,12 +114,13 @@ export default class DamageField extends fields.SchemaField {
                     actor.takeHealing(config.damage).then(updates => targetDamage.push({ token, updates }))
                 );
             else {
-                const configDamage = config.damage.clone();
+                const configDamage = foundry.utils.deepClone(config.damage);
-                configDamage.main &&= configDamage.main.toJSON();
+                const hpDamageMultiplier = config.actionActor?.system.rules?.attack?.damage?.hpDamageMultiplier ?? 1;
-                if (configDamage.main) {
+                const hpDamageTakenMultiplier = actor.system.rules?.attack?.damage?.hpDamageTakenMultiplier;
-                    const multiplier = config.actionActor?.system.rules?.attack?.damage?.hpDamageMultiplier ?? 1;
+                if (configDamage.hitPoints) {
-                    const takenMultiplier = actor.system.rules?.attack?.damage?.hpDamageTakenMultiplier;
+                    for (const part of configDamage.hitPoints.parts) {
-                    configDamage.main.total = Math.ceil(configDamage.main.total * multiplier * takenMultiplier);
+                        part.total = Math.ceil(part.total * hpDamageMultiplier * hpDamageTakenMultiplier);
+                    }
                 }
 
                 damagePromises.push(
@@ -170,19 +183,13 @@ export default class DamageField extends fields.SchemaField {
     /**
      * Prepare formulas for Damage Roll
      * Must be called within Action context or similar.
-     * @param {DHResourceData[]} damageData  Array of DHResourceData
+     * @param {object[]} formulas   Array of formatted formulas object
      * @param {object} data         Action getRollData
      * @returns
      */
-    static formatFormulas(damageData, data) {
+    static formatFormulas(formulas, data) {
-        const formulas = damageData.map(x => ({
-            formula: DamageField.getFormulaValue.call(this, x, data).getFormula(this.actor),
-            damageTypes: x.type ?? new Set(),
-            applyTo: x.applyTo
-        }));
-
         const formattedFormulas = [];
-        for (const formula of formulas) {
+        formulas.forEach(formula => {
             if (isNaN(formula.formula))
                 formula.formula = Roll.replaceFormulaData(formula.formula, this.getRollData(data));
             const same = formattedFormulas.find(
@@ -190,8 +197,7 @@ export default class DamageField extends fields.SchemaField {
             );
             if (same) same.formula += ` + ${formula.formula}`;
             else formattedFormulas.push(formula);
-        }
+        });
-
         return formattedFormulas;
     }
 
@@ -269,18 +275,10 @@ export class DHActionDiceData extends foundry.abstract.DataModel {
         };
     }
 
-    /**
-     * @returns {string} the formula associated with this damage field
-     */
     getFormula() {
-        if (this.custom.enabled) return this.custom.formula;
+        const multiplier = this.multiplier === 'flat' ? this.flatMultiplier : `@${this.multiplier}`,
-
+            bonus = this.bonus ? (this.bonus < 0 ? ` - ${Math.abs(this.bonus)}` : ` + ${this.bonus}`) : '';
-        const multiplier = this.multiplier === 'flat' ? this.flatMultiplier : `@${this.multiplier}`;
+        return this.custom.enabled ? this.custom.formula : `${multiplier ?? 1}${this.dice}${bonus}`;
-        if (!multiplier) return String(this.bonus || 0);
-
-        const dice = `${multiplier ?? 1}${this.dice}`;
-        const sign = this.bonus < 0 ? ' - ' : ' + ';
-        return this.bonus ? `${dice} ${sign} ${Math.abs(this.bonus)}` : dice;
     }
 }
 
@@ -288,7 +286,6 @@ export class DHResourceData extends foundry.abstract.DataModel {
     /** @override */
     static defineSchema() {
         return {
-            base: new fields.BooleanField({ initial: false, readonly: true, label: 'Base' }),
             applyTo: new fields.StringField({
                 choices: CONFIG.DH.GENERAL.healingTypes,
                 required: true,
@@ -311,16 +308,7 @@ export class DHDamageData extends DHResourceData {
     static defineSchema() {
         return {
             ...super.defineSchema(),
-            includeBase: new fields.BooleanField({
+            base: new fields.BooleanField({ initial: false, readonly: true, label: 'Base' }),
-                initial: false,
-                label: 'DAGGERHEART.ACTIONS.Settings.includeBase.label'
-            }),
-            direct: new fields.BooleanField({ initial: false, label: 'DAGGERHEART.CONFIG.DamageType.direct.name' }),
-            groupAttack: new fields.StringField({
-                choices: CONFIG.DH.GENERAL.groupAttackRange,
-                blank: true,
-                label: 'DAGGERHEART.ACTIONS.Settings.groupAttack.label'
-            }),
             type: new fields.SetField(
                 new fields.StringField({
                     choices: CONFIG.DH.GENERAL.damageTypes,

module/data/fields/action/summonField.mjs

@@ -1,4 +1,4 @@
-import { getWorldActor, itemAbleRollParse, triggerChatRollFx } from '../../../helpers/utils.mjs';
+import { itemAbleRollParse, triggerChatRollFx } from '../../../helpers/utils.mjs';
 import FormulaField from '../formulaField.mjs';
 
 const fields = foundry.data.fields;
@@ -23,7 +23,7 @@ export default class DHSummonField extends fields.ArrayField {
         super(summonFields, options, context);
     }
 
-    static async execute(config) {
+    static async execute() {
         if (!canvas.scene) {
             ui.notifications.warn(game.i18n.localize('DAGGERHEART.ACTIONS.TYPES.summon.error'));
             return;
@@ -36,14 +36,13 @@ export default class DHSummonField extends fields.ArrayField {
 
         const rolls = [];
         const summonData = [];
-        const chatMessageData = [];
         for (const summon of this.summon) {
             const roll = new Roll(itemAbleRollParse(summon.count, this.actor, this.item));
             await roll.evaluate();
             const count = roll.total;
             if (!roll.isDeterministic) rolls.push(roll);
 
-            const actor = await getWorldActor(await foundry.utils.fromUuid(summon.actorUUID));
+            const actor = await DHSummonField.getWorldActor(await foundry.utils.fromUuid(summon.actorUUID));
             /* Extending summon data in memory so it's available in actionField.toChat. Think it's harmless, but ugly. Could maybe find a better way. */
             summon.actor = actor.toObject();
 
@@ -55,18 +54,30 @@ export default class DHSummonField extends fields.ArrayField {
                     tokenPreviewName: `${actor.prototypeToken.name}${remaining > 1 ? ` (${remaining}x)` : ''}`
                 });
             }
-
-            chatMessageData.push({
-                data: actor,
-                quantity: countNumber
-            });
         }
 
         if (rolls.length) await triggerChatRollFx(rolls);
 
         this.actor.sheet?.minimize();
-        await CONFIG.ux.TokenManager.createTokensWithPreview(summonData, { elevation: this.actor.token?.elevation });
+        DHSummonField.handleSummon(summonData, this.actor);
-        this.actor.sheet?.maximize();
+    }
-        config.summonData = chatMessageData;
+
+    /* Check for any available instances of the actor present in the world if we're missing artwork in the compendium. If none exists, create one. */
+    static async getWorldActor(baseActor) {
+        const dataType = game.system.api.data.actors[`Dh${baseActor.type.capitalize()}`];
+        if (baseActor.inCompendium && dataType && baseActor.img === dataType.DEFAULT_ICON) {
+            const worldActorCopy = game.actors.find(x => x.name === baseActor.name);
+            if (worldActorCopy) return worldActorCopy;
+
+            return await game.system.api.documents.DhpActor.create(baseActor.toObject());
+        }
+
+        return baseActor;
+    }
+
+    static async handleSummon(summonData, actionActor) {
+        await CONFIG.ux.TokenManager.createTokensWithPreview(summonData, { elevation: actionActor.token?.elevation });
+
+        return actionActor.sheet?.maximize();
     }
 }

module/data/fields/action/transformField.mjs

@@ -1,11 +1,5 @@
-import { getWorldActor } from '../../../helpers/utils.mjs';
-
 const fields = foundry.data.fields;
 
-/**
- * @import DHSummonAction from '../../action/summonAction.mjs'
- */
-
 export default class DHSummonField extends fields.SchemaField {
     /**
      * Action Workflow order
@@ -26,11 +20,6 @@ export default class DHSummonField extends fields.SchemaField {
         super(transformFields, options, context);
     }
 
-    /**
-     * Runs the execute. This is run on behalf of DHSummonAction.
-     * @todo move this function to be on the summon action.
-     * @this DHSummonAction
-     */
     static async execute() {
         if (!this.transform.actorUUID) {
             ui.notifications.warn(game.i18n.localize('DAGGERHEART.ACTIONS.TYPES.transform.noTransformActor'));
@@ -48,37 +37,26 @@ export default class DHSummonField extends fields.SchemaField {
             return false;
         }
 
-        const activeTokens = this.actor.getActiveTokens(false, true);
+        if (this.actor.prototypeToken.actorLink) {
-        const controlledMatchingTokens = canvas.tokens.controlled
+            ui.notifications.warn(game.i18n.localize('DAGGERHEART.ACTIONS.TYPES.transform.actorLinkError'));
-            .filter(x => x.actor && x.actor.uuid === this.actor.uuid)
-            .map(x => x.document);
-        /** @type {typeof game.system.api.documents.DhToken | null} */
-        const token = this.actor.token ?? (
-            activeTokens.length === 1 ? activeTokens[0] :
-                (controlledMatchingTokens.length === 1 ? controlledMatchingTokens[0] : null)
-        );
-
-        if (!this.actor.token && !token) {
-            ui.notifications.warn(game.i18n.localize('DAGGERHEART.ACTIONS.TYPES.transform.linkedSelectedError'));
             return false;
         }
 
-        if (!token) {
+        if (!this.actor.token) {
             ui.notifications.warn(game.i18n.localize('DAGGERHEART.ACTIONS.TYPES.transform.prototypeError'));
             return false;
         }
 
-        const actor = await getWorldActor(baseActor);
+        const actor = await DHSummonField.getWorldActor(baseActor);
         const tokenSizes = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Homebrew).tokenSizes;
         const tokenSize = actor?.system.metadata.usesSize ? tokenSizes[actor.system.size] : actor.prototypeToken.width;
 
-        // Update token. Avoid using recursive: false, since that prevents animations
+        await this.actor.token.update(
-        await token.update(
+            { ...actor.prototypeToken.toJSON(), actorId: actor.id, width: tokenSize, height: tokenSize },
-            { ...actor.prototypeToken.toObject(), actorId: actor.id, width: tokenSize, height: tokenSize },
+            { diff: false, recursive: false, noHook: true }
-            { diff: false, noHook: true }
         );
 
-        if (token.combatant) {
+        if (this.actor.token.combatant) {
             this.actor.token.combatant.update({ actorId: actor.id, img: actor.prototypeToken.texture.src });
         }
 
@@ -86,17 +64,17 @@ export default class DHSummonField extends fields.SchemaField {
         if (!this.transform.resourceRefresh.hitPoints) {
             marks.hitPoints = Math.min(
                 this.actor.system.resources.hitPoints.value,
-                token.actor.system.resources.hitPoints.max - 1
+                this.actor.token.actor.system.resources.hitPoints.max - 1
             );
         }
         if (!this.transform.resourceRefresh.stress) {
             marks.stress = Math.min(
                 this.actor.system.resources.stress.value,
-                token.actor.system.resources.stress.max - 1
+                this.actor.token.actor.system.resources.stress.max - 1
             );
         }
         if (marks.hitPoints || marks.stress) {
-            token.actor.update({
+            this.actor.token.actor.update({
                 'system.resources': {
                     hitPoints: { value: marks.hitPoints },
                     stress: { value: marks.stress }
@@ -106,9 +84,20 @@ export default class DHSummonField extends fields.SchemaField {
 
         const prevPosition = { ...this.actor.sheet.position };
         this.actor.sheet.close();
-        token.actor.sheet.render({ force: true, position: prevPosition });
+        this.actor.token.actor.sheet.render({ force: true, position: prevPosition });
-        if (token.object.controlled) {
+    }
-            ui.effectsDisplay.refresh();
+
-        }
+    /* Check for any available instances of the actor present in the world, or create a world actor based on compendium */
+    static async getWorldActor(baseActor) {
+        if (!baseActor.inCompendium) return baseActor;
+
+        const dataType = game.system.api.data.actors[`Dh${baseActor.type.capitalize()}`];
+        if (dataType && baseActor.img === dataType.DEFAULT_ICON) {
+            const worldActorCopy = game.actors.find(x => x.name === baseActor.name);
+            if (worldActorCopy) return worldActorCopy;
+        }
+
+        const worldActor = await game.system.api.documents.DhpActor.create(baseActor.toObject());
+        return worldActor;
     }
 }

module/data/fields/actionField.mjs

@@ -238,7 +238,6 @@ export function ActionMixin(Base) {
                 result = this.parent;
             } else {
                 result = await this.item.update({ [path]: updates }, options);
-                if (!result) return result;
             }
 
             return this.inCollection
@@ -270,7 +269,10 @@ export function ActionMixin(Base) {
             return this.delete();
         }
 
-        async toChat(origin, config) {
+        async toChat(origin) {
+            const autoExpandDescription = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.appearance)
+                .expandRollMessage?.desc;
+
             const cls = getDocumentClass('ChatMessage');
             const systemData = {
                 title: game.i18n.localize('DAGGERHEART.CONFIG.FeatureForm.action'),
@@ -280,7 +282,7 @@ export function ActionMixin(Base) {
                     img: this.baseAction ? this.parent.parent.img : this.img,
                     tags: this.tags ? this.tags : ['Spell', 'Arcana', 'Lv 10'],
                     areas: this.areas,
-                    summon: config?.summonData
+                    summon: this.summon
                 },
                 source: {
                     actor: this.actor.uuid,
@@ -305,7 +307,7 @@ export function ActionMixin(Base) {
                 system: systemData,
                 content: await foundry.applications.handlebars.renderTemplate(
                     'systems/daggerheart/templates/ui/chat/action.hbs',
-                    systemData
+                    { ...systemData, open: autoExpandDescription ? 'open' : '' }
                 ),
                 flags: {
                     daggerheart: {

module/data/item/armor.mjs

@@ -8,8 +8,7 @@ export default class DHArmor extends AttachableItem {
             type: 'armor',
             hasDescription: true,
             isInventoryItem: true,
-            hasActions: true,
+            hasActions: true
-            hasResource: true
         });
     }
 

module/data/item/base.mjs

@@ -49,10 +49,7 @@ export default class BaseDataItem extends foundry.abstract.TypeDataModel {
             })
         };
 
-        if (this.metadata.hasDescription) {
+        if (this.metadata.hasDescription) schema.description = new fields.HTMLField({ required: true, nullable: true });
-            schema.description = new fields.HTMLField({ required: true, nullable: true });
-            schema.gmNotes = new fields.HTMLField({ required: true, nullable: true });
-        }
 
         if (this.metadata.hasResource) {
             schema.resource = new fields.SchemaField(
@@ -137,7 +134,7 @@ export default class BaseDataItem extends foundry.abstract.TypeDataModel {
     /**
      * Augments the description for the item with type specific info to display. Implemented in applicable item subtypes.
      * @param {object} [options] - Options that modify the styling of the rendered template. { headerStyle: undefined|'none'|'large' }
-     * @returns {Promise<{ prefix: string | null; value: string | null; suffix: string | null }>}
+     * @returns {string}
      */
     async getDescriptionData(_options) {
         return { prefix: null, value: this.description, suffix: null };
@@ -146,26 +143,16 @@ export default class BaseDataItem extends foundry.abstract.TypeDataModel {
     /**
      * Gets the enriched and augmented description for the item.
      * @param {object} [options] - Options that modify the styling of the rendered template. { headerStyle: undefined|'none'|'large' }
-     * @returns {Promise<string>}
+     * @returns {string}
      */
-    async getEnrichedDescription({ gmNotes = true } = {}) {
+    async getEnrichedDescription() {
         if (!this.metadata.hasDescription) return '';
 
         const { prefix, value, suffix } = await this.getDescriptionData();
-        let fullDescription = [prefix, value, suffix].filter(p => !!p).join('\n<hr>\n');
+        const fullDescription = [prefix, value, suffix].filter(p => !!p).join('\n<hr>\n');
-        if (this.gmNotes && gmNotes) {
-            const gmNotesElement = document.createElement('section');
-            gmNotesElement.classList.add('gm-notes-section');
-            gmNotesElement.dataset.visibility = 'gm';
-            const header = document.createElement('header');
-            header.classList.add('gm-notes');
-            header.textContent = _loc('DAGGERHEART.ITEMS.FIELDS.gmNotes.label');
-            gmNotesElement.innerHTML = header.outerHTML + this.gmNotes;
-            fullDescription += gmNotesElement.outerHTML;
-        }
 
         return await foundry.applications.ux.TextEditor.implementation.enrichHTML(fullDescription, {
-            relativeTo: this.parent,
+            relativeTo: this,
             rollData: this.getRollData(),
             secrets: this.parent.isOwner
         });

module/data/item/weapon.mjs

@@ -9,8 +9,7 @@ export default class DHWeapon extends AttachableItem {
             type: 'weapon',
             hasDescription: true,
             isInventoryItem: true,
-            hasActions: true,
+            hasActions: true
-            hasResource: true
         });
     }
 
@@ -67,7 +66,8 @@ export default class DHWeapon extends AttachableItem {
                         type: 'attack'
                     },
                     damage: {
-                        main: {
+                        parts: {
+                            hitPoints: {
                                 type: ['physical'],
                                 value: {
                                     multiplier: 'prof',
@@ -76,6 +76,7 @@ export default class DHWeapon extends AttachableItem {
                             }
                         }
                     }
+                }
             }),
             rules: new fields.SchemaField({
                 attack: new fields.SchemaField({
@@ -116,14 +117,6 @@ export default class DHWeapon extends AttachableItem {
         return this.weaponFeatures;
     }
 
-    get hasReload() {
-        return Boolean(this.weaponFeatures.find(x => x.value === 'reloading'));
-    }
-
-    get needsReload() {
-        return this.hasReload && this.resource.value === 0;
-    }
-
     /**@inheritdoc */
     async getDescriptionData() {
         const baseDescription = this.description;
@@ -236,12 +229,11 @@ export default class DHWeapon extends AttachableItem {
             game.i18n.localize(`DAGGERHEART.CONFIG.Burden.${burden}`)
         ];
 
-        if (attack.damage.main) {
+        for (const { value, type } of attack.damage.parts) {
-            const { value, type } = attack.damage.main;
             const parts = value.custom.enabled ? [game.i18n.localize('DAGGERHEART.GENERAL.custom')] : [value.dice];
             if (!value.custom.enabled && value.bonus) parts.push(value.bonus.signedString());
 
-            if (type?.size) {
+            if (type.size > 0) {
                 const typeTags = Array.from(type)
                     .map(t => game.i18n.localize(`DAGGERHEART.CONFIG.DamageType.${t}.abbreviation`))
                     .join(' | ');
@@ -265,10 +257,10 @@ export default class DHWeapon extends AttachableItem {
         if (roll.trait) labels.push(game.i18n.localize(`DAGGERHEART.CONFIG.Traits.${roll.trait}.short`));
         if (range) labels.push(game.i18n.localize(`DAGGERHEART.CONFIG.Range.${range}.short`));
 
-        for (const { value, type } of [damage.main, ...damage.resources].filter(d => !!d)) {
+        for (const { value, type } of damage.parts) {
             const str = Roll.replaceFormulaData(value.getFormula(), this.actor?.getRollData() ?? {});
 
-            const icons = Array.from(type ?? [])
+            const icons = Array.from(type)
                 .map(t => CONFIG.DH.GENERAL.damageTypes[t]?.icon)
                 .filter(Boolean);
 

module/data/settings/Appearance.mjs

@@ -41,11 +41,6 @@ export default class DhAppearance extends foundry.abstract.DataModel {
                 choices: CONFIG.DH.GENERAL.fearDisplay,
                 initial: CONFIG.DH.GENERAL.fearDisplay.token.value
             }),
-            fearPosition: new StringField({
-                required: true,
-                choices: CONFIG.DH.GENERAL.fearPosition,
-                initial: CONFIG.DH.GENERAL.fearPosition.topCenter.value
-            }),
             displayCountdownUI: new BooleanField({ initial: true }),
             diceSoNice: new SchemaField({
                 hope: diceStyle({ fg: '#ffffff', bg: '#ffe760', outline: '#000000', edge: '#ffffff' }),

module/data/settings/Automation.mjs

@@ -196,13 +196,6 @@ export default class DhAutomation extends foundry.abstract.DataModel {
                     })
                 })
             }),
-            reload: new fields.StringField({
-                required: true,
-                choices: CONFIG.DH.SETTINGS.reloadChoices,
-                initial: CONFIG.DH.SETTINGS.reloadChoices.manual.id,
-                label: 'DAGGERHEART.SETTINGS.Automation.FIELDS.reload.label',
-                hint: 'DAGGERHEART.SETTINGS.Automation.FIELDS.reload.hint'
-            }),
             autoExpireActiveEffects: new fields.BooleanField({
                 required: true,
                 initial: true,

module/data/tagTeamData.mjs

@@ -1,5 +1,3 @@
-import { ChatDamageData } from './chat-message/chatDamageData.mjs';
-
 export default class TagTeamData extends foundry.abstract.DataModel {
     static defineSchema() {
         const fields = foundry.data.fields;
@@ -39,7 +37,6 @@ export class MemberData extends foundry.abstract.DataModel {
             }),
             rollChoice: new fields.StringField({ nullable: true, initial: null }),
             rollData: new fields.JSONField({ nullable: true, initial: null }),
-            damageRollData: new fields.EmbeddedDataField(ChatDamageData),
             selected: new fields.BooleanField({ initial: false })
         };
     }

module/dice/baseRoll.mjs

@@ -4,8 +4,4 @@ export default class BaseRoll extends Roll {
 
     /** @inheritdoc */
     static TOOLTIP_TEMPLATE = 'systems/daggerheart/templates/ui/chat/foundryRollTooltip.hbs';
-
-    get modifierTotal() {
-        return this.total - this.dice.reduce((acc, dice) => acc + dice.total, 0);
-    }
 }

module/dice/d20Roll.mjs

@@ -185,10 +185,8 @@ export default class D20Roll extends DHRoll {
         return changeKeys;
     }
 
-    static async buildEvaluate(roll, config = {}, message = {}) {
+    static postEvaluate(roll, config = {}) {
-        await super.buildEvaluate(roll, config, message);
+        const data = super.postEvaluate(roll, config);
-
-        const data = config.roll;
         data.type = config.actionType;
         data.difficulty = config.roll.difficulty;
         if (config.targets?.length) {
@@ -224,6 +222,7 @@ export default class D20Roll extends DHRoll {
                 };
             });
         data.modifierTotal = roll.modifierTotal;
+        return data;
     }
 
     resetFormula() {

module/dice/damageRoll.mjs

@@ -13,34 +13,30 @@ export default class DamageRoll extends DHRoll {
 
     static DefaultDialog = DamageDialog;
 
-    static createRollInstance(config) {
+    static async buildEvaluate(roll, config = {}, message = {}) {
-        return new this(undefined, config.data, config);
+        if (config.dialog.configure === false) roll.constructFormula(config);
-    }
+        if (config.evaluate !== false) for (const roll of config.roll) await roll.roll.evaluate();
-
-    /** @inheritdoc */
-    static async buildEvaluate(roll, config = {}) {
-        if (config.dialog.configure === false) roll.constructFormulas(config);
-        
-        const evaluateRoll = async roll => {
-            await roll.roll.evaluate();
-            roll.roll.options = { damageTypes: roll.damageTypes ? [...roll.damageTypes] : [] };
-            return roll.roll;
-        }
-
-        if (!config.damage) config.damage = { main: null, resources: {} };
-
-        if (config.damageFormula) {
-            config.damage.main = await evaluateRoll(config.damageFormula);
-            config.damage.main.options = { damageTypes: 
-                config.damageFormula.damageTypes ? [...config.damageFormula.damageTypes] : []
-            };
-        }
-        
-        for (const roll of config.resourceFormulas) {
-            config.damage.resources[roll.applyTo] = await evaluateRoll(roll);
-        }
 
         roll._evaluated = true;
+
+        const parts = [];
+        for (const roll of config.roll) {
+            parts.push(this.postEvaluate(roll));
+            roll.roll = JSON.stringify(roll.roll.toJSON());
+        }
+
+        config.damage = this.unifyDamageRoll(parts);
+    }
+
+    static postEvaluate(roll, config = {}) {
+        return {
+            ...roll,
+            ...super.postEvaluate(roll.roll, config),
+            damageTypes: [...(roll.damageTypes ?? [])],
+            roll: roll.roll,
+            type: config.type,
+            modifierTotal: this.calculateTotalModifiers(roll.roll)
+        };
     }
 
     static async buildPost(roll, config, message) {
@@ -51,10 +47,9 @@ export default class DamageRoll extends DHRoll {
         const diceRolls = [];
         if (game.modules.get('dice-so-nice')?.active) {
             config.mute = true;
-            const pool = foundry.dice.terms.PoolTerm.fromRolls([
+            const pool = foundry.dice.terms.PoolTerm.fromRolls(
-                ...(config.damage.main ? [config.damage.main] : []),
+                Object.values(config.damage).flatMap(r => r.parts.map(p => p.roll))
-                ...Object.values(config.damage.resources)
+            );
-            ]);
             diceRolls.push(Roll.fromTerms([pool]));
         }
 
@@ -65,14 +60,22 @@ export default class DamageRoll extends DHRoll {
         await super.buildPost(roll, config, message);
 
         if (config.source?.message) {
-            chatMessage.update({ 'system.damage': {
+            chatMessage.update({ 'system.damage': config.damage });
-                ...config.damage.toObject(),
-                main: config.damage.main,
-                resources: config.damage.resources
-            }});
         }
     }
 
+    static unifyDamageRoll(rolls) {
+        const unified = {};
+        rolls.forEach(r => {
+            const resource = unified[r.applyTo] ?? { formula: '', total: 0, parts: [] };
+            resource.formula += `${resource.formula !== '' ? ' + ' : ''}${r.formula}`;
+            resource.total += r.total;
+            resource.parts.push(r);
+            unified[r.applyTo] = resource;
+        });
+        return unified;
+    }
+
     static formatGlobal(rolls) {
         let formula, total;
         const applyTo = new Set(rolls.flatMap(r => r.applyTo));
@@ -121,9 +124,11 @@ export default class DamageRoll extends DHRoll {
         const type = this.options.messageType ?? (this.options.hasHealing ? 'healing' : 'damage');
         const changeKeys = [];
 
-        for (const damageType of this.options.damageFormula?.damageTypes?.values?.() ?? []) {
+        for (const roll of this.options.roll) {
+            for (const damageType of roll.damageTypes?.values?.() ?? []) {
                 changeKeys.push(`system.bonuses.${type}.${damageType}`);
             }
+        }
 
         const item = this.data.parent?.items?.get(this.options.source.item);
         if (item) {
@@ -140,69 +145,62 @@ export default class DamageRoll extends DHRoll {
         return changeKeys;
     }
 
-    constructFormulas(config) {
+    constructFormula(config) {
-        return {
+        this.options.isCritical = config.isCritical;
-            damageFormula: this.constructFormula(this.options.damageFormula, config, true),
+        for (const [index, part] of this.options.roll.entries()) {
-            resourceFormulas: this.options.resourceFormulas.map(x => this.constructFormula(x, config))
+            const isHitpointPart = part.applyTo === CONFIG.DH.GENERAL.healingTypes.hitPoints.id;
-        };
+            part.roll = new Roll(Roll.replaceFormulaData(part.formula, config.data));
+            part.roll.terms = Roll.parse(part.roll.formula, config.data);
+            if (part.applyTo === CONFIG.DH.GENERAL.healingTypes.hitPoints.id) {
+                part.modifiers = this.applyBaseBonus(part);
+                this.addModifiers(part);
+                part.modifiers?.forEach(m => {
+                    part.roll.terms.push(...this.formatModifier(m.value));
+                });
             }
 
-    constructFormula(formulaData, config, isDamage) {
+            /* To Remove When Reaction System */
-        if (!formulaData) return null;
+            if (index === 0 && part.applyTo === CONFIG.DH.GENERAL.healingTypes.hitPoints.id) {
-        this.options.isCritical = config.isCritical;
+                for (const mod in config.modifiers) {
+                    const modifier = config.modifiers[mod];
+                    if (modifier.beforeCrit === true && (modifier.enabled || modifier.value)) modifier.callback(part);
+                }
+            }
 
-        formulaData.roll = new Roll(Roll.replaceFormulaData(formulaData.formula, config.data));
+            if (part.extraFormula) {
-        formulaData.roll.terms = Roll.parse(formulaData.roll.formula, config.data);
+                part.roll.terms.push(
-
-        if (formulaData.extraFormula) {
-            formulaData.roll.terms.push(
                     new foundry.dice.terms.OperatorTerm({ operator: '+' }),
-                ...this.constructor.parse(formulaData.extraFormula, this.options.data)
+                    ...this.constructor.parse(part.extraFormula, this.options.data)
                 );
             }
 
-        if (isDamage && formulaData.applyTo === CONFIG.DH.GENERAL.healingTypes.hitPoints.id) {
+            if (config.damageOptions.groupAttack?.numAttackers > 1 && isHitpointPart) {
-            formulaData.modifiers = this.applyBaseBonus(formulaData);
-            this.addModifiers(formulaData);
-            formulaData.modifiers?.forEach(m => {
-                formulaData.roll.terms.push(...this.formatModifier(m.value));
-            });
-
-            /* To Remove When Reaction System */
-            for (const mod in config.modifiers) {
-                const modifier = config.modifiers[mod];
-                if (
-                    modifier.beforeCrit === true && 
-                    (modifier.enabled || modifier.value)
-                ) modifier.callback(formulaData);
-            }
-
-            /* To Remove When Reaction System */
-            for (const mod in config.modifiers) {
-                const modifier = config.modifiers[mod];
-                if (!modifier.beforeCrit && (modifier.enabled || modifier.value)) modifier.callback(formulaData);
-            }
-
-            if (config.damageOptions.groupAttack?.numAttackers > 1) {
                 const damageTypes = [foundry.dice.terms.Die, foundry.dice.terms.NumericTerm];
-                for (const term of formulaData.roll.terms) {
+                for (const term of part.roll.terms) {
                     if (damageTypes.some(type => term instanceof type)) {
                         term.number *= config.damageOptions.groupAttack.numAttackers;
                     }
                 }
             }
 
-            if (config.isCritical) {
+            if (config.isCritical && isHitpointPart) {
-                const total = formulaData.roll.dice.reduce((acc, term) => acc + term._faces * term._number, 0);
+                const total = part.roll.dice.reduce((acc, term) => acc + term._faces * term._number, 0);
                 if (total > 0) {
-                    formulaData.roll.terms.push(...this.formatModifier(total));
+                    part.roll.terms.push(...this.formatModifier(total));
-                }
                 }
             }
 
-        formulaData.roll._formula = this.constructor.getFormula(formulaData.roll.terms);
+            /* To Remove When Reaction System */
+            if (index === 0 && part.applyTo === CONFIG.DH.GENERAL.healingTypes.hitPoints.id) {
+                for (const mod in config.modifiers) {
+                    const modifier = config.modifiers[mod];
+                    if (!modifier.beforeCrit && (modifier.enabled || modifier.value)) modifier.callback(part);
+                }
+            }
 
-        return formulaData;
+            part.roll._formula = this.constructor.getFormula(part.roll.terms);
+        }
+        return this.options.roll;
     }
 
     /* To Remove When Reaction System */
@@ -295,4 +293,76 @@ export default class DamageRoll extends DHRoll {
         config.modifiers = mods;
         return mods;
     }
+
+    static async reroll(rollPart, dice, result) {
+        let diceIndex = 0;
+        let parsedRoll = game.system.api.dice.DamageRoll.fromData({
+            ...rollPart.roll,
+            terms: rollPart.roll.terms.map(term => {
+                const isDie = term.class === 'Die';
+                const fixedTerm = {
+                    ...term,
+                    ...(isDie ? { results: rollPart.dice[diceIndex].results } : {})
+                };
+
+                if (isDie) diceIndex++;
+                return fixedTerm;
+            }),
+            class: 'DamageRoll',
+            evaluated: false
+        });
+
+        const parsedDiceTerms = Object.keys(parsedRoll.terms).reduce((acc, key) => {
+            const term = parsedRoll.terms[key];
+            if (term instanceof CONFIG.Dice.termTypes.DiceTerm) acc[Object.keys(acc).length] = term;
+            return acc;
+        }, {});
+        const term = parsedDiceTerms[dice];
+        const termResult = parsedDiceTerms[dice].results[result];
+
+        const newIndex = parsedDiceTerms[dice].results.length;
+        await term.reroll(`/r1=${termResult.result}`);
+
+        const diceRolls = [];
+        if (game.modules.get('dice-so-nice')?.active) {
+            const newResult = parsedDiceTerms[dice].results[newIndex];
+            diceRolls.push({
+                _evaluated: true,
+                dice: [
+                    new foundry.dice.terms.Die({
+                        ...term,
+                        total: newResult.result,
+                        faces: term._faces,
+                        results: [newResult]
+                    })
+                ],
+                options: { appearance: {} }
+            });
+        }
+
+        await triggerChatRollFx(diceRolls);
+        await parsedRoll.evaluate();
+
+        const results = parsedRoll.dice[dice].results.map(result => ({
+            ...result,
+            discarded: !result.active
+        }));
+        const newResult = results.splice(results.length - 1, 1);
+        results.splice(Number(result) + 1, 0, newResult[0]);
+
+        const rerolledDice = parsedRoll.dice.map((x, index) => {
+            const isRerollDice = index === Number(dice);
+            if (!isRerollDice) return { ...x, dice: x.denomination };
+            return {
+                dice: parsedRoll.dice[dice].denomination,
+                total: parsedRoll.dice[dice].total,
+                results: results.map(result => ({
+                    ...result,
+                    hasRerolls: result.hasRerolls || isRerollDice
+                }))
+            };
+        });
+
+        return { parsedRoll, rerolledDice };
+    }
 }

module/dice/dhRoll.mjs

@@ -1,8 +1,7 @@
 import D20RollDialog from '../applications/dialogs/d20RollDialog.mjs';
 import { triggerChatRollFx } from '../helpers/utils.mjs';
-import BaseRoll from './baseRoll.mjs';
 
-export default class DHRoll extends BaseRoll {
+export default class DHRoll extends Roll {
     baseTerms = [];
     constructor(formula, data = {}, options = {}) {
         super(formula, data, foundry.utils.mergeObject(options, { roll: [] }, { overwrite: false }));
@@ -34,17 +33,11 @@ export default class DHRoll extends BaseRoll {
 
         if (config.skips?.createMessage) config.messageRoll = roll;
 
-        if (config.evaluate !== false) {
         await this.buildEvaluate(roll, config, (message = {}));
-        }
         await this.buildPost(roll, config, (message = {}));
         return config;
     }
 
-    static createRollInstance(config) {
-        return new this(config.roll.formula, config.data, config);
-    }
-
     /** 
      * @param {Partial<RollConfig>} config 
      * @returns {Promise<RollConfig>}
@@ -62,7 +55,7 @@ export default class DHRoll extends BaseRoll {
 
         this.temporaryModifierBuilder(config);
 
-        let roll = this.createRollInstance(config);
+        let roll = new this(config.roll.formula, config.data, config);
         if (config.dialog.configure !== false) {
             // Open Roll Dialog
             const DialogClass = config.dialog?.class ?? this.DefaultDialog;
@@ -79,30 +72,13 @@ export default class DHRoll extends BaseRoll {
         return roll;
     }
 
-    /** 
-     * Evaluates the roll and assigns roll data into the config. 
-     * This is only called if config.evaluate is not set to false
-     * @protected
-     */
     static async buildEvaluate(roll, config = {}, message = {}) {
+        if (config.evaluate !== false) {
             await roll.evaluate();
-        config.roll = {
+            config.roll = this.postEvaluate(roll, config);
-            ...roll.options.roll,
+        }
-            total: roll.total,
-            formula: roll.formula,
-            dice: roll.dice.map(d => ({
-                dice: d.denomination,
-                total: d.total,
-                formula: d.formula,
-                results: d.results
-            }))
-        };
     }
 
-    /** 
-     * Runs any post configuration events that need to happen towards the end, such as hooks and dice so nice 
-     * @protected
-     */
     static async buildPost(roll, config, message) {
         for (const hook of config.hooks) {
             if (Hooks.call(`${CONFIG.DH.id}.postRoll${hook.capitalize()}`, config, message) === false) return null;
@@ -115,13 +91,23 @@ export default class DHRoll extends BaseRoll {
         }
     }
 
+    static postEvaluate(roll, config = {}) {
+        return {
+            ...roll.options.roll,
+            total: roll.total,
+            formula: roll.formula,
+            dice: roll.dice.map(d => ({
+                dice: d.denomination,
+                total: d.total,
+                formula: d.formula,
+                results: d.results
+            }))
+        };
+    }
+
     static async toMessage(roll, config) {
         const item = config.data.parent?.items?.get?.(config.source.item) ?? null;
-        const actions = item ? [
+        const action = item ? item.system.actions.get(config.source.action) : null;
-            ...item.system.actions,
-            ...(item.system.attack?.id === config.source.action ? [item.system.attack] : [])
-        ] : [];
-        const action = actions.find(x => x.id === config.source.action);
         let actionDescription = null;
         if (action?.chatDisplay) {
             actionDescription = action
@@ -133,14 +119,6 @@ export default class DHRoll extends BaseRoll {
             config.actionChatMessageHandled = true;
         }
 
-        const reloadSetting = 
-            game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Automation).reload;
-        const useReload = 
-            item?.system.hasReload && 
-            action?.type === 'attack' && 
-            reloadSetting === CONFIG.DH.SETTINGS.reloadChoices.auto.id;
-        const reloadResult = useReload ? await action?.handleReload?.() : {};
-        
         const cls = getDocumentClass('ChatMessage'),
             msgData = {
                 type: this.messageType,
@@ -148,11 +126,7 @@ export default class DHRoll extends BaseRoll {
                 title: roll.title,
                 speaker: cls.getSpeaker({ actor: roll.data?.parent }),
                 sound: config.mute ? null : CONFIG.sounds.dice,
-                system: { 
+                system: { ...config, actionDescription },
-                    ...config, 
-                    actionDescription,
-                    reloadCheckValue: reloadResult.rollValue 
-                },
                 rolls: [roll]
             };
 
@@ -174,17 +148,14 @@ export default class DHRoll extends BaseRoll {
         if (!this._evaluated) return;
 
         const metagamingSettings = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Metagaming);
-        const automationSettings = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Automation);
         const chatData = await this._prepareChatRenderContext({ flavor, isPrivate, ...options });
         return foundry.applications.handlebars.renderTemplate(template, {
             roll: this,
             ...chatData,
-            action: chatData.action,
             parent: chatData.parent,
             targetMode: chatData.targetMode,
             areas: chatData.action?.areas,
-            metagamingSettings,
+            metagamingSettings
-            automationSettings
         });
     }
 

module/dice/die/_module.mjs

@@ -3,7 +3,6 @@ import HopeDie from './hopeDie.mjs';
 import FearDie from './fearDie.mjs';
 import AdvantageDie from './advantageDie.mjs';
 import DisadvantageDie from './disadvantageDie.mjs';
-export { default as BaseDie } from './baseDie.mjs';
 
 export const diceTypes = {
     DualityDie,

module/dice/die/advantageDie.mjs

@@ -1,6 +1,4 @@
-import BaseDie from './baseDie.mjs';
+export default class AdvantageDie extends foundry.dice.terms.Die {
-
-export default class AdvantageDie extends BaseDie {
     constructor(options) {
         super(options);
 

module/dice/die/baseDie.mjs

@@ -1,12 +0,0 @@
-export default class BaseDie extends foundry.dice.terms.Die {
-    async rerollResult(resultIndex) {
-        const result = this.results[resultIndex];
-        result.rerolled = true;
-        result.active = false;
-        await this.roll({ reroll: true });
-
-        const rerolledResult = this.results[this.results.length - 1];
-        this.results.splice(this.results.length - 1, 1);
-        this.results.splice(resultIndex, 0, rerolledResult);
-    }
-}

module/dice/die/disadvantageDie.mjs

@@ -1,6 +1,4 @@
-import BaseDie from './baseDie.mjs';
+export default class DisadvantageDie extends foundry.dice.terms.Die {
-
-export default class DisadvantageDie extends BaseDie {
     constructor(options) {
         super(options);
 

module/dice/die/dualityDie.mjs

@@ -1,7 +1,6 @@
-import BaseDie from './baseDie.mjs';
 import { updateResourcesForDualityReroll } from '../helpers.mjs';
 
-export default class DualityDie extends BaseDie {
+export default class DualityDie extends foundry.dice.terms.Die {
     constructor(options) {
         super(options);
 

module/dice/dualityRoll.mjs

@@ -232,41 +232,9 @@ export default class DualityRoll extends D20Roll {
         return changeKeys;
     }
 
-    /** @inheritdoc */
     static async buildEvaluate(roll, config = {}, message = {}) {
         await super.buildEvaluate(roll, config, message);
-        config.roll.hope = {
-            dice: roll.dHope.denomination,
-            value: this.guaranteedCritical ? 0 : roll.dHope.total,
-            rerolled: {
-                any: roll.dHope.results.some(x => x.rerolled),
-                rerolls: roll.dHope.results.filter(x => x.rerolled)
-            }
-        };
-        config.roll.fear = {
-            dice: roll.dFear.denomination,
-            value: this.guaranteedCritical ? 0 : roll.dFear.total,
-            rerolled: {
-                any: roll.dFear.results.some(x => x.rerolled),
-                rerolls: roll.dFear.results.filter(x => x.rerolled)
-            }
-        };
-        config.roll.rally = {
-            dice: roll.dRally?.denomination,
-            value: roll.dRally?.total
-        };
-        config.roll.result = {
-            duality: roll.withHope ? 1 : roll.withFear ? -1 : 0,
-            total: this.guaranteedCritical ? 0 : roll.dHope.total + roll.dFear.total,
-            label: roll.totalLabel
-        };
 
-        if (roll._rallyIndex && roll.data?.parent)
-            roll.data.parent.deleteEmbeddedDocuments('ActiveEffect', [roll._rallyIndex]);
-    }
-    
-    /** @inheritdoc */
-    static async buildPost(roll, config, message) {
         await setDiceSoNiceForDualityRoll(
             roll,
             config.roll.advantage.type,
@@ -274,7 +242,44 @@ export default class DualityRoll extends D20Roll {
             config.roll.fear.dice,
             config.roll.advantage.dice
         );
+    }
 
+    static postEvaluate(roll, config = {}) {
+        const data = super.postEvaluate(roll, config);
+
+        data.hope = {
+            dice: roll.dHope.denomination,
+            value: this.guaranteedCritical ? 0 : roll.dHope.total,
+            rerolled: {
+                any: roll.dHope.results.some(x => x.rerolled),
+                rerolls: roll.dHope.results.filter(x => x.rerolled)
+            }
+        };
+        data.fear = {
+            dice: roll.dFear.denomination,
+            value: this.guaranteedCritical ? 0 : roll.dFear.total,
+            rerolled: {
+                any: roll.dFear.results.some(x => x.rerolled),
+                rerolls: roll.dFear.results.filter(x => x.rerolled)
+            }
+        };
+        data.rally = {
+            dice: roll.dRally?.denomination,
+            value: roll.dRally?.total
+        };
+        data.result = {
+            duality: roll.withHope ? 1 : roll.withFear ? -1 : 0,
+            total: this.guaranteedCritical ? 0 : roll.dHope.total + roll.dFear.total,
+            label: roll.totalLabel
+        };
+
+        if (roll._rallyIndex && roll.data?.parent)
+            roll.data.parent.deleteEmbeddedDocuments('ActiveEffect', [roll._rallyIndex]);
+
+        return data;
+    }
+
+    static async buildPost(roll, config, message) {
         await super.buildPost(roll, config, message);
 
         await DualityRoll.dualityUpdate(config);
@@ -409,4 +414,8 @@ export default class DualityRoll extends D20Roll {
 
         return rerolled;
     }
+
+    fromJSON(json) {
+        return super.fromJSON(json);
+    }
 }

module/dice/fateRoll.mjs

@@ -75,23 +75,25 @@ export default class FateRoll extends D20Roll {
         this.terms[0] = new foundry.dice.terms.Die({ faces: 12 });
     }
 
-    /** @inheritdoc */
     static async buildEvaluate(roll, config = {}, message = {}) {
         await super.buildEvaluate(roll, config, message);
-        config.roll.fate = {
-            dice: roll.fateDie === 'Hope' ? roll.dHope.denomination : roll.dFear.denomination,
-            value: roll.fateDie === 'Hope' ? roll.dHope.total : roll.dFear.total,
-            fateDie: roll.fateDie
-        };
-    }
 
-    /** @inheritdoc */
-    static async buildPost(roll, config, message) {
         if (roll.fateDie === 'Hope') {
             await setDiceSoNiceForHopeFateRoll(roll, config.roll.fate.dice);
         } else {
             await setDiceSoNiceForFearFateRoll(roll, config.roll.fate.dice);
         }
-        return super.buildPost(roll, config, message);
+    }
+
+    static postEvaluate(roll, config = {}) {
+        const data = super.postEvaluate(roll, config);
+
+        data.fate = {
+            dice: roll.fateDie === 'Hope' ? roll.dHope.denomination : roll.dFear.denomination,
+            value: roll.fateDie === 'Hope' ? roll.dHope.total : roll.dFear.total,
+            fateDie: roll.fateDie
+        };
+
+        return data;
     }
 }

module/documents/_types.d.ts

@@ -3,26 +3,17 @@ import DHItem from './item.mjs';
 import BaseDataItem from '../data/item/base.mjs';
 import DhActiveEffect from './activeEffect.mjs';
 import EmbeddedCollection from '@common/abstract/embedded-collection.mjs';
-import DHToken from './token.mjs';
-import Actor from '@client/documents/actor.mjs';
-import Item from '@client/documents/item.mjs';
 
 declare module './actor.mjs' {
-    export default interface DhpActor<T extends BaseDataActor = BaseDataActor> extends Actor {
+    export default interface DhpActor<T extends BaseDataActor = BaseDataActor> {
         system: T;
         items: EmbeddedCollection<DHItem>;
         effects: EmbeddedCollection<DhActiveEffect>;
-        get token(): DHToken | null;
-
-        /** @inheritdoc */
-        getActiveTokens(linked?: boolean, document?: boolean): (DHToken | foundry.canvas.placeables.Token)[];
-        getActiveTokens(linked?: boolean, document: true): DHToken[];
-        getActiveTokens(linked?: boolean, document: false): foundry.canvas.placeables.Token[];
     }
 }
 
 declare module './item.mjs' {
-    export default interface DHItem<T extends BaseDataItem = BaseDataItem> extends Item {
+    export default interface DHItem<T extends BaseDataItem = BaseDataItem> {
         parent: DhpActor;
         actor: DhpActor;
         system: T;

module/documents/actor.mjs

@@ -34,14 +34,12 @@ export default class DhpActor extends Actor {
         super.prepareData();
 
         // Update effects if it is the user's character or is controlled
-        // A timeout avoids an infinite loop when accessing token actors before the delta is finished constructing
+        if (canvas.ready) {
-        window.setTimeout(() => {
-            if (!canvas.ready) return;
             const controlled = canvas.tokens.controlled.some(t => t.actor === this);
             if (game.user.character === this || controlled) {
-                ui.effectsDisplay.refresh();
+                ui.effectsDisplay.render();
+            }
         }
-        }, 0);
     }
 
     /* -------------------------------------------- */
@@ -622,63 +620,53 @@ export default class DhpActor extends Actor {
         return rollData;
     }
 
-    /** 
+    #canReduceDamage(hpDamage, type) {
-     * Checks to see if damage can be reduced in one way or another.
+        const { stressDamageReduction, disabledArmor } = this.system.rules.damageReduction;
-     * @param {number} hpDamage the amount of marked hp that will be marked
-     * @param {Set<string>} types a list of damage types
-     */
-    #canReduceDamage(hpDamage, types) {
-        const { stressDamageReduction, disabledArmor, reduceSeverity, thresholdImmunities } = 
-            this.system.rules.damageReduction;
         if (disabledArmor) return false;
 
         const availableStress = this.system.resources.stress.max - this.system.resources.stress.value;
 
         const canUseArmor =
             this.system.armorScore.value < this.system.armorScore.max &&
-            types.every(t => this.system.armorApplicableDamageTypes[t] === true);
+            type.every(t => this.system.armorApplicableDamageTypes[t] === true);
-
         const canUseStress = Object.keys(stressDamageReduction).reduce((acc, x) => {
             const rule = stressDamageReduction[x];
             if (damageKeyToNumber(x) <= hpDamage) return acc || (rule.enabled && availableStress >= rule.cost);
             return acc;
         }, false);
 
-        const hasReduceSeverity = types.some(t => reduceSeverity[t]);
+        return canUseArmor || canUseStress;
-        
-        const hasThresholdImmunity = Object.entries(thresholdImmunities)
-            .filter(([key, value]) => Boolean(value) && damageKeyToNumber(key) === hpDamage)
-            .length;
-
-        return canUseArmor || canUseStress || hasReduceSeverity || hasThresholdImmunity;
     }
 
-    async takeDamage(args, isDirect = false) {
+    async takeDamage(damages, isDirect = false) {
-        args = this.#parseDamageArgs(args);
+        if (Hooks.call(`${CONFIG.DH.id}.preTakeDamage`, this, damages) === false) return null;
-        if (Hooks.call(`${CONFIG.DH.id}.preTakeDamage`, this, args) === false) return null;
 
         if (this.type === 'companion') {
             await this.modifyResource([{ value: 1, key: 'stress' }]);
             return;
         }
 
-        const updates = args.resourceUpdates;
+        const updates = [];
-        if (args.main) {
-            // todo: avoid side effects, but hook currently requires it
-            args.main.value = this.calculateDamage(args.main.value, args.main.damageTypes);
-        }
 
-        if (Hooks.call(`${CONFIG.DH.id}.postCalculateDamage`, this, args) === false) return null;
+        Object.entries(damages).forEach(([key, damage]) => {
+            damage.parts.forEach(part => {
+                if (part.applyTo === CONFIG.DH.GENERAL.healingTypes.hitPoints.id)
+                    part.total = this.calculateDamage(part.total, part.damageTypes);
+                const update = updates.find(u => u.key === key);
+                if (update) {
+                    update.value += part.total;
+                    update.damageTypes.add(...new Set(part.damageTypes));
+                } else updates.push({ value: part.total, key, damageTypes: new Set(part.damageTypes) });
+            });
+        });
 
-        // Convert deducted resources to a record of updates. Return if nothing to do.
+        if (Hooks.call(`${CONFIG.DH.id}.postCalculateDamage`, this, damages) === false) return null;
-        if (!updates.some(u => u.value) && !args.main) return; 
 
-        if (args.main) {
+        if (!updates.length) return;
-            const hpDamage = { 
+
-                value: this.convertDamageToThreshold(args.main.value),
+        const hpDamage = updates.find(u => u.key === CONFIG.DH.GENERAL.healingTypes.hitPoints.id);
-                damageTypes: new Set(args.main.damageTypes), 
+        if (hpDamage?.value) {
-                key: CONFIG.DH.GENERAL.healingTypes.hitPoints.id
+            hpDamage.value = this.convertDamageToThreshold(hpDamage.value);
-            };
             if (this.type === 'character' && !isDirect && this.#canReduceDamage(hpDamage.value, hpDamage.damageTypes)) {
                 const armorSlotResult = await this.owner.query(
                     'armorSlot',
@@ -693,7 +681,7 @@ export default class DhpActor extends Actor {
                 );
                 if (armorSlotResult) {
                     const { modifiedDamage, armorChanges, stressSpent } = armorSlotResult;
-                    hpDamage.value = modifiedDamage;
+                    updates.find(u => u.key === 'hitPoints').value = modifiedDamage;
                     for (const armorChange of armorChanges) {
                         updates.push({ value: armorChange.amount, key: 'armor', uuid: armorChange.uuid });
                     }
@@ -703,23 +691,19 @@ export default class DhpActor extends Actor {
                         else updates.push({ value: stressSpent, key: 'stress' });
                     }
                 }
-            } else if (this.type === 'adversary') {
+            }
+            if (this.type === 'adversary') {
                 const reducedSeverity = hpDamage.damageTypes.reduce((value, curr) => {
                     return Math.max(this.system.rules.damageReduction.reduceSeverity[curr], value);
                 }, 0);
                 hpDamage.value = Math.max(hpDamage.value - reducedSeverity, 0);
-                if (this.system.rules.damageReduction.thresholdImmunities[getDamageKey(hpDamage.value)]) {
-                    hpDamage.value = Math.max(0, hpDamage.value - 1);
-                }
-            }
 
-            // Merge existing hitPoint deduction with finalised damage deduction
+                if (
-            const existing = updates.find(u => u.key === CONFIG.DH.GENERAL.healingTypes.hitPoints.id);
+                    hpDamage.value &&
-            if (existing) {
+                    this.system.rules.damageReduction.thresholdImmunities[getDamageKey(hpDamage.value)]
-                existing.value += hpDamage.value;
+                ) {
-                existing.damageTypes = hpDamage.damageTypes;
+                    hpDamage.value -= 1;
-            } else {
+                }
-                updates.push(hpDamage);
             }
         }
 
@@ -736,9 +720,11 @@ export default class DhpActor extends Actor {
             resourceMap.updateResources();
         }
 
-        for (const u of updates) {
+        updates.forEach(
-            u.value = u.key === 'fear' || this.system?.resources?.[u.key]?.isReversed === false ? u.value * -1 : u.value;
+            u =>
-        }
+                (u.value =
+                    u.key === 'fear' || this.system?.resources?.[u.key]?.isReversed === false ? u.value * -1 : u.value)
+        );
 
         await this.modifyResource(updates);
 
@@ -747,36 +733,6 @@ export default class DhpActor extends Actor {
         return updates;
     }
 
-    async takeHealing(args) {
-        args = this.#parseDamageArgs({ resources: 'resources' in args ? args.resources : args });
-        if (Hooks.call(`${CONFIG.DH.id}.preTakeHealing`, this, args) === false) return null;
-
-        const updates = args.resourceUpdates;
-        for (const u of updates) {
-            const shouldFlip = !(u.key === 'fear' || this.system?.resources?.[u.key]?.isReversed === false);
-            u.value = shouldFlip ? u.value * -1 : u.value;
-        }
-        await this.modifyResource(updates);
-
-        if (Hooks.call(`${CONFIG.DH.id}.postTakeHealing`, this, updates) === false) return null;
-
-        return updates;
-    }
-
-    /** Parse damage args that may be coming from takeHealing or takeDamage. Used to simplify macro usage and roll vs non-roll usage */
-    #parseDamageArgs(args = {}) {
-        const damageRoll = 'total' in args ? args : (args.main ?? args.damage);
-        const damageValue = typeof damageRoll === 'number' ? damageRoll : damageRoll?.total;
-        const damageTypes = Array.from(damageRoll?.options?.damageTypes ?? damageRoll?.damageTypes ?? []);
-        const main = typeof damageValue === 'number' ? { key: 'damage', value: damageValue, damageTypes } : null;
-        const resourceUpdates = Object.entries(args.resources ?? {}).map(([key, damage]) => ({ 
-            key, 
-            value: typeof damage === 'number' ? damage : damage?.total ?? 0 
-        }));
-
-        return { main, resourceUpdates };
-    }
-
     calculateDamage(baseDamage, type) {
         if (this.canResist(type, 'immunity')) return 0;
         if (this.canResist(type, 'resistance')) baseDamage = Math.ceil(baseDamage / 2);
@@ -801,6 +757,32 @@ export default class DhpActor extends Actor {
         return reduction === Infinity ? 0 : reduction;
     }
 
+    async takeHealing(healings) {
+        if (Hooks.call(`${CONFIG.DH.id}.preTakeHealing`, this, healings) === false) return null;
+
+        const updates = [];
+        Object.entries(healings).forEach(([key, healing]) => {
+            healing.parts.forEach(part => {
+                const update = updates.find(u => u.key === key);
+                if (update) update.value += part.total;
+                else updates.push({ value: part.total, key });
+            });
+        });
+
+        updates.forEach(
+            u =>
+                (u.value = !(u.key === 'fear' || this.system?.resources?.[u.key]?.isReversed === false)
+                    ? u.value * -1
+                    : u.value)
+        );
+
+        await this.modifyResource(updates);
+
+        if (Hooks.call(`${CONFIG.DH.id}.postTakeHealing`, this, updates) === false) return null;
+
+        return updates;
+    }
+
     /**
      * Resources are modified asynchronously, so be careful not to update the same resource in
      * quick succession.

module/documents/chatMessage.mjs

@@ -3,13 +3,6 @@ import { emitGMUpdate, emitGMCreate, GMUpdateEvent } from '../systemRegistration
 export default class DhpChatMessage extends foundry.documents.ChatMessage {
     targetHook = null;
 
-    static #EXPAND_SECTIONS = [
-        { selector: 'roll-section [data-action="expandRoll"]', key: 'roll' },
-        { selector: 'damage-section', key: 'damage' },
-        { selector: 'target-section', key: 'target' },
-        { selector: 'description-section', key: 'desc' }
-    ];
-
     async renderHTML() {
         const actor = game.actors.get(this.speaker.actor);
         const actorData =
@@ -96,26 +89,23 @@ export default class DhpChatMessage extends foundry.documents.ChatMessage {
                 }
             }
 
-            // Check registered selectors and the main item section for expanding
-            // Preserving during re-render is handled by core foundry on anything with [data-action=expandRoll]
             const autoExpandRoll = game.settings.get(
-                CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.appearance
+                    CONFIG.DH.id,
-            ).expandRollMessage;
+                    CONFIG.DH.SETTINGS.gameSettings.appearance
-            for (const { selector, key } of DhpChatMessage.#EXPAND_SECTIONS) {
+                ).expandRollMessage,
-                const elements = html.querySelectorAll(selector);
+                rollSections = html.querySelectorAll('.roll-part'),
-                for (const element of elements) {
+                itemDesc = html.querySelector('.domain-card-move');
-                    element.classList.toggle('expanded', autoExpandRoll[key]);
+            rollSections.forEach(s => {
-                }
+                if (s.classList.contains('roll-section')) {
-            }
+                    const toExpand = s.querySelector('[data-action="expandRoll"]');
-
+                    toExpand.classList.toggle('expanded', autoExpandRoll.roll);
-            // Auto expand the item description. These are not preserved by foundry during re-renders
+                } else if (s.classList.contains('damage-section'))
-            const itemDesc = html.querySelector('details');
+                    s.classList.toggle('expanded', autoExpandRoll.damage);
-            if (itemDesc) {
+                else if (s.classList.contains('target-section')) s.classList.toggle('expanded', autoExpandRoll.target);
-                const existing = document.querySelector(`.chat-message[data-message-id="${this.id}"] details`);
+                else if (s.classList.contains('description-section'))
-                if (existing?.hasAttribute('open') ?? autoExpandRoll.desc) {
+                    s.classList.toggle('expanded', autoExpandRoll.desc);
-                    itemDesc.setAttribute('open', '');
+            });
-                }
+            if (itemDesc && autoExpandRoll.desc) itemDesc.setAttribute('open', '');
-            }
         }
 
         if (!this.isAuthor && !this.speakerActor?.isOwner) {
@@ -286,12 +276,8 @@ export default class DhpChatMessage extends foundry.documents.ChatMessage {
     async onCreateAreas(event) {
         const createArea = async selectedArea => {
             const effects = selectedArea.effects.map(effect => this.system.action.item.effects.get(effect).uuid);
-            const { shape: shapeType, size: range, hasHole } = selectedArea;
+            const { shape: type, size: range } = selectedArea;
-            const shapeData = CONFIG.Canvas.layers.regions.layerClass.getTemplateShape({ 
+            const shapeData = CONFIG.Canvas.layers.regions.layerClass.getTemplateShape({ type, range });
-                shapeType, 
-                range, 
-                hasHole 
-            });
 
             const scene = game.scenes.get(game.user.viewedScene);
             const level = scene.levels.find(x => x.isView);
@@ -319,10 +305,7 @@ export default class DhpChatMessage extends foundry.documents.ChatMessage {
                 visibility: CONST.REGION_VISIBILITY.ALWAYS
             };
             const placeRegion = data => {
-                canvas.regions.placeRegion(data, { 
+                canvas.regions.placeRegion(data, { create: true });
-                    create: true, 
-                    attachToToken: selectedArea.type === CONFIG.DH.ACTIONS.areaTypes.attached.id 
-                });
             };
 
             // Regions with effects must be placed by the GM

module/documents/item.mjs

@@ -208,7 +208,10 @@ export default class DHItem extends foundry.documents.Item {
                 tags: this._getTags()
             },
             actions: item.system.actionsList,
-            description: await this.system.getEnrichedDescription()
+            description: await foundry.applications.ux.TextEditor.implementation.enrichHTML(this.system.description, {
+                relativeTo: this.parent,
+                rollData: this.parent?.getRollData() ?? {}
+            })
         };
 
         const msg = {

module/documents/token.mjs

@@ -38,8 +38,7 @@ export default class DHToken extends CONFIG.Token.documentClass {
                 tokens.filter(x => x.actor).map(x => x.actor)
             );
         }
-        
+        super.createCombatants(tokens, combat ?? {});
-        await super.createCombatants(tokens, combat ?? {});
     }
 
     /**@inheritdoc */

module/documents/tokenManager.mjs

@@ -19,10 +19,10 @@ export default class DhTokenManager {
             }
         }
 
-        const placedData = await canvas.tokens.placeTokens(
+        return await canvas.tokens.placeTokens(
             [
                 {
-                    ...(await actor.getTokenDocument()).toObject(),
+                    ...actor.prototypeToken.toObject(),
                     actorId: actor.id,
                     displayName: 50,
                     ...tokenData
@@ -30,8 +30,6 @@ export default class DhTokenManager {
             ],
             { create: false }
         );
-
-        return placedData?.[0] ?? null;
     }
 
     /**
@@ -48,24 +46,22 @@ export default class DhTokenManager {
 
         const createElevation = elevation ?? level.elevation.bottom;
         for (const tokenData of tokensData) {
-            const previewToken = await this.createPreview(tokenData.actor, {
+            const previewTokens = await this.createPreview(tokenData.actor, {
                 name: tokenData.tokenPreviewName,
                 level: game.user.viewedLevel,
                 elevation: createElevation,
                 flags: { daggerheart: { createPlacement: true } }
             });
-            if (!previewToken) return null;
+            if (!previewTokens?.length) return null;
-
-            const finalTokenData = {
-                ...previewToken.toObject(),
-                name: tokenData.actor.prototypeToken.name,
-                displayName: tokenData.actor.prototypeToken.displayName,
-                flags: tokenData.actor.prototypeToken.flags
-            };
 
             await canvas.scene.createEmbeddedDocuments(
                 'Token',
-                [finalTokenData],
+                previewTokens.map(x => ({
+                    ...x.toObject(),
+                    name: tokenData.actor.prototypeToken.name,
+                    displayName: tokenData.actor.prototypeToken.displayName,
+                    flags: tokenData.actor.prototypeToken.flags
+                })),
                 { controlObject: true, parent: canvas.scene }
             );
         }

module/documents/tooltipManager.mjs

@@ -4,55 +4,15 @@ export default class DhTooltipManager extends foundry.helpers.interaction.Toolti
     #wide = false;
     #bordered = false;
 
-    /** @inheritdoc */
     async activate(element, options = {}) {
-        this.#wide = false;
+        const { TextEditor } = foundry.applications.ux;
-        this.#bordered = false;
-
-        const isMacro = document.getElementById('action-bar').contains(element);
-        const macro = isMacro ? game.macros.get(game.user.hotbar[Number(element.dataset.slot)] ?? null) : null;
-        const macroItemUuid = macro?.type === 'script' ? macro.command.match(/await game\.system\.api\.applications\.ui\.DhHotbar\.useItem\("([^"]+)"\);/)?.[1] : null;
-        if (macroItemUuid && await fromUuid(macroItemUuid, { strict: false })) {
-            element.dataset.tooltip = `#item#${macroItemUuid}`;
-            options.direction = this.constructor.TOOLTIP_DIRECTIONS.UP;
-        }
 
         let html = options.html;
-        const key = element.dataset.tooltip?.match(/^#([\w-]+)#/)?.[1];
+        if (element.dataset.tooltip?.startsWith('#battlepoints#')) {
-        switch (key) {
-            case 'battlepoints':
-                return this.#activateBattlepoints(element, options);
-            case 'effect-display':
-                html = await this.#activateEffectDisplay(element, options);
-                break;
-            case 'item':
-                html = await this.#activateItem(element, options);
-                break;
-            case 'attack':
-                html = await this.#activateAttack(element, options);
-                break;
-            case 'shortRest':
-            case 'longRest':
-                html = await this.#activateRest(element, options);
-                break;
-            case 'advantage':
-            case 'disadvantage':
-                html = await this.#activateAdvantageDisadvantage(element, options);
-                break;
-            case 'deathMove':
-                html = await this.#activateDeathMove(element, options);
-                break;
-        }
-
-        this.noOffset = options.noOffset;
-        super.activate(element, { ...options, html });
-    }
-
-    async #activateBattlepoints(element, options) {
             this.#wide = true;
             this.#bordered = true;
 
-        const html = await this.getBattlepointHTML(element.dataset.combatId);
+            html = await this.getBattlepointHTML(element.dataset.combatId);
             options.direction = this._determineItemTooltipDirection(element);
             super.activate(element, { ...options, html: html });
 
@@ -60,9 +20,13 @@ export default class DhTooltipManager extends foundry.helpers.interaction.Toolti
             lockedTooltip.querySelectorAll('.battlepoint-toggle-container input').forEach(element => {
                 element.addEventListener('input', this.toggleModifier.bind(this));
             });
+            return;
+        } else {
+            this.#wide = false;
+            this.#bordered = false;
         }
 
-    async #activateEffectDisplay(element, options) {
+        if (element.dataset.tooltip === '#effect-display#') {
             this.#bordered = true;
             let effect = {};
             if (element.dataset.uuid) {
@@ -111,7 +75,7 @@ export default class DhTooltipManager extends foundry.helpers.interaction.Toolti
                 };
             }
 
-        const html = await foundry.applications.handlebars.renderTemplate(
+            html = await foundry.applications.handlebars.renderTemplate(
                 `systems/daggerheart/templates/ui/tooltip/effect-display.hbs`,
                 {
                     effect
@@ -120,11 +84,11 @@ export default class DhTooltipManager extends foundry.helpers.interaction.Toolti
 
             this.tooltip.innerHTML = html;
             options.direction = this._determineItemTooltipDirection(element);
-
+        } else {
-        return html;
+            this.#bordered = false;
         }
 
-    async #activateItem(element, options) {
+        if (element.dataset.tooltip?.startsWith('#item#')) {
             const itemUuid = element.dataset.tooltip.slice(6);
             const item = await foundry.utils.fromUuid(itemUuid);
             if (item) {
@@ -133,7 +97,7 @@ export default class DhTooltipManager extends foundry.helpers.interaction.Toolti
                 await this.enrichText(item);
 
                 const type = isAction ? 'action' : isEffect ? 'effect' : item.type;
-            const html = await foundry.applications.handlebars.renderTemplate(
+                html = await foundry.applications.handlebars.renderTemplate(
                     `systems/daggerheart/templates/ui/tooltip/${type}.hbs`,
                     {
                         item: item,
@@ -144,20 +108,17 @@ export default class DhTooltipManager extends foundry.helpers.interaction.Toolti
                 );
 
                 this.tooltip.innerHTML = html;
-            options.direction ??= this._determineItemTooltipDirection(element);
+                options.direction = this._determineItemTooltipDirection(element);
-            return html;
             }
-
+        } else {
-        return null;
+            const attack = element.dataset.tooltip?.startsWith('#attack#');
-    }
+            if (attack) {
-
-    async #activateAttack(element, options) {
                 const actorUuid = element.dataset.tooltip.slice(8);
                 const actor = await foundry.utils.fromUuid(actorUuid);
                 const attack = actor.system.attack;
 
-        const description = await foundry.applications.ux.TextEditor.enrichHTML(attack.description);
+                const description = await TextEditor.enrichHTML(attack.description);
-        const html = await foundry.applications.handlebars.renderTemplate(
+                html = await foundry.applications.handlebars.renderTemplate(
                     `systems/daggerheart/templates/ui/tooltip/attack.hbs`,
                     {
                         attack: attack,
@@ -168,57 +129,19 @@ export default class DhTooltipManager extends foundry.helpers.interaction.Toolti
                 );
 
                 this.tooltip.innerHTML = html;
-        return html;
             }
 
-    async #activateAdvantageDisadvantage(element, options) {
+            const shortRest = element.dataset.tooltip?.startsWith('#shortRest#');
-        const isAdvantage = element.dataset.tooltip?.startsWith('#advantage#');
+            const longRest = element.dataset.tooltip?.startsWith('#longRest#');
-        const actorUuid = element.dataset.tooltip.slice(isAdvantage ? 11 : 14);
+            if (shortRest || longRest) {
-        const actor = await foundry.utils.fromUuid(actorUuid);
+                const key = element.dataset.tooltip.slice(shortRest ? 11 : 10);
 
-        if (actor) {
-            const html = await foundry.applications.handlebars.renderTemplate(
-                `systems/daggerheart/templates/ui/tooltip/advantage.hbs`,
-                {
-                    sources: isAdvantage ? actor.system.advantageSources : actor.system.disadvantageSources
-                }
-            );
-
-            this.tooltip.innerHTML = html;
-            return html;
-        }
-        return null;
-    }
-
-    async #activateDeathMove(element, options) {
-        const name = element.dataset.deathName;
-        const img = element.dataset.deathImg;
-        const description = element.dataset.deathDescription;
-
-        const html = await foundry.applications.handlebars.renderTemplate(
-            `systems/daggerheart/templates/ui/tooltip/death-move.hbs`,
-            {
-                move: { name: name, img: img, description: description }
-            }
-        );
-
-        this.tooltip.innerHTML = html;
-        options.direction = this._determineItemTooltipDirection(
-            element,
-            this.constructor.TOOLTIP_DIRECTIONS.RIGHT
-        );
-        return html;
-    }
-
-    async #activateRest(element, options) {
-        const isShortRest = element.dataset.tooltip?.startsWith('#shortRest#');
-        const key = element.dataset.tooltip.slice(isShortRest ? 11 : 10);
                 const moves = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Homebrew).restMoves[
                     element.dataset.restType
                 ].moves;
                 const move = moves[key];
-        const description = await foundry.applications.ux.TextEditor.enrichHTML(move.description);
+                const description = await TextEditor.enrichHTML(move.description);
-        const html = await foundry.applications.handlebars.renderTemplate(
+                html = await foundry.applications.handlebars.renderTemplate(
                     `systems/daggerheart/templates/ui/tooltip/downtime.hbs`,
                     {
                         move: move,
@@ -231,7 +154,49 @@ export default class DhTooltipManager extends foundry.helpers.interaction.Toolti
                     element,
                     this.constructor.TOOLTIP_DIRECTIONS.RIGHT
                 );
-        return html;
+            }
+
+            const isAdvantage = element.dataset.tooltip?.startsWith('#advantage#');
+            const isDisadvantage = element.dataset.tooltip?.startsWith('#disadvantage#');
+            if (isAdvantage || isDisadvantage) {
+                const actorUuid = element.dataset.tooltip.slice(isAdvantage ? 11 : 14);
+                const actor = await foundry.utils.fromUuid(actorUuid);
+
+                if (actor) {
+                    html = await foundry.applications.handlebars.renderTemplate(
+                        `systems/daggerheart/templates/ui/tooltip/advantage.hbs`,
+                        {
+                            sources: isAdvantage ? actor.system.advantageSources : actor.system.disadvantageSources
+                        }
+                    );
+
+                    this.tooltip.innerHTML = html;
+                }
+            }
+
+            const deathMove = element.dataset.tooltip?.startsWith('#deathMove#');
+            if (deathMove) {
+                const name = element.dataset.deathName;
+                const img = element.dataset.deathImg;
+                const description = element.dataset.deathDescription;
+
+                html = await foundry.applications.handlebars.renderTemplate(
+                    `systems/daggerheart/templates/ui/tooltip/death-move.hbs`,
+                    {
+                        move: { name: name, img: img, description: description }
+                    }
+                );
+
+                this.tooltip.innerHTML = html;
+                options.direction = this._determineItemTooltipDirection(
+                    element,
+                    this.constructor.TOOLTIP_DIRECTIONS.RIGHT
+                );
+            }
+        }
+
+        this.noOffset = options.noOffset;
+        super.activate(element, { ...options, html: html });
     }
 
     _setAnchor(direction) {

module/enrichers/TemplateEnricher.mjs

@@ -58,11 +58,10 @@ export const renderMeasuredTemplate = async event => {
     if (!type || !range || !game.canvas.scene) return;
 
     const shapeData = CONFIG.Canvas.layers.regions.layerClass.getTemplateShape({
-        shapeType: type,
+        type,
         angle,
         range,
-        direction,
+        direction
-        hasHole: false
     });
 
     await canvas.regions.placeRegion(

module/enrichers/_module.mjs

@@ -35,24 +35,23 @@ export const enricherConfig = [
 ];
 
 export const enricherRenderSetup = element => {
-    const clickWrapper = func => event => {
-        event.stopPropagation();
-        func(event);
-    };   
-
     element
         .querySelectorAll('.enriched-damage-button')
-        .forEach(element => element.addEventListener('click', clickWrapper(renderDamageButton)));
+        .forEach(element => element.addEventListener('click', renderDamageButton));
 
     element
         .querySelectorAll('.duality-roll-button')
-        .forEach(element => element.addEventListener('click', clickWrapper(renderDualityButton)));
+        .forEach(element => element.addEventListener('click', renderDualityButton));
 
     element
         .querySelectorAll('.fate-roll-button')
-        .forEach(element => element.addEventListener('click', clickWrapper(renderFateButton)));
+        .forEach(element => element.addEventListener('click', renderFateButton));
 
     element
         .querySelectorAll('.measured-template-button')
-        .forEach(element => element.addEventListener('click', clickWrapper(renderMeasuredTemplate)));
+        .forEach(element => element.addEventListener('click', renderMeasuredTemplate));
+
+    // element
+    //     .querySelectorAll('.enriched-effect')
+    //     .forEach(element => element.addEventListener('dragstart', dragEnrichedEffect));
 };

module/helpers/handlebarsHelper.mjs

@@ -48,8 +48,9 @@ export default class RegisterHandlebarsHelpers {
         return formula;
     }
 
-    static damageSymbols(damageData) {
+    static damageSymbols(damageParts) {
-        const symbols = damageData.type.map(p => CONFIG.DH.GENERAL.damageTypes[p].icon);
+        const allTypes = [...new Set([...damageParts].flatMap(x => Array.from(x.type)))];
+        const symbols = allTypes.map(p => CONFIG.DH.GENERAL.damageTypes[p].icon);
         return new Handlebars.SafeString(Array.from(symbols).map(symbol => `<i class="fa-solid ${symbol}"></i>`));
     }
 

module/helpers/utils.mjs

@@ -1,10 +1,6 @@
 import { diceTypes, getDiceSoNicePresets, getDiceSoNicePreset, range } from '../config/generalConfig.mjs';
 import Tagify from '@yaireo/tagify';
 
-/**
- * @import DhpActor from '../documents/actor.mjs';
- */
-
 export const capitalize = string => {
     return string.charAt(0).toUpperCase() + string.slice(1);
 };
@@ -700,6 +696,19 @@ export async function RefreshFeatures(
     return refreshedActors;
 }
 
+export function getUnusedDamageTypes(parts) {
+    const usedKeys = Object.keys(parts);
+    return Object.keys(CONFIG.DH.GENERAL.healingTypes).reduce((acc, key) => {
+        if (!usedKeys.includes(key))
+            acc.push({
+                value: key,
+                label: game.i18n.localize(CONFIG.DH.GENERAL.healingTypes[key].label)
+            });
+
+        return acc;
+    }, []);
+}
+
 /** Returns resolved armor sources ordered by application order */
 export function getArmorSources(actor) {
     const rawArmorSources = Array.from(actor.allApplicableEffects()).filter(x => x.system.armorData);
@@ -881,32 +890,3 @@ export function shouldUseHopeFearAutomation(options = { gmAsPlayer: true }) {
     const { hopeFear } = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Automation);
     return (!game.user.isGM || options.gmAsPlayer) ? hopeFear.players : hopeFear.gm; 
 }
-
-/**
- * Returns the given actor if its a world actor, 
- * finds a world actor equivalent, 
- * or imports the actor and returns the imported actor.
- * @param {DhpActor} baseActor 
- * @returns {Promise<DhpActor>} a world actor
- */
-export async function getWorldActor(baseActor) {
-    if (baseActor.inCompendium) {
-        const worldActorCandidates = game.actors.filter(x =>
-            x._stats.compendiumSource === baseActor.uuid &&
-            x.prototypeToken.actorLink === baseActor.prototypeToken.actorLink
-        );
-        const worldActorCopy = worldActorCandidates.find(a => a.name === baseActor.name) ?? worldActorCandidates[0];
-        if (worldActorCopy) return worldActorCopy;
-
-        const baseActorData = baseActor;
-        return await game.system.api.documents.DhpActor.create({ 
-            ...baseActorData, 
-            _stats: { 
-                ...baseActorData._stats, 
-                compendiumSource: baseActor.uuid 
-            } 
-        });
-    }
-
-    return baseActor;
-}

module/systemRegistration/migration-handlers/2_5_2.mjs

@@ -1,39 +0,0 @@
-import { MigrationHandlerBase } from './base.mjs';
-
-export class Migration_2_5_2 extends MigrationHandlerBase {
-    version = '2.5.2';
-
-    /** @inheritdoc */
-    async updateActiveEffectSource(effectSource, item) {
-        let shouldUpdate = false;
-        const newChanges = [];
-
-        const srdItem = item?._stats?.compendiumSource ? 
-            await foundry.utils.fromUuid(item?._stats.compendiumSource) : 
-            null;
-        for (let i = 0; i < effectSource.system.changes.length; i++) {
-            const change = effectSource.system.changes[i];
-            const srdEffect = srdItem?.effects.find(x => x.name === effectSource.name);
-            if (change.type === 'custom') {
-                const srdChange = srdEffect ? srdEffect.system.changes[i] : null;
-                if (srdChange &&
-                    change.key === srdChange.key && 
-                    change.value === srdChange.value && 
-                    change.type !== srdChange.type
-                ) {
-                    shouldUpdate = true;
-                    newChanges.push(srdChange);
-                }
-            } else {
-                newChanges.push(change);
-            }
-        }
-
-        if (shouldUpdate) {
-            return {
-                _id: effectSource._id,
-                system: { changes: newChanges }
-            }
-        }
-    }
-}

module/systemRegistration/migration-handlers/2_6_0.mjs

@@ -1,20 +0,0 @@
-import { MigrationHandlerBase } from './base.mjs';
-
-export class Migration_2_6_0 extends MigrationHandlerBase {
-    version = '2.6.0';
-
-    /** @inheritdoc */
-    async updateActorSource(actor) {
-        if (actor.type === 'party' && Object.keys(actor.system.tagTeam.members).length) {
-            return {
-                _id: actor._id,
-                system: {
-                    tagTeam: {
-                        initiator: null,
-                        members: _replace({})
-                    }
-                }
-            };
-        }
-    }
-}

module/systemRegistration/migration-handlers/base.mjs

@@ -1,117 +0,0 @@
-/** 
- * @import DHItem from "../../documents/item.mjs";
-* @import DhActor from "../../documents/actor.mjs";
- */
-/** 
-
- * The base class of an async migration. 
- * These are generally run between versions for things that require compendiums or must be done in post.
- * The migrate() functions calls the various updateXSource() functions.
- * Generally a subclass will override the version and the updateXSource() functions.
- */
-export class MigrationHandlerBase {
-    version = null;
-
-    /**
-     * Gets change data for an active effect's source, or null if no changes
-     * @param {object} effectSource 
-     * @param {DHItem} item 
-     * @returns {Promise<object>}
-     * @protected
-     */
-    async updateActiveEffectSource(effectSource, item) {
-        return null;
-    }
-
-    /**
-     * Update a world actor
-     * @param {DhActor} actor 
-     * @returns {Promise<object>}
-     * @protected
-     */
-    async updateActorSource(actor) {
-        return null;
-    }
-
-    async migrate() {
-        // todo: handle more than just migrating effects. Right now this can only migrate effects
-        // NOTE: the preload is hardcoded, we should not hardcode it
-
-        const numActors = game.actors.size;
-        const numItems = game.items.size;
-        const finalUpdateProgress = 5;
-        const DhProgress = game.system.api.applications.ui.DhProgress;
-        const preRunProgress = game.packs.size;
-        
-        const progress = DhProgress.createMigrationProgress(
-            preRunProgress + numActors + numItems + finalUpdateProgress
-        );
-
-        // Preload. Avoid hardcoding in the future
-        for (const pack of game.packs) {
-            await pack.getDocuments();
-            progress.advance();
-        }
-
-        const batch = [];
-
-        const updateItem = async item => {
-            const itemUpdates = [];
-            for (const effect of item.effects) {
-                const changes = await this.updateActiveEffectSource(effect.toObject(), item);
-                if (changes) itemUpdates.push(changes);
-            }
-            if (itemUpdates.length) {
-                batch.push({
-                    action: 'update',
-                    documentName: 'ActiveEffect',
-                    updates: itemUpdates,
-                    parent: item
-                });
-            }
-        };
-
-        const updateActor = async actor => {
-            const actorUpdate = await this.updateActorSource(actor);
-            if (actorUpdate) {
-                batch.push({
-                    action: 'update',
-                    documentName: 'Actor',
-                    updates: [actorUpdate]
-                });
-            }
-
-            const aeUpdates = [];
-            for (const item of actor.items) {
-                await updateItem(item);
-            }
-            
-            for (const effect of actor.effects) {
-                const changes = await this.updateActiveEffectSource(effect.toObject(), { parent: actor });
-                if (changes) aeUpdates.push(changes);
-            }
-            if (aeUpdates.length) {
-                batch.push({
-                    action: 'update',
-                    documentName: 'ActiveEffect',
-                    updates: aeUpdates,
-                    parent: actor
-                });
-            }
-        }
-
-
-        for (const actor of game.actors) {
-            await updateActor(actor);
-  
-            progress.advance();
-        }
-        for (const item of game.items) {
-            await updateItem(item);
-            progress.advance();
-        }
-
-        await foundry.documents.modifyBatch(batch);
-        progress.advance({ by: finalUpdateProgress });
-    }
-}

module/systemRegistration/migrations.mjs

@@ -1,6 +1,4 @@
 import { defaultRestOptions } from '../config/generalConfig.mjs';
-import { Migration_2_5_2 } from './migration-handlers/2_5_2.mjs';
-import { Migration_2_6_0 } from './migration-handlers/2_6_0.mjs';
 
 export async function runMigrations() {
     let lastMigrationVersion = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.LastMigrationVersion);
@@ -322,20 +320,7 @@ export async function runMigrations() {
 
         lastMigrationVersion = '2.1.0';
     }
+    //#endregion
 
     await game.settings.set(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.LastMigrationVersion, lastMigrationVersion);
-
-    /* -------------------------------------------- */
-    /*  New Style migrations below this point       */
-    /* -------------------------------------------- */
-
-    const migrations = [
-        new Migration_2_5_2(),
-        new Migration_2_6_0()
-    ].filter(m => m.version && foundry.utils.isNewerVersion(m.version, lastMigrationVersion));
-
-    for (const handler of migrations) {
-        await handler.migrate();
-        await game.settings.set(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.LastMigrationVersion, handler.version);
-    }
 }

module/systemRegistration/socket.mjs

@@ -1,5 +1,5 @@
 import DamageReductionDialog from '../applications/dialogs/damageReductionDialog.mjs';
-import PartySheet from '../applications/sheets/actors/party.mjs';
+import Party from '../applications/sheets/actors/party.mjs';
 
 export function handleSocketEvent({ action = null, data = {} } = {}) {
     switch (action) {
@@ -16,7 +16,7 @@ export function handleSocketEvent({ action = null, data = {} } = {}) {
             Hooks.call(socketEvent.Refresh, data);
             break;
         case socketEvent.DowntimeTrigger:
-            PartySheet.downtimeMoveQuery(data);
+            Party.downtimeMoveQuery(data);
             break;
         case socketEvent.TagTeamStart:
             Hooks.callAll(CONFIG.DH.HOOKS.hooksConfig.tagTeamStart, data);

src/packs/adversaries/adversary_Acid_Burrower_89yAh30vaNQOALlz.json

@@ -403,6 +403,12 @@
           "_id": "9PsnogEPsp1OOK64",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "act"

src/packs/adversaries/adversary_Adult_Flickerfly_G7jiltRjgvVhZewm.json

@@ -322,22 +322,31 @@
           "_id": "ia0NUIOxE3RHb45P",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            }
+          },
           "changes": [
             {
               "key": "system.evasion",
-                "type": "override",
+              "mode": 5,
               "value": "@system.evasion / 2",
               "priority": 10
             }
-            ]
+          ],
-          },
-          "duration": {
-            "value": null,
-            "units": "seconds",
-            "expiry": null,
-            "expired": false
-          },
           "disabled": false,
+          "duration": {
+            "startTime": null,
+            "combat": null,
+            "seconds": null,
+            "rounds": null,
+            "turns": null,
+            "startRound": null,
+            "startTurn": null
+          },
           "description": "<p>When the Flickerfl y makes an attack, the target’s Evasion is halved against the attack.</p>",
           "tint": "#ffffff",
           "statuses": [],

src/packs/adversaries/adversary_Arch_Necromancer_WPEOIGfclNJxWb87.json

@@ -491,6 +491,12 @@
           "_id": "KGdf2eqcXkdigg0u",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "temporary",

src/packs/adversaries/adversary_Archer_Guard_JRhrrEg5UroURiAD.json

@@ -325,15 +325,21 @@
           "_id": "wGuxOLokMqdxVSOo",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            }
+          },
           "changes": [
             {
               "key": "system.disadvantageSources",
-                "type": "add",
+              "mode": 2,
               "value": "Agility Rolls",
               "priority": null
             }
-            ]
+          ],
-          },
           "disabled": false,
           "duration": {
             "startTime": null,

src/packs/adversaries/adversary_Archer_Squadron_0ts6CGd93lLqGZI5.json

@@ -472,8 +472,14 @@
       "disabled": true,
       "_id": "bq8hTzQoCXmA3xad",
       "system": {
-        "changes": []
+        "rangeDependence": {
+          "enabled": false,
+          "type": "withinRange",
+          "target": "hostile",
+          "range": "melee"
+        }
       },
+      "changes": [],
       "duration": {
         "startTime": 0,
         "combat": null

src/packs/adversaries/adversary_Assassin_Poisoner_h5RuhzGL17dW5FBT.json

@@ -270,6 +270,12 @@
           "_id": "2iBVUGHtGW3I9VIj",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "temporary",
@@ -335,17 +341,23 @@
           "name": "Out of Nowhere",
           "type": "base",
           "system": {
-            "changes": [
+            "rangeDependence": {
-              {
+              "enabled": false,
-                "key": "system.advantageSources",
+              "type": "withinRange",
-                "type": "add",
+              "target": "hostile",
-                "value": "On attacks if they are Hidden.",
+              "range": "melee"
-                "priority": null
             }
-            ]
           },
           "_id": "qT5nh7OcQ6aGdiWS",
           "img": "icons/magic/perception/shadow-stealth-eyes-purple.webp",
+          "changes": [
+            {
+              "key": "system.advantageSources",
+              "mode": 2,
+              "value": "On attacks if they are Hidden.",
+              "priority": null
+            }
+          ],
           "disabled": false,
           "duration": {
             "startTime": null,
@@ -440,6 +452,12 @@
           "_id": "yP4ot8VqS56RnxnE",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": []
           },
           "disabled": false,

src/packs/adversaries/adversary_Battle_Box_dgH3fW9FTYLaIDvS.json

@@ -680,6 +680,12 @@
           "_id": "9NQcCXMhjyBReJRd",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "temporary",
@@ -921,6 +927,12 @@
           "_id": "S7kJlhnV8Nexzi8l",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "temporary",

src/packs/adversaries/adversary_Bear_71qKDLKO3CsrNkdy.json

@@ -363,6 +363,12 @@
           "_id": "U50Ccm9emMqAxma6",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "temporary",

src/packs/adversaries/adversary_Bladed_Guard_B4LZcGuBAHzyVdzy.json

@@ -371,6 +371,12 @@
           "_id": "LmzztuktRkwOCy1a",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "temporary",

src/packs/adversaries/adversary_Brawny_Zombie_2UeZ0tEe7AzgSJNd.json

@@ -447,6 +447,12 @@
           "_id": "CjMrSdL6kgD8mKRQ",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "temporary"

src/packs/adversaries/adversary_Chaos_Skull_jDmHqGvzg5wjgmxE.json

@@ -232,17 +232,23 @@
           "name": "Levitation",
           "type": "base",
           "system": {
-            "changes": [
+            "rangeDependence": {
-              {
+              "enabled": false,
-                "key": "system.rules.conditionImmunities.restrained",
+              "type": "withinRange",
-                "type": "override",
+              "target": "hostile",
-                "value": "1",
+              "range": "melee"
-                "priority": null
             }
-            ]
           },
           "_id": "k6iaQVfMZhrpwYQj",
           "img": "icons/magic/air/fog-gas-smoke-blue-gray.webp",
+          "changes": [
+            {
+              "key": "system.rules.conditionImmunities.restrained",
+              "mode": 5,
+              "value": "1",
+              "priority": null
+            }
+          ],
           "disabled": false,
           "duration": {
             "startTime": null,
@@ -295,17 +301,23 @@
           "name": "Wards",
           "type": "base",
           "system": {
-            "changes": [
+            "rangeDependence": {
-              {
+              "enabled": false,
-                "key": "system.resistance.magical.resistance",
+              "type": "withinRange",
-                "type": "override",
+              "target": "hostile",
-                "value": "1",
+              "range": "melee"
-                "priority": null
             }
-            ]
           },
           "_id": "4rj6PM0AGrrlbxSc",
           "img": "icons/magic/defensive/barrier-shield-dome-blue-purple.webp",
+          "changes": [
+            {
+              "key": "system.resistance.magical.resistance",
+              "mode": 5,
+              "value": "1",
+              "priority": null
+            }
+          ],
           "disabled": false,
           "duration": {
             "startTime": null,

src/packs/adversaries/adversary_Construct_uOP5oT9QzXPlnf3p.json

@@ -459,15 +459,21 @@
           "_id": "xkDIZk9u2ipDHvOL",
           "img": "icons/creatures/magical/construct-golem-stone-blue.webp",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            }
+          },
           "changes": [
             {
               "key": "system.bonuses.damage.physical.bonus",
-                "type": "add",
+              "mode": 2,
               "value": "10",
               "priority": null
             }
-            ]
+          ],
-          },
           "disabled": true,
           "duration": {
             "startTime": null,

src/packs/adversaries/adversary_Courtesan_ZxWaWPdzFIUPNC62.json

@@ -315,6 +315,12 @@
           "_id": "blcRqns0PHqiuPac",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "temporary",

src/packs/adversaries/adversary_Courtier_CBBuEXAlLKFMJdjg.json

@@ -331,6 +331,12 @@
           "_id": "YNMhgBZW8ndrCjIp",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "scene"

src/packs/adversaries/adversary_Cult_Adept_0NxCSugvKQ4W8OYZ.json

@@ -415,6 +415,12 @@
           "_id": "U9lWz1LgeAiK5L85",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [
               {
                 "key": "system.resistance.magical.resistance",
@@ -540,6 +546,12 @@
           "_id": "lNH6srSPyEprXZ4o",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "temporary",

src/packs/adversaries/adversary_Cult_Fang_tyBOpLfigAhI9bU3.json

@@ -379,6 +379,12 @@
           "_id": "LwWxRz7FTMA80VdA",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "temporary"

src/packs/adversaries/adversary_Deeproot_Defender_9x2xY9zwc3xzbXo5.json

@@ -410,6 +410,12 @@
           "_id": "F3E7fiz01AbF2kr5",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [],
             "duration": {
               "type": "temporary",

src/packs/adversaries/adversary_Demon_of_Avarice_pnyjIGxxvurcWmTv.json

@@ -270,17 +270,23 @@
           "name": "Numbers Must Go Up",
           "type": "base",
           "system": {
-            "changes": [
+            "rangeDependence": {
-              {
+              "enabled": false,
-                "key": "system.bonuses.roll.attack.bonus",
+              "type": "withinRange",
-                "type": "add",
+              "target": "hostile",
-                "value": "ITEM.@system.resource.value",
+              "range": "melee"
-                "priority": 21
             }
-            ]
           },
           "_id": "iLeiOlUtWvYsgbDQ",
           "img": "icons/magic/control/fear-fright-monster-grin-red-orange.webp",
+          "changes": [
+            {
+              "key": "system.bonuses.roll.attack.bonus",
+              "mode": 2,
+              "value": "ITEM.@system.resource.value",
+              "priority": 21
+            }
+          ],
           "disabled": false,
           "duration": {
             "startTime": null,

src/packs/adversaries/adversary_Demon_of_Despair_kE4dfhqmIQpNd44e.json

@@ -238,17 +238,23 @@
           "name": "Depths Of Despair",
           "type": "base",
           "system": {
-            "changes": [
+            "rangeDependence": {
-              {
+              "enabled": false,
-                "key": "system.rules.attack.damage.hpDamageMultiplier",
+              "type": "withinRange",
-                "type": "override",
+              "target": "hostile",
-                "value": "2",
+              "range": "melee"
-                "priority": null
             }
-            ]
           },
           "_id": "nofxm1vGZ2TmceA2",
           "img": "icons/magic/death/skull-horned-worn-fire-blue.webp",
+          "changes": [
+            {
+              "key": "system.rules.attack.damage.hpDamageMultiplier",
+              "mode": 5,
+              "value": "2",
+              "priority": null
+            }
+          ],
           "disabled": true,
           "duration": {
             "startTime": null,
@@ -350,6 +356,12 @@
           "_id": "6WSx03mFbpbPWnOI",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [
               {
                 "key": "system.rules.dualityRoll.defaultHopeDice",

src/packs/adversaries/adversary_Demon_of_Wrath_5lphJAgzoqZI3VoG.json

@@ -319,6 +319,12 @@
           "_id": "gFeHLGgeRoDdd3VG",
           "type": "base",
           "system": {
+            "rangeDependence": {
+              "enabled": false,
+              "type": "withinRange",
+              "target": "hostile",
+              "range": "melee"
+            },
             "changes": [
               {
                 "key": "system.rules.dualityRoll.defaultFearDice",