@openg2p/registry-widgets 1.1.0-dev.11 → 1.1.0-dev.12

package/dist/index.js

@@ -5197,6 +5197,11 @@ const SectionsContainer = ({ sections, dataSourceRequestHandler: propDataSourceR
     // Stable ref for namespace so formHandle useMemo doesn't depend on the (possibly inline) function identity
     const namespaceRef = React.useRef(namespace);
     namespaceRef.current = namespace;
+    // Stable ref for sections so formHandle useMemo doesn't depend on the (possibly unstable) array reference.
+    // Without this, an inline sections={[...]} prop would recreate formHandle every render, causing
+    // onFormReady to fire every render → host setState → infinite re-render loop.
+    const safeSectionsRef = React.useRef(safeSections);
+    safeSectionsRef.current = safeSections;
     // Stable refs so formHandle closure can access current mode and accordion setter without stale captures
     const modeRef = React.useRef(mode);
     modeRef.current = mode;
@@ -5245,6 +5250,9 @@ const SectionsContainer = ({ sections, dataSourceRequestHandler: propDataSourceR
     }, [mode, safeSections.length, expandedSectionIndex]);
     const UNSAVED_CHANGES_ERROR = 'Unsaved changes detected. Please save all sections before submitting.';
     // Form handle for onFormReady - allows host to validate and get all section data from its own Submit button
+    // IMPORTANT: safeSections is intentionally accessed via safeSectionsRef.current (not closed over directly)
+    // so that this memo never depends on the sections array reference. This prevents an infinite re-render
+    // loop when the host passes an unstable sections prop (e.g. inline array or derived state).
     const formHandle = React.useMemo(() => {
         const getValues = () => store.getState().widget?.values || {};
         const getNamespace = (section, index) => {
@@ -5266,10 +5274,11 @@ const SectionsContainer = ({ sections, dataSourceRequestHandler: propDataSourceR
                 if (modeRef.current !== 'IntakeForm')
                     checkNoUnsavedChanges();
                 const values = getValues();
+                const sections = safeSectionsRef.current;
                 let allValid = true;
                 let firstInvalidIndex = null;
-                for (let i = 0; i < safeSections.length; i++) {
-                    const section = safeSections[i];
+                for (let i = 0; i < sections.length; i++) {
+                    const section = sections[i];
                     const ns = getNamespace(section, i);
                     const sectionToValidate = ns ? namespaceSectionConfig(section, ns) : section;
                     const valid = sectionValidate(sectionToValidate, values, dispatch);
@@ -5289,10 +5298,11 @@ const SectionsContainer = ({ sections, dataSourceRequestHandler: propDataSourceR
                 if (modeRef.current !== 'IntakeForm')
                     checkNoUnsavedChanges();
                 const values = getValues();
+                const sections = safeSectionsRef.current;
                 const results = [];
                 let firstInvalidIndex = null;
-                for (let i = 0; i < safeSections.length; i++) {
-                    const section = safeSections[i];
+                for (let i = 0; i < sections.length; i++) {
+                    const section = sections[i];
                     const ns = getNamespace(section, i);
                     const sectionToValidate = ns ? namespaceSectionConfig(section, ns) : section;
                     const valid = sectionValidate(sectionToValidate, values, dispatch);
@@ -5314,16 +5324,18 @@ const SectionsContainer = ({ sections, dataSourceRequestHandler: propDataSourceR
             },
             getStructuredData: () => {
                 const values = getValues();
+                const sections = safeSectionsRef.current;
                 const results = [];
-                for (let i = 0; i < safeSections.length; i++) {
-                    const section = safeSections[i];
+                for (let i = 0; i < sections.length; i++) {
+                    const section = sections[i];
                     const ns = getNamespace(section, i);
                     results.push(buildSectionChanges(section, values, ns));
                 }
                 return results;
             },
         };
-    }, [store, dispatch, safeSections]);
+        // eslint-disable-next-line react-hooks/exhaustive-deps
+    }, [store, dispatch]); // safeSections intentionally omitted — accessed via safeSectionsRef
     // Call onFormReady when form is ready (sections loaded)
     React.useEffect(() => {
         if (onFormReady && safeSections.length > 0) {