Conversation
- Remove bundlesize dependency - Add @size-limit/preset-small-lib and size-limit dependencies - Convert bundlesize config to size-limit format - Add Bundle Size Check job to CI workflow - Update GitHub Actions to v4 (checkout, setup-node) Bundle size limits: - dist/final-form-focus.umd.min.js: 1kB - dist/final-form-focus.es.js: 1.5kB - dist/final-form-focus.cjs.js: 1.5kB
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📝 WalkthroughWalkthroughGitHub Actions workflow updates: checkout and setup-node pinned to v4, Node.js runtime fixed to 22, and a new "Bundle Size Check" CI job added. package.json migrates bundle-size enforcement from bundlesize to size-limit and updates devDependencies and configuration keys. Changes
Sequence Diagram(s)sequenceDiagram
participant GH as GitHub Actions
participant Runner as CI Runner
participant Repo as Repository
participant Node as Node.js 22
participant NPM as Package Manager
participant Build as Build Step
participant Size as size-limit
rect rgba(100,149,237,0.5)
GH->>Runner: trigger workflow (Bundle Size Check)
end
Runner->>Repo: checkout (`actions/checkout@v4`)
Runner->>Node: setup Node.js 22 (`actions/setup-node@v4`)
Runner->>NPM: install dependencies (npm ci / install)
Runner->>Build: run build script
Runner->>Size: run `size-limit` checks
Size-->>Runner: report results
Runner-->>GH: job success/failure
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In @.github/workflows/ci.yml:
- Line 51: The workflow references the older action tag
"codecov/codecov-action@v2.1.0"; update this to the v4 release (e.g.,
"codecov/codecov-action@v4") to match the other v4 upgrades and pull in newer
features/security fixes, then verify the CI run succeeds and adjust any
invocation inputs if the v4 action changed its input names or behavior.
In `@package.json`:
- Around line 74-76: DevDependencies in package.json are out of alphabetical
order; move the entry "@size-limit/preset-small-lib" so devDependencies are
sorted alphabetically (consider ordering entries like
"@size-limit/preset-small-lib", "size-limit", "typescript" appropriately) to
restore the conventional ordering.
- Update codecov action to v4 - Alphabetize devDependencies
|
Fixed all CodeRabbit feedback:
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
Migrates from deprecated bundlesize to size-limit for bundle size monitoring.
Changes
Bundle Size Limits
dist/final-form-focus.umd.min.js: 1kBdist/final-form-focus.es.js: 1.5kBdist/final-form-focus.cjs.js: 1.5kBPart of the bundlesize → size-limit migration across all Final Form repos.
Summary by CodeRabbit