Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Audit package size improvements #2017

Open
45930 opened this issue Feb 7, 2025 · 1 comment
Open

Audit package size improvements #2017

45930 opened this issue Feb 7, 2025 · 1 comment

Comments

@45930
Copy link
Contributor

45930 commented Feb 7, 2025

Out package size is currently 103MB, roughly 40MB of dist/web, 46MB of dist/node and 16MB of src/

In the dist folder, we are not minifying the files, and ~80% of the src folder is taken up by o1js_node.bc.map in the compiled bindings folder.

We can certainly shave off some bytes by minifying some build files in all cases. We may also want to split up some of our packages. For instance, the type data could be installed separately as a dev dependency, and the web version could be released as a standalone package of 40MB.

@mitschabaude
Copy link
Contributor

  • dist/web can always be deleted after building except for index.js since it's all packaged in there. it's already minified.
  • please don't minify dist/node, that would hurt debugging a lot. >95% of the file size is already minified jsoo code or wasm, there's not much to gain
  • the compiled bindings folder can be excluded from publishing except for the TS declaration files. after all, the source is not used after publishing, except for source maps

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

No branches or pull requests

2 participants