@reckona/mreact-compat 0.0.153 → 0.0.155

package/dist/host-reconciler.js

@@ -1,4 +1,4 @@
-import { Activity, ERROR_BOUNDARY_TYPE, FORWARD_REF_TYPE, Fragment, HOST_CHILDREN_ONLY_PROPS_META, HOST_OWN_PROPS_META, LAZY_TYPE, MEMO_TYPE, Profiler, REACTIVE_TEXT_BINDING_META, STRICT_MODE_TYPE, Suspense, SuspenseList, isReactCompatElement, isReactCompatPortal, } from "./element.js";
+import { Activity, ERROR_BOUNDARY_TYPE, FORWARD_REF_TYPE, Fragment, HOST_CHILDREN_ONLY_PROPS_META, LAZY_TYPE, MEMO_TYPE, Profiler, REACTIVE_TEXT_BINDING_META, STRICT_MODE_TYPE, Suspense, SuspenseList, isReactCompatElement, isReactCompatPortal, } from "./element.js";
 import { consumerContext, isReactCompatConsumer, isReactCompatProvider, renderWithContextProvider, useContext, } from "./context.js";
 import { applyPostChildFormProps, applyProps } from "./dom-props.js";
 import { syncChildNodes, syncOwnedChildNodes, syncScopedChildNodes } from "./dom-children.js";
@@ -346,8 +346,8 @@ function getReusableKeyedRowHostFiber(current, row) {
         return undefined;
     }
     const previousRecord = previousProps;
-    if (getHostOwnPropsMeta(previousRecord) !== row.meta ||
-        getDirectHostTextChild(previousRecord.children) !== row.text) {
+    if (getDirectHostTextChild(previousRecord.children) !== row.text ||
+        !hostOwnPropsEqual(previousRecord, row.element.props)) {
         return undefined;
     }
     current.pendingProps = row.element.props;
@@ -364,7 +364,6 @@ function createKeyedRowHostElementScratch() {
         element: undefined,
         key: "",
         type: "",
-        meta: 0,
         text: "",
     };
 }
@@ -375,16 +374,15 @@ function readKeyedRowHostElement(node, row) {
         node.ref !== null) {
         return false;
     }
-    const props = node.props;
-    const meta = getHostOwnPropsMeta(props);
-    const text = meta === undefined ? undefined : getDirectHostTextChild(props.children);
-    if (meta === undefined || text === undefined) {
+    // Any keyed host row whose children collapse to a single text value
+    // qualifies; row props are compared per reuse with hostOwnPropsEqual.
+    const text = getDirectHostTextChild(node.props.children);
+    if (text === undefined) {
         return false;
     }
     row.element = node;
     row.key = node.key;
     row.type = node.type;
-    row.meta = meta;
     row.text = text;
     return true;
 }
@@ -412,11 +410,8 @@ function createKeyedRowHostFiber(parent, current, row, options) {
         return fiber;
     }
     const previousProps = current.memoizedProps ?? current.pendingProps;
-    const previousMeta = typeof previousProps === "object" && previousProps !== null
-        ? getHostOwnPropsMeta(previousProps)
-        : undefined;
     const previousText = getDirectHostTextChild(hostFiberChildrenProp(previousProps));
-    if (previousMeta !== row.meta || previousText !== row.text) {
+    if (previousText !== row.text || !hostOwnPropsEqual(previousProps, row.element.props)) {
         fiber.flags |= Update;
     }
     return fiber;
@@ -1015,7 +1010,9 @@ function commitHostDirtyFiber(fiber, parent, eventRoot, path, options = {}) {
             (fiber.hydrateExisting !== true &&
                 hostPropsAreChildrenOnly(fiber.memoizedProps) &&
                 hostPropsAreChildrenOnly(props));
-        const textOnlyRowUpdate = fiber.hydrateExisting !== true &&
+        const textOnlyRowUpdate = !propsAreUnchanged &&
+            !propsAreChildrenOnly &&
+            fiber.hydrateExisting !== true &&
             isRowTextOnlyUpdate(fiber.memoizedProps, props);
         if (!propsAreUnchanged && !propsAreChildrenOnly && !textOnlyRowUpdate) {
             applyProps(element, props, path, {
@@ -1287,7 +1284,9 @@ function commitHostFiber(fiber, parent, eventRoot, path, options = {}) {
             (fiber.hydrateExisting !== true &&
                 hostPropsAreChildrenOnly(fiber.memoizedProps) &&
                 hostPropsAreChildrenOnly(props));
-        const textOnlyRowUpdate = fiber.hydrateExisting !== true &&
+        const textOnlyRowUpdate = !propsAreUnchanged &&
+            !propsAreChildrenOnly &&
+            fiber.hydrateExisting !== true &&
             isRowTextOnlyUpdate(fiber.memoizedProps, props);
         if (!propsAreUnchanged && !propsAreChildrenOnly && !textOnlyRowUpdate) {
             applyProps(element, props, path, {
@@ -1459,11 +1458,6 @@ function hostOwnPropsEqual(previous, next) {
         return false;
     }
     const previousProps = previous;
-    const previousMeta = getHostOwnPropsMeta(previousProps);
-    const nextMeta = getHostOwnPropsMeta(next);
-    if (previousMeta !== undefined && nextMeta !== undefined) {
-        return previousMeta === nextMeta;
-    }
     let previousCount = 0;
     let nextCount = 0;
     for (const key in previousProps) {
@@ -1485,9 +1479,6 @@ function hostOwnPropsEqual(previous, next) {
     }
     return previousCount === nextCount;
 }
-function getHostOwnPropsMeta(props) {
-    return props[HOST_OWN_PROPS_META];
-}
 function hostDirectTextChildChanged(previous, next) {
     const previousText = getDirectHostTextChild(hostFiberChildrenProp(previous));
     const nextText = getDirectHostTextChild(next.children);
@@ -1556,14 +1547,12 @@ function isRowTextOnlyUpdate(previous, next) {
         return false;
     }
     const previousProps = previous;
-    const previousMeta = getHostOwnPropsMeta(previousProps);
-    const nextMeta = getHostOwnPropsMeta(next);
-    if (previousMeta === undefined || previousMeta !== nextMeta) {
-        return false;
-    }
     const previousText = getDirectHostTextChild(previousProps.children);
     const nextText = getDirectHostTextChild(next.children);
-    return previousText !== undefined && nextText !== undefined && previousText !== nextText;
+    return (previousText !== undefined &&
+        nextText !== undefined &&
+        previousText !== nextText &&
+        hostOwnPropsEqual(previousProps, next));
 }
 function hostFiberChildrenProp(props) {
     return typeof props === "object" && props !== null
@@ -1575,10 +1564,26 @@ function getDirectHostTextChild(children) {
         ? String(children)
         : undefined;
 }
+const hostFastPathMode = (() => {
+    try {
+        // The literal process.env.NODE_ENV member expression is what bundler
+        // define rewriting matches; a globalThis.process indirection is never
+        // rewritten and leaves deployed browser bundles without any fast path.
+        return process.env.NODE_ENV === "production" ? "static-fast" : "dynamic";
+    }
+    catch {
+        // No process global at all: an unbundled browser runtime. Treat it as
+        // production rather than running every host update on the slow path.
+        return "static-fast";
+    }
+})();
 function shouldUseDirectHostTextChild() {
-    const globalProcess = globalThis
-        .process;
-    return globalProcess?.env?.NODE_ENV === "production";
+    if (hostFastPathMode === "static-fast") {
+        return true;
+    }
+    // Node dev/test environments keep the per-call env read so test harnesses
+    // can flip NODE_ENV (vi.stubEnv) without re-importing this module.
+    return process.env.NODE_ENV === "production";
 }
 function syncDirectHostTextChild(element, text) {
     const firstChild = element.firstChild;