@svgrid/grid 2.2.32 → 2.2.33

package/src/editing.ts

@@ -2,126 +2,23 @@
 // reading/writing controller state via the `ctx` handle; the reactive core
 // ($state/$derived/$effect) stays in the controller.
 import {
-  applyExcelFilter,
-  normalizeForFilter,
-  createColumnVirtualizer,
-  createCoreRowModel,
-  createExpandedRowModel,
-  createFilteredRowModel,
-  createGroupedRowModel,
-  createPaginatedRowModel,
-  createSvelteVirtualizer,
-  createSortedRowModel,
-  createSvGrid,
-  filterFns,
-  getGridCellA11yProps,
-  getGridCellDomId,
-  getGridHeaderA11yProps,
-  getGridRootA11yProps,
-  getGridRowA11yProps,
   parseEditorValue,
-  normalizeEditorOptions,
-  sortFns,
-  tableFeatures,
-  rowSortingFeature,
-  columnFilteringFeature,
-  columnGroupingFeature,
   type CellContext,
-  type EditorContext,
-  type CellEditorOption,
   type CellEditorType,
-  type CellFormatter,
-  type CellFormatConfig,
   type Column,
-  type ColumnDef,
   type Row,
   type RowData,
-  type SvGridApi,
   type TableFeatures,
 } from "./index";
 import "./sv-grid-scrollbar";
-import type { Snippet } from "svelte";
-import { getKeyboardIntent, getNextActiveCell } from "./keyboard";
+import { getNextActiveCell } from "./keyboard";
 import {
-  formatNumericWithConfig,
-  getDateFormatter,
-  resolveDatePattern,
-} from "./cell-formatting";
-import {
-  RenderSnippetConfig,
-  RenderComponentConfig,
-} from "./render-component";
-import { buildFillPattern } from "./fill-patterns";
-import { buildSparkline, toSparklineValues } from "./sparkline";
-import {
-  resolveCellFormat,
-  computeColumnStat,
-  formatsNeedingStats,
-  type ColumnStat,
-  type ResolvedCellFormat,
-} from "./conditional-formatting";
-import SvGridDropdown from "./SvGridDropdown.svelte";
-import type {
-  Props,
-  SelectionPoint,
-  SelectionRange,
-  CellEditState,
-  FilterOperator,
-  FilterOption,
-  MenuPosition,
-} from "./SvGrid.types";
-import {
-  cfTextStyle,
-  fmtStat,
   getCellKey,
-  resolveClassList,
-  toDateInputValue,
-  toDateTimeLocalInputValue,
-  getEditableInputValue,
-  getEditorInputType,
   toValueArray,
-  getOptionLabel,
-  getOptionColor,
-  colorfulChipStyle,
-  getEditorClass,
-  asDate,
-  clampMenuX,
-  cssEscape,
-  rawToNumber,
-  formatFacetNumber,
-  formatFacetDate,
 } from "./SvGrid.helpers";
-import { createSelection } from "./selection";
-import { createColumns } from "./columns";
-import { createGridApi } from "./build-api";
-import { createClipboard } from "./clipboard";
-import {
-  filterOperatorOptions,
-  fallbackOperatorOption,
-  TEXT_OPERATORS,
-  NUMBER_OPERATORS,
-  DATE_OPERATORS,
-  CHECKBOX_OPERATORS,
-  operatorOption,
-  operatorsForColumn,
-  defaultOperatorFor,
-  operatorLabelFor,
-} from "./filter-operators";
-import {
-  type FacetBucket,
-  isBucketableColumn,
-  buildBuckets,
-  isInBucket,
-} from "./facet-buckets";
 import {
   getColumnBaseValue,
   isGroupRow,
-  toolPanelHeaderLabel,
-  formatSummaryNumeric,
-  getColumnAlign,
-  getPinnedCellValue,
-  getColumnAccessorValue,
-  columnDefMatchesId,
 } from "./cell-values";
 
 type HistoryStep = {
@@ -224,7 +121,7 @@ export function createEditing<
     ) {
       return false;
     }
-    if (editorType === "number" && !/[0-9.+\-]/.test(char)) return false;
+    if (editorType === "number" && !/[0-9.+-]/.test(char)) return false;
     ctx.editorSelectAll = false;
     ctx.editingCell = {
       rowId: row.id,

package/src/editor-registry.grid.test.ts

@@ -73,7 +73,6 @@ afterEach(() => {
 
 describe('registry wiring in SvGrid', () => {
   it('mounts a registered component when a custom-editorType cell edits', async () => {
-    let committed: unknown = undefined
     registerCellEditor('stars', {
       component: SvRating,
       props: (ctx) => ({
@@ -90,7 +89,6 @@ describe('registry wiring in SvGrid', () => {
       // The registered SvRating editor is mounted in the editing overlay.
       const rating = target.querySelector('.sv-rating')
       expect(rating).not.toBeNull()
-      void committed
     } finally {
       destroy()
     }

package/src/export-format.ts

@@ -521,6 +521,9 @@ const XML_ESCAPE: Record<string, string> = {
 }
 function xmlText(value: unknown): string {
   const s = value == null ? '' : value instanceof Date ? value.toISOString() : String(value)
+  // Stripping control characters is the point: XML 1.0 forbids them outright,
+  // so a stray \x00 from a data source would produce an unopenable file.
+  // eslint-disable-next-line no-control-regex
   return s.replace(/[\x00-\x08\x0B\x0C\x0E-\x1F]/g, '').replace(/[&<>"']/g, (c) => XML_ESCAPE[c]!)
 }
 /** Coerce a field name into a valid XML element name. */

package/src/features.ts

@@ -2,104 +2,14 @@
 // reading/writing controller state via the `ctx` handle; the reactive core
 // ($state/$derived/$effect) stays in the controller.
 import {
-    applyExcelFilter,
-    normalizeForFilter,
-    createColumnVirtualizer,
-    createCoreRowModel,
-    createExpandedRowModel,
-    createFilteredRowModel,
-    createGroupedRowModel,
-    createSvelteVirtualizer,
-    createSortedRowModel,
-    createSvGrid,
-    getGridCellDomId,
-    sortFns,
     tableFeatures,
     rowSortingFeature,
     columnFilteringFeature,
     columnGroupingFeature,
-    type CellEditorOption,
-    type Column,
-    type ColumnDef,
-    type Row,
     type RowData,
     type TableFeatures,
   } from "./index";
 import "./sv-grid-scrollbar";
-import {
-    computeColumnStat,
-    formatsNeedingStats,
-    type ColumnStat,
-  } from "./conditional-formatting";
-import SvGridDropdown from "./SvGridDropdown.svelte";
-import type {
-    Props,
-    SelectionRange,
-    CellEditState,
-    FilterOperator,
-    MenuPosition,
-  } from "./SvGrid.types";
-import {
-    rawToNumber,
-  } from "./SvGrid.helpers";
-import {
-    createScrollSync,
-  } from "./scroll-sync";
-import {
-    createKeyboard,
-  } from "./keyboard-handlers";
-import {
-    createSummaries,
-  } from "./summaries";
-import {
-    createMenus,
-  } from "./menus";
-import {
-    createCellRender,
-  } from "./cell-render";
-import {
-    createEditing,
-  } from "./editing";
-import {
-    createSelection,
-  } from "./selection";
-import {
-    createColumns,
-  } from "./columns";
-import {
-    createGridApi,
-  } from "./build-api";
-import {
-    createClipboard,
-  } from "./clipboard";
-import {
-    filterOperatorOptions,
-    fallbackOperatorOption,
-    TEXT_OPERATORS,
-    NUMBER_OPERATORS,
-    DATE_OPERATORS,
-    CHECKBOX_OPERATORS,
-    operatorOption,
-    operatorsForColumn,
-    defaultOperatorFor,
-    operatorLabelFor,
-  } from "./filter-operators";
-import {
-    type FacetBucket,
-    isBucketableColumn,
-    buildBuckets,
-    isInBucket,
-  } from "./facet-buckets";
-import {
-    getColumnBaseValue,
-    isGroupRow,
-    toolPanelHeaderLabel,
-    formatSummaryNumeric,
-    getColumnAlign,
-    getPinnedCellValue,
-    getColumnAccessorValue,
-    columnDefMatchesId,
-  } from "./cell-values";
 
 export function createFeatures<
   TFeatures extends TableFeatures = TableFeatures,

package/src/keyboard-handlers.coverage.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 import { createKeyboard } from './keyboard-handlers'
 
 // createKeyboard returns imperative handlers that read/write a controller

package/src/keyboard-handlers.ts

@@ -2,130 +2,10 @@
 // reading/writing controller state via the `ctx` handle; the reactive core
 // ($state/$derived/$effect) stays in the controller.
 import {
-  applyExcelFilter,
-  normalizeForFilter,
-  createColumnVirtualizer,
-  createCoreRowModel,
-  createExpandedRowModel,
-  createFilteredRowModel,
-  createGroupedRowModel,
-  createPaginatedRowModel,
-  createSvelteVirtualizer,
-  createSortedRowModel,
-  createSvGrid,
-  filterFns,
-  getGridCellA11yProps,
-  getGridCellDomId,
-  getGridHeaderA11yProps,
-  getGridRootA11yProps,
-  getGridRowA11yProps,
-  parseEditorValue,
-  normalizeEditorOptions,
-  sortFns,
-  tableFeatures,
-  rowSortingFeature,
-  columnFilteringFeature,
-  columnGroupingFeature,
-  type CellContext,
-  type EditorContext,
-  type CellEditorOption,
-  type CellEditorType,
-  type CellFormatter,
-  type CellFormatConfig,
-  type Column,
-  type ColumnDef,
-  type Row,
   type RowData,
-  type SvGridApi,
   type TableFeatures,
 } from "./index";
 import "./sv-grid-scrollbar";
-import type { Snippet } from "svelte";
-import {
-  formatNumericWithConfig,
-  getDateFormatter,
-  resolveDatePattern,
-} from "./cell-formatting";
-import {
-  RenderSnippetConfig,
-  RenderComponentConfig,
-} from "./render-component";
-import { buildFillPattern } from "./fill-patterns";
-import { buildSparkline, toSparklineValues } from "./sparkline";
-import {
-  resolveCellFormat,
-  computeColumnStat,
-  formatsNeedingStats,
-  type ColumnStat,
-  type ResolvedCellFormat,
-} from "./conditional-formatting";
-import SvGridDropdown from "./SvGridDropdown.svelte";
-import type {
-  Props,
-  SelectionPoint,
-  SelectionRange,
-  CellEditState,
-  FilterOperator,
-  FilterOption,
-  MenuPosition,
-} from "./SvGrid.types";
-import {
-  cfTextStyle,
-  fmtStat,
-  getCellKey,
-  resolveClassList,
-  toDateInputValue,
-  toDateTimeLocalInputValue,
-  getEditableInputValue,
-  getEditorInputType,
-  toValueArray,
-  getOptionLabel,
-  getOptionColor,
-  colorfulChipStyle,
-  getEditorClass,
-  asDate,
-  clampMenuX,
-  cssEscape,
-  rawToNumber,
-  formatFacetNumber,
-  formatFacetDate,
-} from "./SvGrid.helpers";
-import { createSummaries } from "./summaries";
-import { createMenus } from "./menus";
-import { createCellRender } from "./cell-render";
-import { createEditing } from "./editing";
-import { createSelection } from "./selection";
-import { createColumns } from "./columns";
-import { createGridApi } from "./build-api";
-import { createClipboard } from "./clipboard";
-import {
-  filterOperatorOptions,
-  fallbackOperatorOption,
-  TEXT_OPERATORS,
-  NUMBER_OPERATORS,
-  DATE_OPERATORS,
-  CHECKBOX_OPERATORS,
-  operatorOption,
-  operatorsForColumn,
-  defaultOperatorFor,
-  operatorLabelFor,
-} from "./filter-operators";
-import {
-  type FacetBucket,
-  isBucketableColumn,
-  buildBuckets,
-  isInBucket,
-} from "./facet-buckets";
-import {
-  getColumnBaseValue,
-  isGroupRow,
-  toolPanelHeaderLabel,
-  formatSummaryNumeric,
-  getColumnAlign,
-  getPinnedCellValue,
-  getColumnAccessorValue,
-  columnDefMatchesId,
-} from "./cell-values";
 
 import { getKeyboardIntent, getNextActiveCell } from "./keyboard";
 

package/src/menus.test.ts

@@ -597,7 +597,6 @@ describe('createMenus - contextMenuItems', () => {
       internalData: [rowA, rowB],
       contextMenuFor: { rowIndex: 1, colIndex: 0, columnId: 'name', rowId: 'r1', row: rowB },
     }))
-    const ctx2 = (m2 as any)
     const removeItem = m2.contextMenuItems().find((i) => i.key === 'remove_row')!
     removeItem.run!()
   })

package/src/menus.ts

@@ -2,131 +2,19 @@
 // reading/writing controller state via the `ctx` handle; the reactive core
 // ($state/$derived/$effect) stays in the controller.
 import {
-  applyExcelFilter,
-  normalizeForFilter,
-  createColumnVirtualizer,
-  createCoreRowModel,
-  createExpandedRowModel,
-  createFilteredRowModel,
-  createGroupedRowModel,
-  createPaginatedRowModel,
-  createSvelteVirtualizer,
-  createSortedRowModel,
-  createSvGrid,
-  filterFns,
-  getGridCellA11yProps,
-  getGridCellDomId,
-  getGridHeaderA11yProps,
-  getGridRootA11yProps,
-  getGridRowA11yProps,
-  parseEditorValue,
-  normalizeEditorOptions,
-  sortFns,
-  tableFeatures,
-  rowSortingFeature,
-  columnFilteringFeature,
-  columnGroupingFeature,
-  type CellContext,
-  type EditorContext,
-  type CellEditorOption,
-  type CellEditorType,
-  type CellFormatter,
-  type CellFormatConfig,
-  type Column,
-  type ColumnDef,
-  type Row,
   type RowData,
-  type SvGridApi,
   type TableFeatures,
 } from "./index";
 import "./sv-grid-scrollbar";
 import { untrack } from "svelte";
-import type { Snippet } from "svelte";
-import { getKeyboardIntent, getNextActiveCell } from "./keyboard";
-import {
-  formatNumericWithConfig,
-  getDateFormatter,
-  resolveDatePattern,
-} from "./cell-formatting";
-import {
-  RenderSnippetConfig,
-  RenderComponentConfig,
-} from "./render-component";
-import { buildFillPattern } from "./fill-patterns";
-import { buildSparkline, toSparklineValues } from "./sparkline";
-import {
-  resolveCellFormat,
-  computeColumnStat,
-  formatsNeedingStats,
-  type ColumnStat,
-  type ResolvedCellFormat,
-} from "./conditional-formatting";
-import SvGridDropdown from "./SvGridDropdown.svelte";
 import type {
-  Props,
-  SelectionPoint,
-  SelectionRange,
-  CellEditState,
   FilterOperator,
-  FilterOption,
-  MenuPosition,
 } from "./SvGrid.types";
 import {
-  cfTextStyle,
-  fmtStat,
-  getCellKey,
-  resolveClassList,
-  toDateInputValue,
-  toDateTimeLocalInputValue,
-  getEditableInputValue,
-  getEditorInputType,
-  toValueArray,
-  getOptionLabel,
-  getOptionColor,
-  colorfulChipStyle,
-  getEditorClass,
-  asDate,
   clampMenuX,
   cssEscape,
-  rawToNumber,
-  formatFacetNumber,
-  formatFacetDate,
 } from "./SvGrid.helpers";
-import { createCellRender } from "./cell-render";
-import { createEditing } from "./editing";
-import { createSelection } from "./selection";
-import { createColumns } from "./columns";
-import { createGridApi } from "./build-api";
-import { createClipboard } from "./clipboard";
-import {
-  filterOperatorOptions,
-  fallbackOperatorOption,
-  TEXT_OPERATORS,
-  NUMBER_OPERATORS,
-  DATE_OPERATORS,
-  CHECKBOX_OPERATORS,
-  operatorOption,
-  operatorsForColumn,
-  defaultOperatorFor,
-  operatorLabelFor,
-} from "./filter-operators";
-import {
-  type FacetBucket,
-  isBucketableColumn,
-  buildBuckets,
-  isInBucket,
-} from "./facet-buckets";
 import { splitInTokens, joinInTokens } from "./filtering/excel-filters";
-import {
-  getColumnBaseValue,
-  isGroupRow,
-  toolPanelHeaderLabel,
-  formatSummaryNumeric,
-  getColumnAlign,
-  getPinnedCellValue,
-  getColumnAccessorValue,
-  columnDefMatchesId,
-} from "./cell-values";
 
 export function createMenus<
   TFeatures extends TableFeatures = TableFeatures,

package/src/new-features.test.ts

@@ -58,7 +58,8 @@ describe('createSvGrid - getRowId', () => {
 
   it('passes the row index to getRowId', () => {
     const seen: Array<{ id: string; index: number }> = []
-    createSvGrid<typeof features, Row>({
+    // The row model is lazy, so it has to be read for getRowId to run at all.
+    const rows = createSvGrid<typeof features, Row>({
       _features: features,
       _rowModels: { coreRowModel: createCoreRowModel<Row>() },
       columns,
@@ -68,6 +69,7 @@ describe('createSvGrid - getRowId', () => {
         return row.id
       },
     }).getRowModel().rows
+    expect(rows).toHaveLength(data.length)
     expect(seen).toEqual([
       { id: 'ada',   index: 0 },
       { id: 'linus', index: 1 },

package/src/row-drag.test.ts

@@ -6,7 +6,7 @@
  * Because the bus is shared, we can create TWO fake ctxs (two grids) and drive
  * a genuine grid-to-grid move without mounting anything.
  */
-import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 import { createRowDrag, rowDropZone } from './row-drag'
 
 type AnyCtx = Record<string, any>

package/src/row-resize.test.ts

@@ -1,6 +1,5 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 import { rowResize } from './row-resize'
-import type { RowResizeOptions } from './row-resize'
 
 const STRIP_CLASS = 'sv-grid-row-resize-handle'
 
@@ -247,7 +246,7 @@ describe('rowResize - drag', () => {
   })
 
   it('ignores pointermove / pointerup when no drag is active', () => {
-    const { host, rows } = buildGrid([{ rowIndex: 0 }])
+    const { host } = buildGrid([{ rowIndex: 0 }])
     track(host)
     const onResize = vi.fn()
     const onResizeMove = vi.fn()

package/src/scroll-sync.test.ts

@@ -102,7 +102,7 @@ describe('showTooltipFor / hideTooltip', () => {
   })
 
   it('clears a pending timer before scheduling a new tooltip', () => {
-    const { ctx, ss } = syncFor()
+    const { ss } = syncFor()
     const clearSpy = vi.spyOn(window, 'clearTimeout')
     const el = anchorEl({ left: 10, right: 30, top: 10, bottom: 30 })
     ss.showTooltipFor(el, 'first')
@@ -204,13 +204,10 @@ describe('scheduleScrollSync / flushScheduledScrollSync', () => {
 })
 
 describe('onBodyScroll', () => {
-  let rafCb: FrameRequestCallback | null = null
   beforeEach(() => {
-    rafCb = null
-    vi.stubGlobal('requestAnimationFrame', (cb: FrameRequestCallback) => {
-      rafCb = cb
-      return 7
-    })
+    // These tests assert what happens BEFORE the frame runs, so the stub just
+    // hands back a handle and never invokes the callback.
+    vi.stubGlobal('requestAnimationFrame', () => 7)
   })
   afterEach(() => {
     vi.unstubAllGlobals()