Skip to content

ci(wasm): raise the size ratchet of the core wasm to 1.75 M - #1244

Merged
andiwand merged 1 commit into
mainfrom
fix/wasm-size-ratchet
Oct 24, 2026
Merged

andiwand merged 1 commit into
mainfrom
fix/wasm-size-ratchet

Conversation

@andiwand

@andiwand andiwand commented Oct 24, 2026 •

Copy link
Copy Markdown
Member

🤖 Generated with Claude Code

Stacked on the clang-tidy fix.

Summary

The wasm job fails on main (run 40111474980): the brotli'd odr-core.wasm is 1,580,440 bytes, over the 1.5 M (1,572,864) ratchet.

No single change crossed the ratchet. The library grew slowly from the ~820 K that the comment states. I measured locally with brotli -q 11:

commit brotli'd bytes
5ae95b9 (last green) 1,573,966
ae40a46 (#1236, first red) 1,580,165
2cc5c82 (main) 1,590,514
#1242 (localized names) 1,606,831

The local build is already over at 5ae95b9, so CI's emscripten build is a little smaller. This PR sets the ratchet to 1.75 M and updates the comment.

odr-formula.js is 637,837 bytes with #1242, which stays under its 768 K ratchet.

@andiwand
andiwand force-pushed the fix/wasm-size-ratchet branch from 6150e7e to 529f0d3 Compare October 24, 2026 17:36
Base automatically changed from fix/formula-tidy to main October 24, 2026 17:37
The brotli'd `odr-core.wasm` grew slowly to the 1.5 M ratchet. The
last green commit on main (5ae95b9) measures 1,573,966 bytes locally,
so no single change crossed it. Main now measures 1,590,514 bytes,
and the localized function names (#1242) add about 16 K.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017QtawyvLP2SCnoBtnVfFed
@andiwand
andiwand force-pushed the fix/wasm-size-ratchet branch from 529f0d3 to 0c8e122 Compare October 24, 2026 17:37
@andiwand
andiwand merged commit 2075dc7 into main Oct 24, 2026
22 checks passed
@andiwand
andiwand deleted the fix/wasm-size-ratchet branch October 24, 2026 17:38
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