Merge community contributions and fix vulnerabilities

.eslintrc.js

@@ -3,21 +3,20 @@ module.exports = {
   root: true,
   parser: '@typescript-eslint/parser',
   parserOptions: {
-    sourceType: 'module',
-    ecmaVersion: 2020,
+    'ecmaVersion': 11,
+    'sourceType': 'module',
+    'tsconfigRootDir': __dirname,
+    'project': ['./tsconfig.json'],
   },
   plugins: ['@typescript-eslint', 'prettier'],
   extends: [
     'plugin:@typescript-eslint/recommended',
-    'prettier/@typescript-eslint', // Uses eslint-config-prettier to disable ESLint rules from @typescript-eslint/eslint-plugin that would conflict with prettier
     'plugin:prettier/recommended', // Enables eslint-plugin-prettier and eslint-config-prettier. This will display prettier errors as ESLint errors. Make sure this is always the last configuration in the extends array.
   ],
   env: {
     browser: true,
-    commonjs: true,
-    node: true,
-    mocha: true,
-    es6: true,
+    es2020: true,
+    node: true
   },
   rules: {
     curly: 'error',
@@ -29,6 +28,7 @@ module.exports = {
     '@typescript-eslint/ban-ts-comment': 'off',
     '@typescript-eslint/class-name-casing': 'off',
     '@typescript-eslint/no-empty-function': 'off',
+    'no-unused-vars': ['error', { varsIgnorePattern: '^_', argsIgnorePattern: '^_' }],
     'prettier/prettier': 'error',
   },
 };

.github/workflows/nodejs.yml

@@ -5,25 +5,24 @@ name: Node.js CI
 
 on:
   push:
-    branches: [ master ]
+    branches: [master]
   pull_request:
-    branches: [ master ]
+    branches: [master]
 
 jobs:
   build:
-
     runs-on: ubuntu-latest
 
     strategy:
       matrix:
-        node-version: [10.x, 12.x]
+        node-version: [16.x, 18.x]
 
     steps:
-    - uses: actions/checkout@v2
-    - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v1
-      with:
-        node-version: ${{ matrix.node-version }}
-    - run: npm ci
-    - run: npm run build --if-present
-    - run: npm test
+      - uses: actions/checkout@v3
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v3
+        with:
+          node-version: ${{ matrix.node-version }}
+      - run: npm i
+      - run: npm run build --if-present
+      - run: npm test

.github/workflows/npmpublish.yml

@@ -5,44 +5,38 @@ name: Node.js Package
 
 on:
   release:
+    branches: [master, dev]
     types: [created]
 
 jobs:
   test:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v1
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
         with:
-          node-version: 12
+          node-version: 18
       - run: npm ci
       - run: npm test
 
   publish-npm:
     needs: test
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v1
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
         with:
-          node-version: 12
+          node-version: 18
           registry-url: https://registry.npmjs.org/
       - run: npm ci
       - run: npm run build
-      - run: npm publish --verbose --access public --tag latest
+      - name: Set distribution tag for branch
+        run: |
+          if [[ $GITHUB_REF == 'refs/heads/master' ]]; then
+              echo "DISTRIBUTION_TAG=latest" >> "$GITHUB_ENV"
+          else
+              echo "DISTRIBUTION_TAG=develop" >> "$GITHUB_ENV"
+          fi
+      - run: npm publish --verbose --access public --tag ${{ env.DISTRIBUTION_TAG }} ./dist
         env:
           NODE_AUTH_TOKEN: ${{secrets.npm_token}}
-
-#  publish-gpr:
-#    needs: build
-#    runs-on: ubuntu-latest
-#    steps:
-#      - uses: actions/checkout@v2
-#      - uses: actions/setup-node@v1
-#        with:
-#          node-version: 12
-#          registry-url: https://npm.pkg.github.com/
-#      - run: npm ci
-#      - run: npm publish
-#        env:
-#          NODE_AUTH_TOKEN: ${{secrets.GITHUB_TOKEN}}

.gitignore

@@ -10,3 +10,5 @@ compiled
 .awcache
 .rpt2_cache
 docs
+src/**/*.js
+src/**/*.map

CHANGELOG.md

@@ -0,0 +1,58 @@
+# Changelog
+
+All notable changes to `@metrichor/jmespath` will be documented in this file.
+
+## [1.0.1] - 1.0.1
+
+Clean up scripts and unused dependencies
+
+## Breaking
+
+- Minimum node version upgraded to >=16
+
+### Fixed
+
+- Fix: CI build script
+- Fix: Update CI publish script to use Node 18
+
+### Changed
+
+- Improvement: Refactor AST to improve types [@shortercode](https://github.com/shortercode)
+- Package dependency updates
+- Upgrade to [email protected]
+- Upgrade to rollup@3
+- Upgrade to deprecated coveralls package in favour of a new community version
+- Cleaned up old linting comments
+
+
+## [1.0.0] - 1.0.0
+
+Update package dependencies to address vulnerabilities
+
+## Breaking
+
+- Minimum node version upgraded to >=14
+- NPM package exports have changed.
+  - Using named exports only
+  - Exporting ems and cjs only (dropped UMD export)
+  - No minified files
+
+### Added
+
+- [JEP-16] Arithmetic Expressions [@springcomp](https://github.com/springcomp)
+- [JEP-19] Pipe evaluation [@springcomp](https://github.com/springcomp)
+- `CHANGELOG.md`: Going to start tracking changes now that the JMESPath community is contributing
+
+### Fixed
+
+- Fix: Catch invalid slice expressions at compile time [@mkantor](https://github.com/mkantor)
+- Fix: Fix unsupported '\\' raw-string escape sequence [@springcomp](https://github.com/springcomp)
+
+### Changed
+
+- Improvement: Refactor AST to improve types [@shortercode](https://github.com/shortercode)
+- Package dependency updates
+- Upgrade to [email protected]
+- Upgrade to rollup@3
+- Upgrade to deprecated coveralls package in favour of a new community version
+- Cleaned up old linting comments