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

package/dist/index.js

@@ -3947,7 +3947,7 @@ function scopedClassSelectors(sectionClassId, classNames) {
  * - Panels wrap when they exceed available width
  * - Sections can sit side-by-side if there's space
  */
-const SectionRenderer = ({ section, dataSourceRequestHandler: propDataSourceRequestHandler, schemaData, onValueChange, gridColumnSpan, onSectionSave, hideEditButton = false, mode = 'RegistryView', namespace, changeRequestType, showChangeRequestLabel = true, dbSectionId, sectionRegisterId, onSectionDirtyChange, sectionIndex, sectionCount, expandedSectionIndex, onExpandSection, onSectionSaveSuccess, onPreviousSection, isDraft, onEditModeChange, forceExitEdit, }) => {
+const SectionRenderer = ({ section, dataSourceRequestHandler: propDataSourceRequestHandler, schemaData, onValueChange, gridColumnSpan, onSectionSave, hideEditButton = false, mode = 'RegistryView', namespace, changeRequestType, showChangeRequestLabel = true, dbSectionId, sectionRegisterId, onSectionDirtyChange, sectionIndex, sectionCount, expandedSectionIndex, onExpandSection, onSectionSaveSuccess, onPreviousSection, isDraft, isAccessible = false, onEditModeChange, forceExitEdit, }) => {
     const { translateConfig, translate } = useWidgetTranslation();
     const resolvedTheme = useWidgetTheme();
     const portalCSSVariables = React.useMemo(() => themeToCSSVariables(resolvedTheme), [resolvedTheme]);
@@ -4013,17 +4013,25 @@ const SectionRenderer = ({ section, dataSourceRequestHandler: propDataSourceRequ
     const isExpandedFromContainer = typeof sectionIndex === 'number' && expandedSectionIndex === sectionIndex;
     const isExpandedStandalone = sectionIndex === undefined && standaloneExpanded;
     const isExpanded = mode === 'IntakeForm' && (isExpandedFromContainer || isExpandedStandalone);
-    // When the section is managed by SectionsContainer (sectionIndex is set), header clicks are
-    // disabled — navigation is only allowed via the Next / Previous buttons.
-    // Standalone usage (sectionIndex undefined) keeps the classic toggle behaviour.
+    // IntakeForm only: tracks whether the user has clicked Next on this section at least once.
+    // Used to unlock the accordion header so the user can navigate back to a visited section.
+    const [hasBeenSavedByUser, setHasBeenSavedByUser] = React.useState(false);
+    // Accordion header click behaviour in IntakeForm mode:
+    //   - Standalone (no sectionIndex): always toggleable.
+    //   - Managed by SectionsContainer: toggleable only when isAccessible is true
+    //     (i.e. the section has been visited OR is the immediate next one).
+    //     Sections beyond that remain locked.
     const handleAccordionToggle = React.useCallback(() => {
         if (mode !== 'IntakeForm')
             return;
         if (sectionIndex === undefined) {
             setStandaloneExpanded(prev => !prev);
         }
-        // Intentionally no-op when sectionIndex is set (managed by SectionsContainer)
-    }, [mode, sectionIndex]);
+        else if (isAccessible && onExpandSection) {
+            onExpandSection(sectionIndex);
+        }
+        // Intentionally no-op for locked sections (isAccessible === false)
+    }, [mode, sectionIndex, isAccessible, onExpandSection]);
     // Recursively count all vertical panels, especially those nested inside horizontal panels
     // Typically: horizontal panels at first level contain vertical panels at second level
     // Accounts for panel-column-span: a panel with column-span 3 counts as 3 columns
@@ -4358,8 +4366,6 @@ const SectionRenderer = ({ section, dataSourceRequestHandler: propDataSourceRequ
     const baselineSnapshotRef = React.useRef(null);
     // IntakeForm only: increment when baseline is updated after save - forces badge to update (refs don't trigger re-renders)
     const [intakeFormBaselineTrigger, setIntakeFormBaselineTrigger] = React.useState(0);
-    // IntakeForm only: tracks whether the user has actually saved this section (prevents "Saved" badge on initial load)
-    const [hasBeenSavedByUser, setHasBeenSavedByUser] = React.useState(false);
     // IntakeForm: treat as edit mode for dirty tracking when isDraft. RegistryView: use isEditMode.
     const effectiveEditModeForDirty = mode === 'IntakeForm' ? (isDraft !== false) : isEditMode;
     // Compute isDirty: compare current store state to baseline (only when in edit mode)
@@ -4592,6 +4598,11 @@ const SectionRenderer = ({ section, dataSourceRequestHandler: propDataSourceRequ
             }
             onSectionDirtyChange?.(sectionId, false);
         }
+        else if (mode === 'IntakeForm') {
+            // No onSectionSave provided, but still mark section as visited so the
+            // user can navigate back to it by clicking the accordion header.
+            setHasBeenSavedByUser(true);
+        }
         // Always navigate to the next section
         onSectionSaveSuccess?.(sectionIndex);
     }, [store, onSectionSave, onSectionSaveSuccess, sectionIndex, originalSection, schemaData, contextSchemaData, namespace, hasSupportingDocuments, dbSectionId, sectionRegisterId, dispatch, buildSectionSnapshot, sectionId, onSectionDirtyChange, mode, isDraft]);
@@ -4863,7 +4874,7 @@ const SectionRenderer = ({ section, dataSourceRequestHandler: propDataSourceRequ
                     }),
                 }, children: mode === 'IntakeForm' ? (
                 /* IntakeForm: accordion layout - header always visible, content only when expanded */
-                jsxRuntimeExports.jsxs(jsxRuntimeExports.Fragment, { children: [jsxRuntimeExports.jsxs("button", { type: "button", id: `intake-form-accordion-header-${sectionId}`, className: "intake-form-accordion-header", onClick: handleAccordionToggle, "aria-expanded": isExpanded, "aria-controls": isExpanded ? `intake-form-accordion-content-${sectionId}` : undefined, "data-interactive": sectionIndex === undefined ? 'true' : 'false', style: {
+                jsxRuntimeExports.jsxs(jsxRuntimeExports.Fragment, { children: [jsxRuntimeExports.jsxs("button", { type: "button", id: `intake-form-accordion-header-${sectionId}`, className: "intake-form-accordion-header", onClick: handleAccordionToggle, "aria-expanded": isExpanded, "aria-controls": isExpanded ? `intake-form-accordion-content-${sectionId}` : undefined, "data-interactive": sectionIndex === undefined || isAccessible ? 'true' : 'false', style: {
                                 width: '100%',
                                 display: 'flex',
                                 alignItems: 'flex-start',
@@ -4873,7 +4884,7 @@ const SectionRenderer = ({ section, dataSourceRequestHandler: propDataSourceRequ
                                 marginBottom: 0,
                                 background: 'none',
                                 border: 'none',
-                                cursor: sectionIndex === undefined ? 'pointer' : 'default',
+                                cursor: sectionIndex === undefined || isAccessible ? 'pointer' : 'default',
                                 textAlign: 'left',
                                 fontFamily: 'Roboto, sans-serif',
                             }, children: [jsxRuntimeExports.jsxs("div", { style: { flex: 1, display: 'flex', alignItems: 'center', gap: '12px', minWidth: 0 }, children: [jsxRuntimeExports.jsx("h2", { className: "text-xl font-semibold", style: { margin: 0 }, children: sectionToRender['section-title']
@@ -5171,6 +5182,11 @@ const SectionsContainer = ({ sections, dataSourceRequestHandler: propDataSourceR
     const dataSourceRequestHandler = propDataSourceRequestHandler || contextDataSourceRequestHandler;
     // IntakeForm mode: accordion state - which section is expanded (null = none; first expanded by default)
     const [expandedSectionIndex, setExpandedSectionIndex] = React.useState(0);
+    // IntakeForm mode: high-water mark of the furthest section the user has clicked Next on.
+    // A section at index i is accessible when i <= maxVisitedIndex + 1
+    // (i.e. every visited section plus the one immediately after it).
+    // Starts at -1 so only section 0 is accessible before any Next is clicked.
+    const [maxVisitedIndex, setMaxVisitedIndex] = React.useState(-1);
     // RegistryView: track which section is currently in edit mode (by section-id); null = none
     const [editingSectionId, setEditingSectionId] = React.useState(null);
     const handleEditModeChange = React.useCallback((sectionId, editing) => {
@@ -5181,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;
@@ -5195,8 +5216,9 @@ const SectionsContainer = ({ sections, dataSourceRequestHandler: propDataSourceR
     const handleExpandSection = React.useCallback((index) => {
         setExpandedSectionIndex(prev => (prev === index ? null : index));
     }, []);
-    // IntakeForm mode: called after section save - collapse current, expand next
+    // IntakeForm mode: called after section save - advance high-water mark, collapse current, expand next
     const handleSectionSaveSuccess = React.useCallback((index) => {
+        setMaxVisitedIndex(prev => Math.max(prev, index));
         if (index + 1 < safeSections.length) {
             setExpandedSectionIndex(index + 1);
         }
@@ -5228,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) => {
@@ -5249,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);
@@ -5272,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);
@@ -5297,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) {
@@ -5390,6 +5419,8 @@ const SectionsContainer = ({ sections, dataSourceRequestHandler: propDataSourceR
                             onSectionSaveSuccess: handleSectionSaveSuccess,
                             onPreviousSection: handlePreviousSection,
                             isDraft,
+                            // Accessible = every visited section + the one immediately after
+                            isAccessible: index <= maxVisitedIndex + 1,
                         }
                         : {};
                     // RegistryView: single-edit coordination props