Skip to content

Use npm Trusted Publisher auth in Release workflow - #2

Merged
Priestch merged 3 commits into
mainfrom
copilot/fix-publish-to-npm-error
May 11, 2026
Merged

Use npm Trusted Publisher auth in Release workflow#2
Priestch merged 3 commits into
mainfrom
copilot/fix-publish-to-npm-error

Conversation

Copilot AI commented May 10, 2026

Copy link
Copy Markdown
Contributor
  • Confirm current workflow and branch state for the Trusted Publisher change
  • Run targeted validation/tests relevant to the workflow change
  • Check recent Actions runs/logs for release-related status
  • Reply to the PR question with concrete test results

Copilot AI changed the title [WIP] Fix the publish to npm error in release.yml github action Fix npm publish auth in Release workflow May 10, 2026
Copilot AI requested a review from Priestch May 10, 2026 07:44
@Priestch

Copy link
Copy Markdown
Owner

@copilot I already setup Trusted Publisher for the github action, why still a token needed? what's the point of trusted publisher?

Copilot AI commented May 10, 2026

Copy link
Copy Markdown
Contributor Author

@copilot I already setup Trusted Publisher for the github action, why still a token needed? what's the point of trusted publisher?

You’re right — with npm Trusted Publisher, a long-lived NPM_TOKEN should not be required. The point of Trusted Publisher is to use GitHub OIDC (id-token: write) to get short-lived publish credentials at runtime, so there’s no secret token to manage/rotate. I’ve updated the workflow to remove token env usage in commit eae2e2d so release publish uses the trusted publisher flow.

Copilot AI changed the title Fix npm publish auth in Release workflow Use npm Trusted Publisher auth in Release workflow May 10, 2026
@Priestch

Copy link
Copy Markdown
Owner

@copilot Did you tested?

Copilot AI commented May 11, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Did you tested?

Yes — I tested what can be tested from this PR branch. I validated .github/workflows/release.yml syntax, ran npm ci, and ran npm run test:routes (it fails in this environment with fetch errors after dev server start, not related to the release workflow edit). I also checked the failed Release run logs: the publish failure shown there is E404 Not Found on package publish, not missing npm auth. The Trusted Publisher change itself is in eae2e2d and needs the next main release run to fully verify end-to-end publish.

@Priestch
Priestch marked this pull request as ready for review May 11, 2026 12:11
@Priestch
Priestch merged commit 8e188bf into main May 11, 2026
1 of 2 checks passed
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.

Fix the publish to npm error in release.yml github action.

2 participants