Merge branch 'main' into next

# Conflicts:
#	jest.config.js
#	package.json
#	src/config/jest.config.js
#	yarn.lock

Author: jrolfs

.github/dependabot.yml

@@ -59,7 +59,7 @@ jobs:
 
       - name: Upload Coverage / Release
         run: yarn ci-after-success
-        if: ${{ matrix.node == '14' }}
+        if: ${{ matrix.node == '16' }}
         env:
           NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
           GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/build.yml

@@ -1 +1 @@
-16.13.0
+18.12.1

.node-version

@@ -294,7 +294,7 @@ This project is actively maintained by engineers at
 [license-badge]: https://img.shields.io/npm/l/@hover/javascript.svg
 [license-link]: https://github.com/hoverinc/hover-javascript/blob/master/LICENSE
 [node-link]: https://nodejs.org/en/download/
-[node-badge]: https://img.shields.io/badge/node-v16.13.0-green
+[node-badge]: https://img.shields.io/badge/node-v18.12.1-green
 [prettier-badge]:
   https://img.shields.io/badge/code_style-prettier-ff69b4.svg?logo=prettier
 [prettier-link]: https://prettierjs.org/en/download/

README.md

@@ -11,13 +11,4 @@ module.exports = {
   roots: ['<rootDir>/src'],
   coverageThreshold: null,
   transformIgnorePatterns: [...transformIgnorePatterns, '.prettierrc.js'],
-  // Specifying ts-jest options via `global` in Jest configuration has been
-  // deprecated so we have to do this in order to add the `exclude` option to
-  // the transform in our Jest configuration that already has `transform` ☹
-  transform: Object.fromEntries(
-    Object.entries(transform).map(([glob, [transformer, options]]) => [
-      glob,
-      [transformer, {...options, exclude: '**/*'}],
-    ]),
-  ),
 }

jest.config.js

@@ -3,9 +3,9 @@
   "version": "0.0.0-semantically-released",
   "description": "CLI toolbox for common scripts for JavaScript + TypeScript projects",
   "engines": {
-    "node": ">=12",
+    "node": ">=v12.22.12",
     "npm": ">=6",
-    "yarn": ">=1"
+    "yarn": ">=1.22.19"
   },
   "publishConfig": {
     "registry": "https://registry.npmjs.org"
@@ -26,7 +26,7 @@
     "start": "run-p start:*",
     "start:source": "yarn build:source --watch",
     "start:types": "tsc -b -w --preserveWatchOutput src/",
-    "test": "node src test",
+    "test": "SWC_NODE_PROJECT=src/tsconfig.json node src test",
     "test:update": "node src test --updateSnapshot",
     "validate": "node src validate",
     "prepare": "husky install"
@@ -50,50 +50,52 @@
   "author": "Jamie Rolfs <[email protected]>",
   "license": "MIT",
   "dependencies": {
-    "@commitlint/cli": "^16.1.0",
-    "@commitlint/config-conventional": "^16.2.1",
-    "@commitlint/prompt": "^16.1.0",
-    "@types/jest": "^29.0.0",
-    "@types/lodash.has": "^4.5.6",
+    "@commitlint/cli": "^17.4.2",
+    "@commitlint/config-conventional": "^17.4.2",
+    "@commitlint/prompt": "^17.4.2",
+    "@swc-node/jest": "^1.5.6",
+    "@swc/core": "^1.3.27",
+    "@swc/helpers": "^0.4.14",
+    "@types/jest": "^29.2.5",
+    "@types/lodash.has": "^4.5.7",
     "@types/mkdirp": "^1.0.2",
-    "@types/node": ">=17.x",
+    "@types/node": "^18.11.18",
     "@types/rimraf": "^3.0.2",
     "@types/which": "^2.0.1",
-    "@typescript-eslint/eslint-plugin": "^5.13.0",
-    "@typescript-eslint/parser": "^5.27.0",
-    "arrify": "^2.0.1",
-    "commitizen": "^4.2.4",
-    "concurrently": "^7.0.0",
-    "cosmiconfig": "^7.0.1",
+    "@typescript-eslint/eslint-plugin": "^5.48.2",
+    "@typescript-eslint/parser": "^5.48.2",
+    "commitizen": "^4.2.6",
+    "concurrently": "^7.6.0",
+    "cosmiconfig": "^8.0.0",
     "cross-env": "^7.0.3",
-    "cross-spawn": "^7.0.1",
-    "doctoc": "^2.1.0",
-    "eslint": "^8.16.0",
-    "eslint-config-airbnb": "19.0.0",
+    "cross-spawn": "^7.0.3",
+    "doctoc": "^2.2.1",
+    "eslint": "^8.32.0",
+    "eslint-config-airbnb": "19.0.4",
     "eslint-config-airbnb-typescript": "^17.0.0",
-    "eslint-config-prettier": "^8.5.0",
-    "eslint-plugin-import": "^2.25.4",
-    "eslint-plugin-jest": "^26.1.1",
-    "eslint-plugin-jsx-a11y": "^6.5.1",
-    "eslint-plugin-prettier": "^4.0.0",
-    "eslint-plugin-react": "^7.30.0",
-    "eslint-plugin-react-hooks": "^4.3.0",
-    "glob": "^8.0.3",
+    "eslint-config-prettier": "^8.6.0",
+    "eslint-plugin-import": "^2.27.5",
+    "eslint-plugin-jest": "^27.2.1",
+    "eslint-plugin-jsx-a11y": "^6.7.1",
+    "eslint-plugin-prettier": "^4.2.1",
+    "eslint-plugin-react": "^7.32.1",
+    "eslint-plugin-react-hooks": "^4.6.0",
+    "glob": "^8.1.0",
     "is-ci": "^3.0.1",
-    "jest": "^29.0.2",
+    "jest": "^29.3.1",
+    "jest-environment-jsdom": "^29.3.1",
     "jest-github-actions-reporter": "^1.0.3",
-    "jest-watch-typeahead": "^2.1.1",
-    "lint-staged": "^12.3.4",
+    "jest-watch-typeahead": "^2.2.1",
+    "lint-staged": "^13.1.0",
     "lodash.has": "^4.5.2",
-    "mkdirp": "^1.0.3",
-    "prettier": "^2.5.1",
+    "mkdirp": "^2.1.3",
+    "prettier": "^2.8.3",
     "read-pkg-up": "^7.0.1",
-    "rimraf": "^3.0.2",
-    "ts-jest": "^29.0.0-next.1",
-    "tslib": "^2.4.0",
+    "rimraf": "^4.1.1",
+    "tslib": "^2.4.1",
     "typescript": "^4",
-    "which": "^2.0.2",
-    "yargs-parser": "^21.0.1"
+    "which": "^3.0.0",
+    "yargs-parser": "^21.1.1"
   },
   "eslintConfig": {
     "parserOptions": {
@@ -109,7 +111,8 @@
       "import/no-unassigned-import": "off",
       "no-console": "off",
       "no-nested-ternary": "off",
-      "no-useless-catch": "off"
+      "no-useless-catch": "off",
+      "jest/prefer-snapshot-hint": "off"
     }
   },
   "eslintIgnore": [
@@ -126,15 +129,23 @@
   },
   "homepage": "https://github.com/hoverinc/hover-javascript#readme",
   "devDependencies": {
-    "@babel/cli": "^7.17.10",
-    "@babel/core": "^7.17.2",
-    "@babel/preset-env": "^7.16.11",
+    "@babel/cli": "^7.20.7",
+    "@babel/core": "^7.20.12",
+    "@babel/preset-env": "^7.20.2",
     "@types/cross-spawn": "^6.0.2",
     "depcheck": "^1.4.3",
-    "eslint-config-kentcdodds": "^20.0.1",
-    "husky": "^7.0.4",
+    "eslint-config-kentcdodds": "^20.4.0",
+    "husky": "^8.0.3",
     "jest-in-case": "^1.0.2",
     "npm-run-all": "^4.1.5",
     "slash": "^3.0.0"
+  },
+  "peerDependencies": {
+    "ts-jest": "^29.0.0"
+  },
+  "peerDependenciesMeta": {
+    "ts-jest": {
+      "optional": true
+    }
   }
 }

package.json

@@ -0,0 +1,13 @@
+{
+  "$schema": "https://docs.renovatebot.com/renovate-schema.json",
+  "extends": [
+    "github>jrolfs/renovate:npm.json5",
+    "github>jrolfs/renovate:npm-feat.json5"
+  ],
+  "packageRules": [
+    {
+      "matchPackagePatterns": ["eslint"],
+      "groupName": "eslint"
+    }
+  ]
+}

renovate.json

@@ -0,0 +1 @@
+!node_modules

src/api/__tests__/__fixtures__/ls/dirs/.gitignore

@@ -26,11 +26,29 @@ describe('ls', () => {
     })
 
     test('with prefix', () => {
-      const result = ls.dirs(`${fixtures}/ls/dirs`, 'prefix')
+      const result = ls.dirs(`${fixtures}/ls/dirs`, {prefix: 'prefix'})
 
       const expected = ['prefix/.baz', 'prefix/bar', 'prefix/foo']
 
       expect(result.sort()).toEqual(expected.sort())
     })
+
+    test('with custom exclude (string)', () => {
+      const result = ls.dirs(`${fixtures}/ls/dirs`, {exclude: 'foo'})
+
+      const expected = ['.baz', 'bar', 'node_modules']
+
+      expect(result.sort()).toEqual(expected.sort())
+    })
+
+    test('with custom exclude (RegExp)', () => {
+      const result = ls.dirs(`${fixtures}/ls/dirs`, {
+        exclude: /(^node_modules|^bar)/,
+      })
+
+      const expected = ['.baz', 'foo']
+
+      expect(result.sort()).toEqual(expected.sort())
+    })
   })
 })

src/api/__tests__/__fixtures__/ls/dirs/node_modules/.gitkeep

@@ -44,17 +44,39 @@ const configs = path =>
         )}`,
     )
 
+/**
+ * @typedef DirsOptions
+ * @property {string} [prefix] prefix to prepend to each scope entry (e.g:
+ * `{ prefix: 'prefix' }` becomes `type(prefix/scope)`)
+ * @property {RegExp | string | null} [exclude] expression for excluding
+ * directories, defaults to `^node_modules`
+ */
+
 /**
  * Enumerate one level of directories
  *
  * @param {string} path - directory to enumerate
- * @param {string} [prefix] - prefix for each scope entry (e.g: `prefix/entry`)
+ * @param {DirsOptions} options -
  */
-const dirs = (path, prefix) =>
-  readdirSync(path)
-    .filter(f => statSync(join(path, f)).isDirectory())
+const dirs = (path, options) => {
+  const {exclude, prefix} = {exclude: /^node_modules/, ...options}
+
+  /**
+   * @param {string} f filename
+   */
+  const test = f => {
+    if (!exclude) return true
+
+    return !(typeof exclude === 'string'
+      ? f.includes(exclude)
+      : exclude.test(f))
+  }
+
+  return readdirSync(path)
+    .filter(f => statSync(join(path, f)).isDirectory() && test(f))
     .map(item => (prefix ? `${prefix}/${item}` : item))
     .map(item => item.toLowerCase())
+}
 
 const ls = {configs, dirs}
 

src/api/__tests__/commit.js

@@ -1,7 +1,7 @@
 const fs = require('fs')
 const depcheck = require('depcheck')
-const hoverSpecials = require('./specials')
 const {hasFile, uniq} = require('../../utils')
+const hoverSpecials = require('./specials')
 
 /**
  * @typedef Config

src/api/__tests__/test.js

@@ -119,6 +119,7 @@ Object {
       },
     ],
     "import/prefer-default-export": "off",
+    "jest/prefer-snapshot-hint": "off",
     "no-implied-eval": "error",
     "no-throw-literal": "error",
     "no-void": Array [
@@ -251,6 +252,7 @@ Object {
       },
     ],
     "import/prefer-default-export": "off",
+    "jest/prefer-snapshot-hint": "off",
     "no-implied-eval": "error",
     "no-throw-literal": "error",
     "no-void": Array [

src/api/commit.js

@@ -69,6 +69,8 @@ const buildConfig = ({withReact = false} = {}) => {
       ],
       'no-void': ['error', {allowAsStatement: true}],
       'prettier/prettier': 'error',
+      // TODO: consider enabling this as a warning?
+      'jest/prefer-snapshot-hint': 'off',
       ...parserRules(false, isReact),
     },
     overrides: [

src/api/depcheck/depcheck.js

@@ -3,7 +3,7 @@ const {resolveHoverScripts, resolveBin} = require('../../utils')
 const hoverScripts = resolveHoverScripts()
 const doctoc = resolveBin('doctoc')
 
-const defaultTestCommand = `${hoverScripts} test --findRelatedTests`
+const defaultTestCommand = `${hoverScripts} test --findRelatedTests --passWithNoTests`
 
 const sourceExtensions = ['js', 'jsx', 'ts', 'tsx']