@lwc/engine-core 8.1.0-alpha.0 → 8.1.0-alpha.2

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}`;
@@ -373,8 +377,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 +6952,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 +8390,5 @@ exports.swapTemplate = swapTemplate;
 exports.track = track;
 exports.unwrap = unwrap;
 exports.wire = wire;
-/** version: 8.0.0 */
+/** version: 8.1.0-alpha.2 */
 //# sourceMappingURL=index.cjs.js.map