@openg2p/registry-widgets 1.1.0-dev.9 → 1.1.0

package/dist/index.js

@@ -1078,16 +1078,12 @@ const getApiDataSource = async (dataSource, allValues, dataSourceRequestHandler,
             console.error('[getApiDataSource] API data source missing endpoint. Use "endpoint" field to specify the operation (e.g., "get_g2p_geo_level_values")');
             return [];
         }
-        let response;
-        try {
-            response = await dataSourceRequestHandler(service, endpoint, method, requestParams, {
-                headers: dataSource.headers,
-            });
-        }
-        catch (error) {
-            console.error('[getApiDataSource] Handler error:', error);
-            throw error;
-        }
+        // Call handler — let any throw propagate to the outer catch so it is logged once
+        // by useBaseWidget rather than double-logged here (which can cascade when
+        // intercept-console-error.js converts console.error calls into thrown errors).
+        const response = await dataSourceRequestHandler(service, endpoint, method, requestParams, {
+            headers: dataSource.headers,
+        });
         // Handle OpenG2P response format (response_body.response_payload)
         if (response && typeof response === 'object') {
             if (response.response_body?.response_payload && Array.isArray(response.response_body.response_payload)) {
@@ -1110,8 +1106,8 @@ const getApiDataSource = async (dataSource, allValues, dataSourceRequestHandler,
         return [];
     }
     catch (error) {
-        console.error('Error fetching API data source:', error);
-        return [];
+        // Rethrow so useBaseWidget's catch can log it with full widget context
+        throw error;
     }
 };
 /**
@@ -2305,7 +2301,7 @@ const useBaseWidget = (options) => {
                 dispatch(setDataSource({ widgetId, data: transformed }));
             }
             catch (error) {
-                console.error(`[useBaseWidget] ERROR loading data source for ${widgetId}:`, error);
+                console.error(`[useBaseWidget] ERROR loading data source for widget "${widgetId}" (type="${dataSource.type}"):`, error, '\nWidget config:', config, '\ndataSourceRequestHandler provided:', Boolean(dataSourceRequestHandler));
                 dispatch(setDataSource({ widgetId, data: [] }));
             }
             finally {
@@ -3947,7 +3943,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,16 +4009,25 @@ const SectionRenderer = ({ section, dataSourceRequestHandler: propDataSourceRequ
     const isExpandedFromContainer = typeof sectionIndex === 'number' && expandedSectionIndex === sectionIndex;
     const isExpandedStandalone = sectionIndex === undefined && standaloneExpanded;
     const isExpanded = mode === 'IntakeForm' && (isExpandedFromContainer || isExpandedStandalone);
+    // 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 (typeof sectionIndex === 'number' && onExpandSection) {
-            onExpandSection(sectionIndex);
-        }
-        else if (sectionIndex === undefined) {
+        if (sectionIndex === undefined) {
             setStandaloneExpanded(prev => !prev);
         }
-    }, [mode, sectionIndex, onExpandSection]);
+        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
@@ -4357,8 +4362,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)
@@ -4591,6 +4594,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]);
@@ -4809,13 +4817,17 @@ const SectionRenderer = ({ section, dataSourceRequestHandler: propDataSourceRequ
         .${sectionClassId}.intake-form-accordion-item .intake-form-accordion-header h2 {
           color: var(--owt-color-primary-dark, #F07B1A);
         }
-        .${sectionClassId}.intake-form-accordion-item .intake-form-accordion-header:hover {
+        /* Hover / focus only shown when the header is actually interactive (standalone mode) */
+        .${sectionClassId}.intake-form-accordion-item .intake-form-accordion-header[data-interactive="true"]:hover {
           opacity: 0.85;
         }
-        .${sectionClassId}.intake-form-accordion-item .intake-form-accordion-header:focus-visible {
+        .${sectionClassId}.intake-form-accordion-item .intake-form-accordion-header[data-interactive="true"]:focus-visible {
           outline: 2px solid var(--owt-color-primary, #F5BB1A);
           outline-offset: 2px;
         }
+        .${sectionClassId}.intake-form-accordion-item .intake-form-accordion-header[data-interactive="false"]:focus-visible {
+          outline: none;
+        }
         .${sectionClassId}.intake-form-accordion-item .intake-form-accordion-content {
           padding-top: 8px;
           padding-bottom: 0px;
@@ -4858,7 +4870,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, 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',
@@ -4868,7 +4880,7 @@ const SectionRenderer = ({ section, dataSourceRequestHandler: propDataSourceRequ
                                 marginBottom: 0,
                                 background: 'none',
                                 border: 'none',
-                                cursor: 'pointer',
+                                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']
@@ -5166,6 +5178,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) => {
@@ -5190,8 +5207,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);
         }
@@ -5385,6 +5403,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
@@ -7367,7 +7387,7 @@ const NumberInputWidget = ({ config }) => {
         if (parsed === null) {
             // Allow empty input or partial input (e.g., "-", ".")
             if (inputValue === '' || inputValue === '-' || inputValue === '.') {
-                onChange('');
+                onChange(null);
             }
             // Don't update if invalid - let user continue typing
             return;
@@ -9731,10 +9751,13 @@ const TextAreaWidget = ({ config }) => {
     // For readonly mode, render as preformatted text using <pre> tag
     if (isReadonly) {
         const displayValue = getStringValue() || '-';
-        return (jsxRuntimeExports.jsxs("div", { className: "mb-[10px] TextAreaDisplayWidget flex flex-col sm:flex-row sm:items-start", children: [label && (jsxRuntimeExports.jsxs("div", { className: "text-base text-gray-600 font-medium md:min-w-[120px] sm:pr-4 mb-1 sm:mb-0", style: { fontFamily: 'Roboto, sans-serif' }, title: label, children: [label, ":"] })), jsxRuntimeExports.jsx("div", { className: "flex-1", children: jsxRuntimeExports.jsx("pre", { className: "text-base text-gray-900 font-medium whitespace-pre-wrap", title: String(displayValue), style: {
+        return (jsxRuntimeExports.jsxs("div", { className: "mb-[10px] TextAreaDisplayWidget flex flex-col sm:flex-row sm:items-start", children: [label && (jsxRuntimeExports.jsxs("div", { className: "text-base text-gray-600 font-medium md:min-w-[120px] sm:pr-4 mb-1 sm:mb-0", style: { fontFamily: 'Roboto, sans-serif' }, title: label, children: [label, ":"] })), jsxRuntimeExports.jsx("div", { className: "flex-1", children: jsxRuntimeExports.jsx("div", { title: String(displayValue), className: "text-base text-gray-900 font-medium overflow-y-auto whitespace-pre-wrap break-words", style: {
                             fontFamily: 'Roboto, sans-serif',
-                            margin: 0,
-                            padding: 0,
+                            height: '56px',
+                            minHeight: '56px',
+                            maxHeight: '56px',
+                            lineHeight: '20px',
+                            padding: '8px 0',
                             backgroundColor: 'transparent',
                             border: 'none',
                         }, children: displayValue }) })] }));