fix: don't reset controlled values

docs/stories/ExternalFilters.stories.tsx

@@ -13,11 +13,11 @@ export default {
 } as ComponentMeta<typeof Table>;
 
 export const Primary = () => {
-  const [sort, setSort] = useState<Sort[]>();
-  const [firstName, setFirstName] = useState<string>();
-  const [lastName, setLastName] = useState<string>();
-  const [jobTitle, setJobTitle] = useState<Set<string>>();
-  const [birthday, setBirthday] = useState<Date | DateRange | null>();
+  const [sort, setSort] = useState(new Array<Sort>());
+  const [firstName, setFirstName] = useState('');
+  const [lastName, setLastName] = useState('');
+  const [jobTitle, setJobTitle] = useState(new Set<string>());
+  const [birthday, setBirthday] = useState<Date | DateRange | null>(null);
 
   return (
     <Box>
@@ -39,6 +39,13 @@ export const Primary = () => {
         sort={sort}
         onSortChange={setSort}
         externalSort
+        onReset={() => {
+          setSort([]);
+          setFirstName('');
+          setLastName('');
+          setJobTitle(new Set());
+          setBirthday(null);
+        }}
         columns={(col) => [
           //
           col((x) => x.avatar, {

src/hooks/useFilter.ts

@@ -14,7 +14,9 @@ export function useFilter<T, V, F, S extends SerializableValue>(impl: FilterImpl
 
   // On mount and reset: Fire onChange
   useEffect(() => {
-    impl.onChange?.(impl.defaultValue);
+    if (impl.value === undefined) {
+      impl.onChange?.(impl.defaultValue);
+    }
   }, [table]);
 
   // Update implementation

src/internalState/useTableState.ts

@@ -15,10 +15,24 @@ export function useTableState<T>(_props: TableProps<T>): [Store<InternalTableSta
 
   function reset() {
     setKey({});
-    props.onSortChange?.(props.defaultSort ?? []);
-    props.onExpandedChange?.(props.defaultExpanded ?? new Set());
-    props.onSelectionChange?.(props.defaultSelection ?? new Set());
-    props.onHiddenColumnsChange?.(props.defaultHiddenColumns ?? new Set());
+
+    if (props.sort === undefined) {
+      props.onSortChange?.(props.defaultSort ?? []);
+    }
+
+    if (props.expanded === undefined) {
+      props.onExpandedChange?.(props.defaultExpanded ?? new Set());
+    }
+
+    if (props.selection === undefined) {
+      props.onSelectionChange?.(props.defaultSelection ?? new Set());
+    }
+
+    if (props.hiddenColumns === undefined) {
+      props.onHiddenColumnsChange?.(props.defaultHiddenColumns ?? new Set());
+    }
+
+    props.onReset?.();
   }
 
   const state = useMemo(

src/types.ts

@@ -246,6 +246,7 @@ export interface TableProps<T> extends PartialTableTheme<T> {
   };
   debug?: (...output: any) => void;
   debugRender?: (...output: any) => void;
+  onReset?: () => void;
 }
 
 export type InternalTableProps<T> = MemoizedFunctions<