Skip to content

build: require Node.js 24.2 or newer - #23

Merged
kodiakhq[bot] merged 2 commits into
mainfrom
build/node-24-2
Aug 10, 2026
Merged

build: require Node.js 24.2 or newer#23
kodiakhq[bot] merged 2 commits into
mainfrom
build/node-24-2

Conversation

@RedStar071

@RedStar071RedStar071 commented Aug 10, 2026

Copy link
Copy Markdown
Member

Raises the project's Node.js floor from 22.18 to 24.2 everywhere the version is declared, so the engines gate, local toolchain pins, CI, and the emitted bundles all agree:

"engines": {
"node": ">=24.2"
}

mise.toml and .github/actions/setup-toolchain now pin Node 24.19.0 (the latest 24.x release), the shared tsdown config emits for node24.2 instead of node20.11, and README, CONTRIBUTING, and AGENTS.md state the new requirement (CLAUDE.md and GEMINI.md are symlinks to AGENTS.md). @types/node was already on ^24.

No source changes: packages keep building the same code, just without down-leveling below the supported runtime.


View with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is enabled.

Confidence Score: 5/5

Safe to merge based on successful dual-format build and runtime loading checks with the changed tsdown configuration.

The changed configuration completed a Node 24.2-targeted ESM/CommonJS build and both generated module formats executed successfully.

Files Needing Attention: No files need follow-up attention; scripts/tsdown.config.ts was exercised directly.

T-Rex T-Rex Logs

What T-Rex did

  • Ran an isolated tsdown 0.22.14 dual-format build targeting Node 24.2 with failOnWarn enabled, first with the legacy-CJS check enabled and it produced a legacy-CJS error and exit code 1.
  • Re-ran the build after disabling the legacy-CJS check (checks.legacyCjs: false), and tsdown produced ESM, CommonJS, and declaration outputs; importing the ESM output and requiring the CommonJS output both returned 42.
  • Attempted a direct repository dependency install for a workspace build, but the configured aube@1.38.0 package manager executable was unavailable.
  • Reviewed the PR diff and verified the after-state shows a successful multi-format package setup with matching configuration; the reproduction source is included for verification.

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (1): Last reviewed commit: "build: keep dual CJS output past tsdown'..." | Re-trigger Greptile

RedStar071and others added 2 commits August 10, 2026 07:50
Raises the runtime floor from 22.18 to 24.2: engines, the mise and CI toolchain pins (Node 24.19.0, latest 24.x), the shared tsdown bundle target, and the contributor docs.
Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
@kodiakhq
kodiakhqBot merged commit 202a519 into mainAug 10, 2026
14 checks passed
@kodiakhq
kodiakhqBot deleted the build/node-24-2 branch August 10, 2026 08:00
Sign up for freeto 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.

1 participant

@RedStar071