mpyw / axios-case-converter / 29545995904

93%

master: 93%

Pull #85

github

chore: harden v2 packaging/CI per Fable+Codex audit

Follow-up fixes from the post-v2 modernization audit:

- Reproducible installs: commit package-lock.json (stop gitignoring it),
  switch CI and release to `npm ci`, add `cache: npm`.
- Publish safety: build via `prepack` so `npm pack`/`npm publish` always
  regenerate dist (prepublishOnly now only gates check/lint/test). Verified by
  removing dist and confirming `npm pack` rebuilds it.
- engines/CI alignment: bump engines to Node >=20 (18/20-era floor) and add
  Node 20 to the CI matrix so the minimum is actually tested.
- ESLint: ignore stale build dirs (es/lib/typings) so `eslint .` doesn't choke
  on leftover v1 output; mark navigator/global read-only (overridden via
  defineProperty, never assigned).
- Tests: snapshot FormData/URLSearchParams prototype methods in beforeEach and
  restore them in afterEach, so a mid-test assertion failure can't leak a
  mutated native prototype into later tests.
- exports: add "./package.json" subpath.
- vitest: keep built-in excludes via configDefaults.exclude.
- Add .prettierignore (dist/coverage/package-lock.json).
- README: note the v2 build is ES2021 ESM and legacy targets need transpilation.

require(ESM) interop on Node >=20.19/22.12 already works via the existing
`default` condition (CJS `require()` returns { default: fn }); no extra
condition needed.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

88 of 98 branches covered (89.8%)

Branch coverage included in aggregate %.

100 of 104 relevant lines covered (96.15%)

143.23 hits per line