@lwc/engine-core 5.0.3 → 5.0.4

package/dist/index.js

@@ -1,7 +1,7 @@
 /**
  * Copyright (C) 2023 salesforce.com, inc.
  */
-import { noop, StringToLowerCase, isNull, ArrayPush as ArrayPush$1, ArrayJoin, isFrozen, isUndefined as isUndefined$1, defineProperty, ArrayIndexOf, ArraySplice, create, seal, isArray as isArray$1, isFunction as isFunction$1, keys, hasOwnProperty as hasOwnProperty$1, globalThis as globalThis$1, AriaPropNameToAttrNameMap, getOwnPropertyDescriptor as getOwnPropertyDescriptor$1, forEach, getPropertyDescriptor, defineProperties, getPrototypeOf as getPrototypeOf$1, setPrototypeOf, assign, isObject, freeze, KEY__SYNTHETIC_MODE, isString, assert, toString as toString$1, isFalse, LWC_VERSION_COMMENT_REGEX, LWC_VERSION, getOwnPropertyNames as getOwnPropertyNames$1, getOwnPropertyDescriptors, htmlPropertyToAttribute, ArraySlice, ArrayMap, KEY__SCOPED_CSS, kebabCaseToCamelCase, StringCharCodeAt, XML_NAMESPACE, XLINK_NAMESPACE, StringSlice, ArrayShift, ArrayUnshift, isTrue, SVG_NAMESPACE, KEY__SHADOW_STATIC, KEY__SHADOW_RESOLVER, ArraySome, ArrayPop, isAPIFeatureEnabled, isNumber, StringReplace, LOWEST_API_VERSION, KEY__NATIVE_GET_ELEMENT_BY_ID, KEY__NATIVE_QUERY_SELECTOR_ALL, ID_REFERENCING_ATTRIBUTES_SET, KEY__SHADOW_TOKEN, ArrayFilter, StringSplit, arrayEvery, ArrayIncludes, ArrayCopyWithin, ArrayFill, ArraySort, ArrayReverse } from '@lwc/shared';
+import { noop, StringToLowerCase, isNull, ArrayPush as ArrayPush$1, ArrayJoin, isFrozen, isUndefined as isUndefined$1, defineProperty, ArrayIndexOf, ArraySplice, create, seal, isArray as isArray$1, isFunction as isFunction$1, keys, hasOwnProperty as hasOwnProperty$1, globalThis as globalThis$1, entries, AriaPropNameToAttrNameMap, getPropertyDescriptor, forEach, defineProperties, getPrototypeOf as getPrototypeOf$1, setPrototypeOf, assign, isObject, freeze, KEY__SYNTHETIC_MODE, isString, assert, toString as toString$1, getOwnPropertyDescriptor as getOwnPropertyDescriptor$1, isFalse, LWC_VERSION_COMMENT_REGEX, LWC_VERSION, getOwnPropertyNames as getOwnPropertyNames$1, getOwnPropertyDescriptors, htmlPropertyToAttribute, ArraySlice, ArrayMap, KEY__SCOPED_CSS, kebabCaseToCamelCase, StringCharCodeAt, XML_NAMESPACE, XLINK_NAMESPACE, StringSlice, ArrayShift, ArrayUnshift, isTrue, SVG_NAMESPACE, KEY__SHADOW_STATIC, KEY__SHADOW_RESOLVER, ArraySome, ArrayPop, isAPIFeatureEnabled, isNumber, StringReplace, LOWEST_API_VERSION, KEY__NATIVE_GET_ELEMENT_BY_ID, KEY__NATIVE_QUERY_SELECTOR_ALL, ID_REFERENCING_ATTRIBUTES_SET, KEY__SHADOW_TOKEN, ArrayFilter, StringSplit, arrayEvery, ArrayIncludes, ArrayCopyWithin, ArrayFill, ArraySort, ArrayReverse } from '@lwc/shared';
 export { setFeatureFlag, setFeatureFlagForTest } from '@lwc/features';
 
 /*
@@ -421,54 +421,53 @@ const instrumentDef = (_a = globalThis$1.__lwc_instrument_cmp_def) !== null && _
 const instrumentInstance = (_b = globalThis$1.__lwc_instrument_cmp_instance) !== null && _b !== void 0 ? _b : noop;
 
 /*
- * Copyright (c) 2023, salesforce.com, inc.
+ * Copyright (c) 2018, salesforce.com, inc.
  * All rights reserved.
  * SPDX-License-Identifier: MIT
  * For full license text, see the LICENSE file in the repo root or https://opensource.org/licenses/MIT
  */
-// Apply ARIA string reflection behavior to a prototype.
-// This is deliberately kept separate from @lwc/aria-reflection. @lwc/aria-reflection is a global polyfill that is
-// needed for backwards compatibility in LEX, whereas `applyAriaReflection` is designed to only apply to our own
-// LightningElement/BaseBridgeElement prototypes.
-function applyAriaReflection(prototype) {
-    for (const propName of keys(AriaPropNameToAttrNameMap)) {
-        const attrName = AriaPropNameToAttrNameMap[propName];
-        if (isUndefined$1(getOwnPropertyDescriptor$1(prototype, propName))) {
-            // Note that we need to call this.{get,set,has,remove}Attribute rather than dereferencing
-            // from Element.prototype, because these methods are overridden in LightningElement.
-            defineProperty(prototype, propName, {
-                get() {
-                    return this.getAttribute(attrName);
-                },
-                set(newValue) {
-                    // TODO [#3284]: there is disagreement between browsers and the spec on how to treat undefined
-                    // Our historical behavior is to only treat null as removing the attribute
-                    // See also https://github.com/w3c/aria/issues/1858
-                    if (isNull(newValue)) {
-                        this.removeAttribute(attrName);
-                    }
-                    else {
-                        this.setAttribute(attrName, newValue);
-                    }
-                },
-                // configurable and enumerable to allow it to be overridden – this mimics Safari's/Chrome's behavior
-                configurable: true,
-                enumerable: true,
-            });
-        }
-    }
-}
+// This is a temporary workaround to get the @lwc/engine-server to evaluate in node without having
+// to inject at runtime.
+const HTMLElementConstructor = typeof HTMLElement !== 'undefined' ? HTMLElement : function () { };
+const HTMLElementPrototype = HTMLElementConstructor.prototype;
 
 /*
- * Copyright (c) 2018, salesforce.com, inc.
+ * Copyright (c) 2023, salesforce.com, inc.
  * All rights reserved.
  * SPDX-License-Identifier: MIT
  * For full license text, see the LICENSE file in the repo root or https://opensource.org/licenses/MIT
  */
-// This is a temporary workaround to get the @lwc/engine-server to evaluate in node without having
-// to inject at runtime.
-const HTMLElementConstructor = typeof HTMLElement !== 'undefined' ? HTMLElement : function () { };
-const HTMLElementPrototype = HTMLElementConstructor.prototype;
+// Apply ARIA string reflection behavior to a prototype.
+// This is deliberately kept separate from @lwc/aria-reflection. @lwc/aria-reflection is a global polyfill that is
+// needed for backwards compatibility in LEX, whereas this is designed to only apply to our own
+// LightningElement/BaseBridgeElement prototypes.
+// Note we only need to handle ARIA reflections that aren't already in Element.prototype
+const ariaReflectionPolyfillDescriptors = create(null);
+for (const [propName, attrName] of entries(AriaPropNameToAttrNameMap)) {
+    if (isUndefined$1(getPropertyDescriptor(HTMLElementPrototype, propName))) {
+        // Note that we need to call this.{get,set,has,remove}Attribute rather than dereferencing
+        // from Element.prototype, because these methods are overridden in LightningElement.
+        ariaReflectionPolyfillDescriptors[propName] = {
+            get() {
+                return this.getAttribute(attrName);
+            },
+            set(newValue) {
+                // TODO [#3284]: there is disagreement between browsers and the spec on how to treat undefined
+                // Our historical behavior is to only treat null as removing the attribute
+                // See also https://github.com/w3c/aria/issues/1858
+                if (isNull(newValue)) {
+                    this.removeAttribute(attrName);
+                }
+                else {
+                    this.setAttribute(attrName, newValue);
+                }
+            },
+            // configurable and enumerable to allow it to be overridden – this mimics Safari's/Chrome's behavior
+            configurable: true,
+            enumerable: true,
+        };
+    }
+}
 
 /*
  * Copyright (c) 2018, salesforce.com, inc.
@@ -1842,11 +1841,22 @@ const lightningBasedDescriptors = create(null);
 for (const propName in HTMLElementOriginalDescriptors) {
     lightningBasedDescriptors[propName] = createBridgeToElementDescriptor(propName, HTMLElementOriginalDescriptors[propName]);
 }
-defineProperties(LightningElement.prototype, lightningBasedDescriptors);
 // Apply ARIA reflection to LightningElement.prototype, on both the browser and server.
 // This allows `this.aria*` property accessors to work from inside a component, and to reflect `aria-*` attrs.
 // Note this works regardless of whether the global ARIA reflection polyfill is applied or not.
-applyAriaReflection(LightningElement.prototype);
+if (process.env.IS_BROWSER) {
+    // In the browser, we use createBridgeToElementDescriptor, so we can get the normal reactivity lifecycle for
+    // aria* properties
+    for (const [propName, descriptor] of entries(ariaReflectionPolyfillDescriptors)) {
+        lightningBasedDescriptors[propName] = createBridgeToElementDescriptor(propName, descriptor);
+    }
+}
+else {
+    // On the server, we cannot use createBridgeToElementDescriptor because getAttribute/setAttribute are
+    // not supported on HTMLElement. So apply the polyfill directly on top of LightningElement
+    defineProperties(LightningElement.prototype, ariaReflectionPolyfillDescriptors);
+}
+defineProperties(LightningElement.prototype, lightningBasedDescriptors);
 defineProperty(LightningElement, 'CustomElementConstructor', {
     get() {
         // If required, a runtime-specific implementation must be defined.
@@ -2696,7 +2706,8 @@ function HTMLBridgeElementFactory(SuperClass, publicProperties, methods, observe
                 // and can break tooling that expects it to be iterable or defined, e.g. Jest:
                 // https://github.com/jestjs/jest/blob/b4c9587/packages/pretty-format/src/plugins/DOMElement.ts#L95
                 // It also doesn't make sense to override e.g. "constructor".
-                .filter((propName) => !(propName in HTMLElementPrototype)));
+                .filter((propName) => !(propName in HTMLElementPrototype) &&
+                !(propName in ariaReflectionPolyfillDescriptors)));
             for (const propName of nonPublicPropertiesToWarnOn) {
                 if (ArrayIndexOf.call(publicProperties, propName) === -1) {
                     descriptors[propName] = createAccessorThatWarns(propName);
@@ -2766,21 +2777,22 @@ function HTMLBridgeElementFactory(SuperClass, publicProperties, methods, observe
     defineProperties(HTMLBridgeElement.prototype, descriptors);
     return HTMLBridgeElement;
 }
-const BaseBridgeElement = HTMLBridgeElementFactory(HTMLElementConstructor, getOwnPropertyNames$1(HTMLElementOriginalDescriptors), [], [], null, false);
-if (process.env.IS_BROWSER) {
-    // This ARIA reflection only really makes sense in the browser. On the server, there is no `renderedCallback()`,
-    // so you cannot do e.g. `this.template.querySelector('x-child').ariaBusy = 'true'`. So we don't need to expose
-    // ARIA props outside the LightningElement
-    //
-    // Apply ARIA reflection to HTMLBridgeElement.prototype. This allows `elm.aria*` property accessors to work from
-    // outside a component, and to reflect `aria-*` attrs. This is especially important because the template compiler
-    // compiles aria-* attrs on components to aria* props.
-    // Note this works regardless of whether the global ARIA reflection polyfill is applied or not.
-    //
-    // Also note that we apply this to BaseBridgeElement.prototype to avoid excessively redefining property
-    // accessors inside the HTMLBridgeElementFactory.
-    applyAriaReflection(BaseBridgeElement.prototype);
-}
+// We do some special handling of non-standard ARIA props like ariaLabelledBy as well as props without (as of this
+// writing) broad cross-browser support like ariaBrailleLabel. This is so the reflection works correctly and preserves
+// backwards compatibility with the previous global polyfill approach.
+//
+// The goal here is to expose `elm.aria*` property accessors to work from outside a component, and to reflect `aria-*`
+// attrs. This is especially important because the template compiler compiles aria-* attrs on components to aria* props.
+// Note this works regardless of whether the global ARIA reflection polyfill is applied or not.
+//
+// Also note this ARIA reflection only really makes sense in the browser. On the server, there is no
+// `renderedCallback()`, so you cannot do e.g. `this.template.querySelector('x-child').ariaBusy = 'true'`. So we don't
+// need to expose ARIA props outside the LightningElement
+const basePublicProperties = [
+    ...getOwnPropertyNames$1(HTMLElementOriginalDescriptors),
+    ...(process.env.IS_BROWSER ? getOwnPropertyNames$1(ariaReflectionPolyfillDescriptors) : []),
+];
+const BaseBridgeElement = HTMLBridgeElementFactory(HTMLElementConstructor, basePublicProperties, [], [], null, false);
 freeze(BaseBridgeElement);
 seal(BaseBridgeElement.prototype);
 
@@ -7313,5 +7325,5 @@ function readonly(obj) {
 }
 
 export { 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, setHooks, swapComponent, swapStyle, swapTemplate, track, unwrap, wire };
-/** version: 5.0.3 */
+/** version: 5.0.4 */
 //# sourceMappingURL=index.js.map