feat!: migrate to ESM, Vitest, and ESLint 9 flat config - #921
Merged
Conversation
BREAKING CHANGE: CJS builds removed, packages are ESM-only. - Convert all packages to ESM (type: module) - Replace Jest with Vitest for all test targets - Migrate ESLint to v9 flat config (eslint.config.js) - Build output ESM-only (removed CJS dist/index.js require path) - Upgrade @tanstack/react-query from v4 to v5 - Upgrade @availity/api-axios to v13 - Drop Node 20 support (engines: ^22.0.0 || ^24.0.0) - Bump React to 19.2.7, MUI to 7.3.11, TypeScript to 5.9.3 - Simplify CI caching with built-in yarn cache action + Nx cache - Modernize nx.json configuration - Update husky hooks to use yarn directly - Replace lint-staged nx affected with direct eslint
- Upgrade GitHub Actions: checkout v6, setup-node v6, nx-set-shas v5, cache v5 - Upgrade commitlint action to v6.2.1 (ESM support) - Replace manual yarn cache with built-in setup-node cache - Update CodeQL language to javascript-typescript - Remove deprecated --all flag from nx run-many scripts - Add test:ci script (vitest run --coverage) - Use test:ci in CI workflows - Fix postinstall to use modern husky invocation - Align lint-staged config with other repos
|
You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool. What Enabling Code Scanning Means:
For more information about GitHub Code Scanning, check out the documentation. |
jordan-a-young
force-pushed
the
feature/esm-vitest-migration
branch
from
June 16, 2026 14:49
abab7f6 to
f1064a3
Compare
Contributor
|
🚀 Preview deployed! |
…ration - Update README with Node 24, ESM-only note, yarn v4 reference - Replace VS Code jest-runner extension with vitest.explorer - Update CONTRIBUTING.md test instructions for Vitest - Update nx-generator templates for ESM-only pattern - Fix stale jest comment in mock server - Update upgrading-to-v2 testing section for ESM - Add upgrading-to-v3 migration guide
gregmartDOTin
approved these changes
Jun 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Comprehensive modernization of the build and test infrastructure.
Breaking Changes
"type": "module")^22.0.0 || ^24.0.0Changes
eslint.config.js)dist/index.jsrequire path)nx.jsonconfigurationWhat was tested
yarn build— ✅ 55 projects built successfullyyarn lint— ✅ 53 projects pass (warnings only, 0 errors)yarn test— ✅ 55 projects pass