@lwc/engine-core 2.27.0 → 2.28.0

package/dist/engine-core.cjs.js

@@ -3756,9 +3756,14 @@ function mountCustomElement(vnode, parent, anchor, renderer) {
     if (features.lwcRuntimeFlags.ENABLE_NATIVE_CUSTOM_ELEMENT_LIFECYCLE) {
       disconnectRootElement(elm);
     }
-  };
+  }; // Should never get a tag with upper case letter at this point; the compiler
+  // should produce only tags with lowercase letters. However, the Java
+  // compiler may generate tagnames with uppercase letters so - for backwards
+  // compatibility, we lower case the tagname here.
+
 
-  const elm = createCustomElement(sel, upgradeCallback, connectedCallback, disconnectedCallback);
+  const normalizedTagname = sel.toLowerCase();
+  const elm = createCustomElement(normalizedTagname, upgradeCallback, connectedCallback, disconnectedCallback);
   vnode.elm = elm;
   vnode.vm = vm;
   linkNodeToShadow(elm, owner, renderer);
@@ -4064,6 +4069,17 @@ function allocateChildren(vnode, vm) {
     shadowMode
   } = vm;
 
+  if (process.env.NODE_ENV !== 'production') {
+    // If any of the children being allocated is a scoped slot fragment, make sure the receiving
+    // component is a light DOM component. This is mainly to validate light dom parent running
+    // in native shadow mode.
+    if (renderMode !== 0
+    /* RenderMode.Light */
+    && shared.ArraySome.call(children, child => !shared.isNull(child) && isVScopedSlotFragment(child))) {
+      logError(`Invalid usage of 'lwc:slot-data' on ${getComponentTag(vm)} tag. Scoped slot content can only be passed to a light dom child.`);
+    }
+  }
+
   if (shadowMode === 1
   /* ShadowMode.Synthetic */
   || renderMode === 0
@@ -4458,27 +4474,42 @@ function s(slotName, data, children, slotset) {
         !shared.isUndefined(slotset.slotAssignments) &&
         !shared.isUndefined(slotset.slotAssignments[slotName]) &&
         slotset.slotAssignments[slotName].length !== 0) {
-        children = slotset.slotAssignments[slotName].reduce((acc, vnode) => {
-            // If the passed slot content is factory, evaluate it and use the produced vnodes
-            if (vnode && isVScopedSlotFragment(vnode)) {
-                const vmBeingRenderedInception = getVMBeingRendered();
-                let children = [];
-                // Evaluate in the scope of the slot content's owner
-                // if a slotset is provided, there will always be an owner. The only case where owner is
-                // undefined is for root components, but root components cannot accept slotted content
-                setVMBeingRendered(slotset.owner);
-                try {
-                    children = vnode.factory(data.slotData);
+        children = slotset.slotAssignments[slotName].reduce((accumulator, vnode) => {
+            if (vnode) {
+                const assignedNodeIsScopedSlot = isVScopedSlotFragment(vnode);
+                // The only sniff test for a scoped <slot> element is the presence of `slotData`
+                const isScopedSlotElement = !shared.isUndefined(data.slotData);
+                // Check if slot types of parent and child are matching
+                if (assignedNodeIsScopedSlot !== isScopedSlotElement) {
+                    if (process.env.NODE_ENV !== 'production') {
+                        logError(`Mismatched slot types for ${slotName === '' ? '(default)' : slotName} slot. Both parent and child component must use standard type or scoped type for a given slot.`, slotset.owner);
+                    }
+                    // Ignore slot content from parent
+                    return accumulator;
                 }
-                finally {
-                    setVMBeingRendered(vmBeingRenderedInception);
+                // If the passed slot content is factory, evaluate it and add the produced vnodes
+                if (assignedNodeIsScopedSlot) {
+                    const vmBeingRenderedInception = getVMBeingRendered();
+                    let scopedSlotChildren = [];
+                    // Evaluate in the scope of the slot content's owner
+                    // if a slotset is provided, there will always be an owner. The only case where owner is
+                    // undefined is for root components, but root components cannot accept slotted content
+                    setVMBeingRendered(slotset.owner);
+                    try {
+                        scopedSlotChildren = vnode.factory(data.slotData);
+                    }
+                    finally {
+                        setVMBeingRendered(vmBeingRenderedInception);
+                    }
+                    return shared.ArrayConcat.call(accumulator, scopedSlotChildren);
+                }
+                else {
+                    // If the slot content is standard type, the content is static, no additional
+                    // processing needed on the vnode
+                    return shared.ArrayConcat.call(accumulator, vnode);
                 }
-                return shared.ArrayConcat.call(acc, children);
-            }
-            else {
-                // If the slot content is a static list of child nodes provided by the parent, nothing to do
-                return shared.ArrayConcat.call(acc, vnode);
             }
+            return accumulator;
         }, []);
     }
     const vmBeingRendered = getVMBeingRendered();
@@ -6888,4 +6919,4 @@ exports.swapTemplate = swapTemplate;
 exports.track = track;
 exports.unwrap = unwrap;
 exports.wire = wire;
-/* version: 2.27.0 */
+/* version: 2.28.0 */

package/dist/engine-core.js

@@ -1,7 +1,7 @@
 /* proxy-compat-disable */
 import { lwcRuntimeFlags } from '@lwc/features';
 export { setFeatureFlag, setFeatureFlagForTest } from '@lwc/features';
-import { seal, create, isUndefined as isUndefined$1, isFunction as isFunction$1, ArrayPush as ArrayPush$1, ArrayIndexOf, ArraySplice, StringToLowerCase, isNull, ArrayJoin, isFrozen, defineProperty, hasOwnProperty as hasOwnProperty$1, assign, forEach, keys, AriaPropNameToAttrNameMap, getPropertyDescriptor, defineProperties, getOwnPropertyNames as getOwnPropertyNames$1, getPrototypeOf as getPrototypeOf$1, setPrototypeOf, isObject, freeze, assert, KEY__SYNTHETIC_MODE, isFalse, isTrue, toString as toString$1, getOwnPropertyDescriptor as getOwnPropertyDescriptor$1, LWC_VERSION_COMMENT_REGEX, LWC_VERSION, htmlPropertyToAttribute, ArraySlice, ArrayMap, isArray as isArray$1, KEY__SCOPED_CSS, StringCharCodeAt, XML_NAMESPACE, XLINK_NAMESPACE, isString, StringSlice, SVG_NAMESPACE, KEY__SHADOW_STATIC, KEY__SHADOW_RESOLVER, ArrayConcat as ArrayConcat$1, isNumber, StringReplace, noop, ArrayUnshift, ArrayFilter, ArrayCopyWithin, ArrayFill, ArraySort, ArrayReverse, ArrayShift, ArrayPop } from '@lwc/shared';
+import { seal, create, isUndefined as isUndefined$1, isFunction as isFunction$1, ArrayPush as ArrayPush$1, ArrayIndexOf, ArraySplice, StringToLowerCase, isNull, ArrayJoin, isFrozen, defineProperty, hasOwnProperty as hasOwnProperty$1, assign, forEach, keys, AriaPropNameToAttrNameMap, getPropertyDescriptor, defineProperties, getOwnPropertyNames as getOwnPropertyNames$1, getPrototypeOf as getPrototypeOf$1, setPrototypeOf, isObject, freeze, assert, KEY__SYNTHETIC_MODE, isFalse, isTrue, toString as toString$1, getOwnPropertyDescriptor as getOwnPropertyDescriptor$1, LWC_VERSION_COMMENT_REGEX, LWC_VERSION, htmlPropertyToAttribute, ArraySlice, ArrayMap, isArray as isArray$1, KEY__SCOPED_CSS, StringCharCodeAt, XML_NAMESPACE, XLINK_NAMESPACE, isString, StringSlice, SVG_NAMESPACE, KEY__SHADOW_STATIC, KEY__SHADOW_RESOLVER, ArraySome, ArrayConcat as ArrayConcat$1, isNumber, StringReplace, noop, ArrayUnshift, ArrayFilter, ArrayCopyWithin, ArrayFill, ArraySort, ArrayReverse, ArrayShift, ArrayPop } from '@lwc/shared';
 
 /*
  * Copyright (c) 2018, salesforce.com, inc.
@@ -3753,9 +3753,14 @@ function mountCustomElement(vnode, parent, anchor, renderer) {
     if (lwcRuntimeFlags.ENABLE_NATIVE_CUSTOM_ELEMENT_LIFECYCLE) {
       disconnectRootElement(elm);
     }
-  };
+  }; // Should never get a tag with upper case letter at this point; the compiler
+  // should produce only tags with lowercase letters. However, the Java
+  // compiler may generate tagnames with uppercase letters so - for backwards
+  // compatibility, we lower case the tagname here.
+
 
-  const elm = createCustomElement(sel, upgradeCallback, connectedCallback, disconnectedCallback);
+  const normalizedTagname = sel.toLowerCase();
+  const elm = createCustomElement(normalizedTagname, upgradeCallback, connectedCallback, disconnectedCallback);
   vnode.elm = elm;
   vnode.vm = vm;
   linkNodeToShadow(elm, owner, renderer);
@@ -4061,6 +4066,17 @@ function allocateChildren(vnode, vm) {
     shadowMode
   } = vm;
 
+  if (process.env.NODE_ENV !== 'production') {
+    // If any of the children being allocated is a scoped slot fragment, make sure the receiving
+    // component is a light DOM component. This is mainly to validate light dom parent running
+    // in native shadow mode.
+    if (renderMode !== 0
+    /* RenderMode.Light */
+    && ArraySome.call(children, child => !isNull(child) && isVScopedSlotFragment(child))) {
+      logError(`Invalid usage of 'lwc:slot-data' on ${getComponentTag(vm)} tag. Scoped slot content can only be passed to a light dom child.`);
+    }
+  }
+
   if (shadowMode === 1
   /* ShadowMode.Synthetic */
   || renderMode === 0
@@ -4455,27 +4471,42 @@ function s(slotName, data, children, slotset) {
         !isUndefined$1(slotset.slotAssignments) &&
         !isUndefined$1(slotset.slotAssignments[slotName]) &&
         slotset.slotAssignments[slotName].length !== 0) {
-        children = slotset.slotAssignments[slotName].reduce((acc, vnode) => {
-            // If the passed slot content is factory, evaluate it and use the produced vnodes
-            if (vnode && isVScopedSlotFragment(vnode)) {
-                const vmBeingRenderedInception = getVMBeingRendered();
-                let children = [];
-                // Evaluate in the scope of the slot content's owner
-                // if a slotset is provided, there will always be an owner. The only case where owner is
-                // undefined is for root components, but root components cannot accept slotted content
-                setVMBeingRendered(slotset.owner);
-                try {
-                    children = vnode.factory(data.slotData);
+        children = slotset.slotAssignments[slotName].reduce((accumulator, vnode) => {
+            if (vnode) {
+                const assignedNodeIsScopedSlot = isVScopedSlotFragment(vnode);
+                // The only sniff test for a scoped <slot> element is the presence of `slotData`
+                const isScopedSlotElement = !isUndefined$1(data.slotData);
+                // Check if slot types of parent and child are matching
+                if (assignedNodeIsScopedSlot !== isScopedSlotElement) {
+                    if (process.env.NODE_ENV !== 'production') {
+                        logError(`Mismatched slot types for ${slotName === '' ? '(default)' : slotName} slot. Both parent and child component must use standard type or scoped type for a given slot.`, slotset.owner);
+                    }
+                    // Ignore slot content from parent
+                    return accumulator;
                 }
-                finally {
-                    setVMBeingRendered(vmBeingRenderedInception);
+                // If the passed slot content is factory, evaluate it and add the produced vnodes
+                if (assignedNodeIsScopedSlot) {
+                    const vmBeingRenderedInception = getVMBeingRendered();
+                    let scopedSlotChildren = [];
+                    // Evaluate in the scope of the slot content's owner
+                    // if a slotset is provided, there will always be an owner. The only case where owner is
+                    // undefined is for root components, but root components cannot accept slotted content
+                    setVMBeingRendered(slotset.owner);
+                    try {
+                        scopedSlotChildren = vnode.factory(data.slotData);
+                    }
+                    finally {
+                        setVMBeingRendered(vmBeingRenderedInception);
+                    }
+                    return ArrayConcat$1.call(accumulator, scopedSlotChildren);
+                }
+                else {
+                    // If the slot content is standard type, the content is static, no additional
+                    // processing needed on the vnode
+                    return ArrayConcat$1.call(accumulator, vnode);
                 }
-                return ArrayConcat$1.call(acc, children);
-            }
-            else {
-                // If the slot content is a static list of child nodes provided by the parent, nothing to do
-                return ArrayConcat$1.call(acc, vnode);
             }
+            return accumulator;
         }, []);
     }
     const vmBeingRendered = getVMBeingRendered();
@@ -6849,4 +6880,4 @@ function getComponentConstructor(elm) {
 }
 
 export { LightningElement, profilerControl as __unstable__ProfilerControl, api$1 as api, connectRootElement, createContextProvider, createVM, disconnectRootElement, freezeTemplate, getAssociatedVMIfPresent, getComponentConstructor, getComponentDef, getComponentHtmlPrototype, hydrateRoot, isComponentConstructor, parseFragment, parseSVGFragment, readonly, register, registerComponent, registerDecorators, registerTemplate, sanitizeAttribute, setHooks, swapComponent, swapStyle, swapTemplate, track, unwrap, wire };
-/* version: 2.27.0 */
+/* version: 2.28.0 */

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@lwc/engine-core",
-    "version": "2.27.0",
+    "version": "2.28.0",
     "description": "Core LWC engine APIs.",
     "homepage": "https://lwc.dev/",
     "repository": {
@@ -24,8 +24,8 @@
         "types/"
     ],
     "dependencies": {
-        "@lwc/features": "2.27.0",
-        "@lwc/shared": "2.27.0"
+        "@lwc/features": "2.28.0",
+        "@lwc/shared": "2.28.0"
     },
     "devDependencies": {
         "observable-membrane": "2.0.0"