@seekora-ai/ui-sdk-react 0.2.23 → 0.2.25

package/dist/src/index.js

@@ -303,6 +303,9 @@ class SearchStateManager {
     constructor(config) {
         this.listeners = [];
         this.debounceTimer = null;
+        this.notifyScheduled = false;
+        this.searchCoalesceTimer = null;
+        this.searchCoalesceResolvers = [];
         this.client = config.client;
         this.autoSearch = config.autoSearch !== false;
         this.debounceMs = config.debounceMs || 300;
@@ -447,13 +450,27 @@ class SearchStateManager {
             this.debouncedSearch();
         }
     }
-    // Manual search trigger
+    // Manual search trigger — coalesces rapid calls within 10ms into a single API request
     async search(additionalOptions) {
         // Clear debounce timer if exists
         if (this.debounceTimer) {
             clearTimeout(this.debounceTimer);
             this.debounceTimer = null;
         }
+        return new Promise((resolve, reject) => {
+            this.searchCoalesceResolvers.push({ resolve, reject });
+            if (this.searchCoalesceTimer) {
+                clearTimeout(this.searchCoalesceTimer);
+            }
+            this.searchCoalesceTimer = setTimeout(() => {
+                this.searchCoalesceTimer = null;
+                const resolvers = [...this.searchCoalesceResolvers];
+                this.searchCoalesceResolvers = [];
+                this._executeSearch(additionalOptions).then((result) => resolvers.forEach(r => r.resolve(result)), (err) => resolvers.forEach(r => r.reject(err)));
+            }, 10);
+        });
+    }
+    async _executeSearch(additionalOptions) {
         this.setState({ loading: true, error: null });
         try {
             const searchOptions = this.buildSearchOptions(additionalOptions);
@@ -550,19 +567,27 @@ class SearchStateManager {
         this.state = { ...this.state, ...updates };
         this.notifyListeners();
     }
-    // Notify all listeners of state changes
+    // Notify all listeners of state changes, batched via microtask.
+    // Multiple synchronous mutations (e.g. addRefinement + page reset)
+    // coalesce into a single listener notification.
     notifyListeners() {
-        const state = this.getState();
-        this.listeners.forEach(listener => {
-            try {
-                listener(state);
-            }
-            catch (err) {
-                const error = err instanceof Error ? err : new Error(String(err));
-                log.error('SearchStateManager: Error in listener', {
-                    error: error.message,
-                });
-            }
+        if (this.notifyScheduled)
+            return;
+        this.notifyScheduled = true;
+        queueMicrotask(() => {
+            this.notifyScheduled = false;
+            const state = this.getState();
+            this.listeners.forEach(listener => {
+                try {
+                    listener(state);
+                }
+                catch (err) {
+                    const error = err instanceof Error ? err : new Error(String(err));
+                    log.error('SearchStateManager: Error in listener', {
+                        error: error.message,
+                    });
+                }
+            });
         });
     }
     /** Explicitly clear results (bypasses keepResultsOnClear) */
@@ -608,10 +633,13 @@ class SearchStateManager {
     async fetchFilters(options) {
         log.verbose('SearchStateManager: Fetching filters', { options });
         try {
-            const filterString = this.buildFilterString();
+            // Do NOT pass refinement-based filters to the Filters API.
+            // Facets should be generated from the search query only, not narrowed
+            // by active filter selections. This keeps facet options stable when
+            // users toggle filters (same behaviour as performSimplifiedFacetSearch
+            // in the search API).
             const response = await this.client.getFilters({
                 q: this.state.query || undefined,
-                filter: filterString || undefined,
                 ...options,
             });
             log.info('SearchStateManager: Filters fetched', {
@@ -3521,13 +3549,22 @@ const useFilters = (options) => {
             }
         }
     }, [stateManager, options?.facetBy, options?.maxFacetValues, options?.disjunctiveFacets?.join(',')]);
-    // Refetch when query or refinements change
+    // Track query + refinements to only refetch when they actually change
+    const prevKeyRef = React.useRef('');
+    // Refetch when query or refinements change (not on every state update)
     React.useEffect(() => {
         if (!autoFetch)
             return;
-        const unsubscribe = stateManager.subscribe((_state) => {
+        const unsubscribe = stateManager.subscribe((state) => {
+            const key = `${state.query}|${state.refinements.map(r => `${r.field}:${r.value}`).sort().join(',')}`;
+            if (key === prevKeyRef.current)
+                return;
+            prevKeyRef.current = key;
             fetchFilters();
         });
+        // subscribe() immediately invokes the listener with current state,
+        // which handles the initial fetch (prevKeyRef starts as '' so key
+        // will differ). No explicit fetchFilters() call needed here.
         return unsubscribe;
     }, [stateManager, autoFetch, fetchFilters]);
     // Fetch schema once on mount
@@ -5189,9 +5226,6 @@ const CurrentRefinements = ({ refinements: refinementsProp, onRefinementClear, o
                     opacity: 0.6,
                 }, "aria-label": `Clear ${refinement.label || refinement.field}: ${refinement.value}`, onMouseEnter: e => (e.currentTarget.style.opacity = '1'), onMouseLeave: e => (e.currentTarget.style.opacity = '0.6') }, renderCloseIcon ? renderCloseIcon() : defaultCloseIcon())));
     };
-    if (refinements.length === 0) {
-        return null;
-    }
     // Group refinements by field for grouped layout
     const groupedRefinements = layout === 'grouped'
         ? refinements.reduce((acc, r) => {
@@ -5203,13 +5237,17 @@ const CurrentRefinements = ({ refinements: refinementsProp, onRefinementClear, o
         : null;
     const containerStyles = {
         ...style,
+        // When a custom list theme is provided, make the container transparent to layout
+        // so the list div becomes the direct layout child (enables overflow scroll from parent)
+        ...(refinementsTheme.list && !style?.display ? { display: 'contents' } : {}),
     };
     const listStyles = {
         display: 'flex',
-        flexWrap: layout === 'vertical' ? 'nowrap' : 'wrap',
         flexDirection: layout === 'vertical' ? 'column' : 'row',
         alignItems: layout === 'vertical' ? 'flex-start' : 'center',
-        marginBottom: showClearAll ? theme.spacing.medium : 0,
+        marginBottom: showClearAll && refinements.length > 0 ? theme.spacing.medium : 0,
+        // Only apply flex-wrap if no custom list theme is provided (let theme classes control wrapping)
+        ...(!refinementsTheme.list ? { flexWrap: layout === 'vertical' ? 'nowrap' : 'wrap' } : {}),
     };
     return (React.createElement("div", { className: clsx(refinementsTheme.container, className), style: containerStyles },
         React.createElement("style", null, `
@@ -5218,34 +5256,35 @@ const CurrentRefinements = ({ refinements: refinementsProp, onRefinementClear, o
           to { opacity: 1; transform: scale(1); }
         }
       `),
-        layout === 'grouped' && groupedRefinements ? (Object.entries(groupedRefinements).map(([field, items]) => (React.createElement("div", { key: field, className: refinementsTheme.group, style: { marginBottom: theme.spacing.medium } },
-            React.createElement("div", { className: refinementsTheme.groupLabel, style: {
-                    fontSize: theme.typography.fontSize.small,
-                    fontWeight: 600,
-                    color: theme.colors.text,
-                    marginBottom: theme.spacing.small,
-                    textTransform: 'capitalize',
-                } }, items[0]?.label || field),
-            React.createElement("div", { role: "list", style: listStyles }, items.map((refinement, index) => {
+        refinements.length > 0 && (React.createElement(React.Fragment, null,
+            layout === 'grouped' && groupedRefinements ? (Object.entries(groupedRefinements).map(([field, items]) => (React.createElement("div", { key: field, className: refinementsTheme.group, style: { marginBottom: theme.spacing.medium } },
+                React.createElement("div", { className: refinementsTheme.groupLabel, style: {
+                        fontSize: theme.typography.fontSize.small,
+                        fontWeight: 600,
+                        color: theme.colors.text,
+                        marginBottom: theme.spacing.small,
+                        textTransform: 'capitalize',
+                    } }, items[0]?.label || field),
+                React.createElement("div", { role: "list", style: listStyles }, items.map((refinement, index) => {
+                    return renderRefinement
+                        ? renderRefinement(refinement, index)
+                        : defaultRenderRefinement(refinement, index);
+                })))))) : (React.createElement("div", { role: "list", className: refinementsTheme.list, style: listStyles }, refinements.map((refinement, index) => {
                 return renderRefinement
                     ? renderRefinement(refinement, index)
                     : defaultRenderRefinement(refinement, index);
-            })))))) : (React.createElement("div", { role: "list", className: refinementsTheme.list, style: listStyles }, refinements.map((refinement, index) => {
-            return renderRefinement
-                ? renderRefinement(refinement, index)
-                : defaultRenderRefinement(refinement, index);
-        }))),
-        showClearAll && refinements.length > 1 && (React.createElement("button", { type: "button", onClick: handleClearAll, className: refinementsTheme.clearAllButton, style: {
-                padding: `${theme.spacing.small} ${theme.spacing.medium}`,
-                border: `1px solid ${theme.colors.border}`,
-                borderRadius: typeof theme.borderRadius === 'string' ? theme.borderRadius : theme.borderRadius.medium,
-                backgroundColor: theme.colors.background,
-                color: theme.colors.text,
-                cursor: 'pointer',
-                fontSize: theme.typography.fontSize.small,
-                textDecoration: 'underline',
-                transition: `background-color ${TRANSITIONS$7.fast}`,
-            } }, "Clear all filters"))));
+            }))),
+            showClearAll && refinements.length > 1 && (React.createElement("button", { type: "button", onClick: handleClearAll, className: refinementsTheme.clearAllButton, style: {
+                    padding: `${theme.spacing.small} ${theme.spacing.medium}`,
+                    border: `1px solid ${theme.colors.border}`,
+                    borderRadius: typeof theme.borderRadius === 'string' ? theme.borderRadius : theme.borderRadius.medium,
+                    backgroundColor: theme.colors.background,
+                    color: theme.colors.text,
+                    cursor: 'pointer',
+                    fontSize: theme.typography.fontSize.small,
+                    textDecoration: 'underline',
+                    transition: `background-color ${TRANSITIONS$7.fast}`,
+                } }, "Clear all filters"))))));
 };
 
 /**
@@ -5313,16 +5352,17 @@ const ClearRefinements = ({ clearsQuery = false, resetLabel = 'Clear all filters
                 padding: `${theme.spacing.small} ${theme.spacing.medium}`,
                 fontSize: theme.typography.fontSize.medium,
                 fontWeight: theme.typography.fontWeight?.medium || 500,
-                backgroundColor: canClear ? theme.colors.primary : theme.colors.hover,
-                color: canClear ? '#ffffff' : theme.colors.textSecondary,
+                backgroundColor: theme.colors.primary,
+                color: '#ffffff',
                 border: 'none',
                 borderRadius: typeof theme.borderRadius === 'string'
                     ? theme.borderRadius
                     : theme.borderRadius.medium,
-                cursor: canClear ? 'pointer' : 'not-allowed',
-                opacity: canClear ? 1 : 0.6,
-                transition: theme.transitions?.fast || '150ms ease-in-out',
-            }, "aria-label": canClear ? resetLabel : disabledLabel }, canClear ? resetLabel : disabledLabel)));
+                cursor: canClear ? 'pointer' : 'default',
+                opacity: canClear ? 1 : 0,
+                pointerEvents: canClear ? 'auto' : 'none',
+                transition: 'none',
+            }, "aria-label": resetLabel }, resetLabel)));
 };
 
 /**
@@ -5360,7 +5400,7 @@ const SearchLayout = ({ sidebar, children, header, footer, sidebarWidth = '300px
                 padding: responsivePadding,
                 backgroundColor: theme.colors.background,
                 color: theme.colors.text,
-                overflow: isMobile ? 'visible' : 'hidden',
+                overflow: 'visible',
             } },
             sidebar && (!isMobile || showSidebarOnMobile) && (React.createElement("aside", { className: layoutTheme.sidebar, style: {
                     width: isMobile ? '100%' : sidebarWidth,