Skip to content

Pin CI to Node 24.14.1 - #1835

Merged
rdonigian merged 1 commit into
developfrom
pin-ci-node-24.14
May 18, 2026
Merged

Pin CI to Node 24.14.1 #1835
rdonigian merged 1 commit into
developfrom
pin-ci-node-24.14

Conversation

@rdonigian

Copy link
Copy Markdown
Contributor

Summary

@coderabbitai

coderabbitai Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: ae717c29-8c1c-4ac8-839b-8ea24dfc2277

📥 Commits

Reviewing files that changed from the base of the PR and between 06e497d and 822fde1.

📒 Files selected for processing (1)
  • .github/actions/setup/action.yml

📝 Walkthrough

Walkthrough

The GitHub Actions setup composite action pins Node.js from 24 to 24.14.1 to avoid a Yarn PnP CJS-from-zip loader issue in Node 24.15+. Inline comments document the issue and mark this as a temporary workaround.

Changes

Node.js version pinning

Layer / File(s) Summary
Node.js version pin and issue documentation
.github/actions/setup/action.yml
Node.js version is pinned to 24.14.1 with comments documenting the Node 24.15+ / Yarn PnP EBADF incompatibility and marking this as a temporary workaround.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Pin CI to Node 24.14.1' clearly and concisely summarizes the main change: pinning the Node.js version in CI to a specific patch version.
Description check ✅ Passed The description is directly related to the changeset, providing detailed context about the CI failure, root cause analysis, and the rationale for the Node.js version pin.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch pin-ci-node-24.14

Comment @coderabbitai help to get the list of available commands and usage tips.

@rdonigian
rdonigian merged commit 151bd4e into develop May 18, 2026
8 checks passed
@rdonigian
rdonigian deleted the pin-ci-node-24.14 branch May 18, 2026 15:39
rdonigian added a commit that referenced this pull request May 22, 2026
## Summary

- CI started failing on `yarn gql-gen -e` with `EBADF: bad file
descriptor, fstat` after `actions/setup-node` rolled the `24` tag to
**v24.15.0**.
- Root cause is in Yarn PnP, not our code: nodejs/node#61769 (ESM loader
cycle reduction) was backported to Node 24.15.0, but Yarn's
`HAS_BROKEN_FSTAT_FOR_ZIP_FDS` gate in
`yarnpkg-pnp/sources/esm-loader/loaderFlags.ts` only covers `>=25.7`.
Any CJS-from-zip read (e.g. `-r ts-node/register` under PnP) now
crashes. Tracking issue: yarnpkg/berry#7103.
- Pinning
[.github/actions/setup/action.yml](.github/actions/setup/action.yml) to
`24.14.1` (last good patch) until Yarn widens the gate.
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.

3 participants