Carry permissions over when excluding files, fixes #24 (#23)

When excluding files from compiled zips in the plugin, permissions were
lost. JSZip was not detecting or setting them from the old archive but this
patch now correctly does it inside applyZipExclude.

index.js

@@ -68,7 +68,9 @@ const applyZipExclude = bluebird.coroutine(
 
       oldZip
         .filter((path, file) => !_.some(exclude, pattern => minimatch(path, pattern)))
-        .forEach(entry => newZip.file(entry.name, entry.nodeStream("nodebuffer")));
+        .forEach(entry => newZip.file(entry.name,
+                                      entry.nodeStream("nodebuffer"),
+                                      _.pick(entry, ["unixPermissions", "dosPermissions", "comment", "date"])));
 
       const buffer = yield newZip.generateAsync({
         type:        "nodebuffer",

serverless-cljs-plugin/serverless_lumo/build.cljs

@@ -8,6 +8,8 @@
             [lumo.io :as io]
             [serverless-lumo.index :as index]))
 
+(def ^:const zip-comment "Generated by serverless-cljs-plugin on ")
+
 (defn zip!
   "Create the zip in output-dir. Return a promise containing the path.
 
@@ -22,7 +24,8 @@
     - https://archiverjs.com/docs/Archiver.html#file"
   [output-path zip-opts compiler-opts]
 
-  (let [archiver      (archiver "zip" #js {:zlib {:level 9}})
+  (let [archiver      (archiver "zip" #js {:zlib {:level 9}
+                                           :comment (str zip-comment (.toISOString (js/Date.)))})
         output-stream (.createWriteStream fs output-path)]
     (js/Promise.
      (fn [resolve-fn reject-fn]