Problem
useCategoricalFilterFactory and useDatetimeFilterFactory both accept a column argument, but the column is only used in two places:
getFilterSummary — reads column.label for displaybuildTableFilterFn — reads column shape for cell value lookup
isFilterInactive and getActiveFilter (when filter is inactive) ignore
the column entirely. Tests for these methods currently pass {} with
@ts-expect-error because the TypeScript signature requires a full
ColumnConfig even when it's structurally unused.
Proposed change
Make the column argument optional or accept a partial, OR split the returned FilterHook so that display methods (getFilterSummary) and filter methods (buildTableFilterFn) can declare their column dependency explicitly.
Problem
useCategoricalFilterFactoryanduseDatetimeFilterFactoryboth accept acolumnargument, but the column is only used in two places:getFilterSummary— readscolumn.labelfor displaybuildTableFilterFn— reads column shape for cell value lookupisFilterInactiveandgetActiveFilter(when filter is inactive) ignorethe column entirely. Tests for these methods currently pass
{}with@ts-expect-errorbecause the TypeScript signature requires a fullColumnConfigeven when it's structurally unused.Proposed change
Make the column argument optional or accept a partial, OR split the returned
FilterHookso that display methods (getFilterSummary) and filter methods (buildTableFilterFn) can declare their column dependency explicitly.