Add Option to Print Full File Paths

.eslintignore

@@ -1 +1,2 @@
 /dev/new-rule/templates/*
+/coverage

.eslintrc.js → .eslintrc.cjs

@@ -1,25 +1,20 @@
 module.exports = {
+  parser: '@babel/eslint-parser',
   parserOptions: {
-    ecmaVersion: 2019,
-    sourceType: 'script',
+    ecmaVersion: 2021,
+    sourceType: 'module',
+    requireConfigFile: false,
   },
   env: {
     node: true,
   },
-  plugins: [
-    'eslint-comments',
-    'filenames',
-    'import',
-    'import-helpers',
-    'node',
-    'prettier',
-    'unicorn',
-  ],
+  plugins: ['eslint-comments', 'filenames', 'import', 'node', 'prettier', 'unicorn'],
   extends: [
     'eslint:recommended',
     'plugin:eslint-comments/recommended',
     'plugin:import/errors',
     'plugin:import/warnings',
+    'plugin:jest/recommended',
     'plugin:node/recommended',
     'plugin:unicorn/recommended',
     'plugin:prettier/recommended',
@@ -82,7 +77,7 @@ module.exports = {
     'filenames/match-regex': ['error', '^.?[a-z0-9-]+$'], // Kebab-case.
 
     // Optional import rules:
-    'import/extensions': 'error',
+    'import/extensions': ['error', 'always'],
     'import/first': 'error',
     'import/newline-after-import': 'error',
     'import/no-absolute-path': 'error',
@@ -98,16 +93,21 @@ module.exports = {
     'import/no-webpack-loader-syntax': 'error',
     'import/unambiguous': 'error',
 
-    'import-helpers/order-imports': [
+    // Jest rules:
+    'jest/no-conditional-expect': 'off',
+
+    // Node rules:
+    'node/no-unsupported-features/es-syntax': [
+      'error',
+      {
+        ignores: ['dynamicImport', 'modules'], // False positives: https://github.com/mysticatea/eslint-plugin-node/issues/250
+      },
+    ],
+
+    'node/no-extraneous-import': [
       'error',
       {
-        newlinesBetween: 'always',
-        groups: [
-          '/^(assert|async_hooks|buffer|child_process|cluster|console|constants|crypto|dgram|dns|domain|events|fs|http|http2|https|inspector|module|net|os|path|perf_hooks|process|punycode|querystring|readline|repl|stream|string_decoder|timers|tls|trace_events|tty|url|util|v8|vm|zli)/',
-          ['module'],
-          ['parent', 'sibling', 'index'],
-        ],
-        alphabetize: { order: 'asc', ignoreCase: true },
+        allowModules: ['@jest/globals'],
       },
     ],
 
@@ -122,22 +122,33 @@ module.exports = {
     'unicorn/no-useless-undefined': 'off',
     'unicorn/prefer-ternary': 'off',
     'unicorn/prevent-abbreviations': 'off',
+    'unicorn/text-encoding-identifier-case': 'off',
   },
 
   overrides: [
     {
-      files: ['bin/**/*.js'],
+      // CJS
+      files: ['**/*.cjs'],
+      parserOptions: {
+        sourceType: 'script',
+      },
+    },
+    {
+      // CLI
+      files: ['bin/**/*.js', 'lib/helpers/cli.js'],
       rules: {
         'no-console': 'off',
       },
     },
     {
+      // Rules
       files: ['lib/rules/**/*.js'],
       rules: {
         'filenames/match-exported': ['error', 'kebab'],
       },
     },
     {
+      // Tests
       files: ['test/**/*.js'],
       env: {
         jest: true,
@@ -146,5 +157,12 @@ module.exports = {
         'import/no-dynamic-require': 'off',
       },
     },
+    {
+      // Rule tests
+      files: ['test/unit/rules/*.js'],
+      rules: {
+        'jest/no-standalone-expect': 'off', // False positives from using: verifyResults(results) { expect(results).toMatchInlineSnapshot }
+      },
+    },
   ],
 };

.github/dependabot.yml

@@ -0,0 +1,25 @@
+version: 2
+updates:
+- package-ecosystem: npm
+  directory: "/"
+  schedule:
+    interval: weekly
+    time: "04:00"
+    timezone: EST
+  open-pull-requests-limit: 10
+  versioning-strategy: increase
+  ignore:
+  - dependency-name: "@babel/parser"
+    versions:
+    - 7.12.15
+    - 7.12.17
+    - 7.13.13
+    - 7.13.15
+    - 7.13.16
+- package-ecosystem: github-actions
+  directory: "/"
+  schedule:
+    interval: monthly
+    time: "04:00"
+    timezone: EST
+  open-pull-requests-limit: 15

.github/workflows/ci.yml

@@ -10,68 +10,111 @@ on:
   schedule:
     - cron: '0 3 * * *' # daily, at 3am
 
+concurrency:
+  group: ci-${{ github.head_ref || github.ref }}
+  cancel-in-progress: true
+
+permissions:
+  contents: read
+
 jobs:
   test:
-    name: Node 12.x - ubuntu
+    name: Node 16.x - ubuntu
     runs-on: ubuntu-latest
     timeout-minutes: 10
+    env:
+      NODE_OPTIONS: --max-old-space-size=4096
+
 
     steps:
-      - uses: actions/checkout@v2
-      - uses: volta-cli/action@v1
+      - uses: actions/checkout@v3
+
+      - uses: volta-cli/action@v4
 
       - name: install dependencies
-        run: yarn install --frozen-lockfile
+        run: npm ci
 
-      - run: yarn lint
-      - run: yarn test:jest
-      - run: yarn update:readme && git diff --exit-code
+      - run: npm run lint
+
+      - run: npm run test:jest
+
+      - run: npm run update && git diff --exit-code
 
   nodeX:
     name: Node ${{ matrix.node-version }} - ${{ matrix.os }}
     runs-on: ${{ matrix.os }}-latest
-    timeout-minutes: 10
-
-    needs: [test]
+    timeout-minutes: 15
+    env:
+      NODE_OPTIONS: --max-old-space-size=4096
 
     strategy:
       matrix:
         os: [ubuntu, windows]
-        node-version: [10.x, 12.x, 14.x]
+        node-version: [14.x, 16.x, 17.x, 18.x]
 
         # excluded because it is the `test` job above
         exclude:
           - os: ubuntu
-            node-version: 12.x
+            node-version: 16.x
 
     steps:
-      - uses: actions/checkout@v2
-      - uses: volta-cli/action@v1
+      - uses: actions/checkout@v3
+
+      - uses: volta-cli/action@v4
         with:
           node-version: ${{ matrix.node-version }}
 
-      - name: print node node-version
-        run: node --version
+      - name: install dependencies
+        run: npm ci
+
+      # Temporarily exclude `test/unit/get-config-test.js` in Node versions before 16 (TODO: remove eventually).
+      # Tests triggering `resolveProjectConfig` and thus `await import()` for config/plugin loading cause this error:
+      #   ReferenceError: You are trying to `import` a file after the Jest environment has been torn down.
+      # Possibly related: https://github.com/facebook/jest/issues/11438
+      - run: npm run test:jest -- --testPathIgnorePatterns=test/unit/get-config-test.js --testPathIgnorePatterns=test/acceptance/public-api-test.js
+
+  node12:
+    name: Node 12
+    runs-on: ${{ matrix.os }}-latest
+    timeout-minutes: 15
+    env:
+      NODE_OPTIONS: --max-old-space-size=4096
+
+
+    strategy:
+      matrix:
+        os: [ubuntu, windows]
+
+    steps:
+      - uses: actions/checkout@v3
+
+      - uses: volta-cli/action@v4
+        with:
+          node-version: 12.x
 
       - name: install dependencies
-        run: yarn install --frozen-lockfile --ignore-engines
+        # Temporary fix for Node 12 pulling in devDependencies who's engines require > 14
+        run: npm ci --force
 
-      - run: yarn test:jest
+      # Temporarily exclude `test/unit/get-config-test.js` in Node versions before 16 (TODO: remove eventually).
+      # Tests triggering `resolveProjectConfig` and thus `await import()` for config/plugin loading cause this error:
+      #   ReferenceError: You are trying to `import` a file after the Jest environment has been torn down.
+      # Possibly related: https://github.com/facebook/jest/issues/11438
+      - run: npm run test:jest -- --testPathIgnorePatterns=test/unit/get-config-test.js --testPathIgnorePatterns=test/acceptance/public-api-test.js test:jest --testPathIgnorePatterns=test/unit/get-config-test.js --testPathIgnorePatterns=test/acceptance/public-api-test.js
 
   floating-dependencies:
     name: Floating Dependencies
     runs-on: ubuntu-latest
     timeout-minutes: 10
-
-    needs: [test]
+    env:
+      NODE_OPTIONS: --max-old-space-size=4096
 
     steps:
-      - uses: actions/checkout@v2
-      - uses: volta-cli/action@v1
-        with:
-          node-version: 14.x
+      - uses: actions/checkout@v3
+
+      - uses: volta-cli/action@v4
 
       - name: install dependencies
-        run: yarn install --no-lockfile
+        run: npm install --no-package-lock
 
-      - run: yarn test:jest
+      - run: npm run test:jest

.gitignore

@@ -16,3 +16,4 @@
 /libpeerconnection.log
 npm-debug.log
 testem.log
+.vscode

.npmpackagejsonlintrc.json

@@ -0,0 +1,14 @@
+{
+  "rules": {
+    "no-duplicate-properties": "error",
+    "no-repeated-dependencies": "error",
+    "prefer-alphabetical-bundledDependencies": "error",
+    "prefer-alphabetical-dependencies": "error",
+    "prefer-alphabetical-devDependencies": "error",
+    "prefer-alphabetical-optionalDependencies": "error",
+    "prefer-alphabetical-scripts": "error",
+    "prefer-caret-version-dependencies": "error",
+    "prefer-caret-version-devDependencies": "error",
+    "prefer-scripts": ["error", ["lint", "test"]]
+  }
+}

.npmrc

@@ -0,0 +1 @@
+engine-strict=true