Skip to content

Use Node 22#541

Open
bheesham wants to merge 4 commits into
mozilla-iam:masterfrom
bheesham:node-22
Open

Use Node 22#541
bheesham wants to merge 4 commits into
mozilla-iam:masterfrom
bheesham:node-22

Conversation

@bheesham

@bheesham bheesham commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Auth0 no longer recommends using Node 18.

Jira: IAM-1990

@bheesham bheesham changed the title Node 22 Use Node 22 Jun 8, 2026
@bheesham bheesham marked this pull request as ready for review June 8, 2026 23:32
Jira: IAM-1990

Copilot AI left a comment

Copy link
Copy Markdown

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 updates the repository’s Node.js baseline to 22 (per Auth0 guidance) and aligns local tooling plus CI to use pnpm for dependency management.

Changes:

  • Bump Auth0 Action runtimes in Terraform from Node 18 to Node 22.
  • Update project/tooling to Node 22 and introduce pnpm (packageManager, lockfile, mise config).
  • Update GitHub Actions workflows to run on Node 22 and install dependencies with pnpm.

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tf/actions.tf Switch Auth0 Action runtime values to node22 for post-login actions.
pnpm-workspace.yaml Add pnpm workspace configuration (incl. allowBuilds).
pnpm-lock.yaml Add pnpm lockfile for deterministic dependency resolution under pnpm.
package.json Require Node >=22 <23 and declare packageManager: pnpm@11.5.2.
mise.toml Set local tool versions (Node 22, pnpm 11.5.2) and default package manager to pnpm.
mise.lock Lock resolved tool binaries/checksums for mise-managed Node/pnpm.
.github/workflows/test-rules.yaml Move CI install/test steps from npm to pnpm and drop Node 18 from the matrix.
.github/workflows/lint.yaml Move CI install/lint steps from npm to pnpm.

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

Comment on lines +21 to +23
- uses: pnpm/action-setup@v6
- name: Install dependencies
run: npm ci
run: pnpm i
Comment on lines +21 to +23
- uses: pnpm/action-setup@v6
- name: install
run: npm ci
run: pnpm i

@gcoxmoz gcoxmoz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lines up with what Auth0 wants to happen, so I like it enough to say "let's find out!"

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