@lwc/engine-core 7.1.2 → 7.2.1

package/dist/framework/stylesheet.d.ts

@@ -12,7 +12,7 @@ export type Stylesheet = (stylesheetToken: string | undefined, useActualHostSele
  */
 export type Stylesheets = Array<Stylesheet | Stylesheets>;
 export declare function updateStylesheetToken(vm: VM, template: Template, legacy: boolean): void;
-export declare function getStylesheetsContent(vm: VM, template: Template): string[];
+export declare function getStylesheetsContent(vm: VM, template: Template): ReadonlyArray<string>;
 /**
  * If the component that is currently being rendered uses scoped styles,
  * this returns the unique token for that scoped stylesheet. Otherwise
@@ -29,5 +29,5 @@ export declare function getScopeTokenClass(owner: VM, legacy: boolean): string |
  * @param vnode
  */
 export declare function getStylesheetTokenHost(vnode: VCustomElement): string | null;
-export declare function createStylesheet(vm: VM, stylesheets: string[]): VNode[] | null;
+export declare function createStylesheet(vm: VM, stylesheets: ReadonlyArray<string>): VNode[] | null;
 export declare function unrenderStylesheet(stylesheet: Stylesheet): void;

package/dist/index.cjs.js

@@ -3133,15 +3133,22 @@ function evaluateStylesheetsContent(stylesheets, stylesheetToken, vm) {
 function getStylesheetsContent(vm, template) {
     const { stylesheets, stylesheetToken } = template;
     const { stylesheets: vmStylesheets } = vm;
-    let content = [];
-    if (hasStyles(stylesheets)) {
-        content = evaluateStylesheetsContent(stylesheets, stylesheetToken, vm);
+    const hasTemplateStyles = hasStyles(stylesheets);
+    const hasVmStyles = hasStyles(vmStylesheets);
+    if (hasTemplateStyles) {
+        const content = evaluateStylesheetsContent(stylesheets, stylesheetToken, vm);
+        if (hasVmStyles) {
+            // Slow path – merge the template styles and vm styles
+            shared.ArrayPush.apply(content, evaluateStylesheetsContent(vmStylesheets, stylesheetToken, vm));
+        }
+        return content;
     }
-    // VM (component) stylesheets apply after template stylesheets
-    if (hasStyles(vmStylesheets)) {
-        shared.ArrayPush.apply(content, evaluateStylesheetsContent(vmStylesheets, stylesheetToken, vm));
+    if (hasVmStyles) {
+        // No template styles, so return vm styles directly
+        return evaluateStylesheetsContent(vmStylesheets, stylesheetToken, vm);
     }
-    return content;
+    // Fastest path - no styles, so return an empty array
+    return EmptyArray;
 }
 // It might be worth caching this to avoid doing the lookup repeatedly, but
 // perf testing has not shown it to be a huge improvement yet:
@@ -5345,14 +5352,14 @@ function i(iterable, factory) {
     // TODO [#1276]: compiler should give us some sort of indicator when a vnodes collection is dynamic
     sc(list);
     const vmBeingRendered = getVMBeingRendered();
-    if (shared.isUndefined(iterable) || iterable === null) {
+    if (shared.isUndefined(iterable) || shared.isNull(iterable)) {
         if (process.env.NODE_ENV !== 'production') {
-            logError(`Invalid template iteration for value "${shared.toString(iterable)}" in ${vmBeingRendered}. It must be an Array or an iterable Object.`, vmBeingRendered);
+            logError(`Invalid template iteration for value \`${shared.toString(iterable)}\` in ${vmBeingRendered}. It must be an array-like object.`, vmBeingRendered);
         }
         return list;
     }
     if (process.env.NODE_ENV !== 'production') {
-        shared.assert.isFalse(shared.isUndefined(iterable[SymbolIterator]), `Invalid template iteration for value \`${shared.toString(iterable)}\` in ${vmBeingRendered}. It must be an array-like object and not \`null\` nor \`undefined\`.`);
+        shared.assert.isFalse(shared.isUndefined(iterable[SymbolIterator]), `Invalid template iteration for value \`${shared.toString(iterable)}\` in ${vmBeingRendered}. It must be an array-like object.`);
     }
     const iterator = iterable[SymbolIterator]();
     if (process.env.NODE_ENV !== 'production') {
@@ -5489,9 +5496,6 @@ function k(compilerKey, obj) {
 function gid(id) {
     const vmBeingRendered = getVMBeingRendered();
     if (shared.isUndefined(id) || id === '') {
-        if (process.env.NODE_ENV !== 'production') {
-            logError(`Invalid id value "${id}". The id attribute must contain a non-empty string.`, vmBeingRendered);
-        }
         return id;
     }
     // We remove attributes when they are assigned a value of null
@@ -5508,11 +5512,6 @@ function gid(id) {
 function fid(url) {
     const vmBeingRendered = getVMBeingRendered();
     if (shared.isUndefined(url) || url === '') {
-        if (process.env.NODE_ENV !== 'production') {
-            if (shared.isUndefined(url)) {
-                logError(`Undefined url value for "href" or "xlink:href" attribute. Expected a non-empty string.`, vmBeingRendered);
-            }
-        }
         return url;
     }
     // We remove attributes when they are assigned a value of null
@@ -6552,6 +6551,8 @@ function computeShadowMode(def, owner, renderer, hydrated) {
     // on, but components running in actual native shadow mode
     (process.env.NODE_ENV === 'test-karma-lwc' &&
         process.env.FORCE_NATIVE_SHADOW_MODE_FOR_TEST) ||
+        // If synthetic shadow is explicitly disabled, use pure-native
+        lwcRuntimeFlags.DISABLE_SYNTHETIC_SHADOW ||
         // hydration only supports native shadow
         shared.isTrue(hydrated)) {
         return 0 /* ShadowMode.Native */;
@@ -8091,5 +8092,5 @@ exports.swapTemplate = swapTemplate;
 exports.track = track;
 exports.unwrap = unwrap;
 exports.wire = wire;
-/** version: 7.1.2 */
+/** version: 7.2.1 */
 //# sourceMappingURL=index.cjs.js.map