Skip to content

Conversation

@kmruiz
Copy link
Contributor

@kmruiz kmruiz commented Dec 1, 2025

With npm 11, some dependencies related to the AWS driver and Kerberos are not loaded properly. It also complains about issues with prettier (where we use 2.x and 3.x), react and other transitive dependencies. The behaviour looks pretty similar to this one mentioned here: npm/cli#8470.

Using Node.js 24 and npm@10 works too, but to avoid temporary setups this PR just downgrades this specific set of tasks that seem to be affected by the upgrade to npm@11.

With npm 11, some dependencies related to the AWS driver
and Kerberos are not loaded properly. It also complains about
issues with prettier (where we use 2.x and 3.x).

While at some point we will need to update all these dependencies,
we will do it with the whole bump to Node.js 24, which requires other
work before that.
Copilot AI review requested due to automatic review settings December 1, 2025 15:26
@kmruiz kmruiz requested a review from a team as a code owner December 1, 2025 15:26
@kmruiz kmruiz requested a review from addaleax December 1, 2025 15:27
Copilot finished reviewing on behalf of kmruiz December 1, 2025 15:28
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR downgrades the Node.js version from 24.x to 20.x specifically for the cron-tasks workflow to work around npm 11 compatibility issues with AWS driver and Kerberos dependencies. Node.js 24.x comes with npm 11 by default, which has known issues loading certain native dependencies. The change is targeted to only affect the automated file generation tasks that run on a schedule or on main branch pushes.

  • Downgrades Node.js from 24.x to 20.x in the cron-tasks GitHub Actions workflow
  • Addresses npm 11 compatibility issues with AWS driver and Kerberos native dependencies
  • Maintains compatibility with project's minimum Node.js requirement (>= 20.19.3)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kmruiz kmruiz added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Dec 1, 2025
@kmruiz kmruiz changed the title chore: Downgrade cron-tasks environment to Node.js 20/npm 10 fix(ci): Downgrade cron-tasks environment to Node.js 20/npm 10 Dec 1, 2025
@addaleax addaleax changed the title fix(ci): Downgrade cron-tasks environment to Node.js 20/npm 10 fix(ci): downgrade cron-tasks environment to Node.js 20/npm 10 Dec 1, 2025
@kmruiz kmruiz merged commit f7a69db into main Dec 1, 2025
162 of 166 checks passed
@kmruiz kmruiz deleted the fix/ci-third-party-updates branch December 1, 2025 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants