Skip to content

fix: keep Excalidraw's font pack out of the Android bundle (versionCode 24) - #70

Merged
adibhanna merged 2 commits into
mainfrom
fix/exclude-excalidraw-fonts
Sep 16, 2026
Merged

adibhanna merged 2 commits into
mainfrom
fix/exclude-excalidraw-fonts

Conversation

@adibhanna

Copy link
Copy Markdown
Contributor

Summary

The first packaged 1.1.21 build (#69, versionCode 23) grew the Play download from 30.6 MB to 43.8 MB. All of the growth is Excalidraw's font pack, which the core's Vite helper copies into the bundle unless told otherwise: 234 files, about 13 MB, 12 MB of it the 209 Xiaolai CJK handwriting subsets. 1.1.20 shipped without them and drawings resolved fonts as they always have.

  • Pass excalidraw: false to zenNotesAssets in vite.config.ts. The built asset set is back to 47 web fonts and no excalidraw-assets directory; the boot-path closure is unchanged (1.58 MB).
  • Bump versionCode to 24. Version code 23 was already uploaded to Play's bundle library (not released), so the corrected 1.1.21 bundle needs a new code.

Validation

  • npm run boundaries:check, npm test (149), npm run typecheck, npm run build.
  • Gradle unit tests, lint, debug build, release AAB; fresh-install emulator smoke (onboarding, first note, cold relaunch).
  • Play preview size check happens on the resubmitted draft.

The core's Vite helper copies Excalidraw's complete font pack into the
build unless told otherwise: 234 files and about 13 MB, 12 MB of which are
the 209 Xiaolai CJK handwriting subsets. 1.1.20 shipped without them and
drawings resolved their fonts as they always have, so the first packaged
build grew the Play download from 30.6 MB to 43.8 MB for no user-visible
gain. Pass excalidraw: false to restore the previous asset set (47 web
fonts, no excalidraw-assets directory).
Version code 23 was uploaded to Play's bundle library before the font-pack
regression was caught, so the corrected 1.1.21 bundle needs a new code.
@adibhanna
adibhanna merged commit 6f320b2 into main Sep 16, 2026
2 checks passed
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