Details on breaking changes:

1. Adds support for genuine ESM in Node (`type` and `exports` in `package.json` and
    changed `main` path).
2. Changes browser paths to include "-browser" in the file name
3. Adds `browser` to `package.json` to point to browser build for browser-specific bundling.
    Rollup users wishing the browser bundle must ensure `@rollup/plugin-node-resolve`
    has `mainFields` to include `browser` as the first item in the array; Webpack users may
    need to set `resolve` with `aliasFields: ['browser']` and/or
    `mainFields: ['browser', 'module', 'main']`, ensuring that `browser` is in the front of the list
4. Has `module` point to the Node-specific build (with a new path)

- Breaking change: Add `type: 'commonjs'` and `exports: {import, require}`
    (with `node-import-test` npm script to demo)
- Breaking change: Change paths for browser (now is
   `dist/index-browser-umd.js` or `dist/index-browser-es.js`)
   (for Node, `main` and `module` point to new Node-specific dist)
- Breaking enhancement:  Add `browser` for browser bundling;
    allowing static analysis environments, doesn't have however
    conditional code to require `vm`); for ESM browser bundling,
    now must check `browser` in Rollup Node resolver plugin;
    see README
- Linting: quote props; as per latest ash-nazg
- Testing: Fix browser tests
- Travis: add default `dist` field to avoid extra config reporting
- Docs: Update
- npm: Reorder scripts by test execution order
- npm: Stable `mocha-multi-reporters`
- npm: Update devDeps. and package-lock
- npm: Bump to 5.0.0

Author: brettz9

.eslintrc.js

@@ -1,12 +1,12 @@
 'use strict';
 module.exports = {
-    'extends': ['ash-nazg/sauron-node'],
+    extends: ['ash-nazg/sauron-node'],
     parserOptions: {
         ecmaVersion: 2020,
         sourceType: 'module'
     },
-    'settings': {
-        'polyfills': [
+    settings: {
+        polyfills: [
             'Array.isArray',
             'console',
             'Date.now',
@@ -20,7 +20,7 @@ module.exports = {
             'XMLHttpRequest'
         ]
     },
-    'overrides': [
+    overrides: [
         {
             files: ['.eslintrc.js', '.mocharc.js'],
             extends: ['plugin:node/recommended-script'],
@@ -30,12 +30,12 @@ module.exports = {
             }
         },
         {
-            'files': ['src/jsonpath.js', 'test-helpers/node-env.js'],
+            files: ['src/jsonpath-node.js', 'test-helpers/node-env.js'],
             // Apparent bug with `overrides` necessitating this
-            'globals': {
-                'require': 'readonly',
-                'run': 'readonly',
-                'module': 'readonly'
+            globals: {
+                require: 'readonly',
+                run: 'readonly',
+                module: 'readonly'
             },
             rules: {
                 'node/no-unsupported-features/es-syntax': ['error', {
@@ -46,55 +46,63 @@ module.exports = {
             }
         },
         {
-            'files': ['*.md'],
-            'rules': {
+            files: ['*.md'],
+            rules: {
                 'import/unambiguous': 0,
                 'import/no-commonjs': 0,
                 'import/no-unresolved': ['error', {
-                    'ignore': ['jsonpath-plus']
+                    ignore: ['jsonpath-plus']
                 }],
                 'no-undef': 0,
                 'no-unused-vars': ['error', {
-                    'varsIgnorePattern': 'json|result'
+                    varsIgnorePattern: 'json|result'
+                }],
+                'import/no-extraneous-dependencies': 0,
+                'node/no-extraneous-import': ['error', {
+                    allowModules: ['jsonpath-plus']
                 }],
                 'node/no-missing-require': ['error', {
-                    'allowModules': ['jsonpath-plus']
+                    allowModules: ['jsonpath-plus']
                 }],
                 'node/no-missing-import': ['error', {
-                    'allowModules': ['jsonpath-plus']
+                    allowModules: ['jsonpath-plus']
                 }]
             }
         },
         {
-            'files': ['test/**'],
-            'extends': [
+            files: ['test/**'],
+            extends: [
                 'plugin:chai-expect/recommended',
                 'plugin:chai-friendly/recommended'
             ],
-            'globals': {
-                'assert': 'readonly',
-                'expect': 'readonly',
-                'jsonpath': 'readonly'
+            globals: {
+                assert: 'readonly',
+                expect: 'readonly',
+                jsonpath: 'readonly'
             },
-            'env': {'mocha': true},
-            'rules': {
-                'quotes': 0,
+            env: {mocha: true},
+            rules: {
+                quotes: 0,
+                'quote-props': 0,
                 'import/unambiguous': 0,
                 // Todo: Reenable
                 'max-len': 0
             }
         }
     ],
-    'rules': {
-        'indent': ['error', 4, {'outerIIFEBody': 0}],
+    rules: {
+        indent: ['error', 4, {outerIIFEBody: 0}],
         'promise/prefer-await-to-callbacks': 0,
-        'quote-props': 0,
         'require-jsdoc': 0,
 
         // Disable for now
         'eslint-comments/require-description': 0,
+        // Reenable as have time and confirming no longer needing:
+        // https://github.com/babel/babel/issues/8951#issuecomment-508045524 is no
+        'prefer-named-capture-group': 0,
 
         // Reenable when no longer having problems
-        'unicorn/no-unsafe-regex': 0
+        'unicorn/no-unsafe-regex': 0,
+        'unicorn/consistent-destructuring': 0
     }
 };

.mocharc.js

@@ -1,5 +1,5 @@
 'use strict';
 module.exports = {
     delay: true,
-    reporter: 'cypress-multi-reporters'
+    reporter: 'mocha-multi-reporters'
 };

.travis.yml

@@ -1,6 +1,7 @@
 language: node_js
-os:
-  - linux
+os: linux
+dist: xenial
+
 node_js:
   - 10
   - 12

CHANGES.md

@@ -1,20 +1,34 @@
 # CHANGES for jsonpath-plus
 
-## ?
-
+## 5.0.0 (2020-07-11)
+
+- Breaking change: Add `type: 'commonjs'` and `exports: {import, require}`
+    (with `node-import-test` npm script to demo)
+- Breaking change: Change paths for browser (now is
+   `dist/index-browser-umd.js` or `dist/index-browser-es.js`)
+   (for Node, `main` and `module` point to new Node-specific dist)
+- Breaking enhancement:  Add `browser` for browser bundling;
+    allowing static analysis environments, doesn't have however
+    conditional code to require `vm`); for ESM browser bundling,
+    now must check `browser` in Rollup Node resolver plugin;
+    see README
 - Build: Update per latest devDeps.
 - Docs: Add Regex (`.match`) example on value (@jeffreypriebe)
 - Docs: Add Regex (`.match`) example on property
 - Docs: Fix XPath example (@humbertoc-silva)
 - Docs: Link to XPath 2.0 tester
 - Docs: Update badges per latest updates
+- Linting: quote props
 - Linting: As per latest ash-nazg
+- Testing: Fix browser tests
 - Testing: Add test case for setting values in callbacks (issue #126)
 - Testing: Add more at-sign tests
 - Testing: Bump timeout
 - Travis: Check Node 14
+- Travis: add default `dist` field to avoid extra config reporting
 - npm: Update from deprecated `rollup-plugin-babel` to `@rollup/plugin-babel`
     (and make `babelHelpers` explicit)
+- npm: Reorder scripts by test execution order
 - npm: Update devDeps
 
 ## 4.0.0 (2020-04-09)

README.md

@@ -86,7 +86,7 @@ magic is necessary:
       browsers in dist -->
 <script src="node_modules/core-js-bundle/minified.js"></script>
 
-<script src="node_modules/jsonpath-plus/dist/index-umd.js"></script>
+<script src="node_modules/jsonpath-plus/dist/index-browser-umd.js"></script>
 
 <script>
 
@@ -102,15 +102,18 @@ You may also use ES6 Module imports (for modern browsers):
 ```html
 <script type="module">
 
-import {JSONPath} from './node_modules/jsonpath-plus/dist/index-es.js';
+import {JSONPath} from './node_modules/jsonpath-plus/dist/index-browser-es.js';
 const result = JSONPath({path: '...', json: ...});
 
 </script>
 ```
 
 ### ESM (Bundlers)
 
-Or if you are bundling your JavaScript (e.g., with Rollup), just use:
+Or if you are bundling your JavaScript (e.g., with Rollup), just use,
+noting that [`mainFields`](https://github.com/rollup/plugins/tree/master/packages/node-resolve#mainfields)
+should include `browser` for browser builds (for Node, the default, which
+checks `module`, should be fine):
 
 ```js
 import {JSONPath} from 'jsonpath-plus';

demo/index.html

@@ -26,7 +26,7 @@ <h2>JSONPath Demo <i id="demoNode">(To demo on Node instead, see the <a href="ht
         </label>
       </div>
     </form>
-    <script src="../dist/index-umd.js"></script>
+    <script src="../dist/index-browser-umd.js"></script>
     <script src="index.js"></script>
   </body>
 </html>

demo/node-import-test.mjs

@@ -0,0 +1,46 @@
+import {JSONPath} from '../dist/index-node-esm.mjs';
+
+/* eslint-disable quotes, quote-props */
+const json = {
+    "store": {
+        "book": [{
+            "category": "reference",
+            "author": "Nigel Rees",
+            "title": "Sayings of the Century",
+            "price": 8.95
+        },
+        {
+            "category": "fiction",
+            "author": "Evelyn Waugh",
+            "title": "Sword of Honour",
+            "price": 12.99
+        },
+        {
+            "category": "fiction",
+            "author": "Herman Melville",
+            "title": "Moby Dick",
+            "isbn": "0-553-21311-3",
+            "price": 8.99
+        },
+        {
+            "category": "fiction",
+            "author": "J. R. R. Tolkien",
+            "title": "The Lord of the Rings",
+            "isbn": "0-395-19395-8",
+            "price": 22.99
+        }],
+        "bicycle": {
+            "color": "red",
+            "price": 19.95
+        }
+    }
+};
+/* eslint-enable quotes, quote-props */
+
+const result = JSONPath({
+    json,
+    path: '$.store.book[*].author'
+});
+
+// eslint-disable-next-line no-console
+console.log('result', result);