chore: require Node 22+ and bump @types/node to 24#2
Merged
Conversation
- engines.node >=20 → >=22 (upstream pnpm 11.x dropped Node 20; uses node:sqlite available in Node 22.13+ only) - @types/node ^20 → ^24 (LTS line; do not chase 25 current) - CI workflows: node-version matrix simplified to ["22"] Pre-1.0 breaking change. Bundled into the commander 14 PR so the whole breaking surface ships as one release. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
4 tasks
Mirror the workspace pattern (already in siblings): - Add `packageManager: "pnpm@10.32.1"` so workflows resolve consistently - Bump engines.pnpm >=9 → >=10 - Drop `version: ">=9.0.0"` from pnpm/action-setup blocks so they read from packageManager instead Prevents accidental resolution to pnpm 11.x, which dropped support for `pnpm.onlyBuiltDependencies` and requires Node 22.13+ in its sqlite store.
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
1 participant
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
Raises the minimum Node version from 20 to 22 (
engines.node >=22.0.0) and aligns@types/nodeto^24(LTS line).Why
pnpm 11.x dropped Node 20 support — it uses
node:sqlite, available only in Node 22.13+. CI was failing on the matrix Node 20 leg across the workspace. With current download counts low, this is the right window to bump.Changes
engines.node:>=20.0.0→>=22.0.0@types/node:^20.x→^24(LTS line; not chasing 25 current)node-versionmatrix simplified to["22"]Verification
pnpm typecheck && pnpm lint && pnpm test && pnpm build— all green (65 unit tests)Breaking change
Yes: drops Node 20 support. Pre-1.0; acceptable per maintainer policy now that upstream is forcing the issue.
Related
Parallel PRs landing the same bump in sibling repos:
fhir-test-data(bundled into its open commander 14 PR)fhir-resource-diff(bundled into its open commander 14 PR)Test plan
🤖 Generated with Claude Code