Skip to content

ci(repo): improve verdaccio proxy verification with retries#7782

Merged
jacekradko merged 2 commits intomainfrom
jrad/fix-verdaccio-proxy-check
Feb 6, 2026
Merged

ci(repo): improve verdaccio proxy verification with retries#7782
jacekradko merged 2 commits intomainfrom
jrad/fix-verdaccio-proxy-check

Conversation

@jacekradko
Copy link
Member

@jacekradko jacekradko commented Feb 5, 2026

Summary

  • Improves Verdaccio proxy verification to catch proxy failures early
  • Tests multiple packages including update-browserslist-db which caused recent CI failures
  • Adds retry logic (3 attempts with 2s delay) for transient network issues
  • Fails fast if proxy verification fails instead of just logging a warning

Context

Recent CI failures showed integration tests timing out because Verdaccio's npm proxy wasn't working properly:

ERR_PNPM_FETCH_404 GET http://localhost:4873/update-browserslist-db: Not Found - 404

The previous implementation only logged a warning if proxy verification failed, allowing tests to proceed and fail later with cryptic timeout errors.

Test plan

  • CI passes on this PR
  • Monitor subsequent PRs for reduced flaky integration test failures

Summary by CodeRabbit

  • Chores
    • Enhanced package verification during the release process with configurable retry logic and automatic retry attempts between verification cycles. Expanded verification to validate multiple packages simultaneously rather than single-package verification, improving the consistency of the release build validation.

@vercel
Copy link

vercel bot commented Feb 5, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
clerk-js-sandbox Skipped Skipped Feb 5, 2026 9:07pm

Request Review

@changeset-bot
Copy link

changeset-bot bot commented Feb 5, 2026

⚠️ No Changeset found

Latest commit: f2f98e0

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

The GitHub Actions workflow file for Verdaccio proxy verification has been updated to test multiple packages instead of a single package. The change introduces a list of three test packages (semver, browserslist, update-browserslist-db) and implements retry logic with configurable maximum retries and delay between attempts. The verification now iterates through each package, attempting to fetch the specified version up to the configured number of times before ultimately failing or succeeding based on all packages passing verification.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title directly and concisely summarizes the main change: improving Verdaccio proxy verification with retry logic, which aligns with the core improvement in the changeset.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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

@jacekradko jacekradko changed the title ci(repo): improve Verdaccio proxy verification with retries ci(repo): improve verdaccio proxy verification with retries Feb 5, 2026
@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 5, 2026

Open in StackBlitz

@clerk/agent-toolkit

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

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/dev-cli

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

@clerk/expo

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

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@7782

@clerk/upgrade

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

@clerk/vue

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

commit: f2f98e0

@jacekradko jacekradko merged commit 332e020 into main Feb 6, 2026
65 of 66 checks passed
@jacekradko jacekradko deleted the jrad/fix-verdaccio-proxy-check branch February 6, 2026 03:03
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