fix(table-core): auto-remove empty arrHas filter values like the other array filters (TanStack#6356)

* fix(table-core): auto-remove empty arrHas filter values like the other array filters

arrHas was the only array-valued filter function without an autoRemove handler, so an empty array filter value was never removed and hid every row.

* ci: apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

Author: mahirhir

packages/table-core/src/fns/filterFns.ts

@@ -296,16 +296,16 @@ export const filterFn_inNumberRange = Object.assign(
 /**
  * Keeps rows whose scalar column value equals at least one filter value.
  */
-export const filterFn_arrHas = <
-  TFeatures extends TableFeatures,
-  TData extends RowData,
->(
-  row: Row<TFeatures, TData>,
-  columnId: string,
-  filterValue: Array<unknown>,
-) => {
-  return filterValue.some((val) => row.getValue<unknown>(columnId) === val)
-}
+export const filterFn_arrHas = Object.assign(
+  <TFeatures extends TableFeatures, TData extends RowData>(
+    row: Row<TFeatures, TData>,
+    columnId: string,
+    filterValue: Array<unknown>,
+  ) => {
+    return filterValue.some((val) => row.getValue<unknown>(columnId) === val)
+  },
+  { autoRemove: (val: any) => testFalsy(val) || !val?.length },
+)
 
 /**
  * Keeps rows whose array or string column value includes at least one filter value.

packages/table-core/tests/unit/fns/filterFns.test.ts

@@ -563,4 +563,26 @@ describe('Filter Functions', () => {
       })
     })
   })
+
+  describe('Array Filters', () => {
+    describe('filterFns.arrHas.autoRemove', () => {
+      const autoRemove = filterFns.arrHas.autoRemove!
+
+      it('should auto-remove when the filter value is undefined', () => {
+        expect(autoRemove(undefined)).toBe(true)
+      })
+      it('should auto-remove when the filter value is null', () => {
+        expect(autoRemove(null)).toBe(true)
+      })
+      it('should auto-remove when the filter value is an empty string', () => {
+        expect(autoRemove('')).toBe(true)
+      })
+      it('should auto-remove when the filter value is an empty array', () => {
+        expect(autoRemove([])).toBe(true)
+      })
+      it('should NOT auto-remove when the filter value is a non-empty array', () => {
+        expect(autoRemove(['a'])).toBe(false)
+      })
+    })
+  })
 })