@dxos/react-ui-searchlist 0.8.4-main.e8ec1fe → 0.8.4-main.ef1bc66f44

package/src/components/SearchList/SearchList.tsx

@@ -1,76 +1,386 @@
 //
-// Copyright 2023 DXOS.org
+// Copyright 2025 DXOS.org
 //
 
-import { CommandEmpty, CommandInput, CommandItem, CommandList, CommandRoot } from 'cmdk';
-import React, { type ComponentPropsWithRef, forwardRef } from 'react';
+import { useControllableState } from '@radix-ui/react-use-controllable-state';
+import React, {
+  type ChangeEvent,
+  type ComponentPropsWithRef,
+  type KeyboardEvent,
+  type PropsWithChildren,
+  type ReactNode,
+  forwardRef,
+  useCallback,
+  useEffect,
+  useMemo,
+  useRef,
+  useState,
+} from 'react';
 
 import {
-  type TextInputProps,
+  type Density,
+  type Elevation,
+  Icon,
   type ThemedClassName,
   useDensityContext,
   useElevationContext,
   useThemeContext,
   useTranslation,
 } from '@dxos/react-ui';
-import { mx } from '@dxos/react-ui-theme';
+import { descriptionText, mx } from '@dxos/ui-theme';
 
 import { translationKey } from '../../translations';
 
-const commandItem = 'flex items-center overflow-hidden';
-const searchListItem =
-  'plb-1 pli-2 rounded-sm select-none cursor-pointer data-[selected]:bg-hoverOverlay hover:bg-hoverOverlay';
+import {
+  SearchListInputContextProvider,
+  SearchListItemContextProvider,
+  useSearchListInputContext,
+  useSearchListItemContext,
+} from './context';
+
+//
+// Internal types
+//
 
-const SEARCHLIST_NAME = 'SearchList';
-const SEARCHLIST_ITEM_NAME = 'SearchListItem';
+type ItemData = {
+  element: HTMLElement;
+  disabled?: boolean;
+  onSelect?: () => void;
+};
 
 //
 // Root
 //
 
-type SearchListVariant = 'list' | 'menu' | 'listbox';
+type SearchListRootProps = PropsWithChildren<{
+  /** Controlled query value. */
+  value?: string;
+
+  /** Default query value for uncontrolled mode. */
+  defaultValue?: string;
+
+  /** Debounce delay in milliseconds. */
+  debounceMs?: number;
+
+  /** Callback when search query changes (debounced). */
+  onSearch?: (query: string) => void;
+}>;
+
+const SearchListRoot = ({
+  children,
+  value: valueProp,
+  defaultValue = '',
+  debounceMs = 200,
+  onSearch,
+}: SearchListRootProps) => {
+  const [query = '', setQuery] = useControllableState({
+    prop: valueProp,
+    defaultProp: defaultValue,
+    onChange: undefined,
+  });
+
+  const [selectedValue, setSelectedValue] = useState<string | undefined>(undefined);
+
+  // Track registered items: value -> { element, onSelect, disabled }.
+  const itemsRef = useRef<Map<string, ItemData>>(new Map());
+
+  const debounceRef = useRef<ReturnType<typeof setTimeout> | null>(null);
+
+  const handleQueryChange = useCallback(
+    (newQuery: string) => {
+      setQuery(newQuery);
+      // Don't update selectedValue here - let the effect handle it when items actually change.
+      // This prevents unnecessary re-renders of items when query changes.
+
+      // Debounce onSearch callback.
+      if (debounceRef.current) {
+        clearTimeout(debounceRef.current);
+      }
+      debounceRef.current = setTimeout(() => {
+        onSearch?.(newQuery);
+      }, debounceMs);
+    },
+    [setQuery, onSearch, debounceMs],
+  );
+
+  // Track when items change to trigger first-item selection.
+  const [itemVersion, setItemVersion] = useState(0);
+
+  // Cleanup debounce on unmount.
+  useEffect(() => {
+    return () => {
+      if (debounceRef.current) {
+        clearTimeout(debounceRef.current);
+      }
+    };
+  }, []);
+
+  // Auto-select first non-disabled item when items change and no valid selection exists.
+  useEffect(() => {
+    // Check if current selection is still valid (exists and not disabled).
+    const currentItem = selectedValue !== undefined ? itemsRef.current.get(selectedValue) : undefined;
+    const isSelectionValid = currentItem !== undefined && !currentItem.disabled;
+    if (!isSelectionValid && itemsRef.current.size > 0) {
+      // Get first non-disabled item in DOM order.
+      const entries = Array.from(itemsRef.current.entries()).filter(([, data]) => !data.disabled);
+      if (entries.length > 0) {
+        entries.sort(([, a], [, b]) => {
+          const position = a.element.compareDocumentPosition(b.element);
+          if (position & Node.DOCUMENT_POSITION_FOLLOWING) {
+            return -1;
+          }
+          if (position & Node.DOCUMENT_POSITION_PRECEDING) {
+            return 1;
+          }
+          return 0;
+        });
+        const firstValue = entries[0]?.[0];
+        if (firstValue !== undefined && firstValue !== selectedValue) {
+          setSelectedValue(firstValue);
+        }
+      } else if (selectedValue !== undefined) {
+        // No valid items available, clear selection
+        setSelectedValue(undefined);
+      }
+    }
+  }, [itemVersion, selectedValue]);
+
+  const registerItem = useCallback(
+    (value: string, element: HTMLElement | null, onSelect: (() => void) | undefined, disabled?: boolean) => {
+      if (element) {
+        itemsRef.current.set(value, { element, onSelect, disabled });
+        setItemVersion((v) => v + 1);
+      }
+    },
+    [],
+  );
+
+  const unregisterItem = useCallback((value: string) => {
+    itemsRef.current.delete(value);
+    setItemVersion((v) => v + 1);
+  }, []);
+
+  // Get item values in DOM order by sorting registered elements (excludes disabled items).
+  const getItemValues = useCallback(() => {
+    return Array.from(itemsRef.current.entries())
+      .filter(([, data]) => !data.disabled)
+      .sort(([, a], [, b]) => {
+        // Sort by DOM position using compareDocumentPosition.
+        const position = a.element.compareDocumentPosition(b.element);
+        return position & Node.DOCUMENT_POSITION_FOLLOWING ? -1 : position & Node.DOCUMENT_POSITION_PRECEDING ? 1 : 0;
+      })
+      .map(([value]) => value);
+  }, []);
+
+  const triggerSelect = useCallback(() => {
+    if (selectedValue !== undefined) {
+      const item = itemsRef.current.get(selectedValue);
+      item?.onSelect?.();
+    }
+  }, [selectedValue]);
+
+  // Item context; stable, doesn't change when query changes.
+  const itemContextValue = useMemo(
+    () => ({
+      selectedValue,
+      onSelectedValueChange: setSelectedValue,
+      registerItem,
+      unregisterItem,
+    }),
+    [selectedValue, registerItem, unregisterItem],
+  );
 
-type SearchListRootProps = ThemedClassName<ComponentPropsWithRef<typeof CommandRoot>> & {
-  variant?: SearchListVariant;
+  const inputContextValue = useMemo(
+    () => ({
+      query,
+      onQueryChange: handleQueryChange,
+      selectedValue,
+      onSelectedValueChange: setSelectedValue,
+      getItemValues,
+      triggerSelect,
+    }),
+    [query, handleQueryChange, selectedValue, getItemValues, triggerSelect],
+  );
+
+  // NOTE: Separate contexts for items and input to avoid unnecessary re-renders of items when query changes.
+  return (
+    <SearchListInputContextProvider
+      query={inputContextValue.query}
+      onQueryChange={inputContextValue.onQueryChange}
+      selectedValue={inputContextValue.selectedValue}
+      onSelectedValueChange={inputContextValue.onSelectedValueChange}
+      getItemValues={inputContextValue.getItemValues}
+      triggerSelect={inputContextValue.triggerSelect}
+    >
+      <SearchListItemContextProvider
+        selectedValue={itemContextValue.selectedValue}
+        onSelectedValueChange={itemContextValue.onSelectedValueChange}
+        registerItem={itemContextValue.registerItem}
+        unregisterItem={itemContextValue.unregisterItem}
+      >
+        {children}
+      </SearchListItemContextProvider>
+    </SearchListInputContextProvider>
+  );
 };
 
-const SearchListRoot = forwardRef<HTMLDivElement, SearchListRootProps>(
-  ({ children, classNames, ...props }, forwardedRef) => {
+SearchListRoot.displayName = 'SearchList.Root';
+
+//
+// Content
+//
+
+type SearchListContentProps = ThemedClassName<PropsWithChildren<{}>>;
+
+const SearchListContent = forwardRef<HTMLDivElement, SearchListContentProps>(
+  ({ classNames, children }, forwardedRef) => {
+    return (
+      <div
+        role='none'
+        // TODO(burdon): Remove p-1 hack.
+        className={mx('flex flex-col gap-2 bs-full is-full min-bs-0 overflow-hidden p-1', classNames)}
+        ref={forwardedRef}
+      >
+        {children}
+      </div>
+    );
+  },
+);
+
+SearchListContent.displayName = 'SearchList.Content';
+
+//
+// Viewport
+//
+
+type SearchListViewportProps = ThemedClassName<PropsWithChildren>;
+
+/**
+ * Scrollable viewport wrapper for Content.
+ * Only Content wrapped in Viewport will be scrollable.
+ */
+// TODO(burdon): Reconcile with Mosaic.Viewport (factor out common core?).
+const SearchListViewport = forwardRef<HTMLDivElement, SearchListViewportProps>(
+  ({ classNames, children }, forwardedRef) => {
     return (
-      <CommandRoot {...props} className={mx(classNames)} ref={forwardedRef}>
+      <div role='listbox' className={mx('bs-full is-full min-bs-0 overflow-y-auto', classNames)} ref={forwardedRef}>
         {children}
-      </CommandRoot>
+      </div>
     );
   },
 );
 
-SearchListRoot.displayName = SEARCHLIST_NAME;
+SearchListViewport.displayName = 'SearchList.Viewport';
 
 //
 // Input
 //
 
-type CommandInputPrimitiveProps = ComponentPropsWithRef<typeof CommandInput>;
+type InputVariant = 'default' | 'subdued';
 
-// TODO: Harmonize with other inputs’ `onChange` prop.
-type SearchListInputProps = Omit<TextInputProps, 'value' | 'defaultValue' | 'onChange'> &
-  Pick<CommandInputPrimitiveProps, 'value' | 'defaultValue' | 'onValueChange'>;
+type SearchListInputProps = ThemedClassName<
+  Omit<ComponentPropsWithRef<'input'>, 'value'> & {
+    density?: Density;
+    elevation?: Elevation;
+    variant?: InputVariant;
+  }
+>;
 
 const SearchListInput = forwardRef<HTMLInputElement, SearchListInputProps>(
-  ({ classNames, density: propsDensity, elevation: propsElevation, variant, ...props }, forwardedRef) => {
+  (
+    { classNames, density: propsDensity, elevation: propsElevation, variant, placeholder, onChange, ...props },
+    forwardedRef,
+  ) => {
+    const { query, onQueryChange, selectedValue, onSelectedValueChange, getItemValues, triggerSelect } =
+      useSearchListInputContext('SearchList.Input');
     const { t } = useTranslation(translationKey);
-    const placeholder = props.placeholder ?? t('search.placeholder');
-
-    // TODO(thure): Keep this in-sync with `TextInput`, or submit a PR for `cmdk` to support `asChild` so we don’t have to.
-    const { hasIosKeyboard } = useThemeContext();
-    const { tx } = useThemeContext();
+    const { hasIosKeyboard, tx } = useThemeContext();
     const density = useDensityContext(propsDensity);
     const elevation = useElevationContext(propsElevation);
+    const defaultPlaceholder = t('search.placeholder');
+
+    const handleChange = useCallback(
+      (event: ChangeEvent<HTMLInputElement>) => {
+        onQueryChange(event.target.value);
+        onChange?.(event);
+      },
+      [onQueryChange, onChange],
+    );
+
+    const handleKeyDown = useCallback(
+      (event: KeyboardEvent<HTMLInputElement>) => {
+        const values = getItemValues();
+        if (values.length === 0) {
+          if (event.key === 'Escape') {
+            onQueryChange('');
+          }
+          return;
+        }
+
+        const currentIndex = selectedValue !== undefined ? values.indexOf(selectedValue) : -1;
+
+        switch (event.key) {
+          case 'ArrowDown': {
+            event.preventDefault();
+            const nextIndex = currentIndex === -1 ? 0 : Math.min(currentIndex + 1, values.length - 1);
+            const nextValue = values[nextIndex];
+            if (nextValue !== undefined) {
+              onSelectedValueChange(nextValue);
+            }
+            break;
+          }
+          case 'ArrowUp': {
+            event.preventDefault();
+            const prevIndex = currentIndex === -1 ? values.length - 1 : Math.max(currentIndex - 1, 0);
+            const prevValue = values[prevIndex];
+            if (prevValue !== undefined) {
+              onSelectedValueChange(prevValue);
+            }
+            break;
+          }
+          case 'Enter': {
+            if (selectedValue !== undefined) {
+              event.preventDefault();
+              triggerSelect();
+            }
+            break;
+          }
+          case 'Home': {
+            event.preventDefault();
+            const firstValue = values[0];
+            if (firstValue !== undefined) {
+              onSelectedValueChange(firstValue);
+            }
+            break;
+          }
+          case 'End': {
+            event.preventDefault();
+            const lastValue = values[values.length - 1];
+            if (lastValue !== undefined) {
+              onSelectedValueChange(lastValue);
+            }
+            break;
+          }
+          case 'Escape': {
+            event.preventDefault();
+            if (selectedValue !== undefined) {
+              onSelectedValueChange(undefined);
+            } else {
+              onQueryChange('');
+            }
+            break;
+          }
+        }
+      },
+      [selectedValue, onSelectedValueChange, getItemValues, triggerSelect, onQueryChange],
+    );
 
     return (
-      <CommandInput
+      <input
         {...props}
-        placeholder={placeholder}
+        {...(props.autoFocus && !hasIosKeyboard && { autoFocus: true })}
+        type='text'
+        placeholder={placeholder ?? defaultPlaceholder}
         className={tx(
           'input.input',
           'input',
@@ -80,65 +390,151 @@ const SearchListInput = forwardRef<HTMLInputElement, SearchListInputProps>(
             density,
             elevation,
           },
-          'mbe-cardSpacingBlock',
           classNames,
         )}
-        {...(props.autoFocus && !hasIosKeyboard && { autoFocus: true })}
+        value={query}
+        onChange={handleChange}
+        onKeyDown={handleKeyDown}
         ref={forwardedRef}
       />
     );
   },
 );
 
+SearchListInput.displayName = 'SearchList.Input';
+
 //
-// Content
+// Item
 //
 
-type SearchListContentProps = ThemedClassName<ComponentPropsWithRef<typeof CommandList>>;
+type SearchListItemProps = ThemedClassName<{
+  /** Unique identifier for the item. */
+  value: string;
+  /** Display label for the item. */
+  label: string;
+  /** Icon to display (string identifier for Icon component). */
+  icon?: string;
+  /** Whether to show a check icon. */
+  checked?: boolean;
+  /** Suffix text to display after the label. */
+  suffix?: string;
+  /** Callback when item is selected. */
+  onSelect?: () => void;
+  /** Whether the item is disabled. */
+  disabled?: boolean;
+}>;
+
+const SearchListItem = forwardRef<HTMLDivElement, SearchListItemProps>(
+  ({ classNames, value, label, icon, checked, suffix, onSelect, disabled }, forwardedRef) => {
+    const { selectedValue, registerItem, unregisterItem } = useSearchListItemContext('SearchList.Item');
+    const internalRef = useRef<HTMLDivElement>(null);
+
+    const isSelected = selectedValue === value && !disabled;
+
+    // Register this item.
+    useEffect(() => {
+      const element = internalRef.current;
+      if (element) {
+        registerItem(value, element, onSelect, disabled);
+      }
+      return () => unregisterItem(value);
+    }, [value, onSelect, disabled, registerItem, unregisterItem]);
+
+    // Scroll into view when selected.
+    useEffect(() => {
+      if (isSelected && internalRef.current) {
+        internalRef.current.scrollIntoView({ block: 'nearest', behavior: 'smooth' });
+      }
+    }, [isSelected]);
+
+    const handleClick = useCallback(() => {
+      if (!disabled) {
+        onSelect?.();
+      }
+    }, [onSelect, disabled]);
 
-const SearchListContent = forwardRef<HTMLDivElement, SearchListContentProps>(
-  ({ children, classNames, ...props }, forwardedRef) => {
     return (
-      <CommandList {...props} className={mx(classNames)} ref={forwardedRef}>
-        {children}
-      </CommandList>
+      <div
+        ref={(node) => {
+          internalRef.current = node;
+          if (typeof forwardedRef === 'function') {
+            forwardedRef(node);
+          } else if (forwardedRef) {
+            forwardedRef.current = node;
+          }
+        }}
+        role='option'
+        aria-selected={isSelected}
+        aria-disabled={disabled}
+        data-selected={isSelected}
+        data-disabled={disabled}
+        data-value={value}
+        tabIndex={-1}
+        className={mx(
+          'flex gap-2 items-center',
+          'plb-1 pli-2 rounded-sm select-none cursor-pointer data-[selected=true]:bg-hoverOverlay hover:bg-hoverOverlay',
+          disabled && 'opacity-50 cursor-not-allowed hover:bg-transparent data-[selected=true]:bg-transparent',
+          classNames,
+        )}
+        onClick={handleClick}
+      >
+        {icon && <Icon icon={icon} size={5} />}
+        <span className='is-0 grow truncate'>{label}</span>
+        {suffix && <span className={mx('shrink-0', descriptionText)}>{suffix}</span>}
+        {checked && <Icon icon='ph--check--regular' size={5} />}
+      </div>
     );
   },
 );
 
+SearchListItem.displayName = 'SearchList.Item';
+
 //
 // Empty
 //
 
-type SearchListEmptyProps = ThemedClassName<ComponentPropsWithRef<typeof CommandEmpty>>;
+type SearchListEmptyProps = ThemedClassName<PropsWithChildren<{}>>;
 
-const SearchListEmpty = forwardRef<HTMLDivElement, SearchListEmptyProps>(
-  ({ children, classNames, ...props }, forwardedRef) => {
-    return (
-      <CommandEmpty {...props} className={mx(classNames)} ref={forwardedRef}>
-        {children}
-      </CommandEmpty>
-    );
-  },
-);
+const SearchListEmpty = ({ classNames, children }: SearchListEmptyProps) => {
+  return (
+    <div role='status' className={mx('flex flex-col is-full pli-2 plb-1', classNames)}>
+      {children}
+    </div>
+  );
+};
+
+SearchListEmpty.displayName = 'SearchList.Empty';
 
 //
-// Item
+// Group
 //
 
-type SearchListItemProps = ThemedClassName<ComponentPropsWithRef<typeof CommandItem>>;
+type SearchListGroupProps = ThemedClassName<
+  PropsWithChildren<{
+    /** Heading for the group. */
+    heading?: ReactNode;
+  }>
+>;
 
-const SearchListItem = forwardRef<HTMLDivElement, SearchListItemProps>(
-  ({ children, classNames, ...props }, forwardedRef) => {
+/**
+ * Groups related search items with an optional heading.
+ */
+const SearchListGroup = forwardRef<HTMLDivElement, SearchListGroupProps>(
+  ({ classNames, heading, children }, forwardedRef) => {
     return (
-      <CommandItem {...props} className={mx(searchListItem, classNames)} ref={forwardedRef}>
+      <div ref={forwardedRef} role='group' className={mx('flex flex-col', classNames)}>
+        {heading && (
+          <div role='presentation' className='pli-2 plb-1 text-xs font-medium text-description'>
+            {heading}
+          </div>
+        )}
         {children}
-      </CommandItem>
+      </div>
     );
   },
 );
 
-SearchListItem.displayName = SEARCHLIST_ITEM_NAME;
+SearchListGroup.displayName = 'SearchList.Group';
 
 //
 // SearchList
@@ -146,18 +542,20 @@ SearchListItem.displayName = SEARCHLIST_ITEM_NAME;
 
 export const SearchList = {
   Root: SearchListRoot,
-  Input: SearchListInput,
   Content: SearchListContent,
-  Empty: SearchListEmpty,
+  Viewport: SearchListViewport,
+  Input: SearchListInput,
   Item: SearchListItem,
+  Empty: SearchListEmpty,
+  Group: SearchListGroup,
 };
 
 export type {
   SearchListRootProps,
-  SearchListInputProps,
   SearchListContentProps,
-  SearchListEmptyProps,
+  SearchListViewportProps,
+  SearchListInputProps,
   SearchListItemProps,
+  SearchListEmptyProps,
+  SearchListGroupProps,
 };
-
-export { commandItem, searchListItem };

package/src/components/SearchList/context.ts

@@ -0,0 +1,43 @@
+//
+// Copyright 2025 DXOS.org
+//
+
+import { createContext } from '@radix-ui/react-context';
+
+/** Context for items - stable, doesn't change when query changes */
+export type SearchListItemContextValue = {
+  /** Currently selected item value for keyboard navigation. */
+  selectedValue: string | undefined;
+  /** Update the selected value. */
+  onSelectedValueChange: (value: string | undefined) => void;
+  /** Register an item for keyboard navigation. */
+  registerItem: (
+    value: string,
+    element: HTMLElement | null,
+    onSelect: (() => void) | undefined,
+    disabled?: boolean,
+  ) => void;
+  /** Unregister an item. */
+  unregisterItem: (value: string) => void;
+};
+
+/** Context for input - can change frequently with query */
+export type SearchListInputContextValue = {
+  /** Current search query. */
+  query: string;
+  /** Update the query value. */
+  onQueryChange: (query: string) => void;
+  /** Currently selected item value for keyboard navigation. */
+  selectedValue: string | undefined;
+  /** Update the selected value. */
+  onSelectedValueChange: (value: string | undefined) => void;
+  /** Get ordered list of registered item values (excludes disabled items). */
+  getItemValues: () => string[];
+  /** Trigger selection of the currently highlighted item. */
+  triggerSelect: () => void;
+};
+
+export const [SearchListItemContextProvider, useSearchListItemContext] =
+  createContext<SearchListItemContextValue>('SearchListItem');
+export const [SearchListInputContextProvider, useSearchListInputContext] =
+  createContext<SearchListInputContextValue>('SearchListInput');

package/src/components/SearchList/hooks/index.ts

@@ -0,0 +1,8 @@
+//
+// Copyright 2025 DXOS.org
+//
+
+export * from './useGlobalFilter';
+export * from './useSearchListInput';
+export * from './useSearchListItem';
+export * from './useSearchListResults';