Repository navigation
ci: build on PRs and shrink full minified bundle to ≤1000 B - #401
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
There was a problem hiding this comment.
Approved. Cursor Bugbot was not running on this PR after the first check poll, so that signal was skipped; remaining signals and approval policy do not require human review. No prior approval from this automation needed to be dismissed.
Sent by Cursor Approval Agent: Pull Request Router and Approver
Restore bundlewatch maxSize to 1000B so the gate matches issue #397 (1kb would allow up to 1024B). Pin checkout, pnpm/action-setup, and setup-node to the current v4 commit SHAs. Co-authored-by: Krish Gupta <ikrishg@users.noreply.github.com>


Related Issue
Closes: #396
Closes: #397
Describe the changes you've made
Follow-up to the open-issue plan after the 3.0.0 audit merge:
.github/workflows/ci.ymlonpull_requestandpushtomain—pnpm install --frozen-lockfilethenpnpm run build(Lightning CSS + aliases + bundlewatch).:rootfont stack insrc/components/global.csstosystem-ui, sans-serif+ emoji families.dist/index.min.cssis now 951 B Brotli (was 1008 B). Bundlewatch max for that file is 1000B. Docs reference snippet updated to match.Hygiene (done outside this PR): closed stale duplicates #375–#380 as duplicates of the already-shipped #381–#386.
How has this been tested?
pnpm run buildlocally — all bundlewatch checks PASS (dist/index.min.css: 951B < 1000B).Checklist
Additional Information
Plan: Reseter 3.0 follow-ups
Code of Conduct

Confidence Score: 5/5
The PR appears safe to merge, with both previous findings fixed and no new issues found.Summary
Adds CI builds for pull requests and pushes to
main, shortens the root font stack, and sets the Brotli limit fordist/index.min.cssto exactly 1000 bytes."1000B", and all three actions use full commit SHAs.Reviews (3) · Last reviewed commit: "chore: acknowledge review fixes for merg..." · Reviewed by Greptile