Skip to content

chore(budget): raise JS bundle budget to 2,472 kB for protocol 0.40.0 - #785

Merged
snekxs merged 1 commit into
OpenMouse-Project:mainfrom
snekxs:chore/budget-gwolves-0-40
Oct 12, 2026
Merged

snekxs merged 1 commit into
OpenMouse-Project:mainfrom
snekxs:chore/budget-gwolves-0-40

Conversation

@snekxs

@snekxs snekxs commented Oct 12, 2026 •

Copy link
Copy Markdown
Member

Unblocks the automated 0.40.0 bump: the hourly update-protocol run fails its size gate at 2,470,894 / 2,470,000 bytes (0.9 kB over).

The growth is the G-Wolves XVI-new driver, 16 more cataloged G-Wolves PIDs with their picker filters, and the Bridge empty-enumeration diagnostic (protocol #203 + app #783). Verified locally with @openmouse/protocol@0.40.0: JS 2,470,897 / 2,472,000. Once this merges, re-trigger update-protocol.yml (or wait for the hourly run) to land the bump.

Summary by CodeRabbit

  • Chores
    • Updated the JavaScript bundle size limit to reflect the latest measured bundle size after a protocol update. This change affects build-size tracking only; it does not alter application functionality or the user experience.

The G-Wolves XVI-new driver, 16 more cataloged G-Wolves PIDs with
their picker filters, and the Bridge empty-enumeration diagnostic
push the measured aggregate to 2,470.9 kB, 0.9 kB over the old
2,470 kB budget. Verified locally with @openmouse/protocol@0.40.0.
@github-actions github-actions Bot added type: maintenance PR title category: maintenance area: ci Automation and build configuration labels Oct 12, 2026
@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: OpenMouse-Project/openmouse/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 39302d29-ac4b-4d10-ba65-e521c901e9e5

📥 Commits

Reviewing files that changed from the base of the PR and between b2c2b9d and 1d55d1a.


📒 Files selected for processing (1)
  • build/check-bundle-size.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 5 remain after this review.



📝 Walkthrough

Walkthrough

The .js bundle-size budget increased from 2,470,000 to 2,472,000 bytes. The accompanying comment records a measured aggregate of 2,470.9 kB and attributes the increase to the protocol 0.40.0 update.

Changes

Bundle Size Budget

Layer / File(s) Summary
Update the bundle-size budget
build/check-bundle-size.ts
The .js budget increased from 2,470,000 to 2,472,000 bytes. The comment records the measured aggregate and protocol version.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: jazzstack


Merge Risk: ⚪ Minimal · up to 1d55d

The updated size limit accommodates the reported bundle measurement, and the existing size check remains in place. No actionable merge-blocking risk is established.

Architecture Summary

Architecture risk: 🔵 Low · up to 1d55d

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

  • observed — Modified behavior in build/check-bundle-size.ts: The .js budget increases from 2,470,000 to 2,472,000 bytes. The updated comment records the protocol 0.40.0 changes and measured aggregate of 2,470.9 kB.

Pre-merge checks | Passed 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: increasing the JavaScript bundle budget for protocol 0.40.0.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@snekxs
snekxs merged commit d1f547a into OpenMouse-Project:main Oct 12, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ci Automation and build configuration type: maintenance PR title category: maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant