Skip to content

ci: build on PRs and shrink full minified bundle to ≤1000 B - #401

Merged
cursor[bot] merged 5 commits into
mainfrom
cursor/ci-and-bundle-size-0c98
Oct 9, 2026
Merged

cursor[bot] merged 5 commits into
mainfrom
cursor/ci-and-bundle-size-0c98

Conversation

@ikrishg

@ikrishg ikrishg commented Oct 9, 2026 •

Copy link
Copy Markdown
Owner

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:

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 build locally — all bundlewatch checks PASS (dist/index.min.css: 951B < 1000B).

Checklist

  • I have performed a self-review of my own code.
  • I have commented on my code, particularly wherever it was hard to understand.
  • I have made corresponding changes to the documentation.
  • I have followed the code style of the project
  • I have tested my code, and it works without errors

Additional Information

Plan: Reseter 3.0 follow-ups

Code of Conduct

Open in Web Open in Cursor 

RetriggerConfidence 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 for dist/index.min.css to exactly 1000 bytes.

  • Source, generated CSS, and the reference snippet use the same shorter font stack.
  • Both previous findings are fixed: the size limit is "1000B", and all three actions use full commit SHAs.
  • Since the previous review, only an explanatory comment was added. No new issues were found.

Reviews (3) · Last reviewed commit: "chore: acknowledge review fixes for merg..." · Reviewed by Greptile

Add GitHub Actions CI that runs pnpm build + bundlewatch on PRs and
main. Trim the :root system font stack so dist/index.min.css is 951 B
Brotli (was 1008 B) and lower the bundlewatch ceiling to 1000 B.

Closes #396
Closes #397

Co-authored-by: Krish Gupta <ikrishg@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. 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: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 5a0fdb40-f5ef-4af5-baa6-0f24428b4dc9

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
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

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

Comment thread .bundlewatch.config.js
@ikrishg
ikrishg marked this pull request as ready for review October 9, 2026 20:41
cursor[bot]
cursor Bot previously approved these changes Oct 9, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

Comment thread .bundlewatch.config.js Outdated
Comment thread .github/workflows/ci.yml Outdated
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>
cursor[bot]
cursor Bot previously approved these changes Oct 9, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Cursor Bugbot was not present after the first check poll, so that signal was skipped; remaining signals and approval policy do not require human review. This automation has no current approval to keep or dismiss.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

@cursor
cursor Bot enabled auto-merge (rebase) October 9, 2026 21:53

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Cursor Bugbot was not present after the first check poll, so that signal was skipped; remaining signals and approval policy do not require human review. This automation has no current approval to keep or dismiss.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

@cursor
cursor Bot merged commit 8858a1f into main Oct 9, 2026
5 checks passed
@cursor
cursor Bot deleted the cursor/ci-and-bundle-size-0c98 branch October 9, 2026 21:55
@ikrishg ikrishg mentioned this pull request Oct 10, 2026
6 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

perf: shrink full minified bundle to ≤1000 B Brotli ci: run pnpm build and bundlewatch on pull requests

2 participants