Skip to content

ci: use the same npmrc auth as metascraper - #26

Merged
Kikobeats merged 1 commit into
masterfrom
ci/npmrc-auth-like-metascraper
Aug 25, 2026
Merged

ci: use the same npmrc auth as metascraper#26
Kikobeats merged 1 commit into
masterfrom
ci/npmrc-auth-like-metascraper

Conversation

@Kikobeats

Copy link
Copy Markdown
Member

Summary

  • The release versioned, tagged, and created GitHub releases, then died on the first npm publish: E404 PUT @microlink/google@1.2.0. That package already exists on npm (1.0.6) — this is missing npm auth, not a first-publish access problem.
  • pnpm config set was already in the workflow. lerna uses libnpmpublish/npm, which does not read that. Same recipe as metascraper / browserless / is-antibot: keep ${NPM_TOKEN} in the project .npmrc for npm, and ignore pnpm-lock.yaml.
  • A ci: merge skips the release job on purpose. Git is already ahead of npm (microlink.io@0.5.0, @microlink/mcp@2.1.0, @microlink/google@1.2.0 tagged, not published). After this lands, those need lerna publish from-package (or the next real package change will leave them unpublished).

Test plan

  • Confirm .npmrc is not rewritten with a raw token in CI
  • After merge, publish the already-tagged versions with lerna publish from-package --yes

Made with Cursor

lerna/npm expand ${NPM_TOKEN} from the project .npmrc; pnpm 11
ignores that line. The release already set the token for pnpm, so
publish died with E404 on @microlink/google.

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 26 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: 61a5be17-765b-4a9f-8ee6-d9e13c79cdcc

📥 Commits

Reviewing files that changed from the base of the PR and between 076d4bf and c73cdf7.

📒 Files selected for processing (2)
  • .gitignore
  • .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.

@Kikobeats
Kikobeats merged commit a26a13d into master Aug 25, 2026
8 checks passed
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 32820910782

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 076d4bf 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.287%

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: 4721
Covered Lines: 3607
Line Coverage: 76.4%
Relevant Branches: 719
Covered Branches: 543
Branch Coverage: 75.52%
Branches in Coverage %: Yes
Coverage Strength: 13.75 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