mixpanel-browser 2.41.0 → 2.42.0

package/dist/mixpanel.umd.js

@@ -6,7 +6,7 @@
 
     var Config = {
         DEBUG: false,
-        LIB_VERSION: '2.41.0'
+        LIB_VERSION: '2.42.0'
     };
 
     // since es6 imports are static and we run unit tests from the console, window won't be defined when importing this file
@@ -1688,28 +1688,6 @@
         return maxlen ? guid.substring(0, maxlen) : guid;
     };
 
-    /**
-     * Check deterministically whether to include or exclude from a feature rollout/test based on the
-     * given string and the desired percentage to include.
-     * @param {String} str - string to run the check against (for instance a project's token)
-     * @param {String} feature - name of feature (for inclusion in hash, to ensure different results
-     * for different features)
-     * @param {Number} percent_allowed - percentage chance that a given string will be included
-     * @returns {Boolean} whether the given string should be included
-     */
-    var determine_eligibility = _.safewrap(function(str, feature, percent_allowed) {
-        str = str + feature;
-
-        // Bernstein's hash: http://www.cse.yorku.ca/~oz/hash.html#djb2
-        var hash = 5381;
-        for (var i = 0; i < str.length; i++) {
-            hash = ((hash << 5) + hash) + str.charCodeAt(i);
-            hash = hash & hash;
-        }
-        var dart = (hash >>> 0) % 100;
-        return dart < percent_allowed;
-    });
-
     // naive way to extract domain name (example.com) from full hostname (my.sub.example.com)
     var SIMPLE_DOMAIN_MATCH_REGEX = /[a-z0-9][a-z0-9-]*\.[a-z]+$/i;
     // this next one attempts to account for some ccSLDs, e.g. extracting oxford.ac.uk from www.oxford.ac.uk
@@ -2448,9 +2426,9 @@
                     } else if (
                         _.isObject(res) &&
                         res.xhr_req &&
-                        (res.xhr_req['status'] >= 500 || res.xhr_req['status'] <= 0)
+                        (res.xhr_req['status'] >= 500 || res.xhr_req['status'] === 429 || res.error === 'timeout')
                     ) {
-                        // network or API error, retry
+                        // network or API error, or 429 Too Many Requests, retry
                         var retryMS = this.flushInterval * 2;
                         var headers = res.xhr_req['responseHeaders'];
                         if (headers) {
@@ -3020,9 +2998,13 @@
      * Permanently delete a group.
      *
      * ### Usage:
+     *
      *     mixpanel.get_group('company', 'mixpanel').delete();
+     *
+     * @param {Function} [callback] If provided, the callback will be called after the tracking event
      */
     MixpanelGroup.prototype['delete'] = addOptOutCheckMixpanelGroup(function(callback) {
+        // bracket notation above prevents a minification error related to reserved words
         var data = this.delete_action();
         return this._send_request(data, callback);
     });
@@ -5924,7 +5906,7 @@
         'inapp_protocol':                    '//',
         'inapp_link_new_window':             false,
         'ignore_dnt':                        false,
-        'batch_requests':                    false, // for now
+        'batch_requests':                    true,
         'batch_size':                        50,
         'batch_flush_interval_ms':           5000,
         'batch_request_timeout_ms':          90000,
@@ -6043,17 +6025,7 @@
         this['config'] = {};
         this['_triggered_notifs'] = [];
 
-        // rollout: enable batch_requests by default for 60% of projects
-        // (only if they have not specified a value in their init config
-        // and they aren't using a custom API host)
-        var variable_features = {};
-        var api_host = config['api_host'];
-        var is_custom_api = !!api_host && !api_host.match(/\.mixpanel\.com$/);
-        if (!('batch_requests' in config) && !is_custom_api && determine_eligibility(token, 'batch', 60)) {
-            variable_features['batch_requests'] = true;
-        }
-
-        this.set_config(_.extend({}, DEFAULT_CONFIG, variable_features, config, {
+        this.set_config(_.extend({}, DEFAULT_CONFIG, config, {
             'name': name,
             'token': token,
             'callback_fn': ((name === PRIMARY_INSTANCE_NAME) ? name : PRIMARY_INSTANCE_NAME + '.' + name) + '._jsc'
@@ -6079,15 +6051,32 @@
             } else {
                 this.init_batchers();
                 if (sendBeacon && window$1.addEventListener) {
-                    window$1.addEventListener('unload', _.bind(function() {
-                        // Before page closes, attempt to flush any events queued up via navigator.sendBeacon.
-                        // Since sendBeacon doesn't report success/failure, events will not be removed from
-                        // the persistent store; if the site is loaded again, the events will be flushed again
-                        // on startup and deduplicated on the Mixpanel server side.
+                    // Before page closes or hides (user tabs away etc), attempt to flush any events
+                    // queued up via navigator.sendBeacon. Since sendBeacon doesn't report success/failure,
+                    // events will not be removed from the persistent store; if the site is loaded again,
+                    // the events will be flushed again on startup and deduplicated on the Mixpanel server
+                    // side.
+                    // There is no reliable way to capture only page close events, so we lean on the
+                    // visibilitychange and pagehide events as recommended at
+                    // https://developer.mozilla.org/en-US/docs/Web/API/Window/unload_event#usage_notes.
+                    // These events fire when the user clicks away from the current page/tab, so will occur
+                    // more frequently than page unload, but are the only mechanism currently for capturing
+                    // this scenario somewhat reliably.
+                    var flush_on_unload = _.bind(function() {
                         if (!this.request_batchers.events.stopped) {
                             this.request_batchers.events.flush({unloading: true});
                         }
-                    }, this));
+                    }, this);
+                    window$1.addEventListener('pagehide', function(ev) {
+                        if (ev['persisted']) {
+                            flush_on_unload();
+                        }
+                    });
+                    window$1.addEventListener('visibilitychange', function() {
+                        if (document$1['visibilityState'] === 'hidden') {
+                            flush_on_unload();
+                        }
+                    });
                 }
             }
         }

package/doc/build-docs.js

@@ -97,6 +97,22 @@ function doxToMD(items) {
   });
 }
 
+// Captures prototype bracket notation property assignment, e.g.:
+// `MixpanelGroup.prototype['delete'] = addOptOutCheckMixpanelGroup(function(callback) {`
+// Based on https://github.com/tj/dox/blob/9fe92e17dfcd31c9b6512f6e5bf0b52c2b6b84d4/lib/dox.js#L592
+dox.contextPatternMatchers.push(function (str) {
+  if (/^\s*([\w$.]+)\s*\.\s*prototype\s*\['\s*([\w$]+)'\]\s*=\s*([^\n;]+)/.exec(str)) {
+    return {
+        type: 'property'
+      , constructor: RegExp.$1
+      , cons: RegExp.$1
+      , name: RegExp.$2
+      , value: RegExp.$3.trim()
+      , string: RegExp.$1 + '.prototype.' + RegExp.$2
+    };
+  }
+});
+
 const rawCode = fs.readFileSync(SOURCE_FILE).toString().trim();
 const parsed = dox.parseComments(rawCode);
 

package/doc/readme.io/javascript-full-api-reference.md

@@ -990,6 +990,24 @@ mixpanel.people.unset(['gender', 'Company']);
 # mixpanel.group
 
 
+___
+## mixpanel.group.delete
+Permanently delete a group.
+
+
+### Usage:
+
+```javascript
+mixpanel.get_group('company', 'mixpanel').delete();
+```
+
+
+
+| Argument | Type | Description |
+| ------------- | ------------- | ----- |
+| **callback** | <span class="mp-arg-type">Function</span></br></span><span class="mp-arg-optional">optional</span> | If provided, the callback will be called after the tracking event |
+
+
 ___
 ## mixpanel.group.remove
 Remove a property from a group. The value will be ignored if doesn't exist.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mixpanel-browser",
-  "version": "2.41.0",
+  "version": "2.42.0",
   "description": "The official Mixpanel JavaScript browser client library",
   "main": "dist/mixpanel.cjs.js",
   "directories": {
@@ -46,10 +46,10 @@
     "jsdom": "11.12.0",
     "jsdom-global": "3.0.2",
     "localStorage": "1.0.4",
-    "lodash": "4.17.19",
+    "lodash": "4.17.21",
     "mocha": "7.1.1",
     "morgan": "1.9.1",
-    "rdme": "3.0.0",
+    "rdme": "4.0.0",
     "request": "2.88.0",
     "rollup": "0.25.8",
     "rollup-plugin-npm": "1.4.0",

package/src/config.js

@@ -1,6 +1,6 @@
 var Config = {
     DEBUG: false,
-    LIB_VERSION: '2.41.0'
+    LIB_VERSION: '2.42.0'
 };
 
 export default Config;

package/src/mixpanel-core.js

@@ -1,6 +1,6 @@
 /* eslint camelcase: "off" */
 import Config from './config';
-import { _, console, userAgent, window, document, navigator, determine_eligibility, slice } from './utils';
+import { _, console, userAgent, window, document, navigator, slice } from './utils';
 import { FormTracker, LinkTracker } from './dom-trackers';
 import { RequestBatcher } from './request-batcher';
 import { MixpanelGroup } from './mixpanel-group';
@@ -117,7 +117,7 @@ var DEFAULT_CONFIG = {
     'inapp_protocol':                    '//',
     'inapp_link_new_window':             false,
     'ignore_dnt':                        false,
-    'batch_requests':                    false, // for now
+    'batch_requests':                    true,
     'batch_size':                        50,
     'batch_flush_interval_ms':           5000,
     'batch_request_timeout_ms':          90000,
@@ -236,17 +236,7 @@ MixpanelLib.prototype._init = function(token, config, name) {
     this['config'] = {};
     this['_triggered_notifs'] = [];
 
-    // rollout: enable batch_requests by default for 60% of projects
-    // (only if they have not specified a value in their init config
-    // and they aren't using a custom API host)
-    var variable_features = {};
-    var api_host = config['api_host'];
-    var is_custom_api = !!api_host && !api_host.match(/\.mixpanel\.com$/);
-    if (!('batch_requests' in config) && !is_custom_api && determine_eligibility(token, 'batch', 60)) {
-        variable_features['batch_requests'] = true;
-    }
-
-    this.set_config(_.extend({}, DEFAULT_CONFIG, variable_features, config, {
+    this.set_config(_.extend({}, DEFAULT_CONFIG, config, {
         'name': name,
         'token': token,
         'callback_fn': ((name === PRIMARY_INSTANCE_NAME) ? name : PRIMARY_INSTANCE_NAME + '.' + name) + '._jsc'
@@ -272,15 +262,32 @@ MixpanelLib.prototype._init = function(token, config, name) {
         } else {
             this.init_batchers();
             if (sendBeacon && window.addEventListener) {
-                window.addEventListener('unload', _.bind(function() {
-                    // Before page closes, attempt to flush any events queued up via navigator.sendBeacon.
-                    // Since sendBeacon doesn't report success/failure, events will not be removed from
-                    // the persistent store; if the site is loaded again, the events will be flushed again
-                    // on startup and deduplicated on the Mixpanel server side.
+                // Before page closes or hides (user tabs away etc), attempt to flush any events
+                // queued up via navigator.sendBeacon. Since sendBeacon doesn't report success/failure,
+                // events will not be removed from the persistent store; if the site is loaded again,
+                // the events will be flushed again on startup and deduplicated on the Mixpanel server
+                // side.
+                // There is no reliable way to capture only page close events, so we lean on the
+                // visibilitychange and pagehide events as recommended at
+                // https://developer.mozilla.org/en-US/docs/Web/API/Window/unload_event#usage_notes.
+                // These events fire when the user clicks away from the current page/tab, so will occur
+                // more frequently than page unload, but are the only mechanism currently for capturing
+                // this scenario somewhat reliably.
+                var flush_on_unload = _.bind(function() {
                     if (!this.request_batchers.events.stopped) {
                         this.request_batchers.events.flush({unloading: true});
                     }
-                }, this));
+                }, this);
+                window.addEventListener('pagehide', function(ev) {
+                    if (ev['persisted']) {
+                        flush_on_unload();
+                    }
+                });
+                window.addEventListener('visibilitychange', function() {
+                    if (document['visibilityState'] === 'hidden') {
+                        flush_on_unload();
+                    }
+                });
             }
         }
     }

package/src/mixpanel-group.js

@@ -110,9 +110,13 @@ MixpanelGroup.prototype.union = addOptOutCheckMixpanelGroup(function(list_name,
  * Permanently delete a group.
  *
  * ### Usage:
+ *
  *     mixpanel.get_group('company', 'mixpanel').delete();
+ *
+ * @param {Function} [callback] If provided, the callback will be called after the tracking event
  */
 MixpanelGroup.prototype['delete'] = addOptOutCheckMixpanelGroup(function(callback) {
+    // bracket notation above prevents a minification error related to reserved words
     var data = this.delete_action();
     return this._send_request(data, callback);
 });

package/src/request-batcher.js

@@ -148,9 +148,9 @@ RequestBatcher.prototype.flush = function(options) {
                 } else if (
                     _.isObject(res) &&
                     res.xhr_req &&
-                    (res.xhr_req['status'] >= 500 || res.xhr_req['status'] <= 0)
+                    (res.xhr_req['status'] >= 500 || res.xhr_req['status'] === 429 || res.error === 'timeout')
                 ) {
-                    // network or API error, retry
+                    // network or API error, or 429 Too Many Requests, retry
                     var retryMS = this.flushInterval * 2;
                     var headers = res.xhr_req['responseHeaders'];
                     if (headers) {

package/src/utils.js

@@ -1682,28 +1682,6 @@ var cheap_guid = function(maxlen) {
     return maxlen ? guid.substring(0, maxlen) : guid;
 };
 
-/**
- * Check deterministically whether to include or exclude from a feature rollout/test based on the
- * given string and the desired percentage to include.
- * @param {String} str - string to run the check against (for instance a project's token)
- * @param {String} feature - name of feature (for inclusion in hash, to ensure different results
- * for different features)
- * @param {Number} percent_allowed - percentage chance that a given string will be included
- * @returns {Boolean} whether the given string should be included
- */
-var determine_eligibility = _.safewrap(function(str, feature, percent_allowed) {
-    str = str + feature;
-
-    // Bernstein's hash: http://www.cse.yorku.ca/~oz/hash.html#djb2
-    var hash = 5381;
-    for (var i = 0; i < str.length; i++) {
-        hash = ((hash << 5) + hash) + str.charCodeAt(i);
-        hash = hash & hash;
-    }
-    var dart = (hash >>> 0) % 100;
-    return dart < percent_allowed;
-});
-
 // naive way to extract domain name (example.com) from full hostname (my.sub.example.com)
 var SIMPLE_DOMAIN_MATCH_REGEX = /[a-z0-9][a-z0-9-]*\.[a-z]+$/i;
 // this next one attempts to account for some ccSLDs, e.g. extracting oxford.ac.uk from www.oxford.ac.uk
@@ -1762,7 +1740,6 @@ export {
     navigator,
     cheap_guid,
     console_with_prefix,
-    determine_eligibility,
     extract_domain,
     localStorageSupported,
     JSONStringify,

package/.travis.yml

@@ -1,6 +0,0 @@
-node_js:
-- '12'
-- '14'
-language: node_js
-script:
-- npm test