Skip to content

Conversation

@jwndlng
Copy link
Member

@jwndlng jwndlng commented Oct 24, 2025

Motivation

NPM package publishing will be migrated to use Trusted Publishing (via OIDC).

Changes

  • Adds id-token permissions for JWT creation
  • Removes NPM token
  • Pins external action ncipollo/release-action to commit SHA

## Motivation
NPM package publishing will be migrated to use Trusted Publishing (via OIDC).
## Changes
* Pins node version to 24
* Removes NPM token
* Pins external action to commit SHA
@jwndlng jwndlng requested a review from a team as a code owner October 24, 2025 14:17
Copilot AI review requested due to automatic review settings October 24, 2025 14:17
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 migrates NPM package publishing from token-based authentication to OIDC-based Trusted Publishing by updating the GitHub Actions workflow configuration.

Key Changes:

  • Added OIDC permissions and Node.js 24 requirement for npm 11 support
  • Removed NPM_ACCESS_TOKEN usage in favor of OIDC authentication
  • Pinned external action to specific commit SHA for security

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

jwndlng and others added 3 commits October 24, 2025 16:51
Copy link
Member

@ilbertt ilbertt left a comment

Choose a reason for hiding this comment

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

Approving, but let's wait for #191 before merging this one

@ilbertt ilbertt enabled auto-merge October 27, 2025 08:32
@ilbertt ilbertt added this pull request to the merge queue Oct 27, 2025
Merged via the queue into main with commit 28c98d4 Oct 27, 2025
13 checks passed
@ilbertt ilbertt deleted the jwndlng-patch-1 branch October 27, 2025 08:45
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