@pack/react 4.3.3 → 4.3.4

package/dist/pack/render-sections.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"render-sections.d.ts","sourceRoot":"","sources":["../../src/pack/render-sections.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAgD,MAAM,OAAO,CAAC;AAMrE,UAAU,kBAAkB;IAC1B,sDAAsD;IACtD,OAAO,EAAE,GAAG,CAAC;CACd;AAuCD;;;GAGG;AACH,wBAAgB,aAAa,CAAC,EAAE,OAAO,EAAE,EAAE,kBAAkB,4BAG5D;AAGD,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,GAAG,EAAE,uCAYnD;AAUD,UAAU,mBAAmB;IAC3B,OAAO,CAAC,EAAE,GAAG,CAAC;CACf;AA8BD,wBAAgB,cAAc,CAAC,KAAK,EAAE,mBAAmB,4BAKxD;AAED,wBAAgB,WAAW,CAAC,KAAK,EAAE,mBAAmB,uCAOrD"}
+{"version":3,"file":"render-sections.d.ts","sourceRoot":"","sources":["../../src/pack/render-sections.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAwD,MAAM,OAAO,CAAC;AAM7E,UAAU,kBAAkB;IAC1B,sDAAsD;IACtD,OAAO,EAAE,GAAG,CAAC;CACd;AAuCD;;;GAGG;AACH,wBAAgB,aAAa,CAAC,EAAE,OAAO,EAAE,EAAE,kBAAkB,4BAG5D;AAGD,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,GAAG,EAAE,uCAYnD;AAUD,UAAU,mBAAmB;IAC3B,OAAO,CAAC,EAAE,GAAG,CAAC;CACf;AAkED,wBAAgB,cAAc,CAAC,KAAK,EAAE,mBAAmB,4BAKxD;AAED,wBAAgB,WAAW,CAAC,KAAK,EAAE,mBAAmB,uCAOrD"}

package/dist/pack/render-sections.js

@@ -1,4 +1,4 @@
-import React, { useMemo, useEffect } from "react";
+import React, { useMemo, useEffect, useRef } from "react";
 import { usePackContext } from "./pack-context";
 import { sectionMap } from "../register-section";
 import { generateSectionStylesheet } from "../lib/style-utils";
@@ -57,12 +57,43 @@ function Sections({ sections }) {
 }
 function useRenderSections({ content }) {
     const { isPreview, liveContent, setLiveContent } = usePackContext();
-    // Update content in context for preview mode
+    // The loader `content` object we last pushed into liveContent. While
+    // liveContent is still that object, nothing has been received from the
+    // customizer for this page, so keep following loader data (navigation and
+    // ordinary preview links stay fresh). Once the customizer replaces it via
+    // SET_PAGE_DATA, a loader update for the *same page* must not win: any
+    // React Router action (e.g. a fetcher POST) revalidates every loader and
+    // hands us a fresh `content` object holding the persisted page, which used
+    // to silently overwrite unsaved customizer changes — a just-shown section
+    // rendered for a beat, then vanished.
+    //
+    // "Same page" is the handle (localized variants share a handle but not an
+    // id, see ff67a5a) plus the pathname the customizer payload arrived on.
+    // Handles are only unique per content type, so a client-side navigation
+    // from /products/hats to /collections/hats changes the pathname and hands
+    // ownership back to loader data instead of pinning the stale product page.
+    const syncedLoaderContentRef = useRef(null);
+    const customizerContentRef = useRef(null);
+    const customizerPathnameRef = useRef(null);
     useEffect(() => {
-        if (isPreview && setLiveContent) {
-            setLiveContent(content);
+        if (!isPreview || !setLiveContent)
+            return;
+        if (liveContent === content)
+            return;
+        const pathname = typeof window === "undefined" ? null : window.location.pathname;
+        const fromCustomizer = !!liveContent && liveContent !== syncedLoaderContentRef.current;
+        if (fromCustomizer && liveContent !== customizerContentRef.current) {
+            customizerContentRef.current = liveContent;
+            customizerPathnameRef.current = pathname;
         }
-    }, [content, isPreview, setLiveContent]);
+        const customizerOwnsPage = fromCustomizer &&
+            liveContent?.handle === content?.handle &&
+            customizerPathnameRef.current === pathname;
+        if (customizerOwnsPage)
+            return;
+        syncedLoaderContentRef.current = content;
+        setLiveContent(content);
+    }, [content, isPreview, liveContent, setLiveContent]);
     // Smart content selection: prefer current content when it changes (navigation),
     // but allow liveContent to take precedence when it's updated by customizer
     const contentToRender = useMemo(() => {

package/dist/version.js

@@ -1 +1 @@
-export const VERSION = "4.3.3";
+export const VERSION = "4.3.4";

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@pack/react",
   "description": "React",
-  "version": "4.3.3",
+  "version": "4.3.4",
   "exports": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "engines": {