Skip to content

ci: configure npm auth and pass GITHUB_TOKEN for unstable publish#14772

Merged
soberm merged 1 commit intomainfrom
fix/unstable-release-npm-auth
Apr 9, 2026
Merged

ci: configure npm auth and pass GITHUB_TOKEN for unstable publish#14772
soberm merged 1 commit intomainfrom
fix/unstable-release-npm-auth

Conversation

@soberm
Copy link
Copy Markdown
Contributor

@soberm soberm commented Apr 9, 2026

Description of changes

changeset publish calls npm publish under the hood, and npm requires auth credentials in ~/.npmrc. Previously, changesets/action handled writing the npm token to ~/.npmrc automatically. When the switch to a direct run: step was made, that setup was lost — NPM_TOKEN was passed as an env var, but npm doesn't read auth from environment variables.

This PR adds a dedicated step to write the npm auth token to ~/.npmrc before publishing, matching the pattern used by the existing npm-publish composite action in this repo (.github/actions/npm-publish/action.yml).

Issue #, if available

Fixes the Release Unstable workflow ENEEDAUTH failure: https://github.com/aws-amplify/amplify-js/actions/runs/24179532870/job/70568727701

Description of how you validated changes

Checklist

  • PR description included
  • yarn test passes
  • Unit Tests are changed or added
  • Relevant documentation is changed or added (and PR referenced)

Checklist for repo maintainers

  • Verify E2E tests for existing workflows are working as expected or add E2E tests for newly added workflows
  • New source file paths included in this PR have been added to CODEOWNERS, if appropriate

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 9, 2026

⚠️ No Changeset found

Latest commit: bd5cd8e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@soberm soberm marked this pull request as ready for review April 9, 2026 14:44
@soberm soberm requested a review from a team as a code owner April 9, 2026 14:44
@soberm soberm merged commit 8968564 into main Apr 9, 2026
42 checks passed
@soberm soberm deleted the fix/unstable-release-npm-auth branch April 9, 2026 14:52
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