supportsNodeVM check that works in GOJA, node and ReactNative.

Author: legander

dist/index-es.js

@@ -135,9 +135,16 @@ function _possibleConstructorReturn(self, call) {
 
 /* eslint-disable no-eval, jsdoc/check-types */
 // Todo: Reenable jsdoc/check-types once PR merged: https://github.com/gajus/eslint-plugin-jsdoc/pull/270
-var globalEval = eval; // eslint-disable-next-line import/no-commonjs
+var globalEval = eval; // Only Node.JS has a process variable that is of [[Class]] process
+
+var supportsNodeVM = function supportsNodeVM() {
+  try {
+    return Object.prototype.toString.call(global.process) === '[object process]';
+  } catch (e) {
+    return false;
+  }
+};
 
-var supportsNodeVM = typeof module !== 'undefined' && Boolean(module.exports) && !(typeof navigator !== 'undefined' && navigator.product === 'ReactNative');
 var allowedResultTypes = ['value', 'path', 'pointer', 'parent', 'parentProperty', 'all'];
 var hasOwnProp = Object.prototype.hasOwnProperty;
 /**
@@ -171,7 +178,7 @@ var moveToAnotherArray = function moveToAnotherArray(source, target, conditionCb
   }
 };
 
-var vm = supportsNodeVM ? require('vm') : {
+var vm = supportsNodeVM() ? require('vm') : {
   /**
    * @param {string} expr Expression to evaluate
    * @param {PlainObject} context Object whose items will be added to evaluation

dist/index-es.min.js

@@ -141,9 +141,16 @@
 
   /* eslint-disable no-eval, jsdoc/check-types */
   // Todo: Reenable jsdoc/check-types once PR merged: https://github.com/gajus/eslint-plugin-jsdoc/pull/270
-  var globalEval = eval; // eslint-disable-next-line import/no-commonjs
+  var globalEval = eval; // Only Node.JS has a process variable that is of [[Class]] process
+
+  var supportsNodeVM = function supportsNodeVM() {
+    try {
+      return Object.prototype.toString.call(global.process) === '[object process]';
+    } catch (e) {
+      return false;
+    }
+  };
 
-  var supportsNodeVM = typeof module !== 'undefined' && Boolean(module.exports) && !(typeof navigator !== 'undefined' && navigator.product === 'ReactNative');
   var allowedResultTypes = ['value', 'path', 'pointer', 'parent', 'parentProperty', 'all'];
   var hasOwnProp = Object.prototype.hasOwnProperty;
   /**
@@ -177,7 +184,7 @@
     }
   };
 
-  var vm = supportsNodeVM ? require('vm') : {
+  var vm = supportsNodeVM() ? require('vm') : {
     /**
      * @param {string} expr Expression to evaluate
      * @param {PlainObject} context Object whose items will be added to evaluation

dist/index-es.min.js.map

@@ -1,7 +1,7 @@
 {
   "author": "Stefan Goessner",
   "name": "jsonpath-plus",
-  "version": "0.20.1",
+  "version": "0.20.2",
   "main": "dist/index-umd.js",
   "module": "dist/index-es.js",
   "description": "A JS implementation of JSONPath with some additional operators",

dist/index-umd.js

@@ -2,9 +2,14 @@
 
 // Todo: Reenable jsdoc/check-types once PR merged: https://github.com/gajus/eslint-plugin-jsdoc/pull/270
 const globalEval = eval;
-// eslint-disable-next-line import/no-commonjs
-const supportsNodeVM = typeof module !== 'undefined' && Boolean(module.exports) &&
-    !(typeof navigator !== 'undefined' && navigator.product === 'ReactNative');
+// Only Node.JS has a process variable that is of [[Class]] process
+const supportsNodeVM = function () {
+    try {
+        return Object.prototype.toString.call(global.process) === '[object process]';
+    } catch (e) {
+        return false;
+    }
+};
 const allowedResultTypes = ['value', 'path', 'pointer', 'parent', 'parentProperty', 'all'];
 const {hasOwnProperty: hasOwnProp} = Object.prototype;
 
@@ -36,7 +41,7 @@ const moveToAnotherArray = function (source, target, conditionCb) {
     }
 };
 
-const vm = supportsNodeVM
+const vm = supportsNodeVM()
     ? require('vm')
     : {
         /**