Repository navigation
ci: drop the unscoped _authToken line that npm 10 rejects - #145
Merged
Merged
Conversation
Since CI moved to Node 22 (5bcd03e), npm rejects a bare _authToken in .npmrc with ERR_INVALID_AUTH, so creating the release PR and publishing both fail at npm whoami. The registry-scoped line written right after it already carries the token. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01S2yW3aohcrkGZ96iyLpv3a
dianaKhortiuk-frontegg
marked this pull request as ready for review
October 29, 2026 11:35
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
npm whoami:_authTokenline in.npmrc. Both workflows write one. On the older Node this job passed; its last success was the 13 October run.onReleaseMerged.yamlwrites the same line, so publishing to npm would fail too.Change
Removes
echo "_authToken=$NPM_TOKEN" >> .npmrcfromonPullRequestMerged.yamlandonReleaseMerged.yaml. The next line in both files already writes the registry-scoped//registry.npmjs.org/:_authToken=$NPM_TOKEN, so npm still gets the token. Ionic Capacitor's workflows write only the scoped line.Testing
I ran
npm whoamilocally on Node 22.22 / npm 10.9 with a dummy token and the same.npmrclines as CI:ERR_INVALID_AUTH, the CI error.E401).The real check is merging this PR: it should run "Create Release Pull Request" and open the release PR for the changes since v1.2.25.
Generated by Claude Code
Note
Low Risk
CI-only
.npmrctweak with no application or runtime behavior change; publishing still uses the same scoped registry token line.Overview
Fixes release automation that broke after CI moved to Node 22 / npm 10: both merge workflows were writing a legacy unscoped
_authTokenline into.npmrc, which npm 10 rejects and causednpm whoamito fail in Create Release Pull Request and would block release publish.This PR removes
echo "_authToken=$NPM_TOKEN" >> .npmrcfromonPullRequestMerged.yamlandonReleaseMerged.yaml. Auth is unchanged in practice—the workflows still set//registry.npmjs.org/:_authToken=$NPM_TOKEN, which is what npm 10 requires.Reviewed by Cursor Bugbot for commit dc74f24. Bugbot is set up for automated code reviews on this repo. Configure here.