Skip to content

ci: fix OIDC publish — drop npm@latest engine break#825

Open
omegent-app[bot] wants to merge 1 commit into
mainfrom
fix/oidc-npm-node-engine
Open

ci: fix OIDC publish — drop npm@latest engine break#825
omegent-app[bot] wants to merge 1 commit into
mainfrom
fix/oidc-npm-node-engine

Conversation

@omegent-app

@omegent-app omegent-app Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Summary

The Changesets version job failed before publish:

npm error engine Not compatible with your version of node/npm: npm@12.0.1
npm error notsup Required: {"node":"^22.22.2 || ^24.15.0 || >=26.0.0"}
npm error notsup Actual:   {"node":"v24.14.1","npm":"11.11.0"}

npm install -g npm@latest pulled npm 12, which needs Node ≥ 24.15. The runner is on Node 24.14 with npm 11.11.0, which already satisfies OIDC trusted publishing (≥ 11.5.1).

This PR updates .github/release.oidc.yaml (template) to remove that upgrade step.

Maintainer action (bot cannot push workflow files)

After merge, or immediately on main:

# delete these 3 lines from .github/workflows/release.yaml:
#      - name: Ensure npm supports OIDC trusted publishing
#        run: npm install -g npm@latest && npm --version

# or re-copy the fixed template:
cp .github/release.oidc.yaml .github/workflows/release.yaml
git add .github/workflows/release.yaml
git commit -m "ci: remove npm@latest upgrade breaking OIDC publish"
git push

Then re-run the failed Changesets workflow (or wait for the push).

Test plan

  • Live release.yaml no longer runs npm@latest upgrade
  • Version job reaches publish
  • Log shows OIDC path: No NPM_TOKEN found, but OIDC is available - using npm trusted publishing

npm@12 requires Node ^24.15; setup-node 24.14 ships npm 11.11 which
already satisfies trusted publishing (>=11.5.1). Remove the global
upgrade step from the release.oidc.yaml template so the version job
can reach publish after re-copying to workflows/release.yaml.
@pkg-pr-new

pkg-pr-new Bot commented Jul 23, 2026

Copy link
Copy Markdown

Open in StackBlitz

@effect-app/cli

npm i https://pkg.pr.new/effect-app/libs/@effect-app/cli@825

effect-app

npm i https://pkg.pr.new/effect-app/libs/effect-app@825

@effect-app/eslint-codegen-model

npm i https://pkg.pr.new/effect-app/libs/@effect-app/eslint-codegen-model@825

@effect-app/eslint-shared-config

npm i https://pkg.pr.new/effect-app/libs/@effect-app/eslint-shared-config@825

@effect-app/infra

npm i https://pkg.pr.new/effect-app/libs/@effect-app/infra@825

@effect-app/vue

npm i https://pkg.pr.new/effect-app/libs/@effect-app/vue@825

@effect-app/vue-components

npm i https://pkg.pr.new/effect-app/libs/@effect-app/vue-components@825

commit: f74d200

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.

0 participants