humanspeak / svelte-headless-table / 26471880821

81%

main: 80%

push

github

perf(addSortBy): memoize per-cell hook + repo-wide lint cleanup (#221)

* chore: clear all trunk lint issues across the repo

Closes the gap between local `trunk check --all` and the CI
trunk-action (which runs in diff-mode against the PR base, so it
passes even when older files carry lint debt). All 283 files now
pass `trunk check --all` clean.

Categories addressed:

- ESLint config gap (docs/eslint.config.mjs)
  `.svelte.ts` / `.svelte.js` files — Svelte 5 typed-runes files —
  were falling through to svelte-eslint-parser without `ts.parser`
  as the inner parser, so generic-call syntax like `$state<T>()`
  threw a parse error. Broadened the existing `**/*.svelte` block
  to also cover the new extensions.

- Missing `{#each ... (key)}` keys (require-each-key, high)
  - SimpleHiddenColumnsDemo.svelte: keyed by `id`
  - SelectFilter.svelte: keyed by `value`

- Duplicate import (no-duplicate-imports, high)
  - SimpleFilteringDemo.svelte: collapse value + type import from
    `@humanspeak/svelte-headless-table/plugins` into one statement

- Unused destructured / let:* bindings on Subscribe components
  - Removed `props={cell.props()} let:props` from <th> blocks that
    only used `attrs`, in 4 demo files
  - Applied `{...rowAttrs}` to `<tr>` in SimpleGroupByDemo.svelte
    (the binding existed; the spread was missing)
  - Dropped unused `rows` from `(_, { rows, pageRows })` destructure
    in SimplePaginationDemo.svelte

- Unused parameter names in interface signatures (store.ts)
  Prefixed with `_` per the project's
  `argsIgnorePattern: '^_'` ESLint convention.

- Useless assignment (no-useless-assignment, high)
  - headerRows.ts:285: `let endIdx = 1` was overwritten by
    `endIdx = startIdx + 1` before any read. Dropped the initializer
    and added a one-line comment explaining where it's first set.

- Trunk + Svelte minor auto-fixes from previous runs
  - .trunk/trunk.yaml: grype/actionlint/oxipng patch bumps
  - app.d.ts: prettier — dropped trai... (continued)

483 of 652 branches covered (74.08%)

Branch coverage included in aggregate %.

22 of 22 new or added lines in 2 files covered. (100.0%)

1342 of 1609 relevant lines covered (83.41%)

1486.18 hits per line