Fix: bad code generated in minified code (#52)

* bumped version
* fixed bad snippet.min() results
* fixed missing SNIPPET_VERSION in minified code

Author: King Long Tse

lib/index.js

@@ -53,7 +53,7 @@ function defaults(options) {
   options || (options = {});
   options.apiKey || (options.apiKey = 'YOUR_API_KEY');
   options.host || (options.host = 'cdn.segment.com');
-  options.ajsPath || (options.ajsPath = '/analytics.js/v1/\' + key + \'/analytics.min.js');
+  options.ajsPath || (options.ajsPath = '/analytics.js/v1/\" + key + \"/analytics.min.js');
   if (!has.call(options, 'page')) options.page = true;
   if (!has.call(options, 'load')) options.load = true;
   return options;

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "@segment/snippet",
   "author": "Segment.io <[email protected]>",
-  "version": "4.13.0",
+  "version": "4.13.1",
   "repository": "git://github.com/segmentio/snippet.git",
   "description": "Templating methods for rendering the analytics.js snippet.",
   "main": "lib/index.js",

package.json

@@ -15,10 +15,10 @@ var lineRegex = /(<%= settings\.load %>|<%= settings\.page %>|}}\(\);)/g;
 var versionRegex = /<%= settings\.version %>/;
 var snippet = template(source.replace(loadRegex, '$1').replace(pageRegex, '$1').replace(versionRegex, packageJSON.version), { variable: 'settings' });
 var snippetMin = template(minify(source, {
-  mangle: { except: ['analytics'] },
+  mangle: { except: ['analytics', 'key'] },
   compress: { sequences: false, side_effects: false },
   fromString: true
-}).code.replace(loadRegex, '$1').replace(pageRegex, '$1').replace(lineRegex, '\n$1'), { variable: 'settings' });
+}).code.replace(loadRegex, '$1').replace(pageRegex, '$1').replace(lineRegex, '\n$1').replace(versionRegex, packageJSON.version), { variable: 'settings' });
 
 mkdirp(path.join(__dirname, '../dist'), function() {
   fs.writeFileSync(path.join(__dirname, '../dist/max.template.js'), 'module.exports=' + snippet.source);

scripts/build.js

@@ -68,7 +68,7 @@
     var script = document.createElement('script');
     script.type = 'text/javascript';
     script.async = true;
-    script.src = 'https://<%= settings.host %><%= settings.ajsPath %>';
+    script.src = "https://<%= settings.host %><%= settings.ajsPath %>";
 
     // Insert our script next to the first script element.
     var first = document.getElementsByTagName('script')[0];