feat(deps)!: update commitlint monorepo to v19

BREAKING CHANGE: Drop support for @commitlint v9 up to and indlucing v18
BREAKING CHANGE: Require Node.js 20 minimum

Author: jdbruijn

package-lock.json

@@ -20,19 +20,12 @@
   "license": "GPL-3.0-or-later",
   "author": "Jeroen de Bruijn",
   "type": "module",
-  "exports": {
-    ".": {
-      "import": "./dist/plugin.js",
-      "require": "./dist/index.cjs"
-    }
-  },
+  "exports": "./dist/index.js",
   "files": [
     "./dist/**/!(*.test).{js,d.ts,cjs}"
   ],
   "scripts": {
-    "build": "npm run build:esm && npm run build:cjs",
-    "build:cjs": "esbuild src/index.ts --bundle --target=ES2022 --platform=node --outfile=dist/index.cjs",
-    "build:esm": "tsc",
+    "build": "tsc",
     "format": "prettier --ignore-path .gitignore --write \"**/*.{vue,css,less,scss,html,htm,json,md,markdown,yml,yaml}\" --log-level warn",
     "format:check": "prettier --ignore-path .gitignore --check \"**/*.{vue,css,less,scss,html,htm,json,md,markdown,yml,yaml}\" --log-level warn",
     "lint": "npm run format:check && xo",
@@ -193,37 +186,18 @@
   },
   "devDependencies": {
     "@ava/typescript": "4.1.0",
-    "@commitlint/cli": "18.6.1",
-    "@commitlint/config-conventional": "18.6.3",
-    "@commitlint/lint-10.x": "npm:@commitlint/[email protected]",
-    "@commitlint/lint-11.x": "npm:@commitlint/[email protected]",
-    "@commitlint/lint-12.x": "npm:@commitlint/[email protected]",
-    "@commitlint/lint-13.x": "npm:@commitlint/[email protected]",
-    "@commitlint/lint-14.x": "npm:@commitlint/[email protected]",
-    "@commitlint/lint-15.x": "npm:@commitlint/[email protected]",
-    "@commitlint/lint-16.x": "npm:@commitlint/[email protected]",
-    "@commitlint/lint-17.x": "npm:@commitlint/[email protected]",
-    "@commitlint/lint-18.x": "npm:@commitlint/[email protected]",
-    "@commitlint/lint-9.x": "npm:@commitlint/[email protected]",
-    "@commitlint/load-10.x": "npm:@commitlint/[email protected]",
-    "@commitlint/load-11.x": "npm:@commitlint/[email protected]",
-    "@commitlint/load-12.x": "npm:@commitlint/[email protected]",
-    "@commitlint/load-13.x": "npm:@commitlint/[email protected]",
-    "@commitlint/load-14.x": "npm:@commitlint/[email protected]",
-    "@commitlint/load-15.x": "npm:@commitlint/[email protected]",
-    "@commitlint/load-16.x": "npm:@commitlint/[email protected]",
-    "@commitlint/load-17.x": "npm:@commitlint/[email protected]",
-    "@commitlint/load-18.x": "npm:@commitlint/[email protected]",
-    "@commitlint/load-9.x": "npm:@commitlint/[email protected]",
-    "@commitlint/rules": "18.6.1",
-    "@commitlint/types": "18.6.1",
+    "@commitlint/cli": "19.2.0",
+    "@commitlint/config-conventional": "19.1.0",
+    "@commitlint/lint-19.x": "npm:@commitlint/[email protected]",
+    "@commitlint/load-19.x": "npm:@commitlint/[email protected]",
+    "@commitlint/rules": "19.0.3",
+    "@commitlint/types": "19.0.3",
     "@semantic-release/changelog": "6.0.3",
     "@semantic-release/exec": "6.0.3",
     "@semantic-release/git": "10.0.1",
     "@types/sinon": "17.0.3",
     "ava": "6.1.2",
     "c8": "9.1.0",
-    "esbuild": "0.20.1",
     "husky": "9.0.11",
     "lint-staged": "15.2.2",
     "prettier": "3.2.5",
@@ -233,9 +207,9 @@
     "xo": "0.58.0"
   },
   "peerDependencies": {
-    "@commitlint/lint": ">=9.1.2 <19"
+    "@commitlint/lint": ">=19 <20"
   },
   "engines": {
-    "node": ">=18"
+    "node": ">=20"
   }
 }

package.json

@@ -1,7 +1,7 @@
-import {type Commit, type RuleOutcome} from '@commitlint/types';
+import {type RuleOutcome} from '@commitlint/types';
 import {fake} from 'sinon';
 import test from 'ava';
-import {functionRule} from './function-rule.js';
+import {type Commit, functionRule} from './function-rule.js';
 
 test('calls an implementation fuction once', async (t) => {
   const implementation = fake();

src/function-rule.test.ts

@@ -1,10 +1,10 @@
 import {
-  type Commit,
   type Rule,
   type RuleConfigCondition,
   type RuleOutcome,
 } from '@commitlint/types';
 
+type Commit = Parameters<Rule>[0];
 type FunctionRule = (
   parsed: Commit,
   when: RuleConfigCondition,
@@ -26,4 +26,4 @@ const functionRule: Rule<FunctionRule> = async (
   return functionRule(parsed, when);
 };
 
-export {type FunctionRule, functionRule};
+export {type Commit, functionRule};

src/function-rule.ts

@@ -1,48 +1,40 @@
 import {type RuleOutcome} from '@commitlint/types';
 import test from 'ava';
-import type commitlintLoad from '@commitlint/load-17.x';
-import type commitlintLint from '@commitlint/lint-17.x';
+import load19x from '@commitlint/load-19.x';
+import lint19x from '@commitlint/lint-19.x';
 import {rules} from './rules.js';
-import {plugin} from './plugin.js';
+import * as plugin from './index.js';
 
 test('exports rules', (t) => {
   t.is(plugin.rules, rules);
 });
 
-const loadPlugin = test.macro<[string]>({
+/* eslint-disable @typescript-eslint/naming-convention */
+const versions = {
+  '19.x': {load: load19x, lint: lint19x},
+} as const;
+/* eslint-enable @typescript-eslint/naming-convention */
+
+const loadPlugin = test.macro<[keyof typeof versions]>({
   async exec(t, version) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-    const module = await import(`@commitlint/load-${version}`);
-    const load = module.default.default as typeof commitlintLoad.default;
     /**
      * Specify the package file, which doesn't contain a configuration, to
      * prevent `@commitlint/load` from searching the filesystem for an actual
      * configuration.
      */
-    await t.notThrowsAsync(load({plugins: [plugin]}, {file: 'package.json'}));
+    await t.notThrowsAsync(
+      versions[version].load({plugins: [plugin]}, {file: 'package.json'}),
+    );
   },
   title(_, version) {
     return `@commitlint/load@${version} can load the plugin`;
   },
 });
 
-test(loadPlugin, '9.x');
-test(loadPlugin, '10.x');
-test(loadPlugin, '11.x');
-test(loadPlugin, '12.x');
-test(loadPlugin, '13.x');
-test(loadPlugin, '14.x');
-test(loadPlugin, '15.x');
-test(loadPlugin, '16.x');
-test(loadPlugin, '17.x');
-test(loadPlugin, '18.x');
+test(loadPlugin, '19.x');
 
-const lintUsingPluginRules = test.macro<[string]>({
+const lintUsingPluginRules = test.macro<[keyof typeof versions]>({
   async exec(t, version) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-    const module = await import(`@commitlint/lint-${version}`);
-    const lint = module.default.default as typeof commitlintLint.default;
-
     for await (const rule of Object.keys(plugin.rules)) {
       const configs: Array<{async: boolean; ruleOutcome: RuleOutcome}> = [
         {async: false, ruleOutcome: [true]},
@@ -52,7 +44,7 @@ const lintUsingPluginRules = test.macro<[string]>({
       ];
 
       for await (const config of configs) {
-        const report = await lint(
+        const report = await versions[version].lint(
           'chore: basic commit message',
           {
             [rule]: [
@@ -82,13 +74,4 @@ const lintUsingPluginRules = test.macro<[string]>({
   },
 });
 
-test(lintUsingPluginRules, '9.x');
-test(lintUsingPluginRules, '10.x');
-test(lintUsingPluginRules, '11.x');
-test(lintUsingPluginRules, '12.x');
-test(lintUsingPluginRules, '13.x');
-test(lintUsingPluginRules, '14.x');
-test(lintUsingPluginRules, '15.x');
-test(lintUsingPluginRules, '16.x');
-test(lintUsingPluginRules, '17.x');
-test(lintUsingPluginRules, '18.x');
+test(lintUsingPluginRules, '19.x');

src/plugin.test.ts renamed to src/index.test.ts

@@ -1,11 +1 @@
-import {plugin} from './plugin.js';
-
-/**
- * Export single object, according to the CommonJS model. The CommonJS module is
- * explicitly used here as that's the kind of module commitlint requires for
- * shareable configurations. This project is an ES module for ease of
- * development, and thus requires the error for an export assignment to be
- * disabled.
- */
-// @ts-expect-error TS1203
-export = plugin;
+export {rules} from './rules.js';

src/index.ts

@@ -5,7 +5,7 @@ import {rules} from './rules.js';
 
 const prefix = 'function-rules/';
 const names = Object.keys(rules);
-const commitlintRuleNames = Object.keys(commitlintRules.default).sort();
+const commitlintRuleNames = Object.keys(commitlintRules).sort();
 test('exports the same rules as commitlint', (t) => {
   t.deepEqual(
     names.map((name) => name.slice(prefix.length)),