Skip to content

ci: fix publishing npm requirements - #288

Merged
BrandonStalnaker merged 1 commit into
mainfrom
ci/Trusted-publishing-requires-npm-11.5.1
Jul 23, 2026
Merged

ci: fix publishing npm requirements#288
BrandonStalnaker merged 1 commit into
mainfrom
ci/Trusted-publishing-requires-npm-11.5.1

Conversation

@BrandonStalnaker

Copy link
Copy Markdown
Contributor

Background

  • The release-and-tag job in the Release – Publish workflow failed during the "Update npm" step when running npm install -g npm@latest.
  • npm 12.x now requires Node.js ^22.22.2, ^24.15.0, or >=26.0.0, but the workflow installs Node 20, causing an EBADENGINE error and blocking npm publish.
  • OIDC trusted publishing only requires npm 11.5.1+, so upgrading to the latest npm is unnecessary for this job.

What Has Changed

  • Pin the global npm upgrade in release-publish.yml from npm@latest to npm@11, which is compatible with Node 20 and still meets the trusted publishing requirement.
  • Update the inline comment to note that npm 12 requires Node 22+.

Checklist

  • I have performed a self-review of my own code.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have tested this locally.

Reference Issue (For employees only. Ignore if you are an outside contributor)

@BrandonStalnaker BrandonStalnaker self-assigned this Jul 23, 2026
@BrandonStalnaker
BrandonStalnaker requested a review from a team as a code owner July 23, 2026 20:09
@pr-security-agent

Copy link
Copy Markdown

🔒 Security Scan Complete

No security issues detected

Scanned files against 71 rules. · Reviewed commit: d3d7d064ed

📜 Scan history (1 scan)
Commit Time (UTC) Status Findings
d3d7d06 2026-07-23 20:09:13 ✅ success 0
ℹ️ About PR Security Agent

This scan was performed by PR Security Agent, an automated security review tool for pull requests. See the User Guide for details on severity levels, feedback reactions, and scan configuration.

Reply @pr-security-agent help for the list of commands you can run on this PR.

@BrandonStalnaker
BrandonStalnaker merged commit cd5f70b into main Jul 23, 2026
9 checks passed
@BrandonStalnaker
BrandonStalnaker deleted the ci/Trusted-publishing-requires-npm-11.5.1 branch July 23, 2026 20:14
@rokt-releases rokt-releases Bot mentioned this pull request Jul 23, 2026
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.

2 participants