Should archive files be actually compressed?

[jeromesimeon]

Discussion 🗣

I'm unclear that cicero archive used to create .cta files actually compresses them in the corresponding zip. Should the archive use compression?

Example with cicero archive:

zsh-5.8$ cicero archive 
8:13:38 AM - INFO: Using current directory as template folder
8:13:40 AM - INFO: Creating archive: acceptance-of-delivery@0.15.0.cta
zsh-5.8$ ls -la acceptance-of-delivery@0.15.0.cta 
-rw-r--r--  1 jeromesimeon  staff  36209 Jun 20 08:13 acceptance-of-delivery@0.15.0.cta

Example with zip -r:

zsh-5.8$ zip -r test.zip .
  adding: request.json (deflated 21%)
  adding: state.json (stored 0%)
  adding: test/ (stored 0%)
  adding: test/logic_default.feature (deflated 46%)
  adding: test/logic.feature (deflated 75%)
  adding: .cucumber.js (stored 0%)
  adding: logic/ (stored 0%)
  adding: logic/logic.ergo (deflated 51%)
  adding: README.md (deflated 20%)
  adding: package.json (deflated 58%)
  adding: logo.png (stored 0%)
  adding: model/ (stored 0%)
  adding: model/@models.accordproject.org.accordproject.runtime.cto (deflated 51%)
  adding: model/model.cto (deflated 62%)
  adding: model/@models.accordproject.org.organization.cto (deflated 49%)
  adding: model/@models.accordproject.org.geo.cto (deflated 67%)
  adding: model/@models.accordproject.org.accordproject.contract.cto (deflated 47%)
  adding: model/@models.accordproject.org.address.cto (deflated 47%)
  adding: text/ (stored 0%)
  adding: text/sample.md (deflated 51%)
  adding: text/grammar.tem.md (deflated 54%)
zsh-5.8$ ls -la test.zip
-rw-r--r--  1 jeromesimeon  staff  17131 Jun 20 08:13 test.zip

[jeromesimeon]

There is a compression option on jszip which is STORE (no compression) by default https://stuk.github.io/jszip/documentation/api_jszip/generate_async.html

Option would presumable need to be added to this call: https://github.com/accordproject/cicero/blob/558978c230e1a774e587a26c47f83b9ef284e615/packages/cicero-core/src/templatesaver.js#L118

Default compression would simply need to use DEFLATE https://stuk.github.io/jszip/documentation/api_jszip/generate_async.html#compression-and-compressionoptions-options

[mttrbrts]

I had always assumed that this was what happens already!

Would this be a breaking change?