Skip to content

ci: write the npm token at release time - #27

Merged
Kikobeats merged 2 commits into
masterfrom
ci/release-token-just-in-time
Aug 25, 2026
Merged

ci: write the npm token at release time#27
Kikobeats merged 2 commits into
masterfrom
ci/release-token-just-in-time

Conversation

@Kikobeats

@Kikobeats Kikobeats commented Aug 25, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove ${NPM_TOKEN} from the committed .npmrc so pnpm 11 stops warning on every install.
  • At release, write the token to both pnpm and npm user config (pnpm config set + npm config set). lerna publishes via npm; pnpm's store alone is what caused the @microlink/google E404. Neither write touches the repo.

Test plan

  • CI logs have no "Ignored project-level auth setting" warning
  • After merge, publish already-tagged versions with lerna publish from-package --yes

Note

Low Risk
CI and release credential wiring only; no application runtime or auth logic changes.

Overview
Stops committing npm registry auth in .npmrc so pnpm 11 no longer warns about ignored project-level //registry.npmjs.org/:_authToken on every install.

The release job now writes NPM_TOKEN into the user npm config right before pnpm run release (lerna publish), instead of relying on repo .npmrc or pnpm config set, so publishes authenticate without touching tracked files.

Reviewed by Cursor Bugbot for commit a54e2bd. Bugbot is set up for automated code reviews on this repo. Configure here.

Drop ${NPM_TOKEN} from the project .npmrc so pnpm stops warning.
Write it to both pnpm and npm user config so lerna can publish
without dirtying the tree.
@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 13 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: ac77af28-83a2-44d1-8e2b-ae56ac808b49

📥 Commits

Reviewing files that changed from the base of the PR and between a26a13d and a54e2bd.

📒 Files selected for processing (2)
  • .github/workflows/main.yml
  • .npmrc

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

lerna publishes via npm; pnpm config set is unused.
@Kikobeats
Kikobeats merged commit 3acb541 into master Aug 25, 2026
8 checks passed
@coveralls

coveralls commented Aug 25, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 32821994186

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Warning

No base build found for commit a26a13d on master.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 76.449%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 4729
Covered Lines: 3621
Line Coverage: 76.57%
Relevant Branches: 723
Covered Branches: 547
Branch Coverage: 75.66%
Branches in Coverage %: Yes
Coverage Strength: 13.94 hits per line

💛 - Coveralls

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