Skip to content

chore: modernize build tooling and packaging#451

Open
tleunen wants to merge 1 commit into
masterfrom
chore/modernize-build-tooling
Open

chore: modernize build tooling and packaging#451
tleunen wants to merge 1 commit into
masterfrom
chore/modernize-build-tooling

Conversation

@tleunen

@tleunen tleunen commented Mar 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • migrate to vite-plus, publishing dist/index.mjs and dist/index.cjs
  • replace ESLint and Prettier config with oxlint and oxfmt, and refresh CI/docs around the new workflow
  • remove glob, pkg-up, and reselect, add local package.json discovery for cwd: "packagejson", and expand transformed test helpers to cover vi.*

Testing

  • npm run test
  • npm run build
  • npm run pack --dry-run
  • npm run lint

@tleunen tleunen force-pushed the chore/modernize-build-tooling branch 3 times, most recently from 6a823a6 to 1f9c8b4 Compare March 16, 2026 15:29
@tleunen tleunen force-pushed the chore/modernize-build-tooling branch from 1f9c8b4 to 2a63800 Compare March 16, 2026 16:23
@hiSandog

hiSandog commented Jul 2, 2026

Copy link
Copy Markdown

The packaging modernization is useful, but the CI matrix has a suspicious mismatch: it runs Node 22.x, 24.x, and 25.x, while the Codecov step checks matrix.node-version == '26.x'. That condition will never be true, so coverage upload is effectively disabled. Since this PR also changes the package to publish ESM/CJS builds, it would be worth adding a pack/install smoke test that imports both entrypoints from the generated tarball.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants