Replace implementation with shallow proxy

module/data/actor/base.mjs

@@ -1,6 +1,6 @@
 import DHBaseActorSettings from '../../applications/sheets/api/actor-setting.mjs';
 import DHItem from '../../documents/item.mjs';
-import { getScrollTextData } from '../../helpers/utils.mjs';
+import { createShallowProxy, getScrollTextData } from '../../helpers/utils.mjs';
 
 const fields = foundry.data.fields;
 
@@ -180,8 +180,7 @@ export default class BaseDataActor extends foundry.abstract.TypeDataModel {
      * @returns {object}
      */
     getRollData() {
-        const data = { ...this };
+        return createShallowProxy(this);
-        return data;
     }
 
     /**

module/data/item/base.mjs

@@ -10,7 +10,7 @@
 import {
     addLinkedItemsDiff,
     getScrollTextData,
-    shallowCopyWithGetters,
+    createShallowProxy,
     updateLinkedItemApps
 } from '../../helpers/utils.mjs';
 import { ActionsField } from '../fields/actionField.mjs';
@@ -165,7 +165,7 @@ export default class BaseDataItem extends foundry.abstract.TypeDataModel {
      */
     getRollData(options = {}) {
         const data = this.actor?.getRollData() ?? {};
-        data.item = shallowCopyWithGetters(this);
+        data.item = createShallowProxy(this);
         return data;
     }
 

module/documents/actor.mjs

@@ -1,7 +1,7 @@
 import { emitAsGM, GMUpdateEvent } from '../systemRegistration/socket.mjs';
 import { LevelOptionType } from '../data/levelTier.mjs';
 import DHFeature from '../data/item/feature.mjs';
-import { createScrollText, damageKeyToNumber, getDamageKey, shallowCopyWithGetters } from '../helpers/utils.mjs';
+import { createScrollText, damageKeyToNumber, getDamageKey, createShallowProxy } from '../helpers/utils.mjs';
 import DhCompanionLevelUp from '../applications/levelup/companionLevelup.mjs';
 import { ResourceUpdateMap } from '../data/action/baseAction.mjs';
 import { abilities } from '../config/actorConfig.mjs';
@@ -595,7 +595,7 @@ export default class DhpActor extends Actor {
 
     /**@inheritdoc */
     getRollData() {
-        const rollData = shallowCopyWithGetters(super.getRollData());
+        const rollData = createShallowProxy(super.getRollData());
         rollData.id = this.id;
         rollData.name = this.name;
         rollData.system = this.system.getRollData();

module/helpers/utils.mjs

@@ -810,28 +810,23 @@ export function sortBy(arr, fn) {
     return arr.sort(cmp);
 }
 
-function getAllPropertyNames(obj, names = new Set()) {
-    if (!obj) return names;
-
-    for (const name of Object.getOwnPropertyNames(obj)) {
-        names.add(name);
-    }
-
-    return getAllPropertyNames(Object.getPrototypeOf(obj), names);
-}
-
 /**
- * Returns a shallow copy including getters of the given object.
+ * Creates a proxy that allows retrieval of top data but diverts updates to a different object.
- * Generally used for expanding roll data without side effects
+ * Generally used for roll data
  */
-export function shallowCopyWithGetters(obj) {
+export function createShallowProxy(obj) {
-    const props = getAllPropertyNames(obj);
+    if (obj._isShallowProxy) return obj;
-    const result = {};
+
-    for (const key of props) {
+    const overrides = {};
-        const value = obj[key];
+    return new Proxy(obj, {
-        if (key !== '__proto__' && typeof value !== 'function') {
+        get(target, prop, receiver) {
-            result[key] = obj[key];
+            if (prop === '_isShallowProxy') return true;
+            if (prop in overrides) return overrides[prop];
+            return Reflect.get(target, prop, receiver);
+        },
+        set(_target, prop, newValue) {
+            overrides[prop] = newValue;
+            return true;
         }
-    }
+    });
-    return result;
 }