fix: trim whitespace on Search filter fields to prevent failed searches (#3811)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

Author: Copilot

src/components/Search/Search.tsx

@@ -26,6 +26,13 @@ export const Search = ({
     inputRef,
     ...props
 }: SearchProps) => {
+    const onUpdate = React.useCallback(
+        (newValue: string) => {
+            onChange(newValue.trim());
+        },
+        [onChange],
+    );
+
     return (
         <DebouncedTextInput
             debounce={debounce}
@@ -35,7 +42,7 @@ export const Search = ({
             style={{width}}
             className={b(null, className)}
             value={value}
-            onUpdate={onChange}
+            onUpdate={onUpdate}
             {...props}
         />
     );

src/components/Search/__test__/Search.test.tsx

@@ -0,0 +1,59 @@
+import {render, screen} from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
+
+import {Search} from '../Search';
+
+describe('Search', () => {
+    beforeEach(() => {
+        jest.useFakeTimers();
+    });
+
+    afterEach(() => {
+        jest.useRealTimers();
+    });
+
+    it('trims leading and trailing whitespace from the onChange value', async () => {
+        const onChange = jest.fn();
+        const user = userEvent.setup({advanceTimers: jest.advanceTimersByTime});
+
+        render(<Search onChange={onChange} value="" />);
+
+        const input = screen.getByRole('textbox');
+        await user.type(input, '  hello  ');
+
+        jest.runAllTimers();
+
+        const lastCall = onChange.mock.calls[onChange.mock.calls.length - 1];
+        expect(lastCall[0]).toBe('hello');
+    });
+
+    it('preserves internal spaces in the onChange value', async () => {
+        const onChange = jest.fn();
+        const user = userEvent.setup({advanceTimers: jest.advanceTimersByTime});
+
+        render(<Search onChange={onChange} value="" />);
+
+        const input = screen.getByRole('textbox');
+        await user.type(input, 'hello world');
+
+        jest.runAllTimers();
+
+        const lastCall = onChange.mock.calls[onChange.mock.calls.length - 1];
+        expect(lastCall[0]).toBe('hello world');
+    });
+
+    it('calls onChange with empty string for whitespace-only input', async () => {
+        const onChange = jest.fn();
+        const user = userEvent.setup({advanceTimers: jest.advanceTimersByTime});
+
+        render(<Search onChange={onChange} value="" />);
+
+        const input = screen.getByRole('textbox');
+        await user.type(input, '   ');
+
+        jest.runAllTimers();
+
+        const lastCall = onChange.mock.calls[onChange.mock.calls.length - 1];
+        expect(lastCall[0]).toBe('');
+    });
+});