Skip to content

fix(release): use setup-node v6 with registry-url for OIDC npm auth#390

Merged
mohnoor94 merged 1 commit intomainfrom
fix-release-npm-publish
Mar 19, 2026
Merged

fix(release): use setup-node v6 with registry-url for OIDC npm auth#390
mohnoor94 merged 1 commit intomainfrom
fix-release-npm-publish

Conversation

@mohnoor94
Copy link
Member

@mohnoor94 mohnoor94 commented Mar 19, 2026

Summary

  • Upgrades actions/setup-node from v4 to v6
  • Adds registry-url: https://registry.npmjs.org/ to setup-node config
  • Reverts .releaserc.json to use @semantic-release/npm (plain, no provenance hack)

Why this works

Matches the pattern used by other EG OSS packages (e.g. graphql-component#115):

  • setup-node@v6 + registry-url + id-token: write = OIDC-based npm auth
  • No npm token needed

Previous attempts

All prior PRs (#385-389) failed because we were missing the registry-url config on setup-node, which is what actually enables OIDC auth for npm.

🤖 Generated with Claude Code

@mohnoor94 mohnoor94 requested a review from a team as a code owner March 19, 2026 21:37
Matches the pattern from ExpediaGroup/graphql-component#115:
- Upgrade actions/setup-node from v4 to v6
- Add registry-url: https://registry.npmjs.org/
- setup-node v6 + registry-url + id-token:write handles OIDC auth

Revert .releaserc.json to use @semantic-release/npm since OIDC auth
is now properly configured by setup-node.
@mohnoor94 mohnoor94 force-pushed the fix-release-npm-publish branch from f8d07a2 to 9d3fce0 Compare March 19, 2026 21:39
@mohnoor94 mohnoor94 merged commit 8aedac6 into main Mar 19, 2026
2 checks passed
@mohnoor94 mohnoor94 deleted the fix-release-npm-publish branch March 19, 2026 21:39
mohnoor94 pushed a commit that referenced this pull request Mar 19, 2026
## [2.4.4](v2.4.3...v2.4.4) (2026-03-19)

### Bug Fixes

* **release:** use exec plugin for npm publish to bypass token validation ([#391](#391)) ([bc6f31b](bc6f31b))
* **release:** use setup-node v6 with registry-url for OIDC npm auth ([#390](#390)) ([8aedac6](8aedac6))
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.

1 participant