@lwc/engine-core 6.6.7 → 6.7.0

package/dist/index.cjs.js

@@ -3453,12 +3453,25 @@ function swapTemplate(oldTpl, newTpl) {
 }
 function swapComponent(oldComponent, newComponent) {
     if (process.env.NODE_ENV !== 'production') {
-        if (isComponentConstructor(oldComponent) && isComponentConstructor(newComponent)) {
+        const isOldCtorAComponent = isComponentConstructor(oldComponent);
+        const isNewCtorAComponent = isComponentConstructor(newComponent);
+        if (isOldCtorAComponent && isNewCtorAComponent) {
             swappedComponentMap.set(oldComponent, newComponent);
             return rehydrateHotComponent(oldComponent);
         }
+        else if (isOldCtorAComponent === false && isNewCtorAComponent === true) {
+            throw new TypeError(`Invalid Component: Attempting to swap a non-component with a component`);
+        }
+        else if (isOldCtorAComponent === true && isNewCtorAComponent === false) {
+            throw new TypeError(`Invalid Component: Attempting to swap a component with a non-component`);
+        }
         else {
-            throw new TypeError(`Invalid Component`);
+            // The dev-server relies on the presence of registerComponent() as a way to determine a
+            // component module. However, the compiler cannot definitively add registerComponent()
+            // transformation only to a component constructor. Hence the dev-server may attempt to
+            // hot swap javascript modules that look like a component and should not cause the app
+            // to fail. To allow that, this api ignores such hot swap attempts.
+            return false;
         }
     }
     return false;
@@ -7636,6 +7649,7 @@ function haveCompatibleStaticParts(vnode, renderer) {
     if (shared.isUndefined(parts)) {
         return true;
     }
+    const shouldValidateAttr = (data, attrName) => attrName in data;
     // The validation here relies on 2 key invariants:
     // 1. It's never the case that `parts` is undefined on the server but defined on the client (or vice-versa)
     // 2. It's never the case that `parts` has one length on the server but another on the client
@@ -7647,8 +7661,15 @@ function haveCompatibleStaticParts(vnode, renderer) {
             }
             const { data } = part;
             const hasMatchingAttrs = validateAttrs(vnode, elm, data, renderer, () => true);
-            const hasMatchingStyleAttr = validateStyleAttr(vnode, elm, data, renderer);
-            const hasMatchingClass = validateClassAttr(vnode, elm, data, renderer);
+            // Explicitly skip hydration validation when static parts don't contain `style` or `className`.
+            // This means the style/class attributes are either static or don't exist on the element and
+            // cannot be affected by hydration.
+            const hasMatchingStyleAttr = shouldValidateAttr(data, 'style')
+                ? validateStyleAttr(vnode, elm, data, renderer)
+                : true;
+            const hasMatchingClass = shouldValidateAttr(data, 'className')
+                ? validateClassAttr(vnode, elm, data, renderer)
+                : true;
             if (shared.isFalse(hasMatchingAttrs && hasMatchingStyleAttr && hasMatchingClass)) {
                 return false;
             }
@@ -7977,5 +7998,5 @@ exports.swapTemplate = swapTemplate;
 exports.track = track;
 exports.unwrap = unwrap;
 exports.wire = wire;
-/** version: 6.6.7 */
+/** version: 6.7.0 */
 //# sourceMappingURL=index.cjs.js.map