@lwc/engine-core 2.5.8 → 2.5.11

package/dist/engine-core.cjs.js

@@ -1431,7 +1431,6 @@ const {
   getPrototypeOf,
   create: ObjectCreate,
   defineProperty: ObjectDefineProperty,
-  defineProperties: ObjectDefineProperties,
   isExtensible,
   getOwnPropertyDescriptor,
   getOwnPropertyNames,
@@ -1441,8 +1440,7 @@ const {
 } = Object;
 const {
   push: ArrayPush,
-  concat: ArrayConcat,
-  map: ArrayMap
+  concat: ArrayConcat
 } = Array.prototype;
 const OtS = {}.toString;
 
@@ -1507,7 +1505,9 @@ class BaseProxyHandler {
     //       but it will always be compatible with the previous descriptor
     //       to preserve the object invariants, which makes these lines safe.
 
-    const originalDescriptor = getOwnPropertyDescriptor(originalTarget, key);
+    const originalDescriptor = getOwnPropertyDescriptor(originalTarget, key); // TODO: it should be impossible for the originalDescriptor to ever be undefined, this `if` can be removed
+
+    /* istanbul ignore else */
 
     if (!isUndefined(originalDescriptor)) {
       const wrappedDesc = this.wrapDescriptor(originalDescriptor);
@@ -1535,11 +1535,17 @@ class BaseProxyHandler {
 
     preventExtensions(shadowTarget);
   } // Shared Traps
+  // TODO: apply() is never called
+
+  /* istanbul ignore next */
 
 
   apply(shadowTarget, thisArg, argArray) {
     /* No op */
-  }
+  } // TODO: construct() is never called
+
+  /* istanbul ignore next */
+
 
   construct(shadowTarget, argArray, newTarget) {
     /* No op */
@@ -1652,8 +1658,8 @@ class BaseProxyHandler {
 
 }
 
-const getterMap = new WeakMap();
-const setterMap = new WeakMap();
+const getterMap$1 = new WeakMap();
+const setterMap$1 = new WeakMap();
 const reverseGetterMap = new WeakMap();
 const reverseSetterMap = new WeakMap();
 
@@ -1663,7 +1669,7 @@ class ReactiveProxyHandler extends BaseProxyHandler {
   }
 
   wrapGetter(originalGet) {
-    const wrappedGetter = getterMap.get(originalGet);
+    const wrappedGetter = getterMap$1.get(originalGet);
 
     if (!isUndefined(wrappedGetter)) {
       return wrappedGetter;
@@ -1676,13 +1682,13 @@ class ReactiveProxyHandler extends BaseProxyHandler {
       return handler.wrapValue(originalGet.call(unwrap$1(this)));
     };
 
-    getterMap.set(originalGet, get);
+    getterMap$1.set(originalGet, get);
     reverseGetterMap.set(get, originalGet);
     return get;
   }
 
   wrapSetter(originalSet) {
-    const wrappedSetter = setterMap.get(originalSet);
+    const wrappedSetter = setterMap$1.get(originalSet);
 
     if (!isUndefined(wrappedSetter)) {
       return wrappedSetter;
@@ -1693,7 +1699,7 @@ class ReactiveProxyHandler extends BaseProxyHandler {
       originalSet.call(unwrap$1(this), unwrap$1(v));
     };
 
-    setterMap.set(originalSet, set);
+    setterMap$1.set(originalSet, set);
     reverseSetterMap.set(set, originalSet);
     return set;
   }
@@ -1734,7 +1740,7 @@ class ReactiveProxyHandler extends BaseProxyHandler {
       return unwrap$1(redGet.call(handler.wrapValue(this)));
     };
 
-    getterMap.set(get, redGet);
+    getterMap$1.set(get, redGet);
     reverseGetterMap.set(redGet, get);
     return get;
   }
@@ -1753,7 +1759,7 @@ class ReactiveProxyHandler extends BaseProxyHandler {
       redSet.call(handler.wrapValue(this), handler.wrapValue(v));
     };
 
-    setterMap.set(set, redSet);
+    setterMap$1.set(set, redSet);
     reverseSetterMap.set(redSet, set);
     return set;
   }
@@ -1794,6 +1800,7 @@ class ReactiveProxyHandler extends BaseProxyHandler {
   }
 
   setPrototypeOf(shadowTarget, prototype) {
+    /* istanbul ignore else */
     if (process.env.NODE_ENV !== 'production') {
       throw new Error(`Invalid setPrototypeOf invocation for reactive proxy ${toString(this.originalTarget)}. Prototype of reactive objects cannot be changed.`);
     }
@@ -1807,6 +1814,11 @@ class ReactiveProxyHandler extends BaseProxyHandler {
       preventExtensions(originalTarget); // if the originalTarget is a proxy itself, it might reject
       // the preventExtension call, in which case we should not attempt to lock down
       // the shadow target.
+      // TODO: It should not actually be possible to reach this `if` statement.
+      // If a proxy rejects extensions, then calling preventExtensions will throw an error:
+      // https://codepen.io/nolanlawson-the-selector/pen/QWMOjbY
+
+      /* istanbul ignore if */
 
       if (isExtensible(originalTarget)) {
         return false;
@@ -1848,8 +1860,8 @@ class ReactiveProxyHandler extends BaseProxyHandler {
 
 }
 
-const getterMap$1 = new WeakMap();
-const setterMap$1 = new WeakMap();
+const getterMap = new WeakMap();
+const setterMap = new WeakMap();
 
 class ReadOnlyHandler extends BaseProxyHandler {
   wrapValue(value) {
@@ -1857,7 +1869,7 @@ class ReadOnlyHandler extends BaseProxyHandler {
   }
 
   wrapGetter(originalGet) {
-    const wrappedGetter = getterMap$1.get(originalGet);
+    const wrappedGetter = getterMap.get(originalGet);
 
     if (!isUndefined(wrappedGetter)) {
       return wrappedGetter;
@@ -1870,12 +1882,12 @@ class ReadOnlyHandler extends BaseProxyHandler {
       return handler.wrapValue(originalGet.call(unwrap$1(this)));
     };
 
-    getterMap$1.set(originalGet, get);
+    getterMap.set(originalGet, get);
     return get;
   }
 
   wrapSetter(originalSet) {
-    const wrappedSetter = setterMap$1.get(originalSet);
+    const wrappedSetter = setterMap.get(originalSet);
 
     if (!isUndefined(wrappedSetter)) {
       return wrappedSetter;
@@ -1884,6 +1896,7 @@ class ReadOnlyHandler extends BaseProxyHandler {
     const handler = this;
 
     const set = function (v) {
+      /* istanbul ignore else */
       if (process.env.NODE_ENV !== 'production') {
         const {
           originalTarget
@@ -1892,33 +1905,41 @@ class ReadOnlyHandler extends BaseProxyHandler {
       }
     };
 
-    setterMap$1.set(originalSet, set);
+    setterMap.set(originalSet, set);
     return set;
   }
 
   set(shadowTarget, key, value) {
+    /* istanbul ignore else */
     if (process.env.NODE_ENV !== 'production') {
       const {
         originalTarget
       } = this;
-      throw new Error(`Invalid mutation: Cannot set "${key.toString()}" on "${originalTarget}". "${originalTarget}" is read-only.`);
+      const msg = isArray(originalTarget) ? `Invalid mutation: Cannot mutate array at index ${key.toString()}. Array is read-only.` : `Invalid mutation: Cannot set "${key.toString()}" on "${originalTarget}". "${originalTarget}" is read-only.`;
+      throw new Error(msg);
     }
+    /* istanbul ignore next */
+
 
     return false;
   }
 
   deleteProperty(shadowTarget, key) {
+    /* istanbul ignore else */
     if (process.env.NODE_ENV !== 'production') {
       const {
         originalTarget
       } = this;
       throw new Error(`Invalid mutation: Cannot delete "${key.toString()}" on "${originalTarget}". "${originalTarget}" is read-only.`);
     }
+    /* istanbul ignore next */
+
 
     return false;
   }
 
   setPrototypeOf(shadowTarget, prototype) {
+    /* istanbul ignore else */
     if (process.env.NODE_ENV !== 'production') {
       const {
         originalTarget
@@ -1928,23 +1949,29 @@ class ReadOnlyHandler extends BaseProxyHandler {
   }
 
   preventExtensions(shadowTarget) {
+    /* istanbul ignore else */
     if (process.env.NODE_ENV !== 'production') {
       const {
         originalTarget
       } = this;
       throw new Error(`Invalid mutation: Cannot preventExtensions on ${originalTarget}". "${originalTarget} is read-only.`);
     }
+    /* istanbul ignore next */
+
 
     return false;
   }
 
   defineProperty(shadowTarget, key, descriptor) {
+    /* istanbul ignore else */
     if (process.env.NODE_ENV !== 'production') {
       const {
         originalTarget
       } = this;
       throw new Error(`Invalid mutation: Cannot defineProperty "${key.toString()}" on "${originalTarget}". "${originalTarget}" is read-only.`);
     }
+    /* istanbul ignore next */
+
 
     return false;
   }
@@ -2002,6 +2029,8 @@ const formatter = {
 }; // Inspired from paulmillr/es6-shim
 // https://github.com/paulmillr/es6-shim/blob/master/es6-shim.js#L176-L185
 
+/* istanbul ignore next */
+
 function getGlobal() {
   // the only reliable means to get the global object is `Function('return this')()`
   // However, this causes CSP violations in Chrome apps.
@@ -2026,6 +2055,7 @@ function getGlobal() {
 }
 
 function init() {
+  /* istanbul ignore if */
   if (process.env.NODE_ENV === 'production') {
     // this method should never leak to prod
     throw new ReferenceError();
@@ -2040,6 +2070,8 @@ function init() {
   ArrayPush.call(devtoolsFormatters, formatter);
   global.devtoolsFormatters = devtoolsFormatters;
 }
+/* istanbul ignore else */
+
 
 if (process.env.NODE_ENV !== 'production') {
   init();
@@ -2086,7 +2118,8 @@ class ReactiveMembrane {
     this.valueMutated = defaultValueMutated;
     this.valueObserved = defaultValueObserved;
     this.valueIsObservable = defaultValueIsObservable;
-    this.objectGraph = new WeakMap();
+    this.readOnlyObjectGraph = new WeakMap();
+    this.reactiveObjectGraph = new WeakMap();
 
     if (!isUndefined(options)) {
       const {
@@ -2109,10 +2142,13 @@ class ReactiveMembrane {
     const distorted = this.valueDistortion(unwrappedValue);
 
     if (this.valueIsObservable(distorted)) {
-      const o = this.getReactiveState(unwrappedValue, distorted); // when trying to extract the writable version of a readonly
-      // we return the readonly.
+      if (this.readOnlyObjectGraph.get(distorted) === value) {
+        // when trying to extract the writable version of a readonly
+        // we return the readonly.
+        return value;
+      }
 
-      return o.readOnly === value ? value : o.reactive;
+      return this.getReactiveHandler(unwrappedValue, distorted);
     }
 
     return distorted;
@@ -2123,7 +2159,7 @@ class ReactiveMembrane {
     const distorted = this.valueDistortion(value);
 
     if (this.valueIsObservable(distorted)) {
-      return this.getReactiveState(value, distorted).readOnly;
+      return this.getReadOnlyHandler(value, distorted);
     }
 
     return distorted;
@@ -2133,47 +2169,36 @@ class ReactiveMembrane {
     return unwrap$1(p);
   }
 
-  getReactiveState(value, distortedValue) {
-    const {
-      objectGraph
-    } = this;
-    let reactiveState = objectGraph.get(distortedValue);
+  getReactiveHandler(value, distortedValue) {
+    let proxy = this.reactiveObjectGraph.get(distortedValue);
 
-    if (reactiveState) {
-      return reactiveState;
+    if (isUndefined(proxy)) {
+      // caching the proxy after the first time it is accessed
+      const handler = new ReactiveProxyHandler(this, distortedValue);
+      proxy = new Proxy(createShadowTarget(distortedValue), handler);
+      registerProxy(proxy, value);
+      this.reactiveObjectGraph.set(distortedValue, proxy);
     }
 
-    const membrane = this;
-    reactiveState = {
-      get reactive() {
-        const reactiveHandler = new ReactiveProxyHandler(membrane, distortedValue); // caching the reactive proxy after the first time it is accessed
-
-        const proxy = new Proxy(createShadowTarget(distortedValue), reactiveHandler);
-        registerProxy(proxy, value);
-        ObjectDefineProperty(this, 'reactive', {
-          value: proxy
-        });
-        return proxy;
-      },
+    return proxy;
+  }
 
-      get readOnly() {
-        const readOnlyHandler = new ReadOnlyHandler(membrane, distortedValue); // caching the readOnly proxy after the first time it is accessed
+  getReadOnlyHandler(value, distortedValue) {
+    let proxy = this.readOnlyObjectGraph.get(distortedValue);
 
-        const proxy = new Proxy(createShadowTarget(distortedValue), readOnlyHandler);
-        registerProxy(proxy, value);
-        ObjectDefineProperty(this, 'readOnly', {
-          value: proxy
-        });
-        return proxy;
-      }
+    if (isUndefined(proxy)) {
+      // caching the proxy after the first time it is accessed
+      const handler = new ReadOnlyHandler(this, distortedValue);
+      proxy = new Proxy(createShadowTarget(distortedValue), handler);
+      registerProxy(proxy, value);
+      this.readOnlyObjectGraph.set(distortedValue, proxy);
+    }
 
-    };
-    objectGraph.set(distortedValue, reactiveState);
-    return reactiveState;
+    return proxy;
   }
 
 }
-/** version: 1.0.0 */
+/** version: 1.1.5 */
 
 /*
  * Copyright (c) 2018, salesforce.com, inc.
@@ -4283,7 +4308,7 @@ function vnodesAndElementHaveCompatibleAttrs(vnode, elm) {
   for (const [attrName, attrValue] of Object.entries(attrs)) {
     const elmAttrValue = renderer.getAttribute(elm, attrName);
 
-    if (attrValue !== elmAttrValue) {
+    if (String(attrValue) !== elmAttrValue) {
       logError(`Mismatch hydrating element <${elm.tagName.toLowerCase()}>: attribute "${attrName}" has different values, expected "${attrValue}" but found "${elmAttrValue}"`, vnode.owner);
       nodesAreCompatible = false;
     }
@@ -4305,7 +4330,7 @@ function vnodesAndElementHaveCompatibleClass(vnode, elm) {
   let nodesAreCompatible = true;
   let vnodeClassName;
 
-  if (!shared.isUndefined(className) && className !== elm.className) {
+  if (!shared.isUndefined(className) && String(className) !== elm.className) {
     // className is used when class is bound to an expr.
     nodesAreCompatible = false;
     vnodeClassName = className;
@@ -5100,7 +5125,7 @@ function co(text) {
 
 
 function d(value) {
-  return value == null ? '' : value;
+  return value == null ? '' : String(value);
 } // [b]ind function
 
 
@@ -5475,7 +5500,7 @@ function createStylesheet(vm, stylesheets) {
     for (let i = 0; i < stylesheets.length; i++) {
       renderer.insertGlobalStylesheet(stylesheets[i]);
     }
-  } else if (renderer.ssr || renderer.isHydrating) {
+  } else if (renderer.ssr || renderer.isHydrating()) {
     // Note: We need to ensure that during hydration, the stylesheets method is the same as those in ssr.
     //       This works in the client, because the stylesheets are created, and cached in the VM
     //       the first time the VM renders.
@@ -7185,4 +7210,4 @@ exports.swapTemplate = swapTemplate;
 exports.track = track;
 exports.unwrap = unwrap;
 exports.wire = wire;
-/* version: 2.5.8 */
+/* version: 2.5.11 */

package/dist/engine-core.js

@@ -1428,7 +1428,6 @@ const {
   getPrototypeOf,
   create: ObjectCreate,
   defineProperty: ObjectDefineProperty,
-  defineProperties: ObjectDefineProperties,
   isExtensible,
   getOwnPropertyDescriptor,
   getOwnPropertyNames,
@@ -1438,8 +1437,7 @@ const {
 } = Object;
 const {
   push: ArrayPush,
-  concat: ArrayConcat,
-  map: ArrayMap
+  concat: ArrayConcat
 } = Array.prototype;
 const OtS = {}.toString;
 
@@ -1504,7 +1502,9 @@ class BaseProxyHandler {
     //       but it will always be compatible with the previous descriptor
     //       to preserve the object invariants, which makes these lines safe.
 
-    const originalDescriptor = getOwnPropertyDescriptor(originalTarget, key);
+    const originalDescriptor = getOwnPropertyDescriptor(originalTarget, key); // TODO: it should be impossible for the originalDescriptor to ever be undefined, this `if` can be removed
+
+    /* istanbul ignore else */
 
     if (!isUndefined(originalDescriptor)) {
       const wrappedDesc = this.wrapDescriptor(originalDescriptor);
@@ -1532,11 +1532,17 @@ class BaseProxyHandler {
 
     preventExtensions(shadowTarget);
   } // Shared Traps
+  // TODO: apply() is never called
+
+  /* istanbul ignore next */
 
 
   apply(shadowTarget, thisArg, argArray) {
     /* No op */
-  }
+  } // TODO: construct() is never called
+
+  /* istanbul ignore next */
+
 
   construct(shadowTarget, argArray, newTarget) {
     /* No op */
@@ -1649,8 +1655,8 @@ class BaseProxyHandler {
 
 }
 
-const getterMap = new WeakMap();
-const setterMap = new WeakMap();
+const getterMap$1 = new WeakMap();
+const setterMap$1 = new WeakMap();
 const reverseGetterMap = new WeakMap();
 const reverseSetterMap = new WeakMap();
 
@@ -1660,7 +1666,7 @@ class ReactiveProxyHandler extends BaseProxyHandler {
   }
 
   wrapGetter(originalGet) {
-    const wrappedGetter = getterMap.get(originalGet);
+    const wrappedGetter = getterMap$1.get(originalGet);
 
     if (!isUndefined(wrappedGetter)) {
       return wrappedGetter;
@@ -1673,13 +1679,13 @@ class ReactiveProxyHandler extends BaseProxyHandler {
       return handler.wrapValue(originalGet.call(unwrap$1(this)));
     };
 
-    getterMap.set(originalGet, get);
+    getterMap$1.set(originalGet, get);
     reverseGetterMap.set(get, originalGet);
     return get;
   }
 
   wrapSetter(originalSet) {
-    const wrappedSetter = setterMap.get(originalSet);
+    const wrappedSetter = setterMap$1.get(originalSet);
 
     if (!isUndefined(wrappedSetter)) {
       return wrappedSetter;
@@ -1690,7 +1696,7 @@ class ReactiveProxyHandler extends BaseProxyHandler {
       originalSet.call(unwrap$1(this), unwrap$1(v));
     };
 
-    setterMap.set(originalSet, set);
+    setterMap$1.set(originalSet, set);
     reverseSetterMap.set(set, originalSet);
     return set;
   }
@@ -1731,7 +1737,7 @@ class ReactiveProxyHandler extends BaseProxyHandler {
       return unwrap$1(redGet.call(handler.wrapValue(this)));
     };
 
-    getterMap.set(get, redGet);
+    getterMap$1.set(get, redGet);
     reverseGetterMap.set(redGet, get);
     return get;
   }
@@ -1750,7 +1756,7 @@ class ReactiveProxyHandler extends BaseProxyHandler {
       redSet.call(handler.wrapValue(this), handler.wrapValue(v));
     };
 
-    setterMap.set(set, redSet);
+    setterMap$1.set(set, redSet);
     reverseSetterMap.set(redSet, set);
     return set;
   }
@@ -1791,6 +1797,7 @@ class ReactiveProxyHandler extends BaseProxyHandler {
   }
 
   setPrototypeOf(shadowTarget, prototype) {
+    /* istanbul ignore else */
     if (process.env.NODE_ENV !== 'production') {
       throw new Error(`Invalid setPrototypeOf invocation for reactive proxy ${toString(this.originalTarget)}. Prototype of reactive objects cannot be changed.`);
     }
@@ -1804,6 +1811,11 @@ class ReactiveProxyHandler extends BaseProxyHandler {
       preventExtensions(originalTarget); // if the originalTarget is a proxy itself, it might reject
       // the preventExtension call, in which case we should not attempt to lock down
       // the shadow target.
+      // TODO: It should not actually be possible to reach this `if` statement.
+      // If a proxy rejects extensions, then calling preventExtensions will throw an error:
+      // https://codepen.io/nolanlawson-the-selector/pen/QWMOjbY
+
+      /* istanbul ignore if */
 
       if (isExtensible(originalTarget)) {
         return false;
@@ -1845,8 +1857,8 @@ class ReactiveProxyHandler extends BaseProxyHandler {
 
 }
 
-const getterMap$1 = new WeakMap();
-const setterMap$1 = new WeakMap();
+const getterMap = new WeakMap();
+const setterMap = new WeakMap();
 
 class ReadOnlyHandler extends BaseProxyHandler {
   wrapValue(value) {
@@ -1854,7 +1866,7 @@ class ReadOnlyHandler extends BaseProxyHandler {
   }
 
   wrapGetter(originalGet) {
-    const wrappedGetter = getterMap$1.get(originalGet);
+    const wrappedGetter = getterMap.get(originalGet);
 
     if (!isUndefined(wrappedGetter)) {
       return wrappedGetter;
@@ -1867,12 +1879,12 @@ class ReadOnlyHandler extends BaseProxyHandler {
       return handler.wrapValue(originalGet.call(unwrap$1(this)));
     };
 
-    getterMap$1.set(originalGet, get);
+    getterMap.set(originalGet, get);
     return get;
   }
 
   wrapSetter(originalSet) {
-    const wrappedSetter = setterMap$1.get(originalSet);
+    const wrappedSetter = setterMap.get(originalSet);
 
     if (!isUndefined(wrappedSetter)) {
       return wrappedSetter;
@@ -1881,6 +1893,7 @@ class ReadOnlyHandler extends BaseProxyHandler {
     const handler = this;
 
     const set = function (v) {
+      /* istanbul ignore else */
       if (process.env.NODE_ENV !== 'production') {
         const {
           originalTarget
@@ -1889,33 +1902,41 @@ class ReadOnlyHandler extends BaseProxyHandler {
       }
     };
 
-    setterMap$1.set(originalSet, set);
+    setterMap.set(originalSet, set);
     return set;
   }
 
   set(shadowTarget, key, value) {
+    /* istanbul ignore else */
     if (process.env.NODE_ENV !== 'production') {
       const {
         originalTarget
       } = this;
-      throw new Error(`Invalid mutation: Cannot set "${key.toString()}" on "${originalTarget}". "${originalTarget}" is read-only.`);
+      const msg = isArray(originalTarget) ? `Invalid mutation: Cannot mutate array at index ${key.toString()}. Array is read-only.` : `Invalid mutation: Cannot set "${key.toString()}" on "${originalTarget}". "${originalTarget}" is read-only.`;
+      throw new Error(msg);
     }
+    /* istanbul ignore next */
+
 
     return false;
   }
 
   deleteProperty(shadowTarget, key) {
+    /* istanbul ignore else */
     if (process.env.NODE_ENV !== 'production') {
       const {
         originalTarget
       } = this;
       throw new Error(`Invalid mutation: Cannot delete "${key.toString()}" on "${originalTarget}". "${originalTarget}" is read-only.`);
     }
+    /* istanbul ignore next */
+
 
     return false;
   }
 
   setPrototypeOf(shadowTarget, prototype) {
+    /* istanbul ignore else */
     if (process.env.NODE_ENV !== 'production') {
       const {
         originalTarget
@@ -1925,23 +1946,29 @@ class ReadOnlyHandler extends BaseProxyHandler {
   }
 
   preventExtensions(shadowTarget) {
+    /* istanbul ignore else */
     if (process.env.NODE_ENV !== 'production') {
       const {
         originalTarget
       } = this;
       throw new Error(`Invalid mutation: Cannot preventExtensions on ${originalTarget}". "${originalTarget} is read-only.`);
     }
+    /* istanbul ignore next */
+
 
     return false;
   }
 
   defineProperty(shadowTarget, key, descriptor) {
+    /* istanbul ignore else */
     if (process.env.NODE_ENV !== 'production') {
       const {
         originalTarget
       } = this;
       throw new Error(`Invalid mutation: Cannot defineProperty "${key.toString()}" on "${originalTarget}". "${originalTarget}" is read-only.`);
     }
+    /* istanbul ignore next */
+
 
     return false;
   }
@@ -1999,6 +2026,8 @@ const formatter = {
 }; // Inspired from paulmillr/es6-shim
 // https://github.com/paulmillr/es6-shim/blob/master/es6-shim.js#L176-L185
 
+/* istanbul ignore next */
+
 function getGlobal() {
   // the only reliable means to get the global object is `Function('return this')()`
   // However, this causes CSP violations in Chrome apps.
@@ -2023,6 +2052,7 @@ function getGlobal() {
 }
 
 function init() {
+  /* istanbul ignore if */
   if (process.env.NODE_ENV === 'production') {
     // this method should never leak to prod
     throw new ReferenceError();
@@ -2037,6 +2067,8 @@ function init() {
   ArrayPush.call(devtoolsFormatters, formatter);
   global.devtoolsFormatters = devtoolsFormatters;
 }
+/* istanbul ignore else */
+
 
 if (process.env.NODE_ENV !== 'production') {
   init();
@@ -2083,7 +2115,8 @@ class ReactiveMembrane {
     this.valueMutated = defaultValueMutated;
     this.valueObserved = defaultValueObserved;
     this.valueIsObservable = defaultValueIsObservable;
-    this.objectGraph = new WeakMap();
+    this.readOnlyObjectGraph = new WeakMap();
+    this.reactiveObjectGraph = new WeakMap();
 
     if (!isUndefined(options)) {
       const {
@@ -2106,10 +2139,13 @@ class ReactiveMembrane {
     const distorted = this.valueDistortion(unwrappedValue);
 
     if (this.valueIsObservable(distorted)) {
-      const o = this.getReactiveState(unwrappedValue, distorted); // when trying to extract the writable version of a readonly
-      // we return the readonly.
+      if (this.readOnlyObjectGraph.get(distorted) === value) {
+        // when trying to extract the writable version of a readonly
+        // we return the readonly.
+        return value;
+      }
 
-      return o.readOnly === value ? value : o.reactive;
+      return this.getReactiveHandler(unwrappedValue, distorted);
     }
 
     return distorted;
@@ -2120,7 +2156,7 @@ class ReactiveMembrane {
     const distorted = this.valueDistortion(value);
 
     if (this.valueIsObservable(distorted)) {
-      return this.getReactiveState(value, distorted).readOnly;
+      return this.getReadOnlyHandler(value, distorted);
     }
 
     return distorted;
@@ -2130,47 +2166,36 @@ class ReactiveMembrane {
     return unwrap$1(p);
   }
 
-  getReactiveState(value, distortedValue) {
-    const {
-      objectGraph
-    } = this;
-    let reactiveState = objectGraph.get(distortedValue);
+  getReactiveHandler(value, distortedValue) {
+    let proxy = this.reactiveObjectGraph.get(distortedValue);
 
-    if (reactiveState) {
-      return reactiveState;
+    if (isUndefined(proxy)) {
+      // caching the proxy after the first time it is accessed
+      const handler = new ReactiveProxyHandler(this, distortedValue);
+      proxy = new Proxy(createShadowTarget(distortedValue), handler);
+      registerProxy(proxy, value);
+      this.reactiveObjectGraph.set(distortedValue, proxy);
     }
 
-    const membrane = this;
-    reactiveState = {
-      get reactive() {
-        const reactiveHandler = new ReactiveProxyHandler(membrane, distortedValue); // caching the reactive proxy after the first time it is accessed
-
-        const proxy = new Proxy(createShadowTarget(distortedValue), reactiveHandler);
-        registerProxy(proxy, value);
-        ObjectDefineProperty(this, 'reactive', {
-          value: proxy
-        });
-        return proxy;
-      },
+    return proxy;
+  }
 
-      get readOnly() {
-        const readOnlyHandler = new ReadOnlyHandler(membrane, distortedValue); // caching the readOnly proxy after the first time it is accessed
+  getReadOnlyHandler(value, distortedValue) {
+    let proxy = this.readOnlyObjectGraph.get(distortedValue);
 
-        const proxy = new Proxy(createShadowTarget(distortedValue), readOnlyHandler);
-        registerProxy(proxy, value);
-        ObjectDefineProperty(this, 'readOnly', {
-          value: proxy
-        });
-        return proxy;
-      }
+    if (isUndefined(proxy)) {
+      // caching the proxy after the first time it is accessed
+      const handler = new ReadOnlyHandler(this, distortedValue);
+      proxy = new Proxy(createShadowTarget(distortedValue), handler);
+      registerProxy(proxy, value);
+      this.readOnlyObjectGraph.set(distortedValue, proxy);
+    }
 
-    };
-    objectGraph.set(distortedValue, reactiveState);
-    return reactiveState;
+    return proxy;
   }
 
 }
-/** version: 1.0.0 */
+/** version: 1.1.5 */
 
 /*
  * Copyright (c) 2018, salesforce.com, inc.
@@ -4280,7 +4305,7 @@ function vnodesAndElementHaveCompatibleAttrs(vnode, elm) {
   for (const [attrName, attrValue] of Object.entries(attrs)) {
     const elmAttrValue = renderer.getAttribute(elm, attrName);
 
-    if (attrValue !== elmAttrValue) {
+    if (String(attrValue) !== elmAttrValue) {
       logError(`Mismatch hydrating element <${elm.tagName.toLowerCase()}>: attribute "${attrName}" has different values, expected "${attrValue}" but found "${elmAttrValue}"`, vnode.owner);
       nodesAreCompatible = false;
     }
@@ -4302,7 +4327,7 @@ function vnodesAndElementHaveCompatibleClass(vnode, elm) {
   let nodesAreCompatible = true;
   let vnodeClassName;
 
-  if (!isUndefined$1(className) && className !== elm.className) {
+  if (!isUndefined$1(className) && String(className) !== elm.className) {
     // className is used when class is bound to an expr.
     nodesAreCompatible = false;
     vnodeClassName = className;
@@ -5097,7 +5122,7 @@ function co(text) {
 
 
 function d(value) {
-  return value == null ? '' : value;
+  return value == null ? '' : String(value);
 } // [b]ind function
 
 
@@ -5472,7 +5497,7 @@ function createStylesheet(vm, stylesheets) {
     for (let i = 0; i < stylesheets.length; i++) {
       renderer.insertGlobalStylesheet(stylesheets[i]);
     }
-  } else if (renderer.ssr || renderer.isHydrating) {
+  } else if (renderer.ssr || renderer.isHydrating()) {
     // Note: We need to ensure that during hydration, the stylesheets method is the same as those in ssr.
     //       This works in the client, because the stylesheets are created, and cached in the VM
     //       the first time the VM renders.
@@ -7149,4 +7174,4 @@ function setHooks(hooks) {
 }
 
 export { LightningElement, profilerControl as __unstable__ProfilerControl, api$1 as api, connectRootElement, createContextProvider, createVM, disconnectRootElement, getAssociatedVMIfPresent, getComponentDef, getComponentInternalDef, getUpgradableConstructor, hydrateRootElement, isComponentConstructor, readonly, register, registerComponent, registerDecorators, registerTemplate, sanitizeAttribute, setHooks, swapComponent, swapStyle, swapTemplate, track, unwrap, wire };
-/* version: 2.5.8 */
+/* version: 2.5.11 */

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@lwc/engine-core",
-    "version": "2.5.8",
+    "version": "2.5.11",
     "description": "Core LWC engine APIs.",
     "homepage": "https://lwc.dev/",
     "repository": {
@@ -24,14 +24,14 @@
         "types/"
     ],
     "dependencies": {
-        "@lwc/features": "2.5.8",
-        "@lwc/shared": "2.5.8"
+        "@lwc/features": "2.5.11",
+        "@lwc/shared": "2.5.11"
     },
     "devDependencies": {
-        "observable-membrane": "1.0.1"
+        "observable-membrane": "1.1.5"
     },
     "publishConfig": {
         "access": "public"
     },
-    "gitHead": "223513a6a4e070b3243db0a345024a184f82dc08"
+    "gitHead": "2e72bcacbf105ef92277372bb4e52f8280d7fa72"
 }

package/types/framework/api.d.ts

@@ -12,7 +12,7 @@ declare function i(iterable: Iterable<any>, factory: (value: any, index: number,
 declare function f(items: any[]): any[];
 declare function t(text: string): VText;
 declare function co(text: string): VComment;
-declare function d(value: any): string | any;
+declare function d(value: any): string;
 declare function b(fn: EventListener): EventListener;
 declare function k(compilerKey: number, obj: any): string | void;
 declare function gid(id: string | undefined | null): string | null | undefined;

package/types/framework/renderer.d.ts

@@ -2,7 +2,7 @@ export declare type HostNode = any;
 export declare type HostElement = any;
 export interface Renderer<N = HostNode, E = HostElement> {
     ssr: boolean;
-    readonly isHydrating: boolean;
+    isHydrating(): boolean;
     isNativeShadowDefined: boolean;
     isSyntheticShadowDefined: boolean;
     insert(node: N, parent: E, anchor: N | null): void;