@svgrid/grid 2.2.32 → 2.4.0

package/src/toast-store.test.ts

@@ -1,147 +1,147 @@
-/**
- * Tests for the headless toast store: queueing, variants, auto-dismiss timers,
- * manual dismiss, sticky toasts, and the SR announcement side effect.
- */
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-import { toast, dismissToast, updateToast, clearToasts, pauseToast, resumeToast, toastStore } from './toast-store.svelte'
-
-beforeEach(() => {
-  vi.useFakeTimers()
-})
-
-afterEach(() => {
-  clearToasts()
-  vi.useRealTimers()
-  document.body.innerHTML = ''
-})
-
-describe('toast store', () => {
-  it('queues a toast with defaults', () => {
-    const id = toast('Hello')
-    expect(toastStore.toasts).toHaveLength(1)
-    const t = toastStore.toasts[0]!
-    expect(t.id).toBe(id)
-    expect(t.message).toBe('Hello')
-    expect(t.variant).toBe('info')
-    expect(t.dismissible).toBe(true)
-  })
-
-  it('variant helpers set the variant', () => {
-    toast.success('ok')
-    toast.error('bad')
-    toast.warning('hmm')
-    expect(toastStore.toasts.map((t) => t.variant)).toEqual(['success', 'error', 'warning'])
-  })
-
-  it('auto-dismisses after its duration', () => {
-    toast('bye', { duration: 3000 })
-    expect(toastStore.toasts).toHaveLength(1)
-    vi.advanceTimersByTime(2999)
-    expect(toastStore.toasts).toHaveLength(1)
-    vi.advanceTimersByTime(1)
-    expect(toastStore.toasts).toHaveLength(0)
-  })
-
-  it('duration 0 is sticky (no auto-dismiss)', () => {
-    toast('stay', { duration: 0 })
-    vi.advanceTimersByTime(60_000)
-    expect(toastStore.toasts).toHaveLength(1)
-  })
-
-  it('dismissToast removes by id and cancels its timer', () => {
-    const id = toast('x', { duration: 5000 })
-    dismissToast(id)
-    expect(toastStore.toasts).toHaveLength(0)
-    // Advancing past the original duration must not throw or double-remove.
-    vi.advanceTimersByTime(5000)
-    expect(toastStore.toasts).toHaveLength(0)
-  })
-
-  it('announces via a live region (assertive for errors)', () => {
-    toast.error('boom')
-    vi.runOnlyPendingTimers()
-    // The store routes errors to the assertive live region.
-    expect(document.querySelector('[aria-live="assertive"]')).not.toBeNull()
-  })
-
-  it('pause banks the remaining time and resume finishes it (hover to keep)', () => {
-    const id = toast('hover me', { duration: 3000 })
-    vi.advanceTimersByTime(1000) // 2000 left
-    pauseToast(id)
-    vi.advanceTimersByTime(10_000) // stays while paused
-    expect(toastStore.toasts).toHaveLength(1)
-    resumeToast(id)
-    vi.advanceTimersByTime(1999)
-    expect(toastStore.toasts).toHaveLength(1)
-    vi.advanceTimersByTime(1) // banked 2000ms elapsed
-    expect(toastStore.toasts).toHaveLength(0)
-  })
-
-  it('clearToasts empties the queue', () => {
-    toast('a'); toast('b'); toast('c')
-    expect(toastStore.toasts).toHaveLength(3)
-    clearToasts()
-    expect(toastStore.toasts).toHaveLength(0)
-  })
-
-  it('update patches a toast in place (same id) and restarts its timer', () => {
-    const id = toast('working', { duration: 0 })
-    updateToast(id, { message: 'done', variant: 'success', duration: 3000 })
-    expect(toastStore.toasts).toHaveLength(1)
-    const t = toastStore.toasts[0]!
-    expect(t.id).toBe(id)
-    expect(t.message).toBe('done')
-    expect(t.variant).toBe('success')
-    // The new duration now applies.
-    vi.advanceTimersByTime(3000)
-    expect(toastStore.toasts).toHaveLength(0)
-  })
-
-  it('update is a no-op for an unknown id', () => {
-    updateToast(999, { message: 'nope' })
-    expect(toastStore.toasts).toHaveLength(0)
-  })
-
-  it('promise shows a sticky loader, then updates in place to success', async () => {
-    vi.useRealTimers() // real microtask flush for the promise
-    let resolve!: (v: string) => void
-    const p = new Promise<string>((r) => (resolve = r))
-    const ret = toast.promise(p, { loading: 'Saving...', success: (v) => `Saved ${v}`, error: 'Failed' })
-    expect(ret).toBe(p) // returns the original promise
-    expect(toastStore.toasts).toHaveLength(1)
-    const id = toastStore.toasts[0]!.id
-    expect(toastStore.toasts[0]!.message).toBe('Saving...')
-    expect(toastStore.toasts[0]!.variant).toBe('info')
-    resolve('now')
-    await p
-    await Promise.resolve()
-    const t = toastStore.toasts.find((x) => x.id === id)!
-    expect(t.message).toBe('Saved now')
-    expect(t.variant).toBe('success')
-  })
-
-  it('promise updates in place to error on rejection', async () => {
-    vi.useRealTimers()
-    const p = Promise.reject(new Error('x'))
-    toast.promise(p, { loading: 'Load', success: 'ok', error: (e) => `Err: ${(e as Error).message}` })
-    await p.catch(() => {})
-    await Promise.resolve()
-    const t = toastStore.toasts[0]!
-    expect(t.variant).toBe('error')
-    expect(t.message).toBe('Err: x')
-  })
-
-  it('action + cancel are stored on the toast', () => {
-    const onClick = vi.fn()
-    toast('Undo?', { action: { label: 'Undo', onClick }, cancel: { label: 'Dismiss' } })
-    const t = toastStore.toasts[0]!
-    expect(t.action?.label).toBe('Undo')
-    expect(t.cancel?.label).toBe('Dismiss')
-  })
-
-  it('custom carries a render snippet', () => {
-    const render = (() => {}) as never
-    const id = toast.custom(render, { duration: 0 })
-    expect(toastStore.toasts.find((x) => x.id === id)!.render).toBe(render)
-  })
-})
+/**
+ * Tests for the headless toast store: queueing, variants, auto-dismiss timers,
+ * manual dismiss, sticky toasts, and the SR announcement side effect.
+ */
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { toast, dismissToast, updateToast, clearToasts, pauseToast, resumeToast, toastStore } from './toast-store.svelte'
+
+beforeEach(() => {
+  vi.useFakeTimers()
+})
+
+afterEach(() => {
+  clearToasts()
+  vi.useRealTimers()
+  document.body.innerHTML = ''
+})
+
+describe('toast store', () => {
+  it('queues a toast with defaults', () => {
+    const id = toast('Hello')
+    expect(toastStore.toasts).toHaveLength(1)
+    const t = toastStore.toasts[0]!
+    expect(t.id).toBe(id)
+    expect(t.message).toBe('Hello')
+    expect(t.variant).toBe('info')
+    expect(t.dismissible).toBe(true)
+  })
+
+  it('variant helpers set the variant', () => {
+    toast.success('ok')
+    toast.error('bad')
+    toast.warning('hmm')
+    expect(toastStore.toasts.map((t) => t.variant)).toEqual(['success', 'error', 'warning'])
+  })
+
+  it('auto-dismisses after its duration', () => {
+    toast('bye', { duration: 3000 })
+    expect(toastStore.toasts).toHaveLength(1)
+    vi.advanceTimersByTime(2999)
+    expect(toastStore.toasts).toHaveLength(1)
+    vi.advanceTimersByTime(1)
+    expect(toastStore.toasts).toHaveLength(0)
+  })
+
+  it('duration 0 is sticky (no auto-dismiss)', () => {
+    toast('stay', { duration: 0 })
+    vi.advanceTimersByTime(60_000)
+    expect(toastStore.toasts).toHaveLength(1)
+  })
+
+  it('dismissToast removes by id and cancels its timer', () => {
+    const id = toast('x', { duration: 5000 })
+    dismissToast(id)
+    expect(toastStore.toasts).toHaveLength(0)
+    // Advancing past the original duration must not throw or double-remove.
+    vi.advanceTimersByTime(5000)
+    expect(toastStore.toasts).toHaveLength(0)
+  })
+
+  it('announces via a live region (assertive for errors)', () => {
+    toast.error('boom')
+    vi.runOnlyPendingTimers()
+    // The store routes errors to the assertive live region.
+    expect(document.querySelector('[aria-live="assertive"]')).not.toBeNull()
+  })
+
+  it('pause banks the remaining time and resume finishes it (hover to keep)', () => {
+    const id = toast('hover me', { duration: 3000 })
+    vi.advanceTimersByTime(1000) // 2000 left
+    pauseToast(id)
+    vi.advanceTimersByTime(10_000) // stays while paused
+    expect(toastStore.toasts).toHaveLength(1)
+    resumeToast(id)
+    vi.advanceTimersByTime(1999)
+    expect(toastStore.toasts).toHaveLength(1)
+    vi.advanceTimersByTime(1) // banked 2000ms elapsed
+    expect(toastStore.toasts).toHaveLength(0)
+  })
+
+  it('clearToasts empties the queue', () => {
+    toast('a'); toast('b'); toast('c')
+    expect(toastStore.toasts).toHaveLength(3)
+    clearToasts()
+    expect(toastStore.toasts).toHaveLength(0)
+  })
+
+  it('update patches a toast in place (same id) and restarts its timer', () => {
+    const id = toast('working', { duration: 0 })
+    updateToast(id, { message: 'done', variant: 'success', duration: 3000 })
+    expect(toastStore.toasts).toHaveLength(1)
+    const t = toastStore.toasts[0]!
+    expect(t.id).toBe(id)
+    expect(t.message).toBe('done')
+    expect(t.variant).toBe('success')
+    // The new duration now applies.
+    vi.advanceTimersByTime(3000)
+    expect(toastStore.toasts).toHaveLength(0)
+  })
+
+  it('update is a no-op for an unknown id', () => {
+    updateToast(999, { message: 'nope' })
+    expect(toastStore.toasts).toHaveLength(0)
+  })
+
+  it('promise shows a sticky loader, then updates in place to success', async () => {
+    vi.useRealTimers() // real microtask flush for the promise
+    let resolve!: (v: string) => void
+    const p = new Promise<string>((r) => (resolve = r))
+    const ret = toast.promise(p, { loading: 'Saving...', success: (v) => `Saved ${v}`, error: 'Failed' })
+    expect(ret).toBe(p) // returns the original promise
+    expect(toastStore.toasts).toHaveLength(1)
+    const id = toastStore.toasts[0]!.id
+    expect(toastStore.toasts[0]!.message).toBe('Saving...')
+    expect(toastStore.toasts[0]!.variant).toBe('info')
+    resolve('now')
+    await p
+    await Promise.resolve()
+    const t = toastStore.toasts.find((x) => x.id === id)!
+    expect(t.message).toBe('Saved now')
+    expect(t.variant).toBe('success')
+  })
+
+  it('promise updates in place to error on rejection', async () => {
+    vi.useRealTimers()
+    const p = Promise.reject(new Error('x'))
+    toast.promise(p, { loading: 'Load', success: 'ok', error: (e) => `Err: ${(e as Error).message}` })
+    await p.catch(() => {})
+    await Promise.resolve()
+    const t = toastStore.toasts[0]!
+    expect(t.variant).toBe('error')
+    expect(t.message).toBe('Err: x')
+  })
+
+  it('action + cancel are stored on the toast', () => {
+    const onClick = vi.fn()
+    toast('Undo?', { action: { label: 'Undo', onClick }, cancel: { label: 'Dismiss' } })
+    const t = toastStore.toasts[0]!
+    expect(t.action?.label).toBe('Undo')
+    expect(t.cancel?.label).toBe('Dismiss')
+  })
+
+  it('custom carries a render snippet', () => {
+    const render = (() => {}) as never
+    const id = toast.custom(render, { duration: 0 })
+    expect(toastStore.toasts.find((x) => x.id === id)!.render).toBe(render)
+  })
+})

package/src/tree-row-model.test.ts

@@ -1,169 +1,168 @@
-import { describe, expect, it } from 'vitest'
-import { createTreeRowModel, flattenTreeData } from './core'
-
-type Node = { id: number; parent: number | null; name: string }
-
-/** Minimal Row stand-ins - the model only reads `original`, `id` and spreads. */
-function rowsFrom(data: Node[]) {
-  return data.map((original, index) => ({
-    id: String(original.id),
-    index,
-    original,
-    depth: 0,
-    getCanExpand: () => false,
-    getIsExpanded: () => false,
-    toggleExpanded: () => {},
-    getIsSelected: () => false,
-    toggleSelected: () => {},
-    getAllCells: () => [],
-    getCellValueByColumnId: (c: string) => (original as never as Record<string, unknown>)[c],
-  })) as never[]
-}
-
-let expandedState: Record<string, boolean> = {}
-const table = {
-  getState: () => ({ expanded: expandedState }),
-  setExpanded: (fn: (p: Record<string, boolean>) => Record<string, boolean>) => {
-    expandedState = fn(expandedState)
-  },
-} as never
-
-const run = (data: Node[], opts = { parentField: 'parent' }) =>
-  createTreeRowModel<never>(opts)({ table, rows: rowsFrom(data) } as never)
-
-const ids = (rows: readonly { id: string }[]) => rows.map((r) => r.id)
-
-describe('createTreeRowModel', () => {
-  const data: Node[] = [
-    { id: 1, parent: null, name: 'root' },
-    { id: 2, parent: 1, name: 'child a' },
-    { id: 3, parent: 1, name: 'child b' },
-    { id: 4, parent: 2, name: 'grandchild' },
-    { id: 5, parent: null, name: 'other root' },
-  ]
-
-  it('nests by parent id and returns only roots', () => {
-    const out = run(data)
-    expect(ids(out)).toEqual(['1', '5'])
-  })
-
-  it('records depth down the chain', () => {
-    const [root] = run(data) as never as Array<{ depth: number; subRows: never[] }>
-    expect(root!.depth).toBe(0)
-    const child = (root!.subRows as never as Array<{ id: string; depth: number; subRows: never[] }>)[0]!
-    expect(child.depth).toBe(1)
-    expect((child.subRows as never as Array<{ depth: number }>)[0]!.depth).toBe(2)
-  })
-
-  it('marks rows with children expandable and leaves not', () => {
-    const [root, other] = run(data) as never as Array<{ getCanExpand: () => boolean }>
-    expect(root!.getCanExpand()).toBe(true)
-    expect(other!.getCanExpand()).toBe(false)
-  })
-
-  it('counts descendants, not just direct children', () => {
-    const [root] = run(data) as never as Array<{ leafCount: number }>
-    // children a + b, plus a's grandchild
-    expect(root!.leafCount).toBe(3)
-  })
-
-  it('tags rows so they are not mistaken for group banners', async () => {
-    const { isGroupRow } = await import('./cell-values')
-    const [root] = run(data) as never as Array<never>
-    expect((root as never as { __treeRow: boolean }).__treeRow).toBe(true)
-    // Expandable, but must NOT render as a full-width banner.
-    expect(isGroupRow(root!)).toBe(false)
-  })
-
-  it('drives expansion through the table state', () => {
-    expandedState = {}
-    const [root] = run(data) as never as Array<{
-      getIsExpanded: () => boolean
-      toggleExpanded: () => void
-    }>
-    expect(root!.getIsExpanded()).toBe(false)
-    root!.toggleExpanded()
-    expect(expandedState['1']).toBe(true)
-  })
-
-  it('promotes a row whose parent is missing rather than dropping it', () => {
-    // Parent 99 does not exist - e.g. the filter removed it. The row must still
-    // be reachable.
-    const orphaned: Node[] = [
-      { id: 1, parent: null, name: 'root' },
-      { id: 2, parent: 99, name: 'orphan' },
-    ]
-    expect(ids(run(orphaned))).toEqual(['1', '2'])
-  })
-
-  it('survives a parent cycle', () => {
-    const cyclic: Node[] = [
-      { id: 1, parent: 2, name: 'a' },
-      { id: 2, parent: 1, name: 'b' },
-    ]
-    // Neither is a root by parent link, so nothing is emitted rather than
-    // recursing forever. The guard is what matters.
-    expect(() => run(cyclic)).not.toThrow()
-  })
-
-  it('treats a self-parenting row as a root', () => {
-    expect(ids(run([{ id: 1, parent: 1, name: 'self' }]))).toEqual(['1'])
-  })
-
-  it('passes an empty set straight through', () => {
-    expect(run([])).toEqual([])
-  })
-
-  it('honours a custom idField', () => {
-    type K = { key: number; parent: number | null }
-    const keyed = [
-      { key: 10, parent: null },
-      { key: 11, parent: 10 },
-    ] as never as Node[]
-    const out = createTreeRowModel<never>({ parentField: 'parent', idField: 'key' })({
-      table,
-      rows: rowsFrom(keyed),
-    } as never) as never as Array<{ subRows: never[] }>
-    expect(out).toHaveLength(1)
-    expect(out[0]!.subRows).toHaveLength(1)
-  })
-})
-
-describe('flattenTreeData', () => {
-  const nested = [
-    { id: 1, name: 'a', kids: [{ id: 2, name: 'a1', kids: [{ id: 3, name: 'a1a' }] }] },
-    { id: 4, name: 'b' },
-  ]
-
-  it('emits children directly after their parent', () => {
-    const flat = flattenTreeData(nested as never, { childrenField: 'kids' })
-    expect(flat.map((r) => (r as never as { id: number }).id)).toEqual([1, 2, 3, 4])
-  })
-
-  it('stamps the parent link the tree model reads', () => {
-    const flat = flattenTreeData(nested as never, { childrenField: 'kids' }) as never as Array<
-      Record<string, unknown>
-    >
-    expect(flat[0]!.__parentId).toBeNull()
-    expect(flat[1]!.__parentId).toBe(1)
-    expect(flat[2]!.__parentId).toBe(2)
-  })
-
-  it('round-trips into createTreeRowModel', () => {
-    const flat = flattenTreeData(nested as never, { childrenField: 'kids' })
-    const out = createTreeRowModel<never>({ parentField: '__parentId' })({
-      table,
-      rows: rowsFrom(flat as never as Node[]),
-    } as never) as never as Array<{ id: string; subRows: never[] }>
-    expect(out.map((r) => r.id)).toEqual(['1', '4'])
-    expect(out[0]!.subRows).toHaveLength(1)
-  })
-
-  it('accepts a custom parent field name', () => {
-    const flat = flattenTreeData(nested as never, {
-      childrenField: 'kids',
-      parentField: 'pid',
-    }) as never as Array<Record<string, unknown>>
-    expect(flat[1]!.pid).toBe(1)
-  })
-})
+import { describe, expect, it } from 'vitest'
+import { createTreeRowModel, flattenTreeData } from './core'
+
+type Node = { id: number; parent: number | null; name: string }
+
+/** Minimal Row stand-ins - the model only reads `original`, `id` and spreads. */
+function rowsFrom(data: Node[]) {
+  return data.map((original, index) => ({
+    id: String(original.id),
+    index,
+    original,
+    depth: 0,
+    getCanExpand: () => false,
+    getIsExpanded: () => false,
+    toggleExpanded: () => {},
+    getIsSelected: () => false,
+    toggleSelected: () => {},
+    getAllCells: () => [],
+    getCellValueByColumnId: (c: string) => (original as never as Record<string, unknown>)[c],
+  })) as never[]
+}
+
+let expandedState: Record<string, boolean> = {}
+const table = {
+  getState: () => ({ expanded: expandedState }),
+  setExpanded: (fn: (p: Record<string, boolean>) => Record<string, boolean>) => {
+    expandedState = fn(expandedState)
+  },
+} as never
+
+const run = (data: Node[], opts = { parentField: 'parent' }) =>
+  createTreeRowModel<never>(opts)({ table, rows: rowsFrom(data) } as never)
+
+const ids = (rows: readonly { id: string }[]) => rows.map((r) => r.id)
+
+describe('createTreeRowModel', () => {
+  const data: Node[] = [
+    { id: 1, parent: null, name: 'root' },
+    { id: 2, parent: 1, name: 'child a' },
+    { id: 3, parent: 1, name: 'child b' },
+    { id: 4, parent: 2, name: 'grandchild' },
+    { id: 5, parent: null, name: 'other root' },
+  ]
+
+  it('nests by parent id and returns only roots', () => {
+    const out = run(data)
+    expect(ids(out)).toEqual(['1', '5'])
+  })
+
+  it('records depth down the chain', () => {
+    const [root] = run(data) as never as Array<{ depth: number; subRows: never[] }>
+    expect(root!.depth).toBe(0)
+    const child = (root!.subRows as never as Array<{ id: string; depth: number; subRows: never[] }>)[0]!
+    expect(child.depth).toBe(1)
+    expect((child.subRows as never as Array<{ depth: number }>)[0]!.depth).toBe(2)
+  })
+
+  it('marks rows with children expandable and leaves not', () => {
+    const [root, other] = run(data) as never as Array<{ getCanExpand: () => boolean }>
+    expect(root!.getCanExpand()).toBe(true)
+    expect(other!.getCanExpand()).toBe(false)
+  })
+
+  it('counts descendants, not just direct children', () => {
+    const [root] = run(data) as never as Array<{ leafCount: number }>
+    // children a + b, plus a's grandchild
+    expect(root!.leafCount).toBe(3)
+  })
+
+  it('tags rows so they are not mistaken for group banners', async () => {
+    const { isGroupRow } = await import('./cell-values')
+    const [root] = run(data) as never as Array<never>
+    expect((root as never as { __treeRow: boolean }).__treeRow).toBe(true)
+    // Expandable, but must NOT render as a full-width banner.
+    expect(isGroupRow(root!)).toBe(false)
+  })
+
+  it('drives expansion through the table state', () => {
+    expandedState = {}
+    const [root] = run(data) as never as Array<{
+      getIsExpanded: () => boolean
+      toggleExpanded: () => void
+    }>
+    expect(root!.getIsExpanded()).toBe(false)
+    root!.toggleExpanded()
+    expect(expandedState['1']).toBe(true)
+  })
+
+  it('promotes a row whose parent is missing rather than dropping it', () => {
+    // Parent 99 does not exist - e.g. the filter removed it. The row must still
+    // be reachable.
+    const orphaned: Node[] = [
+      { id: 1, parent: null, name: 'root' },
+      { id: 2, parent: 99, name: 'orphan' },
+    ]
+    expect(ids(run(orphaned))).toEqual(['1', '2'])
+  })
+
+  it('survives a parent cycle', () => {
+    const cyclic: Node[] = [
+      { id: 1, parent: 2, name: 'a' },
+      { id: 2, parent: 1, name: 'b' },
+    ]
+    // Neither is a root by parent link, so nothing is emitted rather than
+    // recursing forever. The guard is what matters.
+    expect(() => run(cyclic)).not.toThrow()
+  })
+
+  it('treats a self-parenting row as a root', () => {
+    expect(ids(run([{ id: 1, parent: 1, name: 'self' }]))).toEqual(['1'])
+  })
+
+  it('passes an empty set straight through', () => {
+    expect(run([])).toEqual([])
+  })
+
+  it('honours a custom idField', () => {
+    const keyed = [
+      { key: 10, parent: null },
+      { key: 11, parent: 10 },
+    ] as never as Node[]
+    const out = createTreeRowModel<never>({ parentField: 'parent', idField: 'key' })({
+      table,
+      rows: rowsFrom(keyed),
+    } as never) as never as Array<{ subRows: never[] }>
+    expect(out).toHaveLength(1)
+    expect(out[0]!.subRows).toHaveLength(1)
+  })
+})
+
+describe('flattenTreeData', () => {
+  const nested = [
+    { id: 1, name: 'a', kids: [{ id: 2, name: 'a1', kids: [{ id: 3, name: 'a1a' }] }] },
+    { id: 4, name: 'b' },
+  ]
+
+  it('emits children directly after their parent', () => {
+    const flat = flattenTreeData(nested as never, { childrenField: 'kids' })
+    expect(flat.map((r) => (r as never as { id: number }).id)).toEqual([1, 2, 3, 4])
+  })
+
+  it('stamps the parent link the tree model reads', () => {
+    const flat = flattenTreeData(nested as never, { childrenField: 'kids' }) as never as Array<
+      Record<string, unknown>
+    >
+    expect(flat[0]!.__parentId).toBeNull()
+    expect(flat[1]!.__parentId).toBe(1)
+    expect(flat[2]!.__parentId).toBe(2)
+  })
+
+  it('round-trips into createTreeRowModel', () => {
+    const flat = flattenTreeData(nested as never, { childrenField: 'kids' })
+    const out = createTreeRowModel<never>({ parentField: '__parentId' })({
+      table,
+      rows: rowsFrom(flat as never as Node[]),
+    } as never) as never as Array<{ id: string; subRows: never[] }>
+    expect(out.map((r) => r.id)).toEqual(['1', '4'])
+    expect(out[0]!.subRows).toHaveLength(1)
+  })
+
+  it('accepts a custom parent field name', () => {
+    const flat = flattenTreeData(nested as never, {
+      childrenField: 'kids',
+      parentField: 'pid',
+    }) as never as Array<Record<string, unknown>>
+    expect(flat[1]!.pid).toBe(1)
+  })
+})