@lwc/engine-core 8.1.0-alpha.1 → 8.1.0-alpha.3

package/dist/index.cjs.js

@@ -328,6 +328,10 @@ function logMutation(reactiveObserver, target, key) {
         if (shared.isUndefined(parentKey)) {
             prop = stringKey;
         }
+        else if (!shared.isString(key)) {
+            // symbol/number, e.g. `obj[Symbol("foo")]` or `obj[1234]`
+            prop = `${shared.toString(parentKey)}[${stringKey}]`;
+        }
         else if (/^\w+$/.test(stringKey)) {
             // Human-readable prop like `items[0].name` on a deep object/array
             prop = `${shared.toString(parentKey)}.${stringKey}`;
@@ -363,6 +367,10 @@ function associateReactiveObserverWithVM(reactiveObserver, vm) {
  */
 function trackTargetForMutationLogging(key, target) {
     assertNotProd();
+    if (targetsToPropertyKeys.has(target)) {
+        // Guard against recursive objects - don't traverse forever
+        return;
+    }
     if (shared.isObject(target) && !shared.isNull(target)) {
         // only track non-primitives; others are invalid as WeakMap keys
         targetsToPropertyKeys.set(target, key);
@@ -373,8 +381,12 @@ function trackTargetForMutationLogging(key, target) {
             }
         }
         else {
-            for (const prop of shared.keys(target)) {
-                trackTargetForMutationLogging(`${shared.toString(key)}.${prop}`, target[prop]);
+            // Track only own property names and symbols (including non-enumerated)
+            // This is consistent with what observable-membrane does:
+            // https://github.com/salesforce/observable-membrane/blob/b85417f/src/base-handler.ts#L142-L143
+            const props = [...shared.getOwnPropertyNames(target), ...shared.getOwnPropertySymbols(target)];
+            for (const prop of props) {
+                trackTargetForMutationLogging(`${shared.toString(key)}.${shared.toString(prop)}`, target[prop]);
             }
         }
     }
@@ -6944,7 +6956,7 @@ let rehydrateQueue = [];
 function flushRehydrationQueue() {
     // Gather the logs before rehydration starts so they can be reported at the end of rehydration.
     // Note that we also clear all existing logs at this point so that subsequent re-renders start from a clean slate.
-    const mutationLogs = process.env.NODE_ENV !== 'production' ? getAndFlushMutationLogs() : undefined;
+    const mutationLogs = process.env.NODE_ENV === 'production' ? undefined : getAndFlushMutationLogs();
     logGlobalOperationStart(8 /* OperationId.GlobalRehydrate */);
     if (process.env.NODE_ENV !== 'production') {
         shared.assert.invariant(rehydrateQueue.length, `If rehydrateQueue was scheduled, it is because there must be at least one VM on this pending queue instead of ${rehydrateQueue}.`);
@@ -8382,5 +8394,5 @@ exports.swapTemplate = swapTemplate;
 exports.track = track;
 exports.unwrap = unwrap;
 exports.wire = wire;
-/** version: 8.1.0-alpha.1 */
+/** version: 8.1.0-alpha.3 */
 //# sourceMappingURL=index.cjs.js.map