@dashforge/tw 0.6.0-beta → 0.8.0-beta

package/src/components/_shared/data/useVirtualizer.ts

@@ -0,0 +1,199 @@
+import { useCallback, useEffect, useLayoutEffect, useRef, useState } from 'react';
+
+/**
+ * Window descriptor — which slice of the row array is currently
+ * mounted in DOM, and how much spacer height to render above and
+ * below the visible window so the scrollbar size matches the
+ * "virtual" total height.
+ */
+export interface VirtualizerWindow {
+  /** First row index inside the visible window (inclusive). */
+  startIndex: number;
+  /** Last row index inside the visible window (inclusive). */
+  endIndex: number;
+  /** Pixels to render above the visible rows (spacer top). */
+  paddingTop: number;
+  /** Pixels to render below the visible rows (spacer bottom). */
+  paddingBottom: number;
+}
+
+export interface UseVirtualizerArgs {
+  /** Total number of rows in the underlying data set. */
+  totalCount: number;
+  /** Fixed row height in pixels. */
+  rowHeight: number;
+  /**
+   * Visible viewport height in pixels. Pass the height of the
+   * scroll container.
+   */
+  viewportHeight: number;
+  /** Current scrollTop of the scroll container. */
+  scrollTop: number;
+  /**
+   * How many extra rows to render above / below the visible window
+   * for smoother scrolling (reduces blank-flash on fast scroll).
+   * @default 5
+   */
+  overscan?: number;
+}
+
+/**
+ * Compute the visible window of rows + the spacer heights given the
+ * current scroll position. Pure function — exported for testability.
+ *
+ * @example
+ * ```ts
+ * computeWindow({ totalCount: 1000, rowHeight: 48, viewportHeight: 600,
+ *                 scrollTop: 0, overscan: 5 });
+ * // → { startIndex: 0, endIndex: 17, paddingTop: 0, paddingBottom: 47136 }
+ * ```
+ */
+export function computeWindow(args: UseVirtualizerArgs): VirtualizerWindow {
+  const { totalCount, rowHeight, viewportHeight, scrollTop, overscan = 5 } = args;
+
+  if (totalCount <= 0 || rowHeight <= 0 || viewportHeight <= 0) {
+    return { startIndex: 0, endIndex: -1, paddingTop: 0, paddingBottom: 0 };
+  }
+
+  // Raw visible range from scrollTop alone.
+  const rawStart = Math.floor(scrollTop / rowHeight);
+  const rawEnd = Math.ceil((scrollTop + viewportHeight) / rowHeight);
+
+  // Apply overscan + clamp to [0, totalCount - 1].
+  const startIndex = Math.max(0, rawStart - overscan);
+  const endIndex = Math.min(totalCount - 1, rawEnd + overscan);
+
+  const paddingTop = startIndex * rowHeight;
+  const paddingBottom = Math.max(0, (totalCount - 1 - endIndex) * rowHeight);
+
+  return { startIndex, endIndex, paddingTop, paddingBottom };
+}
+
+/**
+ * `useVirtualizer` — viewport-based row windowing for `<DataGrid>`.
+ *
+ * Hand-rolled (no `@tanstack/react-virtual` dep — constraint memorized
+ * in `feedback_dashforge_preset_is_identity` companion: "no new
+ * external runtime deps"). Uses scroll event + `requestAnimationFrame`
+ * debounce + `ResizeObserver` on the container. Fixed row height
+ * only in v1 — variable-height with measurement cache deferred to
+ * v1-bis.
+ *
+ * Returns:
+ *  - `scrollRef`: attach to the scroll container `<div>`.
+ *  - `window`: which row indices to mount + top/bottom spacer heights
+ *    to preserve scroll-area size.
+ *  - `viewportHeight`: current container height (re-measured on
+ *    `ResizeObserver` ticks).
+ *
+ * The DataGrid renders `rows.slice(window.startIndex, window.endIndex + 1)`
+ * between two spacer `<tr style={{ height: paddingTop }} />` elements
+ * (above and below). This preserves `<table>` semantics — screen
+ * readers see a continuous structure with `aria-hidden` spacers.
+ *
+ * @example
+ * ```tsx
+ * const { scrollRef, window } = useVirtualizer({
+ *   totalCount: rows.length,
+ *   rowHeight: 48,
+ *   overscan: 5,
+ * });
+ *
+ * return (
+ *   <div ref={scrollRef} style={{ height: 600, overflow: 'auto' }}>
+ *     <table>
+ *       <tbody>
+ *         <tr aria-hidden="true" style={{ height: window.paddingTop }} />
+ *         {rows.slice(window.startIndex, window.endIndex + 1).map(...)}
+ *         <tr aria-hidden="true" style={{ height: window.paddingBottom }} />
+ *       </tbody>
+ *     </table>
+ *   </div>
+ * );
+ * ```
+ */
+export function useVirtualizer(args: {
+  totalCount: number;
+  rowHeight: number;
+  overscan?: number;
+  /**
+   * Initial viewport height — used before the ResizeObserver
+   * settles on first paint. Pass an explicit `height` prop value
+   * if you have one; otherwise the hook re-measures on layout.
+   * @default 400
+   */
+  initialViewportHeight?: number;
+}): {
+  scrollRef: React.RefObject<HTMLDivElement | null>;
+  window: VirtualizerWindow;
+  viewportHeight: number;
+} {
+  const { totalCount, rowHeight, overscan = 5, initialViewportHeight = 400 } = args;
+
+  const scrollRef = useRef<HTMLDivElement | null>(null);
+  const [scrollTop, setScrollTop] = useState(0);
+  const [viewportHeight, setViewportHeight] = useState(initialViewportHeight);
+
+  // Debounce scroll handler via requestAnimationFrame so we never
+  // re-render more than once per paint frame even when the scroll
+  // event fires rapidly during fast scrolling.
+  const rafIdRef = useRef<number | null>(null);
+  const handleScroll = useCallback(() => {
+    if (rafIdRef.current != null) return;
+    rafIdRef.current = requestAnimationFrame(() => {
+      rafIdRef.current = null;
+      const el = scrollRef.current;
+      if (!el) return;
+      setScrollTop(el.scrollTop);
+    });
+  }, []);
+
+  // Attach scroll listener.
+  useEffect(() => {
+    const el = scrollRef.current;
+    if (!el) return;
+    el.addEventListener('scroll', handleScroll, { passive: true });
+    return () => {
+      el.removeEventListener('scroll', handleScroll);
+      if (rafIdRef.current != null) {
+        cancelAnimationFrame(rafIdRef.current);
+        rafIdRef.current = null;
+      }
+    };
+  }, [handleScroll]);
+
+  // Track container viewport height via ResizeObserver — re-virtualize
+  // when the container size changes (window resize, sidebar collapse,
+  // etc.). Use `useLayoutEffect` so the first measurement happens
+  // before paint to avoid a flash of stale window.
+  //
+  // **Defensive**: only accept measurements > 0. jsdom (test env)
+  // returns `clientHeight === 0` for unmounted containers, which would
+  // collapse the window to empty. The hook keeps the
+  // `initialViewportHeight` fallback in that case.
+  useLayoutEffect(() => {
+    const el = scrollRef.current;
+    if (!el) return;
+    if (el.clientHeight > 0) {
+      setViewportHeight(el.clientHeight);
+    }
+    if (typeof ResizeObserver === 'undefined') return;
+    const observer = new ResizeObserver((entries) => {
+      const entry = entries[0];
+      if (!entry || entry.contentRect.height <= 0) return;
+      setViewportHeight(entry.contentRect.height);
+    });
+    observer.observe(el);
+    return () => observer.disconnect();
+  }, []);
+
+  const window = computeWindow({
+    totalCount,
+    rowHeight,
+    viewportHeight,
+    scrollTop,
+    overscan,
+  });
+
+  return { scrollRef, window, viewportHeight };
+}

package/src/components/_shared/themeIdentity.test.ts

@@ -0,0 +1,113 @@
+import { readFileSync, readdirSync } from 'node:fs';
+import { join } from 'node:path';
+import { describe, it, expect } from 'vitest';
+
+/**
+ * Package-level theme-identity regression test.
+ *
+ * Enforces the Dashforge rule
+ * (memory: `feedback_dashforge_preset_is_identity`):
+ * the `@dashforge/tw-theme` default preset auto-inverts the neutral
+ * palette via a CSS-variable swap (`bg-neutral-50` is light surface
+ * in light mode AND dark surface in dark mode — same class, different
+ * physical color). Adding a `dark:` Tailwind variant on a
+ * `neutral-*` class creates a DOUBLE inversion that breaks dark mode.
+ *
+ * The Sprint 4.1 Table fix established the rule; Sprint 4.3 applies
+ * it to the whole catalog and adds this test as a regression guard.
+ *
+ * **Allowed exception** (Category B in `THEME-AUDIT.md`): the pattern
+ * `bg-white dark:bg-neutral-N` / `fill-white dark:fill-neutral-N` is
+ * LEGITIMATE — `bg-white` is hardcoded (no auto-inversion via CSS
+ * vars) so a `dark:` counterpart IS needed to get a dark elevated
+ * surface in dark mode. The regex below strips these legitimate
+ * patterns before scanning for the anti-pattern.
+ *
+ * Color palettes (`primary` / `secondary` / `success` / `warning` /
+ * `danger` / `info`) are EXEMPT from this rule by design — they
+ * do not auto-invert, so `dark:` shifts on them are legitimate
+ * design choices (tone refinement across modes).
+ */
+const COMPONENTS_DIR = join(__dirname, '..');
+
+// The legitimate Category B patterns we allow:
+//  1. `bg-white dark:bg-neutral-N`
+//  2. `fill-white dark:fill-neutral-N`
+//  3. variant-prefixed versions, e.g.
+//     `data-[state=active]:bg-white dark:data-[state=active]:bg-neutral-N`
+//
+// Strip these from the source before scanning for the anti-pattern.
+const CATEGORY_B_RE =
+  /(?:[\w[\]:-]+:)?(?:bg|fill)-white\s+dark:(?:[\w[\]:-]+:)*(?:bg|fill)-neutral-\d+/g;
+
+// Anti-pattern: any `dark:` variant on a `neutral-N` class.
+// Covers `bg`, `text`, `border`, `ring`, `placeholder:text`,
+// `hover:bg`, `focus-visible:bg`, `data-[state=*]:bg`, etc.
+const NEUTRAL_DARK_RE =
+  /dark:(?:[\w[\]:-]+:)*(?:bg|text|border|ring|fill|placeholder|focus-visible|hover|focus|data)?-?neutral-\d+/;
+
+function walkSourceFiles(dir: string): string[] {
+  const entries = readdirSync(dir, { withFileTypes: true });
+  const out: string[] = [];
+  for (const entry of entries) {
+    const full = join(dir, entry.name);
+    if (entry.isDirectory()) {
+      out.push(...walkSourceFiles(full));
+      continue;
+    }
+    if (!entry.isFile()) continue;
+    if (entry.name.endsWith('.test.ts')) continue;
+    if (entry.name.endsWith('.test.tsx')) continue;
+    if (entry.name.endsWith('.spec.ts')) continue;
+    if (entry.name.endsWith('.spec.tsx')) continue;
+    if (!/\.(ts|tsx)$/.test(entry.name)) continue;
+    out.push(full);
+  }
+  return out;
+}
+
+describe('Catalog — theme identity rule (no `dark:` on neutral palette)', () => {
+  const files = walkSourceFiles(COMPONENTS_DIR);
+
+  it('finds the catalog source files', () => {
+    expect(files.length).toBeGreaterThan(20);
+  });
+
+  for (const file of files) {
+    const rel = file.replace(COMPONENTS_DIR + '/', '');
+    it(`${rel} — no \`dark:*-neutral-N\` anti-pattern`, () => {
+      const source = readFileSync(file, 'utf-8');
+      const stripped = source
+        // 1. Strip legitimate Category B patterns (bg-white / fill-white
+        //    paired with a `dark:` neutral target — bg-white doesn't
+        //    auto-invert, so the `dark:` is intentional).
+        .replace(CATEGORY_B_RE, '')
+        // 2. Strip comments — the regex examples in THEME-AUDIT and
+        //    this test source itself contain the anti-pattern as text.
+        .replace(/\/\/.*$/gm, '')
+        .replace(/\/\*[\s\S]*?\*\//g, '');
+
+      const match = stripped.match(NEUTRAL_DARK_RE);
+      if (match) {
+        const idx = stripped.indexOf(match[0]);
+        const context = stripped.slice(
+          Math.max(0, idx - 80),
+          idx + match[0].length + 80,
+        );
+        throw new Error(
+          `Found \`dark:*-neutral-N\` anti-pattern in ${rel}:\n` +
+            `  ...${context.replace(/\s+/g, ' ').trim()}...\n\n` +
+            `Rule: dashforgePreset() auto-inverts neutral palette via CSS-var swap. ` +
+            `Adding \`dark:bg-neutral-N\` (or similar) creates double inversion ` +
+            `and breaks dark mode.\n\n` +
+            `Fix: drop the \`dark:\` variant — the base class auto-inverts.\n\n` +
+            `Exception: \`bg-white dark:bg-neutral-N\` IS allowed (bg-white doesn't ` +
+            `auto-invert, the dark variant is required for proper dark elevation).\n\n` +
+            `See: libs/dashforge/tw/THEME-AUDIT.md, ` +
+            `memory \`feedback_dashforge_preset_is_identity\`.`,
+        );
+      }
+      expect(match).toBeNull();
+    });
+  }
+});

package/src/index.ts

@@ -294,6 +294,8 @@ export type {
   TableSortModel,
   TableFilterItem,
   TableFilterModel,
+  TableFilterOperator,
+  TableFilterType,
   TableRowSelectionMode,
   TableExpandableConfig,
   TableLabels,
@@ -319,7 +321,23 @@ export type {
 } from './components/Table/cells/index.js';
 // Helpers (exported for power users building custom Table-like
 // components on top of the same primitives).
-export { getNestedValue } from './components/Table/_internal/getNestedValue.js';
+export { getNestedValue } from './components/_shared/data/getNestedValue.js';
+
+// F14 tier-5: virtualized data grid for large data sets. Companion to
+// `<Table>` — same column model, sort/search/filter/selection logic
+// (shared via `_shared/data/` helpers) but uses homemade
+// virtualization (`IntersectionObserver` + scroll-event +
+// `requestAnimationFrame` — zero new runtime deps).
+
+export { DataGrid } from './components/DataGrid/DataGrid.js';
+export type {
+  DataGridProps,
+  DataGridSlotProps,
+  DataGridSelectAllScope,
+  DataGridServerSideFlags,
+  DataGridPaginationConfig,
+} from './components/DataGrid/dataGrid.types.js';
+export { dataGridVariants } from './components/DataGrid/dataGrid.variants.js';
 
 // ───── Hooks ─────
 export { useAccessState } from './hooks/useAccessState.js';
@@ -335,4 +353,4 @@ export type { VariantProps } from 'tailwind-variants';
 /**
  * Package version (synced with `package.json` at publish time).
  */
-export const VERSION = '0.6.0-beta';
+export const VERSION = '0.7.0-beta';

package/dist/src/components/Table/_internal/getNestedValue.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"getNestedValue.d.ts","sourceRoot":"","sources":["../../../../../src/components/Table/_internal/getNestedValue.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,cAAc,CAAC,CAAC,SAAS,MAAM,EAC7C,GAAG,EAAE,CAAC,EACN,IAAI,EAAE,MAAM,GACX,OAAO,CAUT"}

package/dist/src/components/Table/_internal/useColumnAutoDetect.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"useColumnAutoDetect.d.ts","sourceRoot":"","sources":["../../../../../src/components/Table/_internal/useColumnAutoDetect.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,WAAW,EAAE,uBAAuB,EAAE,MAAM,mBAAmB,CAAC;AAE9E;;;;;;;;;;GAUG;AACH,wBAAgB,mBAAmB,CAAC,CAAC,SAAS,MAAM,EAClD,IAAI,EAAE,CAAC,EAAE,EACT,IAAI,EAAE,WAAW,CAAC,CAAC,CAAC,EAAE,EACtB,UAAU,SAAK,GACd,GAAG,CAAC,MAAM,EAAE,uBAAuB,CAAC,CAiBtC;AAcD;;;GAGG;AACH,wBAAgB,YAAY,CAC1B,GAAG,EAAE;IAAE,KAAK,CAAC,EAAE,MAAM,GAAG,OAAO,GAAG,QAAQ,CAAA;CAAE,EAC5C,QAAQ,EAAE,uBAAuB,GAAG,SAAS,GAC5C,MAAM,GAAG,OAAO,GAAG,QAAQ,CAU7B;AAED;;;;;;;GAOG;AACH,wBAAgB,kBAAkB,CAChC,GAAG,EAAE;IAAE,WAAW,CAAC,EAAE,OAAO,CAAA;CAAE,EAC9B,QAAQ,EAAE,uBAAuB,GAAG,SAAS,GAC5C,OAAO,CAGT;AAED;;;;;;;;;GASG;AACH,wBAAgB,gBAAgB,CAC9B,GAAG,EAAE;IAAE,SAAS,CAAC,EAAE,OAAO,CAAA;CAAE,GAC3B,OAAO,CAET"}

package/dist/src/components/Table/_internal/useControllableState.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"useControllableState.d.ts","sourceRoot":"","sources":["../../../../../src/components/Table/_internal/useControllableState.ts"],"names":[],"mappings":"AAEA;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,oBAAoB,CAAC,CAAC,EAAE,IAAI,EAAE;IAC5C,eAAe,EAAE,CAAC,GAAG,SAAS,CAAC;IAC/B,YAAY,EAAE,CAAC,CAAC;IAChB,QAAQ,CAAC,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,IAAI,CAAC;CAC/B,GAAG,CAAC,CAAC,EAAE,CAAC,IAAI,EAAE,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,CAAC,KAAK,IAAI,CAAC,CAwB5C"}

package/dist/src/components/Table/_internal/useDebouncedValue.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"useDebouncedValue.d.ts","sourceRoot":"","sources":["../../../../../src/components/Table/_internal/useDebouncedValue.ts"],"names":[],"mappings":"AAEA;;;;;;;GAOG;AACH,wBAAgB,iBAAiB,CAAC,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,OAAO,EAAE,MAAM,GAAG,CAAC,CAajE"}

package/dist/src/components/Table/_internal/useTableFilter.d.ts

@@ -1,13 +0,0 @@
-import type { TableFilterModel } from '../table.types.js';
-/**
- * Filters `rows` by the per-column filter model. v1 only supports
- * the `contains` operator (text substring) — additional ops
- * (`equals`, `in`, `gte`, `lte`, `between`, `before`, `after`)
- * land in v1-bis once the filter UI surface for each shape is
- * fleshed out.
- *
- * AND semantics across columns: a row must match EVERY active
- * filter. Within a single filter, the op decides the predicate.
- */
-export declare function useTableFilter<T extends object>(rows: T[], filterModel: TableFilterModel): T[];
-//# sourceMappingURL=useTableFilter.d.ts.map

package/dist/src/components/Table/_internal/useTableFilter.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"useTableFilter.d.ts","sourceRoot":"","sources":["../../../../../src/components/Table/_internal/useTableFilter.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,mBAAmB,CAAC;AAE1D;;;;;;;;;GASG;AACH,wBAAgB,cAAc,CAAC,CAAC,SAAS,MAAM,EAC7C,IAAI,EAAE,CAAC,EAAE,EACT,WAAW,EAAE,gBAAgB,GAC5B,CAAC,EAAE,CAkBL"}

package/dist/src/components/Table/_internal/useTableSearch.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"useTableSearch.d.ts","sourceRoot":"","sources":["../../../../../src/components/Table/_internal/useTableSearch.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAErD;;;;;;;;;;GAUG;AACH,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,CAqBzD;AAED;;;;;;;GAOG;AACH,wBAAgB,cAAc,CAAC,CAAC,SAAS,MAAM,EAC7C,IAAI,EAAE,CAAC,EAAE,EACT,IAAI,EAAE,WAAW,CAAC,CAAC,CAAC,EAAE,EACtB,KAAK,EAAE,MAAM,GACZ,CAAC,EAAE,CAeL"}

package/dist/src/components/Table/_internal/useTableSelection.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"useTableSelection.d.ts","sourceRoot":"","sources":["../../../../../src/components/Table/_internal/useTableSelection.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,mBAAmB,CAAC;AAE/D,MAAM,WAAW,uBAAuB;IACtC,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,UAAU,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,OAAO,CAAC;IACvC,SAAS,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,IAAI,CAAC;IACnC,SAAS,EAAE,CAAC,SAAS,EAAE,MAAM,EAAE,KAAK,IAAI,CAAC;IACzC,cAAc,EAAE,MAAM,IAAI,CAAC;IAC3B,qDAAqD;IACrD,aAAa,EAAE,CAAC,SAAS,EAAE,MAAM,EAAE,KAAK,OAAO,CAAC;IAChD,uEAAuE;IACvE,eAAe,EAAE,CAAC,SAAS,EAAE,MAAM,EAAE,KAAK,OAAO,CAAC;CACnD;AAED;;;;;;;;;;GAUG;AACH,wBAAgB,iBAAiB,CAC/B,IAAI,EAAE,qBAAqB,EAC3B,cAAc,EAAE,MAAM,EAAE,EACxB,iBAAiB,EAAE,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,CAAC,CAAC,IAAI,EAAE,MAAM,EAAE,KAAK,MAAM,EAAE,CAAC,KAAK,IAAI,GAC3E,uBAAuB,CAqEzB"}

package/dist/src/components/Table/_internal/useTableSort.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"useTableSort.d.ts","sourceRoot":"","sources":["../../../../../src/components/Table/_internal/useTableSort.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EACV,WAAW,EACX,cAAc,EAEf,MAAM,mBAAmB,CAAC;AA+C3B;;;;;;;;GAQG;AACH,wBAAgB,YAAY,CAAC,CAAC,SAAS,MAAM,EAC3C,IAAI,EAAE,CAAC,EAAE,EACT,IAAI,EAAE,WAAW,CAAC,CAAC,CAAC,EAAE,EACtB,SAAS,EAAE,cAAc,GACxB,CAAC,EAAE,CA2CL;AAED;;;;;;GAMG;AACH,wBAAgB,YAAY,CAC1B,OAAO,EAAE,cAAc,EACvB,KAAK,EAAE,MAAM,EACb,KAAK,EAAE,OAAO,GACb,cAAc,CAkBhB"}

package/src/components/Table/_internal/themeIdentity.test.ts

@@ -1,84 +0,0 @@
-import { readFileSync, readdirSync } from 'node:fs';
-import { join } from 'node:path';
-import { describe, it, expect } from 'vitest';
-
-/**
- * Theme-identity regression test.
- *
- * Enforces the Dashforge rule (memory: `feedback_dashforge_preset_is_identity`):
- * the `@dashforge/tw-theme` default preset auto-inverts the neutral
- * palette via a CSS-variable swap. Adding `dark:` Tailwind variants
- * on `neutral-*` classes creates a DOUBLE inversion that breaks
- * dark mode (both classes resolve to the dark value).
- *
- * This test scans Table source files for the anti-pattern and fails
- * if any are introduced. Color palettes (`primary`, `secondary`,
- * `success`, `warning`, `danger`, `info`) are EXEMPT — they do not
- * auto-invert, so `dark:` variants on them are a legitimate design
- * choice (e.g. RenderChip color rows).
- */
-const TABLE_DIR = join(__dirname, '..');
-
-/**
- * Match `dark:<modifier>?:bg-neutral-X` / `text-neutral-X` /
- * `border-neutral-X` / `placeholder:text-neutral-X` / `ring-neutral-X` etc.
- *
- * The `(?:[a-z-]+:)*` chunk eats arbitrary intermediate variants
- * (e.g. `dark:hover:bg-neutral-...`, `dark:placeholder:text-neutral-...`).
- */
-const NEUTRAL_DARK_RE =
-  /dark:(?:[a-z-]+:)*(?:bg|text|border|ring|placeholder:text|focus-visible:bg|hover:bg|hover:text|focus:bg)?-?neutral-\d+/;
-
-function walkSourceFiles(dir: string): string[] {
-  const entries = readdirSync(dir, { withFileTypes: true });
-  const out: string[] = [];
-  for (const entry of entries) {
-    const full = join(dir, entry.name);
-    if (entry.isDirectory()) {
-      out.push(...walkSourceFiles(full));
-      continue;
-    }
-    if (!entry.isFile()) continue;
-    if (entry.name.endsWith('.test.ts')) continue;
-    if (entry.name.endsWith('.test.tsx')) continue;
-    if (!/\.(ts|tsx)$/.test(entry.name)) continue;
-    out.push(full);
-  }
-  return out;
-}
-
-describe('Table — theme identity rule (no `dark:` on neutral palette)', () => {
-  const files = walkSourceFiles(TABLE_DIR).filter(
-    // The test file itself contains the regex example — exclude.
-    (f) => !f.endsWith('themeIdentity.test.ts'),
-  );
-
-  it('finds at least the expected source files', () => {
-    expect(files.length).toBeGreaterThan(5);
-  });
-
-  for (const file of files) {
-    const rel = file.replace(TABLE_DIR + '/', '');
-    it(`${rel} — contains no \`dark:*-neutral-N\` classes`, () => {
-      const source = readFileSync(file, 'utf-8');
-      // Strip line comments to avoid false positives in docstrings
-      const stripped = source
-        .replace(/\/\/.*$/gm, '')
-        .replace(/\/\*[\s\S]*?\*\//g, '');
-      const match = stripped.match(NEUTRAL_DARK_RE);
-      if (match) {
-        // Surface the offending excerpt to make the failure actionable.
-        const idx = stripped.indexOf(match[0]);
-        const context = stripped.slice(Math.max(0, idx - 60), idx + match[0].length + 60);
-        throw new Error(
-          `Found \`dark:\`-on-neutral anti-pattern in ${rel}:\n  ...${context}...\n` +
-            `Rule: dashforgePreset() auto-inverts neutral palette via CSS-var swap. ` +
-            `Adding \`dark:bg-neutral-N\` (etc.) creates double inversion and breaks dark mode.\n` +
-            `Fix: drop the \`dark:\` variant — the same class works in both modes.\n` +
-            `See memory: \`feedback_dashforge_preset_is_identity\`.`,
-        );
-      }
-      expect(match).toBeNull();
-    });
-  }
-});

package/src/components/Table/_internal/useTableFilter.ts

@@ -1,37 +0,0 @@
-import { useMemo } from 'react';
-import { getNestedValue } from './getNestedValue.js';
-import { stringifyForSearch } from './useTableSearch.js';
-import type { TableFilterModel } from '../table.types.js';
-
-/**
- * Filters `rows` by the per-column filter model. v1 only supports
- * the `contains` operator (text substring) — additional ops
- * (`equals`, `in`, `gte`, `lte`, `between`, `before`, `after`)
- * land in v1-bis once the filter UI surface for each shape is
- * fleshed out.
- *
- * AND semantics across columns: a row must match EVERY active
- * filter. Within a single filter, the op decides the predicate.
- */
-export function useTableFilter<T extends object>(
-  rows: T[],
-  filterModel: TableFilterModel,
-): T[] {
-  return useMemo(() => {
-    if (!filterModel || filterModel.length === 0) return rows;
-    return rows.filter((row) =>
-      filterModel.every((filter) => {
-        const value = getNestedValue(row, filter.field);
-        const stringValue = stringifyForSearch(value).toLowerCase();
-        const needle = filter.value.trim().toLowerCase();
-        if (!needle) return true; // empty filter == no-op
-        switch (filter.op) {
-          case 'contains':
-            return stringValue.includes(needle);
-          default:
-            return true;
-        }
-      }),
-    );
-  }, [rows, filterModel]);
-}