ci(repo): raise the clerk.browser.js bundlewatch budget to 76KB - #9376
Conversation
main is at 75.05KB, over the 75KB budget, so the check fails on every PR. #9313 passed at 74.96KB against a two-day-old main and merged without a re-run; drift ate the 40-byte margin before it landed.
🦋 Changeset detectedLatest commit: c8c2245 The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/electron
@clerk/electron-passkeys
@clerk/eslint-plugin
@clerk/expo
@clerk/expo-google-signin
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/hono
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
📝 WalkthroughWalkthroughThe PR raises the Bundlewatch maximum size for Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
API Changes Report
Summary
No API Changes DetectedAll packages have stable APIs with no detected changes. Report generated by Break Check Last ran on |
Description
mainis over theclerk.browser.jsbundlewatch budget, so the Bundle size check fails on every PR opened against it:This raises that budget to 76KB.
No single PR is at fault. #9313 last ran CI on Aug 8 and passed at
74.96KB < 75KB, a 40-byte margin. It merged on Aug 10 without a re-run, and main had grown by then. It did raiseclerk.legacy.browser.jsandclerk.native.jsfor its own growth, butclerk.browser.jsstill fit at the time.Confirmed with #9375, which touches only
integration/and reports the same75.05KB.Worth considering separately: requiring branches to be up to date with
mainbefore merge would prevent this class of failure.Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change