Skip to content

fix(repo): prevent integration tests from installing Core 3 packages#7985

Merged
jacekradko merged 1 commit intorelease/core-2from
jacek/fix-core2-verdaccio-config
Mar 5, 2026
Merged

fix(repo): prevent integration tests from installing Core 3 packages#7985
jacekradko merged 1 commit intorelease/core-2from
jacek/fix-core2-verdaccio-config

Conversation

@jacekradko
Copy link
Member

Summary

  • Remove proxy: npmjs from @clerk/* packages in verdaccio.install.yaml so integration tests only install locally published Core 2 packages instead of resolving higher Core 3 versions from the npm registry

Context

After Core 3 was published to npm, the release/core-2 integration tests started failing because:

  1. CI publishes Core 2 packages (e.g. @clerk/backend@2.33.0) to a local Verdaccio registry
  2. Verdaccio restarts with verdaccio.install.yaml which proxies @clerk/* to npmjs.org
  3. Test apps install @clerk/backend — pnpm resolves Core 3 (3.0.1) from npm since it's a higher version than the locally published Core 2 (2.33.0)
  4. All integration tests fail because they run Core 3 code against Core 2 test fixtures

The fix mirrors what verdaccio.publish.yaml already does — no npm proxy for @clerk/* scoped packages. Non-clerk dependencies continue to proxy to npm normally.

Test plan

  • Integration tests on this PR pass (they should install Core 2 packages from Verdaccio instead of Core 3 from npm)

Remove proxy: npmjs from @clerk/* packages in verdaccio.install.yaml
so integration tests only install locally published Core 2 packages
instead of resolving higher Core 3 versions from npm.
@changeset-bot
Copy link

changeset-bot bot commented Mar 5, 2026

⚠️ No Changeset found

Latest commit: f67686e

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

@vercel
Copy link

vercel bot commented Mar 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clerk-js-sandbox Ready Ready Preview, Comment Mar 5, 2026 1:39am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 5, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: daf97881-1a2c-42e4-ab42-7ad165b5ead4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch jacek/fix-core2-verdaccio-config

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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 5, 2026

Open in StackBlitz

@clerk/agent-toolkit

npm i https://pkg.pr.new/@clerk/agent-toolkit@7985

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@7985

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@7985

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@7985

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@7985

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7985

@clerk/elements

npm i https://pkg.pr.new/@clerk/elements@7985

@clerk/clerk-expo

npm i https://pkg.pr.new/@clerk/clerk-expo@7985

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@7985

@clerk/express

npm i https://pkg.pr.new/@clerk/express@7985

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@7985

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@7985

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@7985

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@7985

@clerk/clerk-react

npm i https://pkg.pr.new/@clerk/clerk-react@7985

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@7985

@clerk/remix

npm i https://pkg.pr.new/@clerk/remix@7985

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@7985

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@7985

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@7985

@clerk/themes

npm i https://pkg.pr.new/@clerk/themes@7985

@clerk/types

npm i https://pkg.pr.new/@clerk/types@7985

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@7985

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@7985

commit: f67686e

@jacekradko jacekradko changed the title fix(ci): prevent integration tests from installing Core 3 packages fix(repo): prevent integration tests from installing Core 3 packages Mar 5, 2026
@jacekradko jacekradko merged commit adb2a14 into release/core-2 Mar 5, 2026
44 of 47 checks passed
@jacekradko jacekradko deleted the jacek/fix-core2-verdaccio-config branch March 5, 2026 01:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants