Skip to content

Conversation

@rfgamaral
Copy link
Member

Overview

This PR updates the repository's GitHub Actions workflow to use npm's Trusted Publishing feature for package releases. Using Trusted Publishing eliminates the need to store long-lived npm tokens in GitHub secrets, reducing security risks and simplifying credential management. This also standardizes the publishing process across repositories.

Important

The npm organization and repository must be linked and authorized for Trusted Publishing before merging.

What's changing:

  • Replaces manual NPM_TOKEN authentication with GitHub's OpenID Connect (OIDC)–based authentication.
  • Updates the release workflow configuration to align with npm's Trusted Publishers documentation
  • Ensures that package publishing permissions are managed directly through GitHub and npm, improving security and maintainability.

@rfgamaral rfgamaral self-assigned this Nov 18, 2025
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@rfgamaral rfgamaral force-pushed the ricardo/secure-release-workflow branch from e95096a to e472dfc Compare November 18, 2025 18:04
@rfgamaral rfgamaral merged commit e8fa777 into main Nov 18, 2025
2 checks passed
@rfgamaral rfgamaral deleted the ricardo/secure-release-workflow branch November 18, 2025 18:06
"sideEffects": false,
"engines": {
"node": ">=20.0.0"
"node": ">=22.0.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Won't this require a breaking change release?

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, damn 😔 I was on autopilot, and the previous repos were already on v22 that I didn't even think about it. v6.1.8 was already published, let me push a v7 with an empty commit.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks for the nudge, Scott. I guess I don't deserve a donation too 😅

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