Bundle size extremely large due to inclusion of cities database

[taylorthurlow]

Hi, I'm wondering if there are any plans to address the size of the included JSON database. Uncompressed, the countries, states, and cities databases are more than 16 megabytes, which is still more than 2.5 when fetched with a browser, gzip compressed.

This is sort of a deal-breaker for using this application in production for just countries and states, but would be 100% more usable if there were an option to exclude the cities database when one has no intention of using it. I don't know much about tree shaking and module exclusion, nor do I know how/if that would related to a big JSON document.

Maybe @dr5hn has an opinion or suggestions on this topic?

[taylorthurlow]

Opened #62 to address this

[dr5hn]

@taylorthurlow
This looks like a critical issue, thanks for putting this up !!

It certainly needs an attention @harpreetkhalsagtbit

[TAnas0]

Interested for the same reasons. My bundle size is getting too big, I started thinking of dropping the NPM package.
Once PR #62 is released, it'd be glad to test it.

[harpreetkhalsagtbit]

@dr5hn @taylorthurlow @TAnas0
I will pick this up tomorrow.

[taylorthurlow]

Great to hear, thanks. I just pushed up rewritten and passing tests to #62.

[harpreetkhalsagtbit]

I have refactored complete package - Tree Shaking
Please test beta version - npm i country-state-city@2.1.0-beta.2

@taylorthurlow @dr5hn @TAnas0

[JakeAi]

@harpreetkhalsagtbit Why was this issue closed? It didn't fix tree shaking. Bundles are still 17+mb. I ended up using @taylorthurlow fork which allows tree shaking.

[cmacdonnacha]

Same here, bundle size is 17Mb. I wonder if this PR was actually included in the release?

[cmacdonnacha]

@taylorthurlow Can you verify that tree shaking now works for you?

[harpreetkhalsagtbit]

@cmacdonnacha @taylorthurlow @dr5hn
I tested this before merging. I don't know whats wrong. I didn't get time to check this again.

Would you all please share the package version you are testing and also share some code that fails.

[cmacdonnacha]

@harpreetkhalsagtbit Sure, package version is 3.0.1 and sample code is

import { Country, State } from 'country-state-city';
console.log(Country.getAllCountries())
console.log(State.getStatesOfCountry('US'))

[taylorthurlow]

@taylorthurlow Can you verify that tree shaking now works for you?

Sorry my priorities changed and I haven't switched over from my fork yet.

[mathycoder]

Hi, any updates on this? My bundle is also over 16 MB, and it seems impossible to tree shake city.json.

[atomauro]

Same issue, mobile is not loading properly due to this JOLY bundle