fix: notFoundContent failed the search state (#1201)

* chore: init test

* fix: add rawOpen to useOpen hook return value

Return both ssrSafeOpen and mergedOpen to prevent incorrect
clearing of search text when mergedOpen differs from rawOpen.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* docs: add comment explaining rawOpen usage

Clarify why rawOpen is used instead of mergedOpen to prevent
premature clearing of search input.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: zombieJ

src/BaseSelect/index.tsx

@@ -362,7 +362,7 @@ const BaseSelect = React.forwardRef<BaseSelectRef, BaseSelectProps>((props, ref)
   // Not trigger `open` when `notFoundContent` is empty
   const emptyListContent = !notFoundContent && emptyOptions;
 
-  const [mergedOpen, triggerOpen, lockOptions] = useOpen(
+  const [rawOpen, mergedOpen, triggerOpen, lockOptions] = useOpen(
     defaultOpen || false,
     open,
     onPopupVisibleChange,
@@ -430,12 +430,15 @@ const BaseSelect = React.forwardRef<BaseSelectRef, BaseSelectProps>((props, ref)
     onSearch(searchText, { source: 'submit' });
   };
 
-  // Close will clean up single mode search text
+  // Clean up search value when the dropdown is closed.
+  // We use `rawOpen` here to avoid clearing the search input when the dropdown is
+  // programmatically closed due to `notFoundContent={null}` and no matching options.
+  // This allows the user to continue typing their search query.
   React.useEffect(() => {
-    if (!mergedOpen && !multiple && mode !== 'combobox') {
+    if (!rawOpen && !multiple && mode !== 'combobox') {
       onInternalSearch('', false, false);
     }
-  }, [mergedOpen]);
+  }, [rawOpen]);
 
   // ============================ Disabled ============================
   // Close dropdown & remove focus state when disabled change

src/hooks/useOpen.ts

@@ -43,7 +43,7 @@ export default function useOpen(
   propOpen: boolean,
   onOpen: (nextOpen: boolean) => void,
   postOpen: (nextOpen: boolean) => boolean,
-): [open: boolean, toggleOpen: TriggerOpenType, lockOptions: boolean] {
+): [rawOpen: boolean, open: boolean, toggleOpen: TriggerOpenType, lockOptions: boolean] {
   // SSR not support Portal which means we need delay `open` for the first time render
   const [rendered, setRendered] = useState(false);
 
@@ -101,5 +101,5 @@ export default function useOpen(
     }
   });
 
-  return [mergedOpen, toggleOpen, lock];
+  return [ssrSafeOpen, mergedOpen, toggleOpen, lock];
 }

tests/Select.test.tsx

@@ -1997,6 +1997,34 @@ describe('Select.Basic', () => {
     expect(container.querySelector('.rc-select-dropdown-empty')).toBeFalsy();
   });
 
+  it('should allow typing when notFoundContent is null and no options match', () => {
+    const onSearch = jest.fn();
+    const { container } = render(
+      <Select showSearch notFoundContent={null} onSearch={onSearch}>
+        <Option value="jack">Jack</Option>
+        <Option value="lucy">Lucy</Option>
+      </Select>,
+    );
+
+    const input = container.querySelector('input');
+
+    // Type 'j' - should match 'Jack'
+    fireEvent.change(input, { target: { value: 'j' } });
+    expect(onSearch).toHaveBeenLastCalledWith('j');
+    expect(input.value).toBe('j');
+    expect(container.querySelectorAll('.rc-select-item-option')).toHaveLength(1);
+
+    // Type 'x' - no match, but input should still work
+    fireEvent.change(input, { target: { value: 'x' } });
+    expect(onSearch).toHaveBeenLastCalledWith('x');
+    expect(input.value).toBe('x');
+
+    // Type more characters - should continue working
+    fireEvent.change(input, { target: { value: 'xyz' } });
+    expect(onSearch).toHaveBeenLastCalledWith('xyz');
+    expect(input.value).toBe('xyz');
+  });
+
   it('click outside to close select', () => {
     jest.useFakeTimers();