tselect-npm / schema / 33315135801

100%

push

github

:boom: Modernize toolchain: pnpm, tsdown, Vitest, Biome, TypeScript 7, CI

Eight concerns, one commit — this repository's ruleset allows squash merges
only, so the gitmoji series from `chore/modernize` is recorded here rather
than replayed onto `main`:

  :bug:                 Repo hygiene: fix what ships and stop tracking build artefacts
  :wrench:              Migrate to pnpm
  :heavy_minus_sign:    Drop the @types/ajv stub and demote ajv to a devDependency
  :sparkles:            Add dual ESM + CJS build via tsdown, on TypeScript 7
  :white_check_mark:    Migrate test suite to Vitest
  :rotating_light:      Replace TSLint with Biome
  :memo:                Add LICENSE, keywords and a full README API section
  :construction_worker: Run CI from the org-wide reusable workflow

`:boom:` rather than the branch's `:wrench:` because a squash leaves one
commit to infer the bump from, and `:wrench:` infers as no release at all.
The `bump: major` override is still the documented way to ship this; the
history now says so on its own as well.

The major is forced four times over: `engines.node >=22`; the `exports` map,
which closes deep imports; `enumeration()`'s return type moving from `any`
to `TJSONSchema`; and `ajv` leaving `dependencies`, which closes a
transitive install for anyone who was resolving it through this package
rather than declaring it.

`ajv` turned out never to be imported by `src/` at all — a test-only
dependency declared as a runtime one — so the 6 → 8 upgrade this repo was
scheduled late for costs consumers nothing.

Headline fix: **the entire package was broken in ESM.** All seven utils used
`import * as Lodash from 'lodash'`, and because `makeSchema` →
`cleanOptions` → `Lodash.omit`, every factory would have thrown on the ESM
entry while CJS kept working. lodash is kept; only the import form changed.

Verified against the published 1.0.0 tarball rather than argued: 105
scenarios across all 30 exports, run against both new entry points —... (continued)

68 of 68 branches covered (100.0%)

Branch coverage included in aggregate %.

24 of 24 new or added lines in 11 files covered. (100.0%)

103 of 103 relevant lines covered (100.0%)

16.42 hits per line