-
Notifications
You must be signed in to change notification settings - Fork 33
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
size audit of aio-cli node_modules, improvement suggestions #349
Labels
enhancement
New feature or request
Comments
Using cost-of-modules for
|
Using cost-of-modules for
|
Using cost-of-modules for
|
After discussion, here are the alternatives:
|
shazron
changed the title
size audit of aio-cli node_modules
size audit of aio-cli node_modules, improvement suggestions
Mar 30, 2022
JIRA issue created: https://jira.corp.adobe.com/browse/ACNA-1578 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Only listing those with at least double digit sizes in megabytes, and only for a production build via
npm i --production
.Total size of
node_modules
is466M
@parcel/transformer-js
is the biggest, mainly because it includes pre-built native binding.node
files for all the platforms. This can be optimized by not including.node
files that are not used by the current platform. There is nothing we can do however to affect this, since the.node
files are packed into the tarball for the package version, and not compiled post-install per platform. The linux.node
files themselves contribute90M
. Parcel is used in the app plugin foraio app run
.aws-sdk
is used in @adobe/aio-lib-web which is in turn used by the app plugin for deploy/undeploy (for S3 only). This is v2 of the sdk - we can improve this if we move to v3 where each service has its own package, and we only use S3.yeoman-environment
is used by the app plugin, to run the yeoman generators when creating an app. The bulk of it is rxjs@7.rxjs
is used by multiple modules - events plugin (events-lib), inquirer (cli UI). v6 is 17M.The text was updated successfully, but these errors were encountered: