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

Emmet occupies way too much space #28874

Closed
SailorMax opened this issue Jun 16, 2017 · 16 comments
Closed

Emmet occupies way too much space #28874

SailorMax opened this issue Jun 16, 2017 · 16 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug emmet Emmet related issues verified Verification succeeded
Milestone

Comments

@SailorMax
Copy link

Can you extract Emmet extension from VSCode and made it as external extension like you did with Git?

For example I dont' use it, but it take 50% of all preinstalled extensions! This is 37MB and 2200+ files! On my tablet update of VSCode can be much faster without it.

thank you.

@mjbvz mjbvz added the emmet Emmet related issues label Jun 16, 2017
@ramya-rao-a
Copy link
Contributor

@SailorMax git is still an internal extension

Are you on stable VS Code or the Insiders?

Can you elaborate on your statement "it take 50% of all preinstalled extensions" ?

@ramya-rao-a ramya-rao-a added the info-needed Issue requires more information from poster label Jun 17, 2017
@SailorMax
Copy link
Author

SailorMax commented Jun 17, 2017

@ramya-rao-a Ok, with Git was wrong :) Then I just offer move Emmet from installation to VSCode marketplace

VSCode 1.13.1
Size of C:\Program Files (x86)\Microsoft VS Code\resources\app\extensions\ = 75MB, 5812 files
Size of C:\Program Files (x86)\Microsoft VS Code\resources\app\extensions\emmet\ = 37MB, 2221 files

When and if NodeJS will support zipped extensions, you can rejoin it again, but currently VSCode already contain too many files for "simple" editor and regular update process takes too many time already.

In this situation I think users, who require Emmet, can download it separately. All others will win 2000+ files and ~30% time of installation and updates (2000+ files is ~30% of total 7000 total files).

@mrmlnc
Copy link
Contributor

mrmlnc commented Jun 17, 2017

I'm in. It seems to me that something went wrong when building the new release of editor.

I'm install all dependencies from dependencies section of Emmet extension and I got:

 ~/Documents/OpenSource/test-emmet    fs node_modules/
3,3M	node_modules/

2M is a mocha module – see emmetio/expand-abbreviation#3.

Also I open \resources\app\extensions\emmet\node_modules on Windows and see that this directory includes typescript and other devDependencies.

@ramya-rao-a
Copy link
Contributor

Thanks for reporting this @SailorMax
Good catch @mrmlnc

I'll first look into the size issue (make a pass at all the dependencies soon and clear the node_modules up) and then circle back on this thread for the internal vs external extension discussion

@ramya-rao-a ramya-rao-a added bug Issue identified by VS Code Team member as probable bug and removed info-needed Issue requires more information from poster labels Jun 19, 2017
@ramya-rao-a ramya-rao-a modified the milestones: July 2017, June 2017 Jun 19, 2017
@ramya-rao-a
Copy link
Contributor

Digging deeper, I found that even after fixing the upstream issue of emmetio/expand-abbreviation#3, the size problem will not be solved

Because this is a problem for all built-in extensions. i.e. dev dependecies are getting pulled into user's machine. Logged #29054 for the same.

Of the 40 something MB for emmet extension, 25MB was coming just from typescript.

Will see if we can have a general solution for this issue

@ramya-rao-a
Copy link
Contributor

ramya-rao-a commented Jun 20, 2017

With 6e366b8 the size of the emmet folder under extensions is reduced to 3.6 MB

and VS Code size is back to 184 MB

@ramya-rao-a
Copy link
Contributor

Also absorbed the change from emmetio/expand-abbreviation#3 with 9100807

This won't reduce the size of the emmet folder until #29054 is fixed though. But is a good cleanup nevertheless

@ramya-rao-a
Copy link
Contributor

ramya-rao-a commented Jun 20, 2017

@SailorMax @mrmlnc With size of emmet folder now down to a few MBs, I believe your initial concern about having emmet as internal extension is now taken care of.

At the moment, we will be going ahead with emmet as internal extension. We will revisit this decision when we make the new emmet model the default in the next iteration (or the one after that)

Thank you very much for reporting this issue though :)

@SailorMax
Copy link
Author

@ramya-rao-a
I just want to emphasize, that core of my report was "2200+ files". 3.6MB in our century - is nothing. Today count of files is more important. Installations and updates of VSCode already is very slow. Few steps in same direction and VSCode setup time will be equal to Visual Studio setup time ;) This is wrong.

How many files did you reduce by your fixes?

@ramya-rao-a
Copy link
Contributor

ramya-rao-a commented Jun 21, 2017

@SailorMax Fixing #29054 reduced all the devdependencies as well. Now we (emmet) are down to 1.3 MB and about 200 files

@ramya-rao-a
Copy link
Contributor

Note to verifier:

  • Check the size and # of files in the extensions\emmet folder of the installed VS Code Insiders.

@ramya-rao-a ramya-rao-a changed the title Emmet as external extension Emmet occupies way too much space Jun 26, 2017
@SailorMax
Copy link
Author

@ramya-rao-a
VS Code Insider.zip looks much better :) Thanks!
But:

  • Insider has node_modules/nsfw with "lodash/" = 1052 files - Is it normal?

  • many extensions (git, html, json, markdown, merge-conflict, typescript) has similar node_modules: applicationinsights, vscode-extension-telemetry, vscode-jsonrpc (twice for css, html and json), vscode-languageclient, vscode-languageserver-types (twice for css, html and json), vscode-nls (twice for css, html and json), winreg. May be more.
    Can you try to reduce these also? We can save more 1000+ files and ~7MB!

@ramya-rao-a
Copy link
Contributor

Good point @SailorMax

@joaomoreno Thoughts?

@joaomoreno
Copy link
Member

Not addressing those issues any time soon.

cc @Tyriar @bpasero for nsfw

@bpasero
Copy link
Member

bpasero commented Jun 26, 2017

@Tyriar can we pick up NSFW from a commit or can we talk them into releasing a new version with your fix for that?

@Tyriar
Copy link
Member

Tyriar commented Jun 26, 2017

Yes I already pushed a PR to get the size of nsfw down Axosoft/nsfw#23, I didn't consider this a blocker to release though as it's only +5mb uncompressed. Made a comment asking for a patch release.

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug emmet Emmet related issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

8 participants