Skip to content

chore: refresh size-gate baselines for 0.17.0 - #4452

Merged
sxlijin merged 1 commit into
canaryfrom
agent/bump-macos-baml-size-gate
Aug 16, 2026
Merged

sxlijin merged 1 commit into
canaryfrom
agent/bump-macos-baml-size-gate

Conversation

@sxlijin

@sxlijin sxlijin commented Aug 16, 2026 •

Copy link
Copy Markdown
Contributor

Refreshes the checked-in size baselines and 3% ceilings from all four platform reports produced by canary CI run 31919654825 at source SHA e8c6ef36ac644dfebd6301227c9258bd7d646532.

The macOS baml-cli grew to 22,259,408 bytes after #4451, exceeding the old hard ceiling by 29.6 KB and the relative 3% gate by a rounding margin. The repository-owned cargo size-gate bake command regenerated all platform baselines and ceilings from that single provenance-bound run. The matching checked-in ceiling assertion is updated from 21.2 MiB to 21.9 MiB.

Validation: cargo test -p cargo-size-gate; cargo fmt --check -p cargo-size-gate; git diff --check.

Summary by CodeRabbit

  • Chores
    • Updated platform-specific size limits and recorded artifact measurements for CLI, packed program, and WebAssembly builds.
    • Refreshed build metadata and timestamps across supported platforms.
    • Updated macOS validation thresholds to reflect current artifact sizes.

@vercel

vercel Bot commented Aug 16, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
beps Ready Ready Preview Aug 16, 2026 3:37am
promptfiddle2 Ready Ready Preview Aug 16, 2026 3:37am

Request Review

@sxlijin
sxlijin marked this pull request as ready for review August 16, 2026 03:29
@github-actions

Copy link
Copy Markdown

⏭️ Performance benchmarks were skipped

Perf benchmarks (CodSpeed) are opt-in on pull requests — they no longer run on every push. They always run automatically after merge to canary/main.

To run them on this PR, do any of the following, then push a commit (or re-run CI):

  • Add RUN_CODSPEED=1 to the PR description, or
  • Include run-perf or /perf in the PR title or any commit message.

@coderabbitai

coderabbitai Bot commented Aug 16, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 54bc43c1-5ce4-478c-aff0-b8521b71659e

📥 Commits

Reviewing files that changed from the base of the PR and between e8c6ef3 and 8477e38.

📒 Files selected for processing (6)
  • baml_language/.cargo/size-gate.toml
  • baml_language/.ci/size-gate/aarch64-apple-darwin.toml
  • baml_language/.ci/size-gate/wasm32-unknown-unknown.toml
  • baml_language/.ci/size-gate/x86_64-pc-windows-msvc.toml
  • baml_language/.ci/size-gate/x86_64-unknown-linux-gnu.toml
  • baml_language/crates/tools_size_gate/src/config.rs

Included review availability: Your plan includes up to 8 reviews per rolling hour; 7 remain after this review.


📝 Walkthrough

Walkthrough

The PR raises size-gate ceilings for baml-cli and packed-program, refreshes platform artifact-size records and build metadata, and updates the macOS configuration test expectation.

Changes

Size-gate refresh

Layer / File(s) Summary
Thresholds and validation
baml_language/.cargo/size-gate.toml, baml_language/crates/tools_size_gate/src/config.rs
The size-gate ceilings and the macOS max_file_bytes test expectation are increased.
Platform baseline records
baml_language/.ci/size-gate/*
Platform timestamps, Git SHAs, artifact measurements, and obsolete CI-record comments are updated. baml-cli, packed-program, and bridge WASM baselines change where applicable.

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

Merge Risk: ⚪ Minimal · up to 8477e

The PR refreshes size-gate baselines and ceilings without changing product runtime behavior. No actionable merge-blocking risk remains after normal validation, though the library-only Rust test command should be explicitly confirmed before merge.

Possibly related PRs

Suggested reviewers: antoniosarosi, aaronvg

Poem

A rabbit checks each measured byte,
And nudges ceilings left and right.
The build records hop in line,
With fresh timestamps, neat and fine.
The size gate rests, its numbers true.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the refresh of size-gate baselines for version 0.17.0.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch agent/bump-macos-baml-size-gate

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.

@vercel
vercel Bot temporarily deployed to Preview – beps August 16, 2026 03:30 Inactive
@vercel
vercel Bot temporarily deployed to Preview – promptfiddle2 August 16, 2026 03:37 Inactive
@github-actions

Copy link
Copy Markdown

Binary size checks passed

✅ 7 passed

Artifact Platform File Gzip Gated on Baseline Delta Status
✅ baml-cli Linux 🔒 28.5 MB 12.0 MB file 28.5 MB -40.4 KB (-0.1%) OK
✅ packed-program Linux 🔒 21.7 MB 8.5 MB file 21.7 MB -21.2 KB (-0.1%) OK
✅ baml-cli macOS 🔒 22.3 MB 10.5 MB file 22.2 MB +29.6 KB (+0.1%) OK
✅ packed-program macOS 🔒 17.4 MB 7.6 MB file 17.4 MB +345 B (+0.0%) OK
✅ baml-cli Windows 🔒 24.0 MB 10.7 MB file 24.0 MB +922 B (+0.0%) OK
✅ packed-program Windows 🔒 18.5 MB 7.7 MB file 18.6 MB -18.9 KB (-0.1%) OK
✅ bridge_wasm WASM 17.9 MB 🔒 4.8 MB gzip 4.8 MB -32.5 KB (-0.7%) OK

🔒 = the size this artifact is GATED on (ceiling + delta). Binaries gate on file size (installed binary); WASM gates on gzip (download size). The other size is shown for information only.


Generated by cargo size-gate · workflow run

@sxlijin
sxlijin added this pull request to the merge queue Aug 16, 2026
Merged via the queue into canary with commit 4b08255 Aug 16, 2026
74 checks passed
@sxlijin
sxlijin deleted the agent/bump-macos-baml-size-gate branch August 16, 2026 03:50

This branch was previously deployed

2 inactive deployments
Preview – promptfiddle2 — 8477e38b Deployed Aug 16, 2026 by vercel[bot]
Preview – beps — 8477e38b Deployed Aug 16, 2026 by vercel[bot]
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.

1 participant