@lwc/engine-core 8.20.4 → 8.20.6

package/dist/index.cjs.js

@@ -611,12 +611,23 @@ function componentValueObserved(vm, key, target = {}) {
     if (lwcRuntimeFlags.ENABLE_EXPERIMENTAL_SIGNALS &&
         shared.isObject(target) &&
         !shared.isNull(target) &&
-        shared.isTrustedSignal(target) &&
         process.env.IS_BROWSER &&
         // Only subscribe if a template is being rendered by the engine
         tro.isObserving()) {
-        // Subscribe the template reactive observer's notify method, which will mark the vm as dirty and schedule hydration.
-        subscribeToSignal(component, target, tro.notify.bind(tro));
+        /**
+         * The legacy validation behavior was that this check should only
+         * be performed for runtimes that have provided a trustedSignals set.
+         * However, this resulted in a bug as all object values were
+         * being considered signals in environments where the trustedSignals
+         * set had not been defined. The runtime flag has been added as a killswitch
+         * in case the fix needs to be reverted.
+         */
+        if (lwcRuntimeFlags.ENABLE_LEGACY_SIGNAL_CONTEXT_VALIDATION
+            ? shared.legacyIsTrustedSignal(target)
+            : shared.isTrustedSignal(target)) {
+            // Subscribe the template reactive observer's notify method, which will mark the vm as dirty and schedule hydration.
+            subscribeToSignal(component, target, tro.notify.bind(tro));
+        }
     }
 }
 function createReactiveObserver(callback) {
@@ -6822,42 +6833,72 @@ class ContextBinding {
 }
 _ContextBinding_renderer = new WeakMap(), _ContextBinding_providedContextVarieties = new WeakMap(), _ContextBinding_elm = new WeakMap();
 function connectContext(vm) {
+    /**
+     * If ENABLE_LEGACY_CONTEXT_CONNECTION is true, enumerates directly on the component
+     * which can result in the component lifecycle observing properties that are not typically observed.
+     * See PR #5536 for more information.
+     */
+    if (lwcRuntimeFlags.ENABLE_LEGACY_CONTEXT_CONNECTION) {
+        connect(vm, shared.keys(shared.getPrototypeOf(vm.component)), vm.component);
+    }
+    else {
+        // Non-decorated objects
+        connect(vm, shared.keys(vm.cmpFields), vm.cmpFields);
+        // Decorated objects like @api context
+        connect(vm, shared.keys(vm.cmpProps), vm.cmpProps);
+    }
+}
+function disconnectContext(vm) {
+    /**
+     * If ENABLE_LEGACY_CONTEXT_CONNECTION is true, enumerates directly on the component
+     * which can result in the component lifecycle observing properties that are not typically observed.
+     * See PR #5536 for more information.
+     */
+    if (lwcRuntimeFlags.ENABLE_LEGACY_CONTEXT_CONNECTION) {
+        connect(vm, shared.keys(shared.getPrototypeOf(vm.component)), vm.component);
+    }
+    else {
+        // Non-decorated objects
+        disconnect(vm, shared.keys(vm.cmpFields), vm.cmpFields);
+        // Decorated objects like @api context
+        disconnect(vm, shared.keys(vm.cmpProps), vm.cmpProps);
+    }
+}
+function connect(vm, enumerableKeys, contextContainer) {
     const contextKeys = shared.getContextKeys();
     if (shared.isUndefined(contextKeys)) {
         return;
     }
     const { connectContext } = contextKeys;
     const { component } = vm;
-    const enumerableKeys = shared.keys(shared.getPrototypeOf(component));
-    const contextfulKeys = shared.ArrayFilter.call(enumerableKeys, (enumerableKey) => shared.isTrustedContext(component[enumerableKey]));
+    const contextfulKeys = shared.ArrayFilter.call(enumerableKeys, (enumerableKey) => shared.isTrustedContext(contextContainer[enumerableKey]));
     if (contextfulKeys.length === 0) {
         return;
     }
     const providedContextVarieties = new Map();
     try {
         for (let i = 0; i < contextfulKeys.length; i++) {
-            component[contextfulKeys[i]][connectContext](new ContextBinding(vm, component, providedContextVarieties));
+            contextContainer[contextfulKeys[i]][connectContext](new ContextBinding(vm, component, providedContextVarieties));
         }
     }
     catch (err) {
         logWarnOnce(`Attempted to connect to trusted context but received the following error: ${err.message}`);
     }
 }
-function disconnectContext(vm) {
+function disconnect(vm, enumerableKeys, contextContainer) {
     const contextKeys = shared.getContextKeys();
     if (!contextKeys) {
         return;
     }
     const { disconnectContext } = contextKeys;
     const { component } = vm;
-    const enumerableKeys = shared.keys(shared.getPrototypeOf(component));
-    const contextfulKeys = shared.ArrayFilter.call(enumerableKeys, (enumerableKey) => shared.isTrustedContext(component[enumerableKey]));
+    const contextfulKeys = shared.ArrayFilter.call(enumerableKeys, (enumerableKey) => shared.isTrustedContext(contextContainer[enumerableKey]));
     if (contextfulKeys.length === 0) {
         return;
     }
     try {
         for (let i = 0; i < contextfulKeys.length; i++) {
-            component[contextfulKeys[i]][disconnectContext](component);
+            contextContainer[contextfulKeys[i]][disconnectContext](component);
         }
     }
     catch (err) {
@@ -8741,5 +8782,5 @@ exports.swapTemplate = swapTemplate;
 exports.track = track;
 exports.unwrap = unwrap;
 exports.wire = wire;
-/** version: 8.20.4 */
+/** version: 8.20.6 */
 //# sourceMappingURL=index.cjs.js.map

package/dist/index.js

@@ -1,7 +1,7 @@
 /**
  * Copyright (c) 2025 Salesforce, Inc.
  */
-import { noop, StringToLowerCase, isNull, ArrayPush as ArrayPush$1, ArrayJoin, isFrozen, isUndefined as isUndefined$1, defineProperty, seal, create, isAPIFeatureEnabled, isFunction as isFunction$1, keys, toString, isString, ArrayFilter, isObject, isArray as isArray$1, getOwnPropertyNames as getOwnPropertyNames$1, getOwnPropertySymbols as getOwnPropertySymbols$1, ArrayIndexOf, ArrayPop, isFalse, isTrustedSignal, hasOwnProperty as hasOwnProperty$1, entries, AriaPropNameToAttrNameMap, getPropertyDescriptor, AriaAttrNameToPropNameMap, forEach, REFLECTIVE_GLOBAL_PROPERTY_SET, getPrototypeOf as getPrototypeOf$1, setPrototypeOf, defineProperties, assign, freeze, assert, KEY__SYNTHETIC_MODE, getOwnPropertyDescriptor as getOwnPropertyDescriptor$1, LWC_VERSION_COMMENT_REGEX, LWC_VERSION, getOwnPropertyDescriptors, htmlPropertyToAttribute, ArraySlice, ArrayMap, isTrue, KEY__SCOPED_CSS, KEY__NATIVE_ONLY_CSS, ArraySplice, flattenStylesheets, kebabCaseToCamelCase, StringCharCodeAt, XML_NAMESPACE, XLINK_NAMESPACE, StringSlice, SVG_NAMESPACE, KEY__SHADOW_STATIC, ArraySome, KEY__SHADOW_RESOLVER, normalizeClass, sanitizeHtmlContent, StringReplace, isNumber, ArraySort, ArrayFrom, StringCharAt, htmlEscape, LOWEST_API_VERSION, getContextKeys, isTrustedContext, ContextEventName, ArrayUnshift, KEY__NATIVE_GET_ELEMENT_BY_ID, KEY__NATIVE_QUERY_SELECTOR_ALL, KEY__SHADOW_TOKEN, ID_REFERENCING_ATTRIBUTES_SET, StringSplit, arrayEvery, parseStyleText, ArrayCopyWithin, ArrayFill, ArrayReverse, ArrayShift } from '@lwc/shared';
+import { noop, StringToLowerCase, isNull, ArrayPush as ArrayPush$1, ArrayJoin, isFrozen, isUndefined as isUndefined$1, defineProperty, seal, create, isAPIFeatureEnabled, isFunction as isFunction$1, keys, toString, isString, ArrayFilter, isObject, isArray as isArray$1, getOwnPropertyNames as getOwnPropertyNames$1, getOwnPropertySymbols as getOwnPropertySymbols$1, ArrayIndexOf, ArrayPop, isFalse, legacyIsTrustedSignal, isTrustedSignal, hasOwnProperty as hasOwnProperty$1, entries, AriaPropNameToAttrNameMap, getPropertyDescriptor, AriaAttrNameToPropNameMap, forEach, REFLECTIVE_GLOBAL_PROPERTY_SET, getPrototypeOf as getPrototypeOf$1, setPrototypeOf, defineProperties, assign, freeze, assert, KEY__SYNTHETIC_MODE, getOwnPropertyDescriptor as getOwnPropertyDescriptor$1, LWC_VERSION_COMMENT_REGEX, LWC_VERSION, getOwnPropertyDescriptors, htmlPropertyToAttribute, ArraySlice, ArrayMap, isTrue, KEY__SCOPED_CSS, KEY__NATIVE_ONLY_CSS, ArraySplice, flattenStylesheets, kebabCaseToCamelCase, StringCharCodeAt, XML_NAMESPACE, XLINK_NAMESPACE, StringSlice, SVG_NAMESPACE, KEY__SHADOW_STATIC, ArraySome, KEY__SHADOW_RESOLVER, normalizeClass, sanitizeHtmlContent, StringReplace, isNumber, ArraySort, ArrayFrom, StringCharAt, htmlEscape, LOWEST_API_VERSION, getContextKeys, isTrustedContext, ContextEventName, ArrayUnshift, KEY__NATIVE_GET_ELEMENT_BY_ID, KEY__NATIVE_QUERY_SELECTOR_ALL, KEY__SHADOW_TOKEN, ID_REFERENCING_ATTRIBUTES_SET, StringSplit, arrayEvery, parseStyleText, ArrayCopyWithin, ArrayFill, ArrayReverse, ArrayShift } from '@lwc/shared';
 export { addTrustedContext, isTrustedSignal, setContextKeys, setHooks, setTrustedContextSet, setTrustedSignalSet } from '@lwc/shared';
 export { setFeatureFlag, setFeatureFlagForTest } from '@lwc/features';
 export { SignalBaseClass } from '@lwc/signals';
@@ -608,12 +608,23 @@ function componentValueObserved(vm, key, target = {}) {
     if (lwcRuntimeFlags.ENABLE_EXPERIMENTAL_SIGNALS &&
         isObject(target) &&
         !isNull(target) &&
-        isTrustedSignal(target) &&
         process.env.IS_BROWSER &&
         // Only subscribe if a template is being rendered by the engine
         tro.isObserving()) {
-        // Subscribe the template reactive observer's notify method, which will mark the vm as dirty and schedule hydration.
-        subscribeToSignal(component, target, tro.notify.bind(tro));
+        /**
+         * The legacy validation behavior was that this check should only
+         * be performed for runtimes that have provided a trustedSignals set.
+         * However, this resulted in a bug as all object values were
+         * being considered signals in environments where the trustedSignals
+         * set had not been defined. The runtime flag has been added as a killswitch
+         * in case the fix needs to be reverted.
+         */
+        if (lwcRuntimeFlags.ENABLE_LEGACY_SIGNAL_CONTEXT_VALIDATION
+            ? legacyIsTrustedSignal(target)
+            : isTrustedSignal(target)) {
+            // Subscribe the template reactive observer's notify method, which will mark the vm as dirty and schedule hydration.
+            subscribeToSignal(component, target, tro.notify.bind(tro));
+        }
     }
 }
 function createReactiveObserver(callback) {
@@ -6819,42 +6830,72 @@ class ContextBinding {
 }
 _ContextBinding_renderer = new WeakMap(), _ContextBinding_providedContextVarieties = new WeakMap(), _ContextBinding_elm = new WeakMap();
 function connectContext(vm) {
+    /**
+     * If ENABLE_LEGACY_CONTEXT_CONNECTION is true, enumerates directly on the component
+     * which can result in the component lifecycle observing properties that are not typically observed.
+     * See PR #5536 for more information.
+     */
+    if (lwcRuntimeFlags.ENABLE_LEGACY_CONTEXT_CONNECTION) {
+        connect(vm, keys(getPrototypeOf$1(vm.component)), vm.component);
+    }
+    else {
+        // Non-decorated objects
+        connect(vm, keys(vm.cmpFields), vm.cmpFields);
+        // Decorated objects like @api context
+        connect(vm, keys(vm.cmpProps), vm.cmpProps);
+    }
+}
+function disconnectContext(vm) {
+    /**
+     * If ENABLE_LEGACY_CONTEXT_CONNECTION is true, enumerates directly on the component
+     * which can result in the component lifecycle observing properties that are not typically observed.
+     * See PR #5536 for more information.
+     */
+    if (lwcRuntimeFlags.ENABLE_LEGACY_CONTEXT_CONNECTION) {
+        connect(vm, keys(getPrototypeOf$1(vm.component)), vm.component);
+    }
+    else {
+        // Non-decorated objects
+        disconnect(vm, keys(vm.cmpFields), vm.cmpFields);
+        // Decorated objects like @api context
+        disconnect(vm, keys(vm.cmpProps), vm.cmpProps);
+    }
+}
+function connect(vm, enumerableKeys, contextContainer) {
     const contextKeys = getContextKeys();
     if (isUndefined$1(contextKeys)) {
         return;
     }
     const { connectContext } = contextKeys;
     const { component } = vm;
-    const enumerableKeys = keys(getPrototypeOf$1(component));
-    const contextfulKeys = ArrayFilter.call(enumerableKeys, (enumerableKey) => isTrustedContext(component[enumerableKey]));
+    const contextfulKeys = ArrayFilter.call(enumerableKeys, (enumerableKey) => isTrustedContext(contextContainer[enumerableKey]));
     if (contextfulKeys.length === 0) {
         return;
     }
     const providedContextVarieties = new Map();
     try {
         for (let i = 0; i < contextfulKeys.length; i++) {
-            component[contextfulKeys[i]][connectContext](new ContextBinding(vm, component, providedContextVarieties));
+            contextContainer[contextfulKeys[i]][connectContext](new ContextBinding(vm, component, providedContextVarieties));
         }
     }
     catch (err) {
         logWarnOnce(`Attempted to connect to trusted context but received the following error: ${err.message}`);
     }
 }
-function disconnectContext(vm) {
+function disconnect(vm, enumerableKeys, contextContainer) {
     const contextKeys = getContextKeys();
     if (!contextKeys) {
         return;
     }
     const { disconnectContext } = contextKeys;
     const { component } = vm;
-    const enumerableKeys = keys(getPrototypeOf$1(component));
-    const contextfulKeys = ArrayFilter.call(enumerableKeys, (enumerableKey) => isTrustedContext(component[enumerableKey]));
+    const contextfulKeys = ArrayFilter.call(enumerableKeys, (enumerableKey) => isTrustedContext(contextContainer[enumerableKey]));
     if (contextfulKeys.length === 0) {
         return;
     }
     try {
         for (let i = 0; i < contextfulKeys.length; i++) {
-            component[contextfulKeys[i]][disconnectContext](component);
+            contextContainer[contextfulKeys[i]][disconnectContext](component);
         }
     }
     catch (err) {
@@ -8667,5 +8708,5 @@ function readonly(obj) {
 }
 
 export { BaseBridgeElement, LightningElement, profilerControl as __unstable__ProfilerControl, reportingControl as __unstable__ReportingControl, api$1 as api, computeShadowAndRenderMode, connectRootElement, createContextProviderWithRegister, createVM, disconnectRootElement, freezeTemplate, getAssociatedVMIfPresent, getComponentAPIVersion, getComponentConstructor, getComponentDef, getComponentHtmlPrototype, hydrateRoot, isComponentConstructor, parseFragment, parseSVGFragment, readonly, registerComponent, registerDecorators, registerTemplate, runFormAssociatedCallback, runFormDisabledCallback, runFormResetCallback, runFormStateRestoreCallback, sanitizeAttribute, shouldBeFormAssociated, swapComponent, swapStyle, swapTemplate, track, unwrap, wire };
-/** version: 8.20.4 */
+/** version: 8.20.6 */
 //# sourceMappingURL=index.js.map

package/package.json

@@ -4,7 +4,7 @@
         "You can safely modify dependencies, devDependencies, keywords, etc., but other props will be overwritten."
     ],
     "name": "@lwc/engine-core",
-    "version": "8.20.4",
+    "version": "8.20.6",
     "description": "Core LWC engine APIs.",
     "keywords": [
         "lwc"
@@ -46,11 +46,11 @@
         }
     },
     "dependencies": {
-        "@lwc/features": "8.20.4",
-        "@lwc/shared": "8.20.4",
-        "@lwc/signals": "8.20.4"
+        "@lwc/features": "8.20.6",
+        "@lwc/shared": "8.20.6",
+        "@lwc/signals": "8.20.6"
     },
     "devDependencies": {
         "observable-membrane": "2.0.0"
     }
-}
+}