lwc 2.43.0 → 2.44.0

package/dist/engine-dom/umd/es2017/engine-dom.js

@@ -49,7 +49,7 @@
      */
     const { assign, create, defineProperties, defineProperty, freeze, getOwnPropertyDescriptor: getOwnPropertyDescriptor$1, getOwnPropertyNames: getOwnPropertyNames$1, getPrototypeOf: getPrototypeOf$1, hasOwnProperty: hasOwnProperty$1, isFrozen, keys, seal, setPrototypeOf, } = Object;
     const { isArray: isArray$1 } = Array;
-    const { concat: ArrayConcat$1, copyWithin: ArrayCopyWithin, fill: ArrayFill, filter: ArrayFilter, find: ArrayFind, indexOf: ArrayIndexOf, join: ArrayJoin, map: ArrayMap, pop: ArrayPop, push: ArrayPush$1, reduce: ArrayReduce, reverse: ArrayReverse, shift: ArrayShift, slice: ArraySlice, some: ArraySome, sort: ArraySort, splice: ArraySplice, unshift: ArrayUnshift, forEach, } = Array.prototype;
+    const { concat: ArrayConcat$1, copyWithin: ArrayCopyWithin, fill: ArrayFill, filter: ArrayFilter, find: ArrayFind, findIndex: ArrayFindIndex, indexOf: ArrayIndexOf, join: ArrayJoin, map: ArrayMap, pop: ArrayPop, push: ArrayPush$1, reduce: ArrayReduce, reverse: ArrayReverse, shift: ArrayShift, slice: ArraySlice, some: ArraySome, sort: ArraySort, splice: ArraySplice, unshift: ArrayUnshift, forEach, } = Array.prototype;
     const { fromCharCode: StringFromCharCode } = String;
     const { charCodeAt: StringCharCodeAt, replace: StringReplace, split: StringSplit, slice: StringSlice, toLowerCase: StringToLowerCase, } = String.prototype;
     function isUndefined$1(obj) {
@@ -328,24 +328,6 @@
         return propertyName;
     }
 
-    /*
-     * Copyright (c) 2020, 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
-     */
-    const ESCAPED_CHARS = {
-        '"': '"',
-        "'": ''',
-        '<': '&lt;',
-        '>': '&gt;',
-        '&': '&amp;',
-    };
-    function htmlEscape(str, attrMode = false) {
-        const searchValue = attrMode ? /["&]/g : /["'<>&]/g;
-        return str.replace(searchValue, (char) => ESCAPED_CHARS[char]);
-    }
-
     /*
      * Copyright (c) 2018, salesforce.com, inc.
      * All rights reserved.
@@ -353,9 +335,9 @@
      * For full license text, see the LICENSE file in the repo root or https://opensource.org/licenses/MIT
      */
     // Increment whenever the LWC template compiler changes
-    const LWC_VERSION = "2.43.0";
+    const LWC_VERSION = "2.44.0";
     const LWC_VERSION_COMMENT_REGEX = /\/\*LWC compiler v([\d.]+)\*\/\s*}/;
-    /** version: 2.43.0 */
+    /** version: 2.44.0 */
 
     /**
      * Copyright (C) 2018 salesforce.com, inc.
@@ -433,7 +415,7 @@
             setFeatureFlag(name, value);
         }
     }
-    /** version: 2.43.0 */
+    /** version: 2.44.0 */
 
     /**
      * Copyright (C) 2018 salesforce.com, inc.
@@ -497,7 +479,7 @@
             }
         }
     }
-    /** version: 2.43.0 */
+    /** version: 2.44.0 */
 
     /*
      * Copyright (c) 2018, salesforce.com, inc.
@@ -940,101 +922,6 @@
         'tabIndex',
         'title',
     ];
-    function offsetPropertyErrorMessage(name) {
-        return `Using the \`${name}\` property is an anti-pattern because it rounds the value to an integer. Instead, use the \`getBoundingClientRect\` method to obtain fractional values for the size of an element and its position relative to the viewport.`;
-    }
-    // Global HTML Attributes & Properties
-    // https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes
-    // https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement
-    //
-    // If you update this list, check for test files that recapitulate the same list. Searching the codebase
-    // for e.g. "dropzone" should suffice.
-    const globalHTMLProperties = {
-        accessKey: {
-            attribute: 'accesskey',
-        },
-        accessKeyLabel: {
-            readOnly: true,
-        },
-        className: {
-            attribute: 'class',
-            error: 'Using the `className` property is an anti-pattern because of slow runtime behavior and potential conflicts with classes provided by the owner element. Use the `classList` API instead.',
-        },
-        contentEditable: {
-            attribute: 'contenteditable',
-        },
-        dataset: {
-            readOnly: true,
-            error: "Using the `dataset` property is an anti-pattern because it can't be statically analyzed. Expose each property individually using the `@api` decorator instead.",
-        },
-        dir: {
-            attribute: 'dir',
-        },
-        draggable: {
-            attribute: 'draggable',
-        },
-        dropzone: {
-            attribute: 'dropzone',
-            readOnly: true,
-        },
-        hidden: {
-            attribute: 'hidden',
-        },
-        id: {
-            attribute: 'id',
-        },
-        inputMode: {
-            attribute: 'inputmode',
-        },
-        lang: {
-            attribute: 'lang',
-        },
-        slot: {
-            attribute: 'slot',
-            error: 'Using the `slot` property is an anti-pattern.',
-        },
-        spellcheck: {
-            attribute: 'spellcheck',
-        },
-        style: {
-            attribute: 'style',
-        },
-        tabIndex: {
-            attribute: 'tabindex',
-        },
-        title: {
-            attribute: 'title',
-        },
-        translate: {
-            attribute: 'translate',
-        },
-        // additional "global attributes" that are not present in the link above.
-        isContentEditable: {
-            readOnly: true,
-        },
-        offsetHeight: {
-            readOnly: true,
-            error: offsetPropertyErrorMessage('offsetHeight'),
-        },
-        offsetLeft: {
-            readOnly: true,
-            error: offsetPropertyErrorMessage('offsetLeft'),
-        },
-        offsetParent: {
-            readOnly: true,
-        },
-        offsetTop: {
-            readOnly: true,
-            error: offsetPropertyErrorMessage('offsetTop'),
-        },
-        offsetWidth: {
-            readOnly: true,
-            error: offsetPropertyErrorMessage('offsetWidth'),
-        },
-        role: {
-            attribute: 'role',
-        },
-    };
     let controlledElement = null;
     let controlledAttributeName;
     function isAttributeLocked(elm, attrName) {
@@ -1219,8 +1106,7 @@
             }),
             addEventListener: generateDataDescriptor({
                 value(type, listener, options) {
-                    // TODO [#420]: this is triggered when the component author attempts to add a listener
-                    // programmatically into its Component's shadow root
+                    // TODO [#1824]: Potentially relax this restriction
                     if (!isUndefined$1(options)) {
                         logError('The `addEventListener` method on ShadowRoot does not support any options.', getAssociatedVMIfPresent(this));
                     }
@@ -1266,8 +1152,7 @@
             }),
             addEventListener: generateDataDescriptor({
                 value(type, listener, options) {
-                    // TODO [#420]: this is triggered when the component author attempts to add a listener
-                    // programmatically into a lighting element node
+                    // TODO [#1824]: Potentially relax this restriction
                     if (!isUndefined$1(options)) {
                         logError('The `addEventListener` method in `LightningElement` does not support any options.', getAssociatedVMIfPresent(this));
                     }
@@ -1295,7 +1180,7 @@
     function getLightningElementPrototypeRestrictionsDescriptors(proto) {
         assertNotProd(); // this method should never leak to prod
         const originalDispatchEvent = proto.dispatchEvent;
-        const descriptors = {
+        return {
             dispatchEvent: generateDataDescriptor({
                 value(event) {
                     const vm = getAssociatedVM(this);
@@ -1313,32 +1198,6 @@
                 },
             }),
         };
-        forEach.call(getOwnPropertyNames$1(globalHTMLProperties), (propName) => {
-            if (propName in proto) {
-                return; // no need to redefine something that we are already exposing
-            }
-            descriptors[propName] = generateAccessorDescriptor({
-                get() {
-                    const { error, attribute } = globalHTMLProperties[propName];
-                    const msg = [];
-                    msg.push(`Accessing the global HTML property "${propName}" is disabled.`);
-                    if (error) {
-                        msg.push(error);
-                    }
-                    else if (attribute) {
-                        msg.push(`Instead access it via \`this.getAttribute("${attribute}")\`.`);
-                    }
-                    logError(msg.join('\n'), getAssociatedVM(this));
-                },
-                set() {
-                    const { readOnly } = globalHTMLProperties[propName];
-                    if (readOnly) {
-                        logError(`The global HTML property \`${propName}\` is read-only.`, getAssociatedVM(this));
-                    }
-                },
-            });
-        });
-        return descriptors;
     }
     // This routine will prevent access to certain properties on a shadow root instance to guarantee
     // that all components will work fine in IE11 and other browsers without shadow dom support.
@@ -2689,12 +2548,7 @@
     }
     function createPublicAccessorDescriptor(key, descriptor) {
         const { get, set, enumerable, configurable } = descriptor;
-        if (!isFunction$1(get)) {
-            if (process.env.NODE_ENV !== 'production') {
-                assert.invariant(isFunction$1(get), `Invalid compiler output for public accessor ${toString$1(key)} decorated with @api`);
-            }
-            throw new Error();
-        }
+        assert.invariant(isFunction$1(get), `Invalid public accessor ${toString$1(key)} decorated with @api. The property is missing a getter.`);
         return {
             get() {
                 if (process.env.NODE_ENV !== 'production') {
@@ -2816,64 +2670,64 @@
         }
     }
     function validateObservedField(Ctor, fieldName, descriptor) {
+        assertNotProd(); // this method should never leak to prod
         if (!isUndefined$1(descriptor)) {
             const type = getClassDescriptorType(descriptor);
             const message = `Invalid observed ${fieldName} field. Found a duplicate ${type} with the same name.`;
-            // [W-9927596] Ideally we always throw an error when detecting duplicate observed field.
-            // This branch is only here for backward compatibility reasons.
-            if (type === "accessor" /* DescriptorType.Accessor */) {
-                logError(message);
-            }
-            else {
-                assert.fail(message);
-            }
+            // TODO [#3408]: this should throw, not log
+            logError(message);
         }
     }
     function validateFieldDecoratedWithTrack(Ctor, fieldName, descriptor) {
+        assertNotProd(); // this method should never leak to prod
         if (!isUndefined$1(descriptor)) {
             const type = getClassDescriptorType(descriptor);
-            assert.fail(`Invalid @track ${fieldName} field. Found a duplicate ${type} with the same name.`);
+            // TODO [#3408]: this should throw, not log
+            logError(`Invalid @track ${fieldName} field. Found a duplicate ${type} with the same name.`);
         }
     }
     function validateFieldDecoratedWithWire(Ctor, fieldName, descriptor) {
+        assertNotProd(); // this method should never leak to prod
         if (!isUndefined$1(descriptor)) {
             const type = getClassDescriptorType(descriptor);
-            assert.fail(`Invalid @wire ${fieldName} field. Found a duplicate ${type} with the same name.`);
+            // TODO [#3408]: this should throw, not log
+            logError(`Invalid @wire ${fieldName} field. Found a duplicate ${type} with the same name.`);
         }
     }
     function validateMethodDecoratedWithWire(Ctor, methodName, descriptor) {
+        assertNotProd(); // this method should never leak to prod
         if (isUndefined$1(descriptor) || !isFunction$1(descriptor.value) || isFalse(descriptor.writable)) {
-            assert.fail(`Invalid @wire ${methodName} method.`);
+            // TODO [#3441]: This line of code does not seem possible to reach.
+            logError(`Invalid @wire ${methodName} field. The field should have a valid writable descriptor.`);
         }
     }
     function validateFieldDecoratedWithApi(Ctor, fieldName, descriptor) {
+        assertNotProd(); // this method should never leak to prod
         if (!isUndefined$1(descriptor)) {
             const type = getClassDescriptorType(descriptor);
             const message = `Invalid @api ${fieldName} field. Found a duplicate ${type} with the same name.`;
-            // [W-9927596] Ideally we always throw an error when detecting duplicate public properties.
-            // This branch is only here for backward compatibility reasons.
-            if (type === "accessor" /* DescriptorType.Accessor */) {
-                logError(message);
-            }
-            else {
-                assert.fail(message);
-            }
+            // TODO [#3408]: this should throw, not log
+            logError(message);
         }
     }
     function validateAccessorDecoratedWithApi(Ctor, fieldName, descriptor) {
-        if (isUndefined$1(descriptor)) {
-            assert.fail(`Invalid @api get ${fieldName} accessor.`);
-        }
-        else if (isFunction$1(descriptor.set)) {
-            assert.isTrue(isFunction$1(descriptor.get), `Missing getter for property ${fieldName} decorated with @api in ${Ctor}. You cannot have a setter without the corresponding getter.`);
+        assertNotProd(); // this method should never leak to prod
+        if (isFunction$1(descriptor.set)) {
+            if (!isFunction$1(descriptor.get)) {
+                // TODO [#3441]: This line of code does not seem possible to reach.
+                logError(`Missing getter for property ${fieldName} decorated with @api in ${Ctor}. You cannot have a setter without the corresponding getter.`);
+            }
         }
         else if (!isFunction$1(descriptor.get)) {
-            assert.fail(`Missing @api get ${fieldName} accessor.`);
+            // TODO [#3441]: This line of code does not seem possible to reach.
+            logError(`Missing @api get ${fieldName} accessor.`);
         }
     }
     function validateMethodDecoratedWithApi(Ctor, methodName, descriptor) {
+        assertNotProd(); // this method should never leak to prod
         if (isUndefined$1(descriptor) || !isFunction$1(descriptor.value) || isFalse(descriptor.writable)) {
-            assert.fail(`Invalid @api ${methodName} method.`);
+            // TODO [#3441]: This line of code does not seem possible to reach.
+            logError(`Invalid @api ${methodName} method.`);
         }
     }
     /**
@@ -2896,13 +2750,14 @@
                 apiFieldsConfig[fieldName] = propConfig.config;
                 descriptor = getOwnPropertyDescriptor$1(proto, fieldName);
                 if (propConfig.config > 0) {
+                    if (isUndefined$1(descriptor)) {
+                        // TODO [#3441]: This line of code does not seem possible to reach.
+                        throw new Error();
+                    }
                     // accessor declaration
                     if (process.env.NODE_ENV !== 'production') {
                         validateAccessorDecoratedWithApi(Ctor, fieldName, descriptor);
                     }
-                    if (isUndefined$1(descriptor)) {
-                        throw new Error();
-                    }
                     descriptor = createPublicAccessorDescriptor(fieldName, descriptor);
                 }
                 else {
@@ -2942,7 +2797,10 @@
                 descriptor = getOwnPropertyDescriptor$1(proto, fieldOrMethodName);
                 if (method === 1) {
                     if (process.env.NODE_ENV !== 'production') {
-                        assert.isTrue(adapter, `@wire on method "${fieldOrMethodName}": adapter id must be truthy.`);
+                        if (!adapter) {
+                            // TODO [#3408]: this should throw, not log
+                            logError(`@wire on method "${fieldOrMethodName}": adapter id must be truthy.`);
+                        }
                         validateMethodDecoratedWithWire(Ctor, fieldOrMethodName, descriptor);
                     }
                     if (isUndefined$1(descriptor)) {
@@ -2953,7 +2811,10 @@
                 }
                 else {
                     if (process.env.NODE_ENV !== 'production') {
-                        assert.isTrue(adapter, `@wire on field "${fieldOrMethodName}": adapter id must be truthy.`);
+                        if (!adapter) {
+                            // TODO [#3408]: this should throw, not log
+                            logError(`@wire on field "${fieldOrMethodName}": adapter id must be truthy.`);
+                        }
                         validateFieldDecoratedWithWire(Ctor, fieldOrMethodName, descriptor);
                     }
                     descriptor = internalWireFieldDecorator(fieldOrMethodName);
@@ -6828,8 +6689,6 @@
         return renderer.nextSibling(hydratedNode);
     }
     const NODE_VALUE_PROP = 'nodeValue';
-    const PARENT_NODE_PROP = 'parentNode';
-    const TAG_NAME_PROP = 'tagName';
     function textNodeContentsAreEqual(node, vnode, renderer) {
         const { getProperty } = renderer;
         const nodeValue = getProperty(node, NODE_VALUE_PROP);
@@ -6841,22 +6700,6 @@
         if (nodeValue === '\u200D' && vnode.text === '') {
             return true;
         }
-        // Special case for text nodes inside `<style>` tags – these are escaped when rendered server-size,
-        // but not when generated by the engine client-side.
-        const parentNode = getProperty(node, PARENT_NODE_PROP);
-        // Should never be null, but just to be safe, we check.
-        /* istanbul ignore else */
-        if (!isNull(parentNode)) {
-            const tagName = getProperty(parentNode, TAG_NAME_PROP);
-            // If the tagName is STYLE, then the following condition should always be true.
-            // The LWC compiler blocks using `<style>`s inside of templates, so it should be impossible
-            // for component authors to render different `<style>` text content on the client and server.
-            // But just to be safe, we check.
-            /* istanbul ignore next */
-            if (tagName === 'STYLE' && htmlEscape(vnode.text) === nodeValue) {
-                return true;
-            }
-        }
         return false;
     }
     function hydrateText(node, vnode, renderer) {
@@ -7482,12 +7325,12 @@
         if (process.env.NODE_ENV !== 'production') {
             // TODO [#1292]: Remove the readonly decorator
             if (arguments.length !== 1) {
-                assert.fail('@readonly cannot be used as a decorator just yet, use it as a function with one argument to produce a readonly version of the provided value.');
+                logError('@readonly cannot be used as a decorator just yet, use it as a function with one argument to produce a readonly version of the provided value.');
             }
         }
         return getReadOnlyProxy(obj);
     }
-    /* version: 2.43.0 */
+    /* version: 2.44.0 */
 
     /*
      * Copyright (c) 2018, salesforce.com, inc.
@@ -7915,7 +7758,7 @@
         function isNull(obj) {
             return obj === null;
         }
-        /** version: 2.43.0 */
+        /** version: 2.44.0 */
 
         /*
          * Copyright (c) 2023, salesforce.com, inc.
@@ -8534,7 +8377,7 @@
             });
         }));
     }
-    /* version: 2.43.0 */
+    /* version: 2.44.0 */
 
     exports.LightningElement = LightningElement;
     exports.__unstable__ProfilerControl = profilerControl;