Skip to content

Conversation

@risantos
Copy link
Member

@risantos risantos commented Jan 5, 2026

Description

Copilot AI review requested due to automatic review settings January 5, 2026 13:55
@risantos risantos self-assigned this Jan 5, 2026
@risantos risantos requested review from Copilot and diogotorres97 and removed request for Copilot January 5, 2026 13:57
Copy link

Copilot AI left a comment

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 release workflow to use OIDC (OpenID Connect) trusted publishing for npm instead of token-based authentication. While the goal is appropriate and aligns with modern security best practices, there are several critical issues that will prevent the workflow from functioning.

Key Changes:

  • Removes manual npm token configuration in favor of OIDC authentication
  • Updates GitHub Actions versions and Node.js version
  • Adds id-token: write permission for OIDC
  • Adds skipChecks: true to release-it configuration
  • Improves concurrency configuration from concurrency: 1 to a group-based approach

Reviewed changes

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

File Description
.release-it.js Adds skipChecks: true to bypass npm authentication pre-flight checks, necessary for OIDC publishing
.github/workflows/release.yaml Removes npm token authentication step, adds OIDC permissions, updates action versions, improves concurrency configuration, and updates Node.js version

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

@risantos risantos force-pushed the support/update-release branch from 9254844 to f7f5147 Compare January 5, 2026 14:00
@risantos risantos force-pushed the support/update-release branch from f7f5147 to 40ef74e Compare January 5, 2026 14:04
@diogotorres97 diogotorres97 merged commit dcecebe into master Jan 5, 2026
1 check passed
@diogotorres97 diogotorres97 deleted the support/update-release branch January 5, 2026 14:19
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